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

2017-08-22 Thread Gerd Hoffmann
Hi, > These are both from Gerd.  Gerd, do you have any objection to using a > union to provide either the dmabuf fd or region index? No. > > It's like we want to propose a general interface used to share > > guest's buffer with host. And the > > general interface, so far, has two choice: regio

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

2017-08-09 Thread Kirti Wankhede
On 8/8/2017 11:37 PM, Alex Williamson wrote: > On Tue, 8 Aug 2017 14:18:07 +0530 > Kirti Wankhede wrote: > >> On 8/7/2017 11:13 PM, Alex Williamson wrote: >>> On Mon, 7 Aug 2017 08:11:43 + >>> "Zhang, Tina" wrote: >>> After going through the previous discussions, here are some summ

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

2017-08-09 Thread Alex Williamson
Zhang, Tina > > Cc: Tian, Kevin ; intel-...@lists.freedesktop.org; > > dri- > > de...@lists.freedesktop.org; kwankh...@nvidia.com; kra...@redhat.com; > > intel-gvt-...@lists.freedesktop.org; Wang, Zhi A ; Lv, > > Zhiyuan > > Subject: Re: [PATCH v13 5/7] vfio: ABI f

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

2017-08-09 Thread Zhang, Tina
eedesktop.org; kwankh...@nvidia.com; kra...@redhat.com; > intel-gvt-...@lists.freedesktop.org; Wang, Zhi A ; Lv, > Zhiyuan > Subject: Re: [PATCH v13 5/7] vfio: ABI for mdev display dma-buf operation > > On Mon, 7 Aug 2017 08:11:43 + > "Zhang, Tina" wrote: > > &

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

2017-08-08 Thread Kirti Wankhede
On 8/7/2017 11:13 PM, Alex Williamson wrote: > On Mon, 7 Aug 2017 08:11:43 + > "Zhang, Tina" wrote: > >> After going through the previous discussions, here are some summaries may be >> related to the current discussion: >> 1. How does user mode figure the device capabilities between region

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

2017-08-08 Thread Alex Williamson
On Tue, 8 Aug 2017 14:18:07 +0530 Kirti Wankhede wrote: > On 8/7/2017 11:13 PM, Alex Williamson wrote: > > On Mon, 7 Aug 2017 08:11:43 + > > "Zhang, Tina" wrote: > > > >> After going through the previous discussions, here are some summaries may > >> be related to the current discussion:

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

2017-08-07 Thread Alex Williamson
On Mon, 7 Aug 2017 08:11:43 + "Zhang, Tina" wrote: > After going through the previous discussions, here are some summaries may be > related to the current discussion: > 1. How does user mode figure the device capabilities between region and > dma-buf? > VFIO_DEVICE_GET_REGION_INFO could tel

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

2017-08-07 Thread Zhang, Tina
f Alex Williamson > > Sent: Thursday, August 3, 2017 10:43 PM > > To: Zhang, Tina > > Cc: Tian, Kevin ; > > intel-...@lists.freedesktop.org; dri- de...@lists.freedesktop.org; > > kwankh...@nvidia.com; kra...@redhat.com; > > intel-gvt-...@lists.freedesktop.org; Wang

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

2017-08-06 Thread Zhang, Tina
eedesktop.org; kwankh...@nvidia.com; kra...@redhat.com; > intel-gvt-...@lists.freedesktop.org; Wang, Zhi A ; Lv, > Zhiyuan > Subject: Re: [PATCH v13 5/7] vfio: ABI for mdev display dma-buf operation > > On Thu, 3 Aug 2017 07:08:15 + > "Zhang, Tina" wrote: > > >

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

2017-08-03 Thread Alex Williamson
@lists.freedesktop.org; > > dri- > > de...@lists.freedesktop.org; kwankh...@nvidia.com; kra...@redhat.com; > > intel-gvt-...@lists.freedesktop.org; Wang, Zhi A ; Lv, > > Zhiyuan > > Subject: Re: [PATCH v13 5/7] vfio: ABI for mdev display dma-buf operati

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

2017-08-03 Thread Zhang, Tina
t.com; > intel-gvt-...@lists.freedesktop.org; Wang, Zhi A ; Lv, > Zhiyuan > Subject: Re: [PATCH v13 5/7] vfio: ABI for mdev display dma-buf operation > > On Thu, 3 Aug 2017 03:17:09 + > "Zhang, Tina" wrote: > > > > -Original Message- > > >

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

2017-08-02 Thread Alex Williamson
gt; To: Zhang, Tina > > Cc: Tian, Kevin ; kra...@redhat.com; intel- > > g...@lists.freedesktop.org; Wang, Zhi A ; > > kwankh...@nvidia.com; dri-devel@lists.freedesktop.org; intel-gvt- > > d...@lists.freedesktop.org; Lv, Zhiyuan > > Subject: Re: [PATCH v13 5/7] vfio: ABI fo

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

2017-08-02 Thread Zhang, Tina
ang, Zhi A ; > kwankh...@nvidia.com; dri-devel@lists.freedesktop.org; intel-gvt- > d...@lists.freedesktop.org; Lv, Zhiyuan > Subject: Re: [PATCH v13 5/7] vfio: ABI for mdev display dma-buf operation > > On Tue, 25 Jul 2017 17:28:18 +0800 > Tina Zhang wrote: > > > Add VFIO_DEVIC

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

2017-08-02 Thread Kirti Wankhede
On 8/2/2017 2:48 AM, Alex Williamson wrote: > On Tue, 25 Jul 2017 17:28:18 +0800 > Tina Zhang wrote: > >> Add VFIO_DEVICE_QUERY_GFX_PLANE ioctl command to let user mode query and >> get the plan and its related information. >> >> The dma-buf's life cycle is handled by user mode and tracked by k

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

2017-08-01 Thread Alex Williamson
On Tue, 25 Jul 2017 17:28:18 +0800 Tina Zhang wrote: > Add VFIO_DEVICE_QUERY_GFX_PLANE ioctl command to let user mode query and > get the plan and its related information. > > The dma-buf's life cycle is handled by user mode and tracked by kernel. > The returned fd in struct vfio_device_query_gf

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

2017-07-30 Thread Zhang, Tina
esktop.org; dri-devel@lists.freedesktop.org; > ville.syrj...@linux.intel.com; zhen...@linux.intel.com; Lv, Zhiyuan > ; Wang, Zhi A ; > alex.william...@redhat.com; ch...@chris-wilson.co.uk; dan...@ffwll.ch; > kwankh...@nvidia.com; Tian, Kevin > Subject: Re: [PATCH v13 5/7] vfio: ABI for mdev

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

2017-07-28 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 retrieve information about the gfx > plane. + * > + * Return: 0 on success, -errno on failure. I think this should be more verbose, especia