[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 1/3] fdc: Drop deprecated floppy configuration
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH v2 1/3] fdc: Drop deprecated floppy configuration |
Date: |
Thu, 4 Mar 2021 10:52:16 +0000 |
User-agent: |
Mutt/2.0.5 (2021-01-21) |
On Thu, Mar 04, 2021 at 11:00:57AM +0100, Markus Armbruster wrote:
> Drop the crap deprecated in commit 4a27a638e7 "fdc: Deprecate
> configuring floppies with -global isa-fdc" (v5.1.0).
>
> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> ---
> docs/system/deprecated.rst | 26 --
> docs/system/removed-features.rst | 26 ++
> hw/block/fdc.c | 54 +--
> tests/qemu-iotests/172 | 31 +-
> tests/qemu-iotests/172.out | 562 +------------------------------
> 5 files changed, 30 insertions(+), 669 deletions(-)
>
> diff --git a/docs/system/deprecated.rst b/docs/system/deprecated.rst
> index 2fcac7861e..6a22bc07e2 100644
> --- a/docs/system/deprecated.rst
> +++ b/docs/system/deprecated.rst
> @@ -94,32 +94,6 @@ QEMU 5.1 has three options:
> to the user to load all the images they need.
> 3. ``-bios <file>`` - Tells QEMU to load the specified file as the firmwrae.
>
> -``Configuring floppies with ``-global``
> -'''''''''''''''''''''''''''''''''''''''
> -
> -Use ``-device floppy,...`` instead:
> -::
> -
> - -global isa-fdc.driveA=...
> - -global sysbus-fdc.driveA=...
> - -global SUNW,fdtwo.drive=...
> -
> -become
> -::
> -
> - -device floppy,unit=0,drive=...
> -
> -and
> -::
> -
> - -global isa-fdc.driveB=...
> - -global sysbus-fdc.driveB=...
> -
> -become
> -::
> -
> - -device floppy,unit=1,drive=...
> -
> ``-drive`` with bogus interface type
> ''''''''''''''''''''''''''''''''''''
>
> diff --git a/docs/system/removed-features.rst
> b/docs/system/removed-features.rst
> index c8481cafbd..b0e7350408 100644
> --- a/docs/system/removed-features.rst
> +++ b/docs/system/removed-features.rst
> @@ -38,6 +38,32 @@ or ``-display default,show-cursor=on`` instead.
> QEMU 5.0 introduced an alternative syntax to specify the size of the
> translation
> block cache, ``-accel tcg,tb-size=``.
>
> +``Configuring floppies with ``-global`` (removed in 6.0)
> +''''''''''''''''''''''''''''''''''''''''''''''''''''''''
> +
> +Use ``-device floppy,...`` instead:
> +::
> +
> + -global isa-fdc.driveA=...
> + -global sysbus-fdc.driveA=...
> + -global SUNW,fdtwo.drive=...
It looks like we're not actually removing the use of -global, rather
we're removing the driveA= and driveB= properties entirely, which
simply means there's nothing to be set via -global. The distinction
is important, because IIUC, it means that libvirt's use of these
properties via -device is also impacted eg
-device isa-fdc,driveA=drive-fdc0-0-0,bootindexA=1,driveB=drive-fdc0-0-1
will no longer work too ?
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- [PATCH v2 0/3] Drop deprecated floppy config & bogus -drive if=T, Markus Armbruster, 2021/03/04
- [PATCH v2 2/3] fdc: Inline fdctrl_connect_drives() into fdctrl_realize_common(), Markus Armbruster, 2021/03/04
- [PATCH v2 1/3] fdc: Drop deprecated floppy configuration, Markus Armbruster, 2021/03/04
- Re: [PATCH v2 1/3] fdc: Drop deprecated floppy configuration,
Daniel P . Berrangé <=
- Re: [PATCH v2 1/3] fdc: Drop deprecated floppy configuration, Markus Armbruster, 2021/03/04
- Re: [PATCH v2 1/3] fdc: Drop deprecated floppy configuration, Daniel P . Berrangé, 2021/03/04
- Re: [PATCH v2 1/3] fdc: Drop deprecated floppy configuration, Markus Armbruster, 2021/03/04
- Re: [PATCH v2 1/3] fdc: Drop deprecated floppy configuration, Peter Krempa, 2021/03/04
- Re: [PATCH v2 1/3] fdc: Drop deprecated floppy configuration, Daniel P . Berrangé, 2021/03/04
- Re: [PATCH v2 1/3] fdc: Drop deprecated floppy configuration, Markus Armbruster, 2021/03/05
- Re: [PATCH v2 1/3] fdc: Drop deprecated floppy configuration, Markus Armbruster, 2021/03/05
- Re: [PATCH v2 1/3] fdc: Drop deprecated floppy configuration, Daniel P . Berrangé, 2021/03/08
[PATCH v2 3/3] blockdev: Drop deprecated bogus -drive interface type, Markus Armbruster, 2021/03/04