This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] Fix usage of inferior_ptid in two thread_alive implementations
- From: John Baldwin <jhb at freebsd dot org>
- To: gdb-patches at sourceware dot org
- Cc: Pedro Alves <palves at redhat dot com>, Simon Marchi <simon dot marchi at polymtl dot ca>, Simon Marchi <simon dot marchi at ericsson dot com>
- Date: Wed, 29 Mar 2017 12:32:42 -0700
- Subject: Re: [PATCH] Fix usage of inferior_ptid in two thread_alive implementations
- Authentication-results: sourceware.org; auth=none
- References: <20170207212450.2232-1-simon.marchi@ericsson.com> <f20baf9481cbd876a3c87876bbe50445@polymtl.ca> <8688edde-759a-7f84-482d-9823a8f546b3@redhat.com>
On Thursday, February 09, 2017 05:53:16 PM Pedro Alves wrote:
> On 02/09/2017 04:46 PM, Simon Marchi wrote:
> > On 2017-02-08 07:40, Pedro Alves wrote:
> >> On 02/07/2017 09:24 PM, Simon Marchi wrote:
> >>> While inspecting some target code, I noticed that in these two
> >>> implementations of thread_alive, inferior_ptid is referenced directly
> >>> instead of using the ptid passed as parameters. I guess that it is
> >>> wrong, although I can't really test it in both cases.
> >>
> >> I can't test either, but it looks right to me.
> >
> > Soooo.. is this an approval :) ?
>
> OK by the end of the week to give a chance of area
> maintainers or interested folks to comment. E.g., Eli
> is the go32-nat.c maintainer and I don't mean to overstep,
> though that bit does look obvious to me. Mark or someone
> with BSD access could want to comment on the BSD bit. The
> latter you could perhaps test on the compile farm, if you
> have access.
[ Apologies for the late reply ]
I don't think any modern versions of either FreeBSD or OpenBSD
use bsd-uthread.c. The uthread target is for a thread implementation
that only provided threads in userland on top of a single process
(like "green" threads in the JDK). FreeBSD last shipped a release
with support for this thread model (FreeBSD 6.4) in 2008. I can't
speak to when OpenBSD stopped using user-based threads, but the
existence of kernel-based thread support in the obsd-nat target
implies it isn't used anymore either. For FreeBSD at least I think
it would be fine to remove bsd-uthread.c.
(I should also send a patch in to remove FreeBSD/alpha as that
platform was retired at around the same time. The last release
to support alpha was 6.3 also released in 2008.)
--
John Baldwin