[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