[PATCH] Race conditions in QUEUE handling on SMP builds
Alexander Demenshin
aldem-nf@aldem.net
Sat, 22 Jul 2000 02:34:47 +0200
On Sat, Jul 22, 2000 at 10:11:14AM +1000, Rob Thomas wrote:
> I'm just a lurker here, so I won't get too carried away, but, to me, if
> there -is- a race somewhere, that just masks it. Anyway. I'll go back to
> lurking, I apologise for the un-neccesary frothing.
Sorry, but you are wrong here. It does not mask it, it _avoids_ race,
or, in other words, _eliminates_ race. If you know another way to
eliminate race - let me know... May be it is possible to fix it in
different way, I don't know, but solution I found is simple and without
any [visible] performance loss (on the other hand, we can fix the problem
by introducing additional locking - this would be a problem).
If you will look at code, immediately before my skb_get(), you will
find out similar call but for netdevice - it is also used to avoid
race conditions. So why not to do so with skb as well (especially
when it solves the problem)? Or - do you prefer 1000 lines patch for
bug fixes? :)
/Al