Bug 443681 - Fedora 9 kernel can't mount MyBook World Edition via CIFS
Summary: Fedora 9 kernel can't mount MyBook World Edition via CIFS
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: kernel
Version: 9
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jeff Layton
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-04-22 20:27 UTC by Chris Lalancette
Modified: 2014-06-18 07:37 UTC (History)
2 users (show)

Fixed In Version: 2.6.25.3-18.fc9
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-05-14 22:14:56 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
patch -- Fix UNC path prefix on QueryUnixPathInfo to have correct slash (2.85 KB, patch)
2008-04-22 20:46 UTC, Jeff Layton
no flags Details | Diff

Description Chris Lalancette 2008-04-22 20:27:26 UTC
Description of problem:
I recently purchased a MyBook World Edition drive.  It's an external storage
drive with both USB and 1Gbit ethernet; it exports it's disks as CIFS shares.

# smbclient -L //<ipaddress>

Domain=[MYBOOKWORLD] OS=[Unix] Server=[Samba 3.0.23c]

        Sharename       Type      Comment
        ---------       ----      -------
        PUBLIC          Disk      
        IPC$            IPC       IPC Service (MyBookWorld)
Domain=[MYBOOKWORLD] OS=[Unix] Server=[Samba 3.0.23c]

        Server               Comment
        ---------            -------

        Workgroup            Master
        ---------            -------
        WORKGROUP            BERMAN

Using both an F-7 or F-8 kernel, I was able to mount the share using the
following command:

# mount -t cifs -o credentials=/etc/cifs //<ipaddress>/public /mnt/tmp

However, this command no longer works in F-9, kernel 2.6.25-1.fc9.x86_64.  When
I try that same command in F-9, I get:

mount error 20 = Not a directory
Refer to the mount.cifs(8) manual page (e.g.man mount.cifs)

Jeff asked me to try mounting with -o nounix; adding that switch allows the
share to mount properly.

Comment 1 Jeff Layton 2008-04-22 20:46:44 UTC
Created attachment 303378 [details]
patch -- Fix UNC path prefix on QueryUnixPathInfo to have correct slash

This patch should fix it. It's not in mainline yet, but is in Steve French's
cifs-2.6 git tree. It should make its way to mainline soon (hopefully).

Comment 2 Chuck Ebbert 2008-04-23 11:16:42 UTC
The patch changes the function's parameters. Does it depend on some other change
in the CIFS GIT tree?

-static const unsigned char *cifs_get_search_path(struct cifsTconInfo *pTcon,
-					const char *search_path)
+static const unsigned char *cifs_get_search_path(struct cifs_sb_info *cifs_sb,
+						const char *search_path)


Comment 3 Jeff Layton 2008-04-23 11:42:48 UTC
It shouldn't. It applies cleanly to 2.6.25-1.fc9 for me. It changes that
function's parameters, but it also changes the callers.

As a side note, I've emailed Steve F to see about pushing this patch for 2.6.25
stable series. I think we'll probably want to carry this for Fedora for now, but
hopefully we can drop it eventually.


Comment 4 Chris Lalancette 2008-04-23 14:34:46 UTC
Yeah, and just to further what Jeff said, I've now built and booted a kernel
with this patch applied, and I can successfully mount (like I did with the F-8
kernel) without the -nounix option.

Chris Lalancette

Comment 5 Jeff Layton 2008-05-06 13:45:39 UTC
Chuck, is there anything we need to do to get this patch taken into F9 kernels
until it makes it upstream?


Comment 6 Chuck Ebbert 2008-05-13 08:17:52 UTC
Patch in 2.6.25.3-18.fc9

Comment 7 Fedora Update System 2008-05-13 14:27:58 UTC
kernel-2.6.25.3-18.fc9 has been submitted as an update for Fedora 9

Comment 8 Bug Zapper 2008-05-14 09:57:48 UTC
Changing version to '9' as part of upcoming Fedora 9 GA.
More information and reason for this action is here:
http://fedoraproject.org/wiki/BugZappers/HouseKeeping

Comment 9 Fedora Update System 2008-05-14 22:14:51 UTC
kernel-2.6.25.3-18.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.