[PATCH] kvm:remove the unexpected word "that" in comments

Michael Ellerman mpe at ellerman.id.au
Wed Aug 3 12:54:41 AEST 2022


Christophe Leroy <christophe.leroy at csgroup.eu> writes:
> Le 16/07/2022 à 18:50, shaom Deng a écrit :
>> [Vous ne recevez pas souvent de courriers de dengshaomin at cdjrlc.com. Découvrez pourquoi ceci est important à https://aka.ms/LearnAboutSenderIdentification ]
>> 
>> delete the repeated word "that" in comments
>> 
>> Signed-off-by: shaom Deng <dengshaomin at cdjrlc.com>
>> ---
>>   arch/powerpc/kvm/book3s_64_mmu_hv.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/arch/powerpc/kvm/book3s_64_mmu_hv.c b/arch/powerpc/kvm/book3s_64_mmu_hv.c
>> index 514fd45c1994..73c6db20cd8a 100644
>> --- a/arch/powerpc/kvm/book3s_64_mmu_hv.c
>> +++ b/arch/powerpc/kvm/book3s_64_mmu_hv.c
>> @@ -1601,7 +1601,7 @@ long kvm_vm_ioctl_resize_hpt_commit(struct kvm *kvm,
>>    * is valid, it is written to the HPT as if an H_ENTER with the
>>    * exact flag set was done.  When the invalid count is non-zero
>>    * in the header written to the stream, the kernel will make
>> - * sure that that many HPTEs are invalid, and invalidate them
>> + * sure that many HPTEs are invalid, and invalidate them
>
> You are changing the meaning here.
>
> The sentence is:
>
> ... make sure that: that many HPTEs are invalid.
>
> You changed the meaning to:
>
> ... make sure that: many HPTEs are invalid.
>
>
>
> It doesn't mean the same.

Correct.

This was also discussed on an identical patch:

  https://lore.kernel.org/linuxppc-dev/87bktdd6s8.fsf@mpe.ellerman.id.au/

In next I have fixed all genuine occurences of repeated words that I
could find, so hopefully we will have fewer of these patches to process
from now on.

cheers


More information about the Linuxppc-dev mailing list