Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Joel Brobecker <brobecker@adacore.com>
To: Philippe Waroquiers <philippe.waroquiers@skynet.be>
Cc: gdb-patches@sourceware.org
Subject: added to 7.7-branch wiki: Re: RFA [PATCH] Implement 'catch syscall' for gdbserver
Date: Tue, 10 Sep 2013 18:07:00 -0000	[thread overview]
Message-ID: <20130910180718.GA6575@adacore.com> (raw)
In-Reply-To: <1377876388.3041.3.camel@soleil>

For the record, since this patch has been submitted in a reasonable
timeframe prior to the planned 7.7 branch creation, I've added it
to our 7.7 release wiki page. It does not mean that we must get
this patch in the release (for instance if considered too risky),
but hopefully this will place this patch a little higher on the
reviewing queue...

Thank you :)

> Here are the changes logs:
> ChangeLog
> 2013-xx-yy  Philippe Waroquiers  <philippe.waroquiers@skynet.be>
> 
> 	* NEWS: Document new QcatchSyscalls packet and its use
> 	in x86/amd64 linux gdbserver and Valgrind gdbserver.
> 	* remote.c (PACKET_QCatchSyscalls): New.
> 	(remote_protocol_features): Add Qcatchsyscalls.
> 	(remote_set_syscall_catchpoint): New function.
> 	(remote_parse_stop_reply): New stop reasons syscall_entry
> 	and syscall_return.
> 	(init_remote_ops): Registers remote_set_syscall_catchpoint
> 	and the config commands for [PACKET_QCatchSyscalls.
> 	* common/linux-ptrace.c (linux_check_ptrace_features):
> 	Detect PTRACE_O_TRACESYSGOOD for gdbserver.
> 	(ptrace_supports_feature): Initializes ptrace features if needed.
> 
> doc/ChangeLog
> 2013-xx-yy  Philippe Waroquiers  <philippe.waroquiers@skynet.be>
> 
> 	* gdb.texinfo (General Query Packets): Document new QcatchSyscalls
> 	packet.
> 	(Stop Reply Packets): Document new stop reasons syscall_entry and
> 	syscall_return.
> 	(Async Records): fixed syscall-return item name.
> 
> gdbserver/ChangeLog
> 2013-xx-yy  Philippe Waroquiers  <philippe.waroquiers@skynet.be>
> 
> 	* target.h (struct target_ops): Add supports_catch_syscall operation.
> 	* linux-low.h (struct linux_target_ops): Add get_syscall_trapinfo
> 	operation.
> 	* linux-low.c (linux_wait_1): Enables, detects and handle
> 	SYSCALL_SIGTRAP.
> 	(gdb_catched_syscall): New function.
> 	(get_syscall_trapinfo): New function.
>         (linux_supports_catch_syscall): New function.
> 	(struct target_ops linux_target_ops): Set linux_supports_catch_syscall.
> 	* linux-x86-low.c (x86_get_syscall_trapinfo): New function.
> 	(struct linux_target_ops the_low_target): Set x86_get_syscall_trapinfo.
> 	* remote-utils.c (prepare_resume_reply): Handle status kinds
> 	TARGET_WAITKIND_SYSCALL_ENTRY and TARGET_WAITKIND_SYSCALL_RETURN.
> 	* server.h: Declare catch_syscalls_p, catched_syscalls_size and
> 	catched_syscalls.
> 	* server.c: Define catch_syscalls_p, catched_syscalls_size and
> 	catched_syscalls.
> 	(handle_general_set): Handle QCatchSyscalls packet.
> 	(handle_query): Reports if low target supports QCatchSyscalls.
> 	* win32-low.c (struct target_ops win32_target_op): Sets NULL
> 	for supports_catch_syscall.
> 
> testsuite/ChangeLog
> 2013-xx-yy  Philippe Waroquiers  <philippe.waroquiers@skynet.be>
> 
> 	* gdb.base/catch-syscall.exp: Enables test for x86 and amd64
> 	gdbserver.

-- 
Joel


  parent reply	other threads:[~2013-09-10 18:07 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-30 15:26 Philippe Waroquiers
2013-08-30 15:44 ` Eli Zaretskii
2013-08-30 16:02   ` Philippe Waroquiers
2013-09-04 22:38 ` Philippe Waroquiers
2013-09-10 18:05 ` Sergio Durigan Junior
2013-09-10 18:07 ` Joel Brobecker [this message]
2013-09-19  5:29 ` Sergio Durigan Junior
2013-09-19 23:30   ` Philippe Waroquiers
2013-09-20  5:07     ` Sergio Durigan Junior

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20130910180718.GA6575@adacore.com \
    --to=brobecker@adacore.com \
    --cc=gdb-patches@sourceware.org \
    --cc=philippe.waroquiers@skynet.be \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox