<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<p>Please note that the patch is not correctly formatted. I will fix
that before submitting.<br>
</p>
<br>
<div class="moz-cite-prefix">On 04/24/2018 01:25 PM, Dag Moxnes
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:373de57c-4cab-0d04-0021-57b566cefe0d@oracle.com">I was
going to suggest the following correction:
<br>
<br>
<br>
If all agree that this is the correct way of doing it, I can go
ahead and an post it.
<br>
<br>
Regards,
<br>
<br>
-Dag
<br>
<br>
<br>
On 04/24/2018 01:10 PM, Håkon Bugge wrote:
<br>
<blockquote type="cite">
<br>
<blockquote type="cite">On 24 Apr 2018, at 05:27,
<a class="moz-txt-link-abbreviated" href="mailto:santosh.shilimkar@oracle.com">santosh.shilimkar@oracle.com</a> wrote:
<br>
<br>
On 4/23/18 6:39 PM, Zhu Yanjun wrote:
<br>
<blockquote type="cite">This reverts commit
3b12f73a5c2977153f28a224392fd4729b50d1dc.
<br>
After long time discussion and investigations, it seems that
there
<br>
is no mem leak. So this patch is reverted.
<br>
Signed-off-by: Zhu Yanjun <a class="moz-txt-link-rfc2396E" href="mailto:yanjun.zhu@oracle.com"><yanjun.zhu@oracle.com></a>
<br>
---
<br>
</blockquote>
Well your fix was not for any leaks but just proper labels for
<br>
graceful exits. I don't know which long time discussion
<br>
you are referring but there is no need to revert this change
<br>
unless you see any issue with your change.
<br>
</blockquote>
As spotted by Dax Moxnes, the patch misses to call
rds_ib_dev_put() when exiting normally, only on err exit.
<br>
<br>
<br>
Thxs, Håkon
<br>
<br>
<blockquote type="cite">Regards,
<br>
Santosh
<br>
--
<br>
To unsubscribe from this list: send the line "unsubscribe
linux-rdma" in
<br>
the body of a message to <a class="moz-txt-link-abbreviated" href="mailto:majordomo@vger.kernel.org">majordomo@vger.kernel.org</a>
<br>
More majordomo info at
<a class="moz-txt-link-freetext" href="http://vger.kernel.org/majordomo-info.html">http://vger.kernel.org/majordomo-info.html</a>
<br>
</blockquote>
</blockquote>
<br>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
rds-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:rds-devel@oss.oracle.com">rds-devel@oss.oracle.com</a>
<a class="moz-txt-link-freetext" href="https://oss.oracle.com/mailman/listinfo/rds-devel">https://oss.oracle.com/mailman/listinfo/rds-devel</a></pre>
</blockquote>
<br>
</body>
</html>