CAS is broken

dormando dormando at rydia.net
Thu Nov 15 08:56:18 UTC 2007


Dustin Sallings wrote:
> 
> On Nov 11, 2007, at 14:48, Chris Goffinet wrote:
> 
>> Looks good Dustin, just validated using the latest test from Tomash.
>> Also we need to wrap this in pthread mutex's so its compatible with
>> thread version.
> 
> 
>     Yeah, I almost had a comment out there, but I left it out.  I'm not
> sure whether or not the item allocation is already within a mutex.
> 
>     My goal was to provide a sketch of what I was talking about for
> conceptual review.
> 

For posterity, fix is in r635 from chris (not christ!) and dustin.
I read it, played with it a bit, ran tests, ran it, looked okay.
Runs entirely inside a lock, logic appears sane.

Adds 8 bytes to the item struct.

In it goes.

-Dormando


More information about the memcached mailing list