2021-02-13 15:36:37

by Bhaskar Chowdhury

[permalink] [raw]
Subject: [PATCH] arch: s390: kvm: Fix oustanding to outstanding in the file kvm-s390.c


s/oustanding/outstanding/

Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
arch/s390/kvm/kvm-s390.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c
index dbafd057ca6a..1d01afaca9fe 100644
--- a/arch/s390/kvm/kvm-s390.c
+++ b/arch/s390/kvm/kvm-s390.c
@@ -4545,7 +4545,7 @@ int kvm_s390_vcpu_start(struct kvm_vcpu *vcpu)
/*
* As we are starting a second VCPU, we have to disable
* the IBS facility on all VCPUs to remove potentially
- * oustanding ENABLE requests.
+ * outstanding ENABLE requests.
*/
__disable_ibs_on_all_vcpus(vcpu->kvm);
}
--
2.30.1


2021-02-13 15:51:01

by Randy Dunlap

[permalink] [raw]
Subject: Re: [PATCH] arch: s390: kvm: Fix oustanding to outstanding in the file kvm-s390.c

On 2/13/21 7:32 AM, Bhaskar Chowdhury wrote:
>
> s/oustanding/outstanding/
>
> Signed-off-by: Bhaskar Chowdhury <[email protected]>

Acked-by: Randy Dunlap <[email protected]>

> ---
> arch/s390/kvm/kvm-s390.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c
> index dbafd057ca6a..1d01afaca9fe 100644
> --- a/arch/s390/kvm/kvm-s390.c
> +++ b/arch/s390/kvm/kvm-s390.c
> @@ -4545,7 +4545,7 @@ int kvm_s390_vcpu_start(struct kvm_vcpu *vcpu)
> /*
> * As we are starting a second VCPU, we have to disable
> * the IBS facility on all VCPUs to remove potentially
> - * oustanding ENABLE requests.
> + * outstanding ENABLE requests.
> */
> __disable_ibs_on_all_vcpus(vcpu->kvm);
> }
> --
> 2.30.1
>


--
~Randy

2021-02-15 09:10:46

by Janosch Frank

[permalink] [raw]
Subject: Re: [PATCH] arch: s390: kvm: Fix oustanding to outstanding in the file kvm-s390.c

On 2/13/21 4:32 PM, Bhaskar Chowdhury wrote:
>
> s/oustanding/outstanding/

Hey Bhaskar,

while I do encourage anyone to send in changes I'm not a big fan of
comment fixes if they are only a couple of characters and when the
meaning is still intact despite the spelling mistake.

You're creating more work for me than you had writing this patch and the
improvement is close to zero.

Be warned that I might not pick up such patches in the future.


If you're ok with it I'll fix up the subject to this and pick up the patch:
"kvm: s390: Fix comment spelling in kvm_s390_vcpu_start()"

Cheers,
Janosch

>
> Signed-off-by: Bhaskar Chowdhury <[email protected]>
> ---
> arch/s390/kvm/kvm-s390.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c
> index dbafd057ca6a..1d01afaca9fe 100644
> --- a/arch/s390/kvm/kvm-s390.c
> +++ b/arch/s390/kvm/kvm-s390.c
> @@ -4545,7 +4545,7 @@ int kvm_s390_vcpu_start(struct kvm_vcpu *vcpu)
> /*
> * As we are starting a second VCPU, we have to disable
> * the IBS facility on all VCPUs to remove potentially
> - * oustanding ENABLE requests.
> + * outstanding ENABLE requests.
> */
> __disable_ibs_on_all_vcpus(vcpu->kvm);
> }
> --
> 2.30.1
>



Attachments:
OpenPGP_signature (855.00 B)
OpenPGP digital signature

2021-02-16 13:42:23

by Bhaskar Chowdhury

[permalink] [raw]
Subject: Re: [PATCH] arch: s390: kvm: Fix oustanding to outstanding in the file kvm-s390.c

On 10:08 Mon 15 Feb 2021, Janosch Frank wrote:
>On 2/13/21 4:32 PM, Bhaskar Chowdhury wrote:
>>
>> s/oustanding/outstanding/
>
>Hey Bhaskar,
>
>while I do encourage anyone to send in changes I'm not a big fan of
>comment fixes if they are only a couple of characters and when the
>meaning is still intact despite the spelling mistake.
>
>You're creating more work for me than you had writing this patch and the
>improvement is close to zero.
>
>Be warned that I might not pick up such patches in the future.
>
>
>If you're ok with it I'll fix up the subject to this and pick up the patch:
>"kvm: s390: Fix comment spelling in kvm_s390_vcpu_start()"
>
Pls do.

>Cheers,
>Janosch
>
>>
>> Signed-off-by: Bhaskar Chowdhury <[email protected]>
>> ---
>> arch/s390/kvm/kvm-s390.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c
>> index dbafd057ca6a..1d01afaca9fe 100644
>> --- a/arch/s390/kvm/kvm-s390.c
>> +++ b/arch/s390/kvm/kvm-s390.c
>> @@ -4545,7 +4545,7 @@ int kvm_s390_vcpu_start(struct kvm_vcpu *vcpu)
>> /*
>> * As we are starting a second VCPU, we have to disable
>> * the IBS facility on all VCPUs to remove potentially
>> - * oustanding ENABLE requests.
>> + * outstanding ENABLE requests.
>> */
>> __disable_ibs_on_all_vcpus(vcpu->kvm);
>> }
>> --
>> 2.30.1
>>
>
>




Attachments:
(No filename) (1.40 kB)
signature.asc (499.00 B)
Download all attachments