<!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.2900.2963" name=GENERATOR></HEAD>
<BODY>
<DIV><SPAN class=300500910-06092006><FONT face=Arial size=2>Fab
</FONT></SPAN></DIV>
<DIV><SPAN class=300500910-06092006><FONT face=Arial
size=2></FONT></SPAN> </DIV>
<DIV><SPAN class=300500910-06092006><FONT face=Arial size=2>In the begging of
__ioc_query_sa function you check </FONT></SPAN></DIV>
<DIV><SPAN class=300500910-06092006><FONT face=Arial size=2><STRONG>if(
p_svc->h_node_query )<BR></STRONG></FONT></SPAN><SPAN
class=300500910-06092006><FONT face=Arial size=2><STRONG> return
IB_NOT_DONE;<BR><BR> if( p_svc->h_path_query )<BR> return
IB_NOT_DONE;<BR></STRONG></FONT></SPAN><SPAN class=300500910-06092006><FONT
face=Arial size=2></FONT></SPAN></DIV>
<DIV><SPAN class=300500910-06092006><FONT face=Arial size=2> I think that
when ib_query fail we should set p_svc->h_node_query = NULL so in the next
time the sweep will try to query this mad_svc (this port).</FONT></SPAN></DIV>
<DIV><SPAN class=300500910-06092006><FONT face=Arial size=2>The current code
does not do that </FONT></SPAN></DIV>
<DIV><SPAN class=300500910-06092006><FONT face=Arial
size=2></FONT></SPAN> </DIV><SPAN class=300500910-06092006>
<DIV><BR><FONT face=Arial size=2><STRONG> status = ib_query( gh_al,
&query, &p_svc->h_node_query );<BR> if( status != IB_SUCCESS
)<BR> {<BR> cl_atomic_dec( &p_svc->query_cnt
);<BR> deref_al_obj( &p_svc->obj
);<BR> AL_PRINT_EXIT( TRACE_LEVEL_WARNING,
AL_DBG_PNP,<BR> ("ib_query returned %s\n", ib_get_err_str(
status )) );<BR> return status;<BR> }<BR></STRONG></FONT></DIV>
<DIV><FONT face=Arial size=2></FONT> </DIV>
<DIV><SPAN class=300500910-06092006><FONT face=Arial size=2>What do you think
?</FONT></SPAN></DIV>
<DIV><SPAN class=300500910-06092006><FONT face=Arial
size=2></FONT></SPAN> </DIV>
<DIV><SPAN class=300500910-06092006><FONT face=Arial size=2>Yossi
</FONT></SPAN></DIV></SPAN></BODY></HTML>