On 9/24/24 12:23, Edgar E. Iglesias wrote:
> diff --git a/xen/arch/arm/dom0less-build.c b/xen/arch/arm/dom0less-build.c
> index 09b65e44ae..dab24fa9e2 100644
> --- a/xen/arch/arm/dom0less-build.c
> +++ b/xen/arch/arm/dom0less-build.c
> @@ -586,6 +586,189 @@ static int __init domain_handle_dtb_bootmodule(struct
> domain *d,
> return res;
> }
>
> +static int __init make_virtio_pci_domU_node(const struct kernel_info *kinfo)
> +{
> + void *fdt = kinfo->fdt;
> + /* reg is sized to be used for all the needed properties below */
> + __be32 reg[(1 + (GUEST_ROOT_ADDRESS_CELLS * 2) + GUEST_ROOT_SIZE_CELLS)
> + * 2];
> + __be32 irq_map[4 * 4 * 8];
Why two separate arrays on the stack? It looks like they're used for the
same purpose, so I think we may as well keep the bigger one and get rid
of the smaller.
