Messages in this thread | | | Date | Wed, 05 Oct 2011 13:08:01 +0900 | From | Masami Hiramatsu <> | Subject | Re: [PATCH 2/4] [TRIVIAL] ftrace/kprobes: Fix an warning typo |
| |
(2011/10/05 0:58), Valdis.Kletnieks@vt.edu wrote: > On Tue, 04 Oct 2011 19:44:49 +0900, Masami Hiramatsu said: >> Add a space between words. > >> if (ret == -ENOENT && trace_probe_is_on_module(tp)) { >> - pr_warning("This probe might be able to register after" >> + pr_warning("This probe might be able to register after " >> "target module is loaded. Continue.\n"); > > Better would be to make it all one line. Keeping a string together trumps the > 80-column rule. >
Agreed :) I'd like to check whether other messages should be fixed too.
Thank you,
-- Masami HIRAMATSU Software Platform Research Dept. Linux Technology Center Hitachi, Ltd., Yokohama Research Laboratory E-mail: masami.hiramatsu.pt@hitachi.com
| |