[PATCH 2/2] soc: xilinx: Fix event manager CPU hotplug state cleanup
Jay Buddhabhatti
jbuddhab at amd.com
Tue Nov 3 04:26:33 PST 2026
Hi Prasanna,
On Mon, Nov 02, 2026 at 10:35:15PM -0800, Prasanna Kumar T S M wrote:
> On driver removal, the event manager removes the CPUHP_AP_ONLINE_DYN
> CPU hotplug state instead of the its actual state. This can leave the
> event manager callbacks dangling. Use the actual CPU hotplug state
> during driver removal.
>
> Hotplug registration failures are also ignored, allowing probe to
> succeed without the callbacks needed to enable event interrupts on
> CPUs brought online later.
>
> Fixes: c7fdb2404f66 ("drivers: soc: xilinx: add xilinx event
management driver")
> Signed-off-by: Prasanna Kumar T S M ptsm at linux.microsoft.com
This looks good to me.
A couple of small nits:
- "instead of the its actual state" -> "instead of its actual state"
- Minor alignment on the cpuhp_setup_state() continuation lines;
checkpatch --strict complains.
CHECK: Alignment should match open parenthesis
#49: FILE: drivers/soc/xilinx/xlnx_event_manager.c:663:
+ hp_state = cpuhp_setup_state(CPUHP_AP_ONLINE_DYN,
+ "soc/event:starting", xlnx_event_cpuhp_start,
total: 0 errors, 0 warnings, 1 checks, 44 lines checked
Worth Cc: stable at vger.kernel.org as well, given the Fixes tag.
Reviewed-by: Jay Buddhabhatti <jay.buddhabhatti at amd.com>
Thanks,
Jay
More information about the linux-arm-kernel
mailing list