netdev - Re: [GIT]: Networking

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20080721135024.GA18320@elte.hu>
Date:	Mon, 21 Jul 2008 15:50:24 +0200
From:	Ingo Molnar <mingo@...e.hu>
To:	David Miller <davem@...emloft.net>
Cc:	torvalds@...ux-foundation.org, akpm@...ux-foundation.org,
	netdev@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [GIT]: Networking

David,

* David Miller <davem@...emloft.net> wrote:

>       e1000: remove PCI Express device IDs

networking in -git (v2.6.26-5253-g14b395e, dead e1000 interface) 
silently broke on two testboxes of mine earlier today, and i've bisected 
[*] it back to:

| d03157babed7424f5391af43200593768ce69c9a is first bad commit
| commit d03157babed7424f5391af43200593768ce69c9a
| Author: Auke Kok <auke-jan.h.kok@...el.com>
| Date:   Sun Jun 22 15:21:29 2008 -0700
|
|    e1000: remove PCI Express device IDs
|
|    We do not want to prolong the situation much longer that e1000
|    and e1000e support these devices at the same time. As a result,
|    take out the bandage that was added for the interim period
|    and remove all the PCI Express device IDs from e1000.

i have migrated these testboxes to e1000e. (i migrated a third one 
already but forgot about these two and i was stupid enough to do a 
bisection suspecting some new bug.)

I'm wondering, couldnt we warn about the removed pci express support via 
the kernel log? Some simple printk. (Perhaps also with a default-off 
.config option that shuts up this warning for users who intentionally 
boot E1000=y without e1000e support.)

Thanks,

	Ingo

[*] the bisection log:

 # bad:  [14b395e1] Merge branch 'for-2.6.27' of git://linux-nfs.org/~
 # good: [bce7f795] Linux 2.6.26
 # good: [cadc7236] Merge branch 'bkl-removal' into next
 # bad:  [a0c80b8d] pkt_sched: Make default qdisc nonshared-multiqueue
 # good: [30902dc4] ax25: Fix std timer socket destroy handling.
 # bad:  [fbd8f13a] net-sched: sch_htb: move hash and sibling list rem
 # good: [40af48c1] rt2x00: kill URB for all TX queues during disable_
 # bad:  [4977929a] iwlwifi: control 11n capabilities through module p
 # bad:  [28f49d89] Merge branch 'master' of master.kernel.org:/pub/sc
 # bad:  [0caa116d] net: sh_eth: Fix compile error sh_eth
 # good: [485ca22c] DM9000: Re-unite menuconfig entries for DM9000 dri
 # good: [177db6f0] ixgbe: add LRO support
 # bad:  [6e4f6f6f] e1000e: make ioport free
 # bad:  [d03157ba] e1000: remove PCI Express device IDs

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html