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

[Jesse Zhang <jesse.zhang@amd.com>]

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@amd.com>
Cc: Luben Tuikov <luben.tuikov@amd.com>
Cc: Alex Deucher <alexander.deucher@amd.com>
Cc: Christian Koenig <christian.koenig@amd.com>
Cc: Kamil Konieczny <kamil.konieczny@linux.intel.com>

Signed-off-by: Jesse Zhang <Jesse.Zhang@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