[PATCH v9] ACPI / init: Invoke early ACPI initialization earlier

Dou Liyang douly.fnst at cn.fujitsu.com
Tue Sep 26 18:41:49 PDT 2017


Hi Rafael, Zheng,

Welcome any comments!

Now, our patches are in tip tree, they are based on this
independent patch, we hope it can be merged earlier than our
patches to avoid ACPI Error. we need your advice and confirmation.

Thanks,
	dou.

At 09/13/2017 05:17 PM, Dou Liyang wrote:
> Linux uses acpi_early_init() to put the ACPI table management into the
> late stage from the early stage. This two stages are different. the
> mapped ACPI tables in early stage is temporary and should be unmapped,
> but in late stage, it is permanent and don't need to be unmapped.
>
> Originally, mapping and parsing the DMAR table should be in the late stage.
> However, Initializing interrupt delivery mode earlier will move it into
> the early stage. This causes an ACPI error warning when Linux reallocates
> the ACPI root tables.
>
> Commit b064a8fa77df ("ACPI / init: Switch over platform to the ACPI mode
> later") splits the ACPI early initialization code into acpi_early_init()
> and acpi_subsystem_init(). This makes acpi_early_init() more independently
>
> So, invoke acpi_early_init() earlier before late_time_init(), Keep the DMAR
> be mapped and parsed in late stage like before.
>
> Reported-by: Xiaolong Ye <xiaolong.ye at intel.com>
> Signed-off-by: Dou Liyang <douly.fnst at cn.fujitsu.com>
> Cc: linux-acpi at vger.kernel.org
> Cc: Rafael J. Wysocki <rjw at rjwysocki.net>
> Cc: Zheng, Lv <lv.zheng at intel.com>
> Cc: Thomas Gleixner <tglx at linutronix.de>
> Cc: Ingo Molnar <mingo at redhat.com>
> Cc: "H. Peter Anvin" <hpa at zytor.com>
> Cc: x86 at kernel.org
> Cc: Michael Ellerman <mpe at ellerman.id.au>
> Cc: Will Deacon <will.deacon at arm.com>
> Cc: linux-arm-kernel at lists.infradead.org
> Cc: Tony Luck <tony.luck at intel.com>
> Cc: Fenghua Yu <fenghua.yu at intel.com>
> Cc: linux-ia64 at vger.kernel.org
> Cc: yinghai at kernel.org
> Cc: bhe at redhat.com
> Cc: bp at alien8.de
> ---
>  init/main.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/init/main.c b/init/main.c
> index 0ee9c686..2fb98a4 100644
> --- a/init/main.c
> +++ b/init/main.c
> @@ -664,12 +664,12 @@ asmlinkage __visible void __init start_kernel(void)
>  	debug_objects_mem_init();
>  	setup_per_cpu_pageset();
>  	numa_policy_init();
> +	acpi_early_init();
>  	if (late_time_init)
>  		late_time_init();
>  	calibrate_delay();
>  	pidmap_init();
>  	anon_vma_init();
> -	acpi_early_init();
>  #ifdef CONFIG_X86
>  	if (efi_enabled(EFI_RUNTIME_SERVICES))
>  		efi_enter_virtual_mode();
>





More information about the linux-arm-kernel mailing list