[ofw] patch: [IBAL] remove unnecessary ASSERT

[Fab Tillier]

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: ofw-bounces at lists.openfabrics.org [mailto:ofw-bounces at lists.openfabrics.org] On Behalf Of Tzachi Dar
Sent: Tuesday, August 31, 2010 8:29 AM
To: ofw at lists.openfabrics.org
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;
                }

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openfabrics.org/pipermail/ofw/attachments/20100831/97d074e7/attachment.html>