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);