[Intel-gfx] [PATCH 2/4] drm: Create a format/modifier blob
Ben Widawsky
ben at bwidawsk.net
Fri Jul 14 18:41:49 UTC 2017
On 17-06-29 22:49:44, Ville Syrjälä wrote:
[snip]
>
>... but here it's ALIGN(formats_offset+formats_size). I think we should
>be aligning the same thing in both cases, or we add a BUILD_BUG_ON to
>make sure the header size always stays a multiple of 8 bytes.
>
>That's mainly because the design of the structure seems geared towards
>expanding the header in the future (as in why else would you have the
>offsets?).
>
I guess I don't quite understand what you're asking for. The first thing is
determining a size, the second is finding an offset into the blob.
I don't mind changing this, but tell me what you want.
BUILD_BUG_ON sounds good to me regardless.
[snip]
--
Ben Widawsky, Intel Open Source Technology Center
More information about the Intel-gfx
mailing list