All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 net-next] selftests: net: dsa: symlink the tc_actions.sh test
@ 2022-08-31 17:08 Vladimir Oltean
  2022-09-02  2:06 ` Florian Fainelli
  2022-09-02  4:00 ` patchwork-bot+netdevbpf
  0 siblings, 2 replies; 3+ messages in thread
From: Vladimir Oltean @ 2022-08-31 17:08 UTC (permalink / raw)
  To: netdev
  Cc: Jakub Kicinski, David S. Miller, Paolo Abeni, Eric Dumazet,
	Florian Fainelli, Vivien Didelot, Andrew Lunn, Vladimir Oltean,
	Claudiu Manoil, Alexandre Belloni, UNGLinuxDriver,
	Xiaoliang Yang, Colin Foster, Martin Blumenstingl

This has been validated on the Ocelot/Felix switch family (NXP LS1028A)
and should be relevant to any switch driver that offloads the tc-flower
and/or tc-matchall actions trap, drop, accept, mirred, for which DSA has
operations.

TEST: gact drop and ok (skip_hw)                                    [ OK ]
TEST: mirred egress flower redirect (skip_hw)                       [ OK ]
TEST: mirred egress flower mirror (skip_hw)                         [ OK ]
TEST: mirred egress matchall mirror (skip_hw)                       [ OK ]
TEST: mirred_egress_to_ingress (skip_hw)                            [ OK ]
TEST: gact drop and ok (skip_sw)                                    [ OK ]
TEST: mirred egress flower redirect (skip_sw)                       [ OK ]
TEST: mirred egress flower mirror (skip_sw)                         [ OK ]
TEST: mirred egress matchall mirror (skip_sw)                       [ OK ]
TEST: trap (skip_sw)                                                [ OK ]
TEST: mirred_egress_to_ingress (skip_sw)                            [ OK ]

Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
---
v1->v2:
- also change the Makefile to install the symlinks

Previous version at
https://patchwork.kernel.org/project/netdevbpf/patch/20220503120804.840463-4-vladimir.oltean@nxp.com/
was part of a series that did not apply cleanly to net-next at the time.
The other patches were resent and accepted, but not this one, it seems.

 tools/testing/selftests/drivers/net/dsa/Makefile      | 3 ++-
 tools/testing/selftests/drivers/net/dsa/tc_actions.sh | 1 +
 tools/testing/selftests/drivers/net/dsa/tc_common.sh  | 1 +
 3 files changed, 4 insertions(+), 1 deletion(-)
 create mode 120000 tools/testing/selftests/drivers/net/dsa/tc_actions.sh
 create mode 120000 tools/testing/selftests/drivers/net/dsa/tc_common.sh

diff --git a/tools/testing/selftests/drivers/net/dsa/Makefile b/tools/testing/selftests/drivers/net/dsa/Makefile
index 2a731d5c6d85..c393e7b73805 100644
--- a/tools/testing/selftests/drivers/net/dsa/Makefile
+++ b/tools/testing/selftests/drivers/net/dsa/Makefile
@@ -8,9 +8,10 @@ TEST_PROGS = bridge_locked_port.sh \
 	bridge_vlan_unaware.sh \
 	local_termination.sh \
 	no_forwarding.sh \
+	tc_actions.sh \
 	test_bridge_fdb_stress.sh
 
-TEST_PROGS_EXTENDED := lib.sh
+TEST_PROGS_EXTENDED := lib.sh tc_common.sh
 
 TEST_FILES := forwarding.config
 
diff --git a/tools/testing/selftests/drivers/net/dsa/tc_actions.sh b/tools/testing/selftests/drivers/net/dsa/tc_actions.sh
new file mode 120000
index 000000000000..306213d9430e
--- /dev/null
+++ b/tools/testing/selftests/drivers/net/dsa/tc_actions.sh
@@ -0,0 +1 @@
+../../../net/forwarding/tc_actions.sh
\ No newline at end of file
diff --git a/tools/testing/selftests/drivers/net/dsa/tc_common.sh b/tools/testing/selftests/drivers/net/dsa/tc_common.sh
new file mode 120000
index 000000000000..bc3465bdc36b
--- /dev/null
+++ b/tools/testing/selftests/drivers/net/dsa/tc_common.sh
@@ -0,0 +1 @@
+../../../net/forwarding/tc_common.sh
\ No newline at end of file
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH v2 net-next] selftests: net: dsa: symlink the tc_actions.sh test
  2022-08-31 17:08 [PATCH v2 net-next] selftests: net: dsa: symlink the tc_actions.sh test Vladimir Oltean
@ 2022-09-02  2:06 ` Florian Fainelli
  2022-09-02  4:00 ` patchwork-bot+netdevbpf
  1 sibling, 0 replies; 3+ messages in thread
From: Florian Fainelli @ 2022-09-02  2:06 UTC (permalink / raw)
  To: Vladimir Oltean, netdev
  Cc: Jakub Kicinski, David S. Miller, Paolo Abeni, Eric Dumazet,
	Vivien Didelot, Andrew Lunn, Vladimir Oltean, Claudiu Manoil,
	Alexandre Belloni, UNGLinuxDriver, Xiaoliang Yang, Colin Foster,
	Martin Blumenstingl



On 8/31/2022 10:08 AM, Vladimir Oltean wrote:
> This has been validated on the Ocelot/Felix switch family (NXP LS1028A)
> and should be relevant to any switch driver that offloads the tc-flower
> and/or tc-matchall actions trap, drop, accept, mirred, for which DSA has
> operations.
> 
> TEST: gact drop and ok (skip_hw)                                    [ OK ]
> TEST: mirred egress flower redirect (skip_hw)                       [ OK ]
> TEST: mirred egress flower mirror (skip_hw)                         [ OK ]
> TEST: mirred egress matchall mirror (skip_hw)                       [ OK ]
> TEST: mirred_egress_to_ingress (skip_hw)                            [ OK ]
> TEST: gact drop and ok (skip_sw)                                    [ OK ]
> TEST: mirred egress flower redirect (skip_sw)                       [ OK ]
> TEST: mirred egress flower mirror (skip_sw)                         [ OK ]
> TEST: mirred egress matchall mirror (skip_sw)                       [ OK ]
> TEST: trap (skip_sw)                                                [ OK ]
> TEST: mirred_egress_to_ingress (skip_sw)                            [ OK ]
> 
> Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>

Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
-- 
Florian

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v2 net-next] selftests: net: dsa: symlink the tc_actions.sh test
  2022-08-31 17:08 [PATCH v2 net-next] selftests: net: dsa: symlink the tc_actions.sh test Vladimir Oltean
  2022-09-02  2:06 ` Florian Fainelli
@ 2022-09-02  4:00 ` patchwork-bot+netdevbpf
  1 sibling, 0 replies; 3+ messages in thread
From: patchwork-bot+netdevbpf @ 2022-09-02  4:00 UTC (permalink / raw)
  To: Vladimir Oltean
  Cc: netdev, kuba, davem, pabeni, edumazet, f.fainelli,
	vivien.didelot, andrew, olteanv, claudiu.manoil,
	alexandre.belloni, UNGLinuxDriver, xiaoliang.yang_1,
	colin.foster, martin.blumenstingl

Hello:

This patch was applied to netdev/net-next.git (master)
by Jakub Kicinski <kuba@kernel.org>:

On Wed, 31 Aug 2022 20:08:39 +0300 you wrote:
> This has been validated on the Ocelot/Felix switch family (NXP LS1028A)
> and should be relevant to any switch driver that offloads the tc-flower
> and/or tc-matchall actions trap, drop, accept, mirred, for which DSA has
> operations.
> 
> TEST: gact drop and ok (skip_hw)                                    [ OK ]
> TEST: mirred egress flower redirect (skip_hw)                       [ OK ]
> TEST: mirred egress flower mirror (skip_hw)                         [ OK ]
> TEST: mirred egress matchall mirror (skip_hw)                       [ OK ]
> TEST: mirred_egress_to_ingress (skip_hw)                            [ OK ]
> TEST: gact drop and ok (skip_sw)                                    [ OK ]
> TEST: mirred egress flower redirect (skip_sw)                       [ OK ]
> TEST: mirred egress flower mirror (skip_sw)                         [ OK ]
> TEST: mirred egress matchall mirror (skip_sw)                       [ OK ]
> TEST: trap (skip_sw)                                                [ OK ]
> TEST: mirred_egress_to_ingress (skip_sw)                            [ OK ]
> 
> [...]

Here is the summary with links:
  - [v2,net-next] selftests: net: dsa: symlink the tc_actions.sh test
    https://git.kernel.org/netdev/net-next/c/1ab3d4175775

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-09-02  4:00 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-08-31 17:08 [PATCH v2 net-next] selftests: net: dsa: symlink the tc_actions.sh test Vladimir Oltean
2022-09-02  2:06 ` Florian Fainelli
2022-09-02  4:00 ` patchwork-bot+netdevbpf

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.