From: Stefan Hajnoczi <stefanha@gmail.com>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: Hannes Reinecke <hare@suse.de>,
Zachary Amsden <zamsden@redhat.com>,
qemu-devel@nongnu.org,
"Nicholas A. Bellinger" <nab@linux-iscsi.org>,
"Michael S. Tsirkin" <mst@redhat.com>
Subject: Re: [Qemu-devel] [RFC PATCH] implement vmware pvscsi device
Date: Fri, 15 Apr 2011 15:01:15 +0100 [thread overview]
Message-ID: <BANLkTim9ftsP5yJg0MZA41nymRSrPbsc6Q@mail.gmail.com> (raw)
In-Reply-To: <1302874976-22248-1-git-send-email-pbonzini@redhat.com>
On Fri, Apr 15, 2011 at 2:42 PM, Paolo Bonzini <pbonzini@redhat.com> wrote:
> Lightly tested with Linux guests; at least it can successfully partition
> and format a disk. scsi-generic also lightly tested.
>
> Doesn't do migration, doesn't do hotplug (the device would support that,
> but it is not 100% documented and the Linux driver in particular cannot
> initiate hot-unplug). I did it as quick one-day hack to study the SCSI
> subsystem and it is my first real foray into device model land, please
> be gentle. :)
>
> vmw_pvscsi.h is taken from Linux, so it doesn't fully respect coding
> standards. I think that's fair.
>
> Size is curiously close to the recently added sPAPR adapter:
>
> 911 2354 25553 hw/vmw_pvscsi.c
> 988 3177 29628 hw/spapr_vscsi.c
>
> Sounds like that's just the amount of code it takes to implement a SCSI
> HBA in QEMU. :)
Interesting, thanks for posting this. I've been playing with virtio
SCSI and it is still in the early stages. Nicholas A. Bellinger and I
have been wiring the in-kernel SCSI target up to KVM using vhost.
Feel free to take a peek at the work-in-progress:
http://repo.or.cz/w/qemu/stefanha.git/shortlog/refs/heads/virtio-scsi
http://git.kernel.org/?p=linux/kernel/git/nab/lio-core-2.6.git;a=shortlog;h=refs/heads/tcm_vhost
I think SCSI brings many benefits. Guests can deal with it better
than these alien vdX virtio-blk devices, which makes migration easier.
It becomes possible to attach many disks without burning through free
PCI slots. We don't need to update guests to add cache control,
discard, and other commands because they are part of SCSI. We can
pass through more exotic devices. The list goes on...
Stefan
next prev parent reply other threads:[~2011-04-15 14:01 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-15 13:42 [Qemu-devel] [RFC PATCH] implement vmware pvscsi device Paolo Bonzini
2011-04-15 14:01 ` Stefan Hajnoczi [this message]
2011-04-15 14:17 ` Paolo Bonzini
2011-04-15 14:28 ` Stefan Hajnoczi
2011-04-15 14:37 ` Paolo Bonzini
2011-04-15 15:04 ` Stefan Hajnoczi
2011-04-15 20:56 ` Paolo Bonzini
2011-04-18 14:05 ` Hannes Reinecke
2011-04-18 15:27 ` Stefan Hajnoczi
2011-04-18 16:09 ` Paolo Bonzini
2011-04-15 14:55 ` Hannes Reinecke
2011-04-15 14:59 ` Paolo Bonzini
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=BANLkTim9ftsP5yJg0MZA41nymRSrPbsc6Q@mail.gmail.com \
--to=stefanha@gmail.com \
--cc=hare@suse.de \
--cc=mst@redhat.com \
--cc=nab@linux-iscsi.org \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=zamsden@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).