[Glamor] [PATCH] xv: workaround corruption

[Alex Deucher]

On Fri, Jun 6, 2014 at 5:51 AM, Grigori Goronzy <greg at chown.ath.cx> wrote:
> Keeping the video pixmaps persistent sometimes will lead to video
> corruption (shifted chroma planes) for unknown reasons. Possibly
> there are some subtle pixmap handling bugs in xv or other parts of
> the glamor code. To workaround, simply destroy pixmaps after they
> have been used once.
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=72821

Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

> ---
>  src/glamor_xv.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/src/glamor_xv.c b/src/glamor_xv.c
> index a89b4cd..604a4d7 100644
> --- a/src/glamor_xv.c
> +++ b/src/glamor_xv.c
> @@ -559,6 +559,7 @@ static int glamor_xv_put_image(ScrnInfoPtr pScrn,
>         port_priv->h = height;
>         port_priv->pDraw = pDrawable;
>         glamor_display_textured_video(port_priv);
> +       glamor_xv_stop_video(pScrn, port_priv, TRUE);
>         return Success;
>  }
>
> --
> 1.8.3.2
>
> _______________________________________________
> Glamor mailing list
> Glamor at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/glamor