[PATCH] libertas: reduce command retry time
David Woodhouse
dwmw2 at redhat.com
Wed Apr 30 16:43:16 EDT 2008
On Wed, 2008-04-30 at 16:34 -0400, Dan Williams wrote:
> On Wed, 2008-04-30 at 16:44 +0200, Holger Schurig wrote:
> > In the normal case, an unsuccessful command would be tried for
> > 10*5 seconds, or 10*10 seconds in the worst case. This patch
> > reduces this to 3*1 seconds, or 3*5 seconds in the worst case.
> >
> > Also changed the needless del_timer_sync(&priv->command_timer);
> > to del_timer().
> >
> > Signed-off-by: Holger Schurig <hs4233 at mail.mn-solutions.de>
>
> Sounds fine to me... Woodhouse?
I think I've seen it take a few of the 5-second timeouts. The firmware
on this thing really is a steaming pile of shit. I wouldn't like to
decrease the maximum so drastically.
Also, that del_timer_sync() happens in the module unload path, doesn't
it? I'd want to see why you think it isn't needed.
--
dwmw2
More information about the libertas-dev
mailing list