Project

General

Profile

Actions

Bug #316

closed

mds: track dirty snapped metadata without faking a cap

Added by Sage Weil almost 14 years ago. Updated over 7 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
-
% Done:

0%

Source:
Tags:
Backport:
Regression:
Severity:
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
Labels (FS):
Pull request ID:
Crash signature (v1):
Crash signature (v2):
Actions #1

Updated by Sage Weil almost 14 years ago

  • Assignee set to Sage Weil
Actions #2

Updated by Sage Weil over 13 years ago

this illustrates the problem:

mkdir mnt/foo
echo asdf > mnt/foo/a
mkdir mnt/foo/.snap/s
ln mnt/foo/a mnt/foo/b
cat mnt/foo/.snap/s/a
echo "this will hang..." 
echo qwer >> mnt/foo/b   
echo "oh, it didn't hang! good job." 
rmdir mnt/foo/.snap/s

the 'ln' clones the cap (badness). the cat of the snapped version revokes the cloned cap, which makes the client state diverge from the mds state for the live cap. the append then hangs because the client doesn't have the w caps but the mds thinks it does.

Actions #3

Updated by Sage Weil over 13 years ago

ok, the cap tracking is done.

remaining issues:
- how to make an OPEN or STAT on the snapped inode trigger the head lock to sync.
- delayed cap writeback of dir attributes (needs to go into old_inodes)
- rdlock on snapped dir shouldn't succeed when in filelock is in EXCL state

Actions #4

Updated by Sage Weil over 13 years ago

incompat bit?

need a way to remove bad (in)compat bits from mdsmap compatset (via teh montior)

Actions #5

Updated by Sage Weil over 13 years ago

  • Status changed from New to Resolved
  • Target version changed from v0.21.1 to v0.22

see commit:ec91ea1db61290cd02a565be1d2076ceb74ba0c1

Actions #6

Updated by John Spray over 7 years ago

  • Project changed from Ceph to CephFS
  • Category deleted (1)
  • Target version deleted (v0.22)

Bulk updating project=ceph category=mds bugs so that I can remove the MDS category from the Ceph project to avoid confusion.

Actions

Also available in: Atom PDF