[igt-dev] [PATCH i-g-t] tests/amdgpu: fix secure bounce test issue

[Jesse Zhang]

To the secure memory, the destination data(bo2) has been encrypted,
after copying it via SDMA from bo to bo2.
It cannot compare bo2 with the original data.
Therefore, skip this memory check.

Cc: Vitaly Prosyak <vitaly.prosyak at amd.com>
Cc: Luben Tuikov <luben.tuikov at amd.com>
Cc: Alex Deucher <alexander.deucher at amd.com>
Cc: Christian Koenig <christian.koenig at amd.com>
Cc: Kamil Konieczny <kamil.konieczny at linux.intel.com>

Signed-off-by: Jesse Zhang <Jesse.Zhang at amd.com>
---
 tests/amdgpu/amd_security.c | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/tests/amdgpu/amd_security.c b/tests/amdgpu/amd_security.c
index d1146a7ce..678270fdf 100644
--- a/tests/amdgpu/amd_security.c
+++ b/tests/amdgpu/amd_security.c
@@ -225,14 +225,16 @@ amdgpu_secure_bounce(amdgpu_device_handle device_handle, int fd,
 			secure == true ? 1 : 0);
 
 	/* Verify the contents of Bob. */
-	for (pp = (__typeof__(pp))ring_context->bo2_cpu;
-	     pp < (__typeof__(pp)) ring_context->bo2_cpu + SECURE_BUFFER_SIZE;
-	     pp += sizeof(secure_pattern)) {
-		r = memcmp(pp, secure_pattern, sizeof(secure_pattern));
-		if (r) {
-			// test failure
-			igt_assert(false);
-			break;
+	if (!secure) {
+		for (pp = (__typeof__(pp))ring_context->bo2_cpu;
+			pp < (__typeof__(pp)) ring_context->bo2_cpu + SECURE_BUFFER_SIZE;
+			pp += sizeof(secure_pattern)) {
+			r = memcmp(pp, secure_pattern, sizeof(secure_pattern));
+			if (r) {
+				// test failure
+				igt_assert(false);
+				break;
+			}
 		}
 	}
 
-- 
2.25.1