Re: [Qemu-devel] [PATCH] i386: pci-assign: Fix MSI-X table size

2016-06-20 Thread Michael S. Tsirkin
On Sat, Jun 18, 2016 at 04:42:05PM -0400, Ido Yariv wrote: > The current code creates a whole page mmio region for the MSI-X table > size. > > However, the page containing the MSI-X table may contain other registers > not related to MSI-X. Creating an mmio region for the whole page masks > such re

Re: [Qemu-devel] [PATCH] i386: pci-assign: Fix MSI-X table size

2016-06-20 Thread Alex Williamson
On Mon, 20 Jun 2016 11:40:17 -0400 Ido Yariv wrote: > Hi Paolo, > > On Mon, Jun 20, 2016 at 04:54:17PM +0200, Paolo Bonzini wrote: > > On 18/06/2016 22:42, Ido Yariv wrote: > > > The current code creates a whole page mmio region for the MSI-X table > > > size. > > > > > > However, the page co

Re: [Qemu-devel] [PATCH] i386: pci-assign: Fix MSI-X table size

2016-06-20 Thread Ido Yariv
Hi Paolo, On Mon, Jun 20, 2016 at 04:54:17PM +0200, Paolo Bonzini wrote: > On 18/06/2016 22:42, Ido Yariv wrote: > > The current code creates a whole page mmio region for the MSI-X table > > size. > > > > However, the page containing the MSI-X table may contain other registers > > not related to

Re: [Qemu-devel] [PATCH] i386: pci-assign: Fix MSI-X table size

2016-06-20 Thread Paolo Bonzini
On 18/06/2016 22:42, Ido Yariv wrote: > The current code creates a whole page mmio region for the MSI-X table > size. > > However, the page containing the MSI-X table may contain other registers > not related to MSI-X. Creating an mmio region for the whole page masks > such registers and may bre

[Qemu-devel] [PATCH] i386: pci-assign: Fix MSI-X table size

2016-06-18 Thread Ido Yariv
The current code creates a whole page mmio region for the MSI-X table size. However, the page containing the MSI-X table may contain other registers not related to MSI-X. Creating an mmio region for the whole page masks such registers and may break drivers in the guest OS. Since maximal number of