Re: [PATCH 2/7] atm: replace del_timer by del_timer_sync

From: Eric Dumazet
Date: Wed Mar 26 2014 - 18:32:11 EST


On Wed, 2014-03-26 at 15:19 -0700, Vijay Subramanian wrote:
> >
> > It seems this timer rearms itself, so this patch wont be enough I fear.
> >
> >
> >
> Does that matter? It seems del_timer_sync() may be intended to work
> despite this (i.e. timer will not fire after call to this function).
> My earlier attempt to fix a similar issue in sch_red.c resulted in
> this explanation.
> http://www.spinics.net/lists/netdev/msg256410.html

I know this very well, thank you.

If documentation is stale, it needs clarification.

If not, we need to make sure timer wont be restarted.



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/