diff options
author | Jiri Slaby <jslaby@suse.cz> | 2016-10-13 17:45:20 +0200 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-07-27 15:03:23 -0700 |
commit | d1ef14e5db17cd0642e3e9621182e206cdbceca5 (patch) | |
tree | 8bede2fe7a76a03a86af16145478c940fa862333 | |
parent | 6d098f86e28d282ab21735381227f0c7e774400d (diff) | |
download | linux-stable-d1ef14e5db17cd0642e3e9621182e206cdbceca5.tar.gz linux-stable-d1ef14e5db17cd0642e3e9621182e206cdbceca5.tar.bz2 linux-stable-d1ef14e5db17cd0642e3e9621182e206cdbceca5.zip |
kvm: x86: memset whole irq_eoi
commit 8678654e3c7ad7b0f4beb03fa89691279cba71f9 upstream.
gcc 7 warns:
arch/x86/kvm/ioapic.c: In function 'kvm_ioapic_reset':
arch/x86/kvm/ioapic.c:597:2: warning: 'memset' used with length equal to number of elements without multiplication by element size [-Wmemset-elt-size]
And it is right. Memset whole array using sizeof operator.
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: Radim Krčmář <rkrcmar@redhat.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: H. Peter Anvin <hpa@zytor.com>
Cc: x86@kernel.org
Cc: kvm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
[Added x86 subject tag]
Signed-off-by: Radim Krčmář <rkrcmar@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | virt/kvm/ioapic.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/virt/kvm/ioapic.c b/virt/kvm/ioapic.c index 0ba4057d271b..531e01becf40 100644 --- a/virt/kvm/ioapic.c +++ b/virt/kvm/ioapic.c @@ -610,7 +610,7 @@ static void kvm_ioapic_reset(struct kvm_ioapic *ioapic) ioapic->ioregsel = 0; ioapic->irr = 0; ioapic->id = 0; - memset(ioapic->irq_eoi, 0x00, IOAPIC_NUM_PINS); + memset(ioapic->irq_eoi, 0x00, sizeof(ioapic->irq_eoi)); rtc_irq_eoi_tracking_reset(ioapic); update_handled_vectors(ioapic); } |