On 5/31/2017 11:48 AM, Chen, Xiaoguang wrote: > Hi, > >> -----Original Message----- >> From: Gerd Hoffmann [mailto:kraxel@xxxxxxxxxx] >> Sent: Monday, May 29, 2017 3:20 PM >> To: Chen, Xiaoguang <xiaoguang.chen@xxxxxxxxx>; >> alex.williamson@xxxxxxxxxx; chris@xxxxxxxxxxxxxxxxxx; intel- >> gfx@xxxxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; >> zhenyuw@xxxxxxxxxxxxxxx; Lv, Zhiyuan <zhiyuan.lv@xxxxxxxxx>; intel-gvt- >> dev@xxxxxxxxxxxxxxxxxxxxx; Wang, Zhi A <zhi.a.wang@xxxxxxxxx>; Tian, Kevin >> <kevin.tian@xxxxxxxxx> >> 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 :) > Can you define what are the expected values of 'flags' would be? Thanks, Kirti > Chenxg. > _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
