[PATCH V3] nvme-pci: add sgl support
Sagi Grimberg
sagi at grimberg.me
Mon Jul 31 04:18:02 PDT 2017
>> The fact that you introduce a sgl_threshold implies that prps
>> are better than sgls for small I/O sizes, are sgls better
>> than prps to large I/O size?
>
> It can't possibly be just a matter of the IO size. It must be a matter of
> physical continuity as well. Considering that each SG entry is twice the
> size of a PRP entry, PRP must be faster for a physically discontiguous
> buffer if only because the PRP list is half the size.
Fully agree
> I've no device to test with, but I'd guess SGL is only better if the
> number of SGE's happens to be less than half the number of the equivalent
> PRP entries. Something like this:
>
> if (iod->nents <= ((blk_rq_bytes(req) / dev->page_size) / 2))
> /* Use SGL */
> else
> /* Use PRP */
That's a device character really. I don't think we can/should come up
with heuristics that matches all the devices out there...
More information about the Linux-nvme
mailing list