[PATCH] perf test: fix spelling mistake "leadking" -> "leaking"

[PATCH] perf test: fix spelling mistake "leadking" -> "leaking"

[Colin King]

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

Re: [PATCH] perf test: fix spelling mistake "leadking" -> "leaking"

[Mukesh Ojha]

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;
>   }

Re: [PATCH] perf test: fix spelling mistake "leadking" -> "leaking"

[Colin Ian King]

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;
>>   }

Re: [PATCH] perf test: fix spelling mistake "leadking" -> "leaking"

[Arnaldo Carvalho de Melo]

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

[tip:perf/core] perf test: Fix spelling mistake "leadking" -> "leaking"

[tip-bot for Colin Ian King]

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;
 }