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

From: Rafael J. Wysocki
Date: Wed Dec 07 2016 - 17:56:55 EST


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