[Mesa-dev] [PATCH] winsys/radeon: add env var to disable VM on Cayman/Trinity

[Martin Andersson]

On Mon, Jun 10, 2013 at 11:55 PM, Alex Deucher <alexdeucher at gmail.com> wrote:
> On Mon, Jun 10, 2013 at 5:36 PM, Marek Olšák <maraeo at gmail.com> wrote:
>> Reviewed-by: Marek Olšák <maraeo at gmail.com>
>>
>> FYI, Cayman works for me as I said in
>> https://bugs.freedesktop.org/show_bug.cgi?id=62959 . I don't remember
>> if the issue has been fixed upstream.
>
> Yes, the fixes should be upstream, however, Tom has had some strange
> behavior with compute using VM and there are a number of strange bugs
> on TN and cayman that may still be VM related.

I have a bunch of regressions on a 6950 with va enabled, no hardlocks though.

Some of test results are random, one example is
all-colors.shader_test. It consists of a bunch of tests like this and
it fails at different pixels each time I run it:

clear color 0.0 0.0 0.0 0.0
clear
uniform vec4 color 0.0 0.0 0.0 0.0
draw arrays GL_LINES 0 2
probe all rgba 0.0 0.0 0.0 0.0

If i remove the draw arrays line the test always passes, I have spent
some time investigating it, but haven't found anything yet.

//Martin

> Alex
>
>>
>> Marek
>>
>> On Mon, Jun 10, 2013 at 10:34 PM,  <alexdeucher at gmail.com> wrote:
>>> From: Alex Deucher <alexander.deucher at amd.com>
>>>
>>> Set env var RADEON_VA=0 to disable VM on Cayman/Trinity.
>>> Useful for debugging.
>>>
>>> Note: this is a candidate for the 9.1 branch.
>>>
>>> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>
>>> ---
>>>  src/gallium/winsys/radeon/drm/radeon_drm_winsys.c |    2 ++
>>>  1 files changed, 2 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/src/gallium/winsys/radeon/drm/radeon_drm_winsys.c b/src/gallium/winsys/radeon/drm/radeon_drm_winsys.c
>>> index 15d5d31..ee4dfa1 100644
>>> --- a/src/gallium/winsys/radeon/drm/radeon_drm_winsys.c
>>> +++ b/src/gallium/winsys/radeon/drm/radeon_drm_winsys.c
>>> @@ -399,6 +399,8 @@ 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))
>>> +               ws->info.r600_virtual_address = FALSE;
>>>      }
>>>
>>>      /* Get max pipes, this is only needed for compute shaders.  All evergreen+
>>> --
>>> 1.7.7.5
>>>
>>> _______________________________________________
>>> mesa-dev mailing list
>>> mesa-dev at lists.freedesktop.org
>>> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev