[Mesa-dev] [PATCH] r600g: disable GPUVM by default

[Christian König]

Am 09.08.2013 03:15, schrieb Alex Deucher:
> Cayman and trinity systems still seem to suffer from
> stability problems with GPUVM.  This also fixes compute
> on these asics.  It can still be enabled for testing
> by setting env var RADEON_VA=true.
>
> Fixes:
> https://bugs.freedesktop.org/show_bug.cgi?id=65958
>
> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>
> CC: "9.2" <mesa-stable at lists.freedesktop.org>
> CC: "9.1" <mesa-stable at lists.freedesktop.org>

Sounds like a good idea to me.

Reviewed-by: Christian König <christian.koenig at amd.com>

> ---
>   src/gallium/winsys/radeon/drm/radeon_drm_winsys.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/winsys/radeon/drm/radeon_drm_winsys.c b/src/gallium/winsys/radeon/drm/radeon_drm_winsys.c
> index 033e78f..69c42a0 100644
> --- a/src/gallium/winsys/radeon/drm/radeon_drm_winsys.c
> +++ b/src/gallium/winsys/radeon/drm/radeon_drm_winsys.c
> @@ -404,7 +404,7 @@ static boolean do_winsys_init(struct radeon_drm_winsys *ws)
>                                         &ws->info.r600_ib_vm_max_size))
>                   ws->info.r600_virtual_address = FALSE;
>           }
> -	if (ws->gen == DRV_R600 && !debug_get_bool_option("RADEON_VA", TRUE))
> +	if (ws->gen == DRV_R600 && !debug_get_bool_option("RADEON_VA", FALSE))
>   		ws->info.r600_virtual_address = FALSE;
>       }
>