From: Hannes Reinecke <hare@suse.de>
To: Alexander Graf <agraf@suse.de>
Cc: "qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
"Nicholas A. Bellinger" <nab@linux-iscsi.org>
Subject: Re: [Qemu-devel] [PATCH 0/7] scsi and megasas update
Date: Fri, 05 Nov 2010 08:21:52 +0100 [thread overview]
Message-ID: <4CD3B090.9060306@suse.de> (raw)
In-Reply-To: <CEF8133A-7A5B-4F84-A4EB-E6DB6E524D8C@suse.de>
On 11/04/2010 09:49 PM, Alexander Graf wrote:
>
> On 15.06.2010, at 11:15, Hannes Reinecke wrote:
>
>> Hi all,
>>
>> I've run an I/O exerciser against the megasas emulation and
>> found quite some issues, both with the HBA and the SCSI
>> emulation.
>>
>> - SG_IO returns -EDOM if the internal queue is exhausted;
>> we should be mapping this to QUEUE_FULL.
>> - Codingstyle fixes for megasas as suggested by blueswirl
>> - We should be using the status codes as defined by SAM,
>> not the linux ones (which are shifted by one)
>> - Each driver has it's own mechanism of storing the
>> sense data, so we should be implementing an accessor
>> scsi_req_sense() to retrieve it.
>> - And, of course, a rather largish megasas update
>>
>> Patches are relative to nab's qemu-kvm git tree at
>> git://git.kernel.org/pub/scm/virt/kvm/nab/qemu-kvm.git
>>
>> Nab, I think this patchset should make Windows happy.
>
> Got an updated version? :)
Sadly, no. I've got a git repository setup at kernel.org,
but so far failed to do anything with it.
Real life intercepting as per normal.
But on the good side I have the patchset cleaned up
to avoid the contested SCSI update.
IE the megasas emulation is now using the 'standard'
SCSI callback mechanism.
I see to have the patchset pushed to kernel.org
and will send a pointer then.
Cheers,
Hannes
--
Dr. Hannes Reinecke zSeries & Storage
hare@suse.de +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: Markus Rex, HRB 16746 (AG Nürnberg)
next prev parent reply other threads:[~2010-11-05 7:20 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-15 15:15 [Qemu-devel] [PATCH 0/7] scsi and megasas update Hannes Reinecke
2010-11-04 20:49 ` Alexander Graf
2010-11-05 7:21 ` Hannes Reinecke [this message]
2010-11-06 10:44 ` Stefan Hajnoczi
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=4CD3B090.9060306@suse.de \
--to=hare@suse.de \
--cc=agraf@suse.de \
--cc=nab@linux-iscsi.org \
--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).