public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
From: Tom de Vries <tdevries@suse.de>
To: gdb-patches@sourceware.org
Subject: [PATCH][gdb] Fix warning in foreach_arch selftests
Date: Wed, 1 Jun 2022 12:41:31 +0200	[thread overview]
Message-ID: <20220601104130.GA24730@delia.home> (raw)

Hi,

When running the selftests, I run into:
...
$ gdb -q -batch -ex "maint selftest"
  ...
Running selftest execute_cfa_program::aarch64:ilp32.
warning: A handler for the OS ABI "GNU/Linux" is not built into this
configuration of GDB.  Attempting to continue with the default aarch64:ilp32
settings.
...
and likewise for execute_cfa_program::i8086 and
execute_cfa_program::ia64-elf32.

The warning can easily be reproduced outside the selftests by doing:
...
$ gdb -q -batch -ex "set arch aarch64:ilp32"
...
and can be prevented by first doing "set osabi none".

Fix the warning by setting osabi to none while doing selftests that iterate
over all architectures.

Tested on x86_64-linux.

Any comments?

Thanks,
- Tom

[gdb] Fix warning in foreach_arch selftests

---
 gdb/osabi.c         | 13 +++++++++++++
 gdb/osabi.h         |  6 ++++++
 gdb/selftest-arch.c | 15 ++++++++++++++-
 3 files changed, 33 insertions(+), 1 deletion(-)

diff --git a/gdb/osabi.c b/gdb/osabi.c
index bbd7635532f..c3f221df969 100644
--- a/gdb/osabi.c
+++ b/gdb/osabi.c
@@ -633,6 +633,19 @@ set_osabi (const char *args, int from_tty, struct cmd_list_element *c)
     internal_error (__FILE__, __LINE__, _("Updating OS ABI failed."));
 }
 
+void
+set_osabi (const char *arg)
+{
+  set_osabi_string = arg;
+  set_osabi (NULL, 0, NULL);
+}
+
+const char *
+get_osabi ()
+{
+  return set_osabi_string;
+}
+
 static void
 show_osabi (struct ui_file *file, int from_tty, struct cmd_list_element *c,
 	    const char *value)
diff --git a/gdb/osabi.h b/gdb/osabi.h
index be016732cbc..eb5d88699e7 100644
--- a/gdb/osabi.h
+++ b/gdb/osabi.h
@@ -89,4 +89,10 @@ const char *osabi_triplet_regexp (enum gdb_osabi osabi);
 void generic_elf_osabi_sniff_abi_tag_sections (bfd *, asection *,
 					       enum gdb_osabi *);
 
+/* Set osabi to ARG.  */
+extern void set_osabi (const char *arg);
+
+/* Return current osabi setting.  */
+extern const char *get_osabi ();
+
 #endif /* OSABI_H */
diff --git a/gdb/selftest-arch.c b/gdb/selftest-arch.c
index f434da718d5..a631f52e31e 100644
--- a/gdb/selftest-arch.c
+++ b/gdb/selftest-arch.c
@@ -66,12 +66,25 @@ foreach_arch_test_generator (const std::string &name,
       auto test_fn
 	= ([=] ()
 	   {
+	     /* Prevent warnings when setting architecture with current osabi
+		settings, like:
+		  A handler for the OS ABI "GNU/Linux" is not built into this
+		  configuration of GDB.  Attempting to continue with the
+		  default aarch64:ilp32 settings.  */
+	     const char *save_osabi = get_osabi ();
+	     set_osabi ("none");
+
 	     struct gdbarch_info info;
 	     info.bfd_arch_info = bfd_scan_arch (arch);
 	     struct gdbarch *gdbarch = gdbarch_find_by_info (info);
 	     SELF_CHECK (gdbarch != NULL);
+
 	     function (gdbarch);
-	     reset ();
+
+	     SCOPE_EXIT {
+	       reset ();
+	       set_osabi (save_osabi);
+	     };
 	   });
 
       tests.emplace_back (string_printf ("%s::%s", name.c_str (), arch),

             reply	other threads:[~2022-06-01 10:41 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-06-01 10:41 Tom de Vries [this message]
2022-06-01 14:32 ` Pedro Alves
2022-06-01 16:36   ` Tom de Vries
2022-06-01 17:10     ` Pedro Alves
2022-06-01 19:08     ` Simon Marchi
2022-06-02 15:44       ` Tom de Vries
2022-06-02 17:44         ` Simon Marchi
2022-06-02 18:35           ` Tom de Vries
2022-06-04  7:23             ` Tom de Vries

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=20220601104130.GA24730@delia.home \
    --to=tdevries@suse.de \
    --cc=gdb-patches@sourceware.org \
    /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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).