[Cake] [PATCH net] sch_cake: diffserv8 CS1 should be bulk

Kevin 'ldir' Darbyshire-Bryant ldir at darbyshire-bryant.me.uk
Thu Jan 27 04:00:14 EST 2022



> On 25 Jan 2022, at 10:58, Toke Høiland-Jørgensen <toke at redhat.com> wrote:
> 
> Matt Johnston <matt at codeconstruct.com.au> writes:
> 
>> The CS1 priority (index 0x08) was changed from 0 to 1 when LE (index
>> 0x01) was added. This looks unintentional, it doesn't match the
>> docs and CS1 shouldn't be the same tin as AF1x
> 
> Hmm, Kevin, any comments?
> 
> -Toke
> 

I’ll have to find my thinking head & time machine :-)
This would be a lot easier if we had ‘diffserv9’, LE could have simply
been added as the ‘if you’ve really nothing better to do’ class that it
is.  And it’s why I’ve personally argued for a diffserv5: lowest;low;normal;high;highest
moving on.

I think I screwed up when LE was added to diffserv8 - Matt the CS1 change from 0 to 1 IS intentional
and IIRC I tried to bump everything else up 1 to compensate.. I may have missed some things though.


Cheers,

Kevin D-B

gpg: 012C ACB2 28C6 C53E 9775  9123 B3A2 389B 9DE2 334A

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Message signed with OpenPGP
URL: <https://lists.bufferbloat.net/pipermail/cake/attachments/20220127/b8a0368f/attachment.sig>


More information about the Cake mailing list