lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 11 Apr 2008 10:46:00 +1000 From: Philip Craig <philipc@...pgear.com> To: "Kok, Auke" <auke-jan.h.kok@...el.com> CC: Ingo Molnar <mingo@...e.hu>, Jeff Garzik <jeff@...zik.org>, Matthew Wilcox <matthew@....cx>, Linux Kernel Mailing List <linux-kernel@...r.kernel.org>, NetDev <netdev@...r.kernel.org>, e1000-list <e1000-devel@...ts.sourceforge.net>, linux-pci maillist <linux-pci@...ey.karlin.mff.cuni.cz>, Andrew Morton <akpm@...ux-foundation.org>, "David S. Miller" <davem@...emloft.net>, Linus Torvalds <torvalds@...ux-foundation.org>, Jesse Brandeburg <jesse.brandeburg@...el.com>, "Ronciak, John" <john.ronciak@...el.com>, "Allan, Bruce W" <bruce.w.allan@...el.com>, Greg KH <greg@...ah.com>, Arjan van de Ven <arjan@...ux.intel.com>, "Rafael J. Wysocki" <rjw@...k.pl> Subject: Re: [patch] e1000=y && e1000e=m regression fix Kok, Auke wrote: > Again - If there is a way to auto-enable e1000e in the right way so that more > systems migrate better then I'm all for it (even if forcing E1000E=y). But it > seems that the various patches proposed don't cut it and frankly Kconfig is > completely inadequate as a hardware enabling script since it knows absolutely > nothing about the hardware in the first place. And it wasn't meant for that > either. `make oldconfig` is not the answer ;). One problem is that the meaning of E1000 has been changed. It covers less hardware than it used to. You could add a new option to control the e1000 driver, and make E1000 set both this new option and E1000E. Thus it will always cover at least all the IDs that the original e1000 driver handled. config E1000 tristate "Both Intel(R) PRO/1000 Gigabit Ethernet support" depends on PCI config E1000_ONLY tristate "Intel(R) PRO/1000 Gigabit Ethernet support" if E1000=n default E1000 depends on PCI config E1000E tristate "Intel(R) PRO/1000 PCI-Express Gigabit Ethernet support" if E1000=n default E1000 depends on PCI The E1000E prompt restriction is required to upgrade existing E1000=y, E1000E=m configs to E1000E=y. But it will also upgrade E1000=y/m,E1000E=n to E1000E=y/m, which may not always be right. This still doesn't solve any problems with loading modules for E1000=m. Loading the e1000 module will still load support for less than it used to. (Because make oldconfig is not the answer ;-) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists