This is the mail archive of the
gdb-cvs@sourceware.org
mailing list for the GDB project.
gdb and binutils branch master updated. 7096e886259fad196b955ecaff9a61891281d117
- From: gary at sourceware dot org
- To: gdb-cvs at sourceware dot org
- Date: 28 Aug 2014 14:15:18 -0000
- Subject: gdb and binutils branch master updated. 7096e886259fad196b955ecaff9a61891281d117
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "gdb and binutils".
The branch, master has been updated
via 7096e886259fad196b955ecaff9a61891281d117 (commit)
via 14ce3192392edde90aa444c153eacede6c1ba5c5 (commit)
via 50278d599da5f7536a1243a887c2a5fa12e099c1 (commit)
via aa96c426c674844a90e8fbea2cd184462191b498 (commit)
via f7160e97410b4b65330e5f23d82e8879c908de50 (commit)
via 38e08fcac0966f52dac0ec97870fdea4de304d86 (commit)
from 6aefca7e4978b93f502ff8123a6c3cab6e0171f1 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=7096e886259fad196b955ecaff9a61891281d117
commit 7096e886259fad196b955ecaff9a61891281d117
Author: Gary Benson <gbenson@redhat.com>
Date: Wed Aug 6 13:23:59 2014 +0100
Remove fatal function and prototype
This commit removes the now-unused fatal function and prototype.
gdb/gdbserver/ChangeLog:
* utils.h (fatal): Remove declaration.
* utils.c (fatal): Remove function.
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=14ce3192392edde90aa444c153eacede6c1ba5c5
commit 14ce3192392edde90aa444c153eacede6c1ba5c5
Author: Gary Benson <gbenson@redhat.com>
Date: Wed Aug 6 14:43:17 2014 +0100
Convert fatal to perror_with_name in IPA code
This commit converts four calls to fatal into calls to
perror_with_name. perror_with_name calls error, which
in IPA terminates with exit (1) rather than longjmp, so
there is no functional change here.
gdb/gdbserver/ChangeLog:
* tracepoint.c (gdb_agent_init): Replace fatal with
perror_with_name.
(initialize_tracepoint): Likewise.
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=50278d599da5f7536a1243a887c2a5fa12e099c1
commit 50278d599da5f7536a1243a887c2a5fa12e099c1
Author: Gary Benson <gbenson@redhat.com>
Date: Wed Aug 6 13:47:55 2014 +0100
Convert fatal to error in remote_prepare
This commit converts a call to fatal in remote_prepare with a call to
error. remote_prepare is called precisely once, from main, at a point
where jumping to toplevel will call exit (1), so error and fatal are
functionally equivalent at this point. Note that remote_prepare calls
perror_with_name (which calls error) so callers of remote_prepare must
already handle the fact that it may exit via longjmp.
gdb/gdbserver/ChangeLog:
* remote-utils.c (remote_prepare): Replace fatal with error.
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=aa96c426c674844a90e8fbea2cd184462191b498
commit aa96c426c674844a90e8fbea2cd184462191b498
Author: Gary Benson <gbenson@redhat.com>
Date: Wed Aug 6 11:57:14 2014 +0100
Downgrade fatal to warning in linux_async
This commit downgrades a fatal error to a warning in linux_async.
linux_async is called from two different places in gdbserver:
Via target_async from handle_accept_event. The argument
is always zero, so the warning will never be printed here.
Via start_non_stop from handle_general_set. This prints
its own error message to stderr on failure, which will
be preceded by the warning if it is emitted.
gdb/gdbserver/ChangeLog:
* linux-low.c (linux_async): Replace fatal with warning.
Tidy up and return.
(linux_start_non_stop): Return -1 if linux_async failed.
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=f7160e97410b4b65330e5f23d82e8879c908de50
commit f7160e97410b4b65330e5f23d82e8879c908de50
Author: Gary Benson <gbenson@redhat.com>
Date: Wed Aug 6 14:00:14 2014 +0100
Convert fatal to gdb_assert in both i386_dr_low_set_addr
This commit converts if..fatal checks in both i386_dr_low_set_addr
implementations to gdb_asserts. It's not obvious from the context,
but the conditional in both cases is changed to match the equivalent
conditional in the i386_dr_low_get_addr implementations. Nothing
fundamental has changed because DR_FIRSTADDR is zero. This commit
also removes a vague comment in Linux i386_dr_low_get_addr. I could
have reworded the comment (and replicated it three times for the other
identical assertions) but I think the existence of specific functions
for the status and control registers makes it fairly obvious what is
going on.
gdb/gdbserver/ChangeLog:
* linux-x86-low.c (i386_dr_low_set_addr): Replace check with
gdb_assert.
(i386_dr_low_get_addr): Remove vague comment.
* win32-i386-low.c (i386_dr_low_set_addr): Replace check with
gdb_assert.
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=38e08fcac0966f52dac0ec97870fdea4de304d86
commit 38e08fcac0966f52dac0ec97870fdea4de304d86
Author: Gary Benson <gbenson@redhat.com>
Date: Wed Aug 6 11:56:25 2014 +0100
Straightforward fatal to internal_error conversions
This commit replaces most of the calls to fatal that represent
internal errors with calls to internal_error, either directly
or via gdb_assert and gdb_assert_not_reached.
gdb/gdbserver/ChangeLog:
* inferiors.c (get_thread_process): Replace check with gdb_assert.
* linux-low.c (linux_wait_for_event_filtered): Replace fatal with
internal_error.
(linux_resume_one_lwp): Likewise.
* linux-x86-low.c (x86_siginfo_fixup): Replace checks with
gdb_assert.
* mem-break.c (raw_bkpt_type_to_target_hw_bp_type): Replace fatal
with internal_error.
* regcache.c (get_thread_regcache): Replace check with gdb_assert.
(init_register_cache): Replace fatal with gdb_assert_not_reached.
(find_register_by_name): Replace fatal with internal_error.
(find_regno): Likewise.
* tdesc.c (init_target_desc): Replace check with gdb_assert.
* thread-db.c (thread_db_create_event): Likewise.
(thread_db_load_search): Likewise.
(try_thread_db_load_1): Likewise.
* tracepoint.c (get_jump_space_head): Replace fatal with
internal_error.
(claim_trampoline_space): Likewise.
(have_fast_tracepoint_trampoline_buffer): Likewise.
(cmd_qtstart): Likewise.
(stop_tracing): Likewise.
(fast_tracepoint_collecting): Likewise.
(target_malloc): Likewise.
(download_tracepoint): Likewise.
(download_trace_state_variables): Replace check with gdb_assert.
(upload_fast_traceframes): Replace fatal with internal_error.
-----------------------------------------------------------------------
Summary of changes:
gdb/gdbserver/ChangeLog | 59 ++++++++++++++++++++++++
gdb/gdbserver/inferiors.c | 4 +-
gdb/gdbserver/linux-low.c | 26 +++++++++--
gdb/gdbserver/linux-x86-low.c | 10 +---
gdb/gdbserver/mem-break.c | 3 +-
gdb/gdbserver/regcache.c | 13 +++---
gdb/gdbserver/remote-utils.c | 2 +-
gdb/gdbserver/tdesc.c | 3 +-
gdb/gdbserver/thread-db.c | 9 +---
gdb/gdbserver/tracepoint.c | 98 +++++++++++++++++++++++++++-------------
gdb/gdbserver/utils.c | 19 --------
gdb/gdbserver/utils.h | 1 -
gdb/gdbserver/win32-i386-low.c | 3 +-
13 files changed, 165 insertions(+), 85 deletions(-)
hooks/post-receive
--
gdb and binutils