2023-04-17 17:54:53

by Colin Ian King

[permalink] [raw]
Subject: [PATCH][next] perf script: task-analyzer: Fix spelling mistake "miliseconds" -> "millseconds"

There is a spelling mistake in the help for the --ms option. Fix it.

Signed-off-by: Colin Ian King <[email protected]>
---
tools/perf/scripts/python/task-analyzer.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/perf/scripts/python/task-analyzer.py b/tools/perf/scripts/python/task-analyzer.py
index 52e8dae9b1f0..3f1df9894246 100755
--- a/tools/perf/scripts/python/task-analyzer.py
+++ b/tools/perf/scripts/python/task-analyzer.py
@@ -114,7 +114,7 @@ def _parse_args():
"--ns", action="store_true", help="show timestamps in nanoseconds"
)
parser.add_argument(
- "--ms", action="store_true", help="show timestamps in miliseconds"
+ "--ms", action="store_true", help="show timestamps in milliseconds"
)
parser.add_argument(
"--extended-times",
--
2.30.2


2023-04-17 18:27:47

by Ian Rogers

[permalink] [raw]
Subject: Re: [PATCH][next] perf script: task-analyzer: Fix spelling mistake "miliseconds" -> "millseconds"

On Mon, Apr 17, 2023 at 10:48 AM Colin Ian King <[email protected]> wrote:
>
> There is a spelling mistake in the help for the --ms option. Fix it.
>
> Signed-off-by: Colin Ian King <[email protected]>

Seems like a similar fix is needed to the patch Subject of
"millseconds" :-) Other than that:
Acked-by: Ian Rogers <[email protected]>

Thanks,
Ian

> ---
> tools/perf/scripts/python/task-analyzer.py | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/perf/scripts/python/task-analyzer.py b/tools/perf/scripts/python/task-analyzer.py
> index 52e8dae9b1f0..3f1df9894246 100755
> --- a/tools/perf/scripts/python/task-analyzer.py
> +++ b/tools/perf/scripts/python/task-analyzer.py
> @@ -114,7 +114,7 @@ def _parse_args():
> "--ns", action="store_true", help="show timestamps in nanoseconds"
> )
> parser.add_argument(
> - "--ms", action="store_true", help="show timestamps in miliseconds"
> + "--ms", action="store_true", help="show timestamps in milliseconds"
> )
> parser.add_argument(
> "--extended-times",
> --
> 2.30.2
>

2023-04-17 18:30:40

by Hagen Paul Pfeifer

[permalink] [raw]
Subject: Re: [PATCH][next] perf script: task-analyzer: Fix spelling mistake "miliseconds" -> "millseconds"

* Colin Ian King | 2023-04-17 18:48:26 [+0100]:

>There is a spelling mistake in the help for the --ms option. Fix it.
>
>Signed-off-by: Colin Ian King <[email protected]>

Good catch - thank you Colin!

Acked-by: Hagen Paul Pfeifer <[email protected]>

2023-04-18 01:26:27

by Arnaldo Carvalho de Melo

[permalink] [raw]
Subject: Re: [PATCH][next] perf script: task-analyzer: Fix spelling mistake "miliseconds" -> "millseconds"

Em Mon, Apr 17, 2023 at 08:27:07PM +0200, Hagen Paul Pfeifer escreveu:
> * Colin Ian King | 2023-04-17 18:48:26 [+0100]:
>
> >There is a spelling mistake in the help for the --ms option. Fix it.
> >
> >Signed-off-by: Colin Ian King <[email protected]>
>
> Good catch - thank you Colin!
>
> Acked-by: Hagen Paul Pfeifer <[email protected]>

Thanks, applied, and fixed the subject line typo.

- Arnaldo