On 29/05/2024 8:55 pm, Oleksii Kurochko wrote: > diff --git a/xen/arch/riscv/mm.c b/xen/arch/riscv/mm.c > index fe3a43be20..2c3fb7d72e 100644 > --- a/xen/arch/riscv/mm.c > +++ b/xen/arch/riscv/mm.c > @@ -1,5 +1,6 @@ > /* SPDX-License-Identifier: GPL-2.0-only */ > > +#include <xen/bug.h> > #include <xen/cache.h> > #include <xen/compiler.h> > #include <xen/init.h> > @@ -14,6 +15,9 @@ > #include <asm/page.h> > #include <asm/processor.h> > > +unsigned long __ro_after_init frametable_base_pdx; > +unsigned long __ro_after_init frametable_virt_end;
Along with removing base_pdx, virt_end appears to be unique to ARM before and after the series. I'd recommend dropping it too. ~Andrew