[ovs-dev] [PATCH] cmap: Fix deadlock with ovs-atomic-pthreads

Jarno Rajahalme jrajahalme at nicira.com
Mon Jun 2 19:56:59 UTC 2014


On Jun 2, 2014, at 12:51 PM, Ben Pfaff <blp at nicira.com> wrote:

> On Mon, Jun 02, 2014 at 12:50:45PM -0700, Jarno Rajahalme wrote:
>> 
>> On Jun 2, 2014, at 11:45 AM, Ben Pfaff <blp at nicira.com> wrote:
>> 
>>> I'd really hope that any serious OVS implementation would be able to
>>> use C11 or GCC or Clang or other compiler-specific techniques to get a
>>> "real" implementation of atomics.  Really the pthreads version is just
>>> to make porting easier.
>> 
>> Are we already documenting this somewhere?
> 
> I don't think so.  It wasn't the attitude I started out with;
> originally I hoped that the pthreads implementation would be viable
> for real use, but I'm really not sure about that any longer.

I just ran “time ./ovstest test-cmap check 1”, with ovs-atomics-pthreads it takes 16 times longer than on master. So it is likely unusable in practice, so this would be the time to document it, I guess.

  Jarno




More information about the dev mailing list