[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [PATCH] xen/arm: livepatch: Include xen/mm.h rather than asm/mm.h
On Fri, Jan 15, 2021 at 07:29:47PM +0000, Julien Grall wrote: > From: Julien Grall <jgrall@xxxxxxxxxx> > > Livepatch fails to build on Arm after commit ced9795c6cb4 "mm: split > out mfn_t / gfn_t / pfn_t definitions and helpers": > > In file included from livepatch.c:13:0: > /oss/xen/xen/include/asm/mm.h:32:28: error: field ‘list’ has incomplete type > struct page_list_entry list; > ^~~~ > /oss/xen/xen/include/asm/mm.h:53:43: error: ‘MAX_ORDER’ undeclared here (not > in a function); did you mean ‘PFN_ORDER’? > unsigned long first_dirty:MAX_ORDER + 1; > ^~~~~~~~~ > PFN_ORDER > /oss/xen/xen/include/asm/mm.h:53:31: error: bit-field ‘first_dirty’ width not > an integer constant > unsigned long first_dirty:MAX_ORDER + 1; > ^~~~~~~~~~~ > > This is happening because asm/mm.h is included directly by livepatch.c. > Yet it depends on xen/mm.h to be included first so MAX_ORDER is defined. > > Resolve the build failure by including xen/mm.h rather than asm/mm.h. > > Fixes: ced9795c6cb4 ("mm: split out mfn_t / gfn_t / pfn_t definitions and > helpers") Reviewed-by: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx> Thank you! > Signed-off-by: Julien Grall <jgrall@xxxxxxxxxx> > --- > xen/arch/arm/livepatch.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/xen/arch/arm/livepatch.c b/xen/arch/arm/livepatch.c > index 915e9d926a11..75e8adcfd6a1 100644 > --- a/xen/arch/arm/livepatch.c > +++ b/xen/arch/arm/livepatch.c > @@ -6,11 +6,11 @@ > #include <xen/lib.h> > #include <xen/livepatch_elf.h> > #include <xen/livepatch.h> > +#include <xen/mm.h> > #include <xen/vmap.h> > > #include <asm/cpufeature.h> > #include <asm/livepatch.h> > -#include <asm/mm.h> > > /* Override macros from asm/page.h to make them work with mfn_t */ > #undef virt_to_mfn > -- > 2.17.1 >
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |