]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commitdiff
Allow TLS access to work in gdb.server/no-thread-db.exp
authorKevin Buettner <kevinb@redhat.com>
Thu, 24 Apr 2025 04:39:28 +0000 (21:39 -0700)
committerKevin Buettner <kevinb@redhat.com>
Thu, 24 Apr 2025 16:54:42 +0000 (09:54 -0700)
The patches later in the series add GDB-internal TLS support for
certain targets.  This commit updates the "print foo" test in
gdb.server/no-thread-db.exp to accept either a TLS failure (when
libthread_db isn't available) or printing the correct answer, which
will occur when GDB's internal TLS address resolution can be used.

I'm making this change prior to the commits which actually add
the GDB-internal TLS support in order to avoid tripping regression
testers.

Tested-By: Luis Machado <luis.machado@arm.com>
Approved-By: Luis Machado <luis.machado@arm.com>
gdb/testsuite/gdb.server/no-thread-db.exp

index cc2470834ba66d2f7ad080bc19db7efbdb0b6fb9..9fd209030b53ce80fdc2cc6ed68f8879f7ac5cdf 100644 (file)
@@ -57,6 +57,8 @@ gdb_breakpoint ${srcfile}:[gdb_get_line_number "after tls assignment"]
 gdb_continue_to_breakpoint "after tls assignment"
 
 # Printing a tls variable should fail gracefully without a libthread_db.
+# Alternately, the correct answer might be printed due GDB's internal
+# TLS support for some targets.
 set re_exec "\[^\r\n\]*[file tail $binfile]"
 gdb_test "print foo" \
-    "Cannot find thread-local storage for Thread \[^,\]+, executable file $re_exec:\[\r\n\]+Remote target failed to process qGetTLSAddr request"
+    "= 1|(?:Cannot find thread-local storage for Thread \[^,\]+, executable file $re_exec:\[\r\n\]+Remote target failed to process qGetTLSAddr request)"