On 01/27/2014 02:46 AM, Topi Pohjolainen wrote:
> This is really not needed as blorp blit programs already sample
> XRGB normally and get alpha channel set to 1.0 automatically by
> the sampler engine. This is simply copied directly to the payload
> of the render target write message and hence there is no need for
> any additional blending support from the pixel processing pipeline.
> 
> Fixes recently modified piglit test gl-3.2-layered-rendering-blit
> on IVB. No regressions on IVB.
> 
> This is effectively revert of c0554141a9b831b4e614747104dcbbe0fe489b9d:
> 
>     i965/blorp: Support overriding destination alpha to 1.0.
> 
>     Currently, Blorp requires the source and destination formats to be
>     equal.  However, we'd really like to be able to blit between XRGB and
>     ARGB formats; our BLT engine paths have supported this for a long time.
> 
>     For ARGB -> XRGB, nothing needs to occur: the missing alpha is already
>     interpreted as 1.0.  For XRGB -> ARGB, we need to smash the alpha
>     channel to 1.0 when writing the destination colors.  This is fairly
>     straightforward with blending.
> 
>     For now, this code is never used, as the source and destination formats
>     still must be equal.  The next patch will relax that restriction.
> 
>     NOTE: This is a candidate for the 9.1 branch.
> 
> CC: Ian Romanick <ian.d.roman...@intel.com>
> CC: Kenneth Graunke <kenn...@whitecape.org>
> CC: Martin Steigerwald <mar...@lichtvoll.de>
> Signed-off-by: Topi Pohjolainen <topi.pohjolai...@intel.com>

I'm still not sure why deleting this would fix your test, but it does
and this code is definitely not necessary to make ARGB -> XRGB blits work.

Cc: mesa-sta...@lists.freedesktop.org (perhaps?)
Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>

Attachment: signature.asc
Description: OpenPGP digital signature

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

Reply via email to