* [PATCH][next] KVM: selftests: Fix spelling mistake "probabalistic" -> "probabilistic"
@ 2022-12-01 9:13 Colin Ian King
2022-12-02 17:21 ` Sean Christopherson
2022-12-02 18:24 ` Paolo Bonzini
0 siblings, 2 replies; 7+ messages in thread
From: Colin Ian King @ 2022-12-01 9:13 UTC (permalink / raw)
To: Paolo Bonzini, Shuah Khan, kvm, linux-kselftest
Cc: kernel-janitors, linux-kernel
There is a spelling mistake in some help text. Fix it.
Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
---
tools/testing/selftests/kvm/dirty_log_perf_test.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/testing/selftests/kvm/dirty_log_perf_test.c b/tools/testing/selftests/kvm/dirty_log_perf_test.c
index c33e89012ae6..e9d6d1aecf89 100644
--- a/tools/testing/selftests/kvm/dirty_log_perf_test.c
+++ b/tools/testing/selftests/kvm/dirty_log_perf_test.c
@@ -398,7 +398,7 @@ static void help(char *name)
printf(" -x: Split the memory region into this number of memslots.\n"
" (default: 1)\n");
printf(" -w: specify the percentage of pages which should be written to\n"
- " as an integer from 0-100 inclusive. This is probabalistic,\n"
+ " as an integer from 0-100 inclusive. This is probabilistic,\n"
" so -w X means each page has an X%% chance of writing\n"
" and a (100-X)%% chance of reading.\n"
" (default: 100 i.e. all pages are written to.)\n");
--
2.38.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH][next] KVM: selftests: Fix spelling mistake "probabalistic" -> "probabilistic"
2022-12-01 9:13 [PATCH][next] KVM: selftests: Fix spelling mistake "probabalistic" -> "probabilistic" Colin Ian King
@ 2022-12-02 17:21 ` Sean Christopherson
2022-12-02 17:43 ` Colin King (gmail)
2022-12-02 18:24 ` Paolo Bonzini
1 sibling, 1 reply; 7+ messages in thread
From: Sean Christopherson @ 2022-12-02 17:21 UTC (permalink / raw)
To: Colin Ian King
Cc: Paolo Bonzini, Shuah Khan, kvm, linux-kselftest, kernel-janitors,
linux-kernel
On Thu, Dec 01, 2022, Colin Ian King wrote:
> There is a spelling mistake in some help text. Fix it.
I assume you have a script/tool of some form to do spell checking? If so, can
you point me at it? I'd love to incorporate something like that into my workflow.
Thanks!
> Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
> ---
Reviewed-by: Sean Christopherson <seanjc@google.com>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH][next] KVM: selftests: Fix spelling mistake "probabalistic" -> "probabilistic"
2022-12-02 17:21 ` Sean Christopherson
@ 2022-12-02 17:43 ` Colin King (gmail)
2022-12-02 18:31 ` Sean Christopherson
0 siblings, 1 reply; 7+ messages in thread
From: Colin King (gmail) @ 2022-12-02 17:43 UTC (permalink / raw)
To: Sean Christopherson
Cc: Paolo Bonzini, Shuah Khan, kvm, linux-kselftest, kernel-janitors,
linux-kernel
On 02/12/2022 17:21, Sean Christopherson wrote:
> On Thu, Dec 01, 2022, Colin Ian King wrote:
>> There is a spelling mistake in some help text. Fix it.
>
> I assume you have a script/tool of some form to do spell checking? If so, can
> you point me at it? I'd love to incorporate something like that into my workflow.
https://github.com/ColinIanKing/kernelscan
It needs an appropriate dictionary to be installed. It's very fast (but
sometimes makes mistakes, I need to fix those!).
./kernelscan -k src_directory > mistakes.txt
I normally run this across the entire kernel git repo on a daily basis,
and diff the new results with the previous results using meld and figure
out what needs to be fixed.
You may be better off with using codespell
Colin
>
> Thanks!
>
>> Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
>> ---
>
> Reviewed-by: Sean Christopherson <seanjc@google.com>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH][next] KVM: selftests: Fix spelling mistake "probabalistic" -> "probabilistic"
2022-12-01 9:13 [PATCH][next] KVM: selftests: Fix spelling mistake "probabalistic" -> "probabilistic" Colin Ian King
2022-12-02 17:21 ` Sean Christopherson
@ 2022-12-02 18:24 ` Paolo Bonzini
1 sibling, 0 replies; 7+ messages in thread
From: Paolo Bonzini @ 2022-12-02 18:24 UTC (permalink / raw)
To: Colin Ian King, Shuah Khan, kvm, linux-kselftest
Cc: kernel-janitors, linux-kernel
On 12/1/22 10:13, Colin Ian King wrote:
> There is a spelling mistake in some help text. Fix it.
>
> Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
> ---
> tools/testing/selftests/kvm/dirty_log_perf_test.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/testing/selftests/kvm/dirty_log_perf_test.c b/tools/testing/selftests/kvm/dirty_log_perf_test.c
> index c33e89012ae6..e9d6d1aecf89 100644
> --- a/tools/testing/selftests/kvm/dirty_log_perf_test.c
> +++ b/tools/testing/selftests/kvm/dirty_log_perf_test.c
> @@ -398,7 +398,7 @@ static void help(char *name)
> printf(" -x: Split the memory region into this number of memslots.\n"
> " (default: 1)\n");
> printf(" -w: specify the percentage of pages which should be written to\n"
> - " as an integer from 0-100 inclusive. This is probabalistic,\n"
> + " as an integer from 0-100 inclusive. This is probabilistic,\n"
> " so -w X means each page has an X%% chance of writing\n"
> " and a (100-X)%% chance of reading.\n"
> " (default: 100 i.e. all pages are written to.)\n");
Queued, thanks.
Paolo
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH][next] KVM: selftests: Fix spelling mistake "probabalistic" -> "probabilistic"
2022-12-02 17:43 ` Colin King (gmail)
@ 2022-12-02 18:31 ` Sean Christopherson
2022-12-02 18:57 ` Sean Christopherson
0 siblings, 1 reply; 7+ messages in thread
From: Sean Christopherson @ 2022-12-02 18:31 UTC (permalink / raw)
To: Colin King (gmail)
Cc: Paolo Bonzini, Shuah Khan, kvm, linux-kselftest, kernel-janitors,
linux-kernel
On Fri, Dec 02, 2022, Colin King (gmail) wrote:
> On 02/12/2022 17:21, Sean Christopherson wrote:
> > On Thu, Dec 01, 2022, Colin Ian King wrote:
> > > There is a spelling mistake in some help text. Fix it.
> >
> > I assume you have a script/tool of some form to do spell checking? If so, can
> > you point me at it? I'd love to incorporate something like that into my workflow.
>
> https://github.com/ColinIanKing/kernelscan
>
> It needs an appropriate dictionary to be installed. It's very fast (but
> sometimes makes mistakes, I need to fix those!).
>
> ./kernelscan -k src_directory > mistakes.txt
Ah, it's specifically checking messages to avoid false postives on code and whatnot.
Neat!
> I normally run this across the entire kernel git repo on a daily basis, and
> diff the new results with the previous results using meld and figure out
> what needs to be fixed.
>
> You may be better off with using codespell
Heh, my kind of nitpicking people :-)
MSDOS->MS-DOS
Thanks a ton, that's exactly what I was looking for!
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH][next] KVM: selftests: Fix spelling mistake "probabalistic" -> "probabilistic"
2022-12-02 18:31 ` Sean Christopherson
@ 2022-12-02 18:57 ` Sean Christopherson
2022-12-02 20:24 ` Colin King (gmail)
0 siblings, 1 reply; 7+ messages in thread
From: Sean Christopherson @ 2022-12-02 18:57 UTC (permalink / raw)
To: Colin King (gmail)
Cc: Paolo Bonzini, Shuah Khan, kvm, linux-kselftest, kernel-janitors,
linux-kernel
On Fri, Dec 02, 2022, Sean Christopherson wrote:
> On Fri, Dec 02, 2022, Colin King (gmail) wrote:
> > You may be better off with using codespell
>
> Heh, my kind of nitpicking people :-)
>
> MSDOS->MS-DOS
>
> Thanks a ton, that's exactly what I was looking for!
For anyone following along and/or laughing at me, checkpatch even supports using
codespell, e.g.
./scripts/checkpatch.pl -g HEAD --codespell
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH][next] KVM: selftests: Fix spelling mistake "probabalistic" -> "probabilistic"
2022-12-02 18:57 ` Sean Christopherson
@ 2022-12-02 20:24 ` Colin King (gmail)
0 siblings, 0 replies; 7+ messages in thread
From: Colin King (gmail) @ 2022-12-02 20:24 UTC (permalink / raw)
To: Sean Christopherson
Cc: Paolo Bonzini, Shuah Khan, kvm, linux-kselftest, kernel-janitors,
linux-kernel
On 02/12/2022 18:57, Sean Christopherson wrote:
> On Fri, Dec 02, 2022, Sean Christopherson wrote:
>> On Fri, Dec 02, 2022, Colin King (gmail) wrote:
>>> You may be better off with using codespell
>>
>> Heh, my kind of nitpicking people :-)
>>
>> MSDOS->MS-DOS
>>
>> Thanks a ton, that's exactly what I was looking for!
>
> For anyone following along and/or laughing at me, checkpatch even supports using
> codespell, e.g.
>
> ./scripts/checkpatch.pl -g HEAD --codespell
checkpatch also uses a misspelling dictionary that I occasionally
contribute to with common spelling mistakes that I find and fix.
Colin
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-12-02 20:24 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-12-01 9:13 [PATCH][next] KVM: selftests: Fix spelling mistake "probabalistic" -> "probabilistic" Colin Ian King
2022-12-02 17:21 ` Sean Christopherson
2022-12-02 17:43 ` Colin King (gmail)
2022-12-02 18:31 ` Sean Christopherson
2022-12-02 18:57 ` Sean Christopherson
2022-12-02 20:24 ` Colin King (gmail)
2022-12-02 18:24 ` Paolo Bonzini
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).