Kenneth Graunke <kenn...@whitecape.org> writes:

> brw_wm_surface_state.c has gotten rather large and unwieldy.  At this
> point, it consists of two separate portions:
>
> 1. Surface format code
>
>    This includes the giant table of surface formats and what features
>    they support on each generation, as well as the code to translate
>    between Mesa formats and hardware formats.
>
>    This is used across all generations.
>
> 2. Binding table (SURFACE_STATE) related code.
>
>    This is the code to generate SURFACE_STATE entries for renderbuffers,
>    textures, transform feedback buffers, constant buffers, and so on, as
>    well as the code to assemble them into binding tables.
>
>    This is only used on Gen4-6; gen7_surface_state.c has Gen7+ code.
>
> Since the two are logically separate, and one is reused on every
> generation while the other is not, it makes a lot of sense to split
> them out.  It should also make finding code easier.

This makes a lot of sense to me.

Reviewed-by: Eric Anholt <e...@anholt.net>

Attachment: pgpyNdq_m3Jps.pgp
Description: PGP signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to