<div dir="ltr">The pie patches that I was using until recently are here:<br><div><br><a href="http://snapon.lab.bufferbloat.net/~cero2/deb/patches/">http://snapon.lab.bufferbloat.net/~cero2/deb/patches/</a><br><br></div><div>
Stephen hemminger and I cleaned up the original pie drop according to KernelStyle in multiple ways. There is a rounding error problem in setting the target value <br></div><div>(we'd recomended they switch to ktime rather than jiffies) that I either added or was in there in the first place.... and we fixed a couple generic bugs...<br>
<br></div><div>In addition to adding ecn support, we also added congestion notification to (a la the linux red and fq_codel algorithms) to provide lossless feedback into the host tcp stack. (this is, incidentally, 1 line of code) This optimization made it possible to do comparisons on host <-> host tests between these qdiscs, with no router in-between.<br>
<br></div><div>Syntactically the iproute2 patches could use identical syntax for time based values as the codel iproute2 patches. Haven't got around to that. (not my job, really, to improve cisco's code)<br><br></div>
<div>Now, I only now - today! - just noticed that the new pie code at cisco's site seems *substantially* different from the older pie code the above was based on, in that it no longer has an optimization for not dropping smaller packets. (bytemode) This is going to invalidate most previous benchmarks of pie that I know of, including all of mine, and most, if not all, of cablelabs'. Sigh.<br>
<br></div><div>I will try to refresh my patches and update the lab over the next week or so.<br></div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Aug 14, 2013 at 2:17 PM, Naeem Khademi <span dir="ltr"><<a href="mailto:naeem.khademi@gmail.com" target="_blank">naeem.khademi@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Vijay<br>
<br>
This code implements basic ECN capability for PIE meaning that it<br>
would CE-mark packets that would be early-dropped otherwise. I have<br>
sanity-checked the code (e.g. successful ECN marking and<br>
receiver/sender reaction). We're now in process of conducting larger<br>
set of experiments with PIE w/wo ECN that will eventually be published<br>
as we move forward.<br>
<br>
Cheers,<br>
Naeem<br>
<div class="HOEnZb"><div class="h5"><br>
On Wed, Aug 14, 2013 at 8:27 PM, Vijay Subramanian<br>
<<a href="mailto:subramanian.vijay@gmail.com">subramanian.vijay@gmail.com</a>> wrote:<br>
> Naeem,<br>
><br>
> Thanks for testing and the contribution! We have ECN support in our current<br>
> code-under-development thanks to earlier help from Dave Taht but I will<br>
> merge your changes as appropriate and<br>
> submit it back for review.<br>
><br>
> I will take a look at your code soon. Do you also have any test results<br>
> which you can share?<br>
><br>
> Thanks again,<br>
> Vijay<br>
><br>
><br>
> On 14 August 2013 07:05, Naeem Khademi <<a href="mailto:naeem.khademi@gmail.com">naeem.khademi@gmail.com</a>> wrote:<br>
>><br>
>> Hi<br>
>><br>
>> The already-existing PIE Linux code maintained at<br>
>> <a href="ftp://ftpeng.cisco.com/pie/" target="_blank">ftp://ftpeng.cisco.com/pie/</a> does not support ECN. Studying the interaction<br>
>> of ECN and AQM algos is probably of interest of some of us. I have added ECN<br>
>> support to PIE Linux code and have given it some basic sanity tests.<br>
>><br>
>> The modified code is available at<br>
>> <a href="http://heim.ifi.uio.no/naeemk/research/PIE/" target="_blank">http://heim.ifi.uio.no/naeemk/research/PIE/</a><br>
>><br>
>> I would appreciate any feedback.<br>
>><br>
>> Regards,<br>
>> Naeem<br>
>><br>
>><br>
>><br>
>> _______________________________________________<br>
>> iccrg mailing list<br>
>> <a href="mailto:iccrg@irtf.org">iccrg@irtf.org</a><br>
>> <a href="https://www.irtf.org/mailman/listinfo/iccrg" target="_blank">https://www.irtf.org/mailman/listinfo/iccrg</a><br>
>><br>
><br>
_______________________________________________<br>
iccrg mailing list<br>
<a href="mailto:iccrg@irtf.org">iccrg@irtf.org</a><br>
<a href="https://www.irtf.org/mailman/listinfo/iccrg" target="_blank">https://www.irtf.org/mailman/listinfo/iccrg</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Dave Täht<br><br>Fixing bufferbloat with cerowrt: <a href="http://www.teklibre.com/cerowrt/subscribe.html" target="_blank">http://www.teklibre.com/cerowrt/subscribe.html</a> 
</div>