qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Nicholas A. Bellinger" <nab@linux-iscsi.org>
To: Hannes Reinecke <hare@suse.de>, Christoph Hellwig <hch@lst.de>,
	Kevin Wolf <kwolf@redhat.com>, Gerd Hoffmann <kraxel@redhat.com>
Cc: Chris Wright <chrisw@redhat.com>,
	Jan Kiszka <jan.kiszka@siemens.com>,
	qemu-devel <qemu-devel@nongnu.org>,
	kvm-devel <kvm@vger.kernel.org>,
	Paul Brook <paul@codesourcery.com>
Subject: [Qemu-devel] MegaSAS 8708EM2 qemu-kvm.git tree updated to v0.12.5
Date: Sun, 15 Aug 2010 02:23:12 -0700	[thread overview]
Message-ID: <1281864192.4930.265.camel@haakon2.linux-iscsi.org> (raw)

Greetings Hannes, hch and Co,

The lastest code from upstream qemu-kvm.git v0.12.5 has been merged into
the megasas HBA emulation friendly qemu-kvm.git/master and scsi-bsg
branches at:

http://git.kernel.org/?p=virt/kvm/nab/qemu-kvm.git;a=summary

The merge commitdiffs for master and scsi-bsg can be found here:

http://git.kernel.org/?p=virt/kvm/nab/qemu-kvm.git;a=commitdiff;h=331578e7e362b33c965d469ea4577956dd431bbc
http://git.kernel.org/?p=virt/kvm/nab/qemu-kvm.git;a=commitdiff;h=2eebcfd04adda1fee641a776c9a85dda95c43b43

The megasas HBA emulation has been given a quick test with scsi-generic
and scsi-bsg backstores into TCM_Loop FILEIO LUNs with x86_64 v2.6.26
guests on a x86_64 v2.6.35 host.  So far things appear to be functioning
as expected with the megasas SGL passthrough logic and v0.12.5 upstream
qemu-kvm code.

There was also some new upstream code in hw/scsi-disk.c which does not
seem to be required with the SGL passthrough logic, and that I ended up
dropping for the v0.12.5 merge.  The code that that was dropped during
the merge starts with scsi_command_complete() at:

http://git.kernel.org/?p=virt/kvm/qemu-kvm.git;a=blob;f=hw/scsi-disk.c;hb=HEAD#l101

and includes everything down to scsi_write_data().  Using scsi-disk was
also given a quick test and appears to be functioning as expected using
passthrough SGL logic and userspace QEMU SCSI CDB emulation.

Note there does appear to be some breakage with the SGL passthrough and
recent upstream changes with hw/lsi53c895a.c to use a local dma_buf
pointer and to get rid of LSIState->select_dev.  While I had verified
that the SGL passthrough code was working with lsi53c895a on v0.12.4
with scsi-generic+scsi-bsg backstores, this is now segfaulting for me
after the latest upstream merge.  I will need to have another look at
this, but if someone who has more knowledge with hw/lsi53c895a.c could
help out, it would be much apperciated.  ;)

Comments are welcome!

Best,

--nab

                 reply	other threads:[~2010-08-15  9:26 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

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=1281864192.4930.265.camel@haakon2.linux-iscsi.org \
    --to=nab@linux-iscsi.org \
    --cc=chrisw@redhat.com \
    --cc=hare@suse.de \
    --cc=hch@lst.de \
    --cc=jan.kiszka@siemens.com \
    --cc=kraxel@redhat.com \
    --cc=kvm@vger.kernel.org \
    --cc=kwolf@redhat.com \
    --cc=paul@codesourcery.com \
    --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).