| Message ID | 1389770748-32698-1-git-send-email-jlee@suse.com |
|---|---|
| State | Accepted |
| Headers | show |
Rafael, you want to take this one? On 01/14/2014 11:25 PM, Lee, Chun-Yi wrote: > This is a variant patch from Rafael J. Wysocki's > ACPI / init: Run acpi_early_init() before efi_enter_virtual_mode() > > According to Matt Fleming, if acpi_early_init() was executed before > efi_enter_virtual_mode(), the EFI initialization could benefit from > it, so Rafael's patch makes that happen. > > And, we want accessing ACPI TAD device to set system clock, so move > acpi_early_init() before timekeeping_init(). This final position is > also before efi_enter_virtual_mode(). > > v2: > Move acpi_early_init() before timekeeping_init() to prepare setting > system clock with ACPI TAD. > > v1: > Rafael J. Wysocki > ACPI / init: Run acpi_early_init() before efi_enter_virtual_mode() > > Cc: Rafael J. Wysocki <rjw@rjwysocki.net> > Cc: Matt Fleming <matt@console-pimps.org> > Cc: H. Peter Anvin <hpa@zytor.com> > Cc: Borislav Petkov <bp@suse.de> > Cc: Matthew Garrett <matthew.garrett@nebula.com> > Tested-by: Toshi Kani <toshi.kani@hp.com> > Signed-off-by: Lee, Chun-Yi <jlee@suse.com> > --- > init/main.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/init/main.c b/init/main.c > index febc511..b6d93c8 100644 > --- a/init/main.c > +++ b/init/main.c > @@ -565,6 +565,7 @@ asmlinkage void __init start_kernel(void) > init_timers(); > hrtimers_init(); > softirq_init(); > + acpi_early_init(); > timekeeping_init(); > time_init(); > sched_clock_postinit(); > @@ -641,7 +642,6 @@ asmlinkage void __init start_kernel(void) > > check_bugs(); > > - acpi_early_init(); /* before LAPIC and SMP init */ > sfi_init_late(); > > if (efi_enabled(EFI_RUNTIME_SERVICES)) { >
On Wednesday, January 15, 2014 04:16:59 AM H. Peter Anvin wrote: > Rafael, you want to take this one? Yes, I can take it. Thanks! > On 01/14/2014 11:25 PM, Lee, Chun-Yi wrote: > > This is a variant patch from Rafael J. Wysocki's > > ACPI / init: Run acpi_early_init() before efi_enter_virtual_mode() > > > > According to Matt Fleming, if acpi_early_init() was executed before > > efi_enter_virtual_mode(), the EFI initialization could benefit from > > it, so Rafael's patch makes that happen. > > > > And, we want accessing ACPI TAD device to set system clock, so move > > acpi_early_init() before timekeeping_init(). This final position is > > also before efi_enter_virtual_mode(). > > > > v2: > > Move acpi_early_init() before timekeeping_init() to prepare setting > > system clock with ACPI TAD. > > > > v1: > > Rafael J. Wysocki > > ACPI / init: Run acpi_early_init() before efi_enter_virtual_mode() > > > > Cc: Rafael J. Wysocki <rjw@rjwysocki.net> > > Cc: Matt Fleming <matt@console-pimps.org> > > Cc: H. Peter Anvin <hpa@zytor.com> > > Cc: Borislav Petkov <bp@suse.de> > > Cc: Matthew Garrett <matthew.garrett@nebula.com> > > Tested-by: Toshi Kani <toshi.kani@hp.com> > > Signed-off-by: Lee, Chun-Yi <jlee@suse.com> > > --- > > init/main.c | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/init/main.c b/init/main.c > > index febc511..b6d93c8 100644 > > --- a/init/main.c > > +++ b/init/main.c > > @@ -565,6 +565,7 @@ asmlinkage void __init start_kernel(void) > > init_timers(); > > hrtimers_init(); > > softirq_init(); > > + acpi_early_init(); > > timekeeping_init(); > > time_init(); > > sched_clock_postinit(); > > @@ -641,7 +642,6 @@ asmlinkage void __init start_kernel(void) > > > > check_bugs(); > > > > - acpi_early_init(); /* before LAPIC and SMP init */ > > sfi_init_late(); > > > > if (efi_enabled(EFI_RUNTIME_SERVICES)) { > > >
diff --git a/init/main.c b/init/main.c index febc511..b6d93c8 100644 --- a/init/main.c +++ b/init/main.c @@ -565,6 +565,7 @@ asmlinkage void __init start_kernel(void) init_timers(); hrtimers_init(); softirq_init(); + acpi_early_init(); timekeeping_init(); time_init(); sched_clock_postinit(); @@ -641,7 +642,6 @@ asmlinkage void __init start_kernel(void) check_bugs(); - acpi_early_init(); /* before LAPIC and SMP init */ sfi_init_late(); if (efi_enabled(EFI_RUNTIME_SERVICES)) {