Re: [PATCH] selftests: tc-testing: add one test for flushing explicitly created chain

From: renmingshuai
Date: Mon Jun 19 2023 - 22:01:59 EST


>Hi renmingshuai:
>On 2023/6/17 11:20, renmingshuai wrote:
>> Add the test for additional reference to chains that are explicitly created
>> by RTM_NEWCHAIN message
>>
>> commit c9a82bec02c3 ("net/sched: cls_api: Fix lockup on flushing explicitly
>> created chain")
>> Signed-off-by: Mingshuai Ren <renmingshuai@xxxxxxxxxx>
>> ---
>> .../tc-testing/tc-tests/infra/filter.json | 25 +++++++++++++++++++
>> 1 file changed, 25 insertions(+)
>> create mode 100644 tools/testing/selftests/tc-testing/tc-tests/infra/filter.json
>>
>> diff --git a/tools/testing/selftests/tc-testing/tc-tests/infra/filter.json b/tools/testing/selftests/tc-testing/tc-tests/infra/filter.json
>> new file mode 100644
>> index 000000000000..c4c778e83da2
>> --- /dev/null
>> +++ b/tools/testing/selftests/tc-testing/tc-tests/infra/filter.json
>> @@ -0,0 +1,25 @@
>> +[
>> + {
>> + "id": "c2b4",
>> + "name": "soft lockup alarm will be not generated after delete the prio 0 filter of the chain",
> "Delete the prio 0 filter of chain" looks better. And adding
> test result in comment also will be better.
>
>Zhengchao Shao
Thanks for your advice, and I have add the test result in comment.
https://lore.kernel.org/netdev/20230620014939.2034054-1-renmingshuai@xxxxxxxxxx/T/#u
The original name can describe the purpose of adding the test case, which would be better.