[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 7/7] scsi-block: always use SG_IO
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 7/7] scsi-block: always use SG_IO |
Date: |
Mon, 23 May 2016 13:49:56 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 05/23/2016 06:54 AM, Paolo Bonzini wrote:
> Using pread/pwrite or io_submit has the advantage of eliminating the
> bounce buffer, but drops the SCSI status. This keeps the guest from
> seeing unit attention codes, as well as statuses such as RESERVATION
> CONFLICT. Because we know scsi-block operates on an SBC device we can
> still use the DMA helpers with SG_IO; just remember to patch the CDBs
> if the transfer is split into multiple segments.
>
> This means that scsi-block will always use the thread-pool unfortunately,
> instead of respecting aio=native.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> hw/scsi/scsi-disk.c | 214
> +++++++++++++++++++++++++++++++++++++++++++++++-----
> 1 file changed, 196 insertions(+), 18 deletions(-)
>
> diff --git a/hw/scsi/scsi-disk.c b/hw/scsi/scsi-disk.c
> index f823781..c0eefc0 100644
> --- a/hw/scsi/scsi-disk.c
> +++ b/hw/scsi/scsi-disk.c
> @@ -83,6 +83,7 @@ typedef struct SCSIDiskReq {
> struct iovec iov;
> QEMUIOVector qiov;
> BlockAcctCookie acct;
> + unsigned char *status;
> } SCSIDiskReq;
>
> #define SCSI_DISK_F_REMOVABLE 0
> @@ -190,6 +191,15 @@ static bool scsi_disk_req_check_error(SCSIDiskReq *r,
> int ret, bool acct_failed)
> return scsi_handle_rw_error(r, -ret, acct_failed);
> }
>
> + if (r->status && *r->status) {
> + if (acct_failed) {
> + SCSIDiskState *s = DO_UPCAST(SCSIDiskState, qdev, r->req.dev);
Another use of DO_UPCAST(); would container_of() be better?
> +
> +static BlockAIOCB *scsi_block_do_sgio(SCSIBlockReq *req,
> + int64_t offset, QEMUIOVector *iov,
> + int direction,
> + BlockCompletionFunc *cb, void *opaque)
> +{
> + sg_io_hdr_t *io_header = &req->io_header;
> + SCSIDiskReq *r = &req->req;
> + SCSIDiskState *s = DO_UPCAST(SCSIDiskState, qdev, r->req.dev);
> + int nb_logical_blocks;
> + uint64_t lba;
> + BlockAIOCB *aiocb;
> +
> + /* This is not supported yet. It can only happen if the guest does
> + * reads and writes that are not aligned to one logical sectors
s/one // ?
> + * _and_ cover multiple MemoryRegions.
> + */
> + assert(offset % s->qdev.blocksize == 0);
> + assert(iov->size % s->qdev.blocksize == 0);
Is asserting for something an ill-behaved guest can do the right behavior?
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-devel] [PATCH 5/7] scsi-disk: add need_fua_emulation to SCSIDiskClass, (continued)
- [Qemu-devel] [PATCH 4/7] scsi-disk: introduce dma_readv and dma_writev, Paolo Bonzini, 2016/05/23
- [Qemu-devel] [PATCH 1/7] dma-helpers: change interface to byte-based, Paolo Bonzini, 2016/05/23
- [Qemu-devel] [PATCH 6/7] scsi-disk: introduce scsi_disk_req_check_error, Paolo Bonzini, 2016/05/23
- [Qemu-devel] [PATCH 3/7] scsi-disk: introduce a common base class, Paolo Bonzini, 2016/05/23
- [Qemu-devel] [PATCH 7/7] scsi-block: always use SG_IO, Paolo Bonzini, 2016/05/23
- Re: [Qemu-devel] [PATCH 7/7] scsi-block: always use SG_IO,
Eric Blake <=
- Re: [Qemu-devel] [PATCH v2 0/7] dma-helpers, scsi-block: use SG_IO for all I/O on scsi-block, Mark Cave-Ayland, 2016/05/23
Re: [Qemu-devel] [Qemu-block] [PATCH v2 0/7] dma-helpers, scsi-block: use SG_IO for all I/O on scsi-block, Kevin Wolf, 2016/05/25