From: Jan Rekorajski <baggins@sith.mimuw.edu.pl>
To: Valerie Aurora <vaurora@redhat.com>
Cc: Jan Blunck <jblunck@suse.de>,
linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org,
viro@zeniv.linux.org.uk, bharata@in.ibm.com, dwmw2@infradead.org,
mszeredi@suse.cz
Subject: Re: [PATCH 00/32] VFS based Union Mount (V3)
Date: Mon, 15 Jun 2009 11:55:28 +0200 [thread overview]
Message-ID: <20090615095528.GG4379@sith.mimuw.edu.pl> (raw)
In-Reply-To: <20090608224831.GF24586@fsbox>
On Mon, 08 Jun 2009, Valerie Aurora wrote:
> On Tue, Jun 09, 2009 at 12:44:06AM +0200, Jan Rekorajski wrote:
> > On Mon, 08 Jun 2009, Valerie Aurora wrote:
> >
> > > You'll get -EBUSY on the second mount of any NFS file system over
> > > another - try it again with the --union flag. Support for NFS on NFS
> > > union mount would have to change this.
> >
> > I did just that, --union didn't change standard NFS behaviour.
>
> Er, excuse me - I mean to type "try it again WITHOUT the --union
> flag." My apologies!
I did, see below.
> >
> > mount -t nfs server:/export/system /mnt
> > mount --union -t nfs server:/export/profile /mnt
> > mount.nfs: /mnt is busy or already mounted
> >
> > I did an experiment by using different IP of the server (same machine)
> > when mounting the second fs, mount worked then, but 'ls -1 /mnt' oopsed.
> > I can reproduce this and send you the oops next week.
>
> Interesting! Does this happen without the --union flag?
Filesystems are exported ro, all mounts nfs3,ro,tcp.
Without --union:
mount -t nfs 10.1.0.4:/nfs/system /mnt -oro,nolock,vers=3,tcp
OK
mount -t nfs 10.1.0.4:/nfs/profile /mnt -oro,nolock,vers=3,tcp
-EBUSY (as expected)
mount -t nfs 10.1.0.3:/nfs/profile /mnt -oro,nolock,vers=3,tcp
(notice different IP - but it's the same machine)
Works, just overmounts /mnt, ls shown contents of /nfs/profile
Now, --union:
mount -t nfs 10.1.0.4:/nfs/system /mnt -oro,nolock,vers=3,tcp
OK
mount --union -t nfs 10.1.0.4:/nfs/profile /mnt -oro,nolock,vers=3,tcp
-EBUSY (/mnt busy or already mounted)
mount --union -t nfs 10.1.0.3:/nfs/profile /mnt -oro,nolock,vers=3,tcp
(notice different IP - but it's the same machine)
mount command works, ls Oopses:
[ 61.766392] creating fallthru for opt
[ 61.766417] BUG: unable to handle kernel NULL pointer dereference at (null)
[ 61.766433] IP: [<(null)>] (null)
[ 61.766482] *pdpt = 000000001e0e9001 *pde = 0000000000000000
[ 61.767324] Oops: 0010 [#1] PREEMPT SMP
[ 61.767324] last sysfs file: /sys/kernel/uevent_seqnum
[ 61.767324] Modules linked in: nfs nfsd lockd nfs_acl auth_rpcgss sunrpc sch_sfq thermal processor thermal_sys rtc_cmos rtc_core i2c_piix4 e1000 ac psmouse rtc_lib hwmon pcspkr button i2c_core sg sr_mod serio_raw evdev cdrom
[ 61.767324]
[ 61.767324] Pid: 2460, comm: ls Not tainted (2.6.29.3 #1) VirtualBox
[ 61.767324] EIP: 0060:[<00000000>] EFLAGS: 00010286 CPU: 0
[ 61.767324] EIP is at 0x0
[ 61.767324] EAX: df6d23c0 EBX: df69d1b0 ECX: e105d3e0 EDX: df69d630
[ 61.767324] ESI: df69d630 EDI: de16d300 EBP: de1b1d18 ESP: de1b1d04
[ 61.767324] DS: 007b ES: 007b FS: 00d8 GS: 0033 SS: 0069
[ 61.767324] Process ls (pid: 2460, ti=de1b0000 task=de0ee3f0 task.ti=de1b0000)
[ 61.767324] Stack:
[ 61.767324] c02d03d9 c054f57d df69d698 00000002 00000000 de1b1d7c e10357f0 00000002
[ 61.767324] 00000000 0180677d 00000000 0000000a c02d0340 df69d1b0 00000000 00000002
[ 61.767324] 0180677d 00000000 df6d2128 de16d300 de1b1f08 de1b1f08 df69d5a0 de1b1ed4
[ 61.767324] Call Trace:
[ 61.767324] [<c02d03d9>] union_copyup_dir_one+0x99/0xc0
[ 61.767324] [<e10357f0>] nfs_do_filldir+0x210/0x570 [nfs]
[ 61.767324] [<c02d0340>] union_copyup_dir_one+0x0/0xc0
[ 61.767324] [<e1036072>] nfs_readdir+0x522/0xa10 [nfs]
[ 61.767324] [<c02d0340>] union_copyup_dir_one+0x0/0xc0
[ 61.767324] [<c0293502>] __mem_cgroup_commit_charge+0x42/0x100
[ 61.767324] [<c0293c07>] mem_cgroup_charge_common+0x57/0x70
[ 61.767324] [<e1039f18>] __put_nfs_open_context+0x28/0xb0 [nfs]
[ 61.767324] [<e0a6fbe2>] rpcauth_lookup_credcache+0x152/0x1f0 [sunrpc]
[ 61.767324] [<e0a6f92d>] rpcauth_lookupcred+0x5d/0xb0 [sunrpc]
[ 61.767324] [<e1049c10>] nfs3_decode_dirent+0x0/0x220 [nfs]
[ 61.767324] [<c02d0ad1>] union_copyup_dir+0x111/0x160
[ 61.767324] [<c02a6190>] filldir64+0x0/0x110
[ 61.767324] [<c02a6575>] vfs_readdir+0xd5/0xf0
[ 61.767324] [<c02a65fd>] sys_getdents64+0x6d/0xc0
[ 61.767324] [<c02033da>] syscall_call+0x7/0xb
[ 61.767324] Code: Bad EIP value.
[ 61.767324] EIP: [<00000000>] 0x0 SS:ESP 0068:de1b1d04
[ 61.816597] ---[ end trace 60fb13bae2f23426 ]---
--
Jan Rekorajski | ALL SUSPECTS ARE GUILTY. PERIOD!
baggins<at>mimuw.edu.pl | OTHERWISE THEY WOULDN'T BE SUSPECTS, WOULD THEY?
BOFH, MANIAC | -- TROOPS by Kevin Rubio
next prev parent reply other threads:[~2009-06-15 9:55 UTC|newest]
Thread overview: 68+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-05-18 16:08 [PATCH 00/32] VFS based Union Mount (V3) Jan Blunck
2009-05-18 16:08 ` [PATCH 01/32] atomic: Only take lock when the counter drops to zero on UP as well Jan Blunck
2009-05-18 16:08 ` [PATCH 02/32] VFS: BUG() if somebody tries to rehash an already hashed dentry Jan Blunck
2009-05-18 16:08 ` [PATCH 03/32] VFS: propagate mnt_flags into do_loopback Jan Blunck
2009-05-18 16:09 ` [PATCH 04/32] VFS: Make lookup_hash() return a struct path Jan Blunck
2009-05-18 16:09 ` [PATCH 05/32] VFS: Remove unnecessary micro-optimization in cached_lookup() Jan Blunck
2009-05-18 16:09 ` [PATCH 06/32] VFS: Make real_lookup() return a struct path Jan Blunck
2009-05-18 16:09 ` [PATCH 07/32] VFS: Introduce dput() variant that maintains a kill-list Jan Blunck
2009-05-18 16:09 ` [PATCH 08/32] whiteout: Don't return information about whiteouts to userspace Jan Blunck
2009-05-18 16:09 ` [PATCH 09/32] whiteout: Add vfs_whiteout() and whiteout inode operation Jan Blunck
2009-05-18 16:09 ` [PATCH 10/32] whiteout: Set S_OPAQUE inode flag when creating directories Jan Blunck
2009-05-18 16:09 ` [PATCH 11/32] whiteout: Add whiteout support to tmpfs Jan Blunck
2009-05-18 16:09 ` [PATCH 12/32] whiteout: Split of ext2_append_link() from ext2_add_link() Jan Blunck
2009-05-18 16:09 ` [PATCH 13/32] whiteout: Add whiteout support to ext2 Jan Blunck
2009-05-18 16:09 ` [PATCH 14/32] whiteout: Add path_whiteout() helper Jan Blunck
2009-05-18 16:09 ` [PATCH 15/32] union-mount: Documentation Jan Blunck
2009-05-25 6:25 ` hooanon05
2009-05-25 8:03 ` Arnd Bergmann
2009-05-25 8:43 ` hooanon05
2009-06-18 19:05 ` Valerie Aurora
2009-06-19 1:53 ` hooanon05
2009-05-18 16:09 ` [PATCH 16/32] union-mount: Introduce MNT_UNION and MS_UNION flags Jan Blunck
2009-05-18 16:09 ` [PATCH 17/32] union-mount: Introduce union_mount structure Jan Blunck
2009-05-18 16:09 ` [PATCH 18/32] union-mount: Drive the union cache via dcache Jan Blunck
2009-05-18 16:09 ` [PATCH 19/32] union-mount: Some checks during namespace changes Jan Blunck
2009-05-18 16:09 ` [PATCH 20/32] union-mount: Changes to the namespace handling Jan Blunck
2009-05-18 16:09 ` [PATCH 21/32] union-mount: Make lookup work for union-mounted file systems Jan Blunck
2009-05-19 16:15 ` Miklos Szeredi
2009-05-19 17:30 ` Valerie Aurora
2009-05-20 10:21 ` Miklos Szeredi
2009-05-18 16:09 ` [PATCH 22/32] union-mount: stop lookup when directory has S_OPAQUE flag set Jan Blunck
2009-05-18 16:09 ` [PATCH 23/32] union-mount: stop lookup when finding a whiteout Jan Blunck
2009-05-18 16:09 ` [PATCH 24/32] union-mount: in-kernel file copy between union mounted filesystems Jan Blunck
2009-05-18 16:09 ` [PATCH 25/32] union-mount: check for logically empty directory (FIXME) Jan Blunck
2009-05-18 16:09 ` [PATCH 26/32] union-mount: call do_whiteout() on unlink and rmdir Jan Blunck
2009-05-18 16:09 ` [PATCH 27/32] union-mount: Always create topmost directory on open Jan Blunck
2009-05-18 16:09 ` [PATCH 28/32] union-mount: Basic fallthru definitions Jan Blunck
2009-05-18 16:09 ` [PATCH 29/32] union mount: Support for fallthru entries in union mount lookup Jan Blunck
2009-05-18 16:09 ` [PATCH 30/32] union mount: ext2 fallthru support Jan Blunck
2009-05-18 16:32 ` Andreas Dilger
2009-05-19 9:42 ` Jan Blunck
2009-05-19 14:05 ` Andreas Dilger
2009-05-19 16:13 ` Jan Blunck
2009-05-18 16:09 ` [PATCH 31/32] union-mount: tmpfs " Jan Blunck
2009-05-18 16:09 ` [PATCH 32/32] union-mount: Copy up directory entries on first readdir() Jan Blunck
2009-05-18 20:40 ` [PATCH] Userland for VFS based Union Mount (V3) Valerie Aurora
2009-05-21 13:53 ` Andreas Dilger
2009-06-18 3:22 ` Valerie Aurora
2009-05-19 9:48 ` [PATCH 00/32] " Miklos Szeredi
2009-05-19 10:29 ` Jan Blunck
2009-05-19 10:35 ` Miklos Szeredi
2009-05-19 10:39 ` Jan Blunck
2009-05-19 11:54 ` Arnd Bergmann
2009-05-19 12:15 ` Jan Blunck
2009-05-19 12:21 ` Arnd Bergmann
2009-05-19 13:10 ` Jan Blunck
2009-05-19 17:23 ` Valerie Aurora
2009-05-20 9:05 ` Miklos Szeredi
2009-06-08 19:44 ` Valerie Aurora
2009-06-16 15:19 ` Miklos Szeredi
2009-05-21 12:54 ` Jan Rekorajski
2009-06-08 19:57 ` Valerie Aurora
2009-06-08 22:44 ` Jan Rekorajski
2009-06-08 22:48 ` Valerie Aurora
2009-06-15 9:55 ` Jan Rekorajski [this message]
2009-06-18 3:23 ` Valerie Aurora
2009-06-04 11:38 ` Scott James Remnant
2009-06-09 22:15 ` Valerie Aurora
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20090615095528.GG4379@sith.mimuw.edu.pl \
--to=baggins@sith.mimuw.edu.pl \
--cc=bharata@in.ibm.com \
--cc=dwmw2@infradead.org \
--cc=jblunck@suse.de \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mszeredi@suse.cz \
--cc=vaurora@redhat.com \
--cc=viro@zeniv.linux.org.uk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).