drm/amdgpu:use clear_ring to clr RB
authorMonk Liu <Monk.Liu@amd.com>
Wed, 8 Feb 2017 08:51:06 +0000 (16:51 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 30 Mar 2017 03:53:14 +0000 (23:53 -0400)
In resume routine, we need clr RB prior to the
ring test of engine, otherwise some engine hang
duplicated during GPU reset.

Signed-off-by: Monk Liu <Monk.Liu@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c
drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c

index 7c842b7f1004dc6d4a4c52eac6af76035a7e9ee5..cead88ac3788c1fd49c21e18703c6c2a52e6b668 100644 (file)
@@ -230,7 +230,7 @@ int amdgpu_ring_init(struct amdgpu_device *adev, struct amdgpu_ring *ring,
                        dev_err(adev->dev, "(%d) ring create failed\n", r);
                        return r;
                }
-               memset((void *)ring->ring, 0, ring->ring_size);
+               amdgpu_ring_clear_ring(ring);
        }
        ring->ptr_mask = (ring->ring_size / 4) - 1;
        ring->max_dw = max_dw;
index b2bfee3e5b675c9cf05d9bc98dafe041848a6dc0..e09dd5e5b58a63d666e595ed26d843fa37603e54 100644 (file)
@@ -4515,6 +4515,7 @@ static int gfx_v8_0_cp_gfx_resume(struct amdgpu_device *adev)
        }
 
        /* start the ring */
+       amdgpu_ring_clear_ring(ring);
        gfx_v8_0_cp_gfx_start(adev);
        ring->ready = true;
        r = amdgpu_ring_test_ring(ring);
index 47e6f146aa2575d728e0182a1517e8f3cd6b7e54..1ac876fe3d5ad114f6b3cba934a89603eca660ca 100644 (file)
@@ -615,6 +615,7 @@ static int sdma_v3_0_gfx_resume(struct amdgpu_device *adev)
 
        for (i = 0; i < adev->sdma.num_instances; i++) {
                ring = &adev->sdma.instance[i].ring;
+               amdgpu_ring_clear_ring(ring);
                wb_offset = (ring->rptr_offs * 4);
 
                mutex_lock(&adev->srbm_mutex);