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;