[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] x86/PV: guest_get_eff_kern_l1e() may still need to switch page tables


  • To: Jan Beulich <jbeulich@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
  • Date: Mon, 14 Dec 2020 14:37:27 +0000
  • Authentication-results: esa3.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none
  • Cc: Wei Liu <wl@xxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Manuel Bouyer <bouyer@xxxxxxxxxxxxxxx>
  • Delivery-date: Mon, 14 Dec 2020 14:37:42 +0000
  • Ironport-sdr: dko5nyLuNZiqVXaAGA9niPDUim9pTpy9+1cOSEfvAozg2DPcGfy4bhYqIl7w9ruz7Ni3VBPmQ/ DQfIX1z94jGlAlN9F2yQXp61BOjZ1MOHGUhXVtgGrU7sCabF5ymXWq5f8Lwjl6Tl9z8YoZTMDL TRogQBlZ1mHi4LHDY/XtLcK6amROih9m2a2lyjTggaXwYWHvu5vILYK5+SZmOY7EdXrYjDMzdm VCP1ikvBdZVxta4OkwiSm8Chu/5jUaMhfpWRgtFcZSdgUn8onPwvXNdhQ8HJSVXgJFFopv0fzg 4zk=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 14/12/2020 13:57, Jan Beulich wrote:
> While indeed unnecessary for pv_ro_page_fault(), pv_map_ldt_shadow_page()
> may run when guest user mode is active, and hence may need to switch to
> the kernel page tables in order to retrieve an LDT page mapping.
>
> Fixes: 9ff970564764 ("x86/mm: drop guest_get_eff_l1e()")
> Reported-by: Manuel Bouyer <bouyer@xxxxxxxxxxxxxxx>
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
> Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> ---
> Manuel, could you test this again, just to be on the safe side
> before we throw it in (at which point we could then also again
> add a Tested-by)? Thanks.

I've got a repro of the issue (literally - just booting the
netinstaller), and this does fix it.

~Andrew



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.