All of lore.kernel.org
 help / color / mirror / Atom feed
* [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.