Re: [Intel-gfx] [PATCH v6 4/6] vfio: Define vfio based vgpu's dma-buf operations

2017-06-06 Thread Gerd Hoffmann
On Mon, 2017-06-05 at 13:56 +0530, Kirti Wankhede wrote: > > On 6/2/2017 2:08 PM, Gerd Hoffmann wrote: > > > > > struct vfio_vgpu_surface_info { > > > __u64 start; > > > __u32 width; > > > __u32 height; > > > __u32 stride; > > > __u32 size; > > > __

Re: [Intel-gfx] [PATCH v6 4/6] vfio: Define vfio based vgpu's dma-buf operations

2017-06-05 Thread Kirti Wankhede
On 6/2/2017 2:08 PM, Gerd Hoffmann wrote: > >> struct vfio_vgpu_surface_info { >> __u64 start; >> __u32 width; >> __u32 height; >> __u32 stride; >> __u32 size; >> __u32 x_pos; >> __u32 y_pos; >> __u32 padding; >> /* Only use

Re: [Intel-gfx] [PATCH v6 4/6] vfio: Define vfio based vgpu's dma-buf operations

2017-06-02 Thread Gerd Hoffmann
> struct vfio_vgpu_surface_info { > __u64 start; > __u32 width; > __u32 height; > __u32 stride; > __u32 size; > __u32 x_pos; > __u32 y_pos; > __u32 padding; > /* Only used when VFIO_VGPU_SURFACE_DMABUF_* flags set */ >    

Re: [Intel-gfx] [PATCH v6 4/6] vfio: Define vfio based vgpu's dma-buf operations

2017-06-01 Thread Kirti Wankhede
On 6/1/2017 10:08 PM, Alex Williamson wrote: > On Thu, 1 Jun 2017 03:01:28 + > "Chen, Xiaoguang" wrote: > >> Hi Kirti, >> >>> -Original Message- >>> From: Kirti Wankhede [mailto:kwankh...@nvidia.com] >>> Sent: Thursday, June 01, 2017 1:23 AM >>> To: Chen, Xiaoguang ; Gerd Hoffmann >

Re: [Intel-gfx] [PATCH v6 4/6] vfio: Define vfio based vgpu's dma-buf operations

2017-06-01 Thread Alex Williamson
On Thu, 1 Jun 2017 03:01:28 + "Chen, Xiaoguang" wrote: > Hi Kirti, > > >-Original Message- > >From: Kirti Wankhede [mailto:kwankh...@nvidia.com] > >Sent: Thursday, June 01, 2017 1:23 AM > >To: Chen, Xiaoguang ; Gerd Hoffmann > >; alex.william...@redhat.com; ch...@chris-wilson.co.uk;

Re: [Intel-gfx] [PATCH v6 4/6] vfio: Define vfio based vgpu's dma-buf operations

2017-05-31 Thread Chen, Xiaoguang
Hi Kirti, >-Original Message- >From: Kirti Wankhede [mailto:kwankh...@nvidia.com] >Sent: Thursday, June 01, 2017 1:23 AM >To: Chen, Xiaoguang ; Gerd Hoffmann >; alex.william...@redhat.com; ch...@chris-wilson.co.uk; >intel-gfx@lists.freedesktop.org; linux-ker...@vger.kernel.org; >zhen...@li

Re: [Intel-gfx] [PATCH v6 4/6] vfio: Define vfio based vgpu's dma-buf operations

2017-05-31 Thread Kirti Wankhede
On 5/31/2017 11:48 AM, Chen, Xiaoguang wrote: > Hi, > >> -Original Message- >> From: Gerd Hoffmann [mailto:kra...@redhat.com] >> Sent: Monday, May 29, 2017 3:20 PM >> To: Chen, Xiaoguang ; >> alex.william...@redhat.com; ch...@chris-wilson.co.uk; intel- >> g...@lists.freedesktop.org; linu

Re: [Intel-gfx] [PATCH v6 4/6] vfio: Define vfio based vgpu's dma-buf operations

2017-05-30 Thread Chen, Xiaoguang
Hi, >-Original Message- >From: Gerd Hoffmann [mailto:kra...@redhat.com] >Sent: Monday, May 29, 2017 3:20 PM >To: Chen, Xiaoguang ; >alex.william...@redhat.com; ch...@chris-wilson.co.uk; intel- >g...@lists.freedesktop.org; linux-ker...@vger.kernel.org; >zhen...@linux.intel.com; Lv, Zhiyuan

Re: [Intel-gfx] [PATCH v6 4/6] vfio: Define vfio based vgpu's dma-buf operations

2017-05-29 Thread Gerd Hoffmann
> +struct vfio_vgpu_dmabuf_info { > + __u32 argsz; > + __u32 flags; > + struct vfio_vgpu_plane_info plane_info; > + __s32 fd; > + __u32 pad; > +}; Hmm, now you have argsz and flags twice in vfio_vgpu_dmabuf_info ... I think we should have something like this: struct vfio_vgpu

[Intel-gfx] [PATCH v6 4/6] vfio: Define vfio based vgpu's dma-buf operations

2017-05-27 Thread Xiaoguang Chen
Here we defined a new ioctl to create a fd for a vfio device based on the input type. Now only one type is supported that is a dma-buf management fd. Two ioctls are defined for the dma-buf management fd: query the vfio vgpu's plane information and create a dma-buf for a plane. Signed-off-by: Xiaog