On Tuesday 13 January 2009, Alan Stern wrote: > On Mon, 12 Jan 2009, Rafael J. Wysocki wrote: > > > On Monday 12 January 2009, Alan Stern wrote: > > > On Fri, 9 Jan 2009, Rafael J. Wysocki wrote: > > > > > > > > My proposal: > > > > > > > > > > Devices and events that are clearly associated with system > > > > > wakeup should be enabled by default. For example: Power > > > > > button and laptop lid. > > > > > > > > Agreed. > > > > > > > > > All other devices capable of waking up the system should be > > > > > disabled by default. This presumably includes every PCI > > > > > device. If users want keyboard or mouse events to cause > > > > > a system resume then they will have to configure their > > > > > desktop management program to enable it. > > > > > > > > I generally agree, with one exception. There are network adapters which > > > > can be enabled to wake up by the BIOS and their drivers set them up for WoL > > > > currently on this basis. These should remain enabled IMO. > > > > > > How do we know which adapters these are? IMO the PCI core should > > > disable wakeup by default for all devices when they are detected. > > > > This is what we do. > > > > > Is it sufficient to have the adapter drivers enable wakeup during > > > their probe routines? > > > > Yes, it is. > > Therefore all we need is a patch to change the USB host controller > drivers. They should call device_set_wakeup_capable() instead of > device_init_wakeup(). > > I just got back from vacation so things will be busy for a few days, > but I'll send out such a patch soon. Great, thanks a lot! Rafael _______________________________________________ linux-pm mailing list linux-pm@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/linux-pm
