Project

General

Profile

Actions
Copy link

Cleanup #588

closed

Allow Inodes to have multiple parent Dentries

Added by Greg Farnum over 13 years ago. Updated over 7 years ago.

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

0%

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

Description

Right now, cached Inodes can only have one parent Dentry. This is unfortunate when there are multiple hard links to a single inode -- see #583.

Re-work the caching to allow Inodes to have multiple parents, and the necessary functions to work with that.

Actions
Copy link
 #1

Updated by Sage Weil over 13 years ago

  • Target version set to v0.25
Actions
Copy link
 #2

Updated by Sage Weil over 13 years ago

  • Source changed from 0 to 2
Actions
Copy link
 #3

Updated by Sage Weil over 13 years ago

  • Translation missing: en.field_position set to 3
Actions
Copy link
 #4

Updated by Sage Weil over 13 years ago

  • Translation missing: en.field_position deleted (5)
  • Translation missing: en.field_position set to 6
Actions
Copy link
 #5

Updated by Sage Weil over 13 years ago

  • Translation missing: en.field_position deleted (6)
  • Translation missing: en.field_position set to 371
Actions
Copy link
 #6

Updated by Sage Weil over 13 years ago

  • Assignee set to Greg Farnum
Actions
Copy link
 #7

Updated by Sage Weil over 13 years ago

  • Translation missing: en.field_position deleted (372)
  • Translation missing: en.field_position set to 309
Actions
Copy link
 #8

Updated by Greg Farnum over 13 years ago

  • Status changed from New to 7

Pushed the branch uclient_dentries; it's only a few commits.

There were actually surprisingly few things that needed changes that I saw, and not many invariants about dentries and inodes either!

This has passed pjd and snaptest-0/1. Still in the process of testing other stuff.

Actions
Copy link
 #9

Updated by Greg Farnum over 13 years ago

When the client changes are merged into testing it also passes snaptest-upchildrealms and snaptest-2. I suspect this is good to go!

Actions
Copy link
 #10

Updated by Greg Farnum over 13 years ago

  • Status changed from 7 to In Progress

Okay, going to swap this with an implementation using sets rather than lists, then done!

Actions
Copy link
 #11

Updated by Greg Farnum over 13 years ago

  • Status changed from In Progress to Resolved

Merged to unstable in commit:b6271764cda3652d7fb0e3472732ae31c1adbbe9.

Actions
Copy link
 #12

Updated by John Spray over 7 years ago

  • Project changed from Ceph to CephFS
  • Category deleted (11)
  • Target version deleted (v0.25)

Bulk updating project=ceph category=ceph-fuse issues to move to fs project so that we can remove the ceph-fuse category from the ceph project

Actions
Copy link

Also available in: Atom PDF