Re: [Qemu-devel] [PATCH v13 09/13] add check reset mechanism when hotplug vfio device

2015-11-16 Thread Chen Fan
On 11/17/2015 12:05 AM, Alex Williamson wrote: On Mon, 2015-11-16 at 18:18 +0800, Chen Fan wrote: Hi Alex, Thanks for your detailed explanation. during my test, I found that maybe there was another problem in vfio driver, I use a dual-port NIC which address are: 06:00.0 and 06:00.1 two

Re: [Qemu-devel] [PATCH v13 09/13] add check reset mechanism when hotplug vfio device

2015-11-16 Thread Alex Williamson
On Mon, 2015-11-16 at 18:18 +0800, Chen Fan wrote: > Hi Alex, > >Thanks for your detailed explanation. >during my test, I found that maybe there was another problem in vfio > driver, > I use a dual-port NIC which address are: 06:00.0 and 06:00.1 two functions. > then I use aer-inject to i

Re: [Qemu-devel] [PATCH v13 09/13] add check reset mechanism when hotplug vfio device

2015-11-16 Thread Chen Fan
Hi Alex, Thanks for your detailed explanation. during my test, I found that maybe there was another problem in vfio driver, I use a dual-port NIC which address are: 06:00.0 and 06:00.1 two functions. then I use aer-inject to inject one error to one function like following: AER ID :06:00

Re: [Qemu-devel] [PATCH v13 09/13] add check reset mechanism when hotplug vfio device

2015-11-13 Thread Alex Williamson
On Fri, 2015-11-13 at 11:28 +0800, Cao jin wrote: > > On 11/12/2015 07:51 PM, Michael S. Tsirkin wrote: > > On Wed, Nov 11, 2015 at 06:34:27PM +0800, Cao jin wrote: > >> From: Chen Fan > >> > >> Since we support multi-function hotplug. the function 0 indicate > >> the closure of the slot, so we h

Re: [Qemu-devel] [PATCH v13 09/13] add check reset mechanism when hotplug vfio device

2015-11-12 Thread Cao jin
On 11/12/2015 07:51 PM, Michael S. Tsirkin wrote: On Wed, Nov 11, 2015 at 06:34:27PM +0800, Cao jin wrote: From: Chen Fan Since we support multi-function hotplug. the function 0 indicate the closure of the slot, so we have the chance to do the check. Signed-off-by: Chen Fan --- hw/pci/pc

Re: [Qemu-devel] [PATCH v13 09/13] add check reset mechanism when hotplug vfio device

2015-11-12 Thread Michael S. Tsirkin
On Wed, Nov 11, 2015 at 06:34:27PM +0800, Cao jin wrote: > From: Chen Fan > > Since we support multi-function hotplug. the function 0 indicate > the closure of the slot, so we have the chance to do the check. > > Signed-off-by: Chen Fan > --- > hw/pci/pci.c | 29 +++

[Qemu-devel] [PATCH v13 09/13] add check reset mechanism when hotplug vfio device

2015-11-11 Thread Cao jin
From: Chen Fan Since we support multi-function hotplug. the function 0 indicate the closure of the slot, so we have the chance to do the check. Signed-off-by: Chen Fan --- hw/pci/pci.c | 29 + hw/vfio/pci.c| 19 +++ hw/vfio/pc