Re: [PATCH] net: fix race in the receive/select

From: David Miller
Date: Thu Jun 25 2009 - 22:11:50 EST


From: Davide Libenzi <davidel@xxxxxxxxxxxxxxx>
Date: Thu, 25 Jun 2009 19:04:04 -0700 (PDT)

> On Fri, 26 Jun 2009, Eric Dumazet wrote:
>
>> Adding a plain smp_mb() in tcp_poll() for example would slowdown select()/poll() with NULL
>> timeout.
>
> Do you think of it as good design adding an MB on a subsystem, because of
> the special locking logic of another one?
> The (eventual) slowdown, IMO can be argued sideways, by saying that
> non-socket users will pay the price for their polls.

Perhaps every performance argument is moot since spin_unlock() used to
have the barrier :-)
--
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/