Re: [Qemu-devel] [RFC PATCH] vfio: Add sysfsdev property for pci & platform

2016-02-01 Thread Eric Auger
On 01/26/2016 06:08 PM, Alex Williamson wrote: > On Tue, 2016-01-26 at 16:03 +0100, Eric Auger wrote: >> >> Hi Alex, >> >> I did a try with both legacy cmd line and new one and it works fine for >> vfio platform too: >> -device vfio-calxeda-xgmac,host="fff51000.ethernet" >> -device >> vfio-calxeda-

Re: [Qemu-devel] [RFC PATCH] vfio: Add sysfsdev property for pci & platform

2016-01-26 Thread Alex Williamson
On Tue, 2016-01-26 at 16:03 +0100, Eric Auger wrote: > > Hi Alex, > > I did a try with both legacy cmd line and new one and it works fine for > vfio platform too: > -device vfio-calxeda-xgmac,host="fff51000.ethernet" > -device > vfio-calxeda-xgmac,sysfsdev="/sys/bus/platform/devices/fff51000.ethe

Re: [Qemu-devel] [RFC PATCH] vfio: Add sysfsdev property for pci & platform

2016-01-26 Thread Eric Auger
Hi Alex, I did a try with both legacy cmd line and new one and it works fine for vfio platform too: -device vfio-calxeda-xgmac,host="fff51000.ethernet" -device vfio-calxeda-xgmac,sysfsdev="/sys/bus/platform/devices/fff51000.ethernet" Tested-by: Eric Auger Reviewed-by: Eric Auger just 1 questio

Re: [Qemu-devel] [RFC PATCH] vfio: Add sysfsdev property for pci & platform

2016-01-25 Thread Tian, Kevin
> From: Alex Williamson [mailto:alex.william...@redhat.com] > Sent: Thursday, January 21, 2016 11:15 PM > > On Thu, 2016-01-21 at 07:51 +, Tian, Kevin wrote: > > > From: Alex Williamson [mailto:alex.william...@redhat.com] > > > Sent: Thursday, January 21, 2016 2:07 AM > > > > > > vfio-pci curr

Re: [Qemu-devel] [RFC PATCH] vfio: Add sysfsdev property for pci & platform

2016-01-21 Thread Alex Williamson
On Thu, 2016-01-21 at 07:51 +, Tian, Kevin wrote: > > From: Alex Williamson [mailto:alex.william...@redhat.com] > > Sent: Thursday, January 21, 2016 2:07 AM > > > > vfio-pci currently requires a host= parameter, which comes in the > > form of a PCI address in [domain:] notation.  We > > expect

Re: [Qemu-devel] [RFC PATCH] vfio: Add sysfsdev property for pci & platform

2016-01-20 Thread Tian, Kevin
> From: Alex Williamson [mailto:alex.william...@redhat.com] > Sent: Thursday, January 21, 2016 2:07 AM > > vfio-pci currently requires a host= parameter, which comes in the > form of a PCI address in [domain:] notation. We > expect to find a matching entry in sysfs for that under > /sys/bus/pci/d

Re: [Qemu-devel] [RFC PATCH] vfio: Add sysfsdev property for pci & platform

2016-01-20 Thread P J P
+-- On Wed, 20 Jan 2016, Alex Williamson wrote --+ | > > +error_printf("Warning : Device at %s is known to cause | > > system instability issues during option rom execution. Proceeding | > > anyway since user specified romfile\n", | > > + vdev->vbasedev.name); |

Re: [Qemu-devel] [RFC PATCH] vfio: Add sysfsdev property for pci & platform

2016-01-20 Thread Alex Williamson
On Wed, 2016-01-20 at 18:11 +, Daniel P. Berrange wrote: > On Wed, Jan 20, 2016 at 11:06:55AM -0700, Alex Williamson wrote: > > vfio-pci currently requires a host= parameter, which comes in the > > form of a PCI address in [domain:] notation.  We > > expect to find a matching entry in sysfs for

Re: [Qemu-devel] [RFC PATCH] vfio: Add sysfsdev property for pci & platform

2016-01-20 Thread Daniel P. Berrange
On Wed, Jan 20, 2016 at 11:06:55AM -0700, Alex Williamson wrote: > vfio-pci currently requires a host= parameter, which comes in the > form of a PCI address in [domain:] notation. We > expect to find a matching entry in sysfs for that under > /sys/bus/pci/devices/. vfio-platform takes a similar a

[Qemu-devel] [RFC PATCH] vfio: Add sysfsdev property for pci & platform

2016-01-20 Thread Alex Williamson
vfio-pci currently requires a host= parameter, which comes in the form of a PCI address in [domain:] notation. We expect to find a matching entry in sysfs for that under /sys/bus/pci/devices/. vfio-platform takes a similar approach, but defines the host= parameter to be a string, which can be mat