[Cake] [PATCH] Keep an internal counter for total queue length

Jonathan Morton chromatix99 at gmail.com
Fri Jul 6 09:23:32 EDT 2018


> On 6 Jul, 2018, at 4:21 pm, Toke Høiland-Jørgensen <toke at toke.dk> wrote:
> 
>> I'm handling it without using any extra permanent state, just making
>> use of what's already there. Take a look at latest commit.
> 
> Yup, that also fixes the infinite loop issue. I'm fine with doing it
> this way; however, it doesn't handle the other places where we check
> whether sch->q.qlen is nonzero. Are you sure that is not going to give
> us more problems down the road? :)

The other places are checking whether the queue is empty for the purposes of updating the shaper state.  They are therefore much less critical, and I think self-correcting.

 - Jonathan Morton



More information about the Cake mailing list