[PATCH] nvme-rdma: Always signal fabrics private commands

Steve Wise swise at opengridcomputing.com
Wed Jul 13 07:28:07 PDT 2016


 
> 
> > Hi Steve,
> >
> > Sorry for the late response, I was on vacation.
> >
> > The patch looks fine,
> >
> > Reviewed-by: Sagi Grimberg <sagi at imberg.me>
> 
> Although, I think that we really should fix the
> double completion for unsignaled wqes in cxgb4...
> it might bite us elsewhere down the road.

Double completion?  When the QP exits RTS with pending unsignaled SQ WRs, cxgb4
doesn't know if those were actually completed by hardware, so they are completed
with FLUSH_ERR status.  I _could_ change cxgb4 to just eat those, but I'm a
little worried about breaking the iWARP Verbs semantics.  Perhaps I shouldn't
be.  It does seem to be causing lots of pain...






More information about the Linux-nvme mailing list