Re: [PATCH] IB/core: Make ib_dealloc_pd return void

[Sagi Grimberg <sagig-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org>]

On 8/11/2015 8:57 AM, Jason Gunthorpe wrote:
> On Thu, Aug 06, 2015 at 08:31:58PM +0300, Sagi Grimberg wrote:
>
>> This looks generally good. Would it make sense to go the extra mile
>> here and just fixup ocrdma (the only driver that seems to be able to
>> fail) to WARN_ON() instead of propagating an error and make it go
>> away from the core?
>
> I've remarked to the ocrdma folks about fixing this in the past, I'm
> not sure where they are on that..

I was simply suggesting to move the (specific to ocrdma) WARN_ON() from
the core to ocrdma. The ocrdma folks can take care of that if they feel
like it.

>
> There are also a few other dealloc APIs (mr, etc) that need a similar
> treatment. I was planning to address the ULP side and driver side in
> two steps to keep things simpler.

mr deallocation can only fail if there are memory windows on it. I
don't know if I would bother changing it. Perhaps ULPs that don't use
windows (all at the moment) will knowingly ignore it and ULPs that will
use windows will be required to check it.
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html