On 1/12/18 12:37 PM, Andrew Cooper wrote:
> Adjust handle_ldt_mapping_fault() exclude the use of this fixup path for
> non-PV guests.  Well-formed code shouldn't reference the LDT while in HVM vcpu
> context, but currently on a context switch from PV to HVM context, there may
> be a stale LDT selector loaded, over an unmapped region.
> 
> By explicitly excluding HVM context at this point, we avoid erroneous
> hypervisor execution resulting in a cascade failure, by falling into
> pv_map_ldt_shadow_page().
> 
> Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com>

Obviously a behavior change here but the rationale behind it seems clear
to me and well worth doing for the net positive result:

Reviewed-by: Doug Goldstein <car...@cardoe.com>

-- 
Doug Goldstein

Attachment: signature.asc
Description: OpenPGP digital signature

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

Reply via email to