[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v9 00/27] gdbstub: Refactor command packets hand
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [PATCH v9 00/27] gdbstub: Refactor command packets handler |
Date: |
Wed, 15 May 2019 19:00:35 +0100 |
User-agent: |
mu4e 1.3.1; emacs 26.1 |
Jon Doron <address@hidden> writes:
> This patch series refactors the old gdbstub command packets handler
> with a new infrastructure which should ease extending and adding new
> and missing gdb command packets.
Jon,
I've finished my review and things are looking pretty good. The code is
a good clean-up and makes adding new features a lot easier. Thanks for
the examples of extensions - they were worth it to see how this might be
used although we shouldn't include them in the first merge. As they
extend the gdbserver ABI we'll want to think carefully about exactly
what we want to expose before we include it in master.
Going forwards aside from the various comments on each patch it would be
worth making sure the branch has gone through at least one CI run to
make sure the non-x86 builds (and disable-tcg and other exotica) haven't
been broken.
It would be nice if we could extend the testing of the gdbserver. Have
you been testing this with the gdb test suite or just manually? Now we
have system test and linux-user binaries being built we could probably
do better than the manually run tests/guest-debug/test-gdbstub.py.
Finally it would be nice if we could modernise the membuf and strbuf
handling with a more robust glib based approach. I understand if you
don't want to do that now and I'll happily accept the patches without it
but I did notice you can send the gdbserver a bit loopy if you send it
some very long maint packets so it would be nice to have that a bit
safer.
--
Alex Bennée
- Re: [Qemu-devel] [PATCH v9 13/27] gdbstub: Implement write all registers (G pkt) with new infra, (continued)
- [Qemu-devel] [PATCH v9 24/27] gdbstub: Add another handler for setting qemu.sstep, Jon Doron, 2019/05/02
- [Qemu-devel] [PATCH v9 15/27] gdbstub: Implement file io (F pkt) with new infra, Jon Doron, 2019/05/02
- [Qemu-devel] [PATCH v9 21/27] gdbstub: Clear unused variables in gdb_handle_packet, Jon Doron, 2019/05/02
- [Qemu-devel] [PATCH v9 25/27] kvm: Add API to read/write a CPU MSR value, Jon Doron, 2019/05/02
- [Qemu-devel] [PATCH v9 27/27] gdbstub: Add support to write a MSR for KVM target, Jon Doron, 2019/05/02
- Re: [Qemu-devel] [PATCH v9 00/27] gdbstub: Refactor command packets handler, Alex Bennée, 2019/05/15
- Re: [Qemu-devel] [PATCH v9 00/27] gdbstub: Refactor command packets handler,
Alex Bennée <=