From: Jon Doron <arilou@gmail.com>
To: qemu-devel@nongnu.org
Cc: alex.bennee@linaro.org, Jon Doron <arilou@gmail.com>
Subject: [Qemu-devel] [PATCH v11 00/20] gdbstub: Refactor command packets handler
Date: Fri, 24 May 2019 19:00:58 +0300 [thread overview]
Message-ID: <20190524160118.31134-1-arilou@gmail.com> (raw)
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.
version 11 changes:
- Add reviewed by tag
- Requires review:
gdbstub: Implement deatch (D pkt) with new infra
gdbstub: Implement breakpoint commands (Z/z pkt) with new infra
gdbstub: Implement set register (P pkt) with new infra
gdbstub: Implement get register (p pkt) with new infra
gdbstub: Implement file io (F pkt) with new infra
gdbstub: Implement v commands with new infra
gdbstub: Implement generic set/query (Q/q pkt) with new infra
gdbstub: Implement target halted (? pkt) with new infra
gdbstub: Implement qemu physical memory mode
- Already reviewed:
gdbstub: Add infrastructure to parse cmd packets
gdbstub: Implement thread_alive (T pkt) with new infra
gdbstub: Implement continue (c pkt) with new infra
gdbstub: Implement continue with signal (C pkt) with new infra
gdbstub: Implement set_thread (H pkt) with new infra
gdbstub: Implement write memory (M pkt) with new infra
gdbstub: Implement read memory (m pkt) with new infra
gdbstub: Implement write all registers (G pkt) with new infra
gdbstub: Implement read all registers (g pkt) with new infra
gdbstub: Implement step (s pkt) with new infra
gdbstub: Clear unused variables in gdb_handle_packet
version 10 changes:
- Remove kvm added API as this is not really required and can be
accomplished by defining a coprocessor callback with a system
specific xml (see: 200bf5b7ffea635079cc05fdfb363372b9544ce7)
- Remove the new QEMU extended commands to read KVM MSRs
- Various fixes from Code Review by Alex Bennee
- Change the QEMU specific command to read physical memory to non-User QEMU
- Per patch changes:
gdbstub: Add infrastructure to parse cmd packets
* remove the union for the flags in GdbCmdParseEntry
gdbstub: Implement deatch (D pkt) with new infra
* Changed default handling for error flow / command not found
gdbstub: Implement continue with signal (C pkt) with new infra
* Added comment we dont support C sig;[addr] commands
gdbstub: Implement set_thread (H pkt) with new infra
* Change num_params check to be equal and not less than
gdbstub: Implement breakpoint commands (Z/z pkt) with new infra
* Merged z/Z commands into a single patch
gdbstub: Implement read memory (m pkt) with new infra
* Change num_params check to be equal and not less than
gdbstub: Implement file io (F pkt) with new infra
* Changed to have a single command parser
gdbstub: Implement generic set/query (Q/q pkt) with new infra
* Merged q/Q and qemu.Supported patches into a single patch
gdbstub: Implement target halted (? pkt) with new infra
* Removed TODO comment and added a note about it in the commit msg
gdbstub: Implement qemu physical memory mode
* Added CONFIG_USER_ONLY where required
version 9 changes:
- checkpatch fixes
version 8 changes:
- Add new command to display the Supported qemu generic query/sets
- kvm: Add API to read/write a MSR
- Add new commands specific for qemu:
* Command to swap the memory GDB sees to be the physical memory
* Commands to read and write a MSR
version 7 changes:
- Fixed few checkpatch complaints
- Feedback from Alex Bennee
version 4-6 changes:
- mostly feedback from Richard Henderson
version 3 changes
- Split the single patch to many individual patches for easier reviewing
version 2 changes
- Code convention fixes
Jon Doron (20):
gdbstub: Add infrastructure to parse cmd packets
gdbstub: Implement deatch (D pkt) with new infra
gdbstub: Implement thread_alive (T pkt) with new infra
gdbstub: Implement continue (c pkt) with new infra
gdbstub: Implement continue with signal (C pkt) with new infra
gdbstub: Implement set_thread (H pkt) with new infra
gdbstub: Implement breakpoint commands (Z/z pkt) with new infra
gdbstub: Implement set register (P pkt) with new infra
gdbstub: Implement get register (p pkt) with new infra
gdbstub: Implement write memory (M pkt) with new infra
gdbstub: Implement read memory (m pkt) with new infra
gdbstub: Implement write all registers (G pkt) with new infra
gdbstub: Implement read all registers (g pkt) with new infra
gdbstub: Implement file io (F pkt) with new infra
gdbstub: Implement step (s pkt) with new infra
gdbstub: Implement v commands with new infra
gdbstub: Implement generic set/query (Q/q pkt) with new infra
gdbstub: Implement target halted (? pkt) with new infra
gdbstub: Clear unused variables in gdb_handle_packet
gdbstub: Implement qemu physical memory mode
gdbstub.c | 1753 ++++++++++++++++++++++++++++++++++++++---------------
1 file changed, 1262 insertions(+), 491 deletions(-)
--
2.21.0
next reply other threads:[~2019-05-24 16:07 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-05-24 16:00 Jon Doron [this message]
2019-05-24 16:00 ` [Qemu-devel] [PATCH v11 01/20] gdbstub: Add infrastructure to parse cmd packets Jon Doron
2019-05-24 16:01 ` [Qemu-devel] [PATCH v11 02/20] gdbstub: Implement deatch (D pkt) with new infra Jon Doron
2019-05-27 8:37 ` Alex Bennée
2019-05-27 8:54 ` Alex Bennée
2019-05-27 8:58 ` Alex Bennée
2019-05-24 16:01 ` [Qemu-devel] [PATCH v11 03/20] gdbstub: Implement thread_alive (T " Jon Doron
2019-05-24 16:01 ` [Qemu-devel] [PATCH v11 04/20] gdbstub: Implement continue (c " Jon Doron
2019-05-24 16:01 ` [Qemu-devel] [PATCH v11 05/20] gdbstub: Implement continue with signal (C " Jon Doron
2019-05-24 16:01 ` [Qemu-devel] [PATCH v11 06/20] gdbstub: Implement set_thread (H " Jon Doron
2019-05-24 16:01 ` [Qemu-devel] [PATCH v11 07/20] gdbstub: Implement breakpoint commands (Z/z " Jon Doron
2019-05-27 9:52 ` Alex Bennée
2019-05-24 16:01 ` [Qemu-devel] [PATCH v11 08/20] gdbstub: Implement set register (P " Jon Doron
2019-05-24 16:01 ` [Qemu-devel] [PATCH v11 09/20] gdbstub: Implement get register (p " Jon Doron
2019-05-24 16:01 ` [Qemu-devel] [PATCH v11 10/20] gdbstub: Implement write memory (M " Jon Doron
2019-05-24 16:01 ` [Qemu-devel] [PATCH v11 11/20] gdbstub: Implement read memory (m " Jon Doron
2019-05-24 16:01 ` [Qemu-devel] [PATCH v11 12/20] gdbstub: Implement write all registers (G " Jon Doron
2019-05-24 16:01 ` [Qemu-devel] [PATCH v11 13/20] gdbstub: Implement read all registers (g " Jon Doron
2019-05-24 16:01 ` [Qemu-devel] [PATCH v11 14/20] gdbstub: Implement file io (F " Jon Doron
2019-05-24 16:01 ` [Qemu-devel] [PATCH v11 15/20] gdbstub: Implement step (s " Jon Doron
2019-05-24 16:01 ` [Qemu-devel] [PATCH v11 16/20] gdbstub: Implement v commands " Jon Doron
2019-05-24 16:01 ` [Qemu-devel] [PATCH v11 17/20] gdbstub: Implement generic set/query (Q/q pkt) " Jon Doron
2019-05-24 16:01 ` [Qemu-devel] [PATCH v11 18/20] gdbstub: Implement target halted (? " Jon Doron
2019-05-24 16:01 ` [Qemu-devel] [PATCH v11 19/20] gdbstub: Clear unused variables in gdb_handle_packet Jon Doron
2019-05-24 16:01 ` [Qemu-devel] [PATCH v11 20/20] gdbstub: Implement qemu physical memory mode Jon Doron
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190524160118.31134-1-arilou@gmail.com \
--to=arilou@gmail.com \
--cc=alex.bennee@linaro.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.