Thanks for noticing, though, since it implies that somebody read the patch carefully. It's always a bit scary to change fundamental inline asms, especially for something like a locking routine where the core _semantics_ are so subtle (and if you get locking wrong, things may still work - you just end up with some really odd race conditions).