Date:   Fri, 12 Nov 2021 19:48:17 +0100
From:   Borislav Petkov <bp@alien8.de>
To:     Cristian Marussi <cristian.marussi@arm.com>
Cc:     LKML <linux-kernel@vger.kernel.org>, linux-crypto@vger.kernel.org
Subject: Re: [PATCH v0 39/42] crypto: ccree - check notifier registration
 return value
Message-ID: <YY628XtLN+IH8D6J@zn.tnic>
References: <20211108101157.15189-1-bp@alien8.de>
 <20211108101157.15189-40-bp@alien8.de>
 <20211112183233.GB6655@e120937-lin>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <20211112183233.GB6655@e120937-lin>
Precedence: bulk

On Fri, Nov 12, 2021 at 06:32:33PM +0000, Cristian Marussi wrote:
> Looking at the implementation of atomic_notifier_chain_register() and
> its internal helper down below, I can see that atomic_notifier_chain_register()
> ALWAYS return 0 (O_o) and anyway there is a WARN() in the notifiier core already
> to alert of double registrations.

You can ignore it - we agreed (at least no one has complained against it
yet :)) on doing this instead:

https://lore.kernel.org/r/YYk1xi3eJdMJdjHC@zn.tnic

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette