[Mesa-dev] [PATCH 11/15] anv: add new formats KHR_sampler_ycbcr_conversion

[Lionel Landwerlin]

On 15/09/17 23:36, Jason Ekstrand wrote:
>
>      struct anv_format_plane {
>         enum isl_format isl_format:16;
>         struct isl_swizzle swizzle;
>     +
>     +   /* Whether this plane contains chroma channels */
>     +   bool has_chroma;
>     +
>     +   /* For downscaling of YUV planes */
>     +   uint8_t denominator_scales[2];
>     +
>     +   /* How to map sampled ycbcr planes to a single 4 component
>     element. */
>     +   struct isl_swizzle ycbcr_swizzle;
>
>
> I'm getting a bit concerned about how bloated the format table is 
> getting.  The old table had 184 entries each of which was 32 bits.  
> Now we're looking at more like 32 bytes per entry and most of it isn't 
> used for your average format.  It probably isn't a huge deal but it 
> does seem a bit excessive.  I'm not quite sure what to do about it though.

Maybe some kind of linked list in the style of pNext in vulkan?

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170917/20ec0c5f/attachment.html>