| Message ID | 20170814142418.13267-23-julien.grall@arm.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hi, On 14/08/17 15:24, Julien Grall wrote: > We should consider the early boot period to end when we stop using the > boot allocator. This is inline with x86 and will be helpful to know > whether we should allocate memory from the boot allocator or xenheap. > > Signed-off-by: Julien Grall <julien.grall@arm.com> Reviewed-by: Andre Przywara <andre.przywara@arm.com> Cheers, Andre. > --- > xen/arch/arm/setup.c | 8 ++++++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > > diff --git a/xen/arch/arm/setup.c b/xen/arch/arm/setup.c > index 277b566b88..46737a2eca 100644 > --- a/xen/arch/arm/setup.c > +++ b/xen/arch/arm/setup.c > @@ -757,6 +757,12 @@ void __init start_xen(unsigned long boot_phys_offset, > > end_boot_allocator(); > > + /* > + * The memory subsystem has been initialized, we can now switch from > + * early_boot -> boot. > + */ > + system_state = SYS_STATE_boot; > + > vm_init(); > > if ( acpi_disabled ) > @@ -779,8 +785,6 @@ void __init start_xen(unsigned long boot_phys_offset, > console_init_preirq(); > console_init_ring(); > > - system_state = SYS_STATE_boot; > - > processor_id(); > > smp_init_cpus(); >
diff --git a/xen/arch/arm/setup.c b/xen/arch/arm/setup.c index 277b566b88..46737a2eca 100644 --- a/xen/arch/arm/setup.c +++ b/xen/arch/arm/setup.c @@ -757,6 +757,12 @@ void __init start_xen(unsigned long boot_phys_offset, end_boot_allocator(); + /* + * The memory subsystem has been initialized, we can now switch from + * early_boot -> boot. + */ + system_state = SYS_STATE_boot; + vm_init(); if ( acpi_disabled ) @@ -779,8 +785,6 @@ void __init start_xen(unsigned long boot_phys_offset, console_init_preirq(); console_init_ring(); - system_state = SYS_STATE_boot; - processor_id(); smp_init_cpus();
We should consider the early boot period to end when we stop using the boot allocator. This is inline with x86 and will be helpful to know whether we should allocate memory from the boot allocator or xenheap. Signed-off-by: Julien Grall <julien.grall@arm.com> --- xen/arch/arm/setup.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-)