<br><br>
<div><span class="gmail_quote">On 5/22/08, <b class="gmail_sendername">Hal Rosenstock</b> <<a href="mailto:hrosenstock@xsigo.com">hrosenstock@xsigo.com</a>> wrote:</span>
<blockquote class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid">Olga,<br><br>On Thu, 2008-05-22 at 18:28 +0300, Olga Shern (Voltaire) wrote:<br>> Hal,<br>> You pointed out that we cannot rely on the assumption that on SM<br>
> failover there is not path change.<br>> In the previous patch we only flush multicast.<br>> What Moni changed in this patch is that on SM failover (SM change<br>> event), we will flush not only multicast but also all paths but<br>
> without destroying ah.<br><br>I missed that in the patch :-( It addresses the first level of concern<br>in terms of the unicast paths but leaves open the path parameter changes<br>(rate, etc.) as the address handles are preserved as Moni stated in<br>
other words. I agree it's in the right direction. I would like to see<br>the whole problem solved. Is the cost of recreating the AHs too much or<br>is something else leading towards preserving the AHs ? That's what's<br>
needed to be resolved for a complete solution.</blockquote>
<div> </div>
<div>I didn't explain it well, I will try again :)</div>
<div> </div>
<div>On SM change event we will not destroy ah but will flush paths, therefore unicast traffic will continue without packets lost.</div>
<div>When there will be arp probe (issued by the kernel) it will look for a path and because we have deleted it will issue path query to SM and after reply from sm it will create a new ah that will replace the old ah.</div>
<div> </div>
<div>Before this patch all packets were dropped till there is a new ah, this patch creating new ah at the background.</div>
<div> </div>
<div>I hope it is clear now.</div>
<div> </div>
<div>Olga </div><br>
<blockquote class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid">-- Hal<br><br>> Olga<br><br></blockquote></div><br>