Re: [PATCH] dev-assignment: handle device with incorrect PCIe Cap structure size

2011-08-01 Thread Marcelo Tosatti
On Tue, Jul 26, 2011 at 06:08:09PM -0400, Donald Dutile wrote: The bcm5761 provides a PCIe Cap structure (capid=0x10) that is invalid, providing one that is 8 bytes shorter than the v2 PCIe spec defines. This leads to a memory corruption when mapped for device-assigment. Add a check in

Re: [PATCH] dev-assignment: handle device with incorrect PCIe Cap structure size

2011-08-01 Thread Avi Kivity
On 08/01/2011 06:50 PM, Marcelo Tosatti wrote: On Tue, Jul 26, 2011 at 06:08:09PM -0400, Donald Dutile wrote: The bcm5761 provides a PCIe Cap structure (capid=0x10) that is invalid, providing one that is 8 bytes shorter than the v2 PCIe spec defines. This leads to a memory corruption

Re: [PATCH] dev-assignment: handle device with incorrect PCIe Cap structure size

2011-08-01 Thread Marcelo Tosatti
On Mon, Aug 01, 2011 at 06:53:49PM +0300, Avi Kivity wrote: On 08/01/2011 06:50 PM, Marcelo Tosatti wrote: On Tue, Jul 26, 2011 at 06:08:09PM -0400, Donald Dutile wrote: The bcm5761 provides a PCIe Cap structure (capid=0x10) that is invalid, providing one that is 8 bytes shorter than

Re: [PATCH] dev-assignment: handle device with incorrect PCIe Cap structure size

2011-08-01 Thread Avi Kivity
On 08/01/2011 06:56 PM, Marcelo Tosatti wrote: OK. Its not lost: commit f9c29774d2174df6ffc20becec20928948198914 Author: Donald Dutileddut...@redhat.com Date: Tue Jul 26 18:08:09 2011 -0400 device-assignment: handle device with incorrect PCIe Cap structure size Ah, I looked for it in

Re: [PATCH] dev-assignment: handle device with incorrect PCIe Cap structure size

2011-07-27 Thread Michael S. Tsirkin
On Tue, Jul 26, 2011 at 06:08:09PM -0400, Donald Dutile wrote: The bcm5761 provides a PCIe Cap structure (capid=0x10) that is invalid, providing one that is 8 bytes shorter than the v2 PCIe spec defines. This leads to a memory corruption when mapped for device-assigment. Add a check in

Re: [PATCH] dev-assignment: handle device with incorrect PCIe Cap structure size

2011-07-27 Thread Avi Kivity
On 07/27/2011 01:08 AM, Donald Dutile wrote: The bcm5761 provides a PCIe Cap structure (capid=0x10) that is invalid, providing one that is 8 bytes shorter than the v2 PCIe spec defines. This leads to a memory corruption when mapped for device-assigment. Add a check in

[PATCH] dev-assignment: handle device with incorrect PCIe Cap structure size

2011-07-26 Thread Donald Dutile
The bcm5761 provides a PCIe Cap structure (capid=0x10) that is invalid, providing one that is 8 bytes shorter than the v2 PCIe spec defines. This leads to a memory corruption when mapped for device-assigment. Add a check in assigned_device_pci_cap_init() to correct this hw error for this device,

Re: [PATCH] dev-assignment: handle device with incorrect PCIe Cap structure size

2011-07-26 Thread Alex Williamson
On Tue, 2011-07-26 at 18:08 -0400, Donald Dutile wrote: The bcm5761 provides a PCIe Cap structure (capid=0x10) that is invalid, providing one that is 8 bytes shorter than the v2 PCIe spec defines. This leads to a memory corruption when mapped for device-assigment. Add a check in