Re: [PATCH v9 12/18] vfio: Register/unregister irq_bypass_producer

2015-09-22 Thread Eric Auger
aolo Bonzini; Wu, Feng; alex.william...@redhat.com; j...@8bytes.org; >> mtosa...@redhat.com >> Cc: kvm@vger.kernel.org; io...@lists.linux-foundation.org; >> linux-ker...@vger.kernel.org >> Subject: Re: [PATCH v9 12/18] vfio: Register/unregister irq_bypass_producer >> >>

RE: [PATCH v9 12/18] vfio: Register/unregister irq_bypass_producer

2015-09-22 Thread Wu, Feng
dhat.com > Cc: kvm@vger.kernel.org; io...@lists.linux-foundation.org; > linux-ker...@vger.kernel.org > Subject: Re: [PATCH v9 12/18] vfio: Register/unregister irq_bypass_producer > > Hi, > On 09/21/2015 03:02 PM, Paolo Bonzini wrote: > > > > > > On 21/09/2015 14:53,

RE: [PATCH v9 12/18] vfio: Register/unregister irq_bypass_producer

2015-09-21 Thread Wu, Feng
.linux-foundation.org; linux-ker...@vger.kernel.org > Subject: Re: [PATCH v9 12/18] vfio: Register/unregister irq_bypass_producer > > > > On 21/09/2015 10:56, Wu, Feng wrote: > > Hi Paolo & Alex, > > > > I find that there is a build error in the followin

RE: [PATCH v9 12/18] vfio: Register/unregister irq_bypass_producer

2015-09-21 Thread Wu, Feng
.linux-foundation.org; linux-ker...@vger.kernel.org > Subject: Re: [PATCH v9 12/18] vfio: Register/unregister irq_bypass_producer > > > > On 21/09/2015 10:56, Wu, Feng wrote: > > Hi Paolo & Alex, > > > > I find that there is a build error in the followin

Re: [PATCH v9 12/18] vfio: Register/unregister irq_bypass_producer

2015-09-21 Thread Paolo Bonzini
On 21/09/2015 14:53, Wu, Feng wrote: >>> > > I think the point is that we cannot trigger the build of irqbypass >>> > > manager inside KVM or VFIO, we need trigger the build at a high >>> > > level and it should be built before VFIO and KVM. Any ideas? >> > >> > We can add virt/Makefile and

Re: [PATCH v9 12/18] vfio: Register/unregister irq_bypass_producer

2015-09-21 Thread Paolo Bonzini
On 21/09/2015 13:35, Wu, Feng wrote: >>> > > I think the point is that we cannot trigger the build of irqbypass >>> > > manager inside KVM or VFIO, we need trigger the build at a high >>> > > level and it should be built before VFIO and KVM. Any ideas? >> > >> > We can add virt/Makefile and

RE: [PATCH v9 12/18] vfio: Register/unregister irq_bypass_producer

2015-09-21 Thread Wu, Feng
foundation.org; linux-ker...@vger.kernel.org > Subject: Re: [PATCH v9 12/18] vfio: Register/unregister irq_bypass_producer > > > > On 21/09/2015 13:35, Wu, Feng wrote: > >>> > > I think the point is that we cannot trigger the build of irqbypass > >>> &

Re: [PATCH v9 12/18] vfio: Register/unregister irq_bypass_producer

2015-09-21 Thread Paolo Bonzini
On 21/09/2015 10:56, Wu, Feng wrote: > Hi Paolo & Alex, > > I find that there is a build error in the following two cases: > - KVM is configured as 'M' and VFIO as 'Y' > The reason is the build of irqbypass manager is triggered in > arch/x86/kvm/Makefile, and VFIO is built before KVM, hence >

Re: [PATCH v9 12/18] vfio: Register/unregister irq_bypass_producer

2015-09-21 Thread Eric Auger
Hi, On 09/21/2015 03:02 PM, Paolo Bonzini wrote: > > > On 21/09/2015 14:53, Wu, Feng wrote: >> I think the point is that we cannot trigger the build of irqbypass >> manager inside KVM or VFIO, we need trigger the build at a high >> level and it should be built before VFIO and KVM.

RE: [PATCH v9 12/18] vfio: Register/unregister irq_bypass_producer

2015-09-21 Thread Wu, Feng
Hi Paolo & Alex, I find that there is a build error in the following two cases: - KVM is configured as 'M' and VFIO as 'Y' The reason is the build of irqbypass manager is triggered in arch/x86/kvm/Makefile, and VFIO is built before KVM, hence it cannot find the symbols in irqbypass manager. -

Re: [PATCH v9 12/18] vfio: Register/unregister irq_bypass_producer

2015-09-18 Thread Alex Williamson
On Fri, 2015-09-18 at 22:29 +0800, Feng Wu wrote: > This patch adds the registration/unregistration of an > irq_bypass_producer for MSI/MSIx on vfio pci devices. > > Signed-off-by: Feng Wu On nit, Paolo could you please fix the spelling of "registration" in the dev_info,