public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
* [RFC] [gdb/cli] Add maint info frame-unwinders
@ 2023-02-09 13:06 Tom de Vries
  2023-02-09 13:53 ` Eli Zaretskii
  2023-02-09 14:41 ` Tom Tromey
  0 siblings, 2 replies; 6+ messages in thread
From: Tom de Vries @ 2023-02-09 13:06 UTC (permalink / raw)
  To: gdb-patches

Add a new command "maint info frame-unwinders"
...
(gdb) help maint info frame-unwinders
List frame unwinders, starting with the highest priority.
...

Output for i386:
...
$ gdb -q -batch -ex "set arch i386" -ex "maint info frame-unwinders"
The target architecture is set to "i386".
dummy                   DUMMY_FRAME
dwarf2 tailcall         TAILCALL_FRAME
inline                  INLINE_FRAME
i386 epilogue           NORMAL_FRAME
dwarf2                  NORMAL_FRAME
dwarf2 signal           SIGTRAMP_FRAME
i386 stack tramp        NORMAL_FRAME
i386 sigtramp           SIGTRAMP_FRAME
i386 prologue           NORMAL_FRAME
...

Output for x86_64:
...
$ gdb -q -batch -ex "set arch i386:x86-64" -ex "maint info frame-unwinders"
The target architecture is set to "i386:x86-64".
dummy                   DUMMY_FRAME
dwarf2 tailcall         TAILCALL_FRAME
inline                  INLINE_FRAME
python                  NORMAL_FRAME
amd64 epilogue          NORMAL_FRAME
i386 epilogue           NORMAL_FRAME
dwarf2                  NORMAL_FRAME
dwarf2 signal           SIGTRAMP_FRAME
amd64 sigtramp          SIGTRAMP_FRAME
amd64 prologue          NORMAL_FRAME
i386 stack tramp        NORMAL_FRAME
i386 sigtramp           SIGTRAMP_FRAME
i386 prologue           NORMAL_FRAME
...

Tested on x86_64-linux.
---
 gdb/doc/gdb.texinfo |  4 ++++
 gdb/frame-unwind.c  | 28 ++++++++++++++++++++++++++++
 gdb/frame.c         |  4 ++--
 gdb/frame.h         |  4 ++++
 4 files changed, 38 insertions(+), 2 deletions(-)

diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo
index eebb6dd422a..709d4766317 100644
--- a/gdb/doc/gdb.texinfo
+++ b/gdb/doc/gdb.texinfo
@@ -40986,6 +40986,10 @@ with the DWARF frame unwinders enabled.
 If DWARF frame unwinders are not supported for a particular target
 architecture, then enabling this flag does not cause them to be used.
 
+@kindex maint info frame-unwinders
+@item maint info frame-unwinders
+List frame unwinders, starting with the highest priority.
+
 @kindex maint set worker-threads
 @kindex maint show worker-threads
 @item maint set worker-threads
diff --git a/gdb/frame-unwind.c b/gdb/frame-unwind.c
index b708c64f967..43321e22230 100644
--- a/gdb/frame-unwind.c
+++ b/gdb/frame-unwind.c
@@ -28,6 +28,7 @@
 #include "target.h"
 #include "gdbarch.h"
 #include "dwarf2/frame-tailcall.h"
+#include "cli/cli-cmds.h"
 
 struct frame_unwind_table_entry
 {
@@ -337,3 +338,30 @@ frame_unwind_got_address (frame_info_ptr frame, int regnum,
 	     register_type (gdbarch, regnum), addr);
   return reg_val;
 }
+
+/* Implement "maintenance info frame-unwinders" command.  */
+
+static void
+maintenance_info_frame_unwinders (const char *args, int from_tty)
+{
+  struct gdbarch *gdbarch = target_gdbarch ();
+  struct frame_unwind_table *table = get_frame_unwind_table (gdbarch);
+
+  for (struct frame_unwind_table_entry *entry = table->list; entry != NULL;
+       entry = entry->next)
+    {
+      const char *name = entry->unwinder->name;
+      const char *type = frame_type_str (entry->unwinder->type);
+
+      gdb_printf (gdb_stdout, "%-16s\t%-16s\n", name, type);
+    }
+}
+
+void _initialize_frame_unwind ();
+void
+_initialize_frame_unwind ()
+{
+  add_cmd ("frame-unwinders", class_maintenance, maintenance_info_frame_unwinders,
+	   _("List frame unwinders, starting with the highest priority."),
+	   &maintenanceinfolist);
+}
diff --git a/gdb/frame.c b/gdb/frame.c
index 9235a2ceb38..c69a3ea0cb0 100644
--- a/gdb/frame.c
+++ b/gdb/frame.c
@@ -439,9 +439,9 @@ frame_id::to_string () const
   return res;
 }
 
-/* Return a string representation of TYPE.  */
+/* See frame.h.  */
 
-static const char *
+const char *
 frame_type_str (frame_type type)
 {
   switch (type)
diff --git a/gdb/frame.h b/gdb/frame.h
index 4a99bd660df..6ed8db0af56 100644
--- a/gdb/frame.h
+++ b/gdb/frame.h
@@ -203,6 +203,10 @@ enum frame_type
   SENTINEL_FRAME
 };
 
+/* Return a string representation of TYPE.  */
+
+extern const char *frame_type_str (frame_type type);
+
 /* A wrapper for "frame_info *".  frame_info objects are invalidated
    whenever reinit_frame_cache is called.  This class arranges to
    invalidate the pointer when appropriate.  This is done to help

base-commit: c920e5cc604c5b20f9af7c75402eea94aa1e11c6
-- 
2.35.3


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [RFC] [gdb/cli] Add maint info frame-unwinders
  2023-02-09 13:06 [RFC] [gdb/cli] Add maint info frame-unwinders Tom de Vries
@ 2023-02-09 13:53 ` Eli Zaretskii
  2023-02-10  9:15   ` Tom de Vries
  2023-02-09 14:41 ` Tom Tromey
  1 sibling, 1 reply; 6+ messages in thread
From: Eli Zaretskii @ 2023-02-09 13:53 UTC (permalink / raw)
  To: Tom de Vries; +Cc: gdb-patches

> Date: Thu,  9 Feb 2023 14:06:58 +0100
> From: Tom de Vries via Gdb-patches <gdb-patches@sourceware.org>
> 
> Add a new command "maint info frame-unwinders"

Thanks.

> +@kindex maint info frame-unwinders
> +@item maint info frame-unwinders
> +List frame unwinders, starting with the highest priority.
   ^^^^^^^^^^^^^^^^^^^^
I'd suggest "List the frame unwinders currently in effect", or
something like it.  "List frame unwinders" is too terse, and possibly
also not entirely correct English-wise.

What about NEWS? should this be called out there?

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [RFC] [gdb/cli] Add maint info frame-unwinders
  2023-02-09 13:06 [RFC] [gdb/cli] Add maint info frame-unwinders Tom de Vries
  2023-02-09 13:53 ` Eli Zaretskii
@ 2023-02-09 14:41 ` Tom Tromey
  2023-02-10  9:10   ` Tom de Vries
  1 sibling, 1 reply; 6+ messages in thread
From: Tom Tromey @ 2023-02-09 14:41 UTC (permalink / raw)
  To: Tom de Vries via Gdb-patches; +Cc: Tom de Vries

>>>>> "Tom" == Tom de Vries via Gdb-patches <gdb-patches@sourceware.org> writes:

Tom> Add a new command "maint info frame-unwinders"
Tom> ...
Tom> (gdb) help maint info frame-unwinders
Tom> List frame unwinders, starting with the highest priority.
Tom> ...

Makes total sense to me.

Reviewed-By: Tom Tromey <tom@tromey.com>

Tom

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [RFC] [gdb/cli] Add maint info frame-unwinders
  2023-02-09 14:41 ` Tom Tromey
@ 2023-02-10  9:10   ` Tom de Vries
  0 siblings, 0 replies; 6+ messages in thread
From: Tom de Vries @ 2023-02-10  9:10 UTC (permalink / raw)
  To: Tom Tromey, Tom de Vries via Gdb-patches

On 2/9/23 15:41, Tom Tromey wrote:
>>>>>> "Tom" == Tom de Vries via Gdb-patches <gdb-patches@sourceware.org> writes:
> 
> Tom> Add a new command "maint info frame-unwinders"
> Tom> ...
> Tom> (gdb) help maint info frame-unwinders
> Tom> List frame unwinders, starting with the highest priority.
> Tom> ...
> 
> Makes total sense to me.
> 
> Reviewed-By: Tom Tromey <tom@tromey.com>

Thanks for the review, added the tag.

- Tom


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [RFC] [gdb/cli] Add maint info frame-unwinders
  2023-02-09 13:53 ` Eli Zaretskii
@ 2023-02-10  9:15   ` Tom de Vries
  2023-02-10 11:45     ` Eli Zaretskii
  0 siblings, 1 reply; 6+ messages in thread
From: Tom de Vries @ 2023-02-10  9:15 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: gdb-patches

[-- Attachment #1: Type: text/plain, Size: 769 bytes --]

On 2/9/23 14:53, Eli Zaretskii wrote:
>> Date: Thu,  9 Feb 2023 14:06:58 +0100
>> From: Tom de Vries via Gdb-patches <gdb-patches@sourceware.org>
>>
>> Add a new command "maint info frame-unwinders"
> 
> Thanks.
> 
>> +@kindex maint info frame-unwinders
>> +@item maint info frame-unwinders
>> +List frame unwinders, starting with the highest priority.
>     ^^^^^^^^^^^^^^^^^^^^
> I'd suggest "List the frame unwinders currently in effect", or
> something like it.  "List frame unwinders" is too terse, and possibly
> also not entirely correct English-wise.
> 

Thanks for spotting that.

Fixed in both the docs and the help message.

> What about NEWS? should this be called out there?

Added.

I've also added a Reviewed-By tag.

Any further comments?

Thanks,
- Tom

[-- Attachment #2: 0001-gdb-cli-Add-maint-info-frame-unwinders.patch --]
[-- Type: text/x-patch, Size: 5249 bytes --]

From 0e1f0f3797faceaa4a13d3bf11162da2df6ea5d4 Mon Sep 17 00:00:00 2001
From: Tom de Vries <tdevries@suse.de>
Date: Thu, 9 Feb 2023 09:57:34 +0100
Subject: [PATCH] [gdb/cli] Add maint info frame-unwinders

Add a new command "maint info frame-unwinders":
...
(gdb) help maint info frame-unwinders
List the frame unwinders currently in effect, starting with the highest \
  priority.
...

Output for i386:
...
$ gdb -q -batch -ex "set arch i386" -ex "maint info frame-unwinders"
The target architecture is set to "i386".
dummy                   DUMMY_FRAME
dwarf2 tailcall         TAILCALL_FRAME
inline                  INLINE_FRAME
i386 epilogue           NORMAL_FRAME
dwarf2                  NORMAL_FRAME
dwarf2 signal           SIGTRAMP_FRAME
i386 stack tramp        NORMAL_FRAME
i386 sigtramp           SIGTRAMP_FRAME
i386 prologue           NORMAL_FRAME
...

Output for x86_64:
...
$ gdb -q -batch -ex "set arch i386:x86-64" -ex "maint info frame-unwinders"
The target architecture is set to "i386:x86-64".
dummy                   DUMMY_FRAME
dwarf2 tailcall         TAILCALL_FRAME
inline                  INLINE_FRAME
python                  NORMAL_FRAME
amd64 epilogue          NORMAL_FRAME
i386 epilogue           NORMAL_FRAME
dwarf2                  NORMAL_FRAME
dwarf2 signal           SIGTRAMP_FRAME
amd64 sigtramp          SIGTRAMP_FRAME
amd64 prologue          NORMAL_FRAME
i386 stack tramp        NORMAL_FRAME
i386 sigtramp           SIGTRAMP_FRAME
i386 prologue           NORMAL_FRAME
...

Tested on x86_64-linux.

Reviewed-By: Tom Tromey <tom@tromey.com>
Reviewed-By: Eli Zaretskii <eliz@gnu.org>
---
 gdb/NEWS            |  4 ++++
 gdb/doc/gdb.texinfo |  4 ++++
 gdb/frame-unwind.c  | 32 ++++++++++++++++++++++++++++++++
 gdb/frame.c         |  4 ++--
 gdb/frame.h         |  4 ++++
 5 files changed, 46 insertions(+), 2 deletions(-)

diff --git a/gdb/NEWS b/gdb/NEWS
index 1567cbea9bd..b85923cf80d 100644
--- a/gdb/NEWS
+++ b/gdb/NEWS
@@ -52,6 +52,10 @@ maintenance print record-instruction [ N ]
   prints how GDB would undo the N-th previous instruction, and if N is
   positive, it prints how GDB will redo the N-th following instruction.
 
+maintenance info frame-unwinders
+  List the frame unwinders currently in effect, starting with the highest
+  priority.
+
 * MI changes
 
 ** mi now reports 'no-history' as a stop reason when hitting the end of the
diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo
index eebb6dd422a..7b128053b5a 100644
--- a/gdb/doc/gdb.texinfo
+++ b/gdb/doc/gdb.texinfo
@@ -40986,6 +40986,10 @@ with the DWARF frame unwinders enabled.
 If DWARF frame unwinders are not supported for a particular target
 architecture, then enabling this flag does not cause them to be used.
 
+@kindex maint info frame-unwinders
+@item maint info frame-unwinders
+List the frame unwinders currently in effect, starting with the highest priority.
+
 @kindex maint set worker-threads
 @kindex maint show worker-threads
 @item maint set worker-threads
diff --git a/gdb/frame-unwind.c b/gdb/frame-unwind.c
index b708c64f967..76601faa479 100644
--- a/gdb/frame-unwind.c
+++ b/gdb/frame-unwind.c
@@ -28,6 +28,7 @@
 #include "target.h"
 #include "gdbarch.h"
 #include "dwarf2/frame-tailcall.h"
+#include "cli/cli-cmds.h"
 
 struct frame_unwind_table_entry
 {
@@ -337,3 +338,34 @@ frame_unwind_got_address (frame_info_ptr frame, int regnum,
 	     register_type (gdbarch, regnum), addr);
   return reg_val;
 }
+
+/* Implement "maintenance info frame-unwinders" command.  */
+
+static void
+maintenance_info_frame_unwinders (const char *args, int from_tty)
+{
+  struct gdbarch *gdbarch = target_gdbarch ();
+  struct frame_unwind_table *table = get_frame_unwind_table (gdbarch);
+
+  for (struct frame_unwind_table_entry *entry = table->list; entry != NULL;
+       entry = entry->next)
+    {
+      const char *name = entry->unwinder->name;
+      const char *type = frame_type_str (entry->unwinder->type);
+
+      gdb_printf (gdb_stdout, "%-16s\t%-16s\n", name, type);
+    }
+}
+
+void _initialize_frame_unwind ();
+void
+_initialize_frame_unwind ()
+{
+  /* Add "maint info frame-unwinders".  */
+  add_cmd ("frame-unwinders",
+	   class_maintenance,
+	   maintenance_info_frame_unwinders,
+	   _("List the frame unwinders currently in effect, "
+	     "starting with the highest priority."),
+	   &maintenanceinfolist);
+}
diff --git a/gdb/frame.c b/gdb/frame.c
index 9235a2ceb38..c69a3ea0cb0 100644
--- a/gdb/frame.c
+++ b/gdb/frame.c
@@ -439,9 +439,9 @@ frame_id::to_string () const
   return res;
 }
 
-/* Return a string representation of TYPE.  */
+/* See frame.h.  */
 
-static const char *
+const char *
 frame_type_str (frame_type type)
 {
   switch (type)
diff --git a/gdb/frame.h b/gdb/frame.h
index 4a99bd660df..6ed8db0af56 100644
--- a/gdb/frame.h
+++ b/gdb/frame.h
@@ -203,6 +203,10 @@ enum frame_type
   SENTINEL_FRAME
 };
 
+/* Return a string representation of TYPE.  */
+
+extern const char *frame_type_str (frame_type type);
+
 /* A wrapper for "frame_info *".  frame_info objects are invalidated
    whenever reinit_frame_cache is called.  This class arranges to
    invalidate the pointer when appropriate.  This is done to help

base-commit: aa1807419ba0b6fa4a8b497babc1c89b27e667e5
-- 
2.35.3


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [RFC] [gdb/cli] Add maint info frame-unwinders
  2023-02-10  9:15   ` Tom de Vries
@ 2023-02-10 11:45     ` Eli Zaretskii
  0 siblings, 0 replies; 6+ messages in thread
From: Eli Zaretskii @ 2023-02-10 11:45 UTC (permalink / raw)
  To: Tom de Vries; +Cc: gdb-patches

> Date: Fri, 10 Feb 2023 10:15:42 +0100
> Cc: gdb-patches@sourceware.org
> From: Tom de Vries <tdevries@suse.de>
> 
> >> +@kindex maint info frame-unwinders
> >> +@item maint info frame-unwinders
> >> +List frame unwinders, starting with the highest priority.
> >     ^^^^^^^^^^^^^^^^^^^^
> > I'd suggest "List the frame unwinders currently in effect", or
> > something like it.  "List frame unwinders" is too terse, and possibly
> > also not entirely correct English-wise.
> > 
> 
> Thanks for spotting that.
> 
> Fixed in both the docs and the help message.
> 
> > What about NEWS? should this be called out there?
> 
> Added.
> 
> I've also added a Reviewed-By tag.

Thanks.

> Any further comments?

No further comments to the documentation parts.

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2023-02-10 11:46 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-09 13:06 [RFC] [gdb/cli] Add maint info frame-unwinders Tom de Vries
2023-02-09 13:53 ` Eli Zaretskii
2023-02-10  9:15   ` Tom de Vries
2023-02-10 11:45     ` Eli Zaretskii
2023-02-09 14:41 ` Tom Tromey
2023-02-10  9:10   ` Tom de Vries

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).