From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mout.gmx.net (mout.gmx.net [212.227.17.20]) by sourceware.org (Postfix) with ESMTPS id 53C5A39450EC for ; Wed, 18 Mar 2020 23:18:09 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 53C5A39450EC Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=gmx.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=n54@gmx.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gmx.net; s=badeba3b8450; t=1584573475; bh=VJ+qnAAritYCrLQfP+sYONdVFDTEde9Hs99DW+RMlyI=; h=X-UI-Sender-Class:From:To:Cc:Subject:Date:In-Reply-To:References; b=Gb3Nfqdq5hqDdfv/k75vaKg8pxeGv+6zVyDCIlWW9VFcim+/epSwMtgd9MRUbtu5v Om6oK7D7FLYlKzwu7KqGH/O8VxUPlQi3r9SYDs57RQBSpcCC/5kRSwcv1n8ICofxX1 XGHJ2SQIhjWgIb4pJCI4ISO7UW2/6FGh5Bfcb9K4= X-UI-Sender-Class: 01bb95c1-4bf8-414a-932a-4f6e2808ef9c Received: from localhost.localdomain ([89.79.191.25]) by mail.gmx.com (mrgmx104 [212.227.17.174]) with ESMTPSA (Nemesis) id 1Mlf0U-1jeO2r0H1W-00ioGu; Thu, 19 Mar 2020 00:17:55 +0100 From: Kamil Rytarowski To: gdb-patches@sourceware.org Cc: tom@tromey.com, Kamil Rytarowski Subject: [PATCH v3] Disable get_ptrace_pid for NetBSD Date: Thu, 19 Mar 2020 00:16:51 +0100 Message-Id: <20200318231651.18045-1-n54@gmx.com> X-Mailer: git-send-email 2.25.0 In-Reply-To: <20200318215531.25248-1-n54@gmx.com> References: <20200318215531.25248-1-n54@gmx.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Provags-ID: V03:K1:rMFJbTFSh4sggIdkEY7AWGfSlqW7/6i9KOduJPsW81ra3YHdvKL hKu/Z8TQ+3ZhhMOzG+ColmXHKljoeNMhqAo5Gcm3IuAgcBXtm+krvP2W0afmmCw+pNK5t8p pvKHy1wQDIE7fKLcAZrOFdnyHb7vJXfr5xGwMtVvmQ11cyMAJsDOvTjRQYvoezHG77leXoe e1c0nJY9yo5TpgKNlq+4w== X-UI-Out-Filterresults: notjunk:1;V03:K0:8HZCCv0FY4o=:tdGxg8tsbgi2y/VPnalU8M Ha7j5C1TVFtPt/jhL40/47bqF4DlJrEqa1QX6jfPy7YT1qXQaZnmbbjEVdBC8XxXqkBnKv+tV H3OkQVh+aJU0seGpEPqGBMvbGlJza8doIiBMW7pharB3m68uCnjVEjjqz1kMkt6Th3aDww0x0 nv0nC7NI72bhehY3XMcGHRXrSGF5CnZ6yKGbDd37Zip+yMu0jWKHcvtdckxeagRET/Eo294FR Pj7fesGrKtKfXVN4LiGkG8HPriVidYre7JXAk00u8VTGn2q3CqhlmrimFmWfJRLJcbeRqcZQ/ tnch52CpjUBXGytsWopJ70vjHvC50LUudKyB5iR3FZu2sNb58aVRHcF7YpMzbQs0/MuLV7uFQ Ch1NZUJzjR7PcfJHABhYRe7TIhvHWJnGayfJGhV5w1GL947cQs29YBPPPRbfmAjankuirIzQC iRI6Ts9Z5U17eRbPlTXVhwrFipUW7WuciFfSu0e1KUlq49l09srPgNNzUEqLibfzXuNHIS26E mgZH/KdGTQR20zlmRLzf1dJ9zUeX5j9f09gbwLZ3Kw2LsXcqmNJhPr6bQIWSNYktYi4IDwLIZ udwX5KDbSPOlcv5EmyTlpJkEfrcwBJjFHe32RzhB8C6Atx9rrpbnpPb0lBVznE6tOy9HgNVyp Nu19eDS5W7PH0iNhr2cWjiO6IUiPv8fcZMb2jVAjKZydPJJ2RIJ3AzjCwKPAfxeMvQuWYK8Np 2JhKw4x+aURgx2IpfNYcCBOY2PekwX+GdgRk2QQzXwmKE/SbcUc+mfIdYYucHcqf9/OJunFHV MgyjAh4hPYApYkPBRraRcxopKWWadbmuGgd5dD1P0LE1a34pet7bmneRy8uXPLThBUfyAN63I RFvBN1DqEVv/ifMrG2oU0jaeh2j5BS2lOglp5TMZliVvkqpbrqB44RjuX3wNB0LIvyZaC52Bd euCtJXzqIzbdzCE2lQmyAM7dM7oaSDHAA5zNeOZrX72YfT1FxQOXQw3ljZkUCqpobDcFYzt8C 1qUI/IwE7Ma4wcUKSqWrr1FgB66PfEQUzzZLiA+/fYJvXeWcH65D+fjHxdGdhD+sxcfAPnD4U Lf9gpwfuur8xEi8FATcLxPbHZUMg3ZpFozBjSrAbl3aWXPfz/hSAJGQsJIV0w/oHb/hXglSkB dxd5oLvVaiRd26bmIRaFoT6RTlyY+cz+w7fI2UZogz+z/s1RsOr2hsZbhS5+nn+Tn79uA= X-Spam-Status: No, score=-25.4 required=5.0 tests=DKIM_SIGNED, DKIM_VALID, FREEMAIL_ENVFROM_END_DIGIT, FREEMAIL_FROM, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, RCVD_IN_DNSWL_LOW, SPF_HELO_NONE, SPF_PASS autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 18 Mar 2020 23:18:11 -0000 Unlike most other Operating Systems, NetBSD tracks both pid and lwp. The process id on NetBSD is stored always in the pid field of ptid. gdb/ChangeLog: * inf-ptrace.h: Disable get_ptrace_pid on NetBSD. * inf-ptrace.c: Likewise. * (gdb_ptrace): Add. * (inf_ptrace_target::resume): Update. * (inf_ptrace_target::xfer_partial): Likewise. * (inf_ptrace_peek_poke): Change argument `pid' to `ptid'. * (inf_ptrace_peek_poke): Update. =2D-- gdb/ChangeLog | 10 ++++++++++ gdb/inf-ptrace.c | 45 +++++++++++++++++++++++++++++---------------- gdb/inf-ptrace.h | 7 ++++++- 3 files changed, 45 insertions(+), 17 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 84964dc00ac..f09b4d154c5 100644 =2D-- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,13 @@ +2020-03-18 Kamil Rytarowski + + * inf-ptrace.h: Disable get_ptrace_pid on NetBSD. + * inf-ptrace.c: Likewise. + * (gdb_ptrace): Add. + * (inf_ptrace_target::resume): Update. + * (inf_ptrace_target::xfer_partial): Likewise. + * (inf_ptrace_peek_poke): Change argument `pid' to `ptid'. + * (inf_ptrace_peek_poke): Update. + 2020-03-17 Kamil Rytarowski * regformats/regdef.h: Put reg in gdb namespace. diff --git a/gdb/inf-ptrace.c b/gdb/inf-ptrace.c index db17a76d946..4003b888616 100644 =2D-- a/gdb/inf-ptrace.c +++ b/gdb/inf-ptrace.c @@ -37,6 +37,18 @@ =0C +static int +gdb_ptrace (PTRACE_TYPE_ARG1 request, ptid_t ptid, PTRACE_TYPE_ARG3 addr, + PTRACE_TYPE_ARG4 data) +{ +#ifdef __NetBSD__ + return ptrace (request, ptid.pid (), addr, data); +#else + pid_t pid =3D get_ptrace_pid (ptid); + return ptrace (request, pid, addr, data); +#endif +} + /* A unique_ptr helper to unpush a target. */ struct target_unpusher @@ -313,8 +325,9 @@ inf_ptrace_target::kill () target_mourn_inferior (inferior_ptid); } -/* Return which PID to pass to ptrace in order to observe/control the - tracee identified by PTID. */ +#ifndef __NetBSD__ + +/* See inf-ptrace.h. */ pid_t get_ptrace_pid (ptid_t ptid) @@ -328,6 +341,7 @@ get_ptrace_pid (ptid_t ptid) pid =3D ptid.pid (); return pid; } +#endif /* Resume execution of thread PTID, or all threads if PTID is -1. If STEP is nonzero, single-step it. If SIGNAL is nonzero, give it @@ -336,15 +350,14 @@ get_ptrace_pid (ptid_t ptid) void inf_ptrace_target::resume (ptid_t ptid, int step, enum gdb_signal signal) { - pid_t pid; int request; if (minus_one_ptid =3D=3D ptid) /* Resume all threads. Traditionally ptrace() only supports single-threaded processes, so simply resume the inferior. */ - pid =3D inferior_ptid.pid (); + ptid =3D inferior_ptid; else - pid =3D get_ptrace_pid (ptid); + ptid =3D ptid_t (inferior_ptid.pid ()); if (catch_syscall_enabled () > 0) request =3D PT_SYSCALL; @@ -365,7 +378,7 @@ inf_ptrace_target::resume (ptid_t ptid, int step, enum= gdb_signal signal) where it was. If GDB wanted it to start some other way, we have already written a new program counter value to the child. */ errno =3D 0; - ptrace (request, pid, (PTRACE_TYPE_ARG3)1, gdb_signal_to_host (signal))= ; + gdb_ptrace (request, ptid, (PTRACE_TYPE_ARG3)1, gdb_signal_to_host (sig= nal)); if (errno !=3D 0) perror_with_name (("ptrace")); } @@ -460,7 +473,7 @@ inf_ptrace_target::wait (ptid_t ptid, struct target_wa= itstatus *ourstatus, be non-null. Return the number of transferred bytes. */ static ULONGEST -inf_ptrace_peek_poke (pid_t pid, gdb_byte *readbuf, +inf_ptrace_peek_poke (ptid_t ptid, gdb_byte *readbuf, const gdb_byte *writebuf, ULONGEST addr, ULONGEST len) { @@ -491,8 +504,8 @@ inf_ptrace_peek_poke (pid_t pid, gdb_byte *readbuf, if (readbuf !=3D NULL || chunk < sizeof (PTRACE_TYPE_RET)) { errno =3D 0; - buf.word =3D ptrace (PT_READ_I, pid, - (PTRACE_TYPE_ARG3)(uintptr_t) addr, 0); + buf.word =3D gdb_ptrace (PT_READ_I, ptid, + (PTRACE_TYPE_ARG3)(uintptr_t) addr, 0); if (errno !=3D 0) break; if (readbuf !=3D NULL) @@ -502,15 +515,15 @@ inf_ptrace_peek_poke (pid_t pid, gdb_byte *readbuf, { memcpy (buf.byte + skip, writebuf + n, chunk); errno =3D 0; - ptrace (PT_WRITE_D, pid, (PTRACE_TYPE_ARG3)(uintptr_t) addr, + gdb_ptrace (PT_WRITE_D, ptid, (PTRACE_TYPE_ARG3)(uintptr_t) addr, buf.word); if (errno !=3D 0) { /* Using the appropriate one (I or D) is necessary for Gould NP1, at least. */ errno =3D 0; - ptrace (PT_WRITE_I, pid, (PTRACE_TYPE_ARG3)(uintptr_t) addr, - buf.word); + gdb_ptrace (PT_WRITE_I, ptid, (PTRACE_TYPE_ARG3)(uintptr_t) addr, + buf.word); if (errno !=3D 0) break; } @@ -528,7 +541,7 @@ inf_ptrace_target::xfer_partial (enum target_object ob= ject, const gdb_byte *writebuf, ULONGEST offset, ULONGEST len, ULONGEST *xfered_len) { - pid_t pid =3D get_ptrace_pid (inferior_ptid); + ptid_t ptid =3D inferior_ptid; switch (object) { @@ -552,7 +565,7 @@ inf_ptrace_target::xfer_partial (enum target_object ob= ject, piod.piod_len =3D len; errno =3D 0; - if (ptrace (PT_IO, pid, (caddr_t)&piod, 0) =3D=3D 0) + if (gdb_ptrace (PT_IO, ptid, (caddr_t)&piod, 0) =3D=3D 0) { /* Return the actual number of bytes read or written. */ *xfered_len =3D piod.piod_len; @@ -565,7 +578,7 @@ inf_ptrace_target::xfer_partial (enum target_object ob= ject, return TARGET_XFER_EOF; } #endif - *xfered_len =3D inf_ptrace_peek_poke (pid, readbuf, writebuf, + *xfered_len =3D inf_ptrace_peek_poke (ptid, readbuf, writebuf, offset, len); return *xfered_len !=3D 0 ? TARGET_XFER_OK : TARGET_XFER_EOF; @@ -588,7 +601,7 @@ inf_ptrace_target::xfer_partial (enum target_object ob= ject, piod.piod_len =3D len; errno =3D 0; - if (ptrace (PT_IO, pid, (caddr_t)&piod, 0) =3D=3D 0) + if (gdb_ptrace (PT_IO, ptid, (caddr_t)&piod, 0) =3D=3D 0) { /* Return the actual number of bytes read or written. */ *xfered_len =3D piod.piod_len; diff --git a/gdb/inf-ptrace.h b/gdb/inf-ptrace.h index dd0733736f2..dea82d005e3 100644 =2D-- a/gdb/inf-ptrace.h +++ b/gdb/inf-ptrace.h @@ -78,9 +78,14 @@ struct inf_ptrace_target : public inf_child_target void detach_success (inferior *inf); }; +#ifndef __NetBSD__ /* Return which PID to pass to ptrace in order to observe/control the - tracee identified by PTID. */ + tracee identified by PTID. + + Unlike most other Operating Systems, NetBSD tracks both pid and lwp + and avoids this function. */ extern pid_t get_ptrace_pid (ptid_t); +#endif #endif =2D- 2.25.0