Project

General

Profile

Actions

Feature #630

closed

release caps on inodes unlinked by other clients

Added by Sage Weil over 13 years ago. Updated over 9 years ago.

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

0%

Source:
Tags:
Backport:
Reviewed:
Affected Versions:
Component(FS):
Labels (FS):
Pull request ID:

Description

If client A writes a file, and client B unlinks it, client A needs to drop the inode sooner rather than later.

Or, the MDS needs to delete file data as soon as a stray's wanted drops to 0.

Actions #1

Updated by Sage Weil over 13 years ago

  • Translation missing: en.field_position set to 1
Actions #2

Updated by Sage Weil over 13 years ago

Sage Weil wrote:

Or, the MDS needs to delete file data as soon as a stray's wanted drops to 0.

That won't work, as the client is allowed to reopen a file and add to wanted if it holds the caps.

I think we need an additional cap message that's a hint to the client to drop the inode asap. Or, just a normal cap message that update's the client's nlink to 0 so that it knows it should drop it..

Actions #3

Updated by Sage Weil about 13 years ago

  • Assignee set to Yehuda Sadeh

I think sending a client_caps to (other) clients with caps notifying them of nlink==0 is the way to do this without changing the protocol. Then the client can try to drop the inode('s dentry) immediately if that happens.

Actions #4

Updated by Sage Weil about 13 years ago

  • Assignee changed from Yehuda Sadeh to Greg Farnum
Actions #5

Updated by Sage Weil about 13 years ago

  • Target version changed from v0.25 to 19
Actions #6

Updated by Greg Farnum about 13 years ago

  • Assignee deleted (Greg Farnum)

Putting this back in the queue since we've pushed it back past 1.0.

Actions #7

Updated by Sage Weil over 12 years ago

  • Target version deleted (19)
Actions #8

Updated by Sage Weil over 12 years ago

  • Translation missing: en.field_position deleted (185)
  • Translation missing: en.field_position set to 738
Actions #9

Updated by Sage Weil about 12 years ago

  • Tracker changed from Bug to Feature
Actions #10

Updated by Loïc Dachary over 9 years ago

  • Project changed from Ceph to CephFS
Actions #11

Updated by Zheng Yan over 9 years ago

dup of #5039. already fixed by commit f8a947d92 client: trim deleted inode

Actions #12

Updated by Zheng Yan over 9 years ago

  • Status changed from New to Resolved
Actions

Also available in: Atom PDF