>>> On 28.12.18 at 12:18, <roger....@citrix.com> wrote:
> --- a/xen/arch/x86/hvm/dom0_build.c
> +++ b/xen/arch/x86/hvm/dom0_build.c
> @@ -60,6 +60,18 @@ static struct acpi_madt_interrupt_override __initdata
> *intsrcovr;
> static unsigned int __initdata acpi_nmi_sources;
> static struct acpi_madt_nmi_source __initdata *nmisrc;
>
> +static unsigned int __initdata order_stats[MAX_ORDER + 1] = { };
Pointless initializer.
> +static void __init print_order_stats(void)
> +{
> + unsigned int i;
> +
> + printk("Memory allocation stats:\n");
I think this wants prefixing with "Dom0".
> + for ( i = 0; i < ARRAY_SIZE(order_stats); i++ )
> + if ( order_stats[i] )
> + printk("order: %2u allocations: %u\n", i, order_stats[i]);
Why the first of the two colons?
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
