Project

General

Profile

Actions

Bug #4105

closed

mds: fix up the Dumper

Added by Greg Farnum about 11 years ago. Updated almost 8 years ago.

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

0%

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

Description

The messenger/objecter locking is wrong, and my quick stab at a fix resulted in lockdep warnings and things. Spend some time getting this into shape so we don't need to manually grab mds log objects!

Actions #1

Updated by Greg Farnum about 11 years ago

  • Subject changed from mds: Dumper asserts due to Objecter::client_lock not being held to mds: Dumper is broken
  • Description updated (diff)
  • Status changed from Fix Under Review to 12
Actions #2

Updated by Greg Farnum about 11 years ago

  • Tracker changed from Bug to Feature
  • Subject changed from mds: Dumper is broken to mds: fix up the Dumper
Actions #3

Updated by Greg Farnum about 11 years ago

  • Translation missing: en.field_position set to 38
Actions #4

Updated by Greg Farnum about 11 years ago

  • Priority changed from Normal to High

Promoting this to high as it can be so useful for gathering important debug data; it would be nice to have done before Cuttlefish.

Actions #5

Updated by Greg Farnum about 11 years ago

  • Tracker changed from Feature to Bug
Actions #6

Updated by Greg Farnum almost 11 years ago

  • Assignee set to Greg Farnum

This has annoyed me a couple more times and I think it's now at the top of the queue, so here we go again.

Actions #7

Updated by Greg Farnum almost 11 years ago

  • Status changed from 12 to Fix Under Review

wip-4105-mds-dumper

Wasn't actually that complicated; it's just the locking expectations around the Objecter changed a fair bit (or maybe are just better enforced? Anyway, there was bitrot).

Actions #8

Updated by Greg Farnum almost 11 years ago

  • Status changed from Fix Under Review to Resolved

Merged into next in commit:dfacd1bd805ebb730b5206c9830b28f47cc7f9cf. Hurray!

Actions #9

Updated by Greg Farnum almost 8 years ago

  • Component(FS) MDS added
Actions

Also available in: Atom PDF