fbb51ac204
Tue Nov 15 2005 Jeff Johnston <jjohnstn@redhat.com> 6.3.0.0-1.86 - Fix ia64 user-specified SIGILL handling error. - Bugzilla 165038.
123 lines
4.2 KiB
Diff
123 lines
4.2 KiB
Diff
2005-11-15 Jeff Johnston <jjohnstn@redhat.com>
|
|
|
|
* thread-db.c (thread_db_wait): Don't bother continuing if
|
|
the wait result indicates the program terminated with a signal.
|
|
* linux-nat.c (linux_nat_wait): For SIGILL and SIGTRAP, don't
|
|
throw away the event if the user has specified nostop noprint.
|
|
|
|
gdb/testsuite:
|
|
|
|
2005-11-15 Jeff Johnston <jjohnstn@redhat.com>
|
|
|
|
* gdb.arch/ia64-sigill.c: New test.
|
|
* gdb.arch/ia64-sigill.exp: Ditto.
|
|
|
|
--- gdb-6.3/gdb/testsuite/gdb.arch/ia64-sigill.exp.fix 2005-11-15 17:52:56.000000000 -0500
|
|
+++ gdb-6.3/gdb/testsuite/gdb.arch/ia64-sigill.exp 2005-11-15 17:52:10.000000000 -0500
|
|
@@ -0,0 +1,59 @@
|
|
+# Copyright 2005 Free Software Foundation, Inc.
|
|
+
|
|
+# This program is free software; you can redistribute it and/or modify
|
|
+# it under the terms of the GNU General Public License as published by
|
|
+# the Free Software Foundation; either version 2 of the License, or
|
|
+# (at your option) any later version.
|
|
+#
|
|
+# This program is distributed in the hope that it will be useful,
|
|
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
+# GNU General Public License for more details.
|
|
+#
|
|
+# You should have received a copy of the GNU General Public License
|
|
+# along with this program; if not, write to the Free Software
|
|
+# Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
|
|
+
|
|
+# Please email any bugs, comments, and/or additions to this file to:
|
|
+# bug-gdb@prep.ai.mit.edu
|
|
+
|
|
+# This file was written by Jeff Johnston (jjohnstn@redhat.com)
|
|
+
|
|
+if $tracelevel then {
|
|
+ strace $tracelevel
|
|
+}
|
|
+
|
|
+#
|
|
+# test running programs
|
|
+#
|
|
+set prms_id 0
|
|
+set bug_id 0
|
|
+
|
|
+if ![istarget "ia64-*-*"] then {
|
|
+ return
|
|
+}
|
|
+
|
|
+set testfile "ia64-sigill"
|
|
+set srcfile ${testfile}.c
|
|
+set binfile ${objdir}/${subdir}/${testfile}
|
|
+
|
|
+# Deliberately compile with pthreads, even though test is single-threaded.
|
|
+# We want to force gdb thread code to be exercised.
|
|
+if { [gdb_compile_pthreads "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug additional_flags=-w}] != "" } {
|
|
+ gdb_suppress_entire_file "Testcase compile failed, so all tests in this file will automatically fail."
|
|
+}
|
|
+
|
|
+if [get_compiler_info ${binfile}] {
|
|
+ return -1
|
|
+}
|
|
+
|
|
+gdb_exit
|
|
+gdb_start
|
|
+gdb_reinitialize_dir $srcdir/$subdir
|
|
+gdb_load ${binfile}
|
|
+
|
|
+# We set up SIGILL nostop, noprint, pass and then run the program.
|
|
+# We expect to just see a normal run.
|
|
+gdb_test "handle SIGILL nostop noprint" "SIGILL.*No.*No.*Yes.*" "handle sigill"
|
|
+gdb_test "run" "Starting program.*ia64-sigill.*\[New thread.*\].*hello world.*Program exited normally." "run to exit"
|
|
+
|
|
--- gdb-6.3/gdb/testsuite/gdb.arch/ia64-sigill.c.fix 2005-11-15 17:53:02.000000000 -0500
|
|
+++ gdb-6.3/gdb/testsuite/gdb.arch/ia64-sigill.c 2005-11-15 17:52:05.000000000 -0500
|
|
@@ -0,0 +1,8 @@
|
|
+#include <stdio.h>
|
|
+
|
|
+int main()
|
|
+{
|
|
+ printf ("hello world\n");
|
|
+ return 0;
|
|
+}
|
|
+
|
|
--- gdb-6.3/gdb/thread-db.c.fix 2005-11-15 17:51:09.000000000 -0500
|
|
+++ gdb-6.3/gdb/thread-db.c 2005-11-15 17:51:50.000000000 -0500
|
|
@@ -930,7 +930,8 @@ thread_db_wait (ptid_t ptid, struct targ
|
|
post-processing and bail out early. */
|
|
return ptid;
|
|
|
|
- if (ourstatus->kind == TARGET_WAITKIND_EXITED)
|
|
+ if (ourstatus->kind == TARGET_WAITKIND_EXITED
|
|
+ || ourstatus->kind == TARGET_WAITKIND_SIGNALLED)
|
|
return pid_to_ptid (-1);
|
|
|
|
if (ourstatus->kind == TARGET_WAITKIND_STOPPED
|
|
--- gdb-6.3/gdb/linux-nat.c.fix 2005-11-15 17:51:14.000000000 -0500
|
|
+++ gdb-6.3/gdb/linux-nat.c 2005-11-15 17:51:46.000000000 -0500
|
|
@@ -2216,7 +2216,8 @@ retry:
|
|
threads can be a bit time-consuming so if we want decent
|
|
performance with heavily multi-threaded programs, especially when
|
|
they're using a high frequency timer, we'd better avoid it if we
|
|
- can. */
|
|
+ can. For possible trap signals like SIGTRAP and SIGILL, don't
|
|
+ avoid reporting. */
|
|
|
|
if (WIFSTOPPED (status))
|
|
{
|
|
@@ -2224,7 +2225,9 @@ retry:
|
|
|
|
if (signal_stop_state (signo) == 0
|
|
&& signal_print_state (signo) == 0
|
|
- && signal_pass_state (signo) == 1)
|
|
+ && signal_pass_state (signo) == 1
|
|
+ && signo != TARGET_SIGNAL_ILL
|
|
+ && signo != TARGET_SIGNAL_TRAP)
|
|
{
|
|
/* FIMXE: kettenis/2001-06-06: Should we resume all threads
|
|
here? It is not clear we should. GDB may not expect
|