Re: [systemd-devel] [PATCH] Add support for Intel Rapid Start

2013-10-13 Thread Bastien Nocera
On Sun, 2013-10-13 at 22:45 +0200, Lennart Poettering wrote: > On Sun, 13.10.13 22:40, Zbigniew Jędrzejewski-Szmek (zbys...@in.waw.pl) wrote: > > Humm. So I am very conservative regarding adding vendor specific hacks > to userspace that go that low-level into the system. Given that the > kernel ha

Re: [systemd-devel] [PATCH] Add support for Intel Rapid Start

2013-10-13 Thread Bastien Nocera
On Sun, 2013-10-13 at 22:40 +0200, Zbigniew Jędrzejewski-Szmek wrote: > On Sun, Oct 13, 2013 at 09:50:59PM +0200, Bastien Nocera wrote: > > > > Instead of using the kernel's hybrid sleep, use the firmware for > > laptops that support Intel Rapid Start, as explained in: > > http://mjg59.dreamwidth.

Re: [systemd-devel] [PATCH] Add support for Intel Rapid Start

2013-10-13 Thread Lennart Poettering
On Sun, 13.10.13 22:40, Zbigniew Jędrzejewski-Szmek (zbys...@in.waw.pl) wrote: Humm. So I am very conservative regarding adding vendor specific hacks to userspace that go that low-level into the system. Given that the kernel has support for interl rapidstart anyway, why isn't this exposed properly

Re: [systemd-devel] [PATCH] Add support for Intel Rapid Start

2013-10-13 Thread Zbigniew Jędrzejewski-Szmek
On Sun, Oct 13, 2013 at 09:50:59PM +0200, Bastien Nocera wrote: > > Instead of using the kernel's hybrid sleep, use the firmware for > laptops that support Intel Rapid Start, as explained in: > http://mjg59.dreamwidth.org/26022.html > and implemented in: > https://git.kernel.org/cgit/linux/kernel/

Re: [systemd-devel] [PATCH] Add support for Intel Rapid Start

2013-10-13 Thread Matthew Garrett
On Sun, Oct 13, 2013 at 09:50:59PM +0200, Bastien Nocera wrote: > +r = > read_one_line_file("/sys/bus/acpi/devices/INT3392:00/wakeup_events", &p); /sys/bus/acpi/drivers/intel_rapid_start/*/wakeup_events would be better, if there's any way to use wildcards here. It's possible that later

Re: [systemd-devel] [PATCH] Add support for Intel Rapid Start

2013-10-13 Thread Matthew Garrett
On Sun, Oct 13, 2013 at 10:09:28PM +0200, Bastien Nocera wrote: > On Sun, 2013-10-13 at 20:57 +0100, Matthew Garrett wrote: > > On Sun, Oct 13, 2013 at 09:50:59PM +0200, Bastien Nocera wrote: > > > > > +r = > > > read_one_line_file("/sys/bus/acpi/devices/INT3392:00/wakeup_events", &p); >

Re: [systemd-devel] [PATCH] Add support for Intel Rapid Start

2013-10-13 Thread Bastien Nocera
On Sun, 2013-10-13 at 20:57 +0100, Matthew Garrett wrote: > On Sun, Oct 13, 2013 at 09:50:59PM +0200, Bastien Nocera wrote: > > > +r = > > read_one_line_file("/sys/bus/acpi/devices/INT3392:00/wakeup_events", &p); > > /sys/bus/acpi/drivers/intel_rapid_start/*/wakeup_events would be better