weston: remove SEGV and ABRT handlers
authorPekka Paalanen <pekka.paalanen@collabora.co.uk>
Wed, 7 Feb 2018 10:51:14 +0000 (12:51 +0200)
committerDaniel Stone <daniels@collabora.com>
Fri, 9 Feb 2018 15:16:07 +0000 (15:16 +0000)
commitbb707dc0fe331c9af112a0552b7aa6fde755dd83
treea40270b0d236e503c01fced3df9af12dde39e9d3
parentc34a9f5ca6611bfc64ded66ed464981285c78eeb
weston: remove SEGV and ABRT handlers

Catching an ABRT is kind of ok, catching a SEGV is russian roulette. We
have been quite lucky with it, but I've started hitting crashes inside
malloc() which causes a deadlock when our SEGV handler needs to malloc()
as well (weston_log_timestamp()).

One reason to catch SEGV and ABRT was to attempt to restore the VT on
the DRM-backend. Nowadays that job is done by logind or weston-launch.

The signal handler also printed a backtrace, which for me personally has
been extremely helpful. Arguably it's not necessary though, when we have
core files and services that catch cores. For instance, if using
systemd, 'coredumpctl gdb' is delightfully easy for getting into the
saved core.

Therefore, this code does more harm than it is useful, so remove it. We
also drop an optional dependency to libunwind.

Signed-off-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
Reviewed-by: Daniel Stone <daniels@collabora.com>
Makefile.am
compositor/main.c
configure.ac
man/weston.man