[PATCH 1/1] nvme: introduce nvme_transport_sgl_desc structure

Sagi Grimberg sagi at grimberg.me
Mon Jul 19 17:25:32 PDT 2021


>>>> Currently the tcp and fc transports use nvme_sgl_desc structure to
>>>> describe transport data blocks. Replace it with a new structure that
>>>> describes the Transport SGL Data Block according to the NVMe
>>>> specification.
>>>
>>> What is the point?
>>
>> I think its ok to be more aligned to the spec, and have the transport
>> sgl represented in the code. Don't see any harm in that.
>>
>>> Note that we should probably still clear the reserved space anyway.
>>
>> Definitely we should clear it.
> 
> don't you think it's cleared already ?
> 
> if not, lets clear also other rsvd fields while we're here.
> 
> Each driver call nvme_setup_cmd before mapping the data and this 
> function clears the SQE (if (!(req->rq_flags & RQF_DONTPREP)))..

Yea, I guess its fine to rely on nvme_setup_cmd for it.



More information about the Linux-nvme mailing list