On Mon, Jun 07, 2021 at 08:44:49PM -0300, Jason Gunthorpe wrote:
On Sun, Jun 06, 2021 at 08:21:51AM +0300, Leon Romanovsky wrote:
I think that better fix is: https://lore.kernel.org/linux-rdma/f72e27d5c82cd9beec7670141afa62786836c569....
Can you resend that with the bug report in the comment?
Done, https://lore.kernel.org/linux-rdma/e2eed941f912b2068e371fd37f43b8cf5082a0e6....
What is the issue? Some error path missed a restrack_del? which one?
All paths that called to rdma_restrack_new(), but didn't set "id_priv->cma_dev" yet. For example, cma_ib_new_conn_id() -> __rdma_create_id() -> do something -> exit on error -> miss call to rdma_restrack_del()
And I still dislike that patch for adding restrack_del and undoing the previous patch which was trying to get rid of it, but if it actually fixes a bug...
And I didn't like the previous patch :).
Thanks
Jason