diff options
author | Lennart Poettering <lennart@poettering.net> | 2018-10-24 12:57:37 +0200 |
---|---|---|
committer | Lennart Poettering <lennart@poettering.net> | 2018-10-24 12:57:37 +0200 |
commit | c695101f47824a6d2142621b439a69f4373f2abf (patch) | |
tree | 56bfabfe58977f863b4971795555fe6b920fc27e /src/sleep | |
parent | eabcf200f7ab271f3f684f380464ad2f8e53fb06 (diff) | |
download | systemd-c695101f47824a6d2142621b439a69f4373f2abf.tar.gz |
sleep: no need to check for resume_offset twice
The W_OK check already checks for existance hence let's remove the F_OK
check.
Diffstat (limited to 'src/sleep')
-rw-r--r-- | src/sleep/sleep.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/src/sleep/sleep.c b/src/sleep/sleep.c index 771a8cd48b..eeaf8d8972 100644 --- a/src/sleep/sleep.c +++ b/src/sleep/sleep.c @@ -54,15 +54,14 @@ static int write_hibernate_location_info(void) { } /* Only available in 4.17+ */ - if (access("/sys/power/resume_offset", F_OK) < 0) { - if (errno == ENOENT) + if (access("/sys/power/resume_offset", W_OK) < 0) { + if (errno == ENOENT) { + log_debug("Kernel too old, can't configure resume offset, ignoring."); return 0; + } - return log_debug_errno(errno, "/sys/power/resume_offset unavailable: %m"); - } - - if (access("/sys/power/resume_offset", W_OK) < 0) return log_debug_errno(errno, "/sys/power/resume_offset not writeable: %m"); + } fd = open(device, O_RDONLY | O_CLOEXEC | O_NONBLOCK); if (fd < 0) |