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

view
thread

!MAILaRCHIVE_VOTE_RePLACE

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."
--
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:

[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

Popular discussions


linux-kernel:
Greg Kroah-Hartman	[PATCH 006/196] Chinese: add translation of oops-tracing.txt
Andrew Morton	Re: -mm merge plans for 2.6.23 -- sys_fallocate
Eric W. Biederman	[PATCH] nfs lockd reclaimer: Convert to kthread API
James Bottomley	Re: Integration of SCST in the mainstream Linux kernel
git:

linux-netdev:
David Miller	[GIT]: Networking
Gerrit Renker	[PATCH 03/37] dccp: List management for new feature negotiation

David Miller	Re: [PATCH] pkt_sched: Destroy gen estimators under rtnl_lock().
openbsd-misc:

Colocation donated by:

Online users

Syndicate