All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jan Kiszka <jan.kiszka@web.de>
To: Eddie Kohler <kohler@cs.ucla.edu>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] Re: [PATCH] i386 debugging stubs: Consider segment bases
Date: Mon, 27 Sep 2010 08:29:55 +0200	[thread overview]
Message-ID: <4CA039E3.9030604@web.de> (raw)
In-Reply-To: <4C9F80A1.7050302@cs.ucla.edu>

[-- Attachment #1: Type: text/plain, Size: 1404 bytes --]

Am 26.09.2010 19:19, Eddie Kohler wrote:
> OK, thanks.  I understand how you're relying on the current behavior.
> 
> I'd rather not change all of QEMU and GDB in one step,

The first step is changing gdb anyway.

> but I'd like to
> address this.  QEMU documentation implies, and new users expect, that
> debugging uses virtual addresses, not the segmentation-specific "linear
> addresses" that are actually used now.
> 
> - How about a maintenance packet type that changed behavior to what I
> would prefer (breakpoints and memory access use virtual addresses, not
> linear addresses)?
> 
> - We could add a "segment identifier" parameter to
> cpu_get_phys_page_debug, ignored on all targets but i386 at first.  Then
> we could pass information through to cpu_get_phys_page_debug about what
> kind of address is being translated.  This change could be propagated to
> cpu_memory_rw_debug (now or later).  Would you object?

These changes would establish a temporary interface for an incomplete
workaround, and that even with impact on non-x86 code. I would prefer if
you could invest your time on the gdb side instead. Anything improved
there is not lost - in contrast to the modifications of qemu. I would
try to support any effort in this direction.

BTW, gdb has nice Python binding these days, so some extensions may also
start their life as a helper script.

Jan


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 259 bytes --]

      reply	other threads:[~2010-09-27  6:30 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-25  0:25 [Qemu-devel] [PATCH] i386 debugging stubs: Consider segment bases Eddie Kohler
2010-09-25  7:22 ` [Qemu-devel] " Jan Kiszka
2010-09-25  8:35   ` Eddie Kohler
2010-09-26  6:44     ` Jan Kiszka
2010-09-26 17:19       ` Eddie Kohler
2010-09-27  6:29         ` Jan Kiszka [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=4CA039E3.9030604@web.de \
    --to=jan.kiszka@web.de \
    --cc=kohler@cs.ucla.edu \
    --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.