On Wed, Jul 10, 2024 at 7:15 PM KP Singh [off-list ref] wrote:

On Wed, Jul 10, 2024 at 10:41 PM Paul Moore [off-list ref] wrote:

quoted

On Jul  9, 2024 KP Singh [off-list ref] wrote:

...

quoted

quoted

A static key guards whether an LSM static call is enabled or not,
without this static key, for LSM hooks that return an int, the presence
of the hook that returns a default value can create side-effects which
has resulted in bugs [1].

I don't want to rehash our previous discussions on this topic, but I do
think we either need to simply delete the paragraph above or update it
to indicate that all known side effects involving LSM callback return
values have been addressed.  Removal is likely easier if for no other
reason than we don't have to go back and forth with edits, but I can

Agreed, we can just delete this paragraph. Thanks!

Okay, I'll do that.  I'll send another note when it is merged into
lsm/dev, but as I said earlier, that is likely a few weeks out.  This
will likely end up in lsm/dev-staging before that for testing, etc.

-- 
paul-moore.com