On Tue, Nov 14, 2017 at 02:10:16PM +0800, Wanpeng Li wrote:
> 2017-11-13 21:02 GMT+08:00 Peter Zijlstra <pet...@infradead.org>:
> > That can be written like:
> >
> >         do {
> >                 if (state & KVM_VCPU_PREEMPTED)
> >                         new_state = state | KVM_VCPU_SHOULD_FLUSH;
> >                 else
> >                         new_state = state | KVM_VCPU_IPI_PENDING;
> >         } while (!try_cmpxchg(&src->preempted, state, new_state);
> >
> >         if (new_state & KVM_VCPU_IPI_PENDING)
> 
> Should be new_state & KVM_VCPU_SHOULD_FLUSH I think.

Quite so indeed.

Reply via email to