From 091de12bf4b91c9326c84a11ce10d51ca41fee0b Mon Sep 17 00:00:00 2001 From: Scott Anderson Date: Wed, 21 Jun 2017 12:47:53 +1200 Subject: [PATCH] Changed session signal to only emit on DRM changes --- backend/drm/backend.c | 48 ++++++++++------------------------ example/simple.c | 2 ++ include/backend/drm.h | 3 +-- include/wlr/session.h | 11 +++----- session/direct.c | 4 +-- session/logind.c | 61 +++++++++++++++++++++++++++---------------- 6 files changed, 60 insertions(+), 69 deletions(-) diff --git a/backend/drm/backend.c b/backend/drm/backend.c index 81a212d2..8859022f 100644 --- a/backend/drm/backend.c +++ b/backend/drm/backend.c @@ -40,36 +40,22 @@ static struct wlr_backend_impl backend_impl = { .destroy = wlr_drm_backend_destroy }; -static void device_paused(struct wl_listener *listener, void *data) { - struct wlr_backend_state *drm = wl_container_of(listener, drm, device_paused); - struct device_arg *arg = data; +static void session_signal(struct wl_listener *listener, void *data) { + struct wlr_backend_state *drm = wl_container_of(listener, drm, session_signal); + struct wlr_session *session = data; - // TODO: Actually pause the renderer or something. - // We currently just expect it to fail its next pageflip. + if (session->active) { + wlr_log(L_INFO, "DRM fd resumed"); - if (arg->dev == drm->dev) { + for (size_t i = 0; i < drm->outputs->length; ++i) { + struct wlr_output_state *output = drm->outputs->items[i]; + wlr_drm_output_start_renderer(output); + } + } else { wlr_log(L_INFO, "DRM fd paused"); - } -} -static void device_resumed(struct wl_listener *listener, void *data) { - struct wlr_backend_state *drm = wl_container_of(listener, drm, device_resumed); - struct device_arg *arg = data; - - if (arg->dev != drm->dev) { - return; - } - - if (dup2(arg->fd, drm->fd) < 0) { - wlr_log(L_ERROR, "dup2 failed: %s", strerror(errno)); - return; - } - - wlr_log(L_INFO, "DRM fd resumed"); - - for (size_t i = 0; i < drm->outputs->length; ++i) { - struct wlr_output_state *output = drm->outputs->items[i]; - wlr_drm_output_start_renderer(output); + // TODO: Actually pause the renderer or something. + // We currently just expect it to fail its next pageflip. } } @@ -139,14 +125,8 @@ struct wlr_backend *wlr_drm_backend_create(struct wl_display *display, goto error_fd; } - wl_list_init(&state->device_paused.link); - wl_list_init(&state->device_paused.link); - - state->device_paused.notify = device_paused; - state->device_resumed.notify = device_resumed; - - wl_signal_add(&session->device_paused, &state->device_paused); - wl_signal_add(&session->device_resumed, &state->device_resumed); + state->session_signal.notify = session_signal; + wl_signal_add(&session->session_signal, &state->session_signal); // TODO: what is the difference between the per-output renderer and this // one? diff --git a/example/simple.c b/example/simple.c index f3a66db9..16d8d460 100644 --- a/example/simple.c +++ b/example/simple.c @@ -42,6 +42,8 @@ static void handle_keyboard_key(struct keyboard_state *kbstate, xkb_keysym_t sym, enum wlr_key_state key_state) { if (sym == XKB_KEY_Escape) { kbstate->compositor->exit = true; + } else if (sym >= XKB_KEY_F1 && sym <= XKB_KEY_F12) { + wlr_session_change_vt(kbstate->compositor->session, sym - XKB_KEY_F1 + 1); } } diff --git a/include/backend/drm.h b/include/backend/drm.h index 10850d1f..7d90ce57 100644 --- a/include/backend/drm.h +++ b/include/backend/drm.h @@ -34,8 +34,7 @@ struct wlr_backend_state { struct wlr_backend *backend; struct wl_event_source *drm_event; - struct wl_listener device_paused; - struct wl_listener device_resumed; + struct wl_listener session_signal; struct wl_listener drm_invalidated; uint32_t taken_crtcs; diff --git a/include/wlr/session.h b/include/wlr/session.h index 28ccb526..4a5b2174 100644 --- a/include/wlr/session.h +++ b/include/wlr/session.h @@ -1,22 +1,17 @@ #ifndef WLR_SESSION_H #define WLR_SESSION_H +#include #include #include struct session_impl; -// Passed to the listeners of device_paused/resumed -struct device_arg { - dev_t dev; - int fd; // Only for device_resumed -}; - struct wlr_session { const struct session_impl *impl; - struct wl_signal device_paused; - struct wl_signal device_resumed; + bool active; + struct wl_signal session_signal; }; struct wlr_session *wlr_session_start(struct wl_display *disp); diff --git a/session/direct.c b/session/direct.c index c735dae4..c2b155e5 100644 --- a/session/direct.c +++ b/session/direct.c @@ -45,8 +45,8 @@ static struct wlr_session *direct_session_start(struct wl_display *disp) { wlr_log(L_INFO, "Successfully loaded direct session"); session->base.impl = &session_direct; - wl_signal_init(&session->base.device_paused); - wl_signal_init(&session->base.device_resumed); + session->base.active = true; + wl_signal_init(&session->base.session_signal); return &session->base; } diff --git a/session/logind.c b/session/logind.c index 73cd70f2..ff710262 100644 --- a/session/logind.c +++ b/session/logind.c @@ -14,6 +14,8 @@ #include #include "common/log.h" +enum { DRM_MAJOR = 226 }; + const struct session_impl session_logind; struct logind_session { @@ -25,12 +27,15 @@ struct logind_session { char *id; char *path; char *seat; + + int drm_fd; }; static int logind_take_device(struct wlr_session *restrict base, const char *restrict path) { struct logind_session *session = wl_container_of(base, session, base); + wlr_log(L_DEBUG, "Taking '%s'", path); int ret; int fd = -1; sd_bus_message *msg = NULL; @@ -58,7 +63,7 @@ static int logind_take_device(struct wlr_session *restrict base, goto error; } - // The original fd seem to be closed when the message is freed + // The original fd seems to be closed when the message is freed // so we just clone it. fd = fcntl(fd, F_DUPFD_CLOEXEC, 0); if (fd == -1) { @@ -67,6 +72,10 @@ static int logind_take_device(struct wlr_session *restrict base, goto error; } + if (major(st.st_rdev) == DRM_MAJOR) { + session->drm_fd = fd; + } + error: sd_bus_error_free(&error); sd_bus_message_unref(msg); @@ -93,6 +102,10 @@ static void logind_release_device(struct wlr_session *base, int fd) { wlr_log(L_ERROR, "Failed to release device '%d'", fd); } + if (major(st.st_rdev) == DRM_MAJOR) { + session->drm_fd = -1; + } + sd_bus_error_free(&error); sd_bus_message_unref(msg); } @@ -185,30 +198,32 @@ static int session_removed(sd_bus_message *msg, void *userdata, sd_bus_error *re } static int pause_device(sd_bus_message *msg, void *userdata, sd_bus_error *ret_error) { - wlr_log(L_INFO, "PauseDevice signal received"); struct logind_session *session = userdata; int ret; uint32_t major, minor; - ret = sd_bus_message_read(msg, "uu", &major, &minor); + const char *type; + ret = sd_bus_message_read(msg, "uus", &major, &minor, &type); if (ret < 0) { wlr_log(L_ERROR, "Failed to parse D-Bus response for PauseDevice: %s", strerror(-ret)); goto error; } + wlr_log(L_INFO, "PauseDevice signal received: (%lu) %s", makedev(major, minor), type); - struct device_arg arg = { - .dev = makedev(major, minor), - .fd = -1, - }; + if (major == DRM_MAJOR) { + session->base.active = false; + wl_signal_emit(&session->base.session_signal, session); + } - wl_signal_emit(&session->base.device_paused, &arg); - - ret = sd_bus_call_method(session->bus, "org.freedesktop.login1", - session->path, "org.freedesktop.login1.Session", "PauseDeviceComplete", - ret_error, &msg, "uu", major, minor); - if (ret < 0) { - wlr_log(L_ERROR, "Failed to send PauseDeviceComplete signal"); + if (strcmp(type, "pause") == 0) { + ret = sd_bus_call_method(session->bus, "org.freedesktop.login1", + session->path, "org.freedesktop.login1.Session", "PauseDeviceComplete", + ret_error, &msg, "uu", major, minor); + if (ret < 0) { + wlr_log(L_ERROR, "Failed to send PauseDeviceComplete signal: %s", + strerror(-ret)); + } } error: @@ -216,7 +231,6 @@ error: } static int resume_device(sd_bus_message *msg, void *userdata, sd_bus_error *ret_error) { - wlr_log(L_INFO, "ResumeDevice signal received"); struct logind_session *session = userdata; int ret; @@ -228,13 +242,13 @@ static int resume_device(sd_bus_message *msg, void *userdata, sd_bus_error *ret_ strerror(-ret)); goto error; } + wlr_log(L_INFO, "ResumeDevice signal received (%lu)", makedev(major, minor)); - struct device_arg arg = { - .dev = makedev(major, minor), - .fd = fd, - }; - - wl_signal_emit(&session->base.device_resumed, &arg); + if (major == DRM_MAJOR) { + dup2(fd, session->drm_fd); + session->base.active = true; + wl_signal_emit(&session->base.session_signal, session); + } error: return 0; @@ -334,9 +348,10 @@ static struct wlr_session *logind_session_start(struct wl_display *disp) { wlr_log(L_INFO, "Successfully loaded logind session"); + session->drm_fd = -1; session->base.impl = &session_logind; - wl_signal_init(&session->base.device_paused); - wl_signal_init(&session->base.device_resumed); + session->base.active = true; + wl_signal_init(&session->base.session_signal); return &session->base; error_bus: