Re: [Xen-devel] [PATCH 22/34] x86/mm: put HVM only code under CONFIG_HVM

2018-08-24 Thread Wei Liu
On Tue, Aug 21, 2018 at 02:07:22AM -0600, Jan Beulich wrote: > #ifdef (also elsewhere; did I overlook similar issues in earlier patches?) > > > --- a/xen/common/vm_event.c > > +++ b/xen/common/vm_event.c > > @@ -429,9 +429,11 @@ void vm_event_resume(struct domain *d, struct > > vm_event_domain

Re: [Xen-devel] [PATCH 22/34] x86/mm: put HVM only code under CONFIG_HVM

2018-08-24 Thread Wei Liu
On Sun, Aug 19, 2018 at 07:27:22PM +0300, Razvan Cojocaru wrote: > On 8/17/18 6:12 PM, Wei Liu wrote: > > Going through the code, nested EPT, EPT, and ALTP2M depend on HVM code. Put > > these components under CONFIG_HVM. This further requires putting one > > of the vm event under CONFIG_HVM. > >

Re: [Xen-devel] [PATCH 22/34] x86/mm: put HVM only code under CONFIG_HVM

2018-08-22 Thread Wei Liu
On Tue, Aug 21, 2018 at 02:07:22AM -0600, Jan Beulich wrote: > >>> On 17.08.18 at 17:12, wrote: > > --- a/xen/arch/x86/mm/Makefile > > +++ b/xen/arch/x86/mm/Makefile > > @@ -2,8 +2,9 @@ subdir-y += shadow > > subdir-y += hap > > > > obj-y += paging.o > > -obj-y += p2m.o p2m-pt.o p2m-ept.o

Re: [Xen-devel] [PATCH 22/34] x86/mm: put HVM only code under CONFIG_HVM

2018-08-21 Thread Jan Beulich
>>> On 17.08.18 at 17:12, wrote: > --- a/xen/arch/x86/mm/Makefile > +++ b/xen/arch/x86/mm/Makefile > @@ -2,8 +2,9 @@ subdir-y += shadow > subdir-y += hap > > obj-y += paging.o > -obj-y += p2m.o p2m-pt.o p2m-ept.o p2m-pod.o > -obj-y += altp2m.o > +obj-y += p2m.o p2m-pt.o p2m-pod.o >

Re: [Xen-devel] [PATCH 22/34] x86/mm: put HVM only code under CONFIG_HVM

2018-08-19 Thread Razvan Cojocaru
On 8/17/18 6:12 PM, Wei Liu wrote: > Going through the code, nested EPT, EPT, and ALTP2M depend on HVM code. Put > these components under CONFIG_HVM. This further requires putting one > of the vm event under CONFIG_HVM. > > Also make hap_enabled evaluate to false when !CONFIG_HVM. This in turn >

[Xen-devel] [PATCH 22/34] x86/mm: put HVM only code under CONFIG_HVM

2018-08-17 Thread Wei Liu
Going through the code, nested EPT, EPT, and ALTP2M depend on HVM code. Put these components under CONFIG_HVM. This further requires putting one of the vm event under CONFIG_HVM. Also make hap_enabled evaluate to false when !CONFIG_HVM. This in turn requires marking a variable in p2m_set_entry as