<div dir="ltr"><div><div><div>I've been looking at v2 of this patch, and have not been able to understand the operation of the for loop in these lines from mpnh_sort:<br><br>          ret = mpnh_copy_lp(x, lp);<br><br>          for (old_prev = x, new_prev = ret;<br>                    old_prev != prev;<br>                    old_prev = old_prev->next, new_prev = new_prev->next);<br><br></div>I'm probably being stupid - but can someone explain how that for loop works?<br><br></div>Thanks,<br></div>     Neil<br><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Aug 22, 2016 at 11:21 AM Neil Jerram <<a href="mailto:neil@tigera.io" target="_blank">neil@tigera.io</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div>Hi Mikhail,<br><br></div>I don't want to sound ungrateful, but that sounds a bit odd.  Is something stopping you from posting the updated patch immediately?  I would really appreciate it today, or as soon as possible after that.<br><br></div>Best wishes,<br></div>      Neil<br><br></div><br><div class="gmail_quote"><div dir="ltr">On Sun, Aug 21, 2016 at 8:19 PM Mikhail Sennikovskii <<a href="mailto:mikhail.sennikovskii@profitbricks.com" target="_blank">mikhail.sennikovskii@profitbricks.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Michael, Neil and all,<div><br></div><div>Thanks for you interest in the patch. I'll try to post the updated patch version within the next few days.</div></div><div dir="ltr"><div><br></div><div>Mikhail</div></div><div class="gmail_extra"><br><div class="gmail_quote">2016-08-20 2:55 GMT+03:00 Michael Elkins <span dir="ltr"><<a href="mailto:me@sigpipe.org" target="_blank">me@sigpipe.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On Thu, Aug 18, 2016 at 06:39:24PM +0200, Arno Töll wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Having that said, I am aware of at least another issue related to the<br>
route scan after an ECMP export, which we fixed internally in v3. In<br>
lack of interest, we did not bother to send it upstream so far.<br>
However, I will ask Mikhail to send it tomorrow, or I will do, so that<br>
other interested folks can at least find it on the list.<br>
</blockquote>
<br></span>
Arno,<br>
<br>
I also wanted to voice interest in this work.  I've been testing the v2 patch, and I've encountered the bug you described (the multipath route gets installed initially, but bird subsequently reports a sync error, and the debug log indicates the multipath route is attempting to be reinstalled indefinitely with error).<br>
<br>
Regards,<br>
<br>
Michael<br>
<br>
</blockquote></div><br></div>
</blockquote></div></blockquote></div></div>