From: "Alex Bennée" <alex.bennee@linaro.org>
To: Jon Doron <arilou@gmail.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v9 00/27] gdbstub: Refactor command packets handler
Date: Thu, 16 May 2019 13:44:06 +0100 [thread overview]
Message-ID: <8736lemuw9.fsf@zen.linaroharston> (raw)
In-Reply-To: <877earmwcc.fsf@zen.linaroharston>
Alex Bennée <alex.bennee@linaro.org> writes:
> Jon Doron <arilou@gmail.com> 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,
>
<snip>
>
> 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.
Actually I had a go at this this morning and it turned out to be quite
fiddly so perhaps this is something best left to a follow-up series once
the re-factoring is in.
--
Alex Bennée
prev parent reply other threads:[~2019-05-16 13:10 UTC|newest]
Thread overview: 70+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-05-02 8:15 [Qemu-devel] [PATCH v9 00/27] gdbstub: Refactor command packets handler Jon Doron
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 01/27] gdbstub: Add infrastructure to parse cmd packets Jon Doron
2019-05-14 18:24 ` Alex Bennée
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 02/27] gdbstub: Implement deatch (D pkt) with new infra Jon Doron
2019-05-14 18:54 ` Alex Bennée
2019-05-21 4:47 ` Jon Doron
2019-05-21 6:43 ` Alex Bennée
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 03/27] gdbstub: Implement thread_alive (T " Jon Doron
2019-05-15 8:27 ` Alex Bennée
2019-05-15 8:33 ` Alex Bennée
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 04/27] gdbstub: Implement continue (c " Jon Doron
2019-05-15 8:34 ` Alex Bennée
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 05/27] gdbstub: Implement continue with signal (C " Jon Doron
2019-05-15 9:43 ` Alex Bennée
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 06/27] gdbstub: Implement set_thread (H " Jon Doron
2019-05-15 10:06 ` Alex Bennée
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 07/27] gdbstub: Implement insert breakpoint (Z " Jon Doron
2019-05-15 10:26 ` Alex Bennée
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 08/27] gdbstub: Implement remove breakpoint (z " Jon Doron
2019-05-15 10:27 ` Alex Bennée
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 09/27] gdbstub: Implement set register (P " Jon Doron
2019-05-15 12:14 ` Alex Bennée
2019-05-19 10:32 ` Jon Doron
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 10/27] gdbstub: Implement get register (p " Jon Doron
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 11/27] gdbstub: Implement write memory (M " Jon Doron
2019-05-15 15:22 ` Alex Bennée
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 12/27] gdbstub: Implement read memory (m " Jon Doron
2019-05-15 15:30 ` Alex Bennée
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 13/27] gdbstub: Implement write all registers (G " Jon Doron
2019-05-15 16:01 ` Alex Bennée
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 14/27] gdbstub: Implement read all registers (g " Jon Doron
2019-05-15 16:10 ` Alex Bennée
2019-05-19 10:42 ` Jon Doron
2019-05-19 14:55 ` Alex Bennée
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 15/27] gdbstub: Implement file io (F " Jon Doron
2019-05-15 16:54 ` Alex Bennée
2019-05-19 11:35 ` Jon Doron
2019-05-19 14:54 ` Alex Bennée
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 16/27] gdbstub: Implement step (s " Jon Doron
2019-05-15 16:55 ` Alex Bennée
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 17/27] gdbstub: Implement v commands " Jon Doron
2019-05-15 17:06 ` Alex Bennée
2019-05-20 4:38 ` Jon Doron
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 18/27] gdbstub: Implement generic query (q pkt) " Jon Doron
2019-05-15 17:12 ` Alex Bennée
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 19/27] gdbstub: Implement generic set (Q " Jon Doron
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 20/27] gdbstub: Implement target halted (? " Jon Doron
2019-05-15 17:20 ` Alex Bennée
2019-05-20 5:32 ` Jon Doron
2019-05-20 12:54 ` Alex Bennée
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 21/27] gdbstub: Clear unused variables in gdb_handle_packet Jon Doron
2019-05-15 17:24 ` Alex Bennée
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 22/27] gdbstub: Implement generic query qemu.Supported Jon Doron
2019-05-15 17:41 ` Alex Bennée
2019-05-20 4:50 ` Jon Doron
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 23/27] gdbstub: Implement qemu physical memory mode Jon Doron
2019-05-15 15:13 ` Alex Bennée
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 24/27] gdbstub: Add another handler for setting qemu.sstep Jon Doron
2019-05-15 17:44 ` Alex Bennée
2019-05-20 5:17 ` Jon Doron
2019-05-20 12:40 ` Alex Bennée
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 25/27] kvm: Add API to read/write a CPU MSR value Jon Doron
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 26/27] gdbstub: Add support to read a MSR for KVM target Jon Doron
2019-05-15 17:48 ` Alex Bennée
2019-05-20 5:24 ` Jon Doron
2019-05-20 12:42 ` Alex Bennée
2019-05-02 8:15 ` [Qemu-devel] [PATCH v9 27/27] gdbstub: Add support to write " Jon Doron
2019-05-15 13:19 ` [Qemu-devel] [PATCH v9 00/27] gdbstub: Refactor command packets handler Alex Bennée
2019-05-15 18:00 ` Alex Bennée
2019-05-16 12:44 ` Alex Bennée [this message]
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=8736lemuw9.fsf@zen.linaroharston \
--to=alex.bennee@linaro.org \
--cc=arilou@gmail.com \
--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.