From: Gary Benson <gbenson@redhat.com>
To: gdb-patches@sourceware.org
Cc: Andrew Burgess <aburgess@broadcom.com>,
Doug Evans <xdje42@gmail.com>,
Eli Zaretskii <eliz@gnu.org>,
Florian Weimer <fw@deneb.enyo.de>,
Mark Kettenis <mark.kettenis@xs4all.nl>,
Pedro Alves <palves@redhat.com>,
Tom Tromey <tromey@redhat.com>
Subject: [PATCH 0/3 v5] Demangler crash handler
Date: Mon, 09 Jun 2014 15:22:00 -0000 [thread overview]
Message-ID: <20140609152229.GA27494@blade.nx> (raw)
Hi all,
This series is an updated version of the demangler crash handler
I posted last week. The main changes are:
1) check_can_dump_core has been renamed back to the can_dump_core,
and I renamed check_can_dump_core_warn as can_dump_core_warn as
per https://sourceware.org/ml/gdb-patches/2014-06/msg00307.html.
2) can_dump_core and can_dump_core_warn have a new parameter,
limit_kind, to specify which limit to check. The original
calls in internal_vproblem use the hard limit, and the
calls from the crash catcher that this series adds use the
soft limit.
3) The crash catcher prints "Attempting to dump core" to alert
the user that a core file might have been created.
The patches are split as before:
1/3 - adds a new category of internal problem for demangler
warnings. This patch is unchanged from the previous
two versions (PATCH 1/2 v3 and v4).
https://sourceware.org/ml/gdb-patches/2014-06/msg00142.html
https://sourceware.org/ml/gdb-patches/2014-06/msg00252.html
2/3 - refactors and exposes the core-dumping functions in utils.c.
This patch differs from the previous version as described in
points 1 and 2 in the above list.
3/3 - the crash catcher itself. This patch differs from the
previous version as described in points 2 and 3 in the
above list.
I would push all three patches as one commit. The news file entries
for the commit would be:
* New options
maint set catch-demangler-crashes (on|off)
maint show catch-demangler-crashes
Control whether GDB should attempt to catch crashes in the
symbol name demangler.
maint set demangler-warning quit (yes|no|ask)
maint show demangler-warning quit
Control whether GDB should exit if it catches a crash in the
symbol name demangler.
* New commands
maint demangler-warning
Cause GDB to call the internal function demangler_warning and
hence behave as though an internal error in the demangler has
been detected.
Is this ok to commit?
Thanks,
Gary
--
http://gbenson.net/
next reply other threads:[~2014-06-09 15:22 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-09 15:22 Gary Benson [this message]
2014-06-09 15:23 ` [PATCH 1/3 v5] Add new internal problem for demangler warnings Gary Benson
2014-06-09 16:19 ` Eli Zaretskii
2014-06-09 15:24 ` [PATCH 3/3 v5] Demangler crash handler Gary Benson
2014-06-09 16:20 ` Eli Zaretskii
2014-06-09 18:25 ` Gary Benson
2014-07-14 0:31 ` Cygwin build failure following " Pierre Muller
2014-07-14 8:52 ` Corinna Vinschen
2014-07-14 12:10 ` Gary Benson
2014-07-14 8:54 ` Gary Benson
2014-07-14 10:28 ` Andreas Schwab
[not found] ` <20140714114241.GA31867@blade.nx>
2014-07-14 16:13 ` Tom Tromey
2014-07-14 17:52 ` Pierre Muller
2014-06-09 15:24 ` [PATCH 2/3 v5] Refactor and expose core-dumping functionality Gary Benson
2014-06-11 20:41 ` [PATCH 0/3 v5] Demangler crash handler Tom Tromey
2014-06-11 21:03 ` Mark Kettenis
2014-06-19 8:23 ` [COMMITTED PATCH " Gary Benson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140609152229.GA27494@blade.nx \
--to=gbenson@redhat.com \
--cc=aburgess@broadcom.com \
--cc=eliz@gnu.org \
--cc=fw@deneb.enyo.de \
--cc=gdb-patches@sourceware.org \
--cc=mark.kettenis@xs4all.nl \
--cc=palves@redhat.com \
--cc=tromey@redhat.com \
--cc=xdje42@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox