blk-mq merge flags
Jens Axboe
axboe at fb.com
Tue Mar 15 15:13:26 PDT 2016
On 03/15/2016 09:08 AM, Christoph Hellwig wrote:
> Jens, Keith -
>
> any comments? I think the lack of BLK_MQ_F_SG_MERGE for nvme
> is rather harmful at the moment, as I get much better request
> packing with it. I'd rather fix this in the block layer, but
> if that's not ok I can also send a pure NVMe patch.
We can default to merging, and have drivers opt out, if they need to.
NVMe is probably the only one that doesn't set the flag currently, so
the net effect of the two different approaches would be the same. We'll
have a slightly higher CPU overhead without a perf win for workloads
that don't get merging, but overall it's always a win if we can merge
commands. So a better default, I think, even for nvme.
--
Jens Axboe
More information about the Linux-nvme
mailing list