Hi Feng, On 09/22/2015 11:01 AM, Wu, Feng wrote: > > >> -----Original Message----- >> From: linux-kernel-owner@xxxxxxxxxxxxxxx >> [mailto:linux-kernel-owner@xxxxxxxxxxxxxxx] On Behalf Of Eric Auger >> Sent: Tuesday, September 22, 2015 3:46 AM >> To: Paolo Bonzini; Wu, Feng; alex.williamson@xxxxxxxxxx; joro@xxxxxxxxxx; >> mtosatti@xxxxxxxxxx >> Cc: kvm@xxxxxxxxxxxxxxx; iommu@xxxxxxxxxxxxxxxxxxxxxxxxxx; >> linux-kernel@xxxxxxxxxxxxxxx >> 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, 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 build virt/lib/ directly, not through >>>>>> arch/x86/kvm. >>>> Thinking about this more, does that mean we need to add the virt directory >>>> in the top Makefile in Linux tree? >>> >>> Yes, it does. >> So I understand this will replace patches 2 & 3 then and will fix the >> arm64 issue then. > > I just sent a patch to fix this build error. BTW, from the reply of Paolo, seems > he dropped patch 3 in this series, maybe he think it doesn't have much > relationship with other patches, so maybe you could include it in your series > when forwarded irq work gets ready. OK no problem. I moved that patch (modified according to the new compilation scheme) in the irq forwarding series. Thanks Eric > > Thanks, > Feng > >> >> Thanks >> >> Eric >> >>> >>> Paolo >>> >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in >> the body of a message to majordomo@xxxxxxxxxxxxxxx >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> Please read the FAQ at http://www.tux.org/lkml/ -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
