[PATCH 1/3] iw_cm: free cm_id resources on the last deref
Steve Wise
swise at opengridcomputing.com
Wed Jul 20 06:51:03 PDT 2016
>
> > Remove the complicated logic to free the cm_id resources in iw_cm event
> > handlers vs when an application thread destroys the device. I'm not sure
> > why this code was written, but simply allowing the last deref to free
> > the memory is cleaner. It also prevents a deadlock when applications
> > try to destroy cm_id's in their cm event handler function.
>
> The description here is misleading. we can never destroy the cm_id
> inside the cm_id handler. Also, I don't think the deadlock was on cm_id
> removal but rather on the qp referenced by the cm_id. I think the change
> log can be improved.
>
I'll reword it.
> The patch looks fine to me.
More information about the Linux-nvme
mailing list