Project

General

Profile

Bug #3186

mds: mds crash on client shutdown

Added by Sam Lang over 11 years ago. Updated over 7 years ago.

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

0%

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

Description

With this libcephfs test: https://gist.github.com/3762534

The first mds crashes during shutdown. The ceph cluster is configured with 3 mds servers (the default for vstart.sh).

mds log: https://gist.github.com/3762542
client log: https://gist.github.com/3762520
config: https://gist.github.com/3762555

Associated revisions

Revision 2b543918 (diff)
Added by Sam Lang over 11 years ago

client: Fix to client filepath initializing

The filepath constructor that takes a const char * is missing the
initializer for the encoded member. This results in uninitialized
memory, so the encoded field is sometimes true, resulting in mds
crashes (see #3186) and client errors with empty components in path
names. This commit fixes #3186 and #2285.

Signed-off-by: Sam Lang <>

History

#1 Updated by Sam Lang over 11 years ago

Just verified the same error occurs with one mds.

#2 Updated by Sage Weil over 11 years ago

can you reproduce with debug mds = 20 ?

#3 Updated by Sam Lang over 11 years ago

Verified that this is only an issue with an empty component in the path (related to #2285).

#4 Updated by Sam Lang over 11 years ago

The getattr segfault is actually from the second mount call in the client test.

#5 Updated by Sam Lang over 11 years ago

  • Status changed from New to Fix Under Review

Proposed fix in wip-3186, specifically c92e1dd7b0106fa986985d8179a8ed66c61e6709. Needs review.

#6 Updated by Sage Weil over 11 years ago

  • Status changed from Fix Under Review to Resolved

#7 Updated by John Spray over 7 years ago

  • Project changed from Ceph to CephFS
  • Category deleted (1)

Bulk updating project=ceph category=mds bugs so that I can remove the MDS category from the Ceph project to avoid confusion.

Also available in: Atom PDF