[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1346629078.2563.75.camel@edumazet-glaptop>
Date: Mon, 03 Sep 2012 01:37:58 +0200
From: Eric Dumazet <eric.dumazet@...il.com>
To: Florian Westphal <fw@...len.de>
Cc: Sami Farin <hvtaifwkbgefbaei@...il.com>, netdev@...r.kernel.org,
e1000-devel@...ts.sourceforge.net
Subject: Re: (ipt_log_packet, sb_add) 3.6.0-rc2 kernel panic - not syncing;
Fatal exception in interrupt
On Mon, 2012-09-03 at 00:53 +0200, Eric Dumazet wrote:
> First I thought changing demux to not do the lookup of TIMEWAIT slot in
> __inet_lookup_established(), then it sounds not optimal to redo the full
> lookup for ESTABLISHED sockets later in TCP stack.
>
> So it seems we should fix xt_LOG instead
>
> Thanks Florian !
>
> [PATCH] xt_LOG: take care of timewait sockets
>
> Sami Farin reported crashes in xt_LOG because it assumes skb->sk is a
> full blown socket.
>
> But with TCP early demux, we can have skb->sk pointing to a timewait
> socket.
Seems other fixes are needed, for example in nfnetlink_log.c
I'll send a v2
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists