Page MenuHomeVyOS Platform

Problems in RED/WRED implementation (QoS)
Open, NormalPublicBUG

Description

Hi all!
I've found that CLI sometimes generate wrong config for RED/WRED qdiscs.

  1. set traffic-policy priority-queue prio1 default queue-type random-detect don't work: no bandwidth parameter is transferred to tc command:

qdisc add dev eth2 parent 1:2 red limit 73728 min 6144 max 18432 avpkt 1024 burst 10 probability 0.1 bandwidth ecn

  1. There is typo in formula, used for burst calculation (https://github.com/vyos/vyatta-cfg-qos/blob/current/lib/Vyatta/Qos/RandomDetect.pm#L121):

my $burst = ( 2 * $pred->{qmin} + $pred->{qmax} ) / 3;
need to be changed to:
my $burst = ( 2 * $pred->{qmin} + $pred->{qmax} ) / ( 3 * $pred->{avpkt} );

Also, there is a predefined average size of packet at 1024 bytes. It is not very good idea to fix this variable, especially at this value. In real Internet traffic average packet size is closer to 500 bytes, than to 1000. Better, if this is possible, will be allow to set own value, as traffic can be very different.

Details

Difficulty level
Unknown (require assessment)
Version
1.2.0-rolling+201812010337
Why the issue appeared?
Will be filled on close
Is it a breaking change?
Unspecified (possibly destroys the router)
Issue type
Bug (incorrect behavior)

Event Timeline

syncer triaged this task as Normal priority.Dec 6 2018, 12:05 AM
syncer edited projects, added VyOS 1.3 Equuleus; removed VyOS 1.2 Crux.
Unknown Object (User) added a subscriber: Unknown Object (User).Jun 10 2020, 8:36 PM
dmbaturin set Is it a breaking change? to Unspecified (possibly destroys the router).
erkin set Issue type to Bug (incorrect behavior).Aug 31 2021, 7:12 PM