bsp: Add cake scheduler kernel module #67

fbl · 2021-01-08T00:36:29+01:00

fbl commented

2021-01-08 00:36:29 +01:00

The cake scheduler is a popular fair queuing scheduler, which is also
capable of shaping traffic. Due to its sensible defaults it is very
easy to set up.

When tunnel traffic exceeds the capability of the transport connection,
firmware users might want to shape traffic, so meaningful queueing can be done
before tunnel packets are dropped. As this is typically combined with a fair
scheduler, cake provides a simple yet very powerful solution for both problems.

Therefore the cake kernel module is now built and included for all of our
devices, except for ath79-tiny, where we are critically low on storage already.

Signed-off-by: Fabian Bläse fabian@blaese.de

The cake scheduler is a popular fair queuing scheduler, which is also capable of shaping traffic. Due to its sensible defaults it is very easy to set up. When tunnel traffic exceeds the capability of the transport connection, firmware users might want to shape traffic, so meaningful queueing can be done before tunnel packets are dropped. As this is typically combined with a fair scheduler, cake provides a simple yet very powerful solution for both problems. Therefore the cake kernel module is now built and included for all of our devices, except for ath79-tiny, where we are critically low on storage already. Signed-off-by: Fabian Bläse <fabian@blaese.de>

fbl added this to the 20210211-beta milestone 2021-01-08 00:36:29 +01:00

fbl added the

feature

layer3

labels 2021-01-08 00:36:29 +01:00

jkimmel commented

2021-01-08 06:31:05 +01:00

Mit cake kann man fast nichts falsch machen. Leistungstechnisch ist cake angeblich auch unproblematisch, also kein Problem für unsere Hardware.
Ist zu überlegen, ob man cake nicht einfach überall default macht, weil es selbst ohne konfigurierte Bandbreite einen guten Job macht, indem es zum Beispiel redundante ACKs bei TCP-Verbindungen filtert.

Acked-by: Johannes Kimmel <fff@bareminimum.eu>

Mit `cake` kann man fast nichts falsch machen. Leistungstechnisch ist `cake` angeblich auch unproblematisch, also kein Problem für unsere Hardware. Ist zu überlegen, ob man `cake` nicht einfach überall default macht, weil es selbst ohne konfigurierte Bandbreite einen guten Job macht, indem es zum Beispiel redundante `ACK`s bei TCP-Verbindungen filtert. `Acked-by: Johannes Kimmel <fff@bareminimum.eu>`

jkimmel commented

2021-01-08 06:33:54 +01:00

Oh, und falls sich jemand fragt, um was es eigentlich geht:

Oh, und falls sich jemand fragt, um was es eigentlich geht: * https://www.bufferbloat.net/projects/codel/wiki/CakeFAQ/ * https://www.bufferbloat.net/projects/codel/wiki/Cake/ * https://www.bufferbloat.net/projects/codel/wiki/CakeTechnical/ * https://www.bufferbloat.net/projects/codel/wiki/CakeRecipes/

ChristianD commented

2021-01-08 09:24:36 +01:00

soll das wirklich auch mit in die Node Firmware? War da bisher nicht immer der Konsens das auf jeden Device das gleiche drauf sein soll? Wäre damit dann nicht mehr gegeben da es auf den tiny fehlt.

Spricht was dagegen dies nur ins L3 zu packen und dort dann als Abhängigkeit in das layer 3 package?

soll das wirklich auch mit in die Node Firmware? War da bisher nicht immer der Konsens das auf jeden Device das gleiche drauf sein soll? Wäre damit dann nicht mehr gegeben da es auf den tiny fehlt. Spricht was dagegen dies nur ins L3 zu packen und dort dann als Abhängigkeit in das layer 3 package?

fbl commented

2021-01-08 11:16:02 +01:00

Das ganze lässt sich nur pro BSP realisieren. ath79-generic und -tiny unterscheiden sich ja sowieso schon.

Dennoch könnte man das auch als Abhängigkeit für das layer3 Paket bauen. Gestern hab ich da gar nicht drüber nachgedacht, dass das auch gehen müsste.

Das ganze lässt sich nur pro BSP realisieren. ath79-generic und -tiny unterscheiden sich ja sowieso schon. Dennoch könnte man das auch als Abhängigkeit für das layer3 Paket bauen. Gestern hab ich da gar nicht drüber nachgedacht, dass das auch gehen müsste.

fbl force-pushed cake from c02dbc7943 to 4ef5d745c8

2021-01-08 18:49:36 +01:00

Compare

fbl commented

2021-01-08 18:50:36 +01:00

Changes:

Add cake as a layer3 dependency instead of statically including it in target configurations

Changes: * Add cake as a layer3 dependency instead of statically including it in target configurations

ChristianD commented

2021-01-08 18:54:52 +01:00

Reviewed-by: Christian Dresel <freifunk@dresel.systems>

``` Reviewed-by: Christian Dresel <freifunk@dresel.systems> ```

ChristianD approved these changes 2021-01-08 18:55:10 +01:00

fbl force-pushed cake from 4ef5d745c8 to 9b161e44d6

2021-01-09 11:19:07 +01:00

Compare

fbl commented

2021-01-09 11:19:17 +01:00

Changes:

Bump PKG_RELEASE

Changes: * Bump PKG_RELEASE

adschm commented

2021-01-13 22:05:27 +01:00

Danke, wird gleich gemergt.

adschm closed this pull request

2021-01-13 22:05:27 +01:00

jkimmel commented

2021-01-13 23:47:34 +01:00

be4c1fe689

be4c1fe689c939193499e85b258ca77ead9e0d0f

fbl deleted branch cake

2021-01-27 16:52:58 +01:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Sign in to join this conversation.

No reviewers

No Label

more details required

No Milestone

No Assignees

4 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: freifunk-franken/firmware#67