github-mirror/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-11-07 06:17:02 +01:00
Code Issues Actions Packages Projects Releases Wiki Activity Graph

notifier: add device-managed registration APIs

Browse Source 
Add devm_atomic_notifier_chain_register() and
devm_blocking_notifier_chain_register() that automatically unregister
the notifier when the device is unbound.

Many drivers repeat the same boilerplate pattern:

  1. Register the notifier with *_notifier_chain_register()
  2. Check for error
  3. Register a devm action to unregister on teardown
  4. Implement a per-driver static unregister callback

With the new devm_*_notifier_chain_register() APIs, this reduces to a
single call with one error path, eliminating per-driver unregister
callbacks entirely.

The implementation follows the established devres pattern used by other
device-managed kernel APIs.

A devm variant for raw notifier chains is intentionally not included
because raw notifiers require the caller to provide all locking. The
devres teardown callback has no way to acquire the caller's lock, and
devres_alloc() with GFP_KERNEL cannot be used if the caller's lock is a
spinlock.

Signed-off-by: Eliav Farber <farbere@amazon.com>
Acked-by: Linus Walleij <linusw@kernel.org>
Tested-by: Bradley Morgan <include@grrlz.net>
Acked-by: Bradley Morgan <include@grrlz.net> # kernel/
Link: https://patch.msgid.link/20260816060648.4030-2-farbere@amazon.com
[ukleinek: Add note to kdoc about devm_atomic_notifier_chain_register()
 not being callable in atomic context.]
Signed-off-by: Uwe Kleine-König <ukleinek@kernel.org>
This commit is contained in:
Eliav Farber 2026-08-16 06:06:00 +00:00 committed by Uwe Kleine-König
parent 8d3ae59288
commit e38b9eda47
2 changed files with 112 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
include/linux/notifier.h
View File

@ -46,6 +46,7 @@
* often but notifier_blocks will seldom be removed.
*/
struct device;
struct notifier_block;
typedef int (*notifier_fn_t)(struct notifier_block *nb,
@ -145,8 +146,14 @@ extern void srcu_init_notifier_head(struct srcu_notifier_head *nh);
extern int atomic_notifier_chain_register(struct atomic_notifier_head *nh,
struct notifier_block *nb);
int devm_atomic_notifier_chain_register(struct device *dev,
struct atomic_notifier_head *nh,
struct notifier_block *nb);
extern int blocking_notifier_chain_register(struct blocking_notifier_head *nh,
struct notifier_block *nb);
int devm_blocking_notifier_chain_register(struct device *dev,
struct blocking_notifier_head *nh,
struct notifier_block *nb);
extern int raw_notifier_chain_register(struct raw_notifier_head *nh,
struct notifier_block *nb);
extern int srcu_notifier_chain_register(struct srcu_notifier_head *nh,

105
kernel/notifier.c
View File

@ -1,4 +1,5 @@
// SPDX-License-Identifier: GPL-2.0-only
#include <linux/device/devres.h>
#include <linux/kdebug.h>
#include <linux/kprobes.h>
#include <linux/export.h>
@ -197,6 +198,58 @@ int atomic_notifier_chain_unregister(struct atomic_notifier_head *nh,
}
EXPORT_SYMBOL_GPL(atomic_notifier_chain_unregister);
struct atomic_notifier_chain_devres {
struct atomic_notifier_head *nh;
struct notifier_block *nb;
};
static void devm_atomic_notifier_chain_unregister(struct device *dev, void *res)
{
struct atomic_notifier_chain_devres *dr = res;
atomic_notifier_chain_unregister(dr->nh, dr->nb);
}
/**
* devm_atomic_notifier_chain_register - Device-managed atomic notifier registration
* @dev: Device to tie the notifier lifetime to
* @nh: Pointer to head of the atomic notifier chain
* @nb: New entry in notifier chain
*
* Adds a notifier to an atomic notifier chain and registers a cleanup
* action to automatically unregister it when @dev is unbound.
*
* Must be called in process context.
*
* Return:
* 0 on success, negative errno on error.
*/
int devm_atomic_notifier_chain_register(struct device *dev,
struct atomic_notifier_head *nh,
struct notifier_block *nb)
{
struct atomic_notifier_chain_devres *dr;
int ret;
dr = devres_alloc(devm_atomic_notifier_chain_unregister,
sizeof(*dr), GFP_KERNEL);
if (!dr)
return -ENOMEM;
ret = atomic_notifier_chain_register(nh, nb);
if (ret) {
devres_free(dr);
return ret;
}
dr->nh = nh;
dr->nb = nb;
devres_add(dev, dr);
return 0;
}
EXPORT_SYMBOL_GPL(devm_atomic_notifier_chain_register);
/**
* atomic_notifier_call_chain - Call functions in an atomic notifier chain
* @nh: Pointer to head of the atomic notifier chain
@ -349,6 +402,58 @@ int blocking_notifier_call_chain_robust(struct blocking_notifier_head *nh,
}
EXPORT_SYMBOL_GPL(blocking_notifier_call_chain_robust);
struct blocking_notifier_chain_devres {
struct blocking_notifier_head *nh;
struct notifier_block *nb;
};
static void devm_blocking_notifier_chain_unregister(struct device *dev,
void *res)
{
struct blocking_notifier_chain_devres *dr = res;
blocking_notifier_chain_unregister(dr->nh, dr->nb);
}
/**
* devm_blocking_notifier_chain_register - Device-managed blocking notifier registration
* @dev: Device to tie the notifier lifetime to
* @nh: Pointer to head of the blocking notifier chain
* @nb: New entry in notifier chain
*
* Adds a notifier to a blocking notifier chain and registers a cleanup
* action to automatically unregister it when @dev is unbound.
* Must be called in process context.
*
* Return:
* 0 on success, negative errno on error.
*/
int devm_blocking_notifier_chain_register(struct device *dev,
struct blocking_notifier_head *nh,
struct notifier_block *nb)
{
struct blocking_notifier_chain_devres *dr;
int ret;
dr = devres_alloc(devm_blocking_notifier_chain_unregister,
sizeof(*dr), GFP_KERNEL);
if (!dr)
return -ENOMEM;
ret = blocking_notifier_chain_register(nh, nb);
if (ret) {
devres_free(dr);
return ret;
}
dr->nh = nh;
dr->nb = nb;
devres_add(dev, dr);
return 0;
}
EXPORT_SYMBOL_GPL(devm_blocking_notifier_chain_register);
/**
* blocking_notifier_call_chain - Call functions in a blocking notifier chain
* @nh: Pointer to head of the blocking notifier chain