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 <so...@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 > > > >