Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/2] lib/amdgpu: add secure flag to sdma packet
@ 2024-01-11 19:25 vitaly.prosyak
  2024-01-11 19:25 ` vitaly.prosyak
                   ` (6 more replies)
  0 siblings, 7 replies; 17+ messages in thread
From: vitaly.prosyak @ 2024-01-11 19:25 UTC (permalink / raw)
  To: igt-dev; +Cc: Alex Deucher, Christian Koenig

From: Vitaly Prosyak <vitaly.prosyak@amd.com>

The change is required to handle the security flag for
trusted memory zone copy operation(TMZ feature).
Previously the security test was its own hardcoded version,
now we are allowed to use ASIC -the dependent version function
to create the SDMA packet.

v2 : added description to commit message (Kamil)

Cc: Jesse Zhang <jesse.zhang@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: Vitaly Prosyak <vitaly.prosyak@amd.com>
Reviewed-by: Jesse Zhang <jesse.zhang@amd.com>
---
 lib/amdgpu/amd_ip_blocks.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/amdgpu/amd_ip_blocks.c b/lib/amdgpu/amd_ip_blocks.c
index 1adea6987..a7ccfa38b 100644
--- a/lib/amdgpu/amd_ip_blocks.c
+++ b/lib/amdgpu/amd_ip_blocks.c
@@ -139,7 +139,7 @@ sdma_ring_copy_linear(const struct amdgpu_ip_funcs *func,
 	} else {
 		context->pm4[i++] = SDMA_PACKET(SDMA_OPCODE_COPY,
 				       SDMA_COPY_SUB_OPCODE_LINEAR,
-				       0);
+					context->secure ? 0x4 : 0);
 		if (func->family_id >= AMDGPU_FAMILY_AI)
 			context->pm4[i++] = context->write_length - 1;
 		else
-- 
2.25.1

^ permalink raw reply related	[flat|nested] 17+ messages in thread
* [PATCH 1/2] lib/amdgpu: add secure flag to sdma packet
@ 2024-01-11  0:31 vitaly.prosyak
  2024-01-11  0:31 ` vitaly.prosyak
  2024-01-11 13:09 ` Kamil Konieczny
  0 siblings, 2 replies; 17+ messages in thread
From: vitaly.prosyak @ 2024-01-11  0:31 UTC (permalink / raw)
  To: igt-dev; +Cc: Alex Deucher, Christian Koenig

From: Vitaly Prosyak <vitaly.prosyak@amd.com>

Cc: Jesse Zhang <jesse.zhang@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: Vitaly Prosyak <vitaly.prosyak@amd.com>
---
 lib/amdgpu/amd_ip_blocks.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/amdgpu/amd_ip_blocks.c b/lib/amdgpu/amd_ip_blocks.c
index 1adea6987..a7ccfa38b 100644
--- a/lib/amdgpu/amd_ip_blocks.c
+++ b/lib/amdgpu/amd_ip_blocks.c
@@ -139,7 +139,7 @@ sdma_ring_copy_linear(const struct amdgpu_ip_funcs *func,
 	} else {
 		context->pm4[i++] = SDMA_PACKET(SDMA_OPCODE_COPY,
 				       SDMA_COPY_SUB_OPCODE_LINEAR,
-				       0);
+					context->secure ? 0x4 : 0);
 		if (func->family_id >= AMDGPU_FAMILY_AI)
 			context->pm4[i++] = context->write_length - 1;
 		else
-- 
2.25.1

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

end of thread, other threads:[~2024-01-18 12:40 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-01-11 19:25 [PATCH 1/2] lib/amdgpu: add secure flag to sdma packet vitaly.prosyak
2024-01-11 19:25 ` vitaly.prosyak
2024-01-11 19:25 ` [PATCH 2/2] tests/amd_security: fix secure bounce test vitaly.prosyak
2024-01-11 19:25   ` vitaly.prosyak
2024-01-15 15:19   ` Kamil Konieczny
2024-01-11 20:23 ` ✓ Fi.CI.BAT: success for series starting with [1/2] lib/amdgpu: add secure flag to sdma packet Patchwork
2024-01-11 21:23 ` ✗ CI.xeBAT: failure " Patchwork
2024-01-12 17:45   ` Kamil Konieczny
2024-01-11 23:39 ` ✗ Fi.CI.IGT: " Patchwork
2024-01-12 17:46   ` Kamil Konieczny
2024-01-18 12:32     ` Illipilli, TejasreeX
2024-01-15 15:16 ` [PATCH 1/2] " Kamil Konieczny
2024-01-16  4:18   ` vitaly prosyak
2024-01-18 12:40 ` ✓ Fi.CI.IGT: success for series starting with [1/2] " Patchwork
  -- strict thread matches above, loose matches on Subject: below --
2024-01-11  0:31 [PATCH 1/2] " vitaly.prosyak
2024-01-11  0:31 ` vitaly.prosyak
2024-01-11 13:09 ` Kamil Konieczny

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