Skip to content

Commit

Permalink
Merge pull request ceph#18786 from taodd/master
Browse files Browse the repository at this point in the history
Client: setattr should drop "Fs" rather than "As" for mtime and size
  • Loading branch information
ukernel authored Nov 19, 2017
2 parents d4c9609 + e1c57f9 commit b730db7
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/client/Client.cc
Original file line number Diff line number Diff line change
Expand Up @@ -6769,7 +6769,7 @@ int Client::_do_setattr(Inode *in, struct ceph_statx *stx, int mask,
}
if (mask & CEPH_SETATTR_MTIME) {
req->head.args.setattr.mtime = utime_t(stx->stx_mtime);
req->inode_drop |= CEPH_CAP_AUTH_SHARED | CEPH_CAP_FILE_RD |
req->inode_drop |= CEPH_CAP_FILE_SHARED | CEPH_CAP_FILE_RD |
CEPH_CAP_FILE_WR;
}
if (mask & CEPH_SETATTR_ATIME) {
Expand All @@ -6786,7 +6786,7 @@ int Client::_do_setattr(Inode *in, struct ceph_statx *stx, int mask,
ldout(cct,10) << "unable to set size to " << stx->stx_size << ". Too large!" << dendl;
return -EFBIG;
}
req->inode_drop |= CEPH_CAP_AUTH_SHARED | CEPH_CAP_FILE_RD |
req->inode_drop |= CEPH_CAP_FILE_SHARED | CEPH_CAP_FILE_RD |
CEPH_CAP_FILE_WR;
}
req->head.args.setattr.mask = mask;
Expand Down

0 comments on commit b730db7

Please sign in to comment.