From: "Nicholas A. Bellinger" <nab@linux-iscsi.org>
To: Boaz Harrosh <bharrosh@panasas.com>
Cc: Kevin Wolf <kwolf@redhat.com>, Pete Wyckoff <pw@padd.com>,
Stefan Hajnoczi <stefanha@gmail.com>,
qemu-devel <qemu-devel@nongnu.org>,
Christoph Hellwig <hch@lst.de>,
FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>,
Gerd Hoffmann <kraxel@redhat.com>,
Paolo Bonzini <pbonzini@redhat.com>,
linux-iscsi-target-dev@googlegroups.com,
Hannes Reinecke <hare@suse.de>
Subject: [Qemu-devel] Re: [PATCH 0/5]: Add hw/scsi-bsg.c backend driver
Date: Wed, 24 Nov 2010 01:45:27 -0800 [thread overview]
Message-ID: <1290591927.30138.223.camel@haakon2.linux-iscsi.org> (raw)
In-Reply-To: <4CECDB57.8040406@panasas.com>
On Wed, 2010-11-24 at 11:31 +0200, Boaz Harrosh wrote:
> On 11/24/2010 10:44 AM, Nicholas A. Bellinger wrote:
> > From: Nicholas Bellinger <nab@linux-iscsi.org>
> >
> > <Quick resend with proper subject>
> >
> > Greetings QEMU SCSI / BLOCK folks,
> >
> > This series adds rebased support for the hw/scsi-bsg.c backstore for scsi-bus
> > compatible HBA emulation in QEMU-KVM on Linux hosts supporting the BSG driver
> > against current mainline qemu-kvm.git/master code.
> >
> > This patch is available from the scsi-bsg branch in the megasas/scsi friendly QEMU-KVM tree at:
> >
> > http://git.kernel.org/?p=virt/kvm/nab/qemu-kvm.git;a=shortlog;h=refs/heads/megasas-upstream-v1
> >
> > which includes Hannes's recent patch series posted recently against mainline v0.13.0 here:
> >
> > Re: [Qemu-devel] [PATCH] Megasas HBA emulation and SCSI update v.2
> > http://lists.gnu.org/archive/html/qemu-devel/2010-11/msg02018.html
> >
> > Note that in order for BSG I/O to function using vectored AIO a kernel patch to
> > linux/block/bsg.c:bsg_map_hdr() is currently required running on a bit paired
> > user/kernel enviroment. The kernel patch in question is here:
> >
> > http://marc.info/?l=linux-scsi&m=127649585524598&w=2
> >
>
> OK I've looked at this patch. I have sent such a patch my self and Pete Wyckoff
> and other people have sent it as well. I could not find in the URL above any
> response to your patch. Where there any?
Hi Boaz,
I found Tomo-san's follow here:
http://ns3.spinics.net/lists/linux-scsi/msg44266.html
>
> Thanks
> Boaz
>
> > The first three patches involve updating block code to support the BSG backstore for scsi-bsg.
> >
> > The forth patch adds the minor changes to hw/scsi-bus.c and hw/scsi-disk.c in order to
> > function with scsi-bsg.
> >
> > And the fifth patch adds the main hw/scsi-bsg.c logic necessary to run the new struct
> > SCSIDeviceInfo and for BSG AIO using struct iovec and paio_submit_len() to function.
> > The scsi-bsg.c code has been refactored to follow the modern SCSIDeviceInfo->alloc_req_iov()
> > and incoming struct iovec * memory for underlying SCSIRequest SGLs
> >
> > So far this has been tested with v2 8708EM2 HBA Emulation on 0.13.0 on a .37-rc2
> > KVM host with TCM_Loop FILEIO backstores into a Debian Lenny v2.6.26 x86_64
> > KVM guest with XFS filesystem mounts.
> >
> > Signed-off-by: Nicholas A. Bellinger <nab@linux-iscsi.org>
> >
> > Nicholas Bellinger (5):
> > block: Add top level BSG support
> > block: Add BSG qemu_open() in block/raw.c:raw_open()
> > block: Add paio_submit_len() non sector sized AIO
> > scsi: Add BSG checks into scsi-bus and scsi-disk
> > scsi-bsg: Add initial support for BSG based SCSIDeviceInfo
> >
> > Makefile.objs | 2 +-
> > block.c | 7 +-
> > block.h | 1 +
> > block/raw-posix-aio.h | 3 +
> > block/raw-posix.c | 65 +++++-
> > block/raw.c | 21 ++
> > block_int.h | 6 +
> > hw/scsi-bsg.c | 720 +++++++++++++++++++++++++++++++++++++++++++++++++
> > hw/scsi-bus.c | 3 +-
> > hw/scsi-disk.c | 5 +
> > posix-aio-compat.c | 29 ++
> > 11 files changed, 855 insertions(+), 7 deletions(-)
> > create mode 100644 hw/scsi-bsg.c
> >
>
prev parent reply other threads:[~2010-11-24 9:51 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-11-24 8:44 [Qemu-devel] [PATCH 0/5]: Add hw/scsi-bsg.c backend driver Nicholas A. Bellinger
2010-11-24 9:17 ` [Qemu-devel] " Kevin Wolf
2010-11-24 9:31 ` Boaz Harrosh
2010-11-24 9:45 ` Nicholas A. Bellinger [this message]
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=1290591927.30138.223.camel@haakon2.linux-iscsi.org \
--to=nab@linux-iscsi.org \
--cc=bharrosh@panasas.com \
--cc=fujita.tomonori@lab.ntt.co.jp \
--cc=hare@suse.de \
--cc=hch@lst.de \
--cc=kraxel@redhat.com \
--cc=kwolf@redhat.com \
--cc=linux-iscsi-target-dev@googlegroups.com \
--cc=pbonzini@redhat.com \
--cc=pw@padd.com \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@gmail.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).