Re: 2.6.24-rc4-mm1

Previous thread: [PATCH 1/6] Use zonelists instead of zones when direct reclaiming pages by Mel Gorman on Tuesday, December 11, 2007 - 1:22 pm. (10 messages)

Next thread: Re: Socket by Grant Likely on Tuesday, December 11, 2007 - 2:09 pm. (1 message)
From: Andrew Morton
Date: Tuesday, December 11, 2007 - 1:37 pm

(cc netdev and Auke)

Yes, that would be very sensible.  CONFIG_E1000E should default to whatever


(cc Greg)

Caused by gregkh-driver-kobject-convert-hvcs-to-use-kref-not-kobject.patch.
--

From: Ingo Molnar
Date: Tuesday, December 11, 2007 - 2:20 pm

Roland says:

| That seems like it must be a tool problem.  The V=1 output would show 
| if those compiles missed -m32 or something.  But even in the wrong 
| mode, this error does not make sense.  The assembly code it's citing 
| is identical to the old arch/x86/ia32/vsyscall-syscall.S code.

	Ingo
--

From: Kok, Auke
Date: Tuesday, December 11, 2007 - 2:26 pm

which is "y" for x86 and friends, ppc, arm and ia64 through 'defconfig'. the
Kconfig files do not have defaults in them.

I can send a patch to adjust the defconfig files, would that be OK? I certainly
think that would be reasonable, I dislike setting defaults through defconfig for
network drivers myself and rather would not do that.

Auke
--

From: Kok, Auke
Date: Tuesday, December 11, 2007 - 2:59 pm

that should read "dislike setting defaults through Kconfig ..."

Auke
--

From: Andrew Morton
Date: Tuesday, December 11, 2007 - 3:10 pm

On Tue, 11 Dec 2007 13:26:58 -0800

I wouldn't be looking at defconfig files - I don't think many people use
them.  Most people use their previous config, via oldconfig.

So what we want here is to give them E1000E if they had previously been
using E1000.  I don't know how one would do this in Kconfig.

--

From: Kok, Auke
Date: Tuesday, December 11, 2007 - 3:17 pm

ditto. I doubt that "SELECT E1000E" would be a good idea here (maybe not even
work), and I can't think of anything else.

Auke
--

From: Randy Dunlap
Date: Tuesday, December 11, 2007 - 4:15 pm

"default E1000" in E1000E seems to work for me.

---

From: Randy Dunlap <randy.dunlap@oracle.com>

Make E1000E default to the same kconfig setting as E1000,
at least for -mm testing.

Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
---
 drivers/net/Kconfig |    1 +
 1 file changed, 1 insertion(+)

--- linux-2.6.24-rc4-mm1.orig/drivers/net/Kconfig
+++ linux-2.6.24-rc4-mm1/drivers/net/Kconfig
@@ -1986,6 +1986,7 @@ config E1000_DISABLE_PACKET_SPLIT
 config E1000E
 	tristate "Intel(R) PRO/1000 PCI-Express Gigabit Ethernet support"
 	depends on PCI
+	default E1000
 	---help---
 	  This driver supports the PCI-Express Intel(R) PRO/1000 gigabit
 	  ethernet family of adapters. For PCI or PCI-X e1000 adapters,
--

Previous thread: [PATCH 1/6] Use zonelists instead of zones when direct reclaiming pages by Mel Gorman on Tuesday, December 11, 2007 - 1:22 pm. (10 messages)

Next thread: Re: Socket by Grant Likely on Tuesday, December 11, 2007 - 2:09 pm. (1 message)