All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paolo Bonzini <pbonzini@redhat.com>
To: Luiz Capitulino <lcapitulino@redhat.com>,
	Claudio Fontana <claudio.fontana@huawei.com>
Cc: Peter Maydell <peter.maydell@linaro.org>,
	Claudio Fontana <hw.claudio@gmail.com>,
	Gonglei <arei.gonglei@huawei.com>,
	qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [RFC v3] monitor: add memory search commands s, sp
Date: Tue, 21 Apr 2015 16:47:42 +0200	[thread overview]
Message-ID: <5536630E.9050808@redhat.com> (raw)
In-Reply-To: <20150421103518.45b69de3@redhat.com>



On 21/04/2015 16:35, Luiz Capitulino wrote:
> > gladly, I am definitely looking for Review, first I wanted to check if the idea itself was acceptable.
> 
> I honestly don't have an opinion on this, which means I'm willing
> to apply if there are no objections.
> 
> On the other hand, isn't it possible to just use existing commands?
> Like, why can't you just dump this memory to disk and search it
> with strings or another tool?

I don't think that can be a serious alternative.  The same would apply
to x and xp, but still, they are incredibly useful.  Now I must say I
don't know of a use case for s or sp, but I don't think what you
suggested is feasible.

Paolo

  reply	other threads:[~2015-04-21 14:48 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-16 10:31 [Qemu-devel] [RFC v3] monitor: add memory search commands s, sp hw.claudio
2015-03-16 10:38 ` Patchew Tool
2015-03-16 14:45 ` Eric Blake
2015-03-16 14:56   ` Claudio Fontana
2015-03-16 15:01   ` Paolo Bonzini
2015-03-16 15:54     ` Claudio Fontana
2015-03-16 15:58       ` Eric Blake
2015-03-16 16:29         ` Paolo Bonzini
2015-03-18 13:30 ` Claudio Fontana
2015-03-27 16:19   ` Claudio Fontana
2015-03-30 14:54     ` Paolo Bonzini
2015-04-20 22:01     ` Luiz Capitulino
2015-04-21 14:25       ` Claudio Fontana
2015-04-21 14:28         ` Paolo Bonzini
2015-04-21 15:12           ` Claudio Fontana
2015-04-21 14:35         ` Luiz Capitulino
2015-04-21 14:47           ` Paolo Bonzini [this message]
2015-04-23  8:56             ` Claudio Fontana

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=5536630E.9050808@redhat.com \
    --to=pbonzini@redhat.com \
    --cc=arei.gonglei@huawei.com \
    --cc=claudio.fontana@huawei.com \
    --cc=hw.claudio@gmail.com \
    --cc=lcapitulino@redhat.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.