github-mirror/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-09-11 20:13:02 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity Graph

selftests/tc-testing: Add cls_route bucket move and change tests

Browse Source 
Add 4 tdc tests for the cls_route bugs fixed earlier in this series:

- Delete a route filter that was moved to another bucket (a7d2):
  Validates that deleting a filter, and making a bucket empty, does not
  leave a dangling empty bucket
- Try to change a route filter onto an already used handle (c05a):
  Validates that attempting to change an existing filter's handle to an
  already taken one fails
- Replace a route filter that shares its key with another filter (3f21):
  Validates that an in-place replace keeps the handle userspace named
  the filter by, rather than dropping the 0x7F00 bits from it
- Replace both route filters sharing a key (9d0e):
  Validates that replacing one of the two does not make the other one
  unreplaceable

Acked-by: Jamal Hadi Salim <jhs@mojatatu.com>
Signed-off-by: Victor Nogueira <victor@mojatatu.com>
Link: https://patch.msgid.link/20260907192133.2639067-5-victor@mojatatu.com
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
This commit is contained in:
Victor Nogueira 2026-09-07 16:21:33 -03:00 committed by Paolo Abeni
parent 41e85e54e5
commit e190a7aabb
1 changed files with 210 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

210
tools/testing/selftests/tc-testing/tc-tests/filters/route.json
View File

@ -202,5 +202,215 @@
"teardown": [
"$TC qdisc del dev $DEV1 parent root drr"
]
},
{
"id": "a7d2",
"name": "Delete a route filter that was moved to another bucket",
"category": [
"filter",
"route"
],
"plugins": {
"requires": "nsPlugin"
},
"setup": [
"$TC qdisc add dev $DEV1 ingress",
"$TC filter add dev $DEV1 parent ffff: protocol ip prio 100 route from 1 to 1 classid 1:1",
"$TC filter change dev $DEV1 parent ffff: protocol ip prio 100 handle 0x10001 route from 1 to 2 classid 1:1",
"$TC filter add dev $DEV1 parent ffff: protocol ip prio 200 route from 5 to 5 classid 1:5"
],
"cmdUnderTest": "$TC filter del dev $DEV1 parent ffff: protocol ip prio 100 handle 0x10002 route from 1 to 2",
"expExitCode": "0",
"verifyCmd": "$TC -j filter show dev $DEV1 parent ffff:",
"matchJSON": [
{
"protocol": "ip",
"pref": 200,
"kind": "route",
"chain": 0
},
{
"protocol": "ip",
"pref": 200,
"kind": "route",
"chain": 0,
"options": {
"fh": "0x50005",
"flowid": "1:5"
}
}
],
"teardown": [
"$TC qdisc del dev $DEV1 ingress"
]
},
{
"id": "c05a",
"name": "Try to change a route filter onto an already used handle",
"category": [
"filter",
"route"
],
"plugins": {
"requires": "nsPlugin"
},
"setup": [
"$TC qdisc add dev $DEV1 ingress",
"$TC filter add dev $DEV1 parent ffff: protocol ip prio 100 route from 1 to 1 classid 1:1 action ok",
"$TC filter add dev $DEV1 parent ffff: protocol ip prio 100 route from 2 to 2 classid 1:2 action drop"
],
"cmdUnderTest": "$TC filter change dev $DEV1 parent ffff: protocol ip prio 100 handle 0x10001 route from 2 to 2 classid 1:1 action ok",
"expExitCode": "2",
"verifyCmd": "$TC -j filter show dev $DEV1 parent ffff:",
"matchJSON": [
{
"protocol": "ip",
"pref": 100,
"kind": "route",
"chain": 0
},
{
"protocol": "ip",
"pref": 100,
"kind": "route",
"chain": 0,
"options": {
"fh": "0x10001",
"flowid": "1:1",
"actions": [
{
"order": 1,
"kind": "gact",
"control_action": {
"type": "pass"
}
}
]
}
},
{
"protocol": "ip",
"pref": 100,
"kind": "route",
"chain": 0,
"options": {
"fh": "0x20002",
"flowid": "1:2",
"actions": [
{
"order": 1,
"kind": "gact",
"control_action": {
"type": "drop"
}
}
]
}
}
],
"teardown": [
"$TC qdisc del dev $DEV1 ingress"
]
},
{
"id": "3f21",
"name": "Replace a route filter that shares its key with another filter",
"category": [
"filter",
"route"
],
"plugins": {
"requires": "nsPlugin"
},
"setup": [
"$TC qdisc add dev $DEV1 ingress",
"$TC filter add dev $DEV1 parent ffff: protocol ip prio 100 handle 0x10101 route from 1 to 1 classid 1:1",
"$TC filter add dev $DEV1 parent ffff: protocol ip prio 100 handle 0x10201 route from 1 to 1 classid 1:2"
],
"cmdUnderTest": "$TC filter replace dev $DEV1 parent ffff: protocol ip prio 100 handle 0x10101 route from 1 to 1 classid 1:9",
"expExitCode": "0",
"verifyCmd": "$TC -j filter show dev $DEV1 parent ffff:",
"matchJSON": [
{
"protocol": "ip",
"pref": 100,
"kind": "route",
"chain": 0
},
{
"protocol": "ip",
"pref": 100,
"kind": "route",
"chain": 0,
"options": {
"fh": "0x10101",
"flowid": "1:9"
}
},
{
"protocol": "ip",
"pref": 100,
"kind": "route",
"chain": 0,
"options": {
"fh": "0x10201",
"flowid": "1:2"
}
}
],
"teardown": [
"$TC qdisc del dev $DEV1 ingress"
]
},
{
"id": "9d0e",
"name": "Replace both route filters sharing a key",
"category": [
"filter",
"route"
],
"plugins": {
"requires": "nsPlugin"
},
"setup": [
"$TC qdisc add dev $DEV1 ingress",
"$TC filter add dev $DEV1 parent ffff: protocol ip prio 100 handle 0x10101 route from 1 to 1 classid 1:1",
"$TC filter add dev $DEV1 parent ffff: protocol ip prio 100 handle 0x10201 route from 1 to 1 classid 1:2",
"$TC filter replace dev $DEV1 parent ffff: protocol ip prio 100 handle 0x10101 route from 1 to 1 classid 1:9"
],
"cmdUnderTest": "$TC filter replace dev $DEV1 parent ffff: protocol ip prio 100 handle 0x10201 route from 1 to 1 classid 1:8",
"expExitCode": "0",
"verifyCmd": "$TC -j filter show dev $DEV1 parent ffff:",
"matchJSON": [
{
"protocol": "ip",
"pref": 100,
"kind": "route",
"chain": 0
},
{
"protocol": "ip",
"pref": 100,
"kind": "route",
"chain": 0,
"options": {
"fh": "0x10101",
"flowid": "1:9"
}
},
{
"protocol": "ip",
"pref": 100,
"kind": "route",
"chain": 0,
"options": {
"fh": "0x10201",
"flowid": "1:8"
}
}
],
"teardown": [
"$TC qdisc del dev $DEV1 ingress"
]
}
]