Re: [PATCH net 9/9] selftests/tc-testing: Add netem test case exercising loops

[Paolo Abeni <pabeni@redhat.com>]

On 4/26/26 9:09 PM, Jamal Hadi Salim wrote:
> From: Victor Nogueira <victor@mojatatu.com>
> 
> Add a netem nested duplicate test case to validate that it won't
> cause an infinite loop
> 
> Acked-by: Jamal Hadi Salim <jhs@mojatatu.com>
> Acked-by: Stephen Hemminger <stephen@networkplumber.org>
> Signed-off-by: Victor Nogueira <victor@mojatatu.com>
> ---
>  .../tc-testing/tc-tests/qdiscs/netem.json     | 33 ++++++++++++++++++-
>  1 file changed, 32 insertions(+), 1 deletion(-)
> 
> diff --git a/tools/testing/selftests/tc-testing/tc-tests/qdiscs/netem.json b/tools/testing/selftests/tc-testing/tc-tests/qdiscs/netem.json
> index 3c4444961488..7c954989069d 100644
> --- a/tools/testing/selftests/tc-testing/tc-tests/qdiscs/netem.json
> +++ b/tools/testing/selftests/tc-testing/tc-tests/qdiscs/netem.json
> @@ -336,5 +336,36 @@
>          "teardown": [
>              "$TC qdisc del dev $DUMMY handle 1: root"
>          ]
> -    }
> +    },
> +    {
> +        "id": "8c17",
> +        "name": "Test netem's recursive duplicate",

This test is apparently consistently failing on our CI:

# not ok 365 8c17 - Test netem's recursive duplicate
# Value doesn't match: bytes: 294 != 196
# Matching against output: {'kind': 'netem', 'handle': '1:', 'root':
True, 'refcnt': 2, 'options': {'limit': 1, 'duplicate': {'duplicate': 1,
'correlation': 0}, 'seed': 17230427318941832146, 'ecn': False, 'gap':
0}, 'bytes': 196, 'packets': 2, 'drops': 1, 'overlimits': 0, 'requeues':
0, 'backlog': 0, 'qlen': 0}

sample full info avail:

https://github.com/p4tc-dev/tc-executor/blob/storage/artifacts/622604/1-tdc-sh/

/P