[PATCH] libertas: Fix ethtool statistics
David Woodhouse
dwmw2 at infradead.org
Sun Apr 20 06:56:32 EDT 2008
On Sun, 2008-04-20 at 10:41 +0200, Sebastian Siewior wrote:
> The function before lbs_ethtool_get_stats() (in net code that
> determinate the number of elements, dunno the name right now) is able to
> return an error. This one could check if the firmware has mesh support.
That's what my second patch does.
> Do I understand this correct: If the firmware has support for mesh
> devices than we have ethX and mshX and only mshX should return the
> statistics?
Well, they are _mesh_ statistics :)
> returning uninitialized nonsense is leaking kernel memory.
Indeed. We need that memset there to handle errors, definitely.
> What about changing lbs_ethtool_get_stats() from void to int? All
> other drivers are reading memory to get this data, we have to go
> through usb/cs/sdio layer and all of them may fail.
Yeah, that might be worthwhile.
> I will try to form a patch around Monday that fixes Dan's comments (if
> nobody else is going to).
Which comments? If we apply your original patch, followed by my patch in
<1208509888.9212.291.camel at pmac.infradead.org> (which for some reason
only went to wireless-dev, not libertas-dev), then I think it's fine.
--
dwmw2
More information about the libertas-dev
mailing list