[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <49460004-94c1-ab6c-a7b7-0d65cb5f6cfb@canonical.com>
Date: Wed, 17 Apr 2019 12:09:25 +0100
From: Colin Ian King <colin.king@...onical.com>
To: Mukesh Ojha <mojha@...eaurora.org>,
Peter Zijlstra <peterz@...radead.org>,
Ingo Molnar <mingo@...hat.com>,
Arnaldo Carvalho de Melo <acme@...nel.org>,
Alexander Shishkin <alexander.shishkin@...ux.intel.com>,
Jiri Olsa <jolsa@...hat.com>,
Namhyung Kim <namhyung@...nel.org>
Cc: kernel-janitors@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] perf test: fix spelling mistake "leadking" -> "leaking"
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@...onical.com>
>>
>> There are a couple of spelling mistakes in test assert messages. Fix
>> them.
>>
>> Signed-off-by: Colin Ian King <colin.king@...onical.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@...eaurora.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;
>> }
Powered by blists - more mailing lists