Am 11.10.19 um 22:50 schrieb Alex Deucher:
We need to allocate a large enough buffer for the
session info, otherwise the IB test can overwrite
other memory.

Bug: https://bugzilla.kernel.org/show_bug.cgi?id=204241
Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>

Acked-by: Christian König <christian.koe...@amd.com> for the series.

---
  drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c | 8 ++++----
  1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c 
b/drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c
index 670784a78512..909bc2ce791f 100644
--- a/drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c
@@ -215,12 +215,12 @@ static int uvd_v6_0_enc_get_create_msg(struct amdgpu_ring 
*ring, uint32_t handle
        uint64_t dummy;
        int i, r;
- r = amdgpu_job_alloc_with_ib(ring->adev, ib_size_dw * 4, &job);
+       r = amdgpu_job_alloc_with_ib(ring->adev, (ib_size_dw * 4) + 1024, &job);
        if (r)
                return r;
ib = &job->ibs[0];
-       dummy = ib->gpu_addr + 1024;
+       dummy = ib->gpu_addr + (ib_size_dw * 4);
ib->length_dw = 0;
        ib->ptr[ib->length_dw++] = 0x00000018;
@@ -277,12 +277,12 @@ static int uvd_v6_0_enc_get_destroy_msg(struct 
amdgpu_ring *ring,
        uint64_t dummy;
        int i, r;
- r = amdgpu_job_alloc_with_ib(ring->adev, ib_size_dw * 4, &job);
+       r = amdgpu_job_alloc_with_ib(ring->adev, (ib_size_dw * 4) + 1024, &job);
        if (r)
                return r;
ib = &job->ibs[0];
-       dummy = ib->gpu_addr + 1024;
+       dummy = ib->gpu_addr + (ib_size_dw * 4);
ib->length_dw = 0;
        ib->ptr[ib->length_dw++] = 0x00000018;

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

Reply via email to