[PATCH] drm/amdkfd: fix double assign skip process context clear

Felix Kuehling felix.kuehling at amd.com
Thu Aug 10 20:04:41 UTC 2023


On 2023-08-10 15:03, Jonathan Kim wrote:
> Remove redundant assignment when skipping process ctx clear.
>
> Signed-off-by: Jonathan Kim <jonathan.kim at amd.com>

Reviewed-by: Felix Kuehling <Felix.Kuehling at amd.com>


> ---
>   drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c | 1 -
>   1 file changed, 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
> index aa5091f18681..89c2bfcb36ce 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
> @@ -227,7 +227,6 @@ static int add_queue_mes(struct device_queue_manager *dqm, struct queue *q,
>   	queue_input.tba_addr = qpd->tba_addr;
>   	queue_input.tma_addr = qpd->tma_addr;
>   	queue_input.trap_en = !kfd_dbg_has_cwsr_workaround(q->device);
> -	queue_input.skip_process_ctx_clear = qpd->pqm->process->debug_trap_enabled;
>   	queue_input.skip_process_ctx_clear = qpd->pqm->process->debug_trap_enabled ||
>   					     kfd_dbg_has_ttmps_always_setup(q->device);
>   


More information about the amd-gfx mailing list