Re: [Xen-devel] [PATCH] x86: correctly set nonlazy_xstate_used when loading full state

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 26.06.18 at 09:57, <wei.liu2@xxxxxxxxxx> wrote:
> On Tue, Jun 26, 2018 at 01:47:12AM -0600, Jan Beulich wrote:
>> In this case, just like xcr0_accum, nonlazy_xstate_used should always be
>> set to the intended new value, rather than possibly leaving the flag set
>> from a prior state load.
>> 
>> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
> 
> Why? Looking at the code it is in fact used as a boolean.

Its boolean nature is irrelevant. The question is whether fully new state
is loaded (in which case the flag should be set according to this state,
without regard to the prior value of the flag) vs ...

> Also you seem to have missed hvm/svm/svm.c and xstate.c?

... existing state being updated.

Jan



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