open_issues/gdb: 695f61ff0f378e1680964128585044799de27015 (2011-09-06)

author: Thomas Schwinge <thomas@codesourcery.com> 2012-11-06 13:25:02 +0100
committer: Thomas Schwinge <thomas@codesourcery.com> 2012-11-06 13:25:02 +0100
commit: 776c4315f601b36021834eb0d44f18203f56e811 (patch)
tree: d778ae5d3e67c54b70f4ddf1493a51fa94f5203a /open_issues
parent: b866ae739bea58218211eeba9b3a7bd0d43ab84c (diff)
3 files changed, 176 insertions, 19 deletions
diff --git a/open_issues/gdb.mdwn b/open_issues/gdb.mdwn
index 1652031b..5b8ac21c 100644
--- a/open_issues/gdb.mdwn
+++ b/open_issues/gdb.mdwn
@@ -24,8 +24,17 @@ Here's what's to be done for maintaining GNU GDB.
 
 # Configuration
 
-Last reviewed up to the [[Git mirror's ea9812279fe436be9a010d07ef1dbe465199a3d7
-(2011-09-07) sources|source_repositories/gdb]].
+<!--
+
+git checkout reviewed
+git log --reverse --topo-order --pretty=fuller --stat=$COLUMNS,$COLUMNS -w -p -C --cc ..sourceware/master
+-i
+/^commit |^Merge:|^---$|hurd|linux|nacl|nptl|glibc|gs:|gnu-nat|i386gnu
+
+-->
+
+Last reviewed up to the [[Git mirror's 70044c11b4bf873773972c0f8784188356389bfc
+(2011-10-16) sources|source_repositories/gdb]].
 
   * Globally
 
@@ -71,9 +80,15 @@ There are several occurences of *error: dereferencing type-punned pointer will
 break strict-aliasing rules* in the MIG-generated stub files; thus no `-Werror`
 until that is resolved ([[strict_aliasing]]).
 
-This takes up around 140 MiB and needs roughly 6 min on kepler.SCHWINGE and 30
+This takes up around 210 MiB and needs roughly 6 min on kepler.SCHWINGE and 25
 min on coulomb.SCHWINGE.
 
+<!--
+
+    $ (make && touch .go-install) 2>&1 | tee log_build_ && test -f .go-install && (make install && touch .go-test) 2>&1 | tee log_install && test -f .go-test && make -k check 2>&1 | tee log_test
+
+-->
+
 
 ## Analysis
 
@@ -81,12 +96,53 @@ x86 GNU/Linux' and GNU/Hurd's configurations are slightly different, thus mask
 out most of the differences that are due to GNU/Linux supporting more core file
 formats and more emulation vectors.
 
-    $ ssh kepler.SCHWINGE 'cd tmp/source/gdb/ && cat hurd/master.build/log_build* | sed -e "s%\(/media/data\)\?${PWD}%[...]%g"' > toolchain/logs/gdb/linux/log_build
-    $ ssh coulomb.SCHWINGE 'cd tmp/gdb/ && cat hurd/master.build/log_build* | sed -e "s%\(/media/erich\)\?${PWD}%[...]%g"' > toolchain/logs/gdb/hurd/log_build
-    $ diff -wu <(sed -f toolchain/logs/gdb/linux/log_build.sed < toolchain/logs/gdb/linux/log_build) <(sed -f toolchain/logs/gdb/hurd/log_build.sed < toolchain/logs/gdb/hurd/log_build) > toolchain/logs/gdb/log_build.diff
+    $ toolchain/logs/process gdb build
+
+  * DFP
+
+        -checking for decimal floating point... bid
+        +checking for decimal floating point... configure: WARNING: decimal float is not supported for this target, ignored
+        +dpd
+
+    Have to sync libdecnumber?
 
   * Why do we specify `-D_GNU_SOURCE`, and GNU/Linux doesn't?
 
+  * GNU/Linux: `gdb/symfile-mem.c` for vDSO.
+
+  * GNU/Linux: `gdb/i386-nat.c` for hardware breakpoints, etc. -- we should
+    probably use that, too.  Related to Samuel's Hurd GDB patch?
+
+  * `gdb/gnu-nat.c`
+
+        gnu-nat.c: In function 'proc_set_exception_port':
+        gnu-nat.c:409:3: warning: format '%d' expects argument of type 'int', but argument 8 has type 'mach_port_t' [-Wformat]
+        gnu-nat.c: In function 'proc_steal_exc_port':
+        gnu-nat.c:449:7: warning: format '%d' expects argument of type 'int', but argument 8 has type 'mach_port_t' [-Wformat]
+        gnu-nat.c:470:7: warning: format '%d' expects argument of type 'int', but argument 8 has type 'mach_port_t' [-Wformat]
+        gnu-nat.c: In function 'make_proc':
+        gnu-nat.c:583:7: warning: format '%d' expects argument of type 'int', but argument 2 has type 'mach_port_t' [-Wformat]
+        gnu-nat.c:586:7: warning: format '%d' expects argument of type 'int', but argument 8 has type 'mach_port_t' [-Wformat]
+        gnu-nat.c: In function 'inf_set_pid':
+        gnu-nat.c:761:3: warning: format '%d' expects argument of type 'int', but argument 7 has type 'task_t' [-Wformat]
+        gnu-nat.c: In function 'inf_validate_procs':
+        gnu-nat.c:1085:6: warning: format '%d' expects argument of type 'int', but argument 8 has type 'thread_t' [-Wformat]
+        gnu-nat.c: In function 'inf_signal':
+        gnu-nat.c:1349:4: warning: format '%d' expects argument of type 'int', but argument 7 has type 'thread_t' [-Wformat]
+        gnu-nat.c:1349:4: warning: format '%d' expects argument of type 'int', but argument 8 has type 'thread_t' [-Wformat]
+        gnu-nat.c: In function 'S_exception_raise_request':
+        gnu-nat.c:1668:3: warning: format '%d' expects argument of type 'int', but argument 7 has type 'thread_t' [-Wformat]
+        gnu-nat.c:1668:3: warning: format '%d' expects argument of type 'int', but argument 8 has type 'task_t' [-Wformat]
+        gnu-nat.c:1705:8: warning: format '%d' expects argument of type 'int', but argument 7 has type 'mach_port_t' [-Wformat]
+        gnu-nat.c:1711:8: warning: format '%d' expects argument of type 'int', but argument 7 has type 'mach_port_t' [-Wformat]
+        gnu-nat.c: In function 'do_mach_notify_dead_name':
+        gnu-nat.c:1762:3: warning: format '%d' expects argument of type 'int', but argument 7 has type 'mach_port_t' [-Wformat]
+        gnu-nat.c: In function 'gnu_write_inferior':
+        gnu-nat.c:2383:8: warning: format '%x' expects argument of type 'unsigned int', but argument 2 has type 'vm_address_t' [-Wformat]
+        gnu-nat.c:2393:8: warning: format '%x' expects argument of type 'unsigned int', but argument 2 has type 'vm_address_t' [-Wformat]
+        gnu-nat.c: In function 'steal_exc_port':
+        gnu-nat.c:2864:5: warning: format '%d' expects argument of type 'int', but argument 2 has type 'mach_port_t' [-Wformat]
+
   * Why does GNU/Linux have an additional `-ldl -rdynamic` when linking `gdb`?
 
 
@@ -101,27 +157,60 @@ min on coulomb.SCHWINGE.
 
 ## Analysis
 
-    $ ssh kepler.SCHWINGE 'cd tmp/source/gdb/ && cat hurd/master.build/log_install | sed -e "s%\(/media/data\)\?${PWD}%[...]%g"' > toolchain/logs/gdb/linux/log_install
-    $ ssh coulomb.SCHWINGE 'cd tmp/gdb/ && cat hurd/master.build/log_install | sed -e "s%\(/media/erich\)\?${PWD}%[...]%g"' > toolchain/logs/gdb/hurd/log_install
-    $ diff -wu <(sed -f toolchain/logs/gdb/linux/log_install.sed < toolchain/logs/gdb/linux/log_install) <(sed -f toolchain/logs/gdb/hurd/log_install.sed < toolchain/logs/gdb/hurd/log_install) > toolchain/logs/gdb/log_install.diff
+    $ toolchain/logs/process gdb install
 
   * `libtool: finish`: `ldconfig` is not run for the Hurd.
 
 
 # Testsuite
 
-On GNU/Hurd, hampered by the [[term_blocking]] issue.
-
     $ make -k check
     [...]
 
-This needs roughly 45 min on kepler.SCHWINGE and TODO min on coulomb.SCHWINGE.
+This needs roughly 11 min on kepler.SCHWINGE and 128 min on coulomb.SCHWINGE.
 
-    $ ssh kepler.SCHWINGE 'cd tmp/source/gdb/ && sed -e "s%\(/media/data\)\?${PWD}%[...]%g" < hurd/master.build/gdb/testsuite/gdb.sum' > toolchain/logs/gdb/linux/sum
-    $ ssh coulomb.SCHWINGE 'cd tmp/gdb/ && sed -e "s%\(/media/erich\)\?${PWD}%[...]%g" < hurd/master.build/gdb/testsuite/gdb.sum' > toolchain/logs/gdb/hurd/sum
-    $ diff -u -F ^Running toolchain/logs/gdb/linux/sum toolchain/logs/gdb/hurd/sum > toolchain/logs/gdb/sum.diff
+When running `make -k check 2>&1 | tee log_test`, at the end of the testsuite
+the `tee` process does not terminate if there are still stray leftover
+processes that [have their stdout/stderr
+open](http://sourceware.org/ml/gdb-patches/2012-10/msg00489.html).  `kill`ing
+these (`SIGKILL` may be needed), makes the `tee` process terminate, too.  On
+GNU/Hurd, this has be seen for `gdb.base/structs-tf-td`, and an unknown (`?`)
+one.
 
 
 ## Analysis
 
+    $ toolchain/logs/process gdb test
+
+  * Disabled
+
+      * `gdb.base/morestack.exp`
+
+        TODO `-fsplit-stack` issue.
+
+      * `gdb.base/readline.exp`
+
+        [[term_blocking]] issue.
+
+      * `gdb.base/sigall.exp`
+
+        From `send signal TSTP` on, all FAIL running into timeouts.
+
+  * `UNSUPPORTED: gdb.threads/ia64-sigill.exp: Couldn't compile ../../../master/gdb/testsuite/gdb.threads/ia64-sigill.c: unrecognized error`
+
+        ../../../master/gdb/testsuite/gdb.threads/ia64-sigill.c:29:24: fatal error: asm/unistd.h: No such file or directory
+
+  * `UNSUPPORTED: gdb.threads/multi-create.exp: Couldn't compile ../../../master/gdb/testsuite/gdb.threads/multi-create.c: unrecognized error`
+        ../../../master/gdb/testsuite/gdb.threads/multi-create.c: In function 'create_function':
+        ../../../master/gdb/testsuite/gdb.threads/multi-create.c:46:39: error: 'PTHREAD_STACK_MIN' undeclared (first use in this function)
+        ../../../master/gdb/testsuite/gdb.threads/multi-create.c:46:39: note: each undeclared identifier is reported only once for each function it appears in
+        ../../../master/gdb/testsuite/gdb.threads/multi-create.c: In function 'main':
+        ../../../master/gdb/testsuite/gdb.threads/multi-create.c:73:39: error: 'PTHREAD_STACK_MIN' undeclared (first use in this function)
+
+  * `UNSUPPORTED: gdb.threads/staticthreads.exp: Couldn't compile ../../../master/gdb/testsuite/gdb.threads/staticthreads.c: unrecognized error`
+
+        ../../../master/gdb/testsuite/gdb.threads/staticthreads.c: In function 'main':
+        ../../../master/gdb/testsuite/gdb.threads/staticthreads.c:52:37: error: 'PTHREAD_STACK_MIN' undeclared (first use in this function)
+        ../../../master/gdb/testsuite/gdb.threads/staticthreads.c:52:37: note: each undeclared identifier is reported only once for each function it appears in
+
 TODO.
diff --git a/open_issues/glibc_ptrace.mdwn b/open_issues/glibc_ptrace.mdwn
index b4c529d7..6704ed80 100644
--- a/open_issues/glibc_ptrace.mdwn
+++ b/open_issues/glibc_ptrace.mdwn
@@ -1,4 +1,4 @@
-[[!meta copyright="Copyright © 2009 Free Software Foundation, Inc."]]
+[[!meta copyright="Copyright © 2009, 2012 Free Software Foundation, Inc."]]
 
 [[!meta license="""[[!toggle id="license" text="GFDL 1.2+"]][[!toggleable
 id="license" text="Permission is granted to copy, distribute and/or modify this
@@ -33,8 +33,8 @@ License|/fdl]]."]]"""]]
         and for us it is a `struct i386_thread_state` from
         `mach/i386/thread_status.h`;
 
-      * Linux probides some functionality that we don't provide, e.g.,
-        `PTRACE_SINGLESTEP`.
+      * Linux provides some functionality that we don't provide:
+        `PTRACE_GETFPXREGS` , `PTRACE_SINGLESTEP`.
 
   * Some parts are wrongly implemented, e.g., `PTRACE_GETREGS` and
     `PTRACE_SETREGS` both do the same thing.
diff --git a/open_issues/term_blocking.mdwn b/open_issues/term_blocking.mdwn
index 19d18d0e..eddc285e 100644
--- a/open_issues/term_blocking.mdwn
+++ b/open_issues/term_blocking.mdwn
@@ -1,4 +1,5 @@
-[[!meta copyright="Copyright © 2009, 2011 Free Software Foundation, Inc."]]
+[[!meta copyright="Copyright © 2009, 2011, 2012 Free Software Foundation,
+Inc."]]
 
 [[!meta license="""[[!toggle id="license" text="GFDL 1.2+"]][[!toggleable
 id="license" text="Permission is granted to copy, distribute and/or modify this
@@ -113,9 +114,76 @@ user started GDB test suite, noticed the PTY it's using; in a root shell
 started GDB (the system one, for `.debug` stuff) on `/hurd/term`, `set
 noninvasive on`, attach to the *term* that GDB is using.
 
+---
 
 [[2011-07-04]].
 
+---
+
+2012-11-05
+
+Log file from a 2011-09-07 run:
+
+    [...]
+    Running ../../../master/gdb/testsuite/gdb.base/readline.exp ...
+    spawn [...]/gdb/testsuite/../../gdb/gdb -nw -nx -data-directory [...]/gdb/testsuite/../data-directory
+    GNU gdb (GDB) 7.3.50.20110906-cvs
+    Copyright (C) 2011 Free Software Foundation, Inc.
+    License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
+    This is free software: you are free to change and redistribute it.
+    There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
+    and "show warranty" for details.
+    This GDB was configured as "i686-unknown-gnu0.3".
+    For bug reporting instructions, please see:
+    <http://www.gnu.org/software/gdb/bugs/>.
+    (gdb) set height 0
+    (gdb) set width 0
+    (gdb) dir
+    Reinitialize source path to empty? (y or n) y
+    Source directories searched: $cdir:$cwd
+    (gdb) dir ../../../master/gdb/testsuite/gdb.base
+    Source directories searched: [...]/gdb/testsuite/../../../master/gdb/testsuite/gdb.base:$cdir:$cwd
+    (gdb) p 1
+    $1 = 1
+    PASS: gdb.base/readline.exp: Simple operate-and-get-next - send p 1
+    (gdb) p 2
+    $2 = 2
+    PASS: gdb.base/readline.exp: Simple operate-and-get-next - send p 2
+    (gdb) p 3
+    $3 = 3
+    PASS: gdb.base/readline.exp: Simple operate-and-get-next - send p 3
+    (gdb) p 3(gdb) p 3PASS: gdb.base/readline.exp: Simple operate-and-get-next - C-p to p 3
+    ^H2(gdb) p 2PASS: gdb.base/readline.exp: Simple operate-and-get-next - C-p to p 2
+    ^H1(gdb) p 1PASS: gdb.base/readline.exp: Simple operate-and-get-next - C-p to p 1
+    ^OFAIL: gdb.base/readline.exp: Simple operate-and-get-next - C-o for p 1
+    FAIL: gdb.base/readline.exp: operate-and-get-next with secondary prompt - send if 1 > 0
+    FAIL: gdb.base/readline.exp: print 42 (timeout)
+    FAIL: gdb.base/readline.exp: arrow keys with secondary prompt (timeout)
+    spawn [...]/gdb/testsuite/../../gdb/gdb -nw -nx -data-directory [...]/gdb/testsuite/../data-directory
+    ERROR: (timeout) GDB never initialized after 10 seconds.
+    ERROR: no fileid for coulomb
+    ERROR: no fileid for coulomb
+    UNRESOLVED: gdb.base/readline.exp: Simple operate-and-get-next - send p 7
+    testcase ../../../master/gdb/testsuite/gdb.base/readline.exp completed in 646 seconds
+    Running ../../../master/gdb/testsuite/gdb.base/wchar.exp ...
+    Executing on host: gcc  -c -g  -o [...]/gdb/testsuite/gdb.base/wchar0.o ../../../master/gdb/testsuite/gdb.base/wchar.c    (timeout = 300)
+    spawn gcc -c -g -o [...]/gdb/testsuite/gdb.base/wchar0.o ../../../master/gdb/testsuite/gdb.base/wchar.c
+    Executing on host: gcc [...]/gdb/testsuite/gdb.base/wchar0.o  -g  -lm   -o [...]/gdb/testsuite/gdb.base/wchar    (timeout = 300)
+    spawn gcc [...]/gdb/testsuite/gdb.base/wchar0.o -g -lm -o [...]/gdb/testsuite/gdb.base/wchar
+    get_compiler_info: gcc-4-6-1
+    spawn [...]/gdb/testsuite/../../gdb/gdb -nw -nx -data-directory [...]/gdb/testsuite/../data-directory
+    ERROR: (timeout) GDB never initialized after 10 seconds.
+    ERROR: no fileid for coulomb
+    ERROR: no fileid for coulomb
+    ERROR: no fileid for coulomb
+    ERROR: couldn't load [...]/gdb/testsuite/gdb.base/wchar into [...]/gdb/testsuite/../../gdb/gdb (timed out).
+    ERROR: no fileid for coulomb
+    ERROR: Delete all breakpoints in delete_breakpoints (timeout)
+    ERROR: no fileid for coulomb
+    UNRESOLVED: gdb.base/wchar.exp: setting breakpoint at wchar.c:34 (timeout)
+    testcase ../../../master/gdb/testsuite/gdb.base/wchar.exp completed in 797 seconds
+    [...]
+
 
 # Formal Verification
author	Thomas Schwinge <thomas@codesourcery.com>	2012-11-06 13:25:02 +0100
committer	Thomas Schwinge <thomas@codesourcery.com>	2012-11-06 13:25:02 +0100
commit	776c4315f601b36021834eb0d44f18203f56e811 (patch)
tree	d778ae5d3e67c54b70f4ddf1493a51fa94f5203a /open_issues
parent	b866ae739bea58218211eeba9b3a7bd0d43ab84c (diff)