This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: hurd: update RPC prototypes
- From: Pedro Alves <palves at redhat dot com>
- To: Samuel Thibault <samuel dot thibault at gnu dot org>, gdb-patches at sourceware dot org
- Cc: Thomas Schwinge <thomas at codesourcery dot com>
- Date: Mon, 4 Sep 2017 13:14:33 +0100
- Subject: Re: hurd: update RPC prototypes
- Authentication-results: sourceware.org; auth=none
- Authentication-results: ext-mx02.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com
- Authentication-results: ext-mx02.extmail.prod.ext.phx2.redhat.com; spf=fail smtp.mailfrom=palves at redhat dot com
- Dmarc-filter: OpenDMARC Filter v1.3.2 mx1.redhat.com ABFF67CB90
- References: <20170827184102.fddwriwehw6kwpv7@var.youpi.perso.aquilenet.fr>
On 08/27/2017 07:41 PM, Samuel Thibault wrote:
> Since hurd's baf7e5c ('hurd: Use polymorphic port types to return some
> rights.'), some RPCs prototypes have changed, gdb needs the
> corresponding update.
>
> * gdb/gnu-nat.c (S_proc_getmsgport_reply, S_proc_task2proc_reply,
> S_proc_pid2proc_reply): Add `mach_msg_type_name_t type' parameter.
Say someone downloads some prebuilt Debian Hurd image or some
such and wants to build newer gdb on that system. I assume that that would be
broken with this change? What's the policy regarding building ToT gdb on
non-ToT Hurd systems? Is the intention to only ever support building
ToT gdb with ToT Hurd?
Thanks,
Pedro Alves
>
> diff --git a/gdb/gnu-nat.c b/gdb/gnu-nat.c
> index d5e3841e68..402027866b 100644
> --- a/gdb/gnu-nat.c
> +++ b/gdb/gnu-nat.c
> @@ -1874,17 +1876,19 @@ ILL_RPC (S_proc_setmsgport_reply,
> mach_port_t oldmsgport)
> ILL_RPC (S_proc_getmsgport_reply,
> mach_port_t reply_port, kern_return_t return_code,
> - mach_port_t msgports)
> + mach_port_t msgports, mach_msg_type_name_t type)
> ILL_RPC (S_proc_pid2task_reply,
> mach_port_t reply_port, kern_return_t return_code, mach_port_t task)
> ILL_RPC (S_proc_task2pid_reply,
> mach_port_t reply_port, kern_return_t return_code, pid_t pid)
> ILL_RPC (S_proc_task2proc_reply,
> - mach_port_t reply_port, kern_return_t return_code, mach_port_t proc)
> + mach_port_t reply_port, kern_return_t return_code,
> + mach_port_t proc, mach_msg_type_name_t type)
> ILL_RPC (S_proc_proc2task_reply,
> mach_port_t reply_port, kern_return_t return_code, mach_port_t task)
> ILL_RPC (S_proc_pid2proc_reply,
> - mach_port_t reply_port, kern_return_t return_code, mach_port_t proc)
> + mach_port_t reply_port, kern_return_t return_code,
> + mach_port_t proc, mach_msg_type_name_t type)
> ILL_RPC (S_proc_getprocinfo_reply,
> mach_port_t reply_port, kern_return_t return_code,
> int flags, procinfo_t procinfo, mach_msg_type_number_t procinfoCnt,
>