On May 20, 2011, at 12:23 PM, Hefty, Sean wrote: >> We had a script which was looping through the devices returned from ibstat >> and attempted to register a SMI agent on an ethernet device. This caused >> a kernel panic. We have fixed our script. This patch prevents the panic. > > Do you remember where the panic was? > > I'm going to guess in ib_register_mad_agent() at this line: > > mad_agent_priv->agent.mr = ib_get_dma_mr(port_priv->qp_info[qpn].qp->pd, > IB_ACCESS_LOCAL_WRITE); > > in which case it may be safer to check for the NULL pointer. Can you confirm > if this was the spot?
Yes it was that spot. I did think of that today after I sent the patch. Do you think it would be safer just to check for both pointers QP0 and 1 (depending on the registration)? Ira > > - Sean -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html