diff options
| author |   Peter Zijlstra <peterz@infradead.org> | 2020-08-18 15:57:37 +0200 |
|---|---|---|
| committer |   Ingo Molnar <mingo@kernel.org> | 2020-09-01 09:58:03 +0200 |
| commit | 0340a6b7fb767f7f296b9bacc9a215920519a644 (patch) | |
| tree | aa04f191d261cc7274fd07c14f46cb06bbf5d969 /kernel/tracepoint.c | |
| parent | notifier: Fix broken error handling pattern (diff) | |
| download | linux-dev-0340a6b7fb767f7f296b9bacc9a215920519a644.tar.xz linux-dev-0340a6b7fb767f7f296b9bacc9a215920519a644.zip | |
module: Fix up module_notifier return values
While auditing all module notifiers I noticed a whole bunch of fail
wrt the return value. Notifiers have a 'special' return semantics.
As is; NOTIFY_DONE vs NOTIFY_OK is a bit vague; but
notifier_from_errno(0) results in NOTIFY_OK and NOTIFY_DONE has a
comment that says "Don't care".
From this I've used NOTIFY_DONE when the function completely ignores
the callback and notifier_to_error() isn't used.
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Reviewed-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Reviewed-by: Joel Fernandes (Google) <joel@joelfernandes.org>
Reviewed-by: Robert Richter <rric@kernel.org>
Acked-by: Steven Rostedt (VMware) <rostedt@goodmis.org>
Link: https://lore.kernel.org/r/20200818135804.385360407@infradead.org
Diffstat (limited to 'kernel/tracepoint.c')
| -rw-r--r-- | kernel/tracepoint.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/tracepoint.c b/kernel/tracepoint.c index 73956eaff8a9..8e05ed2cd39e 100644 --- a/kernel/tracepoint.c +++ b/kernel/tracepoint.c @@ -521,7 +521,7 @@ static int tracepoint_module_notify(struct notifier_block *self, case MODULE_STATE_UNFORMED: break; } - return ret; + return notifier_from_errno(ret); } static struct notifier_block tracepoint_module_nb = { |