On Thu, Jul 11, 2024 at 9:59 AM Paul Moore <paul@xxxxxxxxxxxxxx> wrote:
> On Wed, Jul 10, 2024 at 7:15 PM KP Singh <kpsingh@xxxxxxxxxx> wrote:
> > On Wed, Jul 10, 2024 at 10:41 PM Paul Moore <paul@xxxxxxxxxxxxxx> wrote:
> > > On Jul 9, 2024 KP Singh <kpsingh@xxxxxxxxxx> wrote:
>
> ...
>
> > > > 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.
Quick follow-up that these patches are now in lsm/dev-staging, I'll
send another note when they are merged into lsm/{dev,next}.
--
paul-moore.com
