> @@ -572,6 +573,8 @@ static int nes_dealloc_fmr(struct ib_fmr *ibfmr)
 >      nesmr->ibmw.rkey = ibfmr->rkey;
 >      nesmr->ibmw.uobject = NULL;
 >  
 > +    rc = nes_dealloc_mw(&nesmr->ibmw);
 > +
 >      if (nesfmr->nesmr.pbls_used != 0) {
 >              spin_lock_irqsave(&nesadapter->pbl_lock, flags);
 >              if (nesfmr->nesmr.pbl_4k) {

Can this be right?  nes_dealloc_mw() fails, so the HW still thinks it
owns the resources, and then the function just continues and releases
the PBLs before returning?

[And same issue seems to be there for the change to nes_dereg_mr]

 - R.
_______________________________________________
general mailing list
[email protected]
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to