summaryrefslogtreecommitdiff
path: root/src/sleep
diff options
context:
space:
mode:
authorLennart Poettering <lennart@poettering.net>2018-10-24 12:57:37 +0200
committerLennart Poettering <lennart@poettering.net>2018-10-24 12:57:37 +0200
commitc695101f47824a6d2142621b439a69f4373f2abf (patch)
tree56bfabfe58977f863b4971795555fe6b920fc27e /src/sleep
parenteabcf200f7ab271f3f684f380464ad2f8e53fb06 (diff)
downloadsystemd-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.c11
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)