Re: [Qemu-devel] [PATCH 16/16] cpus-common: lock-free fast path for cpu_exec_start/end

[Richard Henderson]

On 09/23/2016 12:31 AM, Paolo Bonzini wrote:
> +        if (atomic_read(&other_cpu->running)) {
...
> +    atomic_set(&cpu->running, true);
...
> +            cpu->running = false;
...
> +            cpu->running = true;

Inconsistent use of atomics.  I don't see that the cpu_list_lock protects the 
last two lines in any way.


r~