From: Paolo Bonzini <pbonzini@redhat.com>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 0/4] ISCSI: Passthough updates
Date: Sat, 26 May 2012 09:37:44 +0200 [thread overview]
Message-ID: <jpq188$7jd$2@dough.gmane.org> (raw)
In-Reply-To: <1338008201-29078-1-git-send-email-ronniesahlberg@gmail.com>
Il 26/05/2012 06:56, Ronnie Sahlberg ha scritto:
> Paolo, list
>
> Please find a set of patches for iscsi.
>
> 1, The recent change to the eventsystem in iscsi is racy. If the
> async connect of the socket takes longer than almost instant, the
> shortcircuit trying to write directly to the socket may try to do so
> before the socket is established. So we hcange the logic here and
> ONLY try the shortcircuit avoiding setting up POLLOUT if we have been
> fully logged in.
>
> This avoids the race where we might crash if the socket is not yet
> connected.
>
> 2, Swithc to using READCAPACITY10 for MMC devices. Only SBC devices
> support READCAPACITY16 so far.
>
> 3, Switch to use READ10 for MMC devices instead of READ16. With this
> patch ISCSI and MMC devices now work correctly.
>
> 4, If the device is a SMC device, then force the device to be sg. We
> dont support emulation as far as i know for SMV devices, so these
> device types would only work for sg or iscsi passthrough anyway. The
> reason we need to forcer sg is that since we can not READ from a SMC
> device, we need to shortcircuit find_image_format() before it tries
> to do the bdrv_pread() trying to guess what format the image is.
>
> With this patch I have verified I can access and manage a media
> changer via iscsi-passthrough.
Applied 1-2 to scsi-next branch for 1.1, 3 for 1.2.
Paolo
prev parent reply other threads:[~2012-05-26 7:40 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-26 4:56 [Qemu-devel] [PATCH 0/4] ISCSI: Passthough updates Ronnie Sahlberg
2012-05-26 4:56 ` [Qemu-devel] [PATCH 1/4] ISCSI: Only call READCAPACITY16 for SBC devices Ronnie Sahlberg
2012-05-26 4:56 ` [Qemu-devel] [PATCH 2/4] ISCSI: Use READCAPACITY10 for MMC devices Ronnie Sahlberg
2012-05-26 4:56 ` [Qemu-devel] [PATCH 3/4] ISCSI: Only use READ16 for SBC devices. Use READ10 for other device types such as MMC Ronnie Sahlberg
2012-05-26 4:56 ` [Qemu-devel] [PATCH 4/4] ISCSI: If the device we open is a SMC device, then force the use of sg. We dont have any medium changer emulation so only passthrough via real sg or scsi-generic via iscsi would work anyway Ronnie Sahlberg
2012-05-26 7:36 ` Paolo Bonzini
2012-05-27 13:12 ` Andreas Färber
2012-05-28 6:48 ` Paolo Bonzini
2012-05-28 11:55 ` ronnie sahlberg
2012-05-28 12:05 ` Paolo Bonzini
2012-05-26 7:37 ` Paolo Bonzini [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='jpq188$7jd$2@dough.gmane.org' \
--to=pbonzini@redhat.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.