Re: [virtio-dev] Re: [PATCH v2 0/6] Extend vhost-user to support VFIO based accelerators

2018-03-28 Thread Michael S. Tsirkin
On Thu, Mar 29, 2018 at 11:33:29AM +0800, Tiwei Bie wrote: > On Wed, Mar 28, 2018 at 06:33:01PM +0300, Michael S. Tsirkin wrote: > > On Wed, Mar 28, 2018 at 08:24:07PM +0800, Tiwei Bie wrote: > > > > > Update notes > > > > > > > > > > > > > > > IOMMU feature bit check is removed in

Re: [virtio-dev] Re: [PATCH v2 0/6] Extend vhost-user to support VFIO based accelerators

2018-03-28 Thread Tiwei Bie
On Wed, Mar 28, 2018 at 06:33:01PM +0300, Michael S. Tsirkin wrote: > On Wed, Mar 28, 2018 at 08:24:07PM +0800, Tiwei Bie wrote: > > > > Update notes > > > > > > > > > > > > IOMMU feature bit check is removed in this version, because: > > > > > > > > The IOMMU feature is negotiable,

[virtio-dev] on spec clarifications

2018-03-28 Thread Michael S. Tsirkin
Halil, in the past you mentioned that you see some parts of the specification as unclear. Will you have the time to look into this and send patches, or even just open github issues? I'd like us to start looking at wrapping up the next spec version. -- MST

Re: [virtio-dev] [pci PATCH v7 1/5] pci: Add pci_sriov_configure_simple for PFs that don't manage VF resources

2018-03-28 Thread Rustad, Mark D
On Mar 15, 2018, at 11:41 AM, Alexander Duyck wrote: > From: Alexander Duyck > > This patch adds a common configuration function called > pci_sriov_configure_simple that will allow for managing VFs on devices > where the PF is not capable

[virtio-dev] Re: [virtio] [PATCH v12] VIRTIO_F_NOTIFICATION_DATA: extra data to devices

2018-03-28 Thread Michael S. Tsirkin
On Wed, Mar 28, 2018 at 06:56:13PM +0200, Halil Pasic wrote: > > > On 03/27/2018 06:37 PM, Michael S. Tsirkin wrote: > > Some devices benefit from ability to find out the number of available > > descriptors in the ring: for efficiency or as a debugging aid. > > > > To help with these

[virtio-dev] Re: [virtio] [PATCH v12] VIRTIO_F_NOTIFICATION_DATA: extra data to devices

2018-03-28 Thread Halil Pasic
On 03/27/2018 06:37 PM, Michael S. Tsirkin wrote: > Some devices benefit from ability to find out the number of available > descriptors in the ring: for efficiency or as a debugging aid. > > To help with these optimizations, add a new feature: > VIRTIO_F_NOTIFICATION_DATA. When negotiated,

RE: [virtio-dev] [PATCH v10 13/13] split-ring: in order feature

2018-03-28 Thread Lars Ganrot
Missed replying to the lists. Sorry. > From: Michael S. Tsirkin > Sent: 28. marts 2018 16:39 > > On Wed, Mar 28, 2018 at 08:23:38AM +, Lars Ganrot wrote: > > Hi Michael et al > > > > > Behalf Of Michael S. Tsirkin > > > Sent: 9. marts 2018 22:24 > > > > > > For a split

[virtio-dev] Re: [PATCH v2 0/6] Extend vhost-user to support VFIO based accelerators

2018-03-28 Thread Michael S. Tsirkin
On Wed, Mar 28, 2018 at 08:24:07PM +0800, Tiwei Bie wrote: > > > Update notes > > > > > > > > > IOMMU feature bit check is removed in this version, because: > > > > > > The IOMMU feature is negotiable, when an accelerator is used and > > > it doesn't support virtual IOMMU, its

Re: [virtio-dev] [PATCH v10 13/13] split-ring: in order feature

2018-03-28 Thread Michael S. Tsirkin
On Wed, Mar 28, 2018 at 08:23:38AM +, Lars Ganrot wrote: > Hi Michael et al > > > Behalf Of Michael S. Tsirkin > > Sent: 9. marts 2018 22:24 > > > > For a split ring, require that drivers use descriptors in order too. > > This allows devices to skip reading the available ring. > > > >

[virtio-dev] Re: [PATCH v2 0/6] Extend vhost-user to support VFIO based accelerators

2018-03-28 Thread Tiwei Bie
On Thu, Mar 22, 2018 at 06:40:18PM +0200, Michael S. Tsirkin wrote: > On Mon, Mar 19, 2018 at 03:15:31PM +0800, Tiwei Bie wrote: [...] > > > > Below link is an example showing how to setup a such environment > > via nested VM. In this case, the virtio device in the outer VM is > > the

RE: [virtio-dev] [PATCH v10 13/13] split-ring: in order feature

2018-03-28 Thread Lars Ganrot
Hi Michael et al > Behalf Of Michael S. Tsirkin > Sent: 9. marts 2018 22:24 > > For a split ring, require that drivers use descriptors in order too. > This allows devices to skip reading the available ring. > > Signed-off-by: Michael S. Tsirkin > Reviewed-by: Cornelia Huck