Commit 844fa1b5 authored by Dave Kleikamp's avatar Dave Kleikamp

jfs: set i_ctime when setting ACL

This fixes a regression in 3.14-rc1 where xfstests generic/307 fails.

jfs sets the ctime on the inode when writing an xattr. Previously,
jfs went ahead and stored an acl that can be completely represented
in the traditional permission bits, so the ctime was always set in
the xattr code. The new code doesn't bother storing the acl in that
case, thus the ctime isn't getting set.
Signed-off-by: default avatarDave Kleikamp <dave.kleikamp@oracle.com>
Reported-by: default avatarMichael L. Semon <mlsemon35@gmail.com>
parent c18f7b51
...@@ -86,6 +86,8 @@ static int __jfs_set_acl(tid_t tid, struct inode *inode, int type, ...@@ -86,6 +86,8 @@ static int __jfs_set_acl(tid_t tid, struct inode *inode, int type,
rc = posix_acl_equiv_mode(acl, &inode->i_mode); rc = posix_acl_equiv_mode(acl, &inode->i_mode);
if (rc < 0) if (rc < 0)
return rc; return rc;
inode->i_ctime = CURRENT_TIME;
mark_inode_dirty(inode);
if (rc == 0) if (rc == 0)
acl = NULL; acl = NULL;
break; break;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment