qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
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

             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).