Project

General

Profile

Actions
Copy link

Bug #2919

closed

ceph kernel module looks for :/ in path, but / stripped by precise mountall

Added by Dan Mick over 11 years ago. Updated over 11 years ago.

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

0%

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

Description

I think this is really a bug in mountall (see https://bugs.launchpad.net/ubuntu/+source/mountall/+bug/809221), but it's affecting ceph and we might want to make ceph more forgiving, as
apparently nfs is:

in an fstab entry like

cephserver:port:/ /mnt ceph ....

/sbin/mountall will strip the trailing / from the first field, so that what the ceph
kernel module sees is

cephserver:port:

and it checks to see that the field (which represents the "device") ends in :/, which of
course it doesn't anymore.

This was apparently introduced by the fix for https://bugs.launchpad.net/ubuntu/+source/mountall/+bug/443035

Actions
Copy link
 #1

Updated by Dan Mick over 11 years ago

  • Assignee set to Alex Elder
Actions
Copy link
 #2

Updated by Sage Weil over 11 years ago

  • Status changed from New to Fix Under Review
Actions
Copy link
 #3

Updated by Dan Mick over 11 years ago

Was this not fixed?

Actions
Copy link
 #4

Updated by Sage Weil over 11 years ago

  • Status changed from Fix Under Review to Resolved

it's fixed in our testing branch, but i didn't send it to linus in for 3.6; it was late in the -rc cycle to be fixing userspace bugs in the kernel. it'll go upstream in the merge window.

we might want to tell the ubuntu kernel guys to apply it to any kernel they build... but then they are probably better off fixing mountall as well.

Actions
Copy link

Also available in: Atom PDF