On Thu, Jan 13, 2011 at 12:46:57PM +0100, Fabien Chouteau wrote:
> On Windows, this is required to flush the remaining data in the IO stream,
> otherwise Gdb do not receive the last packet.
>
> Version 2:
> Fix linux-user build error.
Hi, It seems a bit odd to explicitely close for system mode but
to implicitely close (at exit) for user mode. But if no one
complains, I'm OK with this version.
Cheers
>
> Signed-off-by: Fabien Chouteau <chout...@adacore.com>
> ---
> gdbstub.c | 6 ++++++
> 1 files changed, 6 insertions(+), 0 deletions(-)
>
> diff --git a/gdbstub.c b/gdbstub.c
> index 0aa081b..d6556c9 100644
> --- a/gdbstub.c
> +++ b/gdbstub.c
> @@ -2391,6 +2391,12 @@ void gdb_exit(CPUState *env, int code)
>
> snprintf(buf, sizeof(buf), "W%02x", (uint8_t)code);
> put_packet(s, buf);
> +
> +#ifndef CONFIG_USER_ONLY
> + if (s->chr) {
> + qemu_chr_close(s->chr);
> + }
> +#endif
> }
>
> #ifdef CONFIG_USER_ONLY
> --
> 1.7.1
>
>
