backlight: ignore error if the backlight device is already removed
authorYu Watanabe <watanabe.yu+github@gmail.com>
Wed, 5 Jan 2022 09:26:46 +0000 (18:26 +0900)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Fri, 7 Jan 2022 14:52:33 +0000 (15:52 +0100)
Fixes #21997.

(cherry picked from commit f0f65087834198d4dabf8b389ddc34223400aab7)

src/backlight/backlight.c

index fd92135fc7541ab96f5e841d5e3a801ac6367d26..5a3095cbbab60d8deb08c909684ba88933bf9bec 100644 (file)
@@ -395,8 +395,16 @@ static int run(int argc, char *argv[]) {
                 return log_error_errno(SYNTHETIC_ERRNO(EINVAL), "Not a backlight or LED device: '%s:%s'", ss, sysname);
 
         r = sd_device_new_from_subsystem_sysname(&device, ss, sysname);
-        if (r < 0)
-                return log_error_errno(r, "Failed to get backlight or LED device '%s:%s': %m", ss, sysname);
+        if (r < 0) {
+                bool ignore = r == -ENODEV;
+
+                /* Some drivers, e.g. for AMD GPU, removes acpi backlight device soon after it is added.
+                 * See issue #21997. */
+                log_full_errno(ignore ? LOG_DEBUG : LOG_ERR, r,
+                               "Failed to get backlight or LED device '%s:%s'%s: %m",
+                               ss, sysname, ignore ? ", ignoring" : "");
+                return ignore ? 0 : r;
+        }
 
         /* If max_brightness is 0, then there is no actual backlight device. This happens on desktops
          * with Asus mainboards that load the eeepc-wmi module. */