drm/amdgpu: just disallow reading untouched registers
authorChristian König <christian.koenig@amd.com>
Fri, 24 Mar 2017 13:32:50 +0000 (14:32 +0100)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 30 Mar 2017 18:44:00 +0000 (14:44 -0400)
Not sure what the original intention was here, but returning a random piece of
kernel memory to userspace because we didn't set the value at all is clearly
not a good idea.

This patch disallows reading the register and returns
a proper error code instead.

Signed-off-by: Christian König <christian.koenig@amd.com>
Acked-by: Alex Deucher <alexander.deucher@amd.com>
Acked-by: Tom St Denis <tom.stdenis@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/vi.c

index 1fe654e8cb7fd805a9322ae7bcea9b0614b2894b..b1132f5e84fc10d2d281e8ba1a3b89143cc31321 100644 (file)
@@ -464,15 +464,9 @@ static void vi_detect_hw_virtualization(struct amdgpu_device *adev)
 }
 
 static const struct amdgpu_allowed_register_entry tonga_allowed_read_registers[] = {
-       {mmGB_MACROTILE_MODE7, true},
 };
 
 static const struct amdgpu_allowed_register_entry cz_allowed_read_registers[] = {
-       {mmGB_TILE_MODE7, true},
-       {mmGB_TILE_MODE12, true},
-       {mmGB_TILE_MODE17, true},
-       {mmGB_TILE_MODE23, true},
-       {mmGB_MACROTILE_MODE7, true},
 };
 
 static const struct amdgpu_allowed_register_entry vi_allowed_read_registers[] = {