[ovs-dev] [PATCH] fatal-signal: Catch SIGSEGV and print backtrace.

William Tu u9012063 at gmail.com
Wed Sep 18 20:16:37 UTC 2019


Thanks for the feedback.

On Wed, Sep 18, 2019 at 11:30 AM Ben Pfaff <blp at ovn.org> wrote:
>
> On Tue, Sep 17, 2019 at 04:13:24PM -0700, William Tu wrote:
> > The patch catches the SIGSEGV signal and prints the backtrace
> > using libunwind, hopefully makes it easier to debug.
> >
> > Signed-off-by: William Tu <u9012063 at gmail.com>
>
> I guess my experience is that this sort of thing is sometimes useful but
> ultimately causes problems because it tries to call a lot of functions
> that a signal handler is not supposed to call.

Do you mean the show_backtrace() tries to call too many functions?
What's the concern about calling lots of functions in a signal handler?

>
> It might be a good idea to put some of this in lib/backtrace.[ch] and
> provide some way to enable it at runtime.

I'm hoping this can be enabled by default, so debugging on systems
without ovs debug symbol package or gdb is easier.

Regards,
William


More information about the dev mailing list