RE: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation

2017-11-07 Thread Zhang, Tina
.@ffwll.ch>; intel-...@lists.freedesktop.org; > intel-gvt-...@lists.freedesktop.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation > > Hi, > > > > Add a head field here?  People asked @ kvm forum about multihead > > > suppo

RE: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation

2017-11-07 Thread Zhang, Tina
.intel.com; > zhen...@linux.intel.com; Lv, Zhiyuan ; Wang, Zhi A > ; Tian, Kevin ; dan...@ffwll.ch; > kwankh...@nvidia.com > Cc: Daniel Vetter ; intel-...@lists.freedesktop.org; > intel-gvt-...@lists.freedesktop.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH v16 5/6] vfio: AB

Re: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation

2017-11-07 Thread Gerd Hoffmann
Hi, > > Add a head field here?  People asked @ kvm forum about multihead > > support. > > Even if the initial driver version doesn't support it we could add > > a field so it > > becomes easier to add it at some point in the future. > > > > Probing for available heads could be done with the

Re: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation

2017-11-07 Thread Gerd Hoffmann
Hi, > > Add a head field here?  People asked @ kvm forum about multihead > > support. > > Even if the initial driver version doesn't support it we could add > > a field so it > > becomes easier to add it at some point in the future. > > > > Probing for available heads could be done with the

RE: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation

2017-11-06 Thread Zhang, Tina
.@ffwll.ch>; intel-...@lists.freedesktop.org; > intel-gvt-...@lists.freedesktop.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation > > Hi, > > > +/** > > + * VFIO_DEVICE_QUERY_GFX_PLANE - _IOW(VFIO_TYPE, VFIO_BASE

RE: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation

2017-11-06 Thread Zhang, Tina
.intel.com; > zhen...@linux.intel.com; Lv, Zhiyuan ; Wang, Zhi A > ; Tian, Kevin ; dan...@ffwll.ch; > kwankh...@nvidia.com > Cc: Daniel Vetter ; intel-...@lists.freedesktop.org; > intel-gvt-...@lists.freedesktop.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH v16 5/6] vfio: AB

RE: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation

2017-11-06 Thread Zhang, Tina
...@lists.freedesktop.org; Wang, Zhi A <zhi.a.w...@intel.com> > Subject: Re: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation > > On Mon, 06 Nov 2017 10:05:34 +0100 > Gerd Hoffmann <kra...@redhat.com> wrote: > > > Hi, > > > > > > I thou

RE: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation

2017-11-06 Thread Zhang, Tina
tel.com; linux-kernel@vger.kernel.org; > zhen...@linux.intel.com; ch...@chris-wilson.co.uk; kwankh...@nvidia.com; > Lv, Zhiyuan ; dan...@ffwll.ch; intel-gvt- > d...@lists.freedesktop.org; Wang, Zhi A > Subject: Re: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation > > On Mon, 06

Re: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation

2017-11-06 Thread Alex Williamson
On Mon, 06 Nov 2017 10:05:34 +0100 Gerd Hoffmann wrote: > Hi, > > > > I thought we had agreed to make this behave similar to > > > VFIO_GROUP_GET_DEVICE_FD, the ioctl would take a __u32 dmabuf_id > > > and > > > return the file descriptor as the ioctl return value.  Thanks,

Re: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation

2017-11-06 Thread Alex Williamson
On Mon, 06 Nov 2017 10:05:34 +0100 Gerd Hoffmann wrote: > Hi, > > > > I thought we had agreed to make this behave similar to > > > VFIO_GROUP_GET_DEVICE_FD, the ioctl would take a __u32 dmabuf_id > > > and > > > return the file descriptor as the ioctl return value.  Thanks, > > > > If we

Re: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation

2017-11-06 Thread Gerd Hoffmann
Hi, > > I thought we had agreed to make this behave similar to > > VFIO_GROUP_GET_DEVICE_FD, the ioctl would take a __u32 dmabuf_id > > and > > return the file descriptor as the ioctl return value.  Thanks, > > If we follow VFIO_GROUP_GET_DEVICE_FD, we would lose flags > functionality. > Zhi

Re: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation

2017-11-06 Thread Gerd Hoffmann
Hi, > > I thought we had agreed to make this behave similar to > > VFIO_GROUP_GET_DEVICE_FD, the ioctl would take a __u32 dmabuf_id > > and > > return the file descriptor as the ioctl return value.  Thanks, > > If we follow VFIO_GROUP_GET_DEVICE_FD, we would lose flags > functionality. > Zhi

Re: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation

2017-11-06 Thread Gerd Hoffmann
Hi, > +/** > + * VFIO_DEVICE_QUERY_GFX_PLANE - _IOW(VFIO_TYPE, VFIO_BASE + 14, > + *struct > vfio_device_query_gfx_plane) > + * > + * Set the drm_plane_type and flags, then retrieve the gfx plane > info. > + * > + * flags supported: > + * -

Re: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation

2017-11-06 Thread Gerd Hoffmann
Hi, > +/** > + * VFIO_DEVICE_QUERY_GFX_PLANE - _IOW(VFIO_TYPE, VFIO_BASE + 14, > + *struct > vfio_device_query_gfx_plane) > + * > + * Set the drm_plane_type and flags, then retrieve the gfx plane > info. > + * > + * flags supported: > + * -

RE: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation

2017-11-05 Thread Zhang, Tina
intel-gvt-...@lists.freedesktop.org; Wang, Zhi A <zhi.a.w...@intel.com>; > kra...@redhat.com > Subject: Re: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation > > On Mon, 6 Nov 2017 10:19:17 +0800 > Tina Zhang <tina.zh...@intel.com> wrote: > > > Add VF

RE: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation

2017-11-05 Thread Zhang, Tina
esktop.org; joonas.lahti...@linux.intel.com; linux- > ker...@vger.kernel.org; zhen...@linux.intel.com; ch...@chris-wilson.co.uk; > kwankh...@nvidia.com; Lv, Zhiyuan ; dan...@ffwll.ch; > intel-gvt-...@lists.freedesktop.org; Wang, Zhi A ; > kra...@redhat.com > Subject: Re: [PATCH v16 5/6] vfio:

Re: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation

2017-11-05 Thread Alex Williamson
On Mon, 6 Nov 2017 10:19:17 +0800 Tina Zhang wrote: > Add VFIO_DEVICE_QUERY_GFX_PLANE ioctl command to let user query and get > a plane and its related information. So far, two types of buffers are > supported: buffers based on dma-buf and buffers based on region. > >

Re: [PATCH v16 5/6] vfio: ABI for mdev display dma-buf operation

2017-11-05 Thread Alex Williamson
On Mon, 6 Nov 2017 10:19:17 +0800 Tina Zhang wrote: > Add VFIO_DEVICE_QUERY_GFX_PLANE ioctl command to let user query and get > a plane and its related information. So far, two types of buffers are > supported: buffers based on dma-buf and buffers based on region. > > This ioctl can be invoked