[Mesa-dev] [PATCH] st/mesa: fix front buffer regression after dropping st_validate_state in Blit

Ilia Mirkin imirkin at alum.mit.edu
Tue Sep 22 18:00:16 PDT 2015


On Tue, Sep 22, 2015 at 8:40 PM, Marek Olšák <maraeo at gmail.com> wrote:
> From: Marek Olšák <marek.olsak at amd.com>
>
> Broken by: d082c5324914212f76e45be497229c7a0681f706
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=92072
>
> Cc: mesa-stable at lists.freedesktop.org

Tested-by: Ilia Mirkin <imirkin at alum.mit.edu>

Sorry, no idea what this defined business is, don't have time to
investigate it right now. Can you put a Cc: "10.6 11.0" tag on there
in case Emil should make another 10.6 release?

Thanks for tracking this down!

> ---
>  src/mesa/state_tracker/st_cb_blit.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/src/mesa/state_tracker/st_cb_blit.c b/src/mesa/state_tracker/st_cb_blit.c
> index 4fdef7f..59c4dcc 100644
> --- a/src/mesa/state_tracker/st_cb_blit.c
> +++ b/src/mesa/state_tracker/st_cb_blit.c
> @@ -229,6 +229,7 @@ st_BlitFramebuffer(struct gl_context *ctx,
>                    st_adjust_blit_for_msaa_resolve(&blit);
>
>                    st->pipe->blit(st->pipe, &blit);
> +                  dstRb->defined = true;
>                 }
>              }
>           }
> @@ -266,6 +267,7 @@ st_BlitFramebuffer(struct gl_context *ctx,
>                    st_adjust_blit_for_msaa_resolve(&blit);
>
>                    st->pipe->blit(st->pipe, &blit);
> +                  dstRb->defined = true;
>                 }
>              }
>           }
> --
> 2.1.4
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list