2021-02-16 15:32:00

by Colin King

[permalink] [raw]
Subject: [PATCH][next][V3] tracing/tools: fix a couple of spelling mistakes

From: Colin Ian King <[email protected]>

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 <[email protected]>
---

V2: Also fix "nummer" spelling mistake.
V3: re-format text, as suggested by Viktor

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

diff --git a/tools/tracing/latency/latency-collector.c b/tools/tracing/latency/latency-collector.c
index 57b20802e71b..b69de9263ee6 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,8 +1711,8 @@ 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\tfunctions are calling other functions.\n\n"
+"\t\t\toption causes ftrace to show the graph of how functions\n"
+"\t\t\tare calling other functions.\n\n"

"-c, --policy POL\tRun the program with scheduling policy POL. POL can be\n"
"\t\t\tother, batch, idle, rr or fifo. The default is rr. When\n"
--
2.30.0


2021-02-16 16:26:49

by Viktor Rosendahl

[permalink] [raw]
Subject: Re: [EXTERNAL] [PATCH][next][V3] tracing/tools: fix a couple of spelling mistakes

This looks good to me.

best regards,

Viktor

On Tue, 2021-02-16 at 15:30 +0000, Colin King wrote:
> EXTERNAL SENDER - be CAUTIOUS, particularly with links and attachments.
> EXTERNER Absender - Bitte VORSICHT beim Öffnen von Links und Anhängen.
> ------------------------------------------------------------------------------
> ------------
>
> From: Colin Ian King <[email protected]>
>
> 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 <[email protected]>
> ---
>
> V2: Also fix "nummer" spelling mistake.
> V3: re-format text, as suggested by Viktor
>
> ---
> tools/tracing/latency/latency-collector.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/tools/tracing/latency/latency-collector.c
> b/tools/tracing/latency/latency-collector.c
> index 57b20802e71b..b69de9263ee6 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,8 +1711,8 @@ 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\tfunctions are calling other functions.\n\n"
> +"\t\t\toption causes ftrace to show the graph of how functions\n"
> +"\t\t\tare calling other functions.\n\n"
>
> "-c, --policy POL\tRun the program with scheduling policy POL. POL can be\n"
> "\t\t\tother, batch, idle, rr or fifo. The default is rr. When\n"