Hi, >-----Original Message----- >From: Gerd Hoffmann [mailto:kra...@redhat.com] >Sent: Monday, May 29, 2017 3:20 PM >To: Chen, Xiaoguang <xiaoguang.c...@intel.com>; >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 <zhiyuan...@intel.com>; intel-gvt- >d...@lists.freedesktop.org; Wang, Zhi A <zhi.a.w...@intel.com>; Tian, Kevin ><kevin.t...@intel.com> >Subject: Re: [PATCH v6 4/6] vfio: Define vfio based vgpu's dma-buf operations > >> +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_plane_info { > __u64 start; > __u64 drm_format_mod; > __u32 drm_format; > __u32 width; > __u32 height; > __u32 stride; > __u32 size; > __u32 x_pos; > __u32 y_pos; > __u32 padding; >}; > >struct vfio_vgpu_query_plane { > __u32 argsz; > __u32 flags; > struct vfio_vgpu_plane_info plane_info; > __u32 plane_id; > __u32 padding; >}; > >struct vfio_vgpu_create_dmabuf { > __u32 argsz; > __u32 flags; > struct vfio_vgpu_plane_info plane_info; > __u32 plane_id; > __s32 fd; >}; Good suggestion will apply in the next version. Thanks for review :)
Chenxg. _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
