]> xenbits.xensource.com Git - people/royger/xen.git/commit
x86/shadow: sh_{make,destroy}_monitor_table() are "even more" HVM-only
authorJan Beulich <jbeulich@suse.com>
Fri, 30 Oct 2020 13:26:46 +0000 (14:26 +0100)
committerJan Beulich <jbeulich@suse.com>
Fri, 30 Oct 2020 13:26:46 +0000 (14:26 +0100)
commit0b841314dacecc94376be22f514da0665055fa3b
tree8fac73d965ae5d254790e22031b355a3f58707db
parent8ac7e452a6e9fb3c8314fe92b2e1ebb0937fa0dd
x86/shadow: sh_{make,destroy}_monitor_table() are "even more" HVM-only

With them depending on just the number of shadow levels, there's no need
for more than one instance of them, and hence no need for any hook (IOW
452219e24648 ["x86/shadow: monitor table is HVM-only"] didn't go quite
far enough). Move the functions to hvm.c while dropping the dead
is_pv_32bit_domain() code paths.

While moving the code, replace a stale comment reference to
sh_install_xen_entries_in_l4(). Doing so made me notice the function
also didn't have its prototype dropped in 8d7b633adab7 ("x86/mm:
Consolidate all Xen L4 slot writing into init_xen_l4_slots()"), which
gets done here as well.

Also make their first parameters const.

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Acked-by: Roger Pau Monné <roger.pau@citrix.com>
Acked-by: Tim Deegan <tim@xen.org>
xen/arch/x86/mm/shadow/common.c
xen/arch/x86/mm/shadow/hvm.c
xen/arch/x86/mm/shadow/multi.c
xen/arch/x86/mm/shadow/private.h
xen/arch/x86/mm/shadow/types.h
xen/include/asm-x86/paging.h