On Wed, Dec 6, 2023 at 1:07 AM Dave Chinner <[email protected]> wrote: > > From: Dave Chinner <[email protected]> > > Because it's a horrible point of lock contention under heavily > concurrent directory traversals... > > - 12.14% d_instantiate > - 12.06% security_d_instantiate > - 12.13% selinux_d_instantiate > - 12.16% inode_doinit_with_dentry > - 15.45% _raw_spin_lock > - do_raw_spin_lock > 14.68% __pv_queued_spin_lock_slowpath > > > Signed-off-by: Dave Chinner <[email protected]> > --- > include/linux/dlock-list.h | 9 ++++ > security/selinux/hooks.c | 72 +++++++++++++++---------------- > security/selinux/include/objsec.h | 6 +-- > 3 files changed, 47 insertions(+), 40 deletions(-)
In the cover letter you talk about testing, but I didn't see any mention of testing with SELinux enabled. Given the lock contention stats in the description above I'm going to assume you did test this and pass along my ACK, but if you haven't tested the changes below please do before sending this anywhere important. Acked-by: Paul Moore <[email protected]> -- paul-moore.com
