* [PATCH -next] KVM: x86: remove set but not used variable 'called'
@ 2019-11-19 3:06 Mao Wenan
2019-11-19 11:58 ` Vitaly Kuznetsov
0 siblings, 1 reply; 14+ messages in thread
From: Mao Wenan @ 2019-11-19 3:06 UTC (permalink / raw)
To: pbonzini, rkrcmar, sean.j.christopherson, vkuznets, wanpengli,
jmattson, joro, tglx, mingo, bp, hpa
Cc: kvm, linux-kernel, kernel-janitors, Mao Wenan
Fixes gcc '-Wunused-but-set-variable' warning:
arch/x86/kvm/x86.c: In function kvm_make_scan_ioapic_request_mask:
arch/x86/kvm/x86.c:7911:7: warning: variable called set but not
used [-Wunused-but-set-variable]
It is not used since commit 7ee30bc132c6 ("KVM: x86: deliver KVM
IOAPIC scan request to target vCPUs")
Signed-off-by: Mao Wenan <maowenan@huawei.com>
---
arch/x86/kvm/x86.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c
index 0d0a682..870f0bc 100644
--- a/arch/x86/kvm/x86.c
+++ b/arch/x86/kvm/x86.c
@@ -7908,12 +7908,11 @@ void kvm_make_scan_ioapic_request_mask(struct kvm *kvm,
unsigned long *vcpu_bitmap)
{
cpumask_var_t cpus;
- bool called;
zalloc_cpumask_var(&cpus, GFP_ATOMIC);
- called = kvm_make_vcpus_request_mask(kvm, KVM_REQ_SCAN_IOAPIC,
- vcpu_bitmap, cpus);
+ kvm_make_vcpus_request_mask(kvm, KVM_REQ_SCAN_IOAPIC,
+ vcpu_bitmap, cpus);
free_cpumask_var(cpus);
}
--
2.7.4
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH -next] KVM: x86: remove set but not used variable 'called'
2019-11-19 3:06 [PATCH -next] KVM: x86: remove set but not used variable 'called' Mao Wenan
@ 2019-11-19 11:58 ` Vitaly Kuznetsov
2019-11-19 12:14 ` Dan Carpenter
2019-11-19 12:42 ` [PATCH -next] " maowenan
0 siblings, 2 replies; 14+ messages in thread
From: Vitaly Kuznetsov @ 2019-11-19 11:58 UTC (permalink / raw)
To: Mao Wenan, pbonzini, rkrcmar, sean.j.christopherson, wanpengli,
jmattson, joro, tglx, mingo, bp, hpa
Cc: kvm, linux-kernel, kernel-janitors, Mao Wenan
Mao Wenan <maowenan@huawei.com> writes:
> Fixes gcc '-Wunused-but-set-variable' warning:
>
> arch/x86/kvm/x86.c: In function kvm_make_scan_ioapic_request_mask:
> arch/x86/kvm/x86.c:7911:7: warning: variable called set but not
> used [-Wunused-but-set-variable]
>
> It is not used since commit 7ee30bc132c6 ("KVM: x86: deliver KVM
> IOAPIC scan request to target vCPUs")
Better expressed as
Fixes: 7ee30bc132c6 ("KVM: x86: deliver KVM IOAPIC scan request to target vCPUs")
>
> Signed-off-by: Mao Wenan <maowenan@huawei.com>
> ---
> arch/x86/kvm/x86.c | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c
> index 0d0a682..870f0bc 100644
> --- a/arch/x86/kvm/x86.c
> +++ b/arch/x86/kvm/x86.c
> @@ -7908,12 +7908,11 @@ void kvm_make_scan_ioapic_request_mask(struct kvm *kvm,
> unsigned long *vcpu_bitmap)
> {
> cpumask_var_t cpus;
> - bool called;
>
> zalloc_cpumask_var(&cpus, GFP_ATOMIC);
>
> - called = kvm_make_vcpus_request_mask(kvm, KVM_REQ_SCAN_IOAPIC,
> - vcpu_bitmap, cpus);
> + kvm_make_vcpus_request_mask(kvm, KVM_REQ_SCAN_IOAPIC,
> + vcpu_bitmap, cpus);
IMHO as kvm_make_vcpus_request_mask() returns value it would probably
make sense to explicitly show that we're not interested in the result,
(void)kvm_make_vcpus_request_mask()
>
> free_cpumask_var(cpus);
> }
--
Vitaly
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH -next] KVM: x86: remove set but not used variable 'called'
2019-11-19 11:58 ` Vitaly Kuznetsov
@ 2019-11-19 12:14 ` Dan Carpenter
2019-11-19 12:28 ` Vitaly Kuznetsov
2019-11-21 9:13 ` Paolo Bonzini
2019-11-19 12:42 ` [PATCH -next] " maowenan
1 sibling, 2 replies; 14+ messages in thread
From: Dan Carpenter @ 2019-11-19 12:14 UTC (permalink / raw)
To: Vitaly Kuznetsov
Cc: Mao Wenan, pbonzini, rkrcmar, sean.j.christopherson, wanpengli,
jmattson, joro, tglx, mingo, bp, hpa, kvm, linux-kernel,
kernel-janitors
On Tue, Nov 19, 2019 at 12:58:54PM +0100, Vitaly Kuznetsov wrote:
> Mao Wenan <maowenan@huawei.com> writes:
>
> > Fixes gcc '-Wunused-but-set-variable' warning:
> >
> > arch/x86/kvm/x86.c: In function kvm_make_scan_ioapic_request_mask:
> > arch/x86/kvm/x86.c:7911:7: warning: variable called set but not
> > used [-Wunused-but-set-variable]
> >
> > It is not used since commit 7ee30bc132c6 ("KVM: x86: deliver KVM
> > IOAPIC scan request to target vCPUs")
>
> Better expressed as
>
> Fixes: 7ee30bc132c6 ("KVM: x86: deliver KVM IOAPIC scan request to target vCPUs")
>
There is sort of a debate about this whether the Fixes tag should be
used if it's only a cleanup.
regards,
dan carpenter
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH -next] KVM: x86: remove set but not used variable 'called'
2019-11-19 12:14 ` Dan Carpenter
@ 2019-11-19 12:28 ` Vitaly Kuznetsov
2019-11-19 12:39 ` Dan Carpenter
2019-11-21 9:13 ` Paolo Bonzini
1 sibling, 1 reply; 14+ messages in thread
From: Vitaly Kuznetsov @ 2019-11-19 12:28 UTC (permalink / raw)
To: Dan Carpenter
Cc: Mao Wenan, pbonzini, rkrcmar, sean.j.christopherson, wanpengli,
jmattson, joro, tglx, mingo, bp, hpa, kvm, linux-kernel,
kernel-janitors
Dan Carpenter <dan.carpenter@oracle.com> writes:
> On Tue, Nov 19, 2019 at 12:58:54PM +0100, Vitaly Kuznetsov wrote:
>> Mao Wenan <maowenan@huawei.com> writes:
>>
>> > Fixes gcc '-Wunused-but-set-variable' warning:
>> >
>> > arch/x86/kvm/x86.c: In function kvm_make_scan_ioapic_request_mask:
>> > arch/x86/kvm/x86.c:7911:7: warning: variable called set but not
>> > used [-Wunused-but-set-variable]
>> >
>> > It is not used since commit 7ee30bc132c6 ("KVM: x86: deliver KVM
>> > IOAPIC scan request to target vCPUs")
>>
>> Better expressed as
>>
>> Fixes: 7ee30bc132c6 ("KVM: x86: deliver KVM IOAPIC scan request to target vCPUs")
>>
>
> There is sort of a debate about this whether the Fixes tag should be
> used if it's only a cleanup.
>
I have to admit I'm involved in doing backporting sometimes and I really
appreciate Fixes: tags. Just so you know on which side of the debate I
am :-)
--
Vitaly
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH -next] KVM: x86: remove set but not used variable 'called'
2019-11-19 12:28 ` Vitaly Kuznetsov
@ 2019-11-19 12:39 ` Dan Carpenter
2019-11-19 13:25 ` Vitaly Kuznetsov
0 siblings, 1 reply; 14+ messages in thread
From: Dan Carpenter @ 2019-11-19 12:39 UTC (permalink / raw)
To: Vitaly Kuznetsov
Cc: Mao Wenan, pbonzini, rkrcmar, sean.j.christopherson, wanpengli,
jmattson, joro, tglx, mingo, bp, hpa, kvm, linux-kernel,
kernel-janitors
On Tue, Nov 19, 2019 at 01:28:32PM +0100, Vitaly Kuznetsov wrote:
> Dan Carpenter <dan.carpenter@oracle.com> writes:
>
> > On Tue, Nov 19, 2019 at 12:58:54PM +0100, Vitaly Kuznetsov wrote:
> >> Mao Wenan <maowenan@huawei.com> writes:
> >>
> >> > Fixes gcc '-Wunused-but-set-variable' warning:
> >> >
> >> > arch/x86/kvm/x86.c: In function kvm_make_scan_ioapic_request_mask:
> >> > arch/x86/kvm/x86.c:7911:7: warning: variable called set but not
> >> > used [-Wunused-but-set-variable]
> >> >
> >> > It is not used since commit 7ee30bc132c6 ("KVM: x86: deliver KVM
> >> > IOAPIC scan request to target vCPUs")
> >>
> >> Better expressed as
> >>
> >> Fixes: 7ee30bc132c6 ("KVM: x86: deliver KVM IOAPIC scan request to target vCPUs")
> >>
> >
> > There is sort of a debate about this whether the Fixes tag should be
> > used if it's only a cleanup.
> >
>
> I have to admit I'm involved in doing backporting sometimes and I really
> appreciate Fixes: tags. Just so you know on which side of the debate I
> am :-)
But we're not going to backport this hopefully?
regards,
dan carpenter
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH -next] KVM: x86: remove set but not used variable 'called'
2019-11-19 11:58 ` Vitaly Kuznetsov
2019-11-19 12:14 ` Dan Carpenter
@ 2019-11-19 12:42 ` maowenan
2019-11-19 13:27 ` Vitaly Kuznetsov
1 sibling, 1 reply; 14+ messages in thread
From: maowenan @ 2019-11-19 12:42 UTC (permalink / raw)
To: Vitaly Kuznetsov, pbonzini, rkrcmar, sean.j.christopherson,
wanpengli, jmattson, joro, tglx, mingo, bp, hpa
Cc: kvm, linux-kernel, kernel-janitors
在 2019/11/19 19:58, Vitaly Kuznetsov 写道:
> Mao Wenan <maowenan@huawei.com> writes:
>
>> Fixes gcc '-Wunused-but-set-variable' warning:
>>
>> arch/x86/kvm/x86.c: In function kvm_make_scan_ioapic_request_mask:
>> arch/x86/kvm/x86.c:7911:7: warning: variable called set but not
>> used [-Wunused-but-set-variable]
>>
>> It is not used since commit 7ee30bc132c6 ("KVM: x86: deliver KVM
>> IOAPIC scan request to target vCPUs")
>
> Better expressed as
>
> Fixes: 7ee30bc132c6 ("KVM: x86: deliver KVM IOAPIC scan request to target vCPUs")
This is just a cleanup, so Fixes tag is no need.
>
>>
>> Signed-off-by: Mao Wenan <maowenan@huawei.com>
>> ---
>> arch/x86/kvm/x86.c | 5 ++---
>> 1 file changed, 2 insertions(+), 3 deletions(-)
>>
>> diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c
>> index 0d0a682..870f0bc 100644
>> --- a/arch/x86/kvm/x86.c
>> +++ b/arch/x86/kvm/x86.c
>> @@ -7908,12 +7908,11 @@ void kvm_make_scan_ioapic_request_mask(struct kvm *kvm,
>> unsigned long *vcpu_bitmap)
>> {
>> cpumask_var_t cpus;
>> - bool called;
>>
>> zalloc_cpumask_var(&cpus, GFP_ATOMIC);
>>
>> - called = kvm_make_vcpus_request_mask(kvm, KVM_REQ_SCAN_IOAPIC,
>> - vcpu_bitmap, cpus);
>> + kvm_make_vcpus_request_mask(kvm, KVM_REQ_SCAN_IOAPIC,
>> + vcpu_bitmap, cpus);
>
> IMHO as kvm_make_vcpus_request_mask() returns value it would probably
> make sense to explicitly show that we're not interested in the result,
>
> (void)kvm_make_vcpus_request_mask()
thanks, but I think is no need to add (void) before kvm_make_vcpus_request_mask()
because we are not interested in it's return value.
>
>>
>> free_cpumask_var(cpus);
>> }
>
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH -next] KVM: x86: remove set but not used variable 'called'
2019-11-19 12:39 ` Dan Carpenter
@ 2019-11-19 13:25 ` Vitaly Kuznetsov
2019-11-22 11:58 ` Nitesh Narayan Lal
0 siblings, 1 reply; 14+ messages in thread
From: Vitaly Kuznetsov @ 2019-11-19 13:25 UTC (permalink / raw)
To: Dan Carpenter
Cc: Mao Wenan, pbonzini, rkrcmar, sean.j.christopherson, wanpengli,
jmattson, joro, tglx, mingo, bp, hpa, kvm, linux-kernel,
kernel-janitors
Dan Carpenter <dan.carpenter@oracle.com> writes:
> On Tue, Nov 19, 2019 at 01:28:32PM +0100, Vitaly Kuznetsov wrote:
>> Dan Carpenter <dan.carpenter@oracle.com> writes:
>>
>> > On Tue, Nov 19, 2019 at 12:58:54PM +0100, Vitaly Kuznetsov wrote:
>> >> Mao Wenan <maowenan@huawei.com> writes:
>> >>
>> >> > Fixes gcc '-Wunused-but-set-variable' warning:
>> >> >
>> >> > arch/x86/kvm/x86.c: In function kvm_make_scan_ioapic_request_mask:
>> >> > arch/x86/kvm/x86.c:7911:7: warning: variable called set but not
>> >> > used [-Wunused-but-set-variable]
>> >> >
>> >> > It is not used since commit 7ee30bc132c6 ("KVM: x86: deliver KVM
>> >> > IOAPIC scan request to target vCPUs")
>> >>
>> >> Better expressed as
>> >>
>> >> Fixes: 7ee30bc132c6 ("KVM: x86: deliver KVM IOAPIC scan request to target vCPUs")
>> >>
>> >
>> > There is sort of a debate about this whether the Fixes tag should be
>> > used if it's only a cleanup.
>> >
>>
>> I have to admit I'm involved in doing backporting sometimes and I really
>> appreciate Fixes: tags. Just so you know on which side of the debate I
>> am :-)
>
> But we're not going to backport this hopefully?
>
In case we're speaking about stable@ kernels, 7ee30bc132c6 doesn't look
like a good candidate (to me) but who knows, it may get pulled in
because of some code dependency or some other 'autosel magic'. And
that's when 'Fixes:' tags become handy.
--
Vitaly
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH -next] KVM: x86: remove set but not used variable 'called'
2019-11-19 12:42 ` [PATCH -next] " maowenan
@ 2019-11-19 13:27 ` Vitaly Kuznetsov
0 siblings, 0 replies; 14+ messages in thread
From: Vitaly Kuznetsov @ 2019-11-19 13:27 UTC (permalink / raw)
To: maowenan
Cc: kvm, linux-kernel, kernel-janitors, pbonzini, rkrcmar,
sean.j.christopherson, wanpengli, jmattson, joro, tglx, mingo,
bp, hpa
maowenan <maowenan@huawei.com> writes:
> 在 2019/11/19 19:58, Vitaly Kuznetsov 写道:
>> Mao Wenan <maowenan@huawei.com> writes:
>>
>>> Fixes gcc '-Wunused-but-set-variable' warning:
>>>
>>> arch/x86/kvm/x86.c: In function kvm_make_scan_ioapic_request_mask:
>>> arch/x86/kvm/x86.c:7911:7: warning: variable called set but not
>>> used [-Wunused-but-set-variable]
>>>
>>> It is not used since commit 7ee30bc132c6 ("KVM: x86: deliver KVM
>>> IOAPIC scan request to target vCPUs")
>>
>> Better expressed as
>>
>> Fixes: 7ee30bc132c6 ("KVM: x86: deliver KVM IOAPIC scan request to target vCPUs")
>
> This is just a cleanup, so Fixes tag is no need.
>>
Just a cleanup -- unless we compile with '-Werror'.
>>>
>>> Signed-off-by: Mao Wenan <maowenan@huawei.com>
>>> ---
>>> arch/x86/kvm/x86.c | 5 ++---
>>> 1 file changed, 2 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c
>>> index 0d0a682..870f0bc 100644
>>> --- a/arch/x86/kvm/x86.c
>>> +++ b/arch/x86/kvm/x86.c
>>> @@ -7908,12 +7908,11 @@ void kvm_make_scan_ioapic_request_mask(struct kvm *kvm,
>>> unsigned long *vcpu_bitmap)
>>> {
>>> cpumask_var_t cpus;
>>> - bool called;
>>>
>>> zalloc_cpumask_var(&cpus, GFP_ATOMIC);
>>>
>>> - called = kvm_make_vcpus_request_mask(kvm, KVM_REQ_SCAN_IOAPIC,
>>> - vcpu_bitmap, cpus);
>>> + kvm_make_vcpus_request_mask(kvm, KVM_REQ_SCAN_IOAPIC,
>>> + vcpu_bitmap, cpus);
>>
>> IMHO as kvm_make_vcpus_request_mask() returns value it would probably
>> make sense to explicitly show that we're not interested in the result,
>>
>> (void)kvm_make_vcpus_request_mask()
>
> thanks, but I think is no need to add (void) before kvm_make_vcpus_request_mask()
> because we are not interested in it's return value.
Hm, that's exactly the reason why I suggested adding it there :-) Not a
big deal, feel free to ignore.
--
Vitaly
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH -next] KVM: x86: remove set but not used variable 'called'
2019-11-19 12:14 ` Dan Carpenter
2019-11-19 12:28 ` Vitaly Kuznetsov
@ 2019-11-21 9:13 ` Paolo Bonzini
2019-11-22 0:48 ` maowenan
2019-11-22 2:39 ` [PATCH -next v2] " Mao Wenan
1 sibling, 2 replies; 14+ messages in thread
From: Paolo Bonzini @ 2019-11-21 9:13 UTC (permalink / raw)
To: Dan Carpenter, Vitaly Kuznetsov
Cc: Mao Wenan, rkrcmar, sean.j.christopherson, wanpengli, jmattson,
joro, tglx, mingo, bp, hpa, kvm, linux-kernel, kernel-janitors
On 19/11/19 13:14, Dan Carpenter wrote:
>> Better expressed as
>>
>> Fixes: 7ee30bc132c6 ("KVM: x86: deliver KVM IOAPIC scan request to target vCPUs")
>
> There is sort of a debate about this whether the Fixes tag should be
> used if it's only a cleanup.
The other debate is whether this is a cleanup, since the build is broken
with -Werror. I agree that code cleanups generally don't deserve Fixes
tags, but this patch IMHO does.
Paolo
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH -next] KVM: x86: remove set but not used variable 'called'
2019-11-21 9:13 ` Paolo Bonzini
@ 2019-11-22 0:48 ` maowenan
2019-11-22 2:39 ` [PATCH -next v2] " Mao Wenan
1 sibling, 0 replies; 14+ messages in thread
From: maowenan @ 2019-11-22 0:48 UTC (permalink / raw)
To: Paolo Bonzini, Dan Carpenter, Vitaly Kuznetsov
Cc: rkrcmar, sean.j.christopherson, wanpengli, jmattson, joro, tglx,
mingo, bp, hpa, kvm, linux-kernel, kernel-janitors
shall we send v2 with fixes tag?
在 2019/11/21 17:13, Paolo Bonzini 写道:
> atch IMHO does.
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH -next v2] KVM: x86: remove set but not used variable 'called'
2019-11-21 9:13 ` Paolo Bonzini
2019-11-22 0:48 ` maowenan
@ 2019-11-22 2:39 ` Mao Wenan
1 sibling, 0 replies; 14+ messages in thread
From: Mao Wenan @ 2019-11-22 2:39 UTC (permalink / raw)
To: pbonzini, rkrcmar, sean.j.christopherson, vkuznets, wanpengli,
jmattson, joro, tglx, mingo, bp, hpa, nitesh
Cc: kvm, linux-kernel, kernel-janitors, Mao Wenan
Fixes gcc '-Wunused-but-set-variable' warning:
arch/x86/kvm/x86.c: In function kvm_make_scan_ioapic_request_mask:
arch/x86/kvm/x86.c:7911:7: warning: variable called set but not
used [-Wunused-but-set-variable]
It is not used since commit 7ee30bc132c6 ("KVM: x86: deliver KVM
IOAPIC scan request to target vCPUs")
Fixes: 7ee30bc132c6 ("KVM: x86: deliver KVM IOAPIC scan request to target vCPUs")
Signed-off-by: Mao Wenan <maowenan@huawei.com>
---
v2: add fixes tag since Paolo and Vitaly proposal.
arch/x86/kvm/x86.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c
index 0d0a682..870f0bc 100644
--- a/arch/x86/kvm/x86.c
+++ b/arch/x86/kvm/x86.c
@@ -7908,12 +7908,11 @@ void kvm_make_scan_ioapic_request_mask(struct kvm *kvm,
unsigned long *vcpu_bitmap)
{
cpumask_var_t cpus;
- bool called;
zalloc_cpumask_var(&cpus, GFP_ATOMIC);
- called = kvm_make_vcpus_request_mask(kvm, KVM_REQ_SCAN_IOAPIC,
- vcpu_bitmap, cpus);
+ kvm_make_vcpus_request_mask(kvm, KVM_REQ_SCAN_IOAPIC,
+ vcpu_bitmap, cpus);
free_cpumask_var(cpus);
}
--
2.7.4
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH -next] KVM: x86: remove set but not used variable 'called'
2019-11-19 13:25 ` Vitaly Kuznetsov
@ 2019-11-22 11:58 ` Nitesh Narayan Lal
2019-11-22 12:25 ` Dan Carpenter
0 siblings, 1 reply; 14+ messages in thread
From: Nitesh Narayan Lal @ 2019-11-22 11:58 UTC (permalink / raw)
To: Vitaly Kuznetsov, Dan Carpenter
Cc: Mao Wenan, pbonzini, rkrcmar, sean.j.christopherson, wanpengli,
jmattson, joro, tglx, mingo, bp, hpa, kvm, linux-kernel,
kernel-janitors, Marcelo Tosatti
[-- Attachment #1.1: Type: text/plain, Size: 1731 bytes --]
On 11/19/19 8:25 AM, Vitaly Kuznetsov wrote:
> Dan Carpenter <dan.carpenter@oracle.com> writes:
>
>> On Tue, Nov 19, 2019 at 01:28:32PM +0100, Vitaly Kuznetsov wrote:
>>> Dan Carpenter <dan.carpenter@oracle.com> writes:
>>>
>>>> On Tue, Nov 19, 2019 at 12:58:54PM +0100, Vitaly Kuznetsov wrote:
>>>>> Mao Wenan <maowenan@huawei.com> writes:
>>>>>
>>>>>> Fixes gcc '-Wunused-but-set-variable' warning:
>>>>>>
>>>>>> arch/x86/kvm/x86.c: In function kvm_make_scan_ioapic_request_mask:
>>>>>> arch/x86/kvm/x86.c:7911:7: warning: variable called set but not
>>>>>> used [-Wunused-but-set-variable]
>>>>>>
>>>>>> It is not used since commit 7ee30bc132c6 ("KVM: x86: deliver KVM
>>>>>> IOAPIC scan request to target vCPUs")
>>>>> Better expressed as
>>>>>
>>>>> Fixes: 7ee30bc132c6 ("KVM: x86: deliver KVM IOAPIC scan request to target vCPUs")
>>>>>
>>>> There is sort of a debate about this whether the Fixes tag should be
>>>> used if it's only a cleanup.
>>>>
>>> I have to admit I'm involved in doing backporting sometimes and I really
>>> appreciate Fixes: tags. Just so you know on which side of the debate I
>>> am :-)
>> But we're not going to backport this hopefully?
>>
> In case we're speaking about stable@ kernels, 7ee30bc132c6 doesn't look
> like a good candidate (to me) but who knows, it may get pulled in
> because of some code dependency or some other 'autosel magic'. And
> that's when 'Fixes:' tags become handy.
Anything I can improve upon? If required I can send fixes on top of it.
For the build error, I didn't trigger it because I didn't compile with
appropriate flags.
I will make a note for myself for next time.
Thanks, Mao for sending the fix.
>
--
Nitesh
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH -next] KVM: x86: remove set but not used variable 'called'
2019-11-22 11:58 ` Nitesh Narayan Lal
@ 2019-11-22 12:25 ` Dan Carpenter
2019-11-22 12:45 ` Nitesh Narayan Lal
0 siblings, 1 reply; 14+ messages in thread
From: Dan Carpenter @ 2019-11-22 12:25 UTC (permalink / raw)
To: Nitesh Narayan Lal
Cc: Vitaly Kuznetsov, Mao Wenan, pbonzini, rkrcmar,
sean.j.christopherson, wanpengli, jmattson, joro, tglx, mingo,
bp, hpa, kvm, linux-kernel, kernel-janitors, Marcelo Tosatti
On Fri, Nov 22, 2019 at 06:58:51AM -0500, Nitesh Narayan Lal wrote:
> For the build error, I didn't trigger it because I didn't compile with
> appropriate flags.
It's going to be a serveral years before we can enable that flag by
default.
regards,
dan carpenter
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH -next] KVM: x86: remove set but not used variable 'called'
2019-11-22 12:25 ` Dan Carpenter
@ 2019-11-22 12:45 ` Nitesh Narayan Lal
0 siblings, 0 replies; 14+ messages in thread
From: Nitesh Narayan Lal @ 2019-11-22 12:45 UTC (permalink / raw)
To: Dan Carpenter
Cc: Vitaly Kuznetsov, Mao Wenan, pbonzini, rkrcmar,
sean.j.christopherson, wanpengli, jmattson, joro, tglx, mingo,
bp, hpa, kvm, linux-kernel, kernel-janitors, Marcelo Tosatti
[-- Attachment #1.1: Type: text/plain, Size: 436 bytes --]
On 11/22/19 7:25 AM, Dan Carpenter wrote:
> On Fri, Nov 22, 2019 at 06:58:51AM -0500, Nitesh Narayan Lal wrote:
>> For the build error, I didn't trigger it because I didn't compile with
>> appropriate flags.
> It's going to be a serveral years before we can enable that flag by
> default.
I see I have made a note of it so that I can do it before sending the patches.
> regards,
> dan carpenter
>
--
Thanks
Nitesh
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2019-11-22 12:45 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-19 3:06 [PATCH -next] KVM: x86: remove set but not used variable 'called' Mao Wenan
2019-11-19 11:58 ` Vitaly Kuznetsov
2019-11-19 12:14 ` Dan Carpenter
2019-11-19 12:28 ` Vitaly Kuznetsov
2019-11-19 12:39 ` Dan Carpenter
2019-11-19 13:25 ` Vitaly Kuznetsov
2019-11-22 11:58 ` Nitesh Narayan Lal
2019-11-22 12:25 ` Dan Carpenter
2019-11-22 12:45 ` Nitesh Narayan Lal
2019-11-21 9:13 ` Paolo Bonzini
2019-11-22 0:48 ` maowenan
2019-11-22 2:39 ` [PATCH -next v2] " Mao Wenan
2019-11-19 12:42 ` [PATCH -next] " maowenan
2019-11-19 13:27 ` Vitaly Kuznetsov
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).