[PATCH 1/2] lib/amdgpu: add secure flag to sdma packet
Kamil Konieczny
kamil.konieczny at linux.intel.com
Thu Jan 11 13:09:37 UTC 2024
Hi Vitaly,
On 2024-01-10 at 19:31:55 -0500, vitaly.prosyak at amd.com wrote:
> From: Vitaly Prosyak <vitaly.prosyak at amd.com>
>
Please describe your change here, why it is needed.
> Cc: Jesse Zhang <jesse.zhang 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: Vitaly Prosyak <vitaly.prosyak at 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);
imho just 4 would be ok.
Regards,
Kamil
> if (func->family_id >= AMDGPU_FAMILY_AI)
> context->pm4[i++] = context->write_length - 1;
> else
> --
> 2.25.1
>
More information about the igt-dev
mailing list