[Public] You can remove BUG:SWDEV-534537 from commit message as it doesn't provide any information to public. With that this patch is
Reviewed-by: Harish Kasiviswanathan <[email protected]. -----Original Message----- From: Clement, Sunday <[email protected]> Sent: Friday, May 23, 2025 7:54 PM To: [email protected] Cc: Tudor, Alexandru <[email protected]>; Kasiviswanathan, Harish <[email protected]>; Clement, Sunday <[email protected]> Subject: [PATCH] drm/amdkfd: Identical code for different branches This patch removes the if/else statement in the cik_event_interrupt_wq function because it is redundant with both branches resulting in identical outcomes, this improves code readibility. BUG:SWDEV-534537 Signed-off-by: Sunday Clement <[email protected]> --- drivers/gpu/drm/amd/amdkfd/cik_event_interrupt.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/gpu/drm/amd/amdkfd/cik_event_interrupt.c b/drivers/gpu/drm/amd/amdkfd/cik_event_interrupt.c index 981d9adcc5e1..25f32771b228 100644 --- a/drivers/gpu/drm/amd/amdkfd/cik_event_interrupt.c +++ b/drivers/gpu/drm/amd/amdkfd/cik_event_interrupt.c @@ -125,11 +125,7 @@ static void cik_event_interrupt_wq(struct kfd_node *dev, return; } - if (info.vmid == vmid) - kfd_signal_vm_fault_event(pdd, &info, NULL); - else - kfd_signal_vm_fault_event(pdd, &info, NULL); - + kfd_signal_vm_fault_event(pdd, &info, NULL); kfd_unref_process(p); } } -- 2.43.0
