Re: [Qemu-devel] [PATCH v9 11/14] target/arm/kvm: Translate the MSI doorbell in kvm_arch_fixup_msi_route

2018-03-15 Thread Auger Eric
Hi Paolo, On 13/03/18 14:37, Paolo Bonzini wrote: > On 12/03/2018 16:16, Auger Eric wrote: >>> It is still unclear to me if we need to register an IOMMUNotifier >>> to handle any change in the MSI doorbell which would occur behind >>> the scene and would not lead to any call to kvm_arch_fixup_msi_

Re: [Qemu-devel] [PATCH v9 11/14] target/arm/kvm: Translate the MSI doorbell in kvm_arch_fixup_msi_route

2018-03-13 Thread Paolo Bonzini
On 12/03/2018 16:16, Auger Eric wrote: >> It is still unclear to me if we need to register an IOMMUNotifier >> to handle any change in the MSI doorbell which would occur behind >> the scene and would not lead to any call to kvm_arch_fixup_msi_route(). > > Paolo, do you know the answer to this ques

Re: [Qemu-devel] [PATCH v9 11/14] target/arm/kvm: Translate the MSI doorbell in kvm_arch_fixup_msi_route

2018-03-12 Thread Auger Eric
Hi Peter, On 12/03/18 12:59, Peter Maydell wrote: > On 17 February 2018 at 18:46, Eric Auger wrote: >> In case the MSI is translated by an IOMMU we need to fixup the >> MSI route with the translated address. >> >> Signed-off-by: Eric Auger >> >> --- >> >> v5 -> v6: >> - use IOMMUMemoryRegionClass

Re: [Qemu-devel] [PATCH v9 11/14] target/arm/kvm: Translate the MSI doorbell in kvm_arch_fixup_msi_route

2018-03-12 Thread Peter Maydell
On 17 February 2018 at 18:46, Eric Auger wrote: > In case the MSI is translated by an IOMMU we need to fixup the > MSI route with the translated address. > > Signed-off-by: Eric Auger > > --- > > v5 -> v6: > - use IOMMUMemoryRegionClass API > > It is still unclear to me if we need to register an

[Qemu-devel] [PATCH v9 11/14] target/arm/kvm: Translate the MSI doorbell in kvm_arch_fixup_msi_route

2018-02-17 Thread Eric Auger
In case the MSI is translated by an IOMMU we need to fixup the MSI route with the translated address. Signed-off-by: Eric Auger --- v5 -> v6: - use IOMMUMemoryRegionClass API It is still unclear to me if we need to register an IOMMUNotifier to handle any change in the MSI doorbell which would