From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Markus Armbruster <armbru@redhat.com>
Cc: kwolf@redhat.com, qemu-block@nongnu.org, libvir-list@redhat.com,
qemu-devel@nongnu.org, mreitz@redhat.com, jsnow@redhat.com
Subject: Re: [PATCH v2 1/3] fdc: Drop deprecated floppy configuration
Date: Thu, 4 Mar 2021 14:34:14 +0000 [thread overview]
Message-ID: <YEDv5l/tGdnDtiHh@redhat.com> (raw)
In-Reply-To: <87v9a7dmfk.fsf@dusky.pond.sub.org>
On Thu, Mar 04, 2021 at 03:26:55PM +0100, Markus Armbruster wrote:
> Daniel P. Berrangé <berrange@redhat.com> writes:
>
> > 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 ?
>
> Correct.
>
> This was deprecated in commit 4a27a638e7 "fdc: Deprecate configuring
> floppies with -global isa-fdc" (v5.1.0). Since then, its use triggers a
> warning:
>
> $ qemu-system-x86_64 -nodefaults -M q35 -display none -drive if=none,id=drive-fdc0-0-0 -device isa-fdc,driveA=drive-fdc0-0-0,bootindexA=1
> qemu-system-x86_64: -device isa-fdc,driveA=drive-fdc0-0-0,bootindexA=1: warning: warning: property isa-fdc.driveA is deprecated
> Use -device floppy,unit=0,drive=... instead.
>
> Note the -M q35. Needed because the default machine type has an onboard
> isa-fdc, which cannot be configured this way.
>
> Sadly, the commit's update of docs/system/deprecated.rst neglects to
> cover this use. Looks the series overtaxed my capacity to juggle
> details; my apologies.
>
> Is libvirt still using these properties?
Unfortunately yes, but it seems like it ought to be fairly easy to
change the syntax. Just need to figure out what the right way to
detect the availability of the new syntax is. Presumably just look
for existance of the 'floppy' device type ?
Can you confirm that switching from -global to the new -device floppy
does /not/ have any live migration impact ?
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 :|
next prev parent reply other threads:[~2021-03-04 14:35 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-04 10:00 [PATCH v2 0/3] Drop deprecated floppy config & bogus -drive if=T Markus Armbruster
2021-03-04 10:00 ` [PATCH v2 1/3] fdc: Drop deprecated floppy configuration Markus Armbruster
2021-03-04 10:52 ` Daniel P. Berrangé
2021-03-04 14:26 ` Markus Armbruster
2021-03-04 14:34 ` Daniel P. Berrangé [this message]
2021-03-04 16:23 ` Markus Armbruster
2021-03-04 16:52 ` Peter Krempa
2021-03-04 16:54 ` Daniel P. Berrangé
2021-03-05 6:41 ` Markus Armbruster
2021-03-05 8:06 ` Markus Armbruster
2021-03-08 11:23 ` Daniel P. Berrangé
2021-03-04 10:00 ` [PATCH v2 2/3] fdc: Inline fdctrl_connect_drives() into fdctrl_realize_common() Markus Armbruster
2021-03-04 10:00 ` [PATCH v2 3/3] blockdev: Drop deprecated bogus -drive interface type Markus Armbruster
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=YEDv5l/tGdnDtiHh@redhat.com \
--to=berrange@redhat.com \
--cc=armbru@redhat.com \
--cc=jsnow@redhat.com \
--cc=kwolf@redhat.com \
--cc=libvir-list@redhat.com \
--cc=mreitz@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).