All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jon Doron <arilou@gmail.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: "Edgar E . Iglesias" <edgar.iglesias@xilinx.com>,
	"Philippe Mathieu-Daudé" <f4bug@amsat.org>,
	"QEMU Developers" <qemu-devel@nongnu.org>,
	"Alistair Francis" <alistair.francis@wdc.com>,
	"Alex Bennée" <alex.bennee@linaro.org>,
	"Luc Michel" <luc.michel@greensocs.com>
Subject: Re: [PATCH-for-5.0] gdbstub: Use correct address space with Qqemu.PhyMemMode packet
Date: Mon, 1 Jun 2020 10:29:48 +0300	[thread overview]
Message-ID: <20200601072948.GG3071@jondnuc> (raw)
In-Reply-To: <CAFEAcA95hfnjBjUT0hCxPUzBnGL9xKDkEjBqDcyVBD6spVZ_4w@mail.gmail.com>

On 31/05/2020, Peter Maydell wrote:
>On Sun, 31 May 2020 at 17:42, Jon Doron <arilou@gmail.com> wrote:
>>
>> On 31/05/2020, Philippe Mathieu-Daudé wrote:
>> >On 3/30/20 6:41 PM, Peter Maydell wrote:
>> >> PS: do we have any documentation of this new command ?
>> >> ab4752ec8d9 has the implementation but no documentation...
>> >
>> >Jon, do you have documentation on the Qqemu.PhyMemMode packet?
>
>> Hi, there is no documentation for this mode, but in general the idea was
>> very simple.
>>
>> I want to have GDB the option to see the physical memory and examine it
>> and have this option toggled.
>>
>> This was useful to me when I was working on nested virtual machine and I
>> wanted to examine different states of the VMCS12 and EPTs.
>>
>> I used this in the following commands:
>> // Enable
>> maint packet Qqemu.PhyMemMode:1
>>
>> // Disable
>> maint packet Qqemu.PhyMemMode:0
>
>docs/system/gdb.rst would be the place to document QEMU-specific
>extensions to the gdb protocol (there's an "advanced debugging
>options" section where we document things like the single-step
>stuff you can also change via 'maint packet').
>
>thanks
>-- PMM

Thanks, I'll know for next time, when I did all that work and 
re-factored gdbstub, that doc did not exist.

Perhaps Peter can just add the documentation to this commit?

-- Jon.


  reply	other threads:[~2020-06-01  7:30 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-03-30 15:30 [PATCH-for-5.0] gdbstub: Use correct address space with Qqemu.PhyMemMode packet Philippe Mathieu-Daudé
2020-03-30 16:08 ` Peter Maydell
2020-03-30 16:21   ` Philippe Mathieu-Daudé
2020-03-30 16:41     ` Peter Maydell
2020-05-31 15:27       ` Philippe Mathieu-Daudé
2020-05-31 16:42         ` Jon Doron
2020-05-31 16:57           ` Peter Maydell
2020-06-01  7:29             ` Jon Doron [this message]
2020-06-01 10:39               ` Alex Bennée
2020-06-01 10:41                 ` 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=20200601072948.GG3071@jondnuc \
    --to=arilou@gmail.com \
    --cc=alex.bennee@linaro.org \
    --cc=alistair.francis@wdc.com \
    --cc=edgar.iglesias@xilinx.com \
    --cc=f4bug@amsat.org \
    --cc=luc.michel@greensocs.com \
    --cc=peter.maydell@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.