On Wed, Nov 04, 2020 at 12:07:16PM -0800, Eric Biggers wrote:
> On Sun, Oct 11, 2020 at 01:29:33AM -0700, Lokesh Gidra wrote:
> > Daniel Colascione (3):
> >   Add a new LSM-supporting anonymous inode interface
> >   Teach SELinux about anonymous inodes
> >   Use secure anon inodes for userfaultfd
> 
> Patches are supposed to have subsystem prefixes, e.g.
> 
>       fs, security: add a new LSM-supporting anonymous inode interface
>       selinux: implement init_security_anon()
>       userfaultfd: use secure anon inodes
> 
> ... but that points to the fact that the first one is really both fs and
> security subsystem changes.  Patches should be one logical change only.  I
> suggest splitting it up into:
> 
>       security: add init_security_anon() LSM hook
>       fs: add anon_inode_getfd_secure()

Correction: it's "inode_init_security_anon()", not "init_security_anon()".

- Eric

Reply via email to