Module: Mesa Branch: master Commit: e9d37d68cff2ac871d8e32a947173898683c0af3 URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=e9d37d68cff2ac871d8e32a947173898683c0af3
Author: Lucas Stach <[email protected]> Date: Fri Sep 22 11:24:08 2017 +0200 etnaviv: fix 16bpp clears util_pack_color may leave undefined values in the upper half of the packed integer. As our hardware needs the upper 16 bits to mirror the lower 16bits, this breaks clears of those formats if the undefined values aren't masked off. I've only observed the issue with R5G6B5_UNORM surfaces, other 16bpp formats seem to work fine. Fixes: d6aa2ba2b2 (etnaviv: replace translate_clear_color with util_pack_color) Cc: [email protected] Signed-off-by: Lucas Stach <[email protected]> Reviewed-by: Wladimir J. van der Laan <[email protected]> Reviewed-by: Christian Gmeiner <[email protected]> --- src/gallium/drivers/etnaviv/etnaviv_clear_blit.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/etnaviv/etnaviv_clear_blit.c b/src/gallium/drivers/etnaviv/etnaviv_clear_blit.c index 92c9107343..d73d0e30b3 100644 --- a/src/gallium/drivers/etnaviv/etnaviv_clear_blit.c +++ b/src/gallium/drivers/etnaviv/etnaviv_clear_blit.c @@ -106,7 +106,7 @@ pack_rgba(enum pipe_format format, const float *rgba) union util_color uc; util_pack_color(rgba, format, &uc); if (util_format_get_blocksize(format) == 2) - return uc.ui[0] << 16 | uc.ui[0]; + return uc.ui[0] << 16 | (uc.ui[0] & 0xffff); else return uc.ui[0]; } _______________________________________________ mesa-commit mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-commit
