Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1481167 > unrolled thread
| Started by | Colin King <colin.king@canonical.com> |
|---|---|
| First post | 2016-09-12 13:50 +0200 |
| Last post | 2016-09-12 19:40 +0200 |
| Articles | 4 — 4 participants |
Back to article view | Back to linux.kernel
[PATCH] tracing/syscalls: add in missing white space in error message text Colin King <colin.king@canonical.com> - 2016-09-12 13:50 +0200
Re: [PATCH] tracing/syscalls: add in missing white space in error message text Steven Rostedt <rostedt@goodmis.org> - 2016-09-12 15:50 +0200
Re: [PATCH] tracing/syscalls: add in missing white space in error message text Ingo Molnar <mingo@kernel.org> - 2016-09-13 20:40 +0200
Re: [PATCH] tracing/syscalls: add in missing white space in error message text Joe Perches <joe@perches.com> - 2016-09-12 19:40 +0200
| From | Colin King <colin.king@canonical.com> |
|---|---|
| Date | 2016-09-12 13:50 +0200 |
| Subject | [PATCH] tracing/syscalls: add in missing white space in error message text |
| Message-ID | <sgxUm-2FS-9@gated-at.bofh.it> |
From: Colin Ian King <colin.king@canonical.com>
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 <colin.king@canonical.com>
---
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");
} 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);
--
2.9.3
[toc] | [next] | [standalone]
| From | Steven Rostedt <rostedt@goodmis.org> |
|---|---|
| Date | 2016-09-12 15:50 +0200 |
| Subject | Re: [PATCH] tracing/syscalls: add in missing white space in error message text |
| Message-ID | <sgzMt-3XR-13@gated-at.bofh.it> |
| In reply to | #1481167 |
On Mon, 12 Sep 2016 12:41:27 +0100
Colin King <colin.king@canonical.com> wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> 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 <colin.king@canonical.com>
> ---
> 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);
[toc] | [prev] | [next] | [standalone]
| From | Ingo Molnar <mingo@kernel.org> |
|---|---|
| Date | 2016-09-13 20:40 +0200 |
| Subject | Re: [PATCH] tracing/syscalls: add in missing white space in error message text |
| Message-ID | <sh0MF-5AJ-13@gated-at.bofh.it> |
| In reply to | #1481295 |
* Steven Rostedt <rostedt@goodmis.org> wrote:
> On Mon, 12 Sep 2016 12:41:27 +0100
> Colin King <colin.king@canonical.com> wrote:
>
> > From: Colin Ian King <colin.king@canonical.com>
> >
> > 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 <colin.king@canonical.com>
> > ---
> > 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).
Agreed that user visible strings should be on a single line. With that change:
Acked-by: Ingo Molnar <mingo@kernel.org>
Thanks,
Ingo
[toc] | [prev] | [next] | [standalone]
| From | Joe Perches <joe@perches.com> |
|---|---|
| Date | 2016-09-12 19:40 +0200 |
| Subject | Re: [PATCH] tracing/syscalls: add in missing white space in error message text |
| Message-ID | <sgDn5-6mO-57@gated-at.bofh.it> |
| In reply to | #1481167 |
On Mon, 2016-09-12 at 12:41 +0100, Colin King wrote:
> pr_info message spans two lines and the literal string is missing
> a white space between words. Add the white space.
Perhaps better would be to coalesce the format string fragments.
And if you didn't already know, checkpatch can find potential
defects like this using --types=split_string,missing_space
For instance:
$ ./scripts/checkpatch.pl --show-types --types=split_string,missing_space --terse --no-summary \
-f kernel/trace/trace_syscalls.c
kernel/trace/trace_syscalls.c:614: WARNING:SPLIT_STRING: quoted string split across lines
kernel/trace/trace_syscalls.c:614: WARNING:MISSING_SPACE: break quoted strings at a space character
kernel/trace/trace_syscalls.c:686: WARNING:SPLIT_STRING: quoted string split across lines
kernel/trace/trace_syscalls.c:686: WARNING:MISSING_SPACE: break quoted strings at a space character
> diff --git 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");
> } 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);
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web