[RESEND PATCH net-next 1/1] tc-testing: updated police, mirred, skbedit and skbmod with more tests
Added extra test cases for control actions (reclassify, pipe etc.), cookies, max index value and police args sanity check. Signed-off-by: Roman Mashak --- .../tc-testing/tc-tests/actions/mirred.json| 192 + .../tc-testing/tc-tests/actions/police.json| 144 .../tc-testing/tc-tests/actions/skbedit.json | 168 ++ .../tc-testing/tc-tests/actions/skbmod.json| 26 ++- 4 files changed, 529 insertions(+), 1 deletion(-) diff --git a/tools/testing/selftests/tc-testing/tc-tests/actions/mirred.json b/tools/testing/selftests/tc-testing/tc-tests/actions/mirred.json index 0fcccf18399b..443c9b3c8664 100644 --- a/tools/testing/selftests/tc-testing/tc-tests/actions/mirred.json +++ b/tools/testing/selftests/tc-testing/tc-tests/actions/mirred.json @@ -171,6 +171,198 @@ ] }, { +"id": "8917", +"name": "Add mirred mirror action with control pass", +"category": [ +"actions", +"mirred" +], +"setup": [ +[ +"$TC actions flush action mirred", +0, +1, +255 +] +], +"cmdUnderTest": "$TC actions add action mirred ingress mirror dev lo pass index 1", +"expExitCode": "0", +"verifyCmd": "$TC actions get action mirred index 1", +"matchPattern": "action order [0-9]*: mirred \\(Ingress Mirror to device lo\\) pass.*index 1 ref", +"matchCount": "1", +"teardown": [ +"$TC actions flush action mirred" +] +}, +{ +"id": "1054", +"name": "Add mirred mirror action with control pipe", +"category": [ +"actions", +"mirred" +], +"setup": [ +[ +"$TC actions flush action mirred", +0, +1, +255 +] +], +"cmdUnderTest": "$TC actions add action mirred ingress mirror dev lo pipe index 15", +"expExitCode": "0", +"verifyCmd": "$TC actions get action mirred index 15", +"matchPattern": "action order [0-9]*: mirred \\(Ingress Mirror to device lo\\) pipe.*index 15 ref", +"matchCount": "1", +"teardown": [ +"$TC actions flush action mirred" +] +}, +{ +"id": "9887", +"name": "Add mirred mirror action with control continue", +"category": [ +"actions", +"mirred" +], +"setup": [ +[ +"$TC actions flush action mirred", +0, +1, +255 +] +], +"cmdUnderTest": "$TC actions add action mirred ingress mirror dev lo continue index 15", +"expExitCode": "0", +"verifyCmd": "$TC actions get action mirred index 15", +"matchPattern": "action order [0-9]*: mirred \\(Ingress Mirror to device lo\\) continue.*index 15 ref", +"matchCount": "1", +"teardown": [ +"$TC actions flush action mirred" +] +}, +{ +"id": "e4aa", +"name": "Add mirred mirror action with control reclassify", +"category": [ +"actions", +"mirred" +], +"setup": [ +[ +"$TC actions flush action mirred", +0, +1, +255 +] +], +"cmdUnderTest": "$TC actions add action mirred ingress mirror dev lo reclassify index 150", +"expExitCode": "0", +"verifyCmd": "$TC actions get action mirred index 150", +"matchPattern": "action order [0-9]*: mirred \\(Ingress Mirror to device lo\\) reclassify.*index 150 ref", +"matchCount": "1", +"teardown": [ +"$TC actions flush action mirred" +] +}, +{ +"id": "ece9", +"name": "Add mirred mirror action with control drop", +"category": [ +"actions", +"mirred" +], +"setup": [ +[ +"$TC actions flush action mirred", +0, +1, +255 +] +], +"cmdUnderTest": "$TC actions add action mirred ingress mirror dev lo drop index 99", +"expExitCode": "0", +"verifyCmd": "$TC actions get action mirred index 99", +"matchPattern": "action order [0-9]*: mirred \\(Ingress Mirror to device lo\\) drop.*index 99 ref", +"matchCount": "1", +"teardown": [ +"$TC actions flush action mirred" +] +}, +{ +"id": "0031", +"name": "Add mirred mirror action with control jump", +"category": [ +"actions", +"mirred" +], +"setup": [ +[ +"$TC actions flush action mirred", +0,
Re: [RESEND PATCH net-next 1/1] tc-testing: updated police, mirred, skbedit and skbmod with more tests
From: Roman Mashak Date: Thu, 22 Mar 2018 15:29:36 -0400 > David Miller writes: > >> From: Roman Mashak >> Date: Thu, 22 Mar 2018 08:23:22 -0400 >> >>> diff --git >>> a/tools/testing/selftests/tc-testing/tc-tests/actions/skbmod.json >>> b/tools/testing/selftests/tc-testing/tc-tests/actions/skbmod.json >>> index 90bba48c3f07..8aa5a88ccb19 100644 >>> --- a/tools/testing/selftests/tc-testing/tc-tests/actions/skbmod.json >>> +++ b/tools/testing/selftests/tc-testing/tc-tests/actions/skbmod.json >> ... >>> ] >>> } >>> -] >>> +] >>> \ No newline at end of file >>> -- >>> 2.7.4 >> >> Please fix this. > > The patch updates police.json, mirred.json, skbedit.json and > skbmod.json files that initially had no newline on the end. My bad, please repost the patch and I'll apply it. Thank you.
Re: [RESEND PATCH net-next 1/1] tc-testing: updated police, mirred, skbedit and skbmod with more tests
David Miller writes: > From: Roman Mashak > Date: Thu, 22 Mar 2018 08:23:22 -0400 > >> diff --git a/tools/testing/selftests/tc-testing/tc-tests/actions/skbmod.json >> b/tools/testing/selftests/tc-testing/tc-tests/actions/skbmod.json >> index 90bba48c3f07..8aa5a88ccb19 100644 >> --- a/tools/testing/selftests/tc-testing/tc-tests/actions/skbmod.json >> +++ b/tools/testing/selftests/tc-testing/tc-tests/actions/skbmod.json > ... >> ] >> } >> -] >> +] >> \ No newline at end of file >> -- >> 2.7.4 > > Please fix this. The patch updates police.json, mirred.json, skbedit.json and skbmod.json files that initially had no newline on the end.
Re: [RESEND PATCH net-next 1/1] tc-testing: updated police, mirred, skbedit and skbmod with more tests
From: Roman Mashak Date: Thu, 22 Mar 2018 08:23:22 -0400 > diff --git a/tools/testing/selftests/tc-testing/tc-tests/actions/skbmod.json > b/tools/testing/selftests/tc-testing/tc-tests/actions/skbmod.json > index 90bba48c3f07..8aa5a88ccb19 100644 > --- a/tools/testing/selftests/tc-testing/tc-tests/actions/skbmod.json > +++ b/tools/testing/selftests/tc-testing/tc-tests/actions/skbmod.json ... > ] > } > -] > +] > \ No newline at end of file > -- > 2.7.4 Please fix this.
Re: [PATCH net-next 1/1] tc-testing: updated police, mirred, skbedit and skbmod with more tests
From: Roman Mashak Date: Wed, 21 Mar 2018 18:17:50 -0400 > diff --git a/tools/testing/selftests/tc-testing/tc-tests/actions/skbmod.json > b/tools/testing/selftests/tc-testing/tc-tests/actions/skbmod.json > index 90bba48c3f07..8aa5a88ccb19 100644 > --- a/tools/testing/selftests/tc-testing/tc-tests/actions/skbmod.json > +++ b/tools/testing/selftests/tc-testing/tc-tests/actions/skbmod.json ... > } > -] > +] > \ No newline at end of file > -- > 2.7.4 Please fix this.
[RESEND PATCH net-next 1/1] tc-testing: updated police, mirred, skbedit and skbmod with more tests
Added extra test cases for control actions (reclassify, pipe etc.), cookies, max index value and police args sanity check. Signed-off-by: Roman Mashak --- .../tc-testing/tc-tests/actions/mirred.json| 192 + .../tc-testing/tc-tests/actions/police.json| 144 .../tc-testing/tc-tests/actions/skbedit.json | 168 ++ .../tc-testing/tc-tests/actions/skbmod.json| 26 ++- 4 files changed, 529 insertions(+), 1 deletion(-) diff --git a/tools/testing/selftests/tc-testing/tc-tests/actions/mirred.json b/tools/testing/selftests/tc-testing/tc-tests/actions/mirred.json index 0fcccf18399b..443c9b3c8664 100644 --- a/tools/testing/selftests/tc-testing/tc-tests/actions/mirred.json +++ b/tools/testing/selftests/tc-testing/tc-tests/actions/mirred.json @@ -171,6 +171,198 @@ ] }, { +"id": "8917", +"name": "Add mirred mirror action with control pass", +"category": [ +"actions", +"mirred" +], +"setup": [ +[ +"$TC actions flush action mirred", +0, +1, +255 +] +], +"cmdUnderTest": "$TC actions add action mirred ingress mirror dev lo pass index 1", +"expExitCode": "0", +"verifyCmd": "$TC actions get action mirred index 1", +"matchPattern": "action order [0-9]*: mirred \\(Ingress Mirror to device lo\\) pass.*index 1 ref", +"matchCount": "1", +"teardown": [ +"$TC actions flush action mirred" +] +}, +{ +"id": "1054", +"name": "Add mirred mirror action with control pipe", +"category": [ +"actions", +"mirred" +], +"setup": [ +[ +"$TC actions flush action mirred", +0, +1, +255 +] +], +"cmdUnderTest": "$TC actions add action mirred ingress mirror dev lo pipe index 15", +"expExitCode": "0", +"verifyCmd": "$TC actions get action mirred index 15", +"matchPattern": "action order [0-9]*: mirred \\(Ingress Mirror to device lo\\) pipe.*index 15 ref", +"matchCount": "1", +"teardown": [ +"$TC actions flush action mirred" +] +}, +{ +"id": "9887", +"name": "Add mirred mirror action with control continue", +"category": [ +"actions", +"mirred" +], +"setup": [ +[ +"$TC actions flush action mirred", +0, +1, +255 +] +], +"cmdUnderTest": "$TC actions add action mirred ingress mirror dev lo continue index 15", +"expExitCode": "0", +"verifyCmd": "$TC actions get action mirred index 15", +"matchPattern": "action order [0-9]*: mirred \\(Ingress Mirror to device lo\\) continue.*index 15 ref", +"matchCount": "1", +"teardown": [ +"$TC actions flush action mirred" +] +}, +{ +"id": "e4aa", +"name": "Add mirred mirror action with control reclassify", +"category": [ +"actions", +"mirred" +], +"setup": [ +[ +"$TC actions flush action mirred", +0, +1, +255 +] +], +"cmdUnderTest": "$TC actions add action mirred ingress mirror dev lo reclassify index 150", +"expExitCode": "0", +"verifyCmd": "$TC actions get action mirred index 150", +"matchPattern": "action order [0-9]*: mirred \\(Ingress Mirror to device lo\\) reclassify.*index 150 ref", +"matchCount": "1", +"teardown": [ +"$TC actions flush action mirred" +] +}, +{ +"id": "ece9", +"name": "Add mirred mirror action with control drop", +"category": [ +"actions", +"mirred" +], +"setup": [ +[ +"$TC actions flush action mirred", +0, +1, +255 +] +], +"cmdUnderTest": "$TC actions add action mirred ingress mirror dev lo drop index 99", +"expExitCode": "0", +"verifyCmd": "$TC actions get action mirred index 99", +"matchPattern": "action order [0-9]*: mirred \\(Ingress Mirror to device lo\\) drop.*index 99 ref", +"matchCount": "1", +"teardown": [ +"$TC actions flush action mirred" +] +}, +{ +"id": "0031", +"name": "Add mirred mirror action with control jump", +"category": [ +"actions", +"mirred" +], +"setup": [ +[ +"$TC actions flush action mirred", +0,
[PATCH net-next 1/1] tc-testing: updated police, mirred, skbedit and skbmod with more tests
Added extra test cases for control actions (reclassify, pipe etc.), cookies, max index value and police args sanity check. Signed-off-by: Roman Mashak --- .../tc-testing/tc-tests/actions/mirred.json| 192 + .../tc-testing/tc-tests/actions/police.json| 144 .../tc-testing/tc-tests/actions/skbedit.json | 168 ++ .../tc-testing/tc-tests/actions/skbmod.json| 26 ++- 4 files changed, 529 insertions(+), 1 deletion(-) diff --git a/tools/testing/selftests/tc-testing/tc-tests/actions/mirred.json b/tools/testing/selftests/tc-testing/tc-tests/actions/mirred.json index 0fcccf18399b..443c9b3c8664 100644 --- a/tools/testing/selftests/tc-testing/tc-tests/actions/mirred.json +++ b/tools/testing/selftests/tc-testing/tc-tests/actions/mirred.json @@ -171,6 +171,198 @@ ] }, { +"id": "8917", +"name": "Add mirred mirror action with control pass", +"category": [ +"actions", +"mirred" +], +"setup": [ +[ +"$TC actions flush action mirred", +0, +1, +255 +] +], +"cmdUnderTest": "$TC actions add action mirred ingress mirror dev lo pass index 1", +"expExitCode": "0", +"verifyCmd": "$TC actions get action mirred index 1", +"matchPattern": "action order [0-9]*: mirred \\(Ingress Mirror to device lo\\) pass.*index 1 ref", +"matchCount": "1", +"teardown": [ +"$TC actions flush action mirred" +] +}, +{ +"id": "1054", +"name": "Add mirred mirror action with control pipe", +"category": [ +"actions", +"mirred" +], +"setup": [ +[ +"$TC actions flush action mirred", +0, +1, +255 +] +], +"cmdUnderTest": "$TC actions add action mirred ingress mirror dev lo pipe index 15", +"expExitCode": "0", +"verifyCmd": "$TC actions get action mirred index 15", +"matchPattern": "action order [0-9]*: mirred \\(Ingress Mirror to device lo\\) pipe.*index 15 ref", +"matchCount": "1", +"teardown": [ +"$TC actions flush action mirred" +] +}, +{ +"id": "9887", +"name": "Add mirred mirror action with control continue", +"category": [ +"actions", +"mirred" +], +"setup": [ +[ +"$TC actions flush action mirred", +0, +1, +255 +] +], +"cmdUnderTest": "$TC actions add action mirred ingress mirror dev lo continue index 15", +"expExitCode": "0", +"verifyCmd": "$TC actions get action mirred index 15", +"matchPattern": "action order [0-9]*: mirred \\(Ingress Mirror to device lo\\) continue.*index 15 ref", +"matchCount": "1", +"teardown": [ +"$TC actions flush action mirred" +] +}, +{ +"id": "e4aa", +"name": "Add mirred mirror action with control reclassify", +"category": [ +"actions", +"mirred" +], +"setup": [ +[ +"$TC actions flush action mirred", +0, +1, +255 +] +], +"cmdUnderTest": "$TC actions add action mirred ingress mirror dev lo reclassify index 150", +"expExitCode": "0", +"verifyCmd": "$TC actions get action mirred index 150", +"matchPattern": "action order [0-9]*: mirred \\(Ingress Mirror to device lo\\) reclassify.*index 150 ref", +"matchCount": "1", +"teardown": [ +"$TC actions flush action mirred" +] +}, +{ +"id": "ece9", +"name": "Add mirred mirror action with control drop", +"category": [ +"actions", +"mirred" +], +"setup": [ +[ +"$TC actions flush action mirred", +0, +1, +255 +] +], +"cmdUnderTest": "$TC actions add action mirred ingress mirror dev lo drop index 99", +"expExitCode": "0", +"verifyCmd": "$TC actions get action mirred index 99", +"matchPattern": "action order [0-9]*: mirred \\(Ingress Mirror to device lo\\) drop.*index 99 ref", +"matchCount": "1", +"teardown": [ +"$TC actions flush action mirred" +] +}, +{ +"id": "0031", +"name": "Add mirred mirror action with control jump", +"category": [ +"actions", +"mirred" +], +"setup": [ +[ +"$TC actions flush action mirred", +0,