Re: [PATCH 2/2] drm/amdgpu: fix KV harvesting

2018-03-06 Thread Zhu, Rex 
Acked-by:  Rex Zhu<re...@amd.com> for the series.


Best Regards
Rex


From: amd-gfx <amd-gfx-boun...@lists.freedesktop.org> on behalf of Alex Deucher 
<alexdeuc...@gmail.com>
Sent: Tuesday, March 6, 2018 2:30 AM
To: amd-gfx@lists.freedesktop.org
Cc: Deucher, Alexander; sta...@vger.kernel.org
Subject: [PATCH 2/2] drm/amdgpu: fix KV harvesting

Always set the graphics values to the max for the
asic type.  E.g., some 1 RB chips are actually 1 RB chips,
others are actually harvested 2 RB chips.

Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=99353
Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>
Cc: sta...@vger.kernel.org
---
 drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c | 30 ++
 1 file changed, 2 insertions(+), 28 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c 
b/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
index 972d421caada..e13d9d83767b 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
@@ -4358,34 +4358,8 @@ static void gfx_v7_0_gpu_early_init(struct amdgpu_device 
*adev)
 case CHIP_KAVERI:
 adev->gfx.config.max_shader_engines = 1;
 adev->gfx.config.max_tile_pipes = 4;
-   if ((adev->pdev->device == 0x1304) ||
-   (adev->pdev->device == 0x1305) ||
-   (adev->pdev->device == 0x130C) ||
-   (adev->pdev->device == 0x130F) ||
-   (adev->pdev->device == 0x1310) ||
-   (adev->pdev->device == 0x1311) ||
-   (adev->pdev->device == 0x131C)) {
-   adev->gfx.config.max_cu_per_sh = 8;
-   adev->gfx.config.max_backends_per_se = 2;
-   } else if ((adev->pdev->device == 0x1309) ||
-  (adev->pdev->device == 0x130A) ||
-  (adev->pdev->device == 0x130D) ||
-  (adev->pdev->device == 0x1313) ||
-  (adev->pdev->device == 0x131D)) {
-   adev->gfx.config.max_cu_per_sh = 6;
-   adev->gfx.config.max_backends_per_se = 2;
-   } else if ((adev->pdev->device == 0x1306) ||
-  (adev->pdev->device == 0x1307) ||
-  (adev->pdev->device == 0x130B) ||
-  (adev->pdev->device == 0x130E) ||
-  (adev->pdev->device == 0x1315) ||
-  (adev->pdev->device == 0x131B)) {
-   adev->gfx.config.max_cu_per_sh = 4;
-   adev->gfx.config.max_backends_per_se = 1;
-   } else {
-   adev->gfx.config.max_cu_per_sh = 3;
-   adev->gfx.config.max_backends_per_se = 1;
-   }
+   adev->gfx.config.max_cu_per_sh = 8;
+   adev->gfx.config.max_backends_per_se = 2;
 adev->gfx.config.max_sh_per_se = 1;
 adev->gfx.config.max_texture_channel_caches = 4;
 adev->gfx.config.max_gprs = 256;
--
2.13.6

___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
amd-gfx Info Page - 
freedesktop.org<https://lists.freedesktop.org/mailman/listinfo/amd-gfx>
lists.freedesktop.org
Subscribing to amd-gfx: Subscribe to amd-gfx by filling out the following form. 
Use of all freedesktop.org lists is subject to our Code of ...



___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

[PATCH 2/2] drm/amdgpu: fix KV harvesting

2018-03-05 Thread Alex Deucher 
Always set the graphics values to the max for the
asic type.  E.g., some 1 RB chips are actually 1 RB chips,
others are actually harvested 2 RB chips.

Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=99353
Signed-off-by: Alex Deucher 
Cc: sta...@vger.kernel.org
---
 drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c | 30 ++
 1 file changed, 2 insertions(+), 28 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c 
b/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
index 972d421caada..e13d9d83767b 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
@@ -4358,34 +4358,8 @@ static void gfx_v7_0_gpu_early_init(struct amdgpu_device 
*adev)
case CHIP_KAVERI:
adev->gfx.config.max_shader_engines = 1;
adev->gfx.config.max_tile_pipes = 4;
-   if ((adev->pdev->device == 0x1304) ||
-   (adev->pdev->device == 0x1305) ||
-   (adev->pdev->device == 0x130C) ||
-   (adev->pdev->device == 0x130F) ||
-   (adev->pdev->device == 0x1310) ||
-   (adev->pdev->device == 0x1311) ||
-   (adev->pdev->device == 0x131C)) {
-   adev->gfx.config.max_cu_per_sh = 8;
-   adev->gfx.config.max_backends_per_se = 2;
-   } else if ((adev->pdev->device == 0x1309) ||
-  (adev->pdev->device == 0x130A) ||
-  (adev->pdev->device == 0x130D) ||
-  (adev->pdev->device == 0x1313) ||
-  (adev->pdev->device == 0x131D)) {
-   adev->gfx.config.max_cu_per_sh = 6;
-   adev->gfx.config.max_backends_per_se = 2;
-   } else if ((adev->pdev->device == 0x1306) ||
-  (adev->pdev->device == 0x1307) ||
-  (adev->pdev->device == 0x130B) ||
-  (adev->pdev->device == 0x130E) ||
-  (adev->pdev->device == 0x1315) ||
-  (adev->pdev->device == 0x131B)) {
-   adev->gfx.config.max_cu_per_sh = 4;
-   adev->gfx.config.max_backends_per_se = 1;
-   } else {
-   adev->gfx.config.max_cu_per_sh = 3;
-   adev->gfx.config.max_backends_per_se = 1;
-   }
+   adev->gfx.config.max_cu_per_sh = 8;
+   adev->gfx.config.max_backends_per_se = 2;
adev->gfx.config.max_sh_per_se = 1;
adev->gfx.config.max_texture_channel_caches = 4;
adev->gfx.config.max_gprs = 256;
-- 
2.13.6

___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx