On Wed, 09 Jul 2025 11:04:08 +0100, Matt Coster wrote:
> These fixes were made as part of investigating the Sparse issue reported
> in [1]; specifically that the sizeof() check for the flexible structure
> struct rogue_fwif_frag_ctx_state was failing under Sparse.
>
> This is actually a fairly pointless check, and the discrepancy is
> related to the implicit padding at the end of the structure that is
> never used due to the flexible array member. A far more useful check is
> to assert that the final member is indeed a flexible array as expected,
> and that one element of said array is of the size that we expect.
>
> [...]
Applied, thanks!
[1/2] drm/imagination: Add and use FLEX_ARRAY_CHECK()
commit: f41ef8e3c65f1c424004493f7792ce06daa147df
[2/2] drm/imagination: Use struct_size_t()
commit: b67ac31d7f42b2b2e9ca77c66bbbb3646f039103
Best regards,
--
Alessio Belle <[email protected]>
