Project

General

Profile

Fix #47983

mds: use proper gather for inode commit ops

Added by Erqi Chen 3 months ago. Updated 2 months ago.

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

0%

Source:
Community (dev)
Tags:
Backport:
octopus,nautilus
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
MDS
Labels (FS):
Pull request ID:
Crash signature:

Description

Improvement for CInode::_commit_ops.

MDS journal handles inodes concurrently by CInode::_commit_ops, fin is C_Gather, so no need to create new C_OnFinisher in gather.

And fin can be Context rather than MDSContext to avoid unnecessary MDS lock.

History

#1 Updated by Patrick Donnelly 3 months ago

  • Status changed from New to Fix Under Review
  • Assignee set to Erqi Chen
  • Target version set to v16.0.0
  • Source set to Community (dev)
  • Backport set to octopus,nautilus

#2 Updated by Erqi Chen 3 months ago

New idea to optimize by pr #37828 optimize batch backtrace store,old pr #37778 is closed and this issue also can be close.

#37828 is small optimization, no issue related.

#3 Updated by Erqi Chen 3 months ago

Erqi Chen wrote:

New idea to optimize by pr 37828 optimize batch backtrace store,old pr 37778 is closed and this issue also can be close.

37828 is small optimization, no issue related.

#4 Updated by Patrick Donnelly 2 months ago

  • Status changed from Fix Under Review to Closed

Also available in: Atom PDF