Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH] lib/amdgpu: fix typo in function compare_pattern
@ 2023-09-08  2:50 vitaly.prosyak
  2023-09-08  5:17 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
                   ` (4 more replies)
  0 siblings, 5 replies; 7+ messages in thread
From: vitaly.prosyak @ 2023-09-08  2:50 UTC (permalink / raw)
  To: igt-dev; +Cc: Alex Deucher, Jesse Zhang, Luben Tuikov, Christian Koenig

From: Jesse Zhang <jesse.zhang@amd.com>

 In function compare_pattern check the destination buffer (bo2_cpu),
 instead of source buffer(bo_cpu).
 Add validation of the return value in the eviction test.

Cc: Luben Tuikov <luben.tuikov@amd.com>
Cc: Alex Deucher <alexander.deucher@amd.com>
Cc: Christian Koenig <christian.koenig@amd.com>
Signed-off-by: Jesse Zhang <Jesse.Zhang@amd.com>
Reviewed-by: Vitaly Prosyak <vitaly.prosyak@amd.com>
---
 lib/amdgpu/amd_ip_blocks.c | 2 +-
 tests/amdgpu/amd_basic.c   | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/lib/amdgpu/amd_ip_blocks.c b/lib/amdgpu/amd_ip_blocks.c
index be9ff38c6..7f8d4a4cd 100644
--- a/lib/amdgpu/amd_ip_blocks.c
+++ b/lib/amdgpu/amd_ip_blocks.c
@@ -296,7 +296,7 @@ x_compare_pattern(const struct amdgpu_ip_funcs *func,
 	int num_compare = ring_context->write_length/div;
 
 	while (i < num_compare) {
-		if (ring_context->bo_cpu[i++] != func->pattern) {
+		if (ring_context->bo2_cpu[i++] != func->pattern) {
 			ret = -1;
 			break;
 		}
diff --git a/tests/amdgpu/amd_basic.c b/tests/amdgpu/amd_basic.c
index dfe11995f..e7a42a335 100644
--- a/tests/amdgpu/amd_basic.c
+++ b/tests/amdgpu/amd_basic.c
@@ -411,6 +411,7 @@ amdgpu_bo_eviction_test(amdgpu_device_handle device_handle)
 			amdgpu_test_exec_cs_helper(device_handle, ip_block->type, ring_context);
 			/* fulfill PM4: test DMA copy linear */
 			r = ip_block->funcs->compare_pattern(ip_block->funcs, ring_context, sdma_write_length);
+			igt_assert_eq(r, 0);
 			amdgpu_bo_unmap_and_free(ring_context->bo, ring_context->va_handle, ring_context->bo_mc,
 						 ring_context->write_length);
 			amdgpu_bo_unmap_and_free(ring_context->bo2, ring_context->va_handle2, ring_context->bo_mc2,
-- 
2.25.1

^ permalink raw reply related	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2023-09-21 19:56 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-09-08  2:50 [igt-dev] [PATCH] lib/amdgpu: fix typo in function compare_pattern vitaly.prosyak
2023-09-08  5:17 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2023-09-08  5:42 ` [igt-dev] ✓ CI.xeBAT: " Patchwork
2023-09-08  8:37 ` [igt-dev] [PATCH] " Christian König
2023-09-08  9:53 ` [igt-dev] ✗ Fi.CI.IGT: failure for " Patchwork
2023-09-08 17:18 ` [igt-dev] [PATCH] " Kamil Konieczny
2023-09-21 19:56   ` vitaly prosyak

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox