drm/amdgpu: default noretry=0 for navi1x and newer (v2)
authorAlex Deucher <alexander.deucher@amd.com>
Mon, 30 Nov 2020 18:55:45 +0000 (13:55 -0500)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 1 Dec 2020 21:04:17 +0000 (16:04 -0500)
There are no performance advantages to setting it to 1 and
it causes stability issues in some cases.

v2: simplify the code

Bug: https://gitlab.freedesktop.org/drm/amd/-/issues/1374
Reviewed-by: Felix Kuehling <Felix.Kuehling@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_gmc.c

index bb66c534a31b0bfe1a06ef17f9b5f7a76d50510c..ff3afc82e73a3905e2bfbe92cbd3c66707bf7ade 100644 (file)
@@ -423,11 +423,6 @@ void amdgpu_gmc_noretry_set(struct amdgpu_device *adev)
        switch (adev->asic_type) {
        case CHIP_VEGA10:
        case CHIP_VEGA20:
-       case CHIP_NAVI10:
-       case CHIP_NAVI14:
-       case CHIP_SIENNA_CICHLID:
-       case CHIP_NAVY_FLOUNDER:
-       case CHIP_DIMGREY_CAVEFISH:
                /*
                 * noretry = 0 will cause kfd page fault tests fail
                 * for some ASICs, so set default to 1 for these ASICs.