diff options
author | Chengguang Xu <cgxu519@gmx.com> | 2018-08-31 10:33:48 -0400 |
---|---|---|
committer | Jan Kara <jack@suse.cz> | 2018-09-03 05:05:03 -0400 |
commit | 59fed3bf8a461ab07a8e57b42b6402e984c7cc62 (patch) | |
tree | b331155ab43493a904f0ff164430cc0416ee506f /fs/ext2/acl.c | |
parent | 849fe89ce6d7bb52d2903c79d2ff089d55e6ff8a (diff) |
ext2: cache NULL when both default_acl and acl are NULL
default_acl and acl of newly created inode will be initiated as
ACL_NOT_CACHED in vfs function inode_init_always() and later will be
updated by calling xxx_init_acl() in specific filesystems. However,
when default_acl and acl are NULL, then they keep the value of
ACL_NOT_CACHED. This patch changes the code to cache NULL for acl /
default_acl in this case to save unnecessary ACL lookup in future.
Signed-off-by: Chengguang Xu <cgxu519@gmx.com>
Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'fs/ext2/acl.c')
-rw-r--r-- | fs/ext2/acl.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/fs/ext2/acl.c b/fs/ext2/acl.c index 224c04abb2e5..cf4c77f8dd08 100644 --- a/fs/ext2/acl.c +++ b/fs/ext2/acl.c | |||
@@ -256,11 +256,15 @@ ext2_init_acl(struct inode *inode, struct inode *dir) | |||
256 | if (default_acl) { | 256 | if (default_acl) { |
257 | error = __ext2_set_acl(inode, default_acl, ACL_TYPE_DEFAULT); | 257 | error = __ext2_set_acl(inode, default_acl, ACL_TYPE_DEFAULT); |
258 | posix_acl_release(default_acl); | 258 | posix_acl_release(default_acl); |
259 | } else { | ||
260 | inode->i_default_acl = NULL; | ||
259 | } | 261 | } |
260 | if (acl) { | 262 | if (acl) { |
261 | if (!error) | 263 | if (!error) |
262 | error = __ext2_set_acl(inode, acl, ACL_TYPE_ACCESS); | 264 | error = __ext2_set_acl(inode, acl, ACL_TYPE_ACCESS); |
263 | posix_acl_release(acl); | 265 | posix_acl_release(acl); |
266 | } else { | ||
267 | inode->i_acl = NULL; | ||
264 | } | 268 | } |
265 | return error; | 269 | return error; |
266 | } | 270 | } |