Re: [Xen-devel] [PATCH v2 2/2] xen: implement VCPUOP_register_runstate_phys_memory_area

[Julien Grall <julien.grall@xxxxxxx>]

Hi Andrii,

On 16/05/2019 15:25, Andrii Anisov wrote:
> Hello Julien,
>
> On 16.05.19 16:48, Julien Grall wrote:
> > The lock can be completely removed anyway. See my previous comments.
> You suggested kinda simplified try_lock with runstate update skipping in case of 
> fail.
> The question here is if we are OK with not updating runstate under 
> circumstances?
Well, if you fail the check then it means someone was modifying it behind your 
back. That someone could update the runstate with the new information once it is 
modified. So I can't see issue with not updating the runstate in the context switch.
Cheers,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel