On 23/11/16 08:27, Jan Beulich wrote: >>>> On 22.11.16 at 17:32, <andrew.coop...@citrix.com> wrote: >> 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 <jbeul...@suse.com> >> As for the mechanical adjustments here, Reviewed-by: Andrew Cooper >> <andrew.coop...@citrix.com> >> >> 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. > But the backlink gets written into the incoming TSS.
Ah - of course it does. I was getting confused by the repeated use of the tss structure. Sorry for the noise. ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
