On 03/02/2015 03:56, Wangting (Kathy) wrote: > Sorry, I find that the patch of "virtio-scsi: Optimize virtio_scsi_init_req" > can slove this problem.
Great that you could confirm that. :) > By the way, can you tell me the reason of the change about cdb and sense? cdb and sense are variable-size items. ANY_LAYOUT support changed VirtIOSCSIReq: instead of having a pointer to the request, it copies the request from guest memory into VirtIOSCSIReq. This is required because the request might not be contiguous in guest memory. And because the request and response headers (e.g. VirtIOSCSICmdReq and VirtIOSCSICmdResp) are included by value in VirtIOSCSIReq, the variable-sized fields have to be treated specially. Only one of them can remain in VirtIOSCSIReq, because you cannot have a flexible array member (e.g. "uint_8 sense[];") in the middle of a struct. cdb is always used, so it is chosen for the variable-sized part of VirtIOSCSIReq: cdb was simply moved from VirtIOSCSICmdReq to VirtIOSCSIReq. Instead, requests that complete with sense data are not a fast path. Hence sense is retrieved from the SCSIRequest, and virtio_scsi_command_complete copies it into the guest buffer via scsi_req_get_sense + qemu_iovec_from_buf. Paolo
