• Stefano Stabellini's avatar
    PCI hotplug: acpiphp: set current_state to D0 in register_slot · 47e9037a
    Stefano Stabellini authored
    If a device doesn't support power management (pm_cap == 0) but it is
    acpi_pci_power_manageable() because there is a _PS0 method declared for
    it and _EJ0 is also declared for the slot then nobody is going to set
    current_state = PCI_D0 for this device.  This is what I think it is
    happening:
    
    pci_enable_device
        |
    __pci_enable_device_flags
    /* here we do not set current_state because !pm_cap */
        |
    do_pci_enable_device
        |
    pci_set_power_state
        |
    __pci_start_power_transition
        |
    pci_platform_power_transition
    /* platform_pci_power_manageable() calls acpi_pci_power_manageable that
     * returns true */
        |
    platform_pci_set_power_state
    /* acpi_pci_set_power_state gets called and does nothing because the
     * acpi device has _EJ0, see the comment "If the ACPI device has _EJ0,
     * ignore the device" */
    
    at this point if we refer to the commit message that introduced the
    comment above (10b3dcae), it is up to
    the hotplug driver to set the state to D0.
    However AFAICT the pci hotplug driver never does, in fact
    drivers/pci/hotplug/acpiphp_glue.c:register_slot sets the slot flags to
    (SLOT_ENABLED | SLOT_POWEREDON) but it does not set the pci device
    current state to PCI_D0.
    
    So my proposed fix is also to set current_state = PCI_D0 in
    register_slot.
    Comments are very welcome.
    Signed-off-by: default avatarStefano Stabellini <stefano.stabellini@eu.citrix.com>
    Signed-off-by: default avatarJesse Barnes <jbarnes@virtuousgeek.org>
    47e9037a
acpiphp_glue.c 35.2 KB