On 11.08.2021 12:23, Wei Chen wrote: > --- a/xen/arch/arm/platform.c > +++ b/xen/arch/arm/platform.c > @@ -27,6 +27,7 @@ extern const struct platform_desc _splatform[], > _eplatform[]; > /* Pointer to the current platform description */ > static const struct platform_desc *platform; > > +extern unsigned int dma_bitsize;
This is a no-go: Declarations need to live in a header which the producer and all consumers include. Else ... > --- a/xen/common/page_alloc.c > +++ b/xen/common/page_alloc.c > @@ -227,7 +227,7 @@ static bool __read_mostly scrub_debug; > * Bit width of the DMA heap -- used to override NUMA-node-first. > * allocation strategy, which can otherwise exhaust low memory. > */ > -static unsigned int dma_bitsize; > +unsigned int dma_bitsize; ... a change here (of e.g. the type) will go unnoticed by the compiler, and the consumer of the variable may no longer work correctly. Also I'm afraid the description does not make clear why this variable is what you want to use. Connected to this is the question why you need to consume it on Arm in the first place, when x86 never had the need. Jan
