drm/radeon: fill in rb backend map on evergreen/ni.

From: Dave Airlie <[email protected]>

This looks to have never gotten filled in, and it seems to
trigger a bug in mesa.

Reported-by Roland Scheidegger <[email protected]>

Signed-off-by Dave Airlie <[email protected]>

---
 drivers/gpu/drm/radeon/evergreen.c | 1 +
 drivers/gpu/drm/radeon/ni.c        | 1 +
 2 files changed, 2 insertions(+)

drivers/gpu/drm/radeon/evergreen.c +1 -0

index 24fe66c..5712d63 100644
@@ -3513,6 +3513,7 @@ static void evergreen_gpu_init(struct radeon_device *rdev)
 		tmp = r6xx_remap_render_backend(rdev, tmp, rdev->config.evergreen.max_backends,
 						EVERGREEN_MAX_BACKENDS, disabled_rb_mask);
 	}
+	rdev->config.evergreen.backend_map = tmp;
 	WREG32(GB_BACKEND_MAP, tmp);
 
 	WREG32(CGTS_SYS_TCC_DISABLE, 0);

drivers/gpu/drm/radeon/ni.c +0 -0

index 9eccd0c..381b0255 100644

FWIW I was wrong that it was never filled in - when the backend map query was introduced, the data indeed got filled in. However that got lost very soon afterwards (over 6 years ago) by some fixes for backend map calculation.
In any case,

Reviewed-by Roland Scheidegger <[email protected]>

Am 09.01.2018 um 04:43 schrieb Dave Airlie:

Quoted 35 lines

From: Dave Airlie <[email protected]>

This looks to have never gotten filled in, and it seems to
trigger a bug in mesa.

Reported-by Roland Scheidegger <[email protected]>

Signed-off-by Dave Airlie <[email protected]>

---
 drivers/gpu/drm/radeon/evergreen.c | 1 +
 drivers/gpu/drm/radeon/ni.c        | 1 +
 2 files changed, 2 insertions(+)

drivers/gpu/drm/radeon/evergreen.c +1 -0

index 24fe66c..5712d63 100644
@@ -3513,6 +3513,7 @@ static void evergreen_gpu_init(struct radeon_device *rdev)
              tmp = r6xx_remap_render_backend(rdev, tmp, rdev->config.evergreen.max_backends,
                                              EVERGREEN_MAX_BACKENDS, disabled_rb_mask);
      }
+     rdev->config.evergreen.backend_map = tmp;
      WREG32(GB_BACKEND_MAP, tmp);
 
      WREG32(CGTS_SYS_TCC_DISABLE, 0);

drivers/gpu/drm/radeon/ni.c +1 -0

index 9eccd0c..381b0255 100644
@@ -1148,6 +1148,7 @@ static void cayman_gpu_init(struct radeon_device *rdev)
                                              rdev->config.cayman.max_shader_engines,
                                              CAYMAN_MAX_BACKENDS, disabled_rb_mask);
      }
+     rdev->config.cayman.backend_map = tmp;
      WREG32(GB_BACKEND_MAP, tmp);
 

drivers/gpu/drm/radeon/ni.c static void cayman_gpu_init(struct radeon_device *rdev) · line 1154

Quoted 2 lines

drivers/gpu/drm/radeon/ni.c +0 -0

      cgts_tcc_disable = 0xffff0000;

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel