Re: [PATCH] selinux: reduce locking overhead in inode_free_security()

2015-06-12 Thread Stephen Smalley
On 06/10/2015 04:17 PM, Waiman Long wrote: > The inode_free_security() function just took the superblock's isec_lock > before checking and trying to remove the inode security struct from the > linked list. In many cases, the list was empty and so the lock taking > is wasteful as no useful work is d

Re: [PATCH] selinux: reduce locking overhead in inode_free_security()

2015-06-11 Thread Waiman Long
On 06/11/2015 08:38 AM, Stephen Smalley wrote: On 06/10/2015 04:17 PM, Waiman Long wrote: The inode_free_security() function just took the superblock's isec_lock before checking and trying to remove the inode security struct from the linked list. In many cases, the list was empty and so the lock

Re: [PATCH] selinux: reduce locking overhead in inode_free_security()

2015-06-11 Thread Stephen Smalley
On 06/10/2015 04:17 PM, Waiman Long wrote: > The inode_free_security() function just took the superblock's isec_lock > before checking and trying to remove the inode security struct from the > linked list. In many cases, the list was empty and so the lock taking > is wasteful as no useful work is d

[PATCH] selinux: reduce locking overhead in inode_free_security()

2015-06-10 Thread Waiman Long
The inode_free_security() function just took the superblock's isec_lock before checking and trying to remove the inode security struct from the linked list. In many cases, the list was empty and so the lock taking is wasteful as no useful work is done. On multi-socket systems with a large number of