From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     netdev@vger.kernel.org
Cc:     "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        UNGLinuxDriver@microchip.com, linux-kernel@vger.kernel.org,
        Andrew Lunn <andrew@lunn.ch>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Richie Pearn <richard.pearn@nxp.com>,
        Xiaoliang Yang <xiaoliang.yang_1@nxp.com>
Subject: [PATCH net 2/2] selftests: ocelot: tc_flower_chains: make test_vlan_ingress_modify() more comprehensive
Date:   Sun,  5 Feb 2023 21:24:09 +0200
Message-Id: <20230205192409.1796428-2-vladimir.oltean@nxp.com>
In-Reply-To: <20230205192409.1796428-1-vladimir.oltean@nxp.com>
References: <20230205192409.1796428-1-vladimir.oltean@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?B7jmWD+gXNOe26vSjK8WLEgQf0tpArf8Ewk19DEmfkLQECqAFUelcTwZPthR?=
 =?us-ascii?Q?94Ak2VfLwWRrHLpOMDUYsHAZp9/8xveg6OUPKP6L8BrjItDTPFJGQWPyT23F?=
 =?us-ascii?Q?h8jGkY8L/98P7Ub3kjW3Y31Of2h7HghObK9DZGnZbT6eYGYFK8EEXIXSf28C?=
 =?us-ascii?Q?O2rw3I0/4MEQOErEqttcJrD84mt9Sdp9Jqs2ukQxSX+eBPXxQqdNUT/y2MoU?=
 =?us-ascii?Q?K8P2I0BVoWG0rOZCldMVSyXwMElo81gFDzcRt2ujn/ZiwfXRZu4w8sU+i5CI?=
 =?us-ascii?Q?kSChNRymjV60gWSJNgFPusmCaN8YLeOxp7KsGTvqreyGsvgbqkukJCUQhuGX?=
 =?us-ascii?Q?pbEN0OdTdcdlyjHBqnGic8SWI0Q7LWgHER4xJKHqwnQkeaHnq062xQvyDMuT?=
 =?us-ascii?Q?tKSSMQWPRkIclgW80v3N2YPMx+CrrDtcTPfuoDWp22MsL9kowtt5FVXe3nmX?=
 =?us-ascii?Q?IZnLdcPJHq6IKbVnmDQN7PsX0zwIXAUo1aeN8kC2ePXXlE/yOSchJS+V5vzI?=
 =?us-ascii?Q?tYNzNhGOvsDtXWRywJfH3errvRehMKrkneXwitvsVqKFozyswVMmfYpc1Y7F?=
 =?us-ascii?Q?omwcSgqvuF3Qg7SXv9HvqxoUVR0P/el2K8BmVknqOWF3+EijtrBp529Z81hd?=
 =?us-ascii?Q?SG7bG651MmdkJldmHFKEiEpb/1vlbSLSlVgR/trC9CGxwCGCnDO0fQSsLWPh?=
 =?us-ascii?Q?fUJy4Bpy7kRa2E1dMoi5wDAujlOwc1gQL9pp7TjyvL7QjGZO2iOe+kQS2UcX?=
 =?us-ascii?Q?7kcbGPJaolsKPFC/1JcNDzT7taihoXZYwQDd2zyPYFBRyUQLlKZDKHen4cQs?=
 =?us-ascii?Q?YE4xG2JP0KGA2T3BbNoQ6eI3peQCtb0q6OahuMyvpp7TrHlzmgiNZLvza+6W?=
 =?us-ascii?Q?HpqbVd48bbL/pFT0g1h328pIaBpbMGt8Iw26ixaGQS3yksRrkRh1AUfmQLZ8?=
 =?us-ascii?Q?W6lqtcBVVr9bMYuIanI76dNqWpCUCdus3x/ZQzkfJPm0jYS95tCEIGhIoQOL?=
 =?us-ascii?Q?5FT4fR/nQgFAM+Q81a2ReC7+5AzOE/jWzgBNKmSyDVmTlGqNkg082ypRz7PL?=
 =?us-ascii?Q?xjxBMiVXgONx7SioR5Lbf7anHJcng937ZooBCLpmqcZEPwVPHvmXvWO5YdT1?=
 =?us-ascii?Q?PcEltA8B8AD2eOCo8WQczblR6FUtb2eQiCNXJfuATldteGbfSk0dJH+Ix5mQ?=
 =?us-ascii?Q?AjxD0ITBMEG6Ex57Et987H9zWXeX/eUOwR4pL4pRMONXFrlliJpKzxOI03ob?=
 =?us-ascii?Q?hmKcvrMqqBNaYDXyHsUDVTyNiyk9LZYIo68+UqvueAGlkhzOb+GVmkT19uSn?=
 =?us-ascii?Q?Sww/hGrTlshankrruKeOeJ20S8gMID/Wv6SQPExNjbmQjmp4btqpB4sxF284?=
 =?us-ascii?Q?FUAHNKjkdftqnFJ8NnybeKYmHY/B7TnAP0iO+KbmSgNBNilas+y++D1yR6AF?=
 =?us-ascii?Q?9Nez0yJTDN1tkm46CjUqEbQTKZnrQ9bwlEvmJazTIqUC5Y10f7HdHixhqhNx?=
 =?us-ascii?Q?OHNnfhp0dO2ROBzsB5uOjQQRvDkpygnc/R5iFU1FVUXD5y29M6GpXiprgouX?=
 =?us-ascii?Q?x7/J1ZGHUuMqN/Tp4wNEXev3b8V+A0bIqDZC/vOwW8Y2bRHlkdXzbWdWZL8Q?=
 =?us-ascii?Q?cQ=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: aac7520a-c306-4c21-0113-08db07ae978c
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5136.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Feb 2023 19:24:25.8014
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 4ElCJRPJVMK6N1Cnwyr7++tydJfXnMPWonRna7WH2UvVzWZpI/UwD3P6d/8YaWTCaIuhz05IS7iooAVIJ9GxqQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXPR04MB8391
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

We have two IS1 filters of the OCELOT_VCAP_KEY_ANY key type (the one with
"action vlan pop" and the one with "action vlan modify") and one of the
OCELOT_VCAP_KEY_IPV4 key type (the one with "action skbedit priority").
But we have no IS1 filter with the OCELOT_VCAP_KEY_ETYPE key type, and
there was an uncaught breakage there.

To increase test coverage, convert one of the OCELOT_VCAP_KEY_ANY
filters to OCELOT_VCAP_KEY_ETYPE, by making the filter also match on the
MAC SA of the traffic sent by mausezahn, $h1_mac.

Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
---
 tools/testing/selftests/drivers/net/ocelot/tc_flower_chains.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/testing/selftests/drivers/net/ocelot/tc_flower_chains.sh b/tools/testing/selftests/drivers/net/ocelot/tc_flower_chains.sh
index 9c79bbcce5a8..aff0a59f92d9 100755
--- a/tools/testing/selftests/drivers/net/ocelot/tc_flower_chains.sh
+++ b/tools/testing/selftests/drivers/net/ocelot/tc_flower_chains.sh
@@ -246,7 +246,7 @@ test_vlan_ingress_modify()
 	bridge vlan add dev $swp2 vid 300
 
 	tc filter add dev $swp1 ingress chain $(IS1 2) pref 3 \
-		protocol 802.1Q flower skip_sw vlan_id 200 \
+		protocol 802.1Q flower skip_sw vlan_id 200 src_mac $h1_mac \
 		action vlan modify id 300 \
 		action goto chain $(IS2 0 0)
 
-- 
2.34.1