[Cake] [sch_cake] Merge branch 'exposeinterval' (5541271)
moeller0 at gmx.de
Mon Oct 12 07:49:12 EDT 2015
On Oct 12, 2015, at 13:16 , Dave Taht <dave.taht at gmail.com> wrote:
> um, no, interval needs to be kept in usec, and converted more like I did.
I have not looked at your code yet. In tc (tc_utils.h or so) there is get_time() that knows how to deal with the default time specifiers like sec ms us… I believe from the tc side this should be used to parse the user input, just like Kevin’s code does. Now this needs to be passed to sch_cake.c, where I believe Kevin’s version expects milliseconds (MS2TIME(q->interval)), which does not do the right thing if the user specified “rtf 123usecs” on tic’s cli… So just getting rid of the MS2TIME called should do the trick, no?
> On Mon, Oct 12, 2015 at 2:36 AM, Dave Taht <dave.taht at gmail.com> wrote:
>> On Mon, Oct 12, 2015 at 11:28 AM, Jonathan Morton <chromatix99 at gmail.com> wrote:
>>>> On 12 Oct, 2015, at 12:24, Dave Taht <dave.taht at gmail.com> wrote:
>>>> This is incorrect for the DC use case. We should be able to have the
>>>> target fall below at least 250us.
>>> Hey, I’m trying to work on this myself, but the code keeps changing under me.
>> Tough. Deal with it. :) Once the rtt option works right toke and I can
>> finally go test the
>> count/2 idea out properly in the testbed, like, today. You otherwise
>> have the pen,
>> as far as I am concerned.
>>> - Jonathan Morton
>> Dave Täht
>> Do you want faster, better, wifi? https://www.patreon.com/dtaht
> Dave Täht
> Do you want faster, better, wifi? https://www.patreon.com/dtaht
More information about the Cake