Re: [PATCH 2/3] module: Fix up module_notifier return values.

"Frank Ch. Eigler" <[email protected]> Mon, 24 Jun 2019 16:58:10 -0400
Newsgroups gmane.linux.kernel,gmane.linux.oprofile,gmane.linux.network
Message-ID <[email protected]>
Hi -

> > While auditing all module notifiers I noticed a whole bunch of fail
> > wrt the return value. Notifiers have a 'special' return semantics.

From peterz's comments, the patches, it's not obvious to me how one is
to choose between 0 (NOTIFY_DONE) and 1 (NOTIFY_OK) in the case of a
routine success.

> [...]
> I have a similar erroneous module notifier return value pattern
> in lttng-modules as well. I'll go fix it right away. CCing
> Frank Eigler from SystemTAP which AFAIK use a copy of
> lttng-tracepoint.c in their project, which should be fixed
> as well. I'm pasting the lttng-modules fix below.

Sure, following suit.  Thanks.

- FChE