jewel: fs: client: nlink count is not maintained correctly #10877

Merged
merged 4 commits into from Oct 17, 2016

Projects

None yet

3 participants

jtlayton added some commits Aug 1, 2016
@jtlayton @ldachary jtlayton client: add mask parameter to _do_lookup
We need to allow callers to specify caps to acquire during a lookup, as
they may need to scrape certain info out of the inode later. Allow them
to pass in a mask. For now, _lookup just passes in 0 for the mask,
but verify_reply_trace passes in the regetattr_mask to match what we'd
request in the _getattr request if there were a traceless reply.

Signed-off-by: Jeff Layton <jlayton@redhat.com>
(cherry picked from commit a2ce16f)
b5cbd57
@jtlayton @ldachary jtlayton client: plumb a mask argument into _lookup
...and attempt to pass in a sane value there, based on what we'll do
with the resulting inode.

Signed-off-by: Jeff Layton <jlayton@redhat.com>
(cherry picked from commit f3605d3)
fb4a939
@jtlayton @ldachary jtlayton client: only skip querying the MDS in _lookup when we have the necess…
…ary caps

If we don't have the caps we'll need later, then we must reissue the
lookup to get them regardless of whether we have a lease on the
dentry.

Fixes: http://tracker.ceph.com/issues/16668
Signed-off-by: Jeff Layton <jlayton@redhat.com>
(cherry picked from commit 9e84767)
66cd43b
@jtlayton @ldachary jtlayton ceph: don't fudge the ctime in stat() unless it's really older than t…
…he mtime

The current code just looks at the sec field, but the difference may
only be evident in the nsec field.

Signed-off-by: Jeff Layton <jlayton@redhat.com>
(cherry picked from commit 14ee7bc)
c0db9fb
@dachary dachary self-assigned this Aug 25, 2016
@dachary dachary added this to the jewel milestone Aug 25, 2016
@dachary
Member
dachary commented Oct 14, 2016

@jcsp does this backport look good to merge ? It passed the fs suite http://tracker.ceph.com/issues/17487#note-13 (except for one known bug that is, I believe, unrelated)

@jcsp jcsp was assigned by dachary Oct 14, 2016
@jcsp
Contributor
jcsp commented Oct 17, 2016

👍

@dachary dachary merged commit b174220 into ceph:jewel Oct 17, 2016

2 checks passed

Signed-off-by all commits in this PR are signed
Details
default Build finished.
Details
@theanalyst theanalyst changed the title from jewel: client: nlink count is not maintained correctly to "jewel: client: nlink count is not maintained correctly" Nov 17, 2016
@theanalyst theanalyst changed the title from "jewel: client: nlink count is not maintained correctly" to jewel: client: nlink count is not maintained correctly Nov 17, 2016
@theanalyst theanalyst changed the title from jewel: client: nlink count is not maintained correctly to jewel: fs: client: nlink count is not maintained correctly Nov 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment