Re: [regression] e1000e broke e1000 (was: Re: [ANNOUNCE] e1000 to e1000e migration of PCI Express devices)

view
thread

Score:

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

To: Ingo Molnar <mingo@...>

Cc: Kok, Auke <auke-jan.h.kok@...>, Linux Kernel Mailing List <linux-kernel@...>, NetDev <netdev@...>, e1000-list <e1000-devel@...>, linux-pci maillist <linux-pci@...>, Jeff Garzik <jeff@...>, Andrew Morton <akpm@...>, David S. Miller <davem@...>, Linus Torvalds <torvalds@...>, Jesse Brandeburg <jesse.brandeburg@...>, Ronciak, John <john.ronciak@...>, Allan, Bruce W <bruce.w.allan@...>, Greg KH <greg@...>, Arjan van de Ven <arjan@...>, Rafael J. Wysocki <rjw@...>

Subject: Re: [regression] e1000e broke e1000 (was: Re: [ANNOUNCE] e1000 to e1000e migration of PCI Express devices)

Date: Tuesday, April 8, 2008 - 3:32 pm

On Tue, Apr 08, 2008 at 08:39:21PM +0200, Ingo Molnar wrote:
quoted text> so the pure presence of the e1000e module breaks the e1000 driver. That 
> is a regression and a bug that should be fixed.

I think you've found the wrong problem ... it looks deliberate to me that
enabling e1000e disables e1000 from claiming the PCI IDs (see the PCIE()
macro right before the e1000_pci_tbl in drivers/net/e1000/e1000_main.c).

The question is why e1000e isn't claiming the device ...

-- 
Intel are signing my paycheques ... these opinions are still mine
"Bill, look, we understand that you're interested in selling us this
operating system, but compare it to ours.  We can't possibly take such
a retrograde step."
--
unsubscribe noticeTo unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:

[regression] e1000e broke e1000 (was: Re: [ANNOUNCE] e1000 t..., Ingo Molnar, (Tue Apr 8, 2:39 pm)

RE: [regression] e1000e broke e1000 (was: Re: [ANNOUNCE] e10..., Brandeburg, Jesse, (Tue Apr 8, 3:43 pm)

Re: [regression] e1000e broke e1000 (was: Re: [ANNOUNCE] e10..., Ingo Molnar, (Tue Apr 8, 3:59 pm)

Re: [regression] e1000e broke e1000 (was: Re: [ANNOUNCE] e10..., Matthew Wilcox, (Tue Apr 8, 4:04 pm)

Re: [regression] e1000e broke e1000 (was: Re: [ANNOUNCE] e10..., Ingo Molnar, (Tue Apr 8, 4:17 pm)

showing which hardware is unclaimed, Rick Jones, (Tue Apr 8, 4:13 pm)

Re: showing which hardware is unclaimed, Martin Mares, (Tue Apr 8, 4:35 pm)

Re: [regression] e1000e broke e1000, Dan Noe, (Tue Apr 8, 4:12 pm)

Re: [regression] e1000e broke e1000, Matthew Wilcox, (Tue Apr 8, 4:20 pm)

Re: [regression] e1000e broke e1000, Dan Noe, (Tue Apr 8, 4:39 pm)

Re: [regression] e1000e broke e1000, Ingo Molnar, (Tue Apr 8, 4:35 pm)

Re: [regression] e1000e broke e1000, Martin Mares, (Tue Apr 8, 4:36 pm)

Re: [regression] e1000e broke e1000 (was: Re: [ANNOUNCE] e10..., Matthew Wilcox, (Tue Apr 8, 3:32 pm)

Re: [regression] e1000e broke e1000 (was: Re: [ANNOUNCE] e10..., Ingo Molnar, (Tue Apr 8, 3:51 pm)

Re: [regression] e1000e broke e1000, Bill Davidsen, (Wed Apr 9, 8:52 pm)

Re: [regression] e1000e broke e1000, Ingo Molnar, (Fri Apr 11, 4:59 am)

Re: [regression] e1000e broke e1000, Jeff Garzik, (Tue Apr 8, 3:56 pm)

Re: [regression] e1000e broke e1000, Ingo Molnar, (Tue Apr 8, 4:06 pm)

Re: [regression] e1000e broke e1000, Kok, Auke, (Tue Apr 8, 4:31 pm)

Re: [regression] e1000e broke e1000, Ingo Molnar, (Wed Apr 9, 3:12 pm)

Re: [regression] e1000e broke e1000, Jeff Garzik, (Wed Apr 9, 3:33 pm)

Re: [regression] e1000e broke e1000, Ingo Molnar, (Fri Apr 11, 7:30 am)

Re: [regression] e1000e broke e1000, Chris Friesen, (Fri Apr 11, 11:40 am)

Re: [regression] e1000e broke e1000, Willy Tarreau, (Fri Apr 11, 3:29 pm)

Re: [regression] e1000e broke e1000, Jeff Garzik, (Tue Apr 8, 4:19 pm)

Re: [regression] e1000e broke e1000, Ingo Molnar, (Tue Apr 8, 4:33 pm)

Re: [regression] e1000e broke e1000, Jeff Garzik, (Tue Apr 8, 4:56 pm)

[patch] e1000=y && e1000e=m regression fix (was: Re:..., Ingo Molnar, (Wed Apr 9, 3:38 pm)

Re: [patch] e1000=y && e1000e=m regression fix (was:..., Frans Pop, (Wed Apr 9, 4:49 pm)

Re: [patch] e1000=y && e1000e=m regression fix (was:..., Krzysztof Halasa, (Wed Apr 9, 7:59 pm)

Re: [patch] e1000=y && e1000e=m regression fix (was:..., Linus Torvalds, (Wed Apr 9, 9:40 pm)

Re: [patch] e1000=y && e1000e=m regression fix (was:..., Krzysztof Halasa, (Thu Apr 10, 5:57 am)

Re: [patch] e1000=y && e1000e=m regression fix (was:..., Linus Torvalds, (Thu Apr 10, 10:30 am)

Re: [patch] e1000=y && e1000e=m regression fix (was:..., Krzysztof Halasa, (Thu Apr 10, 5:35 pm)

Re: [patch] e1000=y && e1000e=m regression fix (was:..., Grant Grundler, (Thu Apr 10, 1:55 pm)

Re: [patch] e1000=y && e1000e=m regression fix (was:..., Linus Torvalds, (Thu Apr 10, 3:27 pm)

Re: [patch] e1000=y && e1000e=m regression fix (was:..., Matthew Wilcox, (Thu Apr 10, 2:04 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Kok, Auke, (Thu Apr 10, 2:26 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Chris Friesen, (Thu Apr 10, 5:20 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Kok, Auke, (Wed Apr 9, 4:12 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Kok, Auke, (Thu Apr 10, 2:29 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Ingo Molnar, (Thu Apr 10, 3:27 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Kok, Auke, (Thu Apr 10, 5:23 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Ingo Molnar, (Fri Apr 11, 7:26 am)

Re: [patch] e1000=y && e1000e=m regression fix, Christoph Hellwig, (Fri Apr 11, 7:36 am)

Re: [patch] e1000=y && e1000e=m regression fix, Ingo Molnar, (Fri Apr 11, 8:16 am)

Re: [patch] e1000=y && e1000e=m regression fix, Kok, Auke, (Fri Apr 11, 12:22 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Martin Mares, (Fri Apr 11, 1:10 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Christoph Hellwig, (Fri Apr 11, 12:45 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Linus Torvalds, (Fri Apr 11, 1:34 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Daniel Barkalow, (Fri Apr 11, 6:06 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Linus Torvalds, (Fri Apr 11, 7:00 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Jeff Garzik, (Fri Apr 11, 7:43 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Linus Torvalds, (Sun Apr 13, 5:13 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Ingo Molnar, (Mon Jun 9, 3:24 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Ondrej Zary, (Sun Apr 13, 5:34 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Christoph Hellwig, (Sat Apr 12, 9:07 am)

Re: [patch] e1000=y && e1000e=m regression fix, , (Fri Apr 11, 7:58 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Daniel Barkalow, (Fri Apr 11, 7:15 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Jeff Garzik, (Fri Apr 11, 6:21 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Daniel Barkalow, (Fri Apr 11, 7:05 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Matthew Wilcox, (Fri Apr 11, 1:53 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Linus Torvalds, (Fri Apr 11, 2:51 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Dan Noe, (Fri Apr 11, 5:01 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Krzysztof Halasa, (Fri Apr 11, 4:22 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Linus Torvalds, (Fri Apr 11, 4:29 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Matthew Wilcox, (Fri Apr 11, 3:01 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Linus Torvalds, (Fri Apr 11, 4:21 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Willy Tarreau, (Fri Apr 11, 3:25 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Matthew Wilcox, (Fri Apr 11, 3:38 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Kok, Auke, (Fri Apr 11, 1:26 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Philip Craig, (Thu Apr 10, 8:46 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Randy Dunlap, (Thu Apr 10, 5:44 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Andi Kleen, (Fri Apr 11, 3:54 am)

Re: [patch] e1000=y && e1000e=m regression fix, Kok, Auke, (Thu Apr 10, 5:52 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Ingo Molnar, (Wed Apr 9, 4:53 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Jeff Garzik, (Wed Apr 9, 3:50 pm)

Re: [patch] e1000=y && e1000e=m regression fix, Ingo Molnar, (Wed Apr 9, 4:04 pm)

Re: [E1000-devel] [regression] e1000e broke e1000, Kok, Auke, (Tue Apr 8, 4:47 pm)

Navigation

Mail archive search

Popular discussions


linux-kernel:
Linus Torvalds	Linux 2.6.27
Linus Torvalds	Linux 2.6.27-rc8
Tejun Heo	[PATCHSET] FUSE: extend FUSE to support more operations
James Bottomley	Re: Integration of SCST in the mainstream Linux kernel
git:
Ken Pratt	pack operation is thrashing my server
Jakub Narebski	Re: VCS comparison table
H. Peter Anvin	Re: git versus CVS (versus bk)
Marco Costalba	[PATCH 11/11] Convert sha1_file.c to use decompress helpers
openbsd-misc:
Richard Stallman	Real men don't attack straw men
Marcos Laufer	dmesg IBM x3650 OpenBSD 4.3
Brian A. Seklecki	Re: GRE over IPsec
sonjaya	openvpn on openbsd 4.1
linux-netdev:
Hugh Dickins	Re: [bug?] tg3: Failed to load firmware "tigon/tg3_tso.bin"
Gilles Chanteperdrix	[PATCH] cs89x0: add support for i.MX31ADS ARM board
Denys Fedoryshchenko	thousands of classes, e1000 TX unit hang
Francois Romieu	Re: 8169 Intermittent ifup Failure Issue With RTL8102E Chipset in Intel's New D945...

Latest forum posts


Shared swap partition	11 hours ago	Linux general
high memory	2 days ago	Linux kernel
semaphore access speed	2 days ago	Applications and Utilities
the kernel how to power off the machine	2 days ago	Linux kernel
Easter Eggs in windows XP	2 days ago	Windows
Root password	2 days ago	Linux general
Where/when DNOTIFY is used?	2 days ago	Linux kernel
How to convert Linux Kernel built-in module into a loadable module	2 days ago	Linux kernel
Linux 2.6.24 and I/O schedulers	2 days ago	Linux kernel
USB Driver -- Interrupt Polling -- A Little Help Please	3 days ago	Linux general

Show all forums...

Recent Tags

2.6.27 Linux bugs Intel quote Linus Torvalds -rc 2.6.27-rc8 -rc8

more tags

Colocation donated by:

Online users

Syndicate