Re: [PATCH 1/3] pvclock: Add CPU barries to get correct version value

From: Andy Lutomirski
Date: Fri May 27 2016 - 11:40:17 EST


On Thu, May 26, 2016 at 11:17 PM, Minfei Huang <mnghuan@xxxxxxxxx> wrote:
> Protocol for the "version" fields is: hypervisor raises it (making it
> uneven) before it starts updating the fields and raises it again (making
> it even) when it is done. Thus the guest can make sure the time values
> it got are consistent by checking the version before and after reading
> them.
>
> Add CPU barries after getting version value just like what function
> vread_pvclock does, because all of callees in this function is inline.

LGTM.