From patchwork Thu Jun 8 03:19:53 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Chaoyong He X-Patchwork-Id: 128365 X-Patchwork-Delegate: ferruh.yigit@amd.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 2FA1B42C57; Thu, 8 Jun 2023 05:20:31 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 24C2642D0C; Thu, 8 Jun 2023 05:20:28 +0200 (CEST) Received: from NAM04-BN8-obe.outbound.protection.outlook.com (mail-bn8nam04on2127.outbound.protection.outlook.com [40.107.100.127]) by mails.dpdk.org (Postfix) with ESMTP id 4B33040042; Thu, 8 Jun 2023 05:20:25 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=bB1Ssc+rTN/A5H1/VYozVUx1iEcpvZs6jQxCLDTkBFVZ+BmvfpVn9NpZxf3lwUbhYB7mxud7r1c6M42CCBvoDoqMSG7jN88El1YiKdGNj3f3tUnlwOQd5cCnTyIauAMbI+Ce7ebvBtHaptYHt5t5yJjhMJwKfuoyvc5BITDxLon/v+CfTcoYTz+j+399MBJLt7zx+8OD6Fbpp3+v3GAzRUavRLTP8K64IwqEpnYJ89qkjRMPpVHK/7Lz5hKISDrWfV5LXEj4ppfhgsK2YSiOhyUl3YtWd2i7a8a+QMOSZlvsl0mnRveWNWklWGEl1RXGUXTmmEo3Bvolt2K9a2izfg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=0Q2Xdo2u0gig00QncctFa3GGSk5nKebplIS1zvlTllk=; b=Df0K/vJnqUB7geHELuUE/j/sYgCIwC+pn+YUXhrckTSNsB5v4Cbtc6QkkIsPS8xAeQWcIBijIXODl90GVT6RTl4O7et6eGCfl7bD5IBHtKukgRVCc40bpOOpRMMuBFpirGk16PibXF1hTcqBhxqxkMllLlX5qKz5/9aEf8UGACGM+3zlzVAhJ8oFGcRHk3NkF/tELNwB5WRrf3reHoUBnMdppGTAnNJKHHav6NOOeGItb15ZRffcZHvApUAJF1VlkCJ9VIDQogBpCOzTHCqr8epC2sQ8BllRrwbfUAGxU36mYeZyot4kVoErKOJLoxRmqKLPzT5kc9ufNQ4IZ2KyDw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=corigine.com; dmarc=pass action=none header.from=corigine.com; dkim=pass header.d=corigine.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=corigine.onmicrosoft.com; s=selector2-corigine-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=0Q2Xdo2u0gig00QncctFa3GGSk5nKebplIS1zvlTllk=; b=pTp0mrbgafS2yqvOepcJLW+9R4GQNU84FoBnVxCI8yMvrH9zLrO9hlsLA1v2oRLbL3tYH/l7yFWHmafzFmZSPpQtBB4OGBpPa/zzOwx5Y/a/tSrVIfOGjeTmB31cUywzSnICNFQBY87iaXjpuyflu8hV55fJ2LoGoXsJE7td0Cc= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=corigine.com; Received: from SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) by DM6PR13MB4432.namprd13.prod.outlook.com (2603:10b6:5:1b8::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6455.32; Thu, 8 Jun 2023 03:20:22 +0000 Received: from SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::d2cb:a650:e1ce:2174]) by SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::d2cb:a650:e1ce:2174%3]) with mapi id 15.20.6455.030; Thu, 8 Jun 2023 03:20:22 +0000 From: Chaoyong He To: dev@dpdk.org Cc: oss-drivers@corigine.com, niklas.soderlund@corigine.com, Chaoyong He , stable@dpdk.org Subject: [PATCH 1/4] net/nfp: fix the IPv6 flow item Date: Thu, 8 Jun 2023 11:19:53 +0800 Message-Id: <20230608031956.814684-2-chaoyong.he@corigine.com> X-Mailer: git-send-email 2.39.1 In-Reply-To: <20230608031956.814684-1-chaoyong.he@corigine.com> References: <20230608031956.814684-1-chaoyong.he@corigine.com> X-ClientProxiedBy: SI2PR02CA0038.apcprd02.prod.outlook.com (2603:1096:4:196::22) To SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ0PR13MB5545:EE_|DM6PR13MB4432:EE_ X-MS-Office365-Filtering-Correlation-Id: 8b762e8f-3682-4c5a-d415-08db67cf4b38 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: sZ4oBelpCKZYkiyO0d0x00ZSA1kjZoRos1EFUWNVze2fx58q4hbrAinIsLxLs3CmzEh5Uv7hQYZE29YU4CRDeYcOj5A0uHPZ8NDWJDOfrRvnoQgsXvDkUR1hKT7r5YiyxJH/Asa2kUSS6djcoxX6Z2G/Td2fefAqwXg45dUiMGYi3MwK07xvyHtgoxTR+ItM21yy5LQHTKK4HuYy7QxvlK/E1GG+8M5etE6tzUBztLe9tSfHM2929mnD38fGYbRQKBXb7GQWX6y8WHzWUH43hAXnCY1OFJN33Zsu8Id/jKCpk6dhbpp2BJw5+p9TKaAMaGDuKgop4YAB3+IPzIXVfpP/guqzL1/BPDfXw1Sm+hUrKKCR7Ojbd5fntYaLvge3SJRiSfZPeWDUFFwkw3dBsdtWPirKWmOtKXJFmRXuH0B3Y5OO5mF8bIjl5dbhoTWiN9Q6aNemQvxPjs5A/NQlK95FBJjLQO3bXZN6CtiH1bIyBuRP4HtcmcA2mGu3c8qTSbjIARVEQmTBvj+IkEOg1o1hgJMoUYJCrkBb7Wp+QbQ9gjKfEzGwGHvw4wCPM0bfTJmIY4f821HcXI/IAuxV98ie1NLuvDBC+Gr3Q0wlrMMFl3gFe0E8IouisagXfvb5 X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:SJ0PR13MB5545.namprd13.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230028)(4636009)(39840400004)(366004)(136003)(346002)(376002)(396003)(451199021)(478600001)(2906002)(2616005)(83380400001)(6506007)(1076003)(26005)(186003)(6512007)(86362001)(38100700002)(38350700002)(52116002)(6486002)(36756003)(6666004)(5660300002)(450100002)(8936002)(8676002)(41300700001)(66476007)(6916009)(4326008)(66946007)(66556008)(316002)(44832011); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?33DCf5w6h8WfsMbH4xtXXLmgqnJq?= =?utf-8?q?otknpyh3rDZL/KQwxofJeLwZmWskm+n3Ef0Z1SAsLVLVUrhnLS3/IuzaQIO/Jw7On?= =?utf-8?q?C75MFHyFsZMOKfDyTYYM2Z/n7nzz0YZ9h/EPH7OI0HWn1rKZamoUGISH/JDmfZXJq?= =?utf-8?q?sMQ8Go6uVeoidznpdhrl3kdUTxR9c7Qm2z6oFnOfTL3C/LJ6VaqUh0Va7LTseqRgW?= =?utf-8?q?GWe31wqCje4GNAu+OoKJnKYVkNqvVUfdj1eJ6Us1UyraZtceqYSZJxvcBCL5lOask?= =?utf-8?q?n+n+vCPJI90b0CBgDm2QhcL0e5LaXdEcocAgsoEnbJMnJvjPVd3o9rl84AkPodTPe?= =?utf-8?q?a2nw7XrN4HWvk1yAI7GTbIL+S+E2xXZES+/3JwYKxuLK9fAMorr00drjNEffud2DH?= =?utf-8?q?/OVS1JUWA4FJ7fU++WehIHJoVi5gTsUthHeR9uPjGericu7O1tySP8j136F2lvVK8?= =?utf-8?q?T8Xyp5IuZBVSBa6Qs4FZMqFgyngV5aliV7q/X77ah4ldsMM6/X83+guykpSwMLjTQ?= =?utf-8?q?HAx1XVctMlWJHmiYaf5/6lXYk+KEhStmo3VVwb6QvZjh/VNd9slBMG9occvyl0fHT?= =?utf-8?q?wWjY/8yezCYQ1jpFLtknJxiC8lM0Xu9eotLQrTXiotJcSG1jn3tMrOoYK+j/fAzIb?= =?utf-8?q?gAYCRthdYs5feT+mm0/YNUNIkqoUKBmt0QdS8jHcAHE4G6pt9thZqQI6IYhROQ93o?= =?utf-8?q?l5H/hNC0DFOYCq17EJKSSE9NvOiVz/kWD2YvfcEjfou4rpKb/oT/9O3uZ6BDnpG3U?= =?utf-8?q?KlbQgX48092vsjeWbPU1inkkWKRWBLIZ6AKk3wgZ7ggUFky7zav5/JLChT7lSw5tC?= =?utf-8?q?uBIhQqOmdUjRQGNotDih6MNjCUXdduyAN3pzt5kcK8ZE4zHeSR3ebThVUQHf3CiOI?= =?utf-8?q?cNa63yHHGOM+YewVUjyKg8vC+24s9v7VuFFKNcZpnfpWmoXg5SHMetoq9oq2xNxc8?= =?utf-8?q?vH4wIgppKHKNaphR1aMP1g64fKMiKJgsJ5YRdj5gJwAmqprK9r2k5RmWtsgvHc+sP?= =?utf-8?q?OlGYF5x3bsLS/d7t3UDQQEtL2OTZ+2JDBViCKb1RIC3Hpppuka4+3FePr7EwI8nND?= =?utf-8?q?YahT5Ym1G0i/+nRfckDfkgtaL0XKFLD9iG7k39Kb6MGjW8pesA7Hv46Mr59CkDD2D?= =?utf-8?q?AJwWcLq2e2GfAnYjUkU1vCKv9XyZ4x3keYvyK7hyNvE/bj8qD2K2+HpvFBU+0FInC?= =?utf-8?q?+MIFLc4TqD5y7lun5QfeDjrfsmfRfk/TALHwmr8obXRBbe7dbKeNprpTeXhRJHyu+?= =?utf-8?q?H0ZfRSnyGtl9d14nsaDpHGWbyU0ohu6QDK3u8g0Wy0Gs06sbQ9YdAlqDi+AZqapX5?= =?utf-8?q?fS9w9R1b/2uzfGzOZKjwm04Wi2f19qlCWT8iKlEoJWz6hhsGWLvxOYlvxtkVI7aFm?= =?utf-8?q?qN+xLWJONInMNLAGu+mOrQEuTO1hBCqhGoxYEMfO9NGUH5PQFG7Is2xwbEAHu0X5b?= =?utf-8?q?euFwivTz2WB7UkqXqjRjj0qieBBSSck2TgQUOxoAhvnPYDGFXHmYh5EqWha0TfQQR?= =?utf-8?q?jixGUVB1Uiw5s5SDzaJQt4BzxrMf7iPh1g=3D=3D?= X-OriginatorOrg: corigine.com X-MS-Exchange-CrossTenant-Network-Message-Id: 8b762e8f-3682-4c5a-d415-08db67cf4b38 X-MS-Exchange-CrossTenant-AuthSource: SJ0PR13MB5545.namprd13.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Jun 2023 03:20:22.3126 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: fe128f2c-073b-4c20-818e-7246a585940c X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: hDFT50j0efSPqRylG+6FJjpPyUmvYTJpEKRkXZN3Isxy7a8pqiQA/kjAGaodI2VfpQdTrUdIMv4hY1D1x2CLbqRRFp3cobcaR0whqvd5DP0= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR13MB4432 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org When process the IPv6 flow item, the former logic perform the mask operation between one big endian value and one CPU endian valude, which cause the match of DSCP filed not work as expect. Fix it by convert the big endian data into CPU endian and remove the unneeded mask operation. Fixes: d96507404463 ("net/nfp: support IPv6 flow item") Cc: stable@dpdk.org Signed-off-by: Chaoyong He Reviewed-by: Niklas Söderlund --- drivers/net/nfp/nfp_flow.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/net/nfp/nfp_flow.c b/drivers/net/nfp/nfp_flow.c index 41b722f4d8..937fb390df 100644 --- a/drivers/net/nfp/nfp_flow.c +++ b/drivers/net/nfp/nfp_flow.c @@ -1232,6 +1232,7 @@ nfp_flow_merge_ipv6(__rte_unused struct nfp_app_fw_flower *app_fw_flower, bool is_mask, bool is_outer_layer) { + uint32_t vtc_flow; struct nfp_flower_ipv6 *ipv6; const struct rte_ipv6_hdr *hdr; struct nfp_flower_meta_tci *meta_tci; @@ -1255,12 +1256,12 @@ nfp_flow_merge_ipv6(__rte_unused struct nfp_app_fw_flower *app_fw_flower, hdr = is_mask ? &mask->hdr : &spec->hdr; + vtc_flow = rte_be_to_cpu_32(hdr->vtc_flow); if (ext_meta && (rte_be_to_cpu_32(ext_meta->nfp_flow_key_layer2) & NFP_FLOWER_LAYER2_GRE)) { ipv6_gre_tun = (struct nfp_flower_ipv6_gre_tun *)*mbuf_off; - ipv6_gre_tun->ip_ext.tos = (hdr->vtc_flow & - RTE_IPV6_HDR_TC_MASK) >> RTE_IPV6_HDR_TC_SHIFT; + ipv6_gre_tun->ip_ext.tos = vtc_flow >> RTE_IPV6_HDR_TC_SHIFT; ipv6_gre_tun->ip_ext.ttl = hdr->hop_limits; memcpy(ipv6_gre_tun->ipv6.ipv6_src, hdr->src_addr, sizeof(ipv6_gre_tun->ipv6.ipv6_src)); @@ -1269,8 +1270,7 @@ nfp_flow_merge_ipv6(__rte_unused struct nfp_app_fw_flower *app_fw_flower, } else { ipv6_udp_tun = (struct nfp_flower_ipv6_udp_tun *)*mbuf_off; - ipv6_udp_tun->ip_ext.tos = (hdr->vtc_flow & - RTE_IPV6_HDR_TC_MASK) >> RTE_IPV6_HDR_TC_SHIFT; + ipv6_udp_tun->ip_ext.tos = vtc_flow >> RTE_IPV6_HDR_TC_SHIFT; ipv6_udp_tun->ip_ext.ttl = hdr->hop_limits; memcpy(ipv6_udp_tun->ipv6.ipv6_src, hdr->src_addr, sizeof(ipv6_udp_tun->ipv6.ipv6_src)); @@ -1291,10 +1291,10 @@ nfp_flow_merge_ipv6(__rte_unused struct nfp_app_fw_flower *app_fw_flower, *mbuf_off += sizeof(struct nfp_flower_tp_ports); hdr = is_mask ? &mask->hdr : &spec->hdr; + vtc_flow = rte_be_to_cpu_32(hdr->vtc_flow); ipv6 = (struct nfp_flower_ipv6 *)*mbuf_off; - ipv6->ip_ext.tos = (hdr->vtc_flow & RTE_IPV6_HDR_TC_MASK) >> - RTE_IPV6_HDR_TC_SHIFT; + ipv6->ip_ext.tos = vtc_flow >> RTE_IPV6_HDR_TC_SHIFT; ipv6->ip_ext.proto = hdr->proto; ipv6->ip_ext.ttl = hdr->hop_limits; memcpy(ipv6->ipv6_src, hdr->src_addr, sizeof(ipv6->ipv6_src)); From patchwork Thu Jun 8 03:19:54 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Chaoyong He X-Patchwork-Id: 128366 X-Patchwork-Delegate: ferruh.yigit@amd.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 8A00542C57; Thu, 8 Jun 2023 05:20:38 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 959BE42D39; Thu, 8 Jun 2023 05:20:29 +0200 (CEST) Received: from NAM04-BN8-obe.outbound.protection.outlook.com (mail-bn8nam04on2129.outbound.protection.outlook.com [40.107.100.129]) by mails.dpdk.org (Postfix) with ESMTP id 354C241153; Thu, 8 Jun 2023 05:20:26 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=FefR1hARUrC6e8psMxtpC2cdu2uE5YfOBSBAsaUJYZyVozfcIQkbdoNwp6djVwCdoeA7ZBtYovQtl7PJMovXdboCtezjl2PEpTYhfbqzuLtNtH5wC/OFUWa52/CPonxYhGvdZfppFJoZpmGw7+GLO5xUmCWCUQ91Q/60F1qUvIZSTYjWt8K1gvY/twkblNNHZvOVwUIH0KKKgL4cUvNXYWCthInn85e36hp+u9IXfrkYkb+L90/VHANEQZCIyy0vuKn/1pqB8+kA983hdRSU/HU3P82BtCUyOhnT3R3EhmLZ5ma+NBmbwIPQ+UdCCyJLCFE6PzvQoT8OVGb0T5bDZA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=VK4hxWT45xBPsSrsRTtRg82OYpPzGDpPDeSTABHWyBc=; b=XwKxX6ltc0MsbzPR3Q2981/XX2Gwu6TEVkHFWQSQopqTgxoxc45BsMeFWaNpbpQEKdsfBF7VuxSei/c1c+r2FumC/8kuYe0z5DCfiBu3T8D6JohkfuMyn2qy1obUPUUiCn9E7SZlTuAWSB7k4qGAiMpsibUblDUfiqJfZwsPIpRpcJF8Jys1VHBf0HFWZOAmcR5ojykcDEoZ+h5AW7rEvtroSLfNyN8NsQYXu4AY7EwWmrSceBRCJ9yAbOu1CtbO2BUdzEZSQ+wARGE7h7gTXY8diL0EMbF/L92lnKnuqxP2FzxjpXw1wiZbTPQW+M0ff2vRwUlpSPHwXK8dAZXYTw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=corigine.com; dmarc=pass action=none header.from=corigine.com; dkim=pass header.d=corigine.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=corigine.onmicrosoft.com; s=selector2-corigine-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=VK4hxWT45xBPsSrsRTtRg82OYpPzGDpPDeSTABHWyBc=; b=oYXJtkHAiHGPpg/Y8j6tZgeiWxjlQNo7u3FD3CHFhAGh9lx6UPTHYJ+1BqFjNoqQszt0Rqut9GLiuD+a68PcuZP7IAKMYTzI7q0PvX572lwQheByTXVUQtqeIpqT+E79hH4Q22tAl/gDWwkuW/yn0d+Y+pGg/aPP9dfQvFVXZM4= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=corigine.com; Received: from SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) by DM6PR13MB4432.namprd13.prod.outlook.com (2603:10b6:5:1b8::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6455.32; Thu, 8 Jun 2023 03:20:25 +0000 Received: from SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::d2cb:a650:e1ce:2174]) by SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::d2cb:a650:e1ce:2174%3]) with mapi id 15.20.6455.030; Thu, 8 Jun 2023 03:20:24 +0000 From: Chaoyong He To: dev@dpdk.org Cc: oss-drivers@corigine.com, niklas.soderlund@corigine.com, Chaoyong He , stable@dpdk.org Subject: [PATCH 2/4] net/nfp: fix TOS of IPv6 VXLAN encap flow action Date: Thu, 8 Jun 2023 11:19:54 +0800 Message-Id: <20230608031956.814684-3-chaoyong.he@corigine.com> X-Mailer: git-send-email 2.39.1 In-Reply-To: <20230608031956.814684-1-chaoyong.he@corigine.com> References: <20230608031956.814684-1-chaoyong.he@corigine.com> X-ClientProxiedBy: SI2PR02CA0038.apcprd02.prod.outlook.com (2603:1096:4:196::22) To SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ0PR13MB5545:EE_|DM6PR13MB4432:EE_ X-MS-Office365-Filtering-Correlation-Id: ea528873-f473-4ed3-d6f8-08db67cf4cb3 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: NFwASPfTffXxuVFwdFOC4WTbRIpzuGPIIzGGMQBLuL53oFats6DeVdZ2JkJ2Za1nOEitnIMbLI99/JZLBxwLxFv/Y5Y6miYs9RYZ9RxvTpCaqu56F0kDEhXXw2aIc0i7aiwoTY0Joz3iU8bL6ZJVVKbAaKzLJcScDv76kzNItcLqqGkXTiASJw6RW20WceNC120u/gfWZDADei/qncTkYQ6TumK+BRfgyUJzH8Wk0hGH28cjPeX3F1c4Z9oSimUQKEaYLhua1nw9isEvsTZGY+Wp7GBSOxtK9LHyAS+Cb8CH2iaaZ2vJZU2uwo3AhHJE733pRFm1DAOa9ts872IQhzPXdtMccWWeGveaE7oXiCFwD3kn71bRlLiTIjGPdpEm3L2+YxV7hEdYqOPIgE80F8UTNwpKafhPE9wvruAklFUXzH+1H4aYj7wwQFuOqK0BH7+femja+OboPVmwJQD/hB0jVBZIJMbAW70Gyox1REKHBgFMj13OMz7arxwIWNJa4mIULDFriKxKRpxiUuBnOIEFMWWBbkdg3RZ7G5Nj/bilglw6X2BweOMndbUNV8BhVvWs0BFiPeHELvg/uSMySHYDh+oA2ACEy5dW+H4+tuIu8EgM8OOthzptalgJ7DRu X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:SJ0PR13MB5545.namprd13.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230028)(4636009)(39840400004)(366004)(136003)(346002)(376002)(396003)(451199021)(478600001)(2906002)(2616005)(83380400001)(6506007)(1076003)(26005)(186003)(6512007)(86362001)(38100700002)(38350700002)(52116002)(6486002)(36756003)(6666004)(5660300002)(450100002)(8936002)(8676002)(41300700001)(66476007)(6916009)(4326008)(66946007)(66556008)(316002)(44832011); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?Ih89Z85mzjtJfdLHcMzNFz2f9SbF?= =?utf-8?q?zgJ9M7wQ+9s72X68JQotdElF3J8qBD3ppwZoFmO1D2Pl34hpPpXC12PjJNucdF+9B?= =?utf-8?q?GYqP7KdHKwjsP8OTNW1YIYjQ7v0rztOHhOveEHBJupc20h1gSNwzITnBYriqJTGjX?= =?utf-8?q?7rAL2vL7/LltIlM0NMlQpDlTj0RH1LDrqGPO5APZNyyO8KCNKHNpU/dfpGsn5F5zG?= =?utf-8?q?kucW9v9d35WP9pjZgCbr0A5ib/3B2n8teHd7oVk8RYQZPRyiNhex3feJ2vRBVNUaP?= =?utf-8?q?sgAqOz6w63JIk15C03LpLDmcfenBipX1bvpCo2PMiZUlHgxyR52NfdhXcdc/4xRYi?= =?utf-8?q?f9AvTpzIkf2YqMbqLCi1AI9YF9zdG77NCax84kenQS8UhZvjvvvfWpcTh5bqjMAgz?= =?utf-8?q?RNPVq0WO1/oeCYTQqqFvMVIg3VZqFvc142B0CBi+Fh/Cssctjm4vId5idGSHCC0bC?= =?utf-8?q?ECvTmWu1SYmLOZloe4rWvSwqeEr3vsbl9PjY0jDhF88DPRhx7jYoR3Hrm2blE0dFP?= =?utf-8?q?AGWVymkTLEbe/A+avp9ENQcyQXPQOjx4Oy+MLpYWbUX87zrBSryplSUvIT7Oc9Uvd?= =?utf-8?q?oYfoDvPm3K3sV0jweZrx12KRnkhDn7c8ZTXUL1DRQqWf4uncuXQOv4Fwe67/zxKJG?= =?utf-8?q?4mNKrYPX0o28rB5WiDRnlcivhWZkkALE20fY2Y9Sm4aTNAooLiAyKTK4MDLn7DMHE?= =?utf-8?q?lM68rcJq+xu3G3OzTkTI1h4//Oo9mj34IcCDqQbgIMWrnvZvaQyqen9VMdTnAHrbj?= =?utf-8?q?x7BIkoRo6UwJiMtmLiDmchHf9SZCkekLUDdp5Zecub8wPeNHQqQBAHnFhNXriomEP?= =?utf-8?q?ju3G9ONcyKJN+BL9FMTUPUVR3432rW9QYNRebnxomy4FIGY2rDck3i88An3gSm7AF?= =?utf-8?q?LI0uYdPmCYhFA7kEtAkMm3RsYMFa1yoUrr24ZTYPDcZBBvdxzD9L+3DCkPYZyF8jK?= =?utf-8?q?jRpbcfAOOhH8lTblFhZwge4eJPIDY4rPtRavPpmYs4f8uQjXagO8TTZIOuhZaVR2e?= =?utf-8?q?zU/prwUipjK4y971WrzQil/1mgvc2RLc+8V6QDNK4IVEG7YvEfOy/mzpWuNG2na6i?= =?utf-8?q?/eG3T7ykp1fW2eURLp4DckWmIQuyXsOxKALa5yh3ORpDloFRdDC1hCeUm/WynDG9K?= =?utf-8?q?X7+LUGuw0vTO0FpSQINIGXrpIvw2SGG/F4rUjPZVVvZzjs9x6mbEq7McB15eLob0L?= =?utf-8?q?xYHAV+8wHJ1ECO0HUhnilEff+J01HNK1THDpzxd3BFmDAHsapLqvjizbsHRet/+ck?= =?utf-8?q?bwd1kB+2lCW9nuzB2QbXl63xx6g7yXFmySmIJ2JDXbSfJdhDuZOhI4hIYjiwNZAn/?= =?utf-8?q?UPqSa9x51fNE8Ff5fMRC/ogcKHO4pgHS886lCJ+F4GZfZE+9p6FvH3UxqSQqajMa6?= =?utf-8?q?A0zWKDty1AQHSZlaV4SQt8tOCd05qBSMO+XH0auDaX3XBlWirj13vkEGd5gcN9NuG?= =?utf-8?q?MA8HFMZZQy9kPS/V42GFym0u+vZMznyaTD6bhnaZZ5AYU32S/JZ0/tkNi61Koj1kQ?= =?utf-8?q?pD3cylHhSuYtSVaLfUeTcR2vd1cq77yfBQ=3D=3D?= X-OriginatorOrg: corigine.com X-MS-Exchange-CrossTenant-Network-Message-Id: ea528873-f473-4ed3-d6f8-08db67cf4cb3 X-MS-Exchange-CrossTenant-AuthSource: SJ0PR13MB5545.namprd13.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Jun 2023 03:20:24.8014 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: fe128f2c-073b-4c20-818e-7246a585940c X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: CCNcQYKh9zbdiD58EbkwMpqDLJK9VRCfKMy21LB8SE2ZlK6GO5iaGKiPCmQU1z9eSbhDbK4LylWTNzP27N1EFue+LbIqilhkGw7KpAlahIk= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR13MB4432 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org The former logic directly do shift operation on big endian data, fix it by convert the big endian data into CPU endian firstly. Fixes: c3b7254093c2 ("net/nfp: support IPv6 VXLAN encap flow action") Cc: stable@dpdk.org Signed-off-by: Chaoyong He Reviewed-by: Niklas Söderlund --- drivers/net/nfp/nfp_flow.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/net/nfp/nfp_flow.c b/drivers/net/nfp/nfp_flow.c index 937fb390df..914afb85e3 100644 --- a/drivers/net/nfp/nfp_flow.c +++ b/drivers/net/nfp/nfp_flow.c @@ -2743,6 +2743,7 @@ nfp_flow_action_vxlan_encap_v6(struct nfp_app_fw_flower *app_fw_flower, struct nfp_fl_rule_metadata *nfp_flow_meta, struct nfp_fl_tun *tun) { + uint8_t tos; uint64_t tun_id; struct nfp_fl_act_pre_tun *pre_tun; struct nfp_fl_act_set_tun *set_tun; @@ -2763,9 +2764,9 @@ nfp_flow_action_vxlan_encap_v6(struct nfp_app_fw_flower *app_fw_flower, set_tun = (struct nfp_fl_act_set_tun *)(act_data + act_pre_size); memset(set_tun, 0, act_set_size); tun_id = rte_be_to_cpu_32(vxlan->hdr.vx_vni); + tos = rte_be_to_cpu_32(ipv6->hdr.vtc_flow) >> RTE_IPV6_HDR_TC_SHIFT; nfp_flow_set_tun_process(set_tun, NFP_FL_TUN_VXLAN, tun_id, - ipv6->hdr.hop_limits, - (ipv6->hdr.vtc_flow >> RTE_IPV6_HDR_TC_SHIFT) & 0xff); + ipv6->hdr.hop_limits, tos); set_tun->tun_flags = vxlan->hdr.vx_flags; /* Send the tunnel neighbor cmsg to fw */ From patchwork Thu Jun 8 03:19:55 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Chaoyong He X-Patchwork-Id: 128367 X-Patchwork-Delegate: ferruh.yigit@amd.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id EB75542C57; Thu, 8 Jun 2023 05:20:48 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 10BCD42D53; Thu, 8 Jun 2023 05:20:33 +0200 (CEST) Received: from NAM04-BN8-obe.outbound.protection.outlook.com (mail-bn8nam04on2139.outbound.protection.outlook.com [40.107.100.139]) by mails.dpdk.org (Postfix) with ESMTP id D344A42D20; Thu, 8 Jun 2023 05:20:28 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=JESulpbFEYnJFMuGF2ZM1KQkQa5oKfZBqRPrk+jYmVgsRzOY2Qcg1/QKRDLGRxU8CMSlJR/kgjCp7wDlER9OflmDjaWmmVQplVZIZPf8L/htDXCW7nTERcBSnI4gia1SQ/9sxMlkzn3jQyt3c1kqkf2Z24aPOWiuH6k25HCmdUnB6xyC3xv4em3ayAcFh8ZcvSy0hcMcsuncajUeZjKqiJ77mi7IIafaDTmutk1DPRF4pztOoxDJJMavB7F7PuaesXaBOU5sITIx2F8u+5sIVsjHix1BPK7QXWUMYRlQGCS7tNOOIawuMSjB5d+Ll/Jd6Lvb8M2lDvIe09qN01dLCw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=Ivom6h+d+55WvxnB/AZkhiya3zM4SeL0VzzvbXUnsHo=; b=LNz6jof5XoyXoBDpxmY9BTE4HCY5d1Z2jsJDn7h3S3XcFwKvueLFQooTOvepZ7RrNoKiCX5PRpDvyXVFEQt8zzIYEGLCz0eP2WoyW04VNeUzwMy51ICliM8tvN5zdqTI+w3+suq1U5lTJwaIKKLmBOYTENqJCEKOfJueNMueFdXJBFqKE2SLRfn2CjB5JsebITYYOw7n7rlNaFUWUaXWbCy66j9N4l31w/yk+QaP/kxTXx3zPXryqWI9fvMpM1mX8wbvqWwbqU9NK0Y+gRJQFpf4hcsYqt5VpnKC4aaxU9hpT+po7WtNCKOMGVVIIo1FBXiM0W1KejLe4v1iGo5RNg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=corigine.com; dmarc=pass action=none header.from=corigine.com; dkim=pass header.d=corigine.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=corigine.onmicrosoft.com; s=selector2-corigine-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=Ivom6h+d+55WvxnB/AZkhiya3zM4SeL0VzzvbXUnsHo=; b=v2/SZMuwpPVm7mSbCMT4YD4ok0f18C7HhnuxejMSvQNwjHolWMvGhwFxw6rE/J0ob7PoMXlrjHQk+U1Fpss5D4wrtxILhe3RSBl3u7oNIkhbmUv1mOHZy+l59LcBrO7PMZd2rs7/Ut0Vy5khlEkE2hKm3x/UPH2aBf5oI1B5Svk= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=corigine.com; Received: from SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) by DM6PR13MB4432.namprd13.prod.outlook.com (2603:10b6:5:1b8::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6455.32; Thu, 8 Jun 2023 03:20:27 +0000 Received: from SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::d2cb:a650:e1ce:2174]) by SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::d2cb:a650:e1ce:2174%3]) with mapi id 15.20.6455.030; Thu, 8 Jun 2023 03:20:27 +0000 From: Chaoyong He To: dev@dpdk.org Cc: oss-drivers@corigine.com, niklas.soderlund@corigine.com, Chaoyong He , stable@dpdk.org Subject: [PATCH 3/4] net/nfp: fix TOS of IPv6 GENEVE encap flow action Date: Thu, 8 Jun 2023 11:19:55 +0800 Message-Id: <20230608031956.814684-4-chaoyong.he@corigine.com> X-Mailer: git-send-email 2.39.1 In-Reply-To: <20230608031956.814684-1-chaoyong.he@corigine.com> References: <20230608031956.814684-1-chaoyong.he@corigine.com> X-ClientProxiedBy: SI2PR02CA0038.apcprd02.prod.outlook.com (2603:1096:4:196::22) To SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ0PR13MB5545:EE_|DM6PR13MB4432:EE_ X-MS-Office365-Filtering-Correlation-Id: deb68863-8d6d-400a-796e-08db67cf4e2c X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: mSKb1A3GmhrXMNcy3cx4GgUoywjBjZy7aqI23ao5185HErsoDkxWBVuqYaO8HGH3O8nLAXSo6ic2zn2/ok+Bwc/YeS71TYg8LI3lodPtXRYBfXgu/ZENXz7ueqUVBY80dJPf8TNVdRKF8tUv8p2kiHKbQmRBNbX9iI2byqwW4GMCQUWdPVw3Tp8EhJ1HfXHzpdG1O/xBgqxO8I0hgUC65/y0Bmxm6+b5bDZr2bDAuSL+U8asXGBfyrLsbR5lPG3gYwxS384dhjnpDlCJ34ZtIYmbrmytbdrjg9iA/EGME/t73YrdpxfgB02tBmsTLFwnj5VewBkL58SpVifi88m7SMsIBW4R5B/rpEKc1x0l8rr7mVbN1QE9DHW5RV8pKByWLmNRoHP41W9LfwrQEl2zhWeoXndhe7AaBGzKv+S6Id1P4jxECYUF7hZNGZwalA/JdPLofBwbyDJmTlPOHS0HC7mgZeO/GMaGyEmvWpL72viznb5AgwkRYTI1rEHUpw1I0I29+6J2HU8B9xarW9Ob9y+9DWwYjrjPtmLA91MVIynNyMoNDzXl53Fbt82g1rXq+akP0AE6tqnFfgscEC3SNzA+IAKRDl7S7GBRUF6K50VzvCTYccq5fDHKAQXTBplJ X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:SJ0PR13MB5545.namprd13.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230028)(4636009)(39840400004)(366004)(136003)(346002)(376002)(396003)(451199021)(478600001)(2906002)(66574015)(2616005)(83380400001)(6506007)(1076003)(26005)(186003)(6512007)(86362001)(38100700002)(38350700002)(52116002)(6486002)(36756003)(6666004)(5660300002)(450100002)(8936002)(8676002)(41300700001)(66476007)(6916009)(4326008)(66946007)(66556008)(316002)(44832011); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?lLrPhgIMF/NcerLG24LVghcY2D/s?= =?utf-8?q?K69TQinNMPnOyvubiZTARaT6Vb7piEBgJYMUQqUiUnCtlXqXdDC4C6IOwh/RKFArL?= =?utf-8?q?xYCA/z8UF5UxDSEtXhuEsWF6Y8BRCehGY+GSlBrgzX03e1mxhlpjzeMRvPj4BcRIP?= =?utf-8?q?C6aJil1i7CV82qSFT4Tvot3Q+pVYAdYAcPNHnlgL2UtesP7KLpsPHJKP6sC+alFTZ?= =?utf-8?q?Y+/9Fyn1awhmEP8hiikB9FIUFkRRYFL7jnUj98hPnRIdVodeYl3tEGoCpZnTpGoqd?= =?utf-8?q?ZOnu678tHPPh3k46KT1FFyGHZWHniWynvq70wDlXKWNxQacDt2jw++avnIY2OFWXg?= =?utf-8?q?MPu4xOUCrIbz/bJpgo6XFU//KCWFv8361SV4Dkpm6EDXhzo2uRsP3uqgplA8dkjLx?= =?utf-8?q?MmiAkfQqcACuNLqa7V4PUqmhL7NMiLugWjn853v/c3qFoRglEIL7t6XJ38W4amWer?= =?utf-8?q?eG1vTbW1hwN9OxJaJVMVsxti4OUooD5oYSW/n/XrC45noU5T+h6WuoDBhneNbdRH9?= =?utf-8?q?WzwwzTtnNy6eDfFdnMAXGGXZhTf1OHTMRmsqT9Z4QsT9bPRfwD1sZeJi9jrkXt4X9?= =?utf-8?q?C6onwNEJBSzEe0A9H+n7J5/tCqZxfOg8UteI4WqcHwMCZzXBy+gqOPuLvkhENKh5j?= =?utf-8?q?sqm+bWoe/T9SSvtYtPeO5M99xspc4Qd9ajxGDut+PH5A+0QZQifk3mcL1J/5Tu4Vx?= =?utf-8?q?zEiLfYS9arCkI2zLPSGFKARw/tXvt+X4YcuPSIOfZM42C8CTCuoLqgH13kNtomV5A?= =?utf-8?q?MGt9PSg0Og8hc4WXtJw6h6Zf6sX8LrnWJXOZxKlWnPzxU3Tp0ujgaO9Af8sciCAVS?= =?utf-8?q?VXFVu30CMkB5QwTRgYLzizy5K8oaGHfRe99X7S7Q7aoKESh5mJgklafbwqmqdDEld?= =?utf-8?q?7GGwiViwIZiGyudS5e/PPp6CTjlMoc3lLrOZ7U2cJQGMFeAagkl6arK5cuBveaNSE?= =?utf-8?q?JvvJ2l8eprBhfoVNvFQku6ijCh/Nyd7WMOkNGxJD/g4J9gpngsNNlVFPBEs1eimvN?= =?utf-8?q?WGwB6gQoGr3knj2vh0SR2SvVf5LeY2ooSbz7IHT1T1E39+Tsggw4X54vKggeZ/ZrN?= =?utf-8?q?Ely0PhK2zosJqijYZ1H4JsVqQqPgvxg6nDXOFp6Mzzix+BDBtc31GgDrI+fQP6Tkr?= =?utf-8?q?2L2n+UDVEbHJxBJNh+eaYPc8Dm90rXSIPtZZZmI3sPxc6CqGn6XqPHV5anhbo/xoU?= =?utf-8?q?BY3RnRmD/ugb714Fvqmj2HtcG2yvfBvOwUJxzM9DfWaeEw25SMzYl/nAnqDxXWVgM?= =?utf-8?q?f8kt7Eu8fK9ukMP4wHv4HWxL7Bo2ylJ4CbUpnzTisxsUi7hCuaaDrYljhhRyS0MfH?= =?utf-8?q?ICiyfCoM0+DI2DDViDq+0PeK3i/O+XIr5hdfVvssXHwKIaaOEW/nCIjlQp6Gau/0t?= =?utf-8?q?S9H7qDGylev9F+CsstPo32eBTkXfHWXReGMg5AsD/Db9yYzDx0jJHnm4g/XRmhcVA?= =?utf-8?q?hBJPp5PWU+E9OPFxDTi7ASS2t3ACylokNUby+BJjCEcN5U5iSOVnxYAnvWT/6qyIc?= =?utf-8?q?SSLPl7Rmbe4IjQ97bFNe+rO8C+3jXeKf4Q=3D=3D?= X-OriginatorOrg: corigine.com X-MS-Exchange-CrossTenant-Network-Message-Id: deb68863-8d6d-400a-796e-08db67cf4e2c X-MS-Exchange-CrossTenant-AuthSource: SJ0PR13MB5545.namprd13.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Jun 2023 03:20:27.3637 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: fe128f2c-073b-4c20-818e-7246a585940c X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: csKU4QSR1UZVw5n/qhb4CyRwucbioH1eJbYXge8dvAuY5XyWcflYT+YBkBG2GUYJFgvRKvbbCAu0TYX1ieVtDlA4ogphZ+rJLt5wiYX66RQ= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR13MB4432 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org The former logic directly do shift operation on big endian data, fix it by convert the big endian data into CPU endian firstly. Fixes: 98fa36eccc4c ("net/nfp: support IPv6 GENEVE encap flow action") Cc: stable@dpdk.org Signed-off-by: Chaoyong He Reviewed-by: Niklas Söderlund --- drivers/net/nfp/nfp_flow.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/nfp/nfp_flow.c b/drivers/net/nfp/nfp_flow.c index 914afb85e3..213d0c7935 100644 --- a/drivers/net/nfp/nfp_flow.c +++ b/drivers/net/nfp/nfp_flow.c @@ -3130,7 +3130,7 @@ nfp_flow_action_geneve_encap_v6(struct nfp_app_fw_flower *app_fw_flower, set_tun = (struct nfp_fl_act_set_tun *)(act_data + act_pre_size); memset(set_tun, 0, act_set_size); - tos = (ipv6->hdr.vtc_flow >> RTE_IPV6_HDR_TC_SHIFT) & 0xff; + tos = rte_be_to_cpu_32(ipv6->hdr.vtc_flow) >> RTE_IPV6_HDR_TC_SHIFT; tun_id = (geneve->vni[0] << 16) | (geneve->vni[1] << 8) | geneve->vni[2]; nfp_flow_set_tun_process(set_tun, NFP_FL_TUN_GENEVE, tun_id, ipv6->hdr.hop_limits, tos); From patchwork Thu Jun 8 03:19:56 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Chaoyong He X-Patchwork-Id: 128368 X-Patchwork-Delegate: ferruh.yigit@amd.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id D9DAF42C57; Thu, 8 Jun 2023 05:20:54 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 0728D42D5A; Thu, 8 Jun 2023 05:20:34 +0200 (CEST) Received: from NAM04-BN8-obe.outbound.protection.outlook.com (mail-bn8nam04on2137.outbound.protection.outlook.com [40.107.100.137]) by mails.dpdk.org (Postfix) with ESMTP id 4E0B642D4A; Thu, 8 Jun 2023 05:20:31 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Se94E3GJlFkosorZdrQs04oI+oP/jllZzBTq4HICDO//MUOE0kJR3bjAWyfwCOO5+kyaiUtygqHPhraFHOVmeBs2tCuJKrFHqtyFqD6hARuWz5ZmURWcM3j/xg6C95T4XzLTsml66RhoXAOJ1I1/1LmdO1z+OmbwDpUpHmOfaXxNugIUkhUpgBlFs8B7/fXfFPHGK26NqlbQgouOhRTxNv2iCqTv+Xpu9gHmrGiP3FJOPu0lNf7f3MHWYxurTbsG64empdIfYN790ST/6Md5acCAqZeruDZIW0rf6QsPZ8UTRRAgXcoVAvM/EROxX8prUasT1h9UaiSmbe9OTz5xQw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=4xk9fFo/Gwb+gOSBC0Z91EQnyNtRflHokKVRugDfBbk=; b=cYNAh7xQ+Kqixgw0PtRUO8i+ukO6fmMC3/Ujv5SrJO3A1v97oxcSs98Nmk+TKP+VH6Wy7AF+5PpZtuctsj/ocMOa6cJxmq8P/TXjNGgQKq2R4vZPHYRYeyzkgOs/QeEv6jWUNpsHCrdkkrw3uMdXqd9gKo1nH7lCbEGRd/CQAH6zT44YAH579FidehfLGaTiBr1UQsSf6bGCDMdc9iTcDao39jkSICk6ITFux5XkwuiGXNuVuki88ZILmNbB4iEtHa2ZUhwqSMQmpJpdc+gh8LInEZC0kwAplTrzsVFkRXpeKCRUzv87ubAQyk6vVR8Wt1JyUnBcrKZ41kfoj9dbKw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=corigine.com; dmarc=pass action=none header.from=corigine.com; dkim=pass header.d=corigine.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=corigine.onmicrosoft.com; s=selector2-corigine-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=4xk9fFo/Gwb+gOSBC0Z91EQnyNtRflHokKVRugDfBbk=; b=oEhP89rxSlssGpRxyKAdzHAgolpahtJBjYI5saQNE12tyAT1uNmsxi6zylnicnqnaBhQbOkjaFXsfLeIghGZ9Y129JdbdgCqhWgEB0D0r6FK33V0rTn4BQyTURhOOmMkqV+XSWyVU60NlwSY/ViV+8uJaZzQQACSLLUldykFjBY= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=corigine.com; Received: from SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) by DM6PR13MB4432.namprd13.prod.outlook.com (2603:10b6:5:1b8::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6455.32; Thu, 8 Jun 2023 03:20:30 +0000 Received: from SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::d2cb:a650:e1ce:2174]) by SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::d2cb:a650:e1ce:2174%3]) with mapi id 15.20.6455.030; Thu, 8 Jun 2023 03:20:29 +0000 From: Chaoyong He To: dev@dpdk.org Cc: oss-drivers@corigine.com, niklas.soderlund@corigine.com, Chaoyong He , stable@dpdk.org Subject: [PATCH 4/4] net/nfp: fix TOS of IPv6 NVGRE encap flow action Date: Thu, 8 Jun 2023 11:19:56 +0800 Message-Id: <20230608031956.814684-5-chaoyong.he@corigine.com> X-Mailer: git-send-email 2.39.1 In-Reply-To: <20230608031956.814684-1-chaoyong.he@corigine.com> References: <20230608031956.814684-1-chaoyong.he@corigine.com> X-ClientProxiedBy: SI2PR02CA0038.apcprd02.prod.outlook.com (2603:1096:4:196::22) To SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ0PR13MB5545:EE_|DM6PR13MB4432:EE_ X-MS-Office365-Filtering-Correlation-Id: d158f433-f5ba-4e72-659d-08db67cf4fb3 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: Ug2D7K3fmH6CqMDlq7S49HaP6kcJofIx+g2vPeqlk9uJ72e/XnOLfx9GJYc32YZhlrE3h4L7eZhs5GT+5Y0vMRkV0p8L3spnlR7mP5V+pnkyxJhXq/0JAsbF0RXRCUrnNhicygfqeViGnXaJIRAhM4Dbg1njgSCTh7mk9nsXHkgxofmFC6/EbnST3JGAeeMgboFEW+kBVgXYyiU2uS0zS9Fma7htT9iSUD+C60vTDPNrJ0sSbVamrTMpNKmAIY6uYJWtkjOjrs53GtTE2d3JododtMkhS1ea1cWSjltGvycgu4DofZ5sdn54nWdHIUlb2txhDP8BMR6+r7eEIv427+ZzT63Owp6mxJl5mdDTcXdCaJEHw4Ua7anALtv93XnJEXwD8Xr3FG4ePpBdvL0ZE17HQ+t5v67XDNQqxq9YTplJNsO3rToMynj4/5uG8Kv59l/jzr2cM1wiYDuke9JyBJMkHEUB8b92zHjDP35WKFetoV7yqdkvYjPjooqBgMr1R+LS9cnDi+9M0Pq3V5mOC4ZqfIbyocp0zVV684Rj3rqZkE+MTtty4lIiTbFf51HR70hmSH5EsJO7IKKpnxbvk5cW8m4cL3A5A+a3fnIMDXXW/KH7Hd9r29M6oO70D45l X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:SJ0PR13MB5545.namprd13.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230028)(4636009)(39840400004)(366004)(136003)(346002)(376002)(396003)(451199021)(478600001)(2906002)(66574015)(2616005)(83380400001)(6506007)(1076003)(26005)(186003)(6512007)(86362001)(38100700002)(38350700002)(52116002)(6486002)(36756003)(6666004)(5660300002)(450100002)(8936002)(8676002)(41300700001)(66476007)(6916009)(4326008)(66946007)(66556008)(316002)(44832011); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?z1x4wg4mfGnAyWc8q4sYUqj3NakZ?= =?utf-8?q?ICINGcD5TPYIKSvuuXB9iDIk853oynshXkO+SsELiWsNvAY3BVt4NUGikRw+sYTht?= =?utf-8?q?HrrIAJaiGBqwmNJlIcHcwylrU02mgxUcvRV9QRMd583zMDpxHDLtR4ADSgATcr84W?= =?utf-8?q?iCiyTCAv+l7qbyVoKQkHG4WFfnDnQ5kXZ+Y0WUHlAji/65G/xi7H68jZnIEXW8SQS?= =?utf-8?q?HMcAfMk3iMRiquWCsngaT17PKIm4zX7VBnvLnfdwAIwBeUlCTykGUNDN4gc/AaicD?= =?utf-8?q?sMNKpDWeMehdHMNgOSIe2URX/LzWqBrg2n7TNX9woyzn8yiJNF7svOsZf5ViYpPrL?= =?utf-8?q?e+MHkuwuPr+Inye9Pbs5fqawSs6L5MlZQ9CEHKU7sZILmciMlp0BLCZU5cSXF7BYh?= =?utf-8?q?0BaTdd9d/hLepnZiHafUS3zHajuI402c6Xv6JfRGmOkyAjUwnPX+25eIwGOas/ytu?= =?utf-8?q?B4lWh7EcJI3gOmZaGt//uI0Q3Yu+JXvC0TSjp4x/SJRKhg8ax7rZUEAnHE9fTPGau?= =?utf-8?q?8SMjmv6xPW78rYoFQ+SYXSs/XjPFzMuGX60W3qylWTIMDvWSrHWJaj0qTvsHodyQs?= =?utf-8?q?SmlyJW0EIAvx4/+/u+WMed+QUOKDlWHRwh/zKiAW0Nt2zADbk31eqi+yX3qp2xqim?= =?utf-8?q?zDacbjKJoOa2nPuRkmY0VzCr8si2a2dmgeiWODkqQ5z+n3V5QXob4DqBBT3Iv5ddi?= =?utf-8?q?zpYMuSIjATpslf+aCDfqrhcpErBzLi2ITOwuGi+eK9IYOH2PrXy1e4ylE8apESVBr?= =?utf-8?q?T0zpVTistpqz8Uh2Z7neVDtTFrTrwQWcYnJQvk+WYT8uAgITj9KdGyngOdi3yBvxE?= =?utf-8?q?lexJhNawaYy8U9l85MstIhl4gLuLknc8W6O4iB0rsJlQTFuyZaBwNhqzf6A+INik0?= =?utf-8?q?ZHiVma5O5wKCUn9uxOUDGXl37dX5buNdP2z+7bzD/07PtIuS/OLuqSHJNEHowhoZQ?= =?utf-8?q?v3mZdjOlvCSEC0ZC5PYJ4QI+Nq24j9Mx8H/pnIL1QAiI0XADCDVuZ79SWy+dMbTGI?= =?utf-8?q?JE8pZTYeNAYIq07dodxmAPSbsOba1yibFf2oaf9Ar3ysi8xadu9eDj6NMI/KbrayD?= =?utf-8?q?OwYkvZPH7SIhkeCfOvupqatgsNgDH3HWnGqw5c8uaqJr6DDLtkTFb4jSpYnnHLEsG?= =?utf-8?q?1K1r1NVx3tCBjgUjHbOR8RH+Z4DLE0wh3zIMxpX6WSqFTAib/zeH2vgYx7CJ2bayD?= =?utf-8?q?tZhbSM/w+krr1W2OxdhN+6EKOGCJNSGaYclOizU+FHmLazYeFlZTlpZtBFDBq2JTB?= =?utf-8?q?eWVfta7lhs6MMxS3EyzQVNJ6b3TzEH520WAC9gb2196PqTnkPiRY0gCBaWMJD6qLz?= =?utf-8?q?FzvBTjl4Intz6MxvNheDrzgeGEgb3AKubm4K/zrP2IAK7GF86oq5g1QORJ11+MOl3?= =?utf-8?q?n0qsFGHyyDzJR/iv++MWhbgRaXMZB2PlY5GG3IN+8zrJgEFK3oFdRy6eJh+agXK6S?= =?utf-8?q?mwxIgBS5Zf8h/TeQZnSaBhX/BVRM3IvK+4OLN+RQ3PtjExJyTVyx6AD48RmKObFlJ?= =?utf-8?q?109xMt2aH7SqYOgjuYTBhOkGkJH5b6WbpA=3D=3D?= X-OriginatorOrg: corigine.com X-MS-Exchange-CrossTenant-Network-Message-Id: d158f433-f5ba-4e72-659d-08db67cf4fb3 X-MS-Exchange-CrossTenant-AuthSource: SJ0PR13MB5545.namprd13.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Jun 2023 03:20:29.9072 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: fe128f2c-073b-4c20-818e-7246a585940c X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 8mWZdM03ULPm21aVCR7FZSpJ+9Hh1wtMF+xqLa2VvMNlVN465OU5hE+jjgbBFokegPC457ECGlyyYtpL7m9Gr1jb0MdiuoIkCZjNSgiCUIk= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR13MB4432 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org The former logic directly do shift operation on big endian data, fix it by convert the big endian data into CPU endian firstly. Fixes: fff680eef7f9 ("net/nfp: support IPv6 NVGRE encap flow action") Cc: stable@dpdk.org Signed-off-by: Chaoyong He Reviewed-by: Niklas Söderlund --- drivers/net/nfp/nfp_flow.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/nfp/nfp_flow.c b/drivers/net/nfp/nfp_flow.c index 213d0c7935..56ce222c9f 100644 --- a/drivers/net/nfp/nfp_flow.c +++ b/drivers/net/nfp/nfp_flow.c @@ -3203,7 +3203,7 @@ nfp_flow_action_nvgre_encap_v6(struct nfp_app_fw_flower *app_fw_flower, set_tun = (struct nfp_fl_act_set_tun *)(act_data + act_pre_size); memset(set_tun, 0, act_set_size); - tos = (ipv6->hdr.vtc_flow >> RTE_IPV6_HDR_TC_SHIFT) & 0xff; + tos = rte_be_to_cpu_32(ipv6->hdr.vtc_flow) >> RTE_IPV6_HDR_TC_SHIFT; nfp_flow_set_tun_process(set_tun, NFP_FL_TUN_GRE, 0, ipv6->hdr.hop_limits, tos); set_tun->tun_proto = gre->protocol;