[Mesa-dev] [PATCH 1/3] i965: Emit the BLEND_STATE pointer directly rather than via atoms.

[Eric Anholt]

Kenneth Graunke <kenneth at whitecape.org> writes:

> Previously, we would:
> 1. Emit the new indirect state.
> 2. Flag CACHE_NEW_BLEND_STATE.
> 3. Rely on later state atoms to notice CACHE_NEW_BLEND_STATE and emit a
>    pointer to the new indirect state.
>
> This is rather cumbersome: it requires two state atoms instead of one,
> and there's a strict ordering dependency in the list.  Plus, the code
> gets spread across two functions (or even files in the case of Gen7+).
>
> Gen7+ has a packet to update just the blend state pointer, so it makes a
> lot of sense to simply emit that right away.  Gen6 has a combined packet
> which updates blending, the color calculator, and depth/stencil state;
> however, each can still be modified independently.
>
> This drops the Gen6 micro-optimization where we tried to only emit one
> packet that changed all three states.  State updates are pretty cheap.
>
> CACHE_NEW_BLEND_STATE is no longer necessary, so drop it.

Series is:

Reviewed-by: Eric Anholt <eric at anholt.net>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20130611/1b052e21/attachment.pgp>