The patch titled Subject: Re: networking crash in current mainline: sk_filter_delayed_uncharge() has been added to the -mm tree. Its filename is try-this.patch *** Remember to use Documentation/SubmitChecklist when testing your code *** See http://www.zip.com.au/~akpm/linux/patches/stuff/added-to-mm.txt to find out what to do about this ------------------------------------------------------ Subject: Subject: Re: networking crash in current mainline: sk_filter_delayed_uncharge() From: David Miller <davem@xxxxxxxxxxxxx> Date: Thu, 18 Oct 2007 23:09:48 -0700 > > powerpc mac G5 > config: http://userweb.kernel.org/~akpm/config-g5.txt > screenshot: http://userweb.kernel.org/~akpm/dsc00005.jpg > > It does this shortly after bringing up eth0 (tg3), in dhclient. Try this: >From 9b013e05e0289c190a53d78ca029e2f21c0e4485 Mon Sep 17 00:00:00 2001 Date: Thu, 18 Oct 2007 21:48:39 -0700 Subject: [PATCH] [NET]: Fix bug in sk_filter race cures. Looks like this might be causing problems, at least for me on ppc. This happened during a normal boot, right around first interface config/dhcp run.. cpu 0x0: Vector: 300 (Data Access) at [c00000000147b820] pc: c000000000435e5c: .sk_filter_delayed_uncharge+0x1c/0x60 lr: c0000000004360d0: .sk_attach_filter+0x170/0x180 sp: c00000000147baa0 msr: 9000000000009032 dar: 4 dsisr: 40000000 current = 0xc000000004780fa0 paca = 0xc000000000650480 pid = 1295, comm = dhclient3 0:mon> t [c00000000147bb20] c0000000004360d0 .sk_attach_filter+0x170/0x180 [c00000000147bbd0] c000000000418988 .sock_setsockopt+0x788/0x7f0 [c00000000147bcb0] c000000000438a74 .compat_sys_setsockopt+0x4e4/0x5a0 [c00000000147bd90] c00000000043955c .compat_sys_socketcall+0x25c/0x2b0 [c00000000147be30] c000000000007508 syscall_exit+0x0/0x40 Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- net/core/filter.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff -puN net/core/filter.c~try-this net/core/filter.c --- a/net/core/filter.c~try-this +++ a/net/core/filter.c @@ -447,7 +447,8 @@ int sk_attach_filter(struct sock_fprog * rcu_assign_pointer(sk->sk_filter, fp); rcu_read_unlock_bh(); - sk_filter_delayed_uncharge(sk, old_fp); + if (old_fp) + sk_filter_delayed_uncharge(sk, old_fp); return 0; } _ Patches currently in -mm which might be from davem@xxxxxxxxxxxxx are origin.patch try-this.patch advansys-depends-on-virt_to_bus.patch - To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html