netdev - Re: forcedeth: Reside in 100Mbit Kconfig but also supports 1000Mbit cards

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 for Android: free password hash cracker in your pocket

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

Date:	Mon, 28 May 2007 22:02:56 +0200
From:	Krzysztof Halasa <khc@...waw.pl>
To:	Stefan Schmidt <stefan@...enfreihafen.org>
Cc:	Randy Dunlap <randy.dunlap@...cle.com>, aabdulla@...dia.com,
	netdev@...r.kernel.org
Subject: Re: forcedeth: Reside in 100Mbit Kconfig but also supports 1000Mbit cards

Stefan Schmidt <stefan@...enfreihafen.org> writes:

> I still like the partition by speed.

Me 2

> It makes it a lot easier to find
> the right driver in the bunch of available ones.

Sure. Unless you have a 100 Mb/s card and you don't know the same
driver works with newer 1000 Mb/s models.

IOW - questionable.

> A solution like a
> Kconfig entry symlink for driver which needs to support cards in two
> or more categories would be my favorite.

It would be a total mess.

However, some clever conditionals could make it better.
One menu for both 100 and 1000 Mb/s, and something like:

tristate 100Mbps (Y/N/M)
tristate 1000Mbps (Y/N/M)

...
if 100Mbps
	Realtek 8139
	Intel E100
fi

if 100Mbps || 1000Mbps
	forcedeath
	some_SIS_driver
	etc.
fi

if 1000Mbps
	E1000
	...

Probably 10 Mbps drivers could lead the list. You could check for PCI,
ISA and what not.
This way each driver has only one entry, but if you know you have,
say, 1 Gb/s Ethernet card only you can select 1 Gb/s and don't
worry about 100-Mbps NVidias.

I'd include that Tulip thing, too - no need for a separate menu
(the directory is a different thing).
-- 
Krzysztof Halasa
-
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