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

> > 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.

Paul

  reply	other threads:[~2009-07-15 10:14 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 [this message]
2009-07-15 10:40         ` Jan Kiszka
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=200907151114.21482.paul@codesourcery.com \
    --to=paul@codesourcery.com \
    --cc=aliguori@us.ibm.com \
    --cc=gleb@redhat.com \
    --cc=jan.kiszka@siemens.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).