diff options
| author |   Marius Vlad <marius.vlad@collabora.com> | 2022-09-20 11:45:09 +0300 |
|---|---|---|
| committer | Marius Vlad <marius.vlad@collabora.com> | 2022-09-23 18:15:11 +0300 |
| commit | 315496ddbb9f6f6079c8cd413507e15d41c3a5e7 (patch) | |
| tree | 8077662e759d0e919841adc982f43c5974c066b1 | |
| parent | f980c0d90329bf83a082c966d69b0015e34c218b (diff) | |
compositor: Stop trapping SIGQUIT
We've been trapping SIGQUIT for a "clean shutdown" since commit 3cad436a
However, sources such as:
http://www.gnu.org/software/libc/manual/html_node/Termination-Signals.html
indicate we probably shouldn't be trapping it at all, as the intent of
SIGQUIT is to leave a core file and debug artifacts from the run.
We should perform the minimal amount of clean up to ensure the system isn't
left in an unusable state - but these days that's performed by other
software such as logind.
We can safely stop trapping SIGQUIT entirely.
(Based on work from https://gitlab.freedesktop.org/wayland/weston/-/merge_requests/817)
Bug-AGL: SPEC-4570
Signed-off-by: Derek Foreman <derek.foreman@collabora.com>
Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
Change-Id: Icf7d4f3e76a58e781bad911a966b65e2798266d0
| -rw-r--r-- | src/compositor.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/compositor.c b/src/compositor.c index e03b860..dbc89e2 100644 --- a/src/compositor.c +++ b/src/compositor.c @@ -1608,7 +1608,7 @@ int wet_main(int argc, char *argv[], const struct weston_testsuite_data *test_da char *cmdline; struct wl_display *display = NULL; struct wl_event_loop *loop; - struct wl_event_source *signals[3] = { 0 }; + struct wl_event_source *signals[2] = { 0 }; struct weston_config_section *section; /* Command line options */ char *backend = NULL; @@ -1709,8 +1709,6 @@ int wet_main(int argc, char *argv[], const struct weston_testsuite_data *test_da display); signals[1] = wl_event_loop_add_signal(loop, SIGINT, on_term_signal, display); - signals[2] = wl_event_loop_add_signal(loop, SIGQUIT, on_term_signal, - display); for (size_t i = 0; i < ARRAY_LENGTH(signals); ++i) if (!signals[i]) |