Hi Michal, > On 3 Dec 2024, at 10:22, Michal Orzel <[email protected]> wrote: > > Commit a14593e3995a extended BOOTINFO_{ACPI,SHMEM}_INIT initializers > list with a new 'type' member but forgot to add trailing commas (they > were present before). This results in a build failure when building > with CONFIG_ACPI=y and CONFIG_STATIC_SHM=y: > ./include/xen/bootfdt.h:155:5: error: request for member 'shmem' in something > not a structure or union > 155 | .shmem.common.max_banks = NR_SHMEM_BANKS, \ > | ^ > ./include/xen/bootfdt.h:168:5: note: in expansion of macro > 'BOOTINFO_SHMEM_INIT' > 168 | BOOTINFO_SHMEM_INIT \ > | ^~~~~~~~~~~~~~~~~~~ > common/device-tree/bootinfo.c:22:39: note: in expansion of macro > 'BOOTINFO_INIT' > 22 | struct bootinfo __initdata bootinfo = BOOTINFO_INIT; > > Fixes: a14593e3995a ("xen/device-tree: Allow region overlapping with > /memreserve/ ranges") > Signed-off-by: Michal Orzel <[email protected]>
Reviewed-by: Bertrand Marquis <[email protected]> Cheers Bertrand > --- > xen/include/xen/bootfdt.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/xen/include/xen/bootfdt.h b/xen/include/xen/bootfdt.h > index cea40ee11706..343c48b73d2c 100644 > --- a/xen/include/xen/bootfdt.h > +++ b/xen/include/xen/bootfdt.h > @@ -145,7 +145,7 @@ struct bootinfo { > #ifdef CONFIG_ACPI > #define BOOTINFO_ACPI_INIT \ > .acpi.common.max_banks = NR_MEM_BANKS, \ > - .acpi.common.type = MEMORY > + .acpi.common.type = MEMORY, > #else > #define BOOTINFO_ACPI_INIT > #endif > @@ -153,7 +153,7 @@ struct bootinfo { > #ifdef CONFIG_STATIC_SHM > #define BOOTINFO_SHMEM_INIT \ > .shmem.common.max_banks = NR_SHMEM_BANKS, \ > - .shmem.common.type = STATIC_SHARED_MEMORY > + .shmem.common.type = STATIC_SHARED_MEMORY, > #else > #define BOOTINFO_SHMEM_INIT > #endif > -- > 2.25.1 >
