From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:37476)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <marcel.a@redhat.com>) id 1WCrZC-00077e-HL
	for qemu-devel@nongnu.org; Mon, 10 Feb 2014 09:10:38 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <marcel.a@redhat.com>) id 1WCrZ6-0002ln-AW
	for qemu-devel@nongnu.org; Mon, 10 Feb 2014 09:10:34 -0500
Received: from mx1.redhat.com ([209.132.183.28]:34253)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <marcel.a@redhat.com>) id 1WCrZ5-0002lb-Tm
	for qemu-devel@nongnu.org; Mon, 10 Feb 2014 09:10:28 -0500
Message-ID: <1392041419.23125.1.camel@localhost.localdomain>
From: Marcel Apfelbaum <marcel.a@redhat.com>
Date: Mon, 10 Feb 2014 16:10:19 +0200
In-Reply-To: <20140202134757.GC25713@redhat.com>
References: <20140131190322.GL29329@ERROL.INI.CMU.EDU>
	<52EBFBD5.5010701@redhat.com> <20140202134757.GC25713@redhat.com>
Content-Type: text/plain; charset="UTF-8"
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [RFC PATCH v2] Add option to switch off FDC
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: "Michael S. Tsirkin" <mst@redhat.com>
Cc: lersek@redhat.com, qemu-devel@nongnu.org, armbru@redhat.com, "Gabriel L. Somlo" <gsomlo@gmail.com>, pbonzini@redhat.com, imammedo@redhat.com

On Sun, 2014-02-02 at 15:47 +0200, Michael S. Tsirkin wrote:
> On Fri, Jan 31, 2014 at 12:39:01PM -0700, Eric Blake wrote:
> > On 01/31/2014 12:03 PM, Gabriel L. Somlo wrote:
> > > Allow guests to omit the configuration of a floppy disk controller.
> > > 
> > > Signed-off-by: Gabriel Somlo <somlo@cmu.edu>
> > > ---
> > > 
> > > New in this version:
> > > 
> > >   - "int have_fdc = 1;" instead of "int no_fdc = 0;"
> > > 
> > >   - use 8bit field for FDC0's _STA method
> > > 
> > 
> > > 
> > > Are we OK with the "mechanism" part, i.e. introducing "have_fdc", and
> > > programatically configuring the FDC in pc.c and patching the DSDT
> > > based on its value ?
> > > 
> > 
> > >  
> > > +DEF("no-fdc", 0, QEMU_OPTION_no_fdc,
> > > +    "-no-fdc         disable FDC\n", QEMU_ARCH_I386)
> > > +STEXI
> > > +@item -no-fdc
> > > +@findex -no-fdc
> > > +Disable FDC support.
> > > +ETEXI
> > 
> > This is not discoverable via the current state of QMP.  It also has the
> > annoyance of being a one-way switch with no way to revert to a default
> > state.  Can you make this instead use qemuOpts (maybe looking like '-fdc
> > yes' vs. '-fdc no')?
> 
> I think Marcel said he's working on an infrastructure to add global
> options as machine properties.
Yes, indeed, I am working on it. Maybe it worth waiting until it will be ready.

Thanks,
Marcel

> 
> > -- 
> > Eric Blake   eblake redhat com    +1-919-301-3266
> > Libvirt virtualization library http://libvirt.org
> > 
> 
>