Re: BUG when doing parallel NFS mounts (WAS: Re: Merge window closed: v2.6.19-rc1)

Trond Myklebust
2006-10-08T18:36:07Z

[-- Attachment #1: Type: text/plain, Size: 1916 bytes --]

On Sun, 2006-10-08 at 17:19 +0200, Peter Osterlund wrote:
> > ------------[ cut here ]------------
> > kernel BUG at fs/nfs/client.c:352!
> > invalid opcode: 0000 [#1]
> > PREEMPT
> > Modules linked in: nfs radeon nfsd exportfs lockd autofs4 sunrpc michael_mic arc4 ecb blkcipher cryptomgr crypto_algapi ieee80211_crypt_tkip binfmt_misc dm_mirror dm_mod lp parport_pc parport pcmcia ehci_hcd joydev ohci1394 ieee1394 ohci_hcd bcm43xx yenta_socket snd_atiixp_modem rsrc_nonstatic ide_cd ieee80211softmac ieee80211 serio_raw 8250_pci pcmcia_core 8250 ieee80211_crypt pcspkr 8139too serial_core cdrom rtc usbcore
> > CPU: 0
> > EIP: 0060:[<e0c836b5>] Not tainted VLI
> > EFLAGS: 00010202 (2.6.19-rc1 #53)
> > EIP is at nfs_create_server+0x9f1/0xa6e [nfs]
> > eax: 00000000 ebx: d5e11b40 ecx: ca01a000 edx: 00000000
> > esi: 00000001 edi: ca01bdac ebp: ca01bd08 esp: ca01bbf8
> > ds: 007b es: 007b ss: 0068
> > Process mount (pid: 4052, ti=ca01a000 task=ca078030 task.ti=ca01a000)
> > Stack: ca01bc58 c1405c68 ca01bc2c c1405c40 c1405c40 00000296 c05cc5d0 ca078030
> > ca07857c d6b03380 00000003 ca97704c ca97705c ca078030 00000000 ca01bc8c
> > c0135211 c01eea52 ca078570 00000246 ca01bc6c 00000246 00000002 ca845cb0
> > Call Trace:
> > [<e0c899b2>] nfs_get_sb+0xbb/0x531 [nfs]
> > [<c0166416>] vfs_kern_mount+0x42/0x8c
> > [<c01664c4>] do_kern_mount+0x3b/0x4e
> > [<c017a8ca>] do_mount+0x25a/0x6a5
> > [<c017adaa>] sys_mount+0x95/0xd4
> > [<c01030c1>] sysenter_past_esp+0x56/0x8d
> > [<b7fa9410>] 0xb7fa9410
> > =======================
> > Code: 85 28 ff ff ff 89 44 24 04 c7 04 24 00 61 c9 e0 e8 16 8e 49 df 8b 9d 28 ff ff ff 31 d2 81 fb 00 f0 ff ff 0f 97 c2 e9 62 f9 ff ff <0f> 0b 60 01 40 50 c9 e0 e9 34 f9 ff ff 81 fa 60 ea 00 00 0f 86
> > EIP: [<e0c836b5>] nfs_create_server+0x9f1/0xa6e [nfs] SS:ESP 0068:ca01bbf8

Does the following patch fix it?

Cheers,
Trond

[-- Attachment #2: linux-2.6.19-002-fix_nfs_get_client.dif --]
[-- Type: message/rfc822, Size: 913 bytes --]

From: Trond Myklebust <Trond.Myklebust@netapp.com>
Subject: NFS: Fix typo in nfs_get_client()
Date: Sun, 8 Oct 2006 14:33:24 -0400
Message-ID: <1160332545.11233.4.camel@lade.trondhjem.org>

NFS_CS_INITING > NFS_CS_READY, so instead of waiting for the structure to
get initialised, we currently immediately jump out of the loop without ever
sleeping.

Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
---

fs/nfs/client.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/fs/nfs/client.c b/fs/nfs/client.c
index 6e4e48c..d2533e2 100644
--- a/fs/nfs/client.c
+++ b/fs/nfs/client.c
@@ -330,7 +330,7 @@ found_client:
for (;;) {
set_current_state(TASK_INTERRUPTIBLE);
if (signal_pending(current) ||
- clp->cl_cons_state > NFS_CS_READY)
+ clp->cl_cons_state != NFS_CS_INITING)
break;
schedule();
}