[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/5] Add "info capabilities" monitor command
From: |
Mark McLoughlin |
Subject: |
Re: [Qemu-devel] [PATCH 0/5] Add "info capabilities" monitor command |
Date: |
Fri, 14 Nov 2008 16:09:07 +0000 |
On Fri, 2008-11-14 at 02:50 +0000, Jamie Lokier wrote:
> > (qemu) info capabilities
> > [qemu]
> > accel=
> > arch=sparc
>
> I'm thinking that a qemu which supports multiple target architectures
> becomes increasingly feasible and likely, especially for system
> emulation. Therefore, [qemu] should have an "archs" property, and
> there should be separate "[arch-sparc]" headers, like this:
>
> [qemu]
> archs=sparc
>
> [arch]
> name=sparc
Yep, that sounds sensible.
> > [machine]
> > name=SS-5
> > max_cpus=1
> > nic_models=
>
> This would be prettier as [machine:SS5] or [machine SS5] imho.
I'd considered that but thought it would be ugly - now that you actually
type it out though, [machine foo] looks fine :-)
> > [machine]
> > name=Voyager
> > max_cpus=0
> > nic_models=
>
> Crumbs, zero CPUs?
Ah, vl.c has:
machine->max_cpus = machine->max_cpus ?: 1; /* Default to UP */
I guess it should do that for all machines, not just the selected one.
> > drive_cache=off,none,writethrough,writeback
>
> Do we need "off" and "none" here?
"none" - yes, I think so.
"off" - probably not, it's deprecated I guess.
Cheers,
Mark.
- [Qemu-devel] Re: [PATCH 4/5] Add new cpu_names() function, (continued)
[Qemu-devel] Re: [PATCH 1/5] Re-factor nic model listing, Anthony Liguori, 2008/11/13
Re: [Qemu-devel] [PATCH 0/5] Add "info capabilities" monitor command, Blue Swirl, 2008/11/13
[Qemu-devel] Re: [PATCH 0/5] Add "info capabilities" monitor command, Anthony Liguori, 2008/11/13
[Qemu-devel] Re: [PATCH 0/5] Add "info capabilities" monitor command, Mark McLoughlin, 2008/11/14
Re: [Qemu-devel] [PATCH 0/5] Add "info capabilities" monitor command, Daniel P. Berrange, 2008/11/14