From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1L11FT-0002ht-A4
	for qemu-devel@nongnu.org; Fri, 14 Nov 2008 11:10:19 -0500
Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43)
	id 1L11FP-0002fe-Ik
	for qemu-devel@nongnu.org; Fri, 14 Nov 2008 11:10:18 -0500
Received: from [199.232.76.173] (port=33658 helo=monty-python.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1L11FP-0002fZ-DT
	for qemu-devel@nongnu.org; Fri, 14 Nov 2008 11:10:15 -0500
Received: from mx2.redhat.com ([66.187.237.31]:37104)
	by monty-python.gnu.org with esmtp (Exim 4.60)
	(envelope-from <markmc@redhat.com>) id 1L11FP-0007wo-GQ
	for qemu-devel@nongnu.org; Fri, 14 Nov 2008 11:10:15 -0500
Subject: Re: [Qemu-devel] [PATCH 0/5] Add "info capabilities" monitor command
From: Mark McLoughlin <markmc@redhat.com>
In-Reply-To: <20081114025052.GG2055@shareable.org>
References: <1226594763-2304-1-git-send-email-markmc@redhat.com>
	<f43fc5580811130949t61b922e8sf5b2ea51dc629a6e@mail.gmail.com>
	<20081114025052.GG2055@shareable.org>
Content-Type: text/plain
Date: Fri, 14 Nov 2008 16:09:07 +0000
Message-Id: <1226678947.9332.104.camel@blaa>
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Reply-To: Mark McLoughlin <markmc@redhat.com>, qemu-devel@nongnu.org
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/pipermail/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Jamie Lokier <jamie@shareable.org>
Cc: qemu-devel <qemu-devel@nongnu.org>

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.