Skip to contentSkip to conversation

drm/amd/amdgpu: Support full range of GFX ring names

PATCH In review Tom St Denis 3 participants 3 messages Sep 1, 2017 1 message not in the archive

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 drm/amd/amdgpu: Support full range of GFX ring names

Right now there's only one but the rest of the code is being setup to support more so might as well fix this up too.

---
 drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c +4 -1
index 4942960b30c6..3548cc02f8d8 100644
@@ -1330,7 +1330,10 @@ static int gfx_v9_0_sw_init(void *handle)
 	for (i = 0; i < adev->gfx.num_gfx_rings; i++) {
 		ring = &adev->gfx.gfx_ring[i];
 		ring->ring_obj = NULL;
-		sprintf(ring->name, "gfx");
+		if (!i)
+			sprintf(ring->name, "gfx");
+		else
+			sprintf(ring->name, "gfx_%d", i);
 		ring->use_doorbell = true;
 		ring->doorbell_index = AMDGPU_DOORBELL64_GFX_RING0 << 1;
 		r = amdgpu_ring_init(adev, ring, 1024,
Signature · 6 lines
2.12.0

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

A message that is not in the archive (sent off-list or not yet received): [email protected]

Reviewed-by
#2 lore raw

Am 01.09.2017 um 15:17 schrieb Tom St Denis:

Quoted 4 lines

Right now there's only one but the rest of the code is being setup to support more so might as well fix this up too.

Quoted 10 lines
---
  drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 5 ++++-
  1 file changed, 4 insertions(+), 1 deletion(-)
drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c +0 -0
index 4942960b30c6..3548cc02f8d8 100644
@@ -1330,7 +1330,10 @@ static int gfx_v9_0_sw_init(void *handle)
  	for (i = 0; i < adev->gfx.num_gfx_rings; i++) {

drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c static int gfx_v9_0_sw_init(void *handle) · line 1339

Quoted 10 lines
drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c +4 -1
  		ring = &adev->gfx.gfx_ring[i];
  		ring->ring_obj = NULL;
-		sprintf(ring->name, "gfx");
+		if (!i)
+			sprintf(ring->name, "gfx");
+		else
+			sprintf(ring->name, "gfx_%d", i);
  		ring->use_doorbell = true;
  		ring->doorbell_index = AMDGPU_DOORBELL64_GFX_RING0 << 1;
  		r = amdgpu_ring_init(adev, ring, 1024,

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

Reviewed-by Alex Deucher
#3 lore raw
In reply to a message not in the archive
Quoted 8 lines

-----Original Message-----
From: amd-gfx [mailto:[email protected]] On Behalf Of Tom St Denis
Sent: Friday, September 01, 2017 9:17 AM
To: [email protected]
Cc: StDenis, Tom
Subject: [PATCH] drm/amd/amdgpu: Support full range of GFX ring names

Quoted 4 lines

Right now there's only one but the rest of the code is being setup to support more so might as well fix this up too.

Quoted 25 lines
---
 drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c +4 -1
index 4942960b30c6..3548cc02f8d8 100644
@@ -1330,7 +1330,10 @@ static int gfx_v9_0_sw_init(void *handle)
 	for (i = 0; i < adev->gfx.num_gfx_rings; i++) {
 		ring = &adev->gfx.gfx_ring[i];
 		ring->ring_obj = NULL;
-		sprintf(ring->name, "gfx");
+		if (!i)
+			sprintf(ring->name, "gfx");
+		else
+			sprintf(ring->name, "gfx_%d", i);
 		ring->use_doorbell = true;
 		ring->doorbell_index = AMDGPU_DOORBELL64_GFX_RING0
 << 1;

r = amdgpu_ring_init(adev, ring, 1024,
--
2.12.0

Quoted 4 lines

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

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