[PATCH v1 1/5] fabrics: define admin sqsize min default, per spec

Verkamp, Daniel daniel.verkamp at intel.com
Mon Aug 15 08:55:48 PDT 2016


On Fri, 2016-08-12 at 17:04 -0700, Jay Freyensee wrote:
> Upon admin queue connect(), the rdma qp was being
> set based on NVMF_AQ_DEPTH.  However, the fabrics layer was
> using the sqsize field value set for I/O queues for the admin
> queue, which through the nvme layer and rdma layer off-whack:

Minor grammar nit: "through" -> "threw".

Otherwise looks OK to me (given the following patches that fix up the
non-AQ case are also applied).

Thanks,
-- Daniel


More information about the Linux-nvme mailing list