From 545ae255da361648d20951fd43bb82b716358ce1 Mon Sep 17 00:00:00 2001 From: Thomas Schwinge Date: Thu, 9 Jan 2014 13:50:30 +0100 Subject: open_issues/binutils: binutils-gdb: 9939e1314f970c6ba568956148a518ac710a280a (2014-01-08) plus [[!message-id "87vbxxhww4.fsf@kepler.schwinge.homeip.net"]], and [[!message-id "8738kyi30l.fsf@kepler.schwinge.homeip.net"]] --- open_issues/binutils.mdwn | 116 ++++++++++++------------------------- open_issues/gdb_non-stop_mode.mdwn | 9 +-- open_issues/gdb_pending_execs.mdwn | 11 ++-- open_issues/gdb_thread_ids.mdwn | 8 +-- toolchain/logs | 2 +- 5 files changed, 54 insertions(+), 92 deletions(-) diff --git a/open_issues/binutils.mdwn b/open_issues/binutils.mdwn index 97b56616..fa384bf4 100644 --- a/open_issues/binutils.mdwn +++ b/open_issues/binutils.mdwn @@ -47,8 +47,8 @@ git diff --patience --stat=$COLUMNS,$COLUMNS --patch --src-prefix=./ --dst-prefi --> -Last reviewed up to Git commit 95e4302a8f872b3f68c4cf5af774ca1bde9d2c1a -(2013-10-21). +Last reviewed up to Git commit 9939e1314f970c6ba568956148a518ac710a280a +(2014-01-08). * Globally @@ -116,6 +116,9 @@ Last reviewed up to Git commit 95e4302a8f872b3f68c4cf5af774ca1bde9d2c1a * [[gdb/gdbserver]] + * In `gdb/gnu-nat.c:gnu_wait`, we don't implement + `gdb/target/wait.h:TARGET_WNOHANG`. What is this needed for? + * `libdecnumber/` Should/can probably align to GNU/Linux. @@ -151,26 +154,22 @@ Last reviewed up to Git commit 95e4302a8f872b3f68c4cf5af774ca1bde9d2c1a * Configure so that Debian system's `/usr/lib/debug/[...]` will be loaded automatically. - * old-style function definition + * [[!message-id "m3ei24vh4l.fsf@fleche.redhat.com"]] + + * [low] c26e9cbb0ce70e8fca32a40c434a0837bf46750a, + `gdb/gnu-nat.c:set_exceptions_cmd`, `Make this take effect immediately in a + running process`. - ../../Ferry_Tagscherer/gdb/gnu-nat.c: In function 'trace_me': - ../../Ferry_Tagscherer/gdb/gnu-nat.c:2106:8: warning: old-style function definition [-Wold-style-definition] - void trace_me () - ^ + * [low] b27caf75c311991772b316fe7c0eecfd5788eeaf, ld, `Add HOSTING_SLIBS and + use it for -pie`. For us, too? # Build Here's a log of a binutils-gdb build run; this is from Git commit -95e4302a8f872b3f68c4cf5af774ca1bde9d2c1a (2013-10-21) plus -36cff971e29987ed61e86891c77d28158c20b090, -a903d6ac5a79c1fb390b9b833c891c0544502cfe, -ae6da67e35733eb8be443ae46ca9a24f8037a75d, -9548bbede51868a9a780d7d21ae16ac13e8bdf9b, -eec6550343414e9ebf12885533609c1ba9b397f1, -689a1abf0dba1f5353039a5625411c6c41098d3a, -8467a2fd5569f7345c025ca94467f50cd329560d, and -`id:"87vbxxhww4.fsf@kepler.schwinge.homeip.net"`, run on +9939e1314f970c6ba568956148a518ac710a280a (2014-01-08) plus +[[!message-id "87vbxxhww4.fsf@kepler.schwinge.homeip.net"]], and +[[!message-id "8738kyi30l.fsf@kepler.schwinge.homeip.net"]], run on kepler.SCHWINGE and coulomb.SCHWINGE. $ export LC_ALL=C @@ -192,7 +191,7 @@ This takes up around 1.3 GiB, and needs roughly 17 min on kepler.SCHWINGE and @@ -246,15 +245,6 @@ formats, and more emulation vectors. `-Wmissing-prototypes` - gnu-nat.c: At top level: - gnu-nat.c:643:1: warning: no previous prototype for 'make_inf' [] - gnu-nat.c: At top level: - gnu-nat.c:879:1: warning: no previous prototype for 'inf_set_traced' [] - gnu-nat.c:980:1: warning: no previous prototype for 'inf_port_to_thread' [] - gnu-nat.c: At top level: - gnu-nat.c:1748:1: warning: no previous prototype for 'inf_task_died_status' [] - gnu-nat.c: At top level: - gnu-nat.c:3415:1: warning: no previous prototype for '_initialize_gnu_nat' [] notify_S.c:305:24: warning: no previous prototype for 'notify_server' [] notify_S.c:341:28: warning: no previous prototype for 'notify_server_routine' [] process_reply_S.c:343:24: warning: no previous prototype for 'process_reply_server' [] @@ -274,15 +264,6 @@ formats, and more emulation vectors. [[!taglink open_issue_glibc]]? - * 00e1c4298d2b6fe040a9a970e98349602b12ddbf..6b25dae901ddedb3f330803542d3eac73cdcae4b: - - +../../Ferry_Tagscherer/gdb/gnu-nat.c: In function 'info_port_rights': - +../../Ferry_Tagscherer/gdb/gnu-nat.c:3058:11: warning: passing argument 1 of 'parse_to_comma_and_eval' from incompatible pointer type [enabled by default] - +In file included from ../../Ferry_Tagscherer/gdb/breakpoint.h:23:0, - + from ../../Ferry_Tagscherer/gdb/inferior.h:37, - + from ../../Ferry_Tagscherer/gdb/gnu-nat.c:56: - +../../Ferry_Tagscherer/gdb/value.h:729:22: note: expected 'const char **' but argument is of type 'char **' - # Install @@ -302,6 +283,22 @@ min on coulomb.SCHWINGE. # Testsuite +Some change in +95e4302a8f872b3f68c4cf5af774ca1bde9d2c1a..b7bba0015567c096541d62f2637b9818b1634a59 +causes all gdb.gdb tests to regress from PASS to ERROR: + + Running [...]/gdb/testsuite/gdb.gdb/complaints.exp ... + -PASS: gdb.gdb/complaints.exp: breakpoint in captured_command_loop + -[...] + +ERROR: remote_download to host of [...]/gdb/testsuite/../../gdb/gdb to [...]/gdb/testsuite/gdb.gdb/xgdb: cp: cannot create regular file '[...]/gdb/testsuite/gdb.gdb/xgdb': No such file or directory + +ERROR: Couldn't load into [...]/gdb/testsuite/../../gdb/gdb. + +WARNING: Couldn't test self + +The gdb/testsuite/gdb.gdb directory does not exist/is no longer created; create +it manually: + + $ mkdir -p gdb/testsuite/gdb.gdb + $ make -k check 2>&1 | tee log_test [...] @@ -333,6 +330,8 @@ like `gdb/testsuite/boards/cc-with-tweaks.exp` would help, or setting in `ld/configure.host`. Perhaps we should finally rewrite this messy code in glibc? Or, something similar to commit 49cc20aa5c416ea4307931cccf6353247368187d `Add HOSTING_SCRT0 for PIE test` + or commit b27caf75c311991772b316fe7c0eecfd5788eeaf `Add HOSTING_SLIBS and + use it for -pie` can be used. Same issue for `FAIL: Common symbol override ifunc *` ones? @@ -874,51 +873,10 @@ like `gdb/testsuite/boards/cc-with-tweaks.exp` would help, or setting TODO. - * `gdb.pascal` - - Both on coulomb.SCHWINGE and kepler.SCHWINGE: - - Running ../../../Ferry_Tagscherer/gdb/testsuite/gdb.pascal/print.exp ... - PASS: gdb.pascal/print.exp: set language pascal - PASS: gdb.pascal/print.exp: set language to "pascal" - -PASS: gdb.pascal/print.exp: check fp + text - -PASS: gdb.pascal/print.exp: p 1. - -PASS: gdb.pascal/print.exp: p 1.5 - -PASS: gdb.pascal/print.exp: p 1.f - -PASS: gdb.pascal/print.exp: p 1.5f - -PASS: gdb.pascal/print.exp: p 1.l - -PASS: gdb.pascal/print.exp: p 1.5l - -PASS: gdb.pascal/print.exp: p 0x1.1 - +FAIL: gdb.pascal/print.exp: check fp + text - +FAIL: gdb.pascal/print.exp: p 1. - +FAIL: gdb.pascal/print.exp: p 1.5 - +FAIL: gdb.pascal/print.exp: p 1.f - +FAIL: gdb.pascal/print.exp: p 1.5f - +FAIL: gdb.pascal/print.exp: p 1.l - +FAIL: gdb.pascal/print.exp: p 1.5l - +FAIL: gdb.pascal/print.exp: p 0x1.1 - PASS: gdb.pascal/print.exp: reject p 1.1x - PASS: gdb.pascal/print.exp: reject p 1.1ff - PASS: gdb.pascal/print.exp: reject p 1.1ll - Running ../../../Ferry_Tagscherer/gdb/testsuite/gdb.pascal/types.exp ... - PASS: gdb.pascal/types.exp: set language pascal - PASS: gdb.pascal/types.exp: set language to "pascal" - -PASS: gdb.pascal/types.exp: pt 123 - +FAIL: gdb.pascal/types.exp: pt 123 - PASS: gdb.pascal/types.exp: pt TRUE - PASS: gdb.pascal/types.exp: pt FALSE - PASS: gdb.pascal/types.exp: pt 'a' - KFAIL: gdb.pascal/types.exp: pt 'a simple string' (PRMS: gdb/2326) - -PASS: gdb.pascal/types.exp: pt .44 - -PASS: gdb.pascal/types.exp: pt 44.0 - -PASS: gdb.pascal/types.exp: pt 10e20 - -PASS: gdb.pascal/types.exp: pt 10E20 - +FAIL: gdb.pascal/types.exp: pt .44 - +FAIL: gdb.pascal/types.exp: pt 44.0 - +FAIL: gdb.pascal/types.exp: pt 10e20 - +FAIL: gdb.pascal/types.exp: pt 10E20 + * `gdb.arch/i386-prologue.exp` - TODO. + On coulomb.SCHWINGE, there are several FAILs, where there are, for example, + unexpected frames showing up in backtraces. TODO. TODO. diff --git a/open_issues/gdb_non-stop_mode.mdwn b/open_issues/gdb_non-stop_mode.mdwn index 418b144c..adaff102 100644 --- a/open_issues/gdb_non-stop_mode.mdwn +++ b/open_issues/gdb_non-stop_mode.mdwn @@ -1,12 +1,13 @@ -[[!meta copyright="Copyright © 2008, 2009 Free Software Foundation, Inc."]] +[[!meta copyright="Copyright © 2008, 2009, 2014 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 document under the terms of the GNU Free Documentation License, Version 1.2 or any later version published by the Free Software Foundation; with no Invariant Sections, no Front-Cover Texts, and no Back-Cover Texts. A copy of the license -is included in the section entitled -[[GNU Free Documentation License|/fdl]]."]]"""]] +is included in the section entitled [[GNU Free Documentation +License|/fdl]]."]]"""]] [[!meta title="GDB's non-stop mode"]] @@ -14,7 +15,7 @@ is included in the section entitled GNU GDB's `gnu-nat.c` doesn't support *non-stop* mode. -Also, from , +Also, from [[!message-id "200810131935.35253.pedro@codesourcery.com"]], GNU GDB's Pedro Alves: > I also notice that when going through the shell in non-stop mode, it would be diff --git a/open_issues/gdb_pending_execs.mdwn b/open_issues/gdb_pending_execs.mdwn index bb0f8419..37a92cb7 100644 --- a/open_issues/gdb_pending_execs.mdwn +++ b/open_issues/gdb_pending_execs.mdwn @@ -1,18 +1,19 @@ -[[!meta copyright="Copyright © 2008, 2009 Free Software Foundation, Inc."]] +[[!meta copyright="Copyright © 2008, 2009, 2014 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 document under the terms of the GNU Free Documentation License, Version 1.2 or any later version published by the Free Software Foundation; with no Invariant Sections, no Front-Cover Texts, and no Back-Cover Texts. A copy of the license -is included in the section entitled -[[GNU Free Documentation License|/fdl]]."]]"""]] +is included in the section entitled [[GNU Free Documentation +License|/fdl]]."]]"""]] [[!meta title="GDB: pending_execs"]] [[!tag open_issue_gdb]] -: +[[!message-id "200810131935.35253.pedro@codesourcery.com"]] GNU GDB's Ulrich Weigand: @@ -25,3 +26,5 @@ GNU GDB's Pedro Alves: >> pending execs handling in gnu-nat.c. An alternative would be to make >> pending_execs a property of inferior.h:`struct inferior' instead of of >> gnu-nat.c:`struct inf'. + +[[!message-id "8738kyi30l.fsf@kepler.schwinge.homeip.net"]] diff --git a/open_issues/gdb_thread_ids.mdwn b/open_issues/gdb_thread_ids.mdwn index c04a10ee..64173b1b 100644 --- a/open_issues/gdb_thread_ids.mdwn +++ b/open_issues/gdb_thread_ids.mdwn @@ -1,5 +1,5 @@ -[[!meta copyright="Copyright © 2008, 2009, 2010, 2011 Free Software Foundation, -Inc."]] +[[!meta copyright="Copyright © 2008, 2009, 2010, 2011, 2014 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 @@ -15,8 +15,8 @@ License|/fdl]]."]]"""]] GNU GDB's Pedro Alves: -> One thing [I asked -> myself](http://lists.gnu.org/archive/html/bug-hurd/2008-10/msg00045.html) +> One thing [[!message-id desc="I asked myself" +> "200810131935.35253.pedro@codesourcery.com"]] > was, if gnu-nat.c couldn't be using the port's id as thread ids instead of a > locally auto-generated number. Maybe the thread id of the main thread would > be preserved across execs this way diff --git a/toolchain/logs b/toolchain/logs index 0dc2a5c6..4350ed2b 160000 --- a/toolchain/logs +++ b/toolchain/logs @@ -1 +1 @@ -Subproject commit 0dc2a5c6f8175b8a5cd4cf8b31a7d76d05d219ab +Subproject commit 4350ed2b0150d38ca945a99cdc7d82b0c2ae564c -- cgit v1.2.3