On Mon, 15 May 2017, Steven Rostedt wrote: > > [ added linux-acpi and took off arm list ;-) ] > > On Sun, 14 May 2017 20:27:23 +0200 > Thomas Gleixner <tglx@xxxxxxxxxxxxx> wrote: > > > To enable smp_processor_id() and might_sleep() debug checks earlier, it's > > required to add system states between SYSTEM_BOOTING and SYSTEM_RUNNING. > > > > Make the decision wether a pci root is hotplugged depend on SYSTEM_RUNNING > > instead of !SYSTEM_BOOTING. > > > > Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx> > > Cc: Catalin Marinas <catalin.marinas@xxxxxxx> > > Cc: Will Deacon <will.deacon@xxxxxxx> > > Cc: Mark Rutland <mark.rutland@xxxxxxx> > > Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > > --- > > drivers/acpi/pci_root.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > --- a/drivers/acpi/pci_root.c > > +++ b/drivers/acpi/pci_root.c > > @@ -523,7 +523,7 @@ static int acpi_pci_root_add(struct acpi > > struct acpi_pci_root *root; > > acpi_handle handle = device->handle; > > int no_aspm = 0; > > - bool hotadd = system_state != SYSTEM_BOOTING; > > + bool hotadd = system_state == SYSTEM_RUNNING; > > What about the other boot states greater than SYSTEM_RUNNING? Can this > be called then too? > > IOW, should it be: > > bool hotadd = system_state >= SYSTEM_RUNNING; Greater than RUNNING is HALT,POWEROFF,RESTART. Having a PCI hotplug event there would be interesting :) I'll amend the changelog. Thanks, tglx -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
