On 04/07/2016 10:32 AM, Emilio G. Cota wrote: > static inline void qemu_spin_lock(QemuSpin *spin) > { > - do { > + while (atomic_xchg(&spin->value, true)) { > while (atomic_read(&spin->value)) { > cpu_relax(); > } > - } while (atomic_xchg(&spin->value, true)); > + } > }
And merge this one as well, please. It's a good improvement, but there's little point to keeping these separate. r~