<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">
<HTML><HEAD>
<META http-equiv=Content-Type content="text/html; charset=us-ascii">
<META content="MSHTML 6.00.6000.16705" name=GENERATOR></HEAD>
<BODY>
<DIV><SPAN class=260503515-13102008><FONT face=Arial color=#0000ff
size=2>Hello,</FONT></SPAN></DIV>
<DIV><SPAN class=260503515-13102008><FONT face=Arial color=#0000ff size=2>
A few small questions.</FONT></SPAN></DIV>
<DIV><SPAN class=260503515-13102008><FONT face=Arial color=#0000ff
size=2></FONT></SPAN> </DIV>
<DIV><SPAN class=260503515-13102008><FONT face=Arial color=#0000ff size=2>Under
what conditions is the p_parent_ext == NULL?</FONT></SPAN></DIV>
<DIV><SPAN class=260503515-13102008><FONT face=Arial color=#0000ff
size=2></FONT></SPAN> </DIV>
<DIV><SPAN class=260503515-13102008><FONT face=Arial color=#0000ff size=2>Why is
it OK in the free build to proceed with a NULL p_parent_ext?</FONT></SPAN></DIV>
<DIV><SPAN class=260503515-13102008><FONT face=Arial color=#0000ff
size=2></FONT></SPAN> </DIV>
<DIV><SPAN class=260503515-13102008><FONT face=Arial color=#0000ff size=2>Should
the code read something like</FONT></SPAN></DIV>
<DIV><SPAN class=260503515-13102008><FONT face=Arial color=#0000ff
size=2></FONT></SPAN> </DIV>
<DIV><SPAN class=260503515-13102008><FONT face=Arial color=#0000ff size=2>if (
!p_bfi->p_ext_p_parent_ext )</FONT></SPAN></DIV>
<DIV><SPAN class=260503515-13102008> <FONT face=Arial
color=#0000ff size=2>return Some-Useful_Return-Code;</FONT></SPAN></DIV>
<DIV><SPAN class=260503515-13102008><FONT face=Arial color=#0000ff
size=2></FONT></SPAN> </DIV>
<DIV><SPAN class=260503515-13102008><FONT face=Arial color=#0000ff
size=2>thanks,</FONT></SPAN></DIV>
<DIV><SPAN class=260503515-13102008><FONT face=Arial color=#0000ff
size=2></FONT></SPAN> </DIV>
<DIV><SPAN class=260503515-13102008><FONT face=Arial color=#0000ff
size=2>stan.</FONT></SPAN></DIV><BR>
<DIV class=OutlookMessageHeader lang=en-us dir=ltr align=left>
<HR tabIndex=-1>
<FONT face=Tahoma size=2><B>From:</B> ofw-bounces@lists.openfabrics.org
[mailto:ofw-bounces@lists.openfabrics.org] <B>On Behalf Of </B>Tzachi
Dar<BR><B>Sent:</B> Sunday, October 12, 2008 6:10 AM<BR><B>To:</B>
ofw@lists.openfabrics.org<BR><B>Subject:</B> [ofw] Patch: Avoid a crash in an
ASSERT during a WHQL test.<BR></FONT><BR></DIV>
<DIV></DIV>
<DIV><FONT face=Arial size=2>Index:
Q:/projinf4/trunk/core/bus/kernel/bus_port_mgr.c<BR>===================================================================<BR>---
Q:/projinf4/trunk/core/bus/kernel/bus_port_mgr.c (revision 3307)<BR>+++
Q:/projinf4/trunk/core/bus/kernel/bus_port_mgr.c (revision 3308)<BR>@@
-1219,7 +1219,9 @@<BR> ("%s NULL h_ca?
p_ext %p\n", p_bfi->whoami, p_ext )
);<BR> return;<BR> }<BR>- CL_ASSERT(p_bfi ==
p_ext->p_parent_ext->bus_filter);<BR>+<BR>+ // Don't crash if
p_ext->p_parent_ext is NULL<BR>+ CL_ASSERT((p_ext->p_parent_ext ==
NULL) || p_bfi ==
p_ext->p_parent_ext->bus_filter);<BR> <BR> cl_mutex_acquire(
&gp_port_mgr->pdo_mutex );<BR> CL_ASSERT( p_ext->h_ca
);<BR></FONT></DIV></BODY></HTML>