RE: [PATCH 0/2] PM / sleep: Support for using suspend-to-idle by default

[Mario.Limonciello]

> -----Original Message-----
> From: rjwysocki@xxxxxxxxx [mailto:rjwysocki@xxxxxxxxx] On Behalf Of
> Rafael J. Wysocki
> Sent: Wednesday, December 7, 2016 4:57 PM
> To: Limonciello, Mario <Mario_Limonciello@xxxxxxxx>
> Cc: Rafael J. Wysocki <rafael@xxxxxxxxxx>; Rafael J. Wysocki
> <rjw@xxxxxxxxxxxxx>; Linux PM <linux-pm@xxxxxxxxxxxxxxx>; ACPI Devel
> Maling List <linux-acpi@xxxxxxxxxxxxxxx>; Len Brown
> <len.brown@xxxxxxxxx>; Linux Kernel Mailing List <linux-
> kernel@xxxxxxxxxxxxxxx>; Andy Lutomirski <luto@xxxxxxxxxx>
> Subject: Re: [PATCH 0/2] PM / sleep: Support for using suspend-to-idle by
> default
> 
> On Wed, Dec 7, 2016 at 10:38 PM,  <Mario.Limonciello@xxxxxxxx> wrote:
> >> -----Original Message-----
> >> From: rjwysocki@xxxxxxxxx [mailto:rjwysocki@xxxxxxxxx] On Behalf Of
> >> Rafael J. Wysocki
> >> Sent: Thursday, November 17, 2016 3:05 PM
> >> To: Limonciello, Mario <Mario_Limonciello@xxxxxxxx>
> >> Cc: Rafael J. Wysocki <rjw@xxxxxxxxxxxxx>; Linux PM <linux-
> >> pm@xxxxxxxxxxxxxxx>; ACPI Devel Maling List <linux-
> acpi@xxxxxxxxxxxxxxx>;
> >> Len Brown <len.brown@xxxxxxxxx>; Linux Kernel Mailing List <linux-
> >> kernel@xxxxxxxxxxxxxxx>; Andy Lutomirski <luto@xxxxxxxxxx>
> >> Subject: Re: [PATCH 0/2] PM / sleep: Support for using suspend-to-idle by
> >> default
> >>
> >> On Thu, Nov 17, 2016 at 9:40 PM,  <Mario.Limonciello@xxxxxxxx> wrote:
> >> > Hi Rafael,
> >> >
> >> >> -----Original Message-----
> >> >> From: Rafael J. Wysocki [mailto:rjw@xxxxxxxxxxxxx]
> >> >> Sent: Wednesday, November 16, 2016 8:19 PM
> >> >> To: Linux PM list <linux-pm@xxxxxxxxxxxxxxx>
> >> >> Cc: ACPI Devel Maling List <linux-acpi@xxxxxxxxxxxxxxx>; Len Brown
> >> >> <len.brown@xxxxxxxxx>; Linux Kernel Mailing List <linux-
> >> >> kernel@xxxxxxxxxxxxxxx>; Limonciello, Mario
> >> >> <Mario_Limonciello@xxxxxxxx>; Andy Lutomirski <luto@xxxxxxxxxx>
> >> >> Subject: [PATCH 0/2] PM / sleep: Support for using suspend-to-idle by
> >> >> default
> >> >>
> >> >> Hi,
> >> >>
> >> >> This series is a follow-up for a BoF discussion during the LPC.
> >> >>
> >> >> The discussion was about supporting Linux on modern laptops and one
> of
> >> >> the issues mentioned was the lack of support for suspend-to-idle in
> user
> >> >> space and if there's anything that can be done about that in the kernel.
> >> >>
> >> >> The following patches are my input. :-)
> >> >>
> >> >> The first one reworks the suspend interface to allow the "mem" string
> >> >> in /sys/power/state to represent multiple things that can be selected
> via
> >> >> an additional sysfs attribute.
> >> >>
> >> >> The second one makes ACPI select suspend-to-idle as the default
> >> suspend
> >> >> mode
> >> >> if so indicated in the FADT.
> >> >>
> >> >> Please let me know what you think.
> >> >>
> >> >> Thanks,
> >> >> Rafael
> >> >
> >> > Thanks for submitting.  I like the approach, especially that it leaves room
> >> > to easily modify the behavior for debugging purposes and adding quirks.
> >>
> >> Well, that's the point. :-)
> >>
> >> > I tested your series on a machine that had this bit set in the FADT and
> >> > confirmed that it set up and used policy properly.
> >> >
> >> > Tested-By: Mario Limonciello <mario.limonciello@xxxxxxxx>
> >>
> >> Thanks!
> >
> > Rafael,
> >
> > I hadn't noticed these merged into any of your linux-pm.git branches.
> >
> > Do you have a plan for merging them?  Or waiting for other feedback?
> 
> They are in the linux-next branch (and therefore in linux-next), so on
> their way to 4.10-rc1.
> 
> I usually only update topic branches at kernel.org when they are ready
> to go into the mainline and the ones I have locally are merged into
> the linux-next one beforehand.
> 
> Thanks,
> Rafael

Ah thanks, I see 'em now.  Sorry for the noise.