<div dir="ltr"><p dir="ltr">> > --- /dev/null<br>
> > +++ b/include/uapi/drm/virtgpu_drm.h<br>
> > @@ -0,0 +1,163 @@<br>
><br>
> > +<br>
> > +struct drm_virtgpu_3d_box {<br>
> > +       uint32_t x, y, z;<br>
> > +       uint32_t w, h, d;<br>
> > +};<br>
> > +<br>
> There was a similar case (multiple variables declared on a single<br>
> line) in drm core that caused confusion and we broke the 32bit compat.<br>
> I thought I mention it - not advocating for/against the above declaration.</p><p dir="ltr"><br></p><p>probably should put some newlines alright,</p><p>><br></p><p dir="ltr">
> > +struct drm_virtgpu_3d_transfer_to_host {<br>
> > +       uint32_t bo_handle;<br>
> > +       struct drm_virtgpu_3d_box box;<br>
> > +       uint32_t level;<br>
> > +       uint32_t offset;<br>
> > +};<br>
> > +<br>
> > +struct drm_virtgpu_3d_transfer_from_host {<br>
> > +       uint32_t bo_handle;<br>
> > +       struct drm_virtgpu_3d_box box;<br>
> > +       uint32_t level;<br>
> > +       uint32_t offset;<br>
> > +};<br>
> > +<br>
> Afaics these seems to be used by the ioctls. If so the current<br>
> declarations are not 32bit compat safe. Things will also go badly if<br>
> you consider expanding struct drm_virtgpu_3d_box in the distant<br>
> future. A u32 pad after bo_handle and a 'pointer' to struct<br>
</p><p dir="ltr">I'm curious what you think we can add to a 3d box.</p><p>So this one is fine, no need to align it either I don't think, though I should confirm that.</p><p>Maybe for safety we can pull the other uint32_t above the box.</p><p>Dave.<br></p></div>