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

[Kevin 'ldir' Darbyshire-Bryant <ldir@darbyshire-bryant.me.uk>]

[-- Attachment #1: Type: text/plain, Size: 998 bytes --]



> On 25 Jan 2022, at 10:58, Toke Høiland-Jørgensen <toke@redhat.com> wrote:
> 
> Matt Johnston <matt@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


[-- Attachment #2: Message signed with OpenPGP --]
[-- Type: application/pgp-signature, Size: 833 bytes --]