Messages in this thread | | | Date | Mon, 24 Jun 2019 16:58:10 -0400 | From | "Frank Ch. Eigler" <> | Subject | Re: [PATCH 2/3] module: Fix up module_notifier return values. |
| |
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
| |