I need to top post because comments are unrelated to past discussion.
rdma_ah_retrieve_dmac() can never fail for RoCE as its returning pointer from
Provider driver doesn't need to check for null pointer as ib/core would never
call provider if it's not RoCE provider.
So this memory leak only exist in theory.
When its null, driver should WARN_ON/BUG_ON in extreme case, but that's not
I have patch is progress under internal review that does nice small cleanup in
many provider drivers that eliminates the check completely.
Waiting for Moni to finish the review.
> -----Original Message-----
> From: linux-rdma-ow...@vger.kernel.org [mailto:linux-rdma-
> ow...@vger.kernel.org] On Behalf Of Johannes Thumshirn
> Sent: Tuesday, August 08, 2017 7:59 AM
> To: Colin Ian King <colin.k...@canonical.com>
> Cc: Lijun Ou <ouli...@huawei.com>; Wei Hu <xavier.hu...@huawei.com>;
> Doug Ledford <dledf...@redhat.com>; Sean Hefty <sean.he...@intel.com>;
> Hal Rosenstock <hal.rosenst...@gmail.com>; linux-r...@vger.kernel.org;
> kernel-janit...@vger.kernel.org; firstname.lastname@example.org
> Subject: Re: [PATCH] IB/core: fix memory leak on ah on error return path
> On Tue, Aug 08, 2017 at 11:28:16AM +0100, Colin Ian King wrote:
> > I was using the same subject start as the patch that introduced the
> > memory leak and touched the same portion of code. I can resend if necessary.
> I think having the hns prefix makes it clearer, as the patch doesn't touch IB
> code but hns code. The reference to the patch which introduced the leak is
> byu the fixes line.
> Johannes Thumshirn Storage
> jthumsh...@suse.de +49 911 74053 689
> SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
> GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg)
> Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850
> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the
> of a message to majord...@vger.kernel.org More majordomo info at