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')?

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

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to