On Mon, 12 Sep 2016 12:41:27 +0100 Colin King <[email protected]> wrote:
> From: Colin Ian King <[email protected]> > > pr_info message spans two lines and the literal string is missing > a white space between words. Add the white space. > > Signed-off-by: Colin Ian King <[email protected]> > --- > kernel/trace/trace_syscalls.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/kernel/trace/trace_syscalls.c b/kernel/trace/trace_syscalls.c > index b2b6efc..79d8752 100644 > --- a/kernel/trace/trace_syscalls.c > +++ b/kernel/trace/trace_syscalls.c > @@ -610,7 +610,7 @@ static int perf_sysenter_enable(struct trace_event_call > *call) > if (!sys_perf_refcount_enter) > ret = register_trace_sys_enter(perf_syscall_enter, NULL); > if (ret) { > - pr_info("event trace: Could not activate" > + pr_info("event trace: Could not activate " > "syscall entry trace point"); Actually, the better fix is to make this one line. The 80 character limit is not applicable when it cuts strings in half, especially when it causes bugs like this. Also, this should go through trivial (with an ack from us). -- Steve > } else { > set_bit(num, enabled_perf_enter_syscalls); > @@ -682,7 +682,7 @@ static int perf_sysexit_enable(struct trace_event_call > *call) > if (!sys_perf_refcount_exit) > ret = register_trace_sys_exit(perf_syscall_exit, NULL); > if (ret) { > - pr_info("event trace: Could not activate" > + pr_info("event trace: Could not activate " > "syscall exit trace point"); > } else { > set_bit(num, enabled_perf_exit_syscalls);
