From: Jason Wang <jasowang@redhat.com>
To: qemu-devel@nongnu.org
Cc: kwolf@redhat.com, mst@redhat.com,
Jason Wang <jasowang@redhat.com>,
stefanha@redhat.com, cornelia.huck@de.ibm.com,
pbonzini@redhat.com
Subject: [Qemu-devel] [PATCH V2 0/5] Set correct blk feature for virtio 1.0
Date: Wed, 15 Jul 2015 13:29:56 +0800 [thread overview]
Message-ID: <1436938201-16766-1-git-send-email-jasowang@redhat.com> (raw)
Hi all:
This series tries to set feature correctly for virtio-blk when virtio
1.0 is supported. Two isssues were addressed according to the spec:
- scsi passthrough was not support in 1.0. This is done by, 1) disable
scsi by defautl for 2.4 machine type and fail the initialization
when both scsi and 1.0 were set.
- any layout must be set for transitional device. This is done by set
any layout when 1.0 is supported.
Please review
Changes from V1:
- Split virtio-net changes out of the series
- Enable VIRTIO_BLK_F_SCSI only when scsi is set
- Disable scsi by default and compat it for legacy machine types
- Let get_features() can fail and fail the initialization of
virito-blk when both 1.0 and scsi were supported.
Jason Wang (5):
virtio: get_features() can fail
virtio-blk: advertise scsi only when scsi is set
virtio-blk: disable scsi passthrough by default
virtio-blk: fail the init when both 1.0 and scsi is set
virtio-blk: set VIRTIO_F_ANY_LAYOUT when 1.0 is supported
hw/9pfs/virtio-9p-device.c | 3 ++-
hw/block/virtio-blk.c | 16 +++++++++++++---
hw/char/virtio-serial-bus.c | 3 ++-
hw/display/virtio-gpu.c | 3 ++-
hw/input/virtio-input.c | 3 ++-
hw/net/virtio-net.c | 3 ++-
hw/scsi/vhost-scsi.c | 3 ++-
hw/scsi/virtio-scsi.c | 3 ++-
hw/virtio/virtio-balloon.c | 3 ++-
hw/virtio/virtio-bus.c | 3 ++-
hw/virtio/virtio-rng.c | 2 +-
include/hw/compat.h | 6 +++++-
include/hw/virtio/virtio.h | 4 +++-
13 files changed, 40 insertions(+), 15 deletions(-)
--
2.1.4
next reply other threads:[~2015-07-15 5:30 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-15 5:29 Jason Wang [this message]
2015-07-15 5:29 ` [Qemu-devel] [PATCH V2 1/5] virtio: get_features() can fail Jason Wang
2015-07-15 9:01 ` Cornelia Huck
2015-07-15 9:11 ` Jason Wang
2015-07-15 11:36 ` Paolo Bonzini
2015-07-22 15:13 ` Markus Armbruster
2015-07-15 5:29 ` [Qemu-devel] [PATCH V2 2/5] virtio-blk: advertise scsi only when scsi is set Jason Wang
2015-07-15 7:57 ` Paolo Bonzini
2015-07-15 8:31 ` Jason Wang
2015-07-15 8:33 ` Paolo Bonzini
2015-07-15 5:29 ` [Qemu-devel] [PATCH V2 3/5] virtio-blk: disable scsi passthrough by default Jason Wang
2015-07-15 12:21 ` Michael S. Tsirkin
2015-07-15 12:47 ` Paolo Bonzini
2015-07-15 14:14 ` Michael S. Tsirkin
2015-07-15 14:18 ` Paolo Bonzini
2015-07-15 14:28 ` Michael S. Tsirkin
2015-07-15 14:45 ` Paolo Bonzini
2015-10-14 10:29 ` Cornelia Huck
2015-10-14 15:58 ` Paolo Bonzini
2015-07-15 5:30 ` [Qemu-devel] [PATCH V2 4/5] virtio-blk: fail the init when both 1.0 and scsi is set Jason Wang
2015-07-15 5:30 ` [Qemu-devel] [PATCH V2 5/5] virtio-blk: set VIRTIO_F_ANY_LAYOUT when 1.0 is supported Jason Wang
2015-07-15 9:11 ` [Qemu-devel] [PATCH V2 0/5] Set correct blk feature for virtio 1.0 Cornelia Huck
2015-07-15 9:39 ` Jason Wang
2015-07-15 11:38 ` Cornelia Huck
2015-07-15 11:52 ` Michael S. Tsirkin
2015-07-15 12:46 ` Cornelia Huck
2015-07-15 14:08 ` Michael S. Tsirkin
2015-07-15 14:20 ` Cornelia Huck
2015-07-15 11:49 ` Michael S. Tsirkin
2015-07-15 12:49 ` Cornelia Huck
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=1436938201-16766-1-git-send-email-jasowang@redhat.com \
--to=jasowang@redhat.com \
--cc=cornelia.huck@de.ibm.com \
--cc=kwolf@redhat.com \
--cc=mst@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@redhat.com \
/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).