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

2015-06-18 Thread Paul Moore
On Monday, June 15, 2015 01:13:39 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 use

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

2015-06-16 Thread Stephen Smalley
On 06/15/2015 01:13 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 v3] selinux: reduce locking overhead in inode_free_security()

2015-06-15 Thread Yury
On 15.06.2015 20:13, 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 done. On m

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

2015-06-15 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