[PATCH 0/3] WOL: Add Wake On LAN support
hadess at hadess.net
Tue Aug 28 13:47:12 PDT 2007
On Tue, 2007-08-28 at 14:12 -0400, David Zeuthen wrote:
> On Tue, 2007-08-14 at 16:54 +0200, Holger Macht wrote:
> > Adding the GetSupported(void) method to the coldplug detection of HAL
> > might make sense, though. I already have code, partly copied from ethtool,
> > to detect this from inside C code. It just makes it more complicated IMO
> > even though I'm an advocator of binary versus script code. However, I'll
> > check this again.
> Actually I think it's nicer to just have a GetSupported() method instead
> of a property. At least badly written drivers won't crap out until it's
> called which is preferable to having the system lock up at boot time.
> Which is what happened back when hal was trying to detect whether a
> cable was plugged in.
Do we actually want to care about drivers like that? They'll get fixed,
and people will downgrade their HAL in the meanwhile. Given that the
property never changes, it doesn't make much sense to check with
GetSupported() all the time. HAL checks once, all the apps know it can
Bastien Nocera <hadess at hadess.net>
More information about the hal