Re: [RFC 0/6] vhost/scsi: Add T10 PI SGL passthrough support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Il 24/02/2014 06:32, Nicholas A. Bellinger ha scritto:
AFAICT up until this point the ->prio field has been unused, but
I'm certainly open to better ways of signaling (to vhost) that some
number of metadata iovs are to be expected..  Any thoughts..?

Hi nab,

the virtio-scsi side of the patch is nice and readable. As requested, here are my thoughts on how to add it to the standard.

The ->prio field is there to mimic SAM's command priority field (8.7 in my copy of the standard). I'd rather leave it alone; I understand this is the main reason why this patch is RFC.

Since we have a new feature bit, we can add a new element before the cdb. It could be a count of scatter/gather list like you did here, or it could be a byte count. Even better, we can add _two_ new fields, one for protection data out and one for protection data in.

Also, do we need an equivalent of the residual field, but for metadata?

Finally, any reason why you put the data sg elements before the metadata sg elements? I would have thought that processing is a bit simpler if either the metadata comes first, or you store in the command header the data count (either sg or byte). Because the virtio buffers form a linked list, it's a bit backwards to put metadata last, and store metadata count in the command header; it prevents you from processing the buffers online because you don't know when the metadata starts. Even though the Linux virtio layer always gives you a buffer count, this need not be the case in general.

Paolo
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux