Re: [Xen-devel] [PATCH V2] x86/altp2m: clean up p2m_{get/set}_suppress_ve()

[George Dunlap <George.Dunlap@xxxxxxxxxxxxx>]

On Tue, Sep 25, 2018 at 9:37 AM Razvan Cojocaru
<rcojocaru@xxxxxxxxxxxxxxx> wrote:
>
> Move p2m_{get/set}_suppress_ve() to p2m.c, replace incorrect
> ASSERT() in p2m-pt.c (since a guest can run in shadow mode even on
> a system with virt exceptions, which would trigger the ASSERT()),
> move the VMX-isms (cpu_has_vmx_virt_exceptions checks) to
> p2m_ept_{get/set}_entry(), and fix locking code in
> p2m_get_suppress_ve().
>
> Signed-off-by: Razvan Cojocaru <rcojocaru@xxxxxxxxxxxxxxx>

This looks good, thanks; but it doesn't apply cleanly to staging
anymore.  Can you double-check to make sure nothing important has
changed in the moved functions within mem_access.c, and then resend a
rebased patch?

Thanks,
 -George

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