From: Paolo Bonzini <pbonzini@redhat.com>
To: Markus Armbruster <armbru@redhat.com>, qemu-devel@nongnu.org
Cc: peter.maydell@linaro.org, kwolf@redhat.com, mreitz@redhat.com,
qemu-block@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v2 0/3] hw: Deprecate unwanted use -drive if=scsi
Date: Wed, 15 Feb 2017 13:39:54 +0100 [thread overview]
Message-ID: <575ab227-e44e-8d66-7b58-3e68e1df8491@redhat.com> (raw)
In-Reply-To: <1487161136-9018-1-git-send-email-armbru@redhat.com>
On 15/02/2017 13:18, Markus Armbruster wrote:
> Drives defined with if=scsi get connected to buses created with
> -device, unlike other interface types. Deprecate this usage.
>
> There is no good default SCSI HBA for PC machines. Deprecate if=scsi
> there entirely.
>
> Before this series, frontends for -drive if=scsi get created by SCSI
> HBAs. Frontends for other interface types get created by machine
> initialization.
>
> After this series, they get created by machine initialization, except
> for deprecated cases, which get created by new function
> scsi_legacy_handle_cmdline().
>
> Based on "[PATCH v3 0/8] More sensible default for -drive interface
> type".
>
> v2:
> * Trivially rebased
> * PATCH 2: Suppress warning for default drives
> * PATCH 3: Update user manual and qemu-iotests/051.pc.out
>
> Markus Armbruster (3):
> hw/scsi: Concentrate -drive if=scsi auto-create in one place
> hw: Deprecate -drive if=scsi with non-onboard HBAs
> hw/i386: Deprecate -drive if=scsi with PC machine types
>
> hw/arm/realview.c | 2 +-
> hw/arm/versatilepb.c | 2 +-
> hw/i386/pc.c | 6 +++++
> hw/scsi/esp-pci.c | 3 ---
> hw/scsi/esp.c | 6 -----
> hw/scsi/lsi53c895a.c | 10 +++++---
> hw/scsi/megasas.c | 4 ----
> hw/scsi/mptsas.c | 4 ----
> hw/scsi/scsi-bus.c | 56 +++++++++++++++++++++++++++++++++++++------
> hw/scsi/spapr_vscsi.c | 5 ++--
> hw/scsi/virtio-scsi.c | 8 -------
> include/hw/pci/pci.h | 2 ++
> include/hw/scsi/scsi.h | 3 ++-
> qemu-options.hx | 5 ----
> tests/qemu-iotests/051.pc.out | 12 ++++++----
> vl.c | 10 ++++++++
> 16 files changed, 88 insertions(+), 50 deletions(-)
>
Because of the ESC characters in 051.pc.out, please send the pull
request for this yourself.
Paolo
next prev parent reply other threads:[~2017-02-15 12:40 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-15 12:18 [Qemu-devel] [PATCH v2 0/3] hw: Deprecate unwanted use -drive if=scsi Markus Armbruster
2017-02-15 12:18 ` [Qemu-devel] [PATCH v2 1/3] hw/scsi: Concentrate -drive if=scsi auto-create in one place Markus Armbruster
2017-02-15 12:18 ` [Qemu-devel] [PATCH v2 2/3] hw: Deprecate -drive if=scsi with non-onboard HBAs Markus Armbruster
2017-02-15 12:18 ` [Qemu-devel] [PATCH v2 3/3] hw/i386: Deprecate -drive if=scsi with PC machine types Markus Armbruster
2017-02-15 12:30 ` [Qemu-devel] [PATCH v2 0/3] hw: Deprecate unwanted use -drive if=scsi no-reply
2017-02-15 12:39 ` Paolo Bonzini [this message]
2017-02-15 13:10 ` Markus Armbruster
2017-02-15 13:19 ` Paolo Bonzini
2017-02-20 12:07 ` Markus Armbruster
2017-02-20 12:53 ` Paolo Bonzini
2017-02-20 13:21 ` Markus Armbruster
2017-02-20 13:23 ` Paolo Bonzini
2017-02-21 12:34 ` 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=575ab227-e44e-8d66-7b58-3e68e1df8491@redhat.com \
--to=pbonzini@redhat.com \
--cc=armbru@redhat.com \
--cc=kwolf@redhat.com \
--cc=mreitz@redhat.com \
--cc=peter.maydell@linaro.org \
--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).