netdev - Re: [patch 04/13] 8390: Split 8390 support into a pausing and a non pausing driver core

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: <20080610005247.6230fec1@core>
Date:	Tue, 10 Jun 2008 00:52:47 +0100
From:	Alan Cox <alan@...rguk.ukuu.org.uk>
To:	akpm@...ux-foundation.org
Cc:	jeff@...zik.org, netdev@...r.kernel.org, akpm@...ux-foundation.org,
	alan@...hat.com, randy.dunlap@...cle.com
Subject: Re: [patch 04/13] 8390: Split 8390 support into a pausing and a non
 pausing driver core

On Mon, 09 Jun 2008 16:33:49 -0700
akpm@...ux-foundation.org wrote:

> From: Alan Cox <alan@...rguk.ukuu.org.uk>
> 
> Only a few ISA controllers need the pausing version of the 8390 core
> while PCMCIA, later ISA and PCI do not. More importantly the ISA delays
> can break non ISA boxes so we must use a different build of 8390.c for
> the two sets of controllers.
> 
> No changes since last time as all the points of concerns raised proved to
> be invalid
> 
> [randy.dunlap@...cle.com: ne2 (MCA) needs both 8390.o and 8390p.o functions]

ne2 does not need both, it hasn't needed both and its completely invalid
to have both in one driver. Please drop Randy's change out and fix the
actual typo instead.

Alan
--
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