This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [Regression] Segfault on native-extended-gdbserver + fork
On 01/29/2018 05:31 PM, Pedro Alves wrote:
> On 01/29/2018 05:03 PM, Simon Marchi wrote:
>> On 2018-01-29 11:58, Pedro Alves wrote:
>> Agreed. Though I don't think Sergio's original patch is needed if we instead remove the detach_inferior call in remote.c, so your patch shouldn't be based on his.
>
> Yeah, my patch was a tangent / orthogonal to his regression fix.
> It was against master. I was thinking more about the whole
> effort to enable "set print inferior-events" on by default
> that Sergio is working on in context of:
> <https://sourceware.org/ml/gdb-patches/2018-01/msg00531.html>
>
And now that I reread that url, I remember that I was actually
proposing to drop the print from within exit_inferior. Otherwise,
we'd end up with (with both that url's patch and the patch I
just posted):
~~
[Inferior 1 (process 2629) exited normally]
[Inferior 1 (process 2629) exited]
~~
which is the sort of redundancy I was talking about.
Likely we'll end up doing the same to detach_inferior
too. So I'll just forget this patch for now, as it'll
probably end up unnecessary.
Thanks,
Pedro Alves