[Mesa-dev] [PATCH 1/1] radeonsi: Reorder checks in si_check_render_feedback
Marek Olšák
maraeo at gmail.com
Thu Apr 5 20:56:22 UTC 2018
Even though I've already fixed the crash in master, I'd like to have this
patch in master too.
Reviewed-by: Marek Olšák <marek.olsak at amd.com>
Marek
On Thu, Apr 5, 2018 at 4:49 PM, Jan Vesely <jan.vesely at rutgers.edu> wrote:
> si_get_total_colormask accesses NULL pointer on compute shaders
> Fixes crashes on clover
> Fixes: 0669dca9c00261849cee14d69fdea0a5e323c7f7 ("radeonsi: skip DCC
> render feedback checking if color writes are disabled")
> CC: Marek Olšák <marek.olsak at amd.com>
> Signed-off-by: Jan Vesely <jan.vesely at rutgers.edu>
> ---
> src/gallium/drivers/radeonsi/si_blit.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/gallium/drivers/radeonsi/si_blit.c b/src/gallium/drivers/
> radeonsi/si_blit.c
> index 40f512a682..35506dd74b 100644
> --- a/src/gallium/drivers/radeonsi/si_blit.c
> +++ b/src/gallium/drivers/radeonsi/si_blit.c
> @@ -710,15 +710,15 @@ static void si_check_render_feedback_resident_images(struct
> si_context *sctx)
>
> static void si_check_render_feedback(struct si_context *sctx)
> {
> + if (!sctx->need_check_render_feedback)
> + return;
> +
> /* There is no render feedback if color writes are disabled.
> * (e.g. a pixel shader with image stores)
> */
> if (!si_get_total_colormask(sctx))
> return;
>
> - if (!sctx->need_check_render_feedback)
> - return;
> -
> for (int i = 0; i < SI_NUM_SHADERS; ++i) {
> si_check_render_feedback_images(sctx, &sctx->images[i]);
> si_check_render_feedback_textures(sctx,
> &sctx->samplers[i]);
> --
> 2.14.3
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20180405/d922ca59/attachment.html>
More information about the mesa-dev
mailing list