On 7/2/07, <b class="gmail_sendername">Sean Hefty</b> <<a href="mailto:mshefty@ichips.intel.com">mshefty@ichips.intel.com</a>> wrote:<div><span class="gmail_quote"></span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
> This seems to be solved with the below patch, however, i see that<br>> for duplicate REQs the code is much more involved, which means i<br>> might be over-simplifying here...<br><br>I don't think you're over-simplifying.  The REQ handling seems more
<br>involved because the connected state machine is more complex.</blockquote><div><br>
OK <br>
</div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">REQ handling deals with duplicate messages by waiting to set the cm id<br>state.  We could do the same in the sidr req handler.
</blockquote><div><br>
Can you clarify what "waiting to set the cm id state" means?<br>
 </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Since we're in this part of the code, I'll create a patch to fix the<br>'todo' comment in this function, to ensure that both patches fit
<br>together cleanly.</blockquote><div><br>
Assuming you refer to "todo: reply with no match" in cm_sidr_req_handler, what else<br>
need to be added to the current code? is  it sending the REP with a different status (ie not 2) or sending a REJ?<br>
</div></div><br>
Or.<br>