https://bugs.freedesktop.org/show_bug.cgi?id=110721
Maybe be a little less trigger happy with the tags, can we get these
reverted in stable :-) and master as well maybe.
Dave.
On Tue, 21 May 2019 at 02:32, Charmaine Lee <charmai...@vmware.com> wrote:
>
>
> >From: Brian Paul <bri...@vmware.com>
> >Sent: Monday, May 20, 2019 6:39 AM
> >To: Charmaine Lee; mesa-dev@lists.freedesktop.org; Neha Bhende;
> >marek.ol...@amd.com
> >Subject: Re: [PATCH 2/2] mesa: unreference current winsys buffers when
> >unbinding winsys buffers
>
> >Both look OK to me. Do they need to be tagged with "Cc:
> mesa-sta...@lists.freedesktop.org" for the stable branches?
>
> >Reviewed-by: Brian Paul <bri...@vmware.com>
>
> Yes, I will add the tag. Thanks Brian for reviewing.
>
> -Charmaine
>
>
>
> >On 05/18/2019 07:46 PM, Charmaine Lee wrote:
> >> This fixes surface leak when no winsys buffers are bound.
> >> ---
> >> src/mesa/main/context.c | 4 ++++
> >> 1 file changed, 4 insertions(+)
> >>
> >> diff --git a/src/mesa/main/context.c b/src/mesa/main/context.c
> >> index 34da16b..04ef4d5 100644
> >> --- a/src/mesa/main/context.c
> >> +++ b/src/mesa/main/context.c
> >> @@ -1765,6 +1765,10 @@ _mesa_make_current( struct gl_context *newCtx,
> >>
> >> check_init_viewport(newCtx, drawBuffer->Width,
> >> drawBuffer->Height);
> >> }
> >> + else {
> >> + _mesa_reference_framebuffer(&newCtx->WinSysDrawBuffer, NULL);
> >> + _mesa_reference_framebuffer(&newCtx->WinSysReadBuffer, NULL);
> >> + }
> >>
> >> if (newCtx->FirstTimeCurrent) {
> >> handle_first_current(newCtx);
> >>
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
