Revert 5fdf2d51c244288ac41443d1bd81365fab7b7b81
authorZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Wed, 21 Nov 2018 09:21:28 +0000 (10:21 +0100)
committerLennart Poettering <lennart@poettering.net>
Wed, 21 Nov 2018 14:04:22 +0000 (15:04 +0100)
commit6d176522f5480ea9e9a83de5ef5ea5e0d95b79cb
tree6aa73058e4daae994218150a318278feb1ee24ec
parent053254e3cb215df3b8c905bc39b920f8817e1c7d
Revert 5fdf2d51c244288ac41443d1bd81365fab7b7b81

This reverts 5fdf2d51c244288ac41443d1bd81365fab7b7b81, except for one improved
log message.

Fixes #10613.

Checking if resume= is configured is a good idea, but it turns out we cannot do
it reliably:
- the code only supported boot options with sd-boot, and it's not very widely
  used. This means that for most systemd we could only check the current
  commandline, not the next one.
- Various systems resume without, e.g. Debian has
  /etc/initramfs-tools/conf.d/resume in the initramfs.

Making those checks better would be possible with enough effort, but there'll
be always new systems that boot in a slightly different way and we would need
to keep adding new cases. Longer term, we want to rely on autodetecting the
resume partition, and then checks like this will not be necessary at all. It is
quite clear from the number of bug reports that the number of poeple impacted
by this is quite high now, so let's just drop this.
src/login/logind-dbus.c
src/shared/sleep-config.c