Project

General

Profile

Actions
Copy link

Feature #47148

closed

mds: get rid of the mds_lock when storing the inode backtrace to meta pool

Added by Xiubo Li over 3 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Xiubo Li
Category:
-
Target version:
Ceph - v16.0.0
% Done:

0%

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

Description

The objector->mutate() may take a long time to finish. We can get rid of the mds_lock when doing this.

Related issues 2 (1 open — 1 closed)

Related to CephFS - Bug #47981: mds: count error of modified dentriesResolvedErqi Chen

Actions
Related to CephFS - Tasks #48620: mds: break the mds_lock or get rid of the mds_lock for some codeIn ProgressXiubo Li

Actions
Actions
Copy link
 #1

Updated by Xiubo Li over 3 years ago

  • Status changed from New to In Progress
Actions
Copy link
 #2

Updated by Patrick Donnelly over 3 years ago

  • Target version set to v16.0.0
  • Source set to Development
Actions
Copy link
 #3

Updated by Xiubo Li over 3 years ago

  • Status changed from In Progress to Fix Under Review
  • Pull request ID set to 36912
Actions
Copy link
 #4

Updated by Xiubo Li over 3 years ago

Currently will queue some encoding excepting the encodings which need to access the CDir/CInode members in the finisher. And also all the related mutate().

Actions
Copy link
 #5

Updated by Patrick Donnelly over 3 years ago

  • Status changed from Fix Under Review to Resolved
Actions
Copy link
 #6

Updated by Nathan Cutler over 3 years ago

  • Related to Bug #47981: mds: count error of modified dentries added
Actions
Copy link
 #7

Updated by Nathan Cutler over 3 years ago

Allegedly the fix for this issue introduced a regression, #47981.

That being the case, this fix should be backported together with the fix for #47981.

Actions
Copy link
 #8

Updated by Xiubo Li over 3 years ago

  • Related to Tasks #48620: mds: break the mds_lock or get rid of the mds_lock for some code added
Actions
Copy link

Also available in: Atom PDF