[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V3] gdbstub: Fix handler for 'F' packet
From: |
Richard Henderson |
Subject: |
Re: [Qemu-devel] [PATCH V3] gdbstub: Fix handler for 'F' packet |
Date: |
Tue, 27 Aug 2019 20:12:38 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 |
On 8/27/19 3:33 PM, Sandra Loosemore wrote:
> Handling of the 'F' packet has been broken since commit
> 4b20fab101b9e2d0fb47454209637a17fc7a13d5, which converted it to use
> the new packet parsing infrastructure. Per the GDB RSP specification
>
> https://sourceware.org/gdb/current/onlinedocs/gdb/The-F-Reply-Packet.html
>
> the second parameter may be omitted, but the rewritten implementation
> was failing to recognize this case. The result was that QEMU was
> repeatedly resending the fileio request and ignoring GDB's replies of
> successful completion. This patch restores the behavior of the
> previous code in allowing the errno parameter to be omitted and
> passing 0 to the callback in that case.
>
> Signed-off-by: Sandra Loosemore <address@hidden>
> ---
> gdbstub.c | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
Reviewed-by: Richard Henderson <address@hidden>
r~