[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v8 07/16] gdbstub: add multiprocess support to (
From: |
Max Filippov |
Subject: |
Re: [Qemu-devel] [PATCH v8 07/16] gdbstub: add multiprocess support to (f|s)ThreadInfo and ThreadExtraInfo |
Date: |
Tue, 29 Jan 2019 12:25:34 -0800 |
On Tue, Jan 29, 2019 at 2:05 AM Peter Maydell <address@hidden> wrote:
> > Starting with this commit it is no longer possible to kill QEMU
> > with the 'kill' command from the gdb. This was a nice feature,
> > was this removal intentional, or is it just an implementation
> > bug?
>
> That sounds like a bug. I think with the multiprocess extensions
> available gdb may switch from killing using the 'k' packet to using
> the 'vKill;pid' packet, which we don't implement? Looking at
> what gdb is sending ('set debug remote 1' turns on logging in
> gdb of remote protocol packets) would let us check that theory.
That's correct:
(gdb) kill
Kill the program being debugged? (y or n) y
Sending packet: $vKill;1#6e...Ack
Packet received:
Packet vKill (kill) is NOT supported
Can't kill process
> It's not clear how our implementation should deal with being asked
> to kill just one process if we have more than one, though...
I'll send a fix that restores previous behavior in case of a single
inferior process.
--
Thanks.
-- Max