Re: 2.5.70-bk16: nfs crash

Maneesh Soni (maneesh@in.ibm.com)
Fri, 13 Jun 2003 18:18:54 +0530


On Thu, Jun 12, 2003 at 09:18:11AM -0700, Linus Torvalds wrote:
>
>
[..]
> ---
> ===== include/linux/dcache.h 1.32 vs edited =====
> --- 1.32/include/linux/dcache.h Tue Jun 10 14:56:43 2003
> +++ edited/include/linux/dcache.h Thu Jun 12 09:12:27 2003
> @@ -174,8 +174,10 @@
>
> static inline void __d_drop(struct dentry *dentry)
> {
> - dentry->d_vfs_flags |= DCACHE_UNHASHED;
> - hlist_del_rcu_init(&dentry->d_hash);
> + if (!(dentry->d_vfs_flags & DCACHE_UNHASHED)) {
> + dentry->d_vfs_flags |= DCACHE_UNHASHED;
> + hlist_del_rcu(&dentry->d_hash);
> + }
> }

Looks like there is some problem in this. With this conditional
d_drop, umounting an NFS mount goes in a loop and oopses in dput()
This is on 2.5.70-bk17.

Unable to handle kernel paging request at virtual address 00100104
printing eip:
c016f9b1
*pde = 00000000
Oops: 0002 [#1]
CPU: 3
EIP: 0060:[<c016f9b1>] Not tainted
EFLAGS: 00010246
EIP is at dput+0x1f1/0x350
eax: 00100100 ebx: f72a8c80 ecx: f72a8c9c edx: 00200200
esi: f64d2000 edi: f72a8c88 ebp: 00000000 esp: f64d3e8c
ds: 007b es: 007b ss: 0068
Process umount (pid: 926, threadinfo=f64d2000 task=f79d9900)
Stack: 00000010 c016007b 0000007b ffffffef f72a8c80 f6448380 f64481e0 c02fccca
f72a8c80 f6448380 f72a8cf8 f72a8820 f72a8820 f6448860 c035f8e0 c02fd1c0
f72a8820 f78775e0 f7fde560 f74971d9 00000005 10ee271a 00000010 00000000
Call Trace:
[<c016007b>] bd_claim+0x2b/0xa0
[<c02fccca>] rpc_depopulate+0x16a/0x190
[<c02fd1c0>] rpc_rmdir+0x60/0xa0
[<c02f2a4d>] rpcauth_destroy+0xd/0x60
[<c02ec45d>] rpc_destroy_client+0x4d/0x70
[<c01b6427>] nfs_put_super+0x17/0x40
[<c015e03f>] generic_shutdown_super+0xbf/0x200
[<c015eec0>] kill_anon_super+0x10/0x80
[<c01b8811>] nfs_kill_super+0x11/0x20
[<c015dcf9>] deactivate_super+0xa9/0x140
[<c0175d08>] __mntput+0x18/0x30
[<c0165e79>] path_release+0x29/0x30
[<c0176641>] sys_umount+0x81/0x90
[<c015710f>] sys_close+0x9f/0x100
[<c017665c>] sys_oldumount+0xc/0x10
[<c0109477>] syscall_call+0x7/0xb

Removing the DCACHE_UNHASHED check makes it work again. Needs more
investigation.

-- 
Maneesh Soni
IBM Linux Technology Center, 
IBM India Software Lab, Bangalore.
Phone: +91-80-5044999 email: maneesh@in.ibm.com
http://lse.sourceforge.net/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/