It is easier to add debug prints for sdp state machine<br><br><div class="gmail_quote">On Thu, Jun 19, 2008 at 8:04 PM, Sean Hefty <<a href="mailto:sean.hefty@intel.com">sean.hefty@intel.com</a>> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">>+static inline void sdp_set_state(struct sock *sk, int state)<br>
>+{<br>
>+      sk->sk_state = state;<br>
>+}<br>
<br>
</div>Why have this abstraction?<br>
<div class="Ih2E3d"><br>
>-      sk->sk_state = TCP_CLOSE;<br>
>+      sdp_set_state(sk, TCP_CLOSE);<br>
<br>
</div>IMO, the previous code is simpler to maintain.<br>
<font color="#888888"><br>
- Sean<br>
</font><div><div></div><div class="Wj3C7c"><br>
_______________________________________________<br>
general mailing list<br>
<a href="mailto:general@lists.openfabrics.org">general@lists.openfabrics.org</a><br>
<a href="http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general" target="_blank">http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general</a><br>
<br>
To unsubscribe, please visit <a href="http://openib.org/mailman/listinfo/openib-general" target="_blank">http://openib.org/mailman/listinfo/openib-general</a><br>
</div></div></blockquote></div><br>