diff options
author | Ray Strode <rstrode@redhat.com> | 2018-08-07 14:04:44 -0400 |
---|---|---|
committer | Ray Strode <rstrode@redhat.com> | 2018-08-08 10:08:47 -0400 |
commit | 8169cd4fc83beb5c3835a6ee1fafc6911a887a81 (patch) | |
tree | 985397f633e501fb30b5700bcb75c79a90584818 /daemon | |
parent | e1633895bf98fcc1c20be270c83e58af5b6ebe7a (diff) | |
download | gdm-8169cd4fc83beb5c3835a6ee1fafc6911a887a81.tar.gz |
session-worker: don't switch VTs if we're already on the right VT
commit 5b5dccbd shows that switching VTs to the same VT isn't
exactly a no-op. In order to prevent unnecessary wakeups, avoid
switching VTs if the worker is already on the correct VT.
Diffstat (limited to 'daemon')
-rw-r--r-- | daemon/gdm-session-worker.c | 22 |
1 files changed, 16 insertions, 6 deletions
diff --git a/daemon/gdm-session-worker.c b/daemon/gdm-session-worker.c index c1d89cab..d211cdb1 100644 --- a/daemon/gdm-session-worker.c +++ b/daemon/gdm-session-worker.c @@ -968,6 +968,8 @@ jump_to_vt (GdmSessionWorker *worker, { int fd; int active_vt_tty_fd; + int active_vt = -1; + struct vt_stat vt_state = { 0 }; g_debug ("GdmSessionWorker: jumping to VT %d", vt_number); active_vt_tty_fd = open ("/dev/tty0", O_RDWR | O_NOCTTY); @@ -993,12 +995,20 @@ jump_to_vt (GdmSessionWorker *worker, handle_terminal_vt_switches (worker, fd); - if (ioctl (fd, VT_ACTIVATE, vt_number) < 0) { - g_debug ("GdmSessionWorker: couldn't initiate jump to VT %d: %m", - vt_number); - } else if (ioctl (fd, VT_WAITACTIVE, vt_number) < 0) { - g_debug ("GdmSessionWorker: couldn't finalize jump to VT %d: %m", - vt_number); + if (ioctl (fd, VT_GETSTATE, &vt_state) <= 0) { + g_debug ("GdmSessionWorker: couldn't get current VT: %m"); + } else { + active_vt = vt_state.v_active; + } + + if (active_vt != vt_number) { + if (ioctl (fd, VT_ACTIVATE, vt_number) < 0) { + g_debug ("GdmSessionWorker: couldn't initiate jump to VT %d: %m", + vt_number); + } else if (ioctl (fd, VT_WAITACTIVE, vt_number) < 0) { + g_debug ("GdmSessionWorker: couldn't finalize jump to VT %d: %m", + vt_number); + } } close (active_vt_tty_fd); |