drm/amdkfd: Fix type of 'dbg_flags' in 'struct kfd_process'
authorSrinivasan Shanmugam <srinivasan.shanmugam@amd.com>
Thu, 21 Dec 2023 02:40:42 +0000 (08:10 +0530)
committerAlex Deucher <alexander.deucher@amd.com>
Wed, 3 Jan 2024 16:16:06 +0000 (11:16 -0500)
dbg_flags looks to be defined with incorrect data type; to process
multiple debug flag options, and hence defined dbg_flags as u32.

Fixes the below:

drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_packet_manager_v9.c:117 pm_map_process_aldebaran() warn: maybe use && instead of &

Fixes: 0de4ec9a0353 ("drm/amdgpu: prepare map process for multi-process debug devices")
Suggested-by: Lijo Lazar <lijo.lazar@amd.com>
Cc: Felix Kuehling <Felix.Kuehling@amd.com>
Cc: Christian König <christian.koenig@amd.com>
Cc: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam@amd.com>
Reviewed-by: Felix Kuehling <Felix.Kuehling@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdkfd/kfd_priv.h

index 45366b4..745024b 100644 (file)
@@ -970,7 +970,7 @@ struct kfd_process {
        struct work_struct debug_event_workarea;
 
        /* Tracks debug per-vmid request for debug flags */
-       bool dbg_flags;
+       u32 dbg_flags;
 
        atomic_t poison;
        /* Queues are in paused stated because we are in the process of doing a CRIU checkpoint */