>>> On 21.04.17 at 12:41, <wei.l...@citrix.com> wrote: > On Wed, Apr 19, 2017 at 09:35:50AM -0600, Jan Beulich wrote: >> >>> On 03.04.17 at 13:22, <wei.l...@citrix.com> wrote: >> > PG_external requires hardware support so the code guarded by that is HVM >> > only. >> > >> > Signed-off-by: Wei Liu <wei.l...@citrix.com> >> > --- >> > xen/arch/x86/hvm/Makefile | 1 + >> > xen/arch/x86/hvm/mm.c | 85 > +++++++++++++++++++++++++++++++++++++++ >> >> I think the moved code would better go under x86/mm/, possibly (but >> not necessarily) into p2m.c. > > I disagree here. What good is there to keep this code when hvm support > is compiled out?
Hmm, do you mean to move e.g. mm/p2m-*.c and mm/hap/ to hvm/mm/ then too? I think we shouldn't go overboard with this shuffling. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
