qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Jan Kiszka <jan.kiszka@siemens.com>
To: Paul Brook <paul@codesourcery.com>
Cc: Anthony Liguori <aliguori@us.ibm.com>,
	qemu-devel@nongnu.org, Gleb Natapov <gleb@redhat.com>
Subject: Re: [Qemu-devel] [PATCH] monitor: Add port write command
Date: Wed, 15 Jul 2009 12:40:49 +0200	[thread overview]
Message-ID: <4A5DB231.7040109@siemens.com> (raw)
In-Reply-To: <200907151114.21482.paul@codesourcery.com>

Paul Brook wrote:
>>> I'd be reluctant to expose the savevm state to the user.
>>>
>>> For debugging qemu I don't see it providing any real benefit over firing
>>> up GDB and poking directly at the device directly.
>> Not all environments where you need to debug things have gdb, qemu
>> sources or even non striped qemu binary.
> 
> If you don't have qemu sources than I really don't care. By definition you're 
> not going to be able to do anything useful even if you do figure out what the 
> problem is.  Note that there's no requirement that you run gdb on the target 
> itself. Remote debug (e.g. via gdbserver on linux) is a well established 
> technique.
> 
> Likewise for debugging stripped production binaries, my answer is "don't do 
> that". There are very rare cases where a bug goes away on a debug build, but 
> in those cases any instrumentation you add is also liable to make the bug go 
> away.

Again, having to debug qemu itself when all you want is some additional
information about the emulated hardware state while debugging a guest
seems like overkill to me. Imagine you are attached to qemu's gdbstub
and stopped the guest that has some issue using, say, the apic. And now
imagine you could simply issue e.g.

(gdb) monitor devstate apic.1
apicbase=...
tpr=...
icr={..., ... }
lvt={..., ... }
...

That would provide you a nice picture of the virtual hardware state
without the need to fire up and operate a second gdb just to obtain this
data. And manipulating some lvt register would only be a trivial
additional monitor command away.

Jan

-- 
Siemens AG, Corporate Technology, CT SE 2
Corporate Competence Center Embedded Linux

  reply	other threads:[~2009-07-15 10:41 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-14  8:20 [Qemu-devel] [PATCH] monitor: Add port write command Jan Kiszka
2009-07-14 15:06 ` Anthony Liguori
     [not found]   ` <m31voj47ht.fsf@neno.mitica>
2009-07-14 15:49     ` [Qemu-devel] " Anthony Liguori
2009-07-14 18:57   ` [Qemu-devel] " Blue Swirl
2009-07-14 19:30   ` Paul Brook
2009-07-14 20:03     ` Anthony Liguori
2009-07-14 20:37       ` [Qemu-devel] " Jan Kiszka
2009-07-15  7:34     ` [Qemu-devel] " Gleb Natapov
2009-07-15 10:14       ` Paul Brook
2009-07-15 10:40         ` Jan Kiszka [this message]
2009-07-15 11:14           ` Gleb Natapov
2009-07-15 11:12         ` Gleb Natapov
2009-07-15 12:36         ` Anthony Liguori
2009-07-15 14:38           ` Paul Brook

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=4A5DB231.7040109@siemens.com \
    --to=jan.kiszka@siemens.com \
    --cc=aliguori@us.ibm.com \
    --cc=gleb@redhat.com \
    --cc=paul@codesourcery.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).