login
Header Space

 
 

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 - 4:22 pm. (11 messages)

Next thread: Re: Socket by Grant Likely on Tuesday, December 11, 2007 - 5:09 pm. (1 message)
To: Martin Bligh <mbligh@...>
Cc: <linux-kernel@...>, Andy Whitcroft <apw@...>, <netdev@...>, Auke Kok <auke-jan.h.kok@...>, Thomas Gleixner <tglx@...>, Ingo Molnar <mingo@...>
Date: Tuesday, December 11, 2007 - 4: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.
--
To: Andrew Morton <akpm@...>
Cc: Martin Bligh <mbligh@...>, <linux-kernel@...>, Andy Whitcroft <apw@...>, <netdev@...>, Thomas Gleixner <tglx@...>, Ingo Molnar <mingo@...>
Date: Tuesday, December 11, 2007 - 5: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
--
To: Kok, Auke <auke-jan.h.kok@...>
Cc: <mbligh@...>, <linux-kernel@...>, <apw@...>, <netdev@...>, <tglx@...>, <mingo@...>
Date: Tuesday, December 11, 2007 - 6: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.

--
To: Andrew Morton <akpm@...>
Cc: <mbligh@...>, <linux-kernel@...>, <apw@...>, <netdev@...>, <tglx@...>, <mingo@...>
Date: Tuesday, December 11, 2007 - 6: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
--
To: Kok, Auke <auke-jan.h.kok@...>
Cc: Andrew Morton <akpm@...>, <mbligh@...>, <linux-kernel@...>, <apw@...>, <netdev@...>, <tglx@...>, <mingo@...>
Date: Tuesday, December 11, 2007 - 7:15 pm

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

---

From: Randy Dunlap &lt;randy.dunlap@oracle.com&gt;

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

Signed-off-by: Randy Dunlap &lt;randy.dunlap@oracle.com&gt;
---
 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,
--
To: Kok, Auke <auke-jan.h.kok@...>
Cc: Andrew Morton <akpm@...>, Martin Bligh <mbligh@...>, <linux-kernel@...>, Andy Whitcroft <apw@...>, <netdev@...>, Thomas Gleixner <tglx@...>, Ingo Molnar <mingo@...>
Date: Tuesday, December 11, 2007 - 5:59 pm

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

Auke
--
To: Andrew Morton <akpm@...>
Cc: Martin Bligh <mbligh@...>, <linux-kernel@...>, Andy Whitcroft <apw@...>, <netdev@...>, Auke Kok <auke-jan.h.kok@...>, Thomas Gleixner <tglx@...>, Roland McGrath <roland@...>
Date: Tuesday, December 11, 2007 - 5: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
--
Previous thread: [PATCH 1/6] Use zonelists instead of zones when direct reclaiming pages by Mel Gorman on Tuesday, December 11, 2007 - 4:22 pm. (11 messages)

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