linux-stable.git/tools/testing/selftests/tc-testing, branch v6.8

net/sched: Remove ipt action tests

2024-01-07T18:23:32+00:00

Commit ba24ea129126 ("net/sched: Retire ipt action") removed the ipt action
but not the testcases. This patch removes the outstanding tdc tests.

Signed-off-by: Jamal Hadi Salim 
Signed-off-by: David S. Miller

net/sched: Retire ipt action

2024-01-02T12:41:16+00:00

The tc ipt action was intended to run all netfilter/iptables target.
Unfortunately it has not benefitted over the years from proper updates when
netfilter changes, and for that reason it has remained rudimentary.
Pinging a bunch of people that i was aware were using this indicates that
removing it wont affect them.
Retire it to reduce maintenance efforts. Buh-bye.

Reviewed-by: Victor Noguiera 
Reviewed-by: Pedro Tammela 
Signed-off-by: Jamal Hadi Salim 
Signed-off-by: David S. Miller

selftests: tc-testing: remove filters/tests.json

2023-12-01T07:22:18+00:00

Remove this generic file and move the tests to their appropriate files

Signed-off-by: Pedro Tammela 
Acked-by: Jamal Hadi Salim 
Link: https://lore.kernel.org/r/20231129222424.910148-5-pctammela@mojatatu.com
Signed-off-by: Jakub Kicinski

selftests: tc-testing: rename concurrency.json to flower.json

2023-12-01T07:22:18+00:00

All tests in this file pertain to flower, so name it appropriately

Signed-off-by: Pedro Tammela 
Acked-by: Jamal Hadi Salim 
Link: https://lore.kernel.org/r/20231129222424.910148-4-pctammela@mojatatu.com
Signed-off-by: Jakub Kicinski

selftests: tc-testing: remove spurious './' from Makefile

2023-12-01T07:22:17+00:00

Patchwork CI didn't like the extra './', so remove it.

Signed-off-by: Pedro Tammela 
Acked-by: Jamal Hadi Salim 
Link: https://lore.kernel.org/r/20231129222424.910148-3-pctammela@mojatatu.com
Signed-off-by: Jakub Kicinski

selftests: tc-testing: remove spurious nsPlugin usage

2023-12-01T07:22:17+00:00

Tests using DEV2 should not be run in a dedicated net namespace,
and in parallel, as this device cannot be shared.

Signed-off-by: Pedro Tammela 
Acked-by: Jamal Hadi Salim 
Link: https://lore.kernel.org/r/20231129222424.910148-2-pctammela@mojatatu.com
Signed-off-by: Jakub Kicinski

selftests: tc-testing: remove unused import

2023-11-28T02:15:43+00:00

Remove this leftover from the times we pre-allocated everything

Signed-off-by: Pedro Tammela 
Acked-by: Jamal Hadi Salim 
Link: https://lore.kernel.org/r/20231124154248.315470-6-pctammela@mojatatu.com
Signed-off-by: Jakub Kicinski

selftests: tc-testing: cleanup on Ctrl-C

2023-11-28T02:15:43+00:00

Cleanup net namespaces and other resources if we get a SIGINT (Ctrl-C).
As user visible resources are allocated on a per test basis, it's only
required to catch this condition when (possibly) running tests.

So far calling post_suite is enough to free up anything that might
linger.

A missing keyword replacement for nsPlugin is also included.

Signed-off-by: Pedro Tammela 
Acked-by: Jamal Hadi Salim 
Link: https://lore.kernel.org/r/20231124154248.315470-5-pctammela@mojatatu.com
Signed-off-by: Jakub Kicinski

selftests: tc-testing: prefix iproute2 functions with "ipr2"

2023-11-28T02:15:42+00:00

As suggested by Simon, prefix the functions that operate on iproute2
commands in contrast with the "nl" netlink prefix.

Cc: Simon Horman 
Signed-off-by: Pedro Tammela 
Acked-by: Jamal Hadi Salim 
Link: https://lore.kernel.org/r/20231124154248.315470-4-pctammela@mojatatu.com
Signed-off-by: Jakub Kicinski

selftests: tc-testing: remove unnecessary time.sleep

2023-11-28T02:15:42+00:00

This operation is redundant and it's not stabilizing nor waiting
for anything.

Signed-off-by: Pedro Tammela 
Acked-by: Jamal Hadi Salim 
Link: https://lore.kernel.org/r/20231124154248.315470-3-pctammela@mojatatu.com
Signed-off-by: Jakub Kicinski