> +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; }; _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx