qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Lucas Meneghel Rodrigues <lmr@redhat.com>
To: Avi Kivity <avi@redhat.com>
Cc: "qemu-ppc@nongnu.org" <qemu-ppc@nongnu.org>,
	Alexander Graf <agraf@suse.de>,
	qemu-devel Developers <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] [PATCH] PPC: Fix via-cuda memory registration
Date: Mon, 12 Sep 2011 17:20:11 -0300	[thread overview]
Message-ID: <4E6E697B.1060002@redhat.com> (raw)
In-Reply-To: <4E6E0ED6.9060507@redhat.com>

On 09/12/2011 10:53 AM, Avi Kivity wrote:
> On 09/12/2011 04:46 PM, Lucas Meneghel Rodrigues wrote:
>> Yes, that would also work, having different variants with different
>> qemu and qemu-img paths. Those binaries would have to be already
>> pre-built, but then we miss the ability autotest has of building the
>> binaries and prepare the environment. It'd be like:
>>
>> variant1:
>> qemu = /path/to/qemu1
>> qemu-img = /path/to/qemu-img1
>> extra_params = "--appropriate --extra --params2"
>>
>>
>> variant2:
>> qemu = /path/to/qemu2
>> qemu-img = /path/to/qemu-img2
>> extra_params = "--appropriate --extra --params2"
>>
>> Something like that. It's a feasible intermediate solution until I
>> finish work on supporting multiple userspaces.
>>
>
> Another option is, now that the binary name 'qemu' is available for
> general use, make it possible to invoke everything with just one binary:
>
> qemu -system -target mips ...
> qemu-system -target mips ...
> qemu-system-mips ...
>
> are all equivalent. autotest should easily be able to pass different
> -target based on the test being run.

Indeed, a good idea, although there are cases where we indeed want to 
run different user spaces (say, a windows reactivation test for 
example), so it's still worth to implement the multiple userspaces thing 
for autotest. But in the case we just want to run multiple targets, this 
would be very handy indeed.

  parent reply	other threads:[~2011-09-12 20:20 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-08 16:54 [Qemu-devel] [PATCH] PPC: Fix via-cuda memory registration Alexander Graf
2011-09-10 18:28 ` Andreas Färber
2011-09-11 10:41 ` Avi Kivity
2011-09-11 11:38   ` Alexander Graf
2011-09-12  9:07     ` Avi Kivity
2011-09-12 10:06       ` Alexander Graf
2011-09-12 13:46       ` Lucas Meneghel Rodrigues
2011-09-12 13:53         ` Avi Kivity
2011-09-12 20:12           ` Blue Swirl
2011-09-12 20:20           ` Lucas Meneghel Rodrigues [this message]
2011-09-12 20:33           ` Peter Maydell
2011-09-12 21:05           ` Anthony Liguori
2011-09-13 19:31             ` [Qemu-devel] [Qemu-ppc] " Blue Swirl
2011-09-13 20:16               ` Anthony Liguori
2011-09-14 19:48                 ` Blue Swirl
2011-09-14 20:01                   ` Avi Kivity
2011-09-14 20:03                     ` Alexander Graf

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=4E6E697B.1060002@redhat.com \
    --to=lmr@redhat.com \
    --cc=agraf@suse.de \
    --cc=avi@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@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).