Re: [PATCH] kvm: fix warning for CONFIG_HAVE_KVM_EVENTFD builds

2018-02-23 Thread Paolo Bonzini
On 23/02/2018 16:21, Christian Borntraeger wrote: > I can see this warnings as well and yes, kvm_arch_irq_routing_update seems to > be available > for CONFIG_HAVE_KVM_EVENTFD=y and =n. > Seems that irqchip.c is compile independly from CONFIG_HAVE_KVM_EVENTFD, > > so > > Acked-by: Christian

Re: [PATCH] kvm: fix warning for CONFIG_HAVE_KVM_EVENTFD builds

2018-02-23 Thread Paolo Bonzini
On 23/02/2018 16:21, Christian Borntraeger wrote: > I can see this warnings as well and yes, kvm_arch_irq_routing_update seems to > be available > for CONFIG_HAVE_KVM_EVENTFD=y and =n. > Seems that irqchip.c is compile independly from CONFIG_HAVE_KVM_EVENTFD, > > so > > Acked-by: Christian

Re: [PATCH] kvm: fix warning for CONFIG_HAVE_KVM_EVENTFD builds

2018-02-23 Thread Christian Borntraeger
I can see this warnings as well and yes, kvm_arch_irq_routing_update seems to be available for CONFIG_HAVE_KVM_EVENTFD=y and =n. Seems that irqchip.c is compile independly from CONFIG_HAVE_KVM_EVENTFD, so Acked-by: Christian Borntraeger On 02/22/2018 01:05 PM,

Re: [PATCH] kvm: fix warning for CONFIG_HAVE_KVM_EVENTFD builds

2018-02-23 Thread Christian Borntraeger
I can see this warnings as well and yes, kvm_arch_irq_routing_update seems to be available for CONFIG_HAVE_KVM_EVENTFD=y and =n. Seems that irqchip.c is compile independly from CONFIG_HAVE_KVM_EVENTFD, so Acked-by: Christian Borntraeger On 02/22/2018 01:05 PM, Sebastian Ott wrote: > Move the

[PATCH] kvm: fix warning for CONFIG_HAVE_KVM_EVENTFD builds

2018-02-22 Thread Sebastian Ott
Move the kvm_arch_irq_routing_update() prototype outside of ifdef CONFIG_HAVE_KVM_EVENTFD guards to fix the following sparse warning: arch/s390/kvm/../../../virt/kvm/irqchip.c:171:28: warning: symbol 'kvm_arch_irq_routing_update' was not declared. Should it be static? Signed-off-by: Sebastian

[PATCH] kvm: fix warning for CONFIG_HAVE_KVM_EVENTFD builds

2018-02-22 Thread Sebastian Ott
Move the kvm_arch_irq_routing_update() prototype outside of ifdef CONFIG_HAVE_KVM_EVENTFD guards to fix the following sparse warning: arch/s390/kvm/../../../virt/kvm/irqchip.c:171:28: warning: symbol 'kvm_arch_irq_routing_update' was not declared. Should it be static? Signed-off-by: Sebastian