diff options
author | Luben Tuikov <luben.tuikov@amd.com> | 2021-07-15 21:28:16 -0400 |
---|---|---|
committer | Luben Tuikov <luben.tuikov@amd.com> | 2021-07-16 12:10:06 -0400 |
commit | 87a68fe9e49605d200ab2604e45b8a9d868d1922 (patch) | |
tree | 3d01fcc06df1261a5dba5ace868fe01b5e179ea0 | |
parent | 9cef5dee3cd817728c83aeb3c2010c1954e4c402 (diff) | |
download | drm-87a68fe9e49605d200ab2604e45b8a9d868d1922.tar.gz |
tests/amdgpu: Fix TMZ secure bounce test
Fix the TMZ secure bounce test, in that Bob's
buffer has to be created encrypted (with the
encrypted flag set), so that when we copy from
Alice's buffer, which is also encrypted, to Bob's
buffer, the copy can be successful and the data
actually copied.
This fixes the test and it no longer fails. Tested
on Sienna Cichlid.
Cc: Alex Deucher <Alexander.Deucher@amd.com>
Cc: Christian König <christian.koenig@amd.com>
Cc: Aaron Liu <aaron.liu@amd.com>
Cc: Huang Rui <ray.huang@amd.com>
Signed-off-by: Luben Tuikov <luben.tuikov@amd.com>
Reviewed-by: Alex Deucher <Alexander.Deucher@amd.com>
-rw-r--r-- | tests/amdgpu/security_tests.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tests/amdgpu/security_tests.c b/tests/amdgpu/security_tests.c index f0eb57a7..c7548dac 100644 --- a/tests/amdgpu/security_tests.c +++ b/tests/amdgpu/security_tests.c @@ -315,7 +315,7 @@ static void amdgpu_secure_bounce(void) SECURE_BUFFER_SIZE, page_size, AMDGPU_GEM_DOMAIN_VRAM, - 0 /* AMDGPU_GEM_CREATE_ENCRYPTED */, + AMDGPU_GEM_CREATE_ENCRYPTED, &bob); if (res) { PRINT_ERROR(res); @@ -323,9 +323,9 @@ static void amdgpu_secure_bounce(void) goto Out_free_Alice; } - /* sDMA clear copy from Alice to Bob. + /* sDMA TMZ copy from Alice to Bob. */ - amdgpu_bo_lcopy(&sb_ctx, &bob, &alice, SECURE_BUFFER_SIZE, 0); + amdgpu_bo_lcopy(&sb_ctx, &bob, &alice, SECURE_BUFFER_SIZE, 1); /* Move Bob to the GTT domain. */ |