[RFC 1/3] [gdb/dap] Fix exit race
Tom de Vries
tdevries@suse.de
Wed Feb 14 15:53:01 GMT 2024
On 2/14/24 16:34, Tom Tromey wrote:
>>>>>> "Tom" == Tom de Vries <tdevries@suse.de> writes:
>
> Tom> Anyway, I've now filed a separate PR for this issue, which I hope
> Tom> makes the discussion a bit clearer, so we have:
> Tom> - a PR for the assertion failure (PR31306)
> Tom> - a PR for ensuring responses are flushed to client before exiting
> Tom> (PR31380)
>
> Ok. FWIW I think the Queue idea seems totally fine, and combining these
> patches seems natural to me.
Sorry for going on this topic, but I'd like to understand why you think
this.
From my point of view, it's not a good idea to combine the patches
because the PRs have distinct root causes, which makes the discussion
about fixing them together confusing.
So, it's not that I _want_ to do this, it's that I'm convinced by the
evidence I've seen that this is the natural and obvious thing to do.
Which might mean I'm misunderstanding or overlooking something.
I start to wonder if perhaps you're thinking of a different race, which
is actually not discussed in the two PRs.
Thanks,
- Tom
More information about the Gdb-patches
mailing list