[PATCH] drm/radeon: avoid UVD corruptions on AGP cards

[Alex Deucher]

On Sun, Sep 15, 2013 at 7:31 AM, Christian König
<deathsimple at vodafone.de> wrote:
> From: Christian König <christian.koenig at amd.com>
>
> Putting everything into VRAM seems to help.
>
> Signed-off-by: Christian König <christian.koenig at amd.com>

Applied.  thanks!

Alex

> ---
>  drivers/gpu/drm/radeon/radeon_cs.c | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_cs.c b/drivers/gpu/drm/radeon/radeon_cs.c
> index 27ea004..ac6ece6 100644
> --- a/drivers/gpu/drm/radeon/radeon_cs.c
> +++ b/drivers/gpu/drm/radeon/radeon_cs.c
> @@ -81,9 +81,11 @@ static int radeon_cs_parser_relocs(struct radeon_cs_parser *p)
>                 p->relocs[i].lobj.bo = p->relocs[i].robj;
>                 p->relocs[i].lobj.written = !!r->write_domain;
>
> -               /* the first reloc of an UVD job is the
> -                  msg and that must be in VRAM */
> -               if (p->ring == R600_RING_TYPE_UVD_INDEX && i == 0) {
> +               /* the first reloc of an UVD job is the msg and that must be in
> +                  VRAM, also but everything into VRAM on AGP cards to avoid
> +                  image corruptions */
> +               if (p->ring == R600_RING_TYPE_UVD_INDEX &&
> +                   (i == 0 || p->rdev->flags & RADEON_IS_AGP)) {
>                         /* TODO: is this still needed for NI+ ? */
>                         p->relocs[i].lobj.domain =
>                                 RADEON_GEM_DOMAIN_VRAM;
> --
> 1.8.1.2
>
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel