Re: [PATCH 0/11] PCI PM: Handling of PCI devices wake-up functionality

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thursday, 26 of June 2008, Jesse Barnes wrote:
> On Monday, June 23, 2008 10:34 am Rafael J. Wysocki wrote:
> > Hi,
> >
> > The purpose of the following series of patches is to rework the power
> > management of PCI devices so that the handling of their wake-up
> > functionality is consistent with the other PM operations and so that the
> > user space can manage the wake-up functionality of PCI devices using the
> > /sys/devices/.../power/wakeup interface regardless of whether the wake-up
> > function is based on the native PCI mechanism (PME#), or it is handled by
> > the platform (eg. ACPI).
> >
> > Apart from the ACPI and PCI changes necessary for this purpose, the series
> > includes some clean-ups that are not strictly required, but make the code
> > more straightforward and (IMHO) easier to follow.
> >
> > Please review.
> 
> This patchset looks really nice, and makes pci_enable_wake and such much more 
> readable, IMO.

Thanks!

> How much testing has it seen? 

A little.  I've been using it on my HP nx6325 for some days with no visible
problems.

> Do you think it's appropriate for 2.6.27?

Well, I think the patches need to be modified to handle the case of devices
that can wake up both through PME# and the ACPI-based mechanism.
Also, I need to make ACPI enable the wake-up GPEs for PCI devices that are
supposed to be handled by it.

For this reasons I'm not sure they will be sufficiently tested before the
2.6.27 merge window.  Perhaps it's better to target the patchset at 2.6.28 and
make sure it gets some serious testing during the .27 cycle.

Thanks,
Rafael
--
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

[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux