* [PATCH] perf test: fix spelling mistake "sythesizing" -> "synthesizing"
@ 2018-11-29 10:31 Colin King
0 siblings, 0 replies; 18+ messages in thread
From: Colin King @ 2018-11-29 10:31 UTC (permalink / raw)
To: Peter Zijlstra, Arnaldo Carvalho de Melo, Alexander Shishkin,
Jiri Olsa, Namhyung Kim
Cc: kernel-janitors, linux-kernel
From: Colin Ian King <colin.king@canonical.com>
There are spelling mistakes in TEST_ASSERT_VAL messages, fix them.
Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
tools/perf/tests/mmap-thread-lookup.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/perf/tests/mmap-thread-lookup.c b/tools/perf/tests/mmap-thread-lookup.c
index b1af2499a3c9..e8498d3e0176 100644
--- a/tools/perf/tests/mmap-thread-lookup.c
+++ b/tools/perf/tests/mmap-thread-lookup.c
@@ -224,11 +224,11 @@ static int mmap_events(synth_cb synth)
int test__mmap_thread_lookup(struct test *test __maybe_unused, int subtest __maybe_unused)
{
/* perf_event__synthesize_threads synthesize */
- TEST_ASSERT_VAL("failed with sythesizing all",
+ TEST_ASSERT_VAL("failed with synthesizing all",
!mmap_events(synth_all));
/* perf_event__synthesize_thread_map synthesize */
- TEST_ASSERT_VAL("failed with sythesizing process",
+ TEST_ASSERT_VAL("failed with synthesizing process",
!mmap_events(synth_process));
return 0;
--
2.19.1
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH] perf test: fix spelling mistake "leadking" -> "leaking"
2018-11-29 10:31 [PATCH] perf test: fix spelling mistake "sythesizing" -> "synthesizing" Colin King
@ 2019-04-17 10:55 ` Colin King
-1 siblings, 0 replies; 18+ messages in thread
From: Colin King @ 2019-04-17 10:55 UTC (permalink / raw)
To: Peter Zijlstra, Ingo Molnar, Arnaldo Carvalho de Melo,
Alexander Shishkin, Jiri Olsa, Namhyung Kim
Cc: kernel-janitors, linux-kernel
From: Colin Ian King <colin.king@canonical.com>
There are a couple of spelling mistakes in test assert messages. Fix them.
Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
tools/perf/tests/dso-data.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/perf/tests/dso-data.c b/tools/perf/tests/dso-data.c
index 7f6c52021e41..946ab4b63acd 100644
--- a/tools/perf/tests/dso-data.c
+++ b/tools/perf/tests/dso-data.c
@@ -304,7 +304,7 @@ int test__dso_data_cache(struct test *test __maybe_unused, int subtest __maybe_u
/* Make sure we did not leak any file descriptor. */
nr_end = open_files_cnt();
pr_debug("nr start %ld, nr stop %ld\n", nr, nr_end);
- TEST_ASSERT_VAL("failed leadking files", nr == nr_end);
+ TEST_ASSERT_VAL("failed leaking files", nr == nr_end);
return 0;
}
@@ -380,6 +380,6 @@ int test__dso_data_reopen(struct test *test __maybe_unused, int subtest __maybe_
/* Make sure we did not leak any file descriptor. */
nr_end = open_files_cnt();
pr_debug("nr start %ld, nr stop %ld\n", nr, nr_end);
- TEST_ASSERT_VAL("failed leadking files", nr == nr_end);
+ TEST_ASSERT_VAL("failed leaking files", nr == nr_end);
return 0;
}
--
2.20.1
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH] perf test: fix spelling mistake "leadking" -> "leaking"
@ 2019-04-17 10:55 ` Colin King
0 siblings, 0 replies; 18+ messages in thread
From: Colin King @ 2019-04-17 10:55 UTC (permalink / raw)
To: Peter Zijlstra, Ingo Molnar, Arnaldo Carvalho de Melo,
Alexander Shishkin, Jiri Olsa, Namhyung Kim
Cc: kernel-janitors, linux-kernel
From: Colin Ian King <colin.king@canonical.com>
There are a couple of spelling mistakes in test assert messages. Fix them.
Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
tools/perf/tests/dso-data.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/perf/tests/dso-data.c b/tools/perf/tests/dso-data.c
index 7f6c52021e41..946ab4b63acd 100644
--- a/tools/perf/tests/dso-data.c
+++ b/tools/perf/tests/dso-data.c
@@ -304,7 +304,7 @@ int test__dso_data_cache(struct test *test __maybe_unused, int subtest __maybe_u
/* Make sure we did not leak any file descriptor. */
nr_end = open_files_cnt();
pr_debug("nr start %ld, nr stop %ld\n", nr, nr_end);
- TEST_ASSERT_VAL("failed leadking files", nr = nr_end);
+ TEST_ASSERT_VAL("failed leaking files", nr = nr_end);
return 0;
}
@@ -380,6 +380,6 @@ int test__dso_data_reopen(struct test *test __maybe_unused, int subtest __maybe_
/* Make sure we did not leak any file descriptor. */
nr_end = open_files_cnt();
pr_debug("nr start %ld, nr stop %ld\n", nr, nr_end);
- TEST_ASSERT_VAL("failed leadking files", nr = nr_end);
+ TEST_ASSERT_VAL("failed leaking files", nr = nr_end);
return 0;
}
--
2.20.1
^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH] perf test: fix spelling mistake "leadking" -> "leaking"
2019-04-17 10:55 ` Colin King
@ 2019-04-17 11:19 ` Mukesh Ojha
-1 siblings, 0 replies; 18+ messages in thread
From: Mukesh Ojha @ 2019-04-17 11:07 UTC (permalink / raw)
To: Colin King, Peter Zijlstra, Ingo Molnar,
Arnaldo Carvalho de Melo, Alexander Shishkin, Jiri Olsa,
Namhyung Kim
Cc: kernel-janitors, linux-kernel
On 4/17/2019 4:25 PM, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> There are a couple of spelling mistakes in test assert messages. Fix them.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
Well, how are you shooting these mistakes one after the other?
Reviewed-by: Mukesh Ojha <mojha@codeaurora.org>
Cheers,
-Mukesh
> ---
> tools/perf/tests/dso-data.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tools/perf/tests/dso-data.c b/tools/perf/tests/dso-data.c
> index 7f6c52021e41..946ab4b63acd 100644
> --- a/tools/perf/tests/dso-data.c
> +++ b/tools/perf/tests/dso-data.c
> @@ -304,7 +304,7 @@ int test__dso_data_cache(struct test *test __maybe_unused, int subtest __maybe_u
> /* Make sure we did not leak any file descriptor. */
> nr_end = open_files_cnt();
> pr_debug("nr start %ld, nr stop %ld\n", nr, nr_end);
> - TEST_ASSERT_VAL("failed leadking files", nr == nr_end);
> + TEST_ASSERT_VAL("failed leaking files", nr == nr_end);
> return 0;
> }
>
> @@ -380,6 +380,6 @@ int test__dso_data_reopen(struct test *test __maybe_unused, int subtest __maybe_
> /* Make sure we did not leak any file descriptor. */
> nr_end = open_files_cnt();
> pr_debug("nr start %ld, nr stop %ld\n", nr, nr_end);
> - TEST_ASSERT_VAL("failed leadking files", nr == nr_end);
> + TEST_ASSERT_VAL("failed leaking files", nr == nr_end);
> return 0;
> }
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] perf test: fix spelling mistake "leadking" -> "leaking"
2019-04-17 11:19 ` Mukesh Ojha
@ 2019-04-17 11:09 ` Colin Ian King
-1 siblings, 0 replies; 18+ messages in thread
From: Colin Ian King @ 2019-04-17 11:09 UTC (permalink / raw)
To: Mukesh Ojha, Peter Zijlstra, Ingo Molnar,
Arnaldo Carvalho de Melo, Alexander Shishkin, Jiri Olsa,
Namhyung Kim
Cc: kernel-janitors, linux-kernel
On 17/04/2019 12:07, Mukesh Ojha wrote:
>
> On 4/17/2019 4:25 PM, Colin King wrote:
>> From: Colin Ian King <colin.king@canonical.com>
>>
>> There are a couple of spelling mistakes in test assert messages. Fix
>> them.
>>
>> Signed-off-by: Colin Ian King <colin.king@canonical.com>
>
> Well, how are you shooting these mistakes one after the other?
I've got a spelling checker and I'm working down a list of 65948
potential mistakes. I'm doing this between kernel build + debug cycles
while I'm waiting for kernels to build and tests to complete.
>
> Reviewed-by: Mukesh Ojha <mojha@codeaurora.org>
>
> Cheers,
> -Mukesh
>
>> ---
>> tools/perf/tests/dso-data.c | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/tools/perf/tests/dso-data.c b/tools/perf/tests/dso-data.c
>> index 7f6c52021e41..946ab4b63acd 100644
>> --- a/tools/perf/tests/dso-data.c
>> +++ b/tools/perf/tests/dso-data.c
>> @@ -304,7 +304,7 @@ int test__dso_data_cache(struct test *test
>> __maybe_unused, int subtest __maybe_u
>> /* Make sure we did not leak any file descriptor. */
>> nr_end = open_files_cnt();
>> pr_debug("nr start %ld, nr stop %ld\n", nr, nr_end);
>> - TEST_ASSERT_VAL("failed leadking files", nr == nr_end);
>> + TEST_ASSERT_VAL("failed leaking files", nr == nr_end);
>> return 0;
>> }
>> @@ -380,6 +380,6 @@ int test__dso_data_reopen(struct test *test
>> __maybe_unused, int subtest __maybe_
>> /* Make sure we did not leak any file descriptor. */
>> nr_end = open_files_cnt();
>> pr_debug("nr start %ld, nr stop %ld\n", nr, nr_end);
>> - TEST_ASSERT_VAL("failed leadking files", nr == nr_end);
>> + TEST_ASSERT_VAL("failed leaking files", nr == nr_end);
>> return 0;
>> }
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] perf test: fix spelling mistake "leadking" -> "leaking"
@ 2019-04-17 11:09 ` Colin Ian King
0 siblings, 0 replies; 18+ messages in thread
From: Colin Ian King @ 2019-04-17 11:09 UTC (permalink / raw)
To: Mukesh Ojha, Peter Zijlstra, Ingo Molnar,
Arnaldo Carvalho de Melo, Alexander Shishkin, Jiri Olsa,
Namhyung Kim
Cc: kernel-janitors, linux-kernel
On 17/04/2019 12:07, Mukesh Ojha wrote:
>
> On 4/17/2019 4:25 PM, Colin King wrote:
>> From: Colin Ian King <colin.king@canonical.com>
>>
>> There are a couple of spelling mistakes in test assert messages. Fix
>> them.
>>
>> Signed-off-by: Colin Ian King <colin.king@canonical.com>
>
> Well, how are you shooting these mistakes one after the other?
I've got a spelling checker and I'm working down a list of 65948
potential mistakes. I'm doing this between kernel build + debug cycles
while I'm waiting for kernels to build and tests to complete.
>
> Reviewed-by: Mukesh Ojha <mojha@codeaurora.org>
>
> Cheers,
> -Mukesh
>
>> ---
>> tools/perf/tests/dso-data.c | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/tools/perf/tests/dso-data.c b/tools/perf/tests/dso-data.c
>> index 7f6c52021e41..946ab4b63acd 100644
>> --- a/tools/perf/tests/dso-data.c
>> +++ b/tools/perf/tests/dso-data.c
>> @@ -304,7 +304,7 @@ int test__dso_data_cache(struct test *test
>> __maybe_unused, int subtest __maybe_u
>> /* Make sure we did not leak any file descriptor. */
>> nr_end = open_files_cnt();
>> pr_debug("nr start %ld, nr stop %ld\n", nr, nr_end);
>> - TEST_ASSERT_VAL("failed leadking files", nr = nr_end);
>> + TEST_ASSERT_VAL("failed leaking files", nr = nr_end);
>> return 0;
>> }
>> @@ -380,6 +380,6 @@ int test__dso_data_reopen(struct test *test
>> __maybe_unused, int subtest __maybe_
>> /* Make sure we did not leak any file descriptor. */
>> nr_end = open_files_cnt();
>> pr_debug("nr start %ld, nr stop %ld\n", nr, nr_end);
>> - TEST_ASSERT_VAL("failed leadking files", nr = nr_end);
>> + TEST_ASSERT_VAL("failed leaking files", nr = nr_end);
>> return 0;
>> }
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] perf test: fix spelling mistake "leadking" -> "leaking"
@ 2019-04-17 11:19 ` Mukesh Ojha
0 siblings, 0 replies; 18+ messages in thread
From: Mukesh Ojha @ 2019-04-17 11:19 UTC (permalink / raw)
To: Colin King, Peter Zijlstra, Ingo Molnar,
Arnaldo Carvalho de Melo, Alexander Shishkin, Jiri Olsa,
Namhyung Kim
Cc: kernel-janitors, linux-kernel
On 4/17/2019 4:25 PM, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> There are a couple of spelling mistakes in test assert messages. Fix them.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
Well, how are you shooting these mistakes one after the other?
Reviewed-by: Mukesh Ojha <mojha@codeaurora.org>
Cheers,
-Mukesh
> ---
> tools/perf/tests/dso-data.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tools/perf/tests/dso-data.c b/tools/perf/tests/dso-data.c
> index 7f6c52021e41..946ab4b63acd 100644
> --- a/tools/perf/tests/dso-data.c
> +++ b/tools/perf/tests/dso-data.c
> @@ -304,7 +304,7 @@ int test__dso_data_cache(struct test *test __maybe_unused, int subtest __maybe_u
> /* Make sure we did not leak any file descriptor. */
> nr_end = open_files_cnt();
> pr_debug("nr start %ld, nr stop %ld\n", nr, nr_end);
> - TEST_ASSERT_VAL("failed leadking files", nr = nr_end);
> + TEST_ASSERT_VAL("failed leaking files", nr = nr_end);
> return 0;
> }
>
> @@ -380,6 +380,6 @@ int test__dso_data_reopen(struct test *test __maybe_unused, int subtest __maybe_
> /* Make sure we did not leak any file descriptor. */
> nr_end = open_files_cnt();
> pr_debug("nr start %ld, nr stop %ld\n", nr, nr_end);
> - TEST_ASSERT_VAL("failed leadking files", nr = nr_end);
> + TEST_ASSERT_VAL("failed leaking files", nr = nr_end);
> return 0;
> }
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] perf test: fix spelling mistake "leadking" -> "leaking"
2019-04-17 11:19 ` Mukesh Ojha
@ 2019-04-17 17:37 ` Arnaldo Carvalho de Melo
-1 siblings, 0 replies; 18+ messages in thread
From: Arnaldo Carvalho de Melo @ 2019-04-17 17:37 UTC (permalink / raw)
To: Mukesh Ojha
Cc: Colin King, Peter Zijlstra, Ingo Molnar, Alexander Shishkin,
Jiri Olsa, Namhyung Kim, kernel-janitors, linux-kernel
Em Wed, Apr 17, 2019 at 04:37:24PM +0530, Mukesh Ojha escreveu:
>
> On 4/17/2019 4:25 PM, Colin King wrote:
> > From: Colin Ian King <colin.king@canonical.com>
> >
> > There are a couple of spelling mistakes in test assert messages. Fix them.
> >
> > Signed-off-by: Colin Ian King <colin.king@canonical.com>
>
> Well, how are you shooting these mistakes one after the other?
>
> Reviewed-by: Mukesh Ojha <mojha@codeaurora.org>
Thanks, applied.
- Arnaldo
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] perf test: fix spelling mistake "leadking" -> "leaking"
@ 2019-04-17 17:37 ` Arnaldo Carvalho de Melo
0 siblings, 0 replies; 18+ messages in thread
From: Arnaldo Carvalho de Melo @ 2019-04-17 17:37 UTC (permalink / raw)
To: Mukesh Ojha
Cc: Colin King, Peter Zijlstra, Ingo Molnar, Alexander Shishkin,
Jiri Olsa, Namhyung Kim, kernel-janitors, linux-kernel
Em Wed, Apr 17, 2019 at 04:37:24PM +0530, Mukesh Ojha escreveu:
>
> On 4/17/2019 4:25 PM, Colin King wrote:
> > From: Colin Ian King <colin.king@canonical.com>
> >
> > There are a couple of spelling mistakes in test assert messages. Fix them.
> >
> > Signed-off-by: Colin Ian King <colin.king@canonical.com>
>
> Well, how are you shooting these mistakes one after the other?
>
> Reviewed-by: Mukesh Ojha <mojha@codeaurora.org>
Thanks, applied.
- Arnaldo
^ permalink raw reply [flat|nested] 18+ messages in thread
* [tip:perf/core] perf test: Fix spelling mistake "leadking" -> "leaking"
2019-04-17 10:55 ` Colin King
(?)
(?)
@ 2019-05-18 8:47 ` tip-bot for Colin Ian King
-1 siblings, 0 replies; 18+ messages in thread
From: tip-bot for Colin Ian King @ 2019-05-18 8:47 UTC (permalink / raw)
To: linux-tip-commits
Cc: alexander.shishkin, jolsa, hpa, namhyung, peterz, acme,
colin.king, mojha, linux-kernel, tglx, mingo
Commit-ID: 1455ea2391be5a5bf0a53258af94fa2abbd73cca
Gitweb: https://git.kernel.org/tip/1455ea2391be5a5bf0a53258af94fa2abbd73cca
Author: Colin Ian King <colin.king@canonical.com>
AuthorDate: Wed, 17 Apr 2019 11:55:39 +0100
Committer: Arnaldo Carvalho de Melo <acme@redhat.com>
CommitDate: Wed, 15 May 2019 16:36:46 -0300
perf test: Fix spelling mistake "leadking" -> "leaking"
There are a couple of spelling mistakes in test assert messages. Fix them.
Signed-off-by: Colin King <colin.king@canonical.com>
Reviewed-by: Mukesh Ojha <mojha@codeaurora.org>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: kernel-janitors@vger.kernel.org
Link: http://lkml.kernel.org/r/20190417105539.5902-1-colin.king@canonical.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
tools/perf/tests/dso-data.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/perf/tests/dso-data.c b/tools/perf/tests/dso-data.c
index 7f6c52021e41..946ab4b63acd 100644
--- a/tools/perf/tests/dso-data.c
+++ b/tools/perf/tests/dso-data.c
@@ -304,7 +304,7 @@ int test__dso_data_cache(struct test *test __maybe_unused, int subtest __maybe_u
/* Make sure we did not leak any file descriptor. */
nr_end = open_files_cnt();
pr_debug("nr start %ld, nr stop %ld\n", nr, nr_end);
- TEST_ASSERT_VAL("failed leadking files", nr == nr_end);
+ TEST_ASSERT_VAL("failed leaking files", nr == nr_end);
return 0;
}
@@ -380,6 +380,6 @@ int test__dso_data_reopen(struct test *test __maybe_unused, int subtest __maybe_
/* Make sure we did not leak any file descriptor. */
nr_end = open_files_cnt();
pr_debug("nr start %ld, nr stop %ld\n", nr, nr_end);
- TEST_ASSERT_VAL("failed leadking files", nr == nr_end);
+ TEST_ASSERT_VAL("failed leaking files", nr == nr_end);
return 0;
}
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH] perf test: fix spelling mistake "allos" -> "allocate"
2018-11-29 10:31 [PATCH] perf test: fix spelling mistake "sythesizing" -> "synthesizing" Colin King
@ 2019-09-11 15:21 ` Colin King
-1 siblings, 0 replies; 18+ messages in thread
From: Colin King @ 2019-09-11 15:21 UTC (permalink / raw)
To: Peter Zijlstra, Ingo Molnar, Arnaldo Carvalho de Melo,
Alexander Shishkin, Jiri Olsa, Namhyung Kim
Cc: kernel-janitors, linux-kernel
From: Colin Ian King <colin.king@canonical.com>
There is a spelling mistake in a TEST_ASSERT_VAL message. Fix it.
Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
tools/perf/tests/event_update.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/perf/tests/event_update.c b/tools/perf/tests/event_update.c
index cac4290e233a..7f0868a31a7f 100644
--- a/tools/perf/tests/event_update.c
+++ b/tools/perf/tests/event_update.c
@@ -92,7 +92,7 @@ int test__event_update(struct test *test __maybe_unused, int subtest __maybe_unu
evsel = perf_evlist__first(evlist);
- TEST_ASSERT_VAL("failed to allos ids",
+ TEST_ASSERT_VAL("failed to allocate ids",
!perf_evsel__alloc_id(evsel, 1, 1));
perf_evlist__id_add(evlist, evsel, 0, 0, 123);
--
2.20.1
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH] perf test: fix spelling mistake "allos" -> "allocate"
@ 2019-09-11 15:21 ` Colin King
0 siblings, 0 replies; 18+ messages in thread
From: Colin King @ 2019-09-11 15:21 UTC (permalink / raw)
To: Peter Zijlstra, Ingo Molnar, Arnaldo Carvalho de Melo,
Alexander Shishkin, Jiri Olsa, Namhyung Kim
Cc: kernel-janitors, linux-kernel
From: Colin Ian King <colin.king@canonical.com>
There is a spelling mistake in a TEST_ASSERT_VAL message. Fix it.
Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
tools/perf/tests/event_update.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/perf/tests/event_update.c b/tools/perf/tests/event_update.c
index cac4290e233a..7f0868a31a7f 100644
--- a/tools/perf/tests/event_update.c
+++ b/tools/perf/tests/event_update.c
@@ -92,7 +92,7 @@ int test__event_update(struct test *test __maybe_unused, int subtest __maybe_unu
evsel = perf_evlist__first(evlist);
- TEST_ASSERT_VAL("failed to allos ids",
+ TEST_ASSERT_VAL("failed to allocate ids",
!perf_evsel__alloc_id(evsel, 1, 1));
perf_evlist__id_add(evlist, evsel, 0, 0, 123);
--
2.20.1
^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH] perf test: fix spelling mistake "allos" -> "allocate"
2019-09-11 15:21 ` Colin King
@ 2019-09-17 7:54 ` Mukesh Ojha
-1 siblings, 0 replies; 18+ messages in thread
From: Mukesh Ojha @ 2019-09-17 7:42 UTC (permalink / raw)
To: Colin King, Peter Zijlstra, Ingo Molnar,
Arnaldo Carvalho de Melo, Alexander Shishkin, Jiri Olsa,
Namhyung Kim
Cc: kernel-janitors, linux-kernel
On 9/11/2019 8:51 PM, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> There is a spelling mistake in a TEST_ASSERT_VAL message. Fix it.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
Reviewed-by: Mukesh Ojha <mojha@codeaurora.org>
Thanks,
Mukesh
> ---
> tools/perf/tests/event_update.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/perf/tests/event_update.c b/tools/perf/tests/event_update.c
> index cac4290e233a..7f0868a31a7f 100644
> --- a/tools/perf/tests/event_update.c
> +++ b/tools/perf/tests/event_update.c
> @@ -92,7 +92,7 @@ int test__event_update(struct test *test __maybe_unused, int subtest __maybe_unu
>
> evsel = perf_evlist__first(evlist);
>
> - TEST_ASSERT_VAL("failed to allos ids",
> + TEST_ASSERT_VAL("failed to allocate ids",
> !perf_evsel__alloc_id(evsel, 1, 1));
>
> perf_evlist__id_add(evlist, evsel, 0, 0, 123);
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] perf test: fix spelling mistake "allos" -> "allocate"
@ 2019-09-17 7:54 ` Mukesh Ojha
0 siblings, 0 replies; 18+ messages in thread
From: Mukesh Ojha @ 2019-09-17 7:54 UTC (permalink / raw)
To: Colin King, Peter Zijlstra, Ingo Molnar,
Arnaldo Carvalho de Melo, Alexander Shishkin, Jiri Olsa,
Namhyung Kim
Cc: kernel-janitors, linux-kernel
On 9/11/2019 8:51 PM, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> There is a spelling mistake in a TEST_ASSERT_VAL message. Fix it.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
Reviewed-by: Mukesh Ojha <mojha@codeaurora.org>
Thanks,
Mukesh
> ---
> tools/perf/tests/event_update.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/perf/tests/event_update.c b/tools/perf/tests/event_update.c
> index cac4290e233a..7f0868a31a7f 100644
> --- a/tools/perf/tests/event_update.c
> +++ b/tools/perf/tests/event_update.c
> @@ -92,7 +92,7 @@ int test__event_update(struct test *test __maybe_unused, int subtest __maybe_unu
>
> evsel = perf_evlist__first(evlist);
>
> - TEST_ASSERT_VAL("failed to allos ids",
> + TEST_ASSERT_VAL("failed to allocate ids",
> !perf_evsel__alloc_id(evsel, 1, 1));
>
> perf_evlist__id_add(evlist, evsel, 0, 0, 123);
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] perf test: fix spelling mistake "allos" -> "allocate"
2019-09-17 7:54 ` Mukesh Ojha
@ 2019-09-18 23:09 ` Arnaldo Carvalho de Melo
-1 siblings, 0 replies; 18+ messages in thread
From: Arnaldo Carvalho de Melo @ 2019-09-18 23:09 UTC (permalink / raw)
To: Mukesh Ojha
Cc: Colin King, Peter Zijlstra, Ingo Molnar, Alexander Shishkin,
Jiri Olsa, Namhyung Kim, kernel-janitors, linux-kernel
Em Tue, Sep 17, 2019 at 01:12:46PM +0530, Mukesh Ojha escreveu:
>
> On 9/11/2019 8:51 PM, Colin King wrote:
> > From: Colin Ian King <colin.king@canonical.com>
> >
> > There is a spelling mistake in a TEST_ASSERT_VAL message. Fix it.
> >
> > Signed-off-by: Colin Ian King <colin.king@canonical.com>
>
> Reviewed-by: Mukesh Ojha <mojha@codeaurora.org>
Thanks, applied.
> Thanks,
> Mukesh
>
> > ---
> > tools/perf/tests/event_update.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/tools/perf/tests/event_update.c b/tools/perf/tests/event_update.c
> > index cac4290e233a..7f0868a31a7f 100644
> > --- a/tools/perf/tests/event_update.c
> > +++ b/tools/perf/tests/event_update.c
> > @@ -92,7 +92,7 @@ int test__event_update(struct test *test __maybe_unused, int subtest __maybe_unu
> > evsel = perf_evlist__first(evlist);
> > - TEST_ASSERT_VAL("failed to allos ids",
> > + TEST_ASSERT_VAL("failed to allocate ids",
> > !perf_evsel__alloc_id(evsel, 1, 1));
> > perf_evlist__id_add(evlist, evsel, 0, 0, 123);
--
- Arnaldo
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] perf test: fix spelling mistake "allos" -> "allocate"
@ 2019-09-18 23:09 ` Arnaldo Carvalho de Melo
0 siblings, 0 replies; 18+ messages in thread
From: Arnaldo Carvalho de Melo @ 2019-09-18 23:09 UTC (permalink / raw)
To: Mukesh Ojha
Cc: Colin King, Peter Zijlstra, Ingo Molnar, Alexander Shishkin,
Jiri Olsa, Namhyung Kim, kernel-janitors, linux-kernel
Em Tue, Sep 17, 2019 at 01:12:46PM +0530, Mukesh Ojha escreveu:
>
> On 9/11/2019 8:51 PM, Colin King wrote:
> > From: Colin Ian King <colin.king@canonical.com>
> >
> > There is a spelling mistake in a TEST_ASSERT_VAL message. Fix it.
> >
> > Signed-off-by: Colin Ian King <colin.king@canonical.com>
>
> Reviewed-by: Mukesh Ojha <mojha@codeaurora.org>
Thanks, applied.
> Thanks,
> Mukesh
>
> > ---
> > tools/perf/tests/event_update.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/tools/perf/tests/event_update.c b/tools/perf/tests/event_update.c
> > index cac4290e233a..7f0868a31a7f 100644
> > --- a/tools/perf/tests/event_update.c
> > +++ b/tools/perf/tests/event_update.c
> > @@ -92,7 +92,7 @@ int test__event_update(struct test *test __maybe_unused, int subtest __maybe_unu
> > evsel = perf_evlist__first(evlist);
> > - TEST_ASSERT_VAL("failed to allos ids",
> > + TEST_ASSERT_VAL("failed to allocate ids",
> > !perf_evsel__alloc_id(evsel, 1, 1));
> > perf_evlist__id_add(evlist, evsel, 0, 0, 123);
--
- Arnaldo
^ permalink raw reply [flat|nested] 18+ messages in thread
* [tip: perf/urgent] perf test: Fix spelling mistake "allos" -> "allocate"
2019-09-11 15:21 ` Colin King
@ 2019-09-20 16:20 ` tip-bot2 for Colin Ian King
-1 siblings, 0 replies; 18+ messages in thread
From: tip-bot2 for Colin Ian King @ 2019-09-20 16:20 UTC (permalink / raw)
To: linux-tip-commits
Cc: Colin King, Mukesh Ojha, Alexander Shishkin, Jiri Olsa,
Namhyung Kim, Peter Zijlstra, kernel-janitors,
Arnaldo Carvalho de Melo, Ingo Molnar, Borislav Petkov,
linux-kernel
The following commit has been merged into the perf/urgent branch of tip:
Commit-ID: ce095c9ac293d1683f9fedb214811727dff0d508
Gitweb: https://git.kernel.org/tip/ce095c9ac293d1683f9fedb214811727dff0d508
Author: Colin Ian King <colin.king@canonical.com>
AuthorDate: Wed, 11 Sep 2019 16:21:48 +01:00
Committer: Arnaldo Carvalho de Melo <acme@redhat.com>
CommitterDate: Fri, 20 Sep 2019 10:28:26 -03:00
perf test: Fix spelling mistake "allos" -> "allocate"
There is a spelling mistake in a TEST_ASSERT_VAL message. Fix it.
Signed-off-by: Colin King <colin.king@canonical.com>
Reviewed-by: Mukesh Ojha <mojha@codeaurora.org>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: kernel-janitors@vger.kernel.org
Link: http://lore.kernel.org/lkml/20190911152148.17031-1-colin.king@canonical.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
tools/perf/tests/event_update.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/perf/tests/event_update.c b/tools/perf/tests/event_update.c
index 4bb772e..0497d90 100644
--- a/tools/perf/tests/event_update.c
+++ b/tools/perf/tests/event_update.c
@@ -94,7 +94,7 @@ int test__event_update(struct test *test __maybe_unused, int subtest __maybe_unu
evsel = perf_evlist__first(evlist);
- TEST_ASSERT_VAL("failed to allos ids",
+ TEST_ASSERT_VAL("failed to allocate ids",
!perf_evsel__alloc_id(evsel, 1, 1));
perf_evlist__id_add(evlist, evsel, 0, 0, 123);
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [tip: perf/urgent] perf test: Fix spelling mistake "allos" -> "allocate"
@ 2019-09-20 16:20 ` tip-bot2 for Colin Ian King
0 siblings, 0 replies; 18+ messages in thread
From: tip-bot2 for Colin Ian King @ 2019-09-20 16:20 UTC (permalink / raw)
To: linux-tip-commits
Cc: Colin King, Mukesh Ojha, Alexander Shishkin, Jiri Olsa,
Namhyung Kim, Peter Zijlstra, kernel-janitors,
Arnaldo Carvalho de Melo, Ingo Molnar, Borislav Petkov,
linux-kernel
The following commit has been merged into the perf/urgent branch of tip:
Commit-ID: ce095c9ac293d1683f9fedb214811727dff0d508
Gitweb: https://git.kernel.org/tip/ce095c9ac293d1683f9fedb214811727dff0d508
Author: Colin Ian King <colin.king@canonical.com>
AuthorDate: Wed, 11 Sep 2019 16:21:48 +01:00
Committer: Arnaldo Carvalho de Melo <acme@redhat.com>
CommitterDate: Fri, 20 Sep 2019 10:28:26 -03:00
perf test: Fix spelling mistake "allos" -> "allocate"
There is a spelling mistake in a TEST_ASSERT_VAL message. Fix it.
Signed-off-by: Colin King <colin.king@canonical.com>
Reviewed-by: Mukesh Ojha <mojha@codeaurora.org>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: kernel-janitors@vger.kernel.org
Link: http://lore.kernel.org/lkml/20190911152148.17031-1-colin.king@canonical.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
tools/perf/tests/event_update.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/perf/tests/event_update.c b/tools/perf/tests/event_update.c
index 4bb772e..0497d90 100644
--- a/tools/perf/tests/event_update.c
+++ b/tools/perf/tests/event_update.c
@@ -94,7 +94,7 @@ int test__event_update(struct test *test __maybe_unused, int subtest __maybe_unu
evsel = perf_evlist__first(evlist);
- TEST_ASSERT_VAL("failed to allos ids",
+ TEST_ASSERT_VAL("failed to allocate ids",
!perf_evsel__alloc_id(evsel, 1, 1));
perf_evlist__id_add(evlist, evsel, 0, 0, 123);
^ permalink raw reply related [flat|nested] 18+ messages in thread
end of thread, other threads:[~2019-09-20 16:22 UTC | newest]
Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-04-17 10:55 [PATCH] perf test: fix spelling mistake "leadking" -> "leaking" Colin King
2019-04-17 10:55 ` Colin King
2019-04-17 11:07 ` Mukesh Ojha
2019-04-17 11:19 ` Mukesh Ojha
2019-04-17 11:09 ` Colin Ian King
2019-04-17 11:09 ` Colin Ian King
2019-04-17 17:37 ` Arnaldo Carvalho de Melo
2019-04-17 17:37 ` Arnaldo Carvalho de Melo
2019-05-18 8:47 ` [tip:perf/core] perf test: Fix " tip-bot for Colin Ian King
-- strict thread matches above, loose matches on Subject: below --
2019-09-11 15:21 [PATCH] perf test: fix spelling mistake "allos" -> "allocate" Colin King
2019-09-11 15:21 ` Colin King
2019-09-17 7:42 ` Mukesh Ojha
2019-09-17 7:54 ` Mukesh Ojha
2019-09-18 23:09 ` Arnaldo Carvalho de Melo
2019-09-18 23:09 ` Arnaldo Carvalho de Melo
2019-09-20 16:20 ` [tip: perf/urgent] perf test: Fix " tip-bot2 for Colin Ian King
2019-09-20 16:20 ` tip-bot2 for Colin Ian King
2018-11-29 10:31 [PATCH] perf test: fix spelling mistake "sythesizing" -> "synthesizing" Colin King
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.