kernel-janitors.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] tracing/tools: fix a couple of spelling mistakes
@ 2021-02-16 14:08 Colin King
  2021-02-16 15:10 ` Viktor.Rosendahl
  0 siblings, 1 reply; 2+ messages in thread
From: Colin King @ 2021-02-16 14:08 UTC (permalink / raw)
  To: Steven Rostedt, Viktor Rosendahl; +Cc: kernel-janitors, linux-kernel

From: Colin Ian King <colin.king@canonical.com>

There is a spelling mistake in the -g help option, I believe
it should be "graph".  There is also a spelling mistake in a
warning message. Fix both mistakes.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
---

V2: Also fix "nummer" spelling mistake.

---
 tools/tracing/latency/latency-collector.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/tracing/latency/latency-collector.c b/tools/tracing/latency/latency-collector.c
index 57b20802e71b..bfe084bc460e 100644
--- a/tools/tracing/latency/latency-collector.c
+++ b/tools/tracing/latency/latency-collector.c
@@ -1650,7 +1650,7 @@ static void start_printthread(void)
 		if (ufd <  0 ||
 		    read(ufd, seed, sizeof(*seed)) != sizeof(*seed)) {
 			printf(
-"Warning! Using trivial random nummer seed, since %s not available\n",
+"Warning! Using trivial random number seed, since %s not available\n",
 			DEV_URANDOM);
 			fflush(stdout);
 			*seed = i;
@@ -1711,7 +1711,7 @@ static void show_usage(void)
 "\t\t\tbeginning, end, and backtrace.\n\n"
 
 "-g, --graph\t\tEnable the display-graph option in trace_option. This\n"
-"\t\t\toption causes ftrace to show the functionph of how\n"
+"\t\t\toption causes ftrace to show the graph of how\n"
 "\t\t\tfunctions are calling other functions.\n\n"
 
 "-c, --policy POL\tRun the program with scheduling policy POL. POL can be\n"
-- 
2.30.0


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] tracing/tools: fix a couple of spelling mistakes
  2021-02-16 14:08 [PATCH] tracing/tools: fix a couple of spelling mistakes Colin King
@ 2021-02-16 15:10 ` Viktor.Rosendahl
  0 siblings, 0 replies; 2+ messages in thread
From: Viktor.Rosendahl @ 2021-02-16 15:10 UTC (permalink / raw)
  To: colin.king; +Cc: rostedt, kernel-janitors, linux-kernel

Hi,

Thanks for finding these mistakes.

On Tue, 2021-02-16 at 14:08 +0000, Colin King wrote:

>  			printf(
> -"Warning! Using trivial random nummer seed, since %s not available\n",
> +"Warning! Using trivial random number seed, since %s not available\n",
>  			DEV_URANDOM);
>  			fflush(stdout);
>  			*seed = i;
> @@ -1711,7 +1711,7 @@ static void show_usage(void)
>  "\t\t\tbeginning, end, and backtrace.\n\n"
>  
>  "-g, --graph\t\tEnable the display-graph option in trace_option. This\n"
> -"\t\t\toption causes ftrace to show the functionph of how\n"
> +"\t\t\toption causes ftrace to show the graph of how\n"
>  "\t\t\tfunctions are calling other functions.\n\n"

This is a large improvement but since the corrected word is so much shorter, we
would ideally move the first word of the next line to this line as well:

"\t\t\toption causes ftrace to show the graph of how functions\n"
"\t\t\tare calling other functions.\n\n"

best regards,

Viktor

>  
>  "-c, --policy POL\tRun the program with scheduling policy POL. POL can be\n"

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-02-16 15:19 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-16 14:08 [PATCH] tracing/tools: fix a couple of spelling mistakes Colin King
2021-02-16 15:10 ` Viktor.Rosendahl

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).