[PATCH] nvme-rdma: Support 2 inline data SGEs for write commands.

Parav Pandit parav at mellanox.com
Wed Feb 22 11:52:41 PST 2017


> -----Original Message-----
> From: Sagi Grimberg [mailto:sagi at grimberg.me]
> Sent: Wednesday, February 22, 2017 1:39 PM
> To: Parav Pandit <parav at mellanox.com>; Christoph Hellwig <hch at lst.de>
> Cc: linux-nvme at lists.infradead.org; axboe at fb.com; keith.busch at intel.com
> Subject: Re: [PATCH] nvme-rdma: Support 2 inline data SGEs for write
> commands.
> 
> 
> > Hi Christoph, Sagi,
> 
> > Does this host side patch look fine? Its unrelated to target side
> implementation.
> 
> I'm still wandering if we want it dynamic according to the target ioccsz (say 1
> per PAGE_SIZE?)

I don't think we are consuming any more resources by increasing to 2SGEs because it's still aligns to hardware's minimum WQE size of 64B or 128B size.
Can we add that additional complexity in incremental manner unless 2 SGEs are less optimal compared to 1?



More information about the Linux-nvme mailing list