On 22/11/16 13:55, Jan Beulich wrote:
> The only field modified (and even that conditionally) is the back link.
> Write only that field, and only when it actually has been written to.
>
> Take the opportunity and also ditch the pointless initializer from the
> "tss" local variable.

It would help to point out that tss is unconditionally filled completely
from guest memory.

> Signed-off-by: Jan Beulich <[email protected]>

As for the mechanical adjustments here, Reviewed-by: Andrew Cooper
<[email protected]>

However, is the position of the backlink write actually correct?  I'd
have thought that all access to the old tss happen before switching cr3.

I can't find a useful description of the order of events in a task
switch in either manual.

~Andrew

_______________________________________________
Xen-devel mailing list
[email protected]
https://lists.xen.org/xen-devel

Reply via email to