Ralph, Can you provide the patch to OFED 1.3.1 today so we will be able to include it in RC1?
Thanks Tziporet -----Original Message----- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Ralph Campbell Sent: Wednesday, May 21, 2008 1:28 AM To: Roland Dreier Cc: Dave Olson; [email protected] Subject: Re: [ofa-general] Re: PATCH 1/1 - fix kernel crash in mad.c whenIB_MAD_RESULT_(SUCCESS|CONSUMED) returned On Tue, 2008-05-20 at 15:17 -0700, Roland Dreier wrote: > > case IB_MAD_RESULT_SUCCESS | IB_MAD_RESULT_CONSUMED: > > kmem_cache_free(ib_mad_cache, mad_priv); > > - break; > > + kfree(local); > > + goto out; > > Seems you need to set ret = 1 here? Otherwise I think ib_post_send_mad > will continue handling the send even though the packet was supposedly > consumed. I agree. > Also as a side note, I think handle_outgoing_dr_smp() would be clearer > if rather than having > > out: > return ret; > > and then doing stuff like > > ret = -EINVAL; > goto out; > > the code just did "return -EINVAL;" > > Maybe I'll do that cleanup for 2.6.27. I also agree but I remember at one point we got pushback from one of the mainline kernel developers who really wanted to see only one return point in the code even if it meant more gotos. I don't remember who though. _______________________________________________ general mailing list [email protected] http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general _______________________________________________ general mailing list [email protected] http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general
