> > cancel_mads(mad_agent_priv); > > port_priv = mad_agent_priv->qp_info->port_priv; > >- cancel_delayed_work(&mad_agent_priv->timed_work); > >+ del_timer_sync(&mad_agent_priv->timeout_timer); > >+ cancel_work_sync(&mad_agent_priv->timeout_work); > > I had to check if there was a race between del_timer_sync() and the worker > thread, but the call to cancel_mads() looks like it prevents any issues.
Yeah, I think it's OK, and in any case any race is already there I think (since cancel_delayed_work is essentially equivalent to del_timer_sync) Thanks. _______________________________________________ general mailing list general@lists.openfabrics.org http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general
