]> git.ipfire.org Git - thirdparty/linux.git/commit
pps: clients: gpio: fix interrupt handling order in remove path
authorEliav Farber <farbere@amazon.com>
Tue, 27 May 2025 05:33:55 +0000 (05:33 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 1 Jul 2025 10:29:24 +0000 (12:29 +0200)
commit6bca1e955830808dc90e0506b2951b4256b81bbb
treebbf782165aa911837ef54e2fc4707f5c9a709690
parentbfb4cf9fb97e4063f0aa62e9e398025fb6625031
pps: clients: gpio: fix interrupt handling order in remove path

The interrupt handler in pps_gpio_probe() is registered after calling
pps_register_source() using devm_request_irq(). However, in the
corresponding remove function, pps_unregister_source() is called before
the IRQ is freed, since devm-managed resources are released after the
remove function completes.

This creates a potential race condition where an interrupt may occur
after the PPS source is unregistered but before the handler is removed,
possibly leading to a kernel panic.

To prevent this, switch from devm-managed IRQ registration to manual
management by using request_irq() and calling free_irq() explicitly in
the remove path before unregistering the PPS source. This ensures the
interrupt handler is safely removed before deactivating the PPS source.

Signed-off-by: Eliav Farber <farbere@amazon.com>
Link: https://lore.kernel.org/r/20250527053355.37185-1-farbere@amazon.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/pps/clients/pps-gpio.c