The assert here is to trap a bad call to alloc_pd. Why is it being removed? Did you have code that hit it? Did you fix that code so it wouldn't hit the assertion? Or was the assertion incorrect? Basically, the assert has been in the code for years now - what suddenly makes it unnecessary?
-Fab From: [email protected] [mailto:[email protected]] On Behalf Of Tzachi Dar Sent: Tuesday, August 31, 2010 8:29 AM To: [email protected] Subject: [ofw] patch: [IBAL] remove unnecessary ASSERT Signed off by: Leonid Mlnx 5599 Index: B:/users/tzachid/projinf9/trunk/core/al/al_pd.c =================================================================== --- B:/users/tzachid/projinf9/trunk/core/al/al_pd.c (revision 5598) +++ B:/users/tzachid/projinf9/trunk/core/al/al_pd.c (revision 5599) @@ -137,7 +137,6 @@ break; default: - CL_ASSERT( h_pd->type == IB_PDT_ALIAS || h_pd->type == IB_PDT_NORMAL ); status = IB_INVALID_PARAMETER; }
_______________________________________________ ofw mailing list [email protected] http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ofw
