Patch is Reviewed-by: Boyuan Zhang <boyuan.zhang@xxxxxxx> Regards, Boyuan -----Original Message----- From: amd-gfx [mailto:amd-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx] On Behalf Of Liu, Leo Sent: December-11-18 4:04 PM To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx; dri-devel@xxxxxxxxxxxxxxxxxxxxx Cc: Liu, Leo Subject: [PATCH libdrm] tests/amdgpu/vcn: fix the nop command in IBs Just make them properly i.e. put 0 to the Nop reg Signed-off-by: Leo Liu <leo.liu@xxxxxxx> --- tests/amdgpu/vcn_tests.c | 18 ++++++++++++------ 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/tests/amdgpu/vcn_tests.c b/tests/amdgpu/vcn_tests.c index d9f05af8..859ec496 100644 --- a/tests/amdgpu/vcn_tests.c +++ b/tests/amdgpu/vcn_tests.c @@ -268,8 +268,10 @@ static void amdgpu_cs_vcn_dec_create(void) ib_cpu[len++] = msg_buf.addr >> 32; ib_cpu[len++] = 0x81C3; ib_cpu[len++] = 0; - for (; len % 16; ++len) - ib_cpu[len] = 0x81ff; + for (; len % 16; ) { + ib_cpu[len++] = 0x81ff; + ib_cpu[len++] = 0; + } r = submit(len, AMDGPU_HW_IP_VCN_DEC); CU_ASSERT_EQUAL(r, 0); @@ -336,8 +338,10 @@ static void amdgpu_cs_vcn_dec_decode(void) ib_cpu[len++] = 0x81C6; ib_cpu[len++] = 0x1; - for (; len % 16; ++len) - ib_cpu[len] = 0x80000000; + for (; len % 16; ) { + ib_cpu[len++] = 0x81ff; + ib_cpu[len++] = 0; + } r = submit(len, AMDGPU_HW_IP_VCN_DEC); CU_ASSERT_EQUAL(r, 0); @@ -373,8 +377,10 @@ static void amdgpu_cs_vcn_dec_destroy(void) ib_cpu[len++] = msg_buf.addr >> 32; ib_cpu[len++] = 0x81C3; ib_cpu[len++] = 0; - for (; len % 16; ++len) - ib_cpu[len] = 0x80000000; + for (; len % 16; ) { + ib_cpu[len++] = 0x81ff; + ib_cpu[len++] = 0; + } r = submit(len, AMDGPU_HW_IP_VCN_DEC); CU_ASSERT_EQUAL(r, 0); -- 2.17.1 _______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx _______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx