Re: [PATCH] staging: lustre: llite: drop acl from cache

2016-05-24 Thread Andreas Gruenbacher
On Tue, May 24, 2016 at 10:38 PM, James Simmons wrote: >> 2016-05-24 2:35 GMT+02:00 James Simmons : >> > Commit b8a7a3a6 change get_acl() for posix xattr to always cache >> > the ACL which increases the reference count. That reference count >> > can be reduced by have ll_get_acl() call forget_cach

Re: [PATCH] staging: lustre: llite: drop acl from cache

2016-05-24 Thread James Simmons
> 2016-05-24 2:35 GMT+02:00 James Simmons : > > Commit b8a7a3a6 change get_acl() for posix xattr to always cache > > the ACL which increases the reference count. That reference count > > can be reduced by have ll_get_acl() call forget_cached_acl() which > > it wasn't. When an inode gets deleted by

Re: [PATCH] staging: lustre: llite: drop acl from cache

2016-05-23 Thread Andreas Grünbacher
2016-05-24 2:35 GMT+02:00 James Simmons : > Commit b8a7a3a6 change get_acl() for posix xattr to always cache > the ACL which increases the reference count. That reference count > can be reduced by have ll_get_acl() call forget_cached_acl() which > it wasn't. When an inode gets deleted by Lustre the

[PATCH] staging: lustre: llite: drop acl from cache

2016-05-23 Thread James Simmons
Commit b8a7a3a6 change get_acl() for posix xattr to always cache the ACL which increases the reference count. That reference count can be reduced by have ll_get_acl() call forget_cached_acl() which it wasn't. When an inode gets deleted by Lustre the POSIX ACL reference count is tested to ensure its