[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