<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=gb2312">
<style>
<!--
body
{line-height:1.5}
body
{font-size:16pt;
font-family:���ź�;
color:rgb(0,128,0);
line-height:1.5}
-->
</style>
</head>
<body>
<div><font color="#000000">Hi</font></div>
<div><font color="#000000"><br>
</font></div>
<div><font color="#000000">1. in the callback o2net_fill_node_map -> <span style="line-height:1.5">o2net_tx_can_proceed()</span></font></div>
<div><font color="#000000"><span style="line-height:1.5">2. </span><span style="line-height:1.5">if the function o2net_tx_can_proceed returns false, then
</span><span style="line-height:1.5">"ret" and sc are uninialized,</span></font></div>
<div><font color="#000000"><span style="line-height:1.5">and re-using the value from the
</span><span style="line-height:1.5">previous iteration. </span><span style="line-height:1.5">I think this is not reasonable.</span><span style="line-height:1.5"> </span></font></div>
<div><span style="font-size:16pt; line-height:1.5; color:rgb(0,0,0); background-color:window">I d</span><font color="#000000"><span style="background-color:rgb(255,255,255)"><span style="line-height:1.5">o not know w</span><span style="line-height:1.5">hether
to hide a</span><span style="line-height:1.5"> bug. </span></span></font></div>
<div><font color="#000000"><span style="background-color:rgb(255,255,255)"><font style="line-height:1.5"><span style="line-height:1.5">checking the return value is harmless and
</span></font><span style="line-height:1.5">robustness.</span></span></font></div>
<div>
<pre><font color="#000000" face="���ź�"><span style="background-color:rgb(255,255,255)">Finally, any feedback about this process </span>(positive or negative) would be greatly appreciated.</font></pre>
</div>
<div>
<pre><span style="font-family:���ź�; color:rgb(0,0,0)">/* Get a map of all nodes to which this node is currently connected to */
<br>void o2net_fill_node_map(unsigned long *map, unsigned bytes)
<br>{<br> struct o2net_sock_container *sc = NULL;<br> int node, ret = 0;<br> BUG_ON(bytes < (BITS_TO_LONGS(O2NM_MAX_NODES) * sizeof(unsigned long)));<br> memset(map, 0, bytes);<br> for (node = 0; node < O2NM_MAX_NODES; ++node) {<br> if (!o2net_tx_can_proceed(o2net_nn_from_num(node), &sc, &ret))<br> continue;<br> if (!ret) {<br> set_bit(node, map);<br> sc_put(sc)</span></pre>
<pre><span style="font-size:16pt; line-height:1.5; font-family:���ź�; color:rgb(0,0,0)"> </span><span style="font-size:16pt; line-height:1.5; color:rgb(0,0,0); font-family:���ź�">}</span></pre>
<pre><font color="#000000" face="���ź�"><span style="line-height:1.5"> </span><span style="line-height:1.5">+</span><span style="line-height:1.5"> </span><span style="line-height:1.5">sc = NULL;</span><span style="line-height:1.5"> </span></font></pre>
<pre><font color="#000000" face="���ź�"><span style="line-height:1.5"> + </span><span style="line-height:1.5">ret=0;</span></font></pre>
<pre><span style="font-family:���ź�; color:rgb(0,0,0)"> </span><font color="#000000" face="���ź�">} <br>}</font></pre>
</div>
<hr color="#b5c4df" size="1" align="left" style="width:210px; height:1px">
<div><span>
<div style="margin:10px; font-family:verdana; font-size:10pt">
<div>zhangguanghui 10102</div>
</div>
</span></div>
<span style="font-size:7.5pt; font-family:����ϸ��; color:gray"><span lang="EN-US">-------------------------------------------------------------------------------------------------------------------------------------<br>
</span>���ʼ����丽�����к��ݻ���ͨ�ż�������˾�ı�����Ϣ�������ڷ��������ַ���г�<span lang="EN-US"><br>
</span>�ĸ��˻�Ⱥ�顣��ֹ�κ����������κ���ʽʹ�ã�������������ȫ���ֵ�й¶�����ơ�<span lang="EN-US"><br>
</span>��ɢ�������ʼ��е���Ϣ������������˱��ʼ������������绰���ʼ�֪ͨ�����˲�ɾ����<span lang="EN-US"><br>
</span>�ʼ���<span lang="EN-US"><br>
</span></span><span lang="EN-US" style="font-size:7.5pt; font-family:"Arial","sans-serif"; color:gray">This e-mail and its attachments contain confidential information from H3C, which is
<br>
intended only for the person or entity whose address is listed above. Any use of the
<br>
information contained herein in any way (including, but not limited to, total or partial
<br>
disclosure, reproduction, or dissemination) by persons other than the intended <br>
recipient(s) is prohibited. If you receive this e-mail in error, please notify the sender
<br>
by phone or email immediately and delete it!</span>
</body>
</html>