Re: [E1000-devel] [TCP]: TCP_DEFER_ACCEPT causes leak sockets

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Ingo Molnar
Date: Wednesday, June 18, 2008 - 4:14 pm

* Kok, Auke <auke-jan.h.kok@intel.com> wrote:


i have reported the problem and even provided a fix.

I have triggered an e1000/e1000e related problem that got introduced in 
the v2.6.25 merge window - one of my testboxes came up with no 
networking and it took me an hour to figure out why. (i wasnt 
particularly focusing on e1000, i just happened to hit that bug in 9 
million lines of Linux kernel code)

I have reported it here, two and a half months ago:

    http://lkml.org/lkml/2008/4/8/256

I even showed you which commit introduced the problem and gave you a 
oneliner fix that i tested (it solved the problem):

    http://bugzilla.kernel.org/attachment.cgi?id=15704&amp;action=view

You were Cc:-ed to that. (attached below again for reference) The bug 
was added to the regression list of v2.6.25. I never expected to spend 
more than 10 minutes on this problem once i found out what's happening - 
we fix dozens of bugs like this per stable kernel release.

I just checked latest -git, my fix is still not upstream (or any 
equivalent solution - i really dont mind how it's solved and i'm not 
maintaining this code).

no alternative patch was sent to me - i offered to test any solution 
back then.

FYI, since i first reported it i've been hit by that problem roughly a 
dozen times. (it happened sporadically so i forgot about it - until i 
again had a system come up with no networking.) It caused me lost time 
and lost work that could have been spent on better things.

	Ingo

------------------------>
Subject: e1000=y && e1000e=m regression fix
From: Ingo Molnar <mingo@elte.hu>
Date: Wed Apr 09 21:09:35 CEST 2008

fix a regression from v2.6.24: do not transfer the e1000e PCI IDs from 
e1000 to e1000e if e1000 is built-in and e1000e is a module.

Built-in drivers take precedence over modules in many ways - and in this 
case it's clear that the user intended the e1000 driver to be the 
primary one. "Silently change behavior and break existing configs" is 
never a good migration strategy. Most users will use distro kernels that 
are not affected by this problem at all - nor are they affected by this 
patch - but this problem can hit users and developers who build their 
kernels themselves and migrate from v2.6.24 to v2.6.25.

this fixes: http://bugzilla.kernel.org/show_bug.cgi?id=10427

Signed-off-by: Ingo Molnar <mingo@elte.hu>
---
 drivers/net/Kconfig |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Index: linux-x86.q/drivers/net/Kconfig
===================================================================
--- linux-x86.q.orig/drivers/net/Kconfig
+++ linux-x86.q/drivers/net/Kconfig
@@ -2022,7 +2022,7 @@ config E1000E
 	  will be called e1000e.
 
 config E1000E_ENABLED
-	def_bool E1000E != n
+	def_bool E1000E = y || ((E1000E != n) && (E1000 = E1000E))
 
 config IP1000
 	tristate "IP1000 Gigabit Ethernet support"
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[TCP]: TCP_DEFER_ACCEPT causes leak sockets, Vitaliy Gusev, (Wed Jun 11, 5:58 am)
Re: [TCP]: TCP_DEFER_ACCEPT causes leak sockets, Alexey Kuznetsov, (Wed Jun 11, 6:57 am)
Re: [TCP]: TCP_DEFER_ACCEPT causes leak sockets, David Miller, (Wed Jun 11, 4:52 pm)
Re: [TCP]: TCP_DEFER_ACCEPT causes leak sockets, David Miller, (Thu Jun 12, 4:32 pm)
Re: [TCP]: TCP_DEFER_ACCEPT causes leak sockets, Ingo Molnar, (Thu Jun 12, 11:30 pm)
Re: [TCP]: TCP_DEFER_ACCEPT causes leak sockets, David Miller, (Fri Jun 13, 2:32 am)
Re: [TCP]: TCP_DEFER_ACCEPT causes leak sockets, Ingo Molnar, (Fri Jun 13, 4:09 am)
Re: [TCP]: TCP_DEFER_ACCEPT causes leak sockets, Ingo Molnar, (Fri Jun 13, 4:47 am)
Re: [TCP]: TCP_DEFER_ACCEPT causes leak sockets, Ingo Molnar, (Fri Jun 13, 2:10 pm)
Re: [TCP]: TCP_DEFER_ACCEPT causes leak sockets, David Miller, (Mon Jun 16, 4:59 pm)
Re: [TCP]: TCP_DEFER_ACCEPT causes leak sockets, Ingo Molnar, (Tue Jun 17, 12:26 am)
Re: [TCP]: TCP_DEFER_ACCEPT causes leak sockets, David Miller, (Tue Jun 17, 12:38 am)
Re: [TCP]: TCP_DEFER_ACCEPT causes leak sockets, Ingo Molnar, (Tue Jun 17, 1:09 am)
Re: [TCP]: TCP_DEFER_ACCEPT causes leak sockets, Ingo Molnar, (Tue Jun 17, 1:32 am)
Re: [TCP]: TCP_DEFER_ACCEPT causes leak sockets, Vitaliy Gusev, (Tue Jun 17, 1:43 am)
Re: [TCP]: TCP_DEFER_ACCEPT causes leak sockets, David Miller, (Tue Jun 17, 2:08 am)
Re: [TCP]: TCP_DEFER_ACCEPT causes leak sockets, Ingo Molnar, (Tue Jun 17, 2:27 am)
Re: [TCP]: TCP_DEFER_ACCEPT causes leak sockets, David Miller, (Tue Jun 17, 2:29 am)
Re: [TCP]: TCP_DEFER_ACCEPT causes leak sockets, Ingo Molnar, (Tue Jun 17, 2:39 am)
Re: [E1000-devel] [TCP]: TCP_DEFER_ACCEPT causes leak sockets, Denys Fedoryshchenko, (Wed Jun 18, 2:41 pm)
Re: [E1000-devel] [TCP]: TCP_DEFER_ACCEPT causes leak sockets, Denys Fedoryshchenko, (Wed Jun 18, 3:44 pm)
Re: [E1000-devel] [TCP]: TCP_DEFER_ACCEPT causes leak sockets, Ingo Molnar, (Wed Jun 18, 4:14 pm)
Re: [TCP]: TCP_DEFER_ACCEPT causes leak sockets, Jarek Poplawski, (Thu Jun 19, 12:06 am)