logind: allow suspending if there are no displays

With proprietary graphics drivers, there won't be any 'drm' devices in
sysfs, so logind will never suspend the system upon closing the lid,
even if only one (internal) display is connected. This has been reported
by multiple users so far.

IMHO, it's better to suspend the system in this case for safety reasons,
to avoid having nvidia blob users' laptops overheat, for the same reason
that sleep inhibitors are overridden (LidSwitchIgnoreInhibited=yes).
This commit is contained in:
Mantas Mikulėnas 2014-04-28 00:58:56 +03:00 committed by Lennart Poettering
parent 2dcf7ec6ec
commit d36d90933a

View file

@ -79,14 +79,12 @@ int manager_handle_action(
return 0;
}
/* If we have more than one or no displays connected,
* don't react to lid closing. The no display case we
* treat like this under the assumption that there is
* no modern drm driver available. */
/* If we have more than one display connected,
* don't react to lid closing. */
n = manager_count_displays(m);
if (n < 0)
log_warning("Display counting failed: %s", strerror(-n));
else if (n != 1) {
else if (n > 1) {
log_debug("Ignoring lid switch request, %i displays connected.", n);
return 0;
}