Subject: Re: [patch 0/6] sky2 driver update (v1.11)

From: Jeff Garzik <[email-masked]>
Date: Tue, 26 Dec 2006 16:44:24 -0500

Stephen Hemminger wrote:
> Patches are in order of severity. 1-3 are bug fixes, 4 is a cleanup
> of the power state code, and 5 adds wake on lan support.
> 
> IMHO, it is bad security policy to allow wake on lan to enabled by default.
> The sky2 driver doesn't do WOL until enabled with ethtool.

While in general I agree with you on the security principle, this seems 
like it might break working setups.

WOL is a partnership between the motherboard and NIC.  The motherboard 
must support WOL, or its useless.  And since the motherboard must 
support WOL, it normally has an on/off switch in BIOS.

As such, you're overriding the admin's chosen BIOS setting here.

	Jeff