On 10.11.2023 17:30, Oleksii Kurochko wrote: > The patch introduces generic paging.h header for Arm, PPC and > RISC-V. > > All mentioned above architectures use hardware virt extensions > and hardware pagetable extensions thereby it makes sense to set > paging_mode_translate and paging_mode_external by default. > > Also in this patch Arm and PPC architectures are switched to > generic paging.h header. > > Signed-off-by: Oleksii Kurochko <oleksii.kuroc...@gmail.com>
Reviewed-by: Jan Beulich <jbeul...@suse.com> with one nit (applicable twice): > --- a/xen/arch/arm/include/asm/Makefile > +++ b/xen/arch/arm/include/asm/Makefile > @@ -1,2 +1,3 @@ > # SPDX-License-Identifier: GPL-2.0-only > generic-y += vm_event.h > +generic-y += paging.h Like in obj-y lists it would be nice for the items here and ... > --- a/xen/arch/ppc/include/asm/Makefile > +++ b/xen/arch/ppc/include/asm/Makefile > @@ -1,2 +1,3 @@ > # SPDX-License-Identifier: GPL-2.0-only > generic-y += vm_event.h > +generic-y += paging.h ... here to be sorted alphabetically from the beginning. Right here I'd be happy to make the change while committing, but I'm not going to promise making the same offer on subsequent changes (should the same issue exist there). Jan
