From patchwork Mon Feb 19 09:31:25 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alan Elder X-Patchwork-Id: 136871 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 6D42243B4B; Mon, 19 Feb 2024 10:31:30 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 56C4940608; Mon, 19 Feb 2024 10:31:30 +0100 (CET) Received: from EUR05-DB8-obe.outbound.protection.outlook.com (mail-db8eur05on2124.outbound.protection.outlook.com [40.107.20.124]) by mails.dpdk.org (Postfix) with ESMTP id 6209340608 for ; Mon, 19 Feb 2024 10:31:28 +0100 (CET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ijtEzT6h5kw/G4o7leRtCPkY0SnruZVc+WRKp6T/5ZVCiIr1rn3T8KSBn++rNgj51EMAjrC4iLKLkz2YWg6FRQ/S3oYIpKvb6DjuquIYCNzAZgA4pJpbaDwkWauT/mym6KQOZBUv81Zp0k/dUMGi4RjCNn2ssfHTa66wCw62juhFNEoF/G5ZsVo/udROZJB1rtY4Z05qwRZzfiA0/9r2G0SMmIxv43UKBm+12VwuzBHKyfRUsKvsuSTwhJ05FIfOhCwXV9UFTjhuCUfIssUnCRIn3ai4LBU5OHbgOol0zaf2I51Xw4C+Xs8AKy5HyTL9aiGhT4xa0xMl7S9koWed4g== 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=kwBGuniDHkxzdFGc3r/A+q3jhkpqug5C3RV+drfarts=; b=JXTSNXQCsfDtRS7s/RDhVbTtnboxfxL4iDybmWFxWkm4kQ3qv+dpM7GI5fX6+78acj76MRnbY5VFgwUYlz+M8S9nuTZlIuMY9U8n7EArlwCbgBnlLeUJkTgOd8rEGu184uIuITznfSc9k1SKqamtNlUvq1yGgN5pRNSeDBz9OMM7BjWkZh1+gbJxmY0ccktqKxhk84ovwGYUv+QOxBym3rx4X7T2B6Wi12hWddoc4T8wBmg8v+N4/0OCLxbva0EEXLFu4uJADh/tQ5j6Za6EKPVg9Z1afe4rQXSbD/A9Z/Hj8ug72RZoT9Wcm91wB8SzIU3iz2u50C5dsHNePWeiXw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=microsoft.com; dmarc=pass action=none header.from=microsoft.com; dkim=pass header.d=microsoft.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=kwBGuniDHkxzdFGc3r/A+q3jhkpqug5C3RV+drfarts=; b=V6q4zkzXu27s7dhL0XLkeSr9WCqkr/XxyUSOl5StAuz8/BYoreTmc9YbbqTqP89ueBRoOO0V01x8htQR5hL+qqR4j6GyZKpKkvgF9mYZcaR2lZRkjosEMuDPBlSz1vP1REJKebtRJCd7Z2bSk0MGZzTQ5anAtfKDWQnvolrUaU0= Received: from PA4PR83MB0526.EURPRD83.prod.outlook.com (2603:10a6:102:26b::17) by DBAPR83MB0421.EURPRD83.prod.outlook.com (2603:10a6:10:190::5) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7339.6; Mon, 19 Feb 2024 09:31:25 +0000 Received: from PA4PR83MB0526.EURPRD83.prod.outlook.com ([fe80::3256:dd6f:ad59:ee34]) by PA4PR83MB0526.EURPRD83.prod.outlook.com ([fe80::3256:dd6f:ad59:ee34%7]) with mapi id 15.20.7339.001; Mon, 19 Feb 2024 09:31:25 +0000 From: Alan Elder To: Ferruh Yigit , Long Li , Andrew Rybchenko CC: "dev@dpdk.org" , stephen Subject: [PATCH v7 1/2] lib/net: fix parsing of VLAN metadata Thread-Topic: [PATCH v7 1/2] lib/net: fix parsing of VLAN metadata Thread-Index: AQHaYxZoWxZbp2xEGkCWc05u90hDgQ== Date: Mon, 19 Feb 2024 09:31:25 +0000 Message-ID: References: <8139ddff-aabf-4ee0-a51b-a2a658691624@amd.com> In-Reply-To: <8139ddff-aabf-4ee0-a51b-a2a658691624@amd.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: msip_labels: MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_ActionId=aca4f2a5-70b0-4b73-b41f-ccdab0e47cbb; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_ContentBits=0; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Enabled=true; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Method=Standard; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Name=Internal; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_SetDate=2024-02-19T09:03:08Z; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_SiteId=72f988bf-86f1-41af-91ab-2d7cd011db47; authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=microsoft.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: PA4PR83MB0526:EE_|DBAPR83MB0421:EE_ x-ms-office365-filtering-correlation-id: 6e1dc4a9-ba3a-42e0-1458-08dc312d8b31 x-ld-processed: 72f988bf-86f1-41af-91ab-2d7cd011db47,ExtAddr x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: zqL3swjnuc1A3Sg0mXRbW7dJp0ueJuGTky87qNkoAMo9JjykBColHcaDIybRqgjKAKih9CZbMS9r16hNBnWs5EK21vOxlTFiyGWIYTDI35xaKtMlQH7FrqyL9xBZ5nKFSg3WKu/UCVJN6w7ux1NAuKn8VgVJK4z+qTo+FnCp69M0k5oFY0VuzTmF1lK/3IGISKjChfjU9ntbg6A5iqO5IbgOti5LGFbM0aycn6QYEXOKL7EAMfzPDDTRCUuMfCUGGVMLf/UE8+anxjvMtsnTaI+TNT28rmVCT97y06TNa8uu6Esxt2USKdfF22aiFscezdkm55L/pZcvTu2A4zZi+KUShgbI24BPiDZB2vBoD9ECbSLMcDBSt9HCFZDWlJlFJ7oX9Fks3mcHoPhQ9RTgGUZDwf02/ZcEW67t/DCYjpT+cuaJec5JgdDGU1XIfHXVbq53f7Wuse7yfY6OElFgGmdD267cRiAW5/sBwpbPxUw9Yv3Mj4nQYv5WN1VLqvT9+ts61meE+N5Cko19oc1p0KDWdhVXq7F/6JslcDYfe9PcFa2TI4e4fXj3O2Cqdka09AkIEXBju3Sl+AZuk7OtgnahfQxiVFRtj2TGgJ2RTw1x85sPMFrX8ZBF7MFv7qX5 x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:PA4PR83MB0526.EURPRD83.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230031)(39860400002)(366004)(346002)(136003)(376002)(396003)(230922051799003)(451199024)(1800799012)(186009)(64100799003)(6506007)(10290500003)(9686003)(7696005)(478600001)(71200400001)(54906003)(316002)(38070700009)(83380400001)(82950400001)(82960400001)(122000001)(33656002)(86362001)(38100700002)(41300700001)(110136005)(26005)(2906002)(44832011)(55016003)(8990500004)(5660300002)(66946007)(76116006)(66476007)(66556008)(4326008)(52536014)(8676002)(64756008)(66446008)(8936002); DIR:OUT; SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?q?OmkO4OCv2vi0ig25XHA9dX4/5vT2?= =?utf-8?q?QDC3z7BXyUI5hPuwaCb3hUFZsvqJvN8cWuFRZPB383pR50n6FtDR52dUZ9MYZ9B33?= =?utf-8?q?CFKmB0tiEoS77W8/gw1f2ZCp014OlAverqUbKskhm7gAilO5cZRoWP4h1qdUYhvNs?= =?utf-8?q?kO/xxovaEu+qq1ie02RDdT8g40CyPoj5VvBiWDAWXv5XUDv6CGlnf08I1E9PYVYak?= =?utf-8?q?Xx675mbLAgOHF5qJ7isfDc4T8DhwyGIQ0tCb1BpEdxPTDi9OilmVipyBp34tJ6eK+?= =?utf-8?q?iw8nhImJKVpjq/L6V1JPtJvXpIFDesvVdg6L56hWnIo1lKfmvOW6n/Xy7tM2w49GA?= =?utf-8?q?9A0RInxnWfue6Tw8/HT5xVVEKPRO1SS7BXYgwME5dQMmKvUxyMMDEdIn8lQB/uhtk?= =?utf-8?q?9cpoJshABowGrO4l7nT6ly2npu7QRyS8Hn8V7yGU2pglUs2ghvxJyJQimgh3Ujqx/?= =?utf-8?q?FZSG1536KBBv2o9Lz7AIVGGYCiREpYMQeWfUHZE/XsVrnWutz/ZXMBK9WhneWD+kz?= =?utf-8?q?sTkW039UNY8qlObUqmyxdfRuF1ICKA5tk6fFuSWCscBi504zwCcGUb5uBxQjbvDZW?= =?utf-8?q?77CjrXg3eeNis1XBBANS9wjYxdMPWHQKCtOiBovVZDrhjli0DyXr0YgQYYgSSckrD?= =?utf-8?q?VV8yKxxCiez2R+vga82jwDHJqD+iTv8ecb0xiPoPmgukKYfTeRc6Rh6EOP1o00ilX?= =?utf-8?q?76uqvKPOzLMEEWap6rIoLu+HPAMu/HLYBQypkmVIVB57OmhGa2tUP1/XUC50lOhDU?= =?utf-8?q?m5ii2CgGLWLeAk9Lv8Jj9P08aeQMYS7HAPkwMjeqLwUk3DmeTyW1Jgbh1oN706hjA?= =?utf-8?q?INm6vqgF90nJAUmN41IgMgq8A0pDhy5stAMzBsdSZpc18/i9WGd2ITQegSIXHVM0w?= =?utf-8?q?/pUQiSfKPGtHl1G7En9fos9j0aKHuFc7cOQVFzk0f8KyQd8RYI5FbCBZeLbG4FZBf?= =?utf-8?q?z3bQHZ8sUxZ7wdYp6xCo1UK7DKxL5tMsuxCcw9Iqa/RZGwbpW8W0NFeEVi4kbViUe?= =?utf-8?q?hsinNKd4ZJ0V69cCfa5ubSQs+NAjbLFHYBVzzj+KRX3UHEW6FZPmEpje+KCbfkKC5?= =?utf-8?q?v5gJzIpjGa+Impz896n/C27DbBlL2FxkrcaMacMQ6kl2cw0Nyp+vb8yTG/xzy71fQ?= =?utf-8?q?iF55L98FDiZLX0jvHzqoU6aOQMwAYmDyXLdJrGA9jwgO0m0N7t3Ob9MeitEUxhUOd?= =?utf-8?q?rB3IGcx8myvdCxQeVjBMOYKn4Z4ne/Uw3Cpcw54Eff9WKEBFjONaktFvGAqoV2C2I?= =?utf-8?q?E6mRsq9R+wwNg6KdKmMAtUYXbhw81ZSzN7jdfO5kkuXZsPXkB7ZTNfB/Hzd3cRCnm?= =?utf-8?q?DH7mfPojcM8NlDjfbd8GAJdNBHsZhv3tlJQJXFx1Sc5WnvXij/k9mY1rur5ehdBar?= =?utf-8?q?CIzbOA9QlbjrIZJitM96Vt6JkMaYQ1ew7zY8N9lwa/FE5zW+uURfhLlpfi0551cqM?= =?utf-8?q?fISf6UOWbRDxe?= MIME-Version: 1.0 X-OriginatorOrg: microsoft.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: PA4PR83MB0526.EURPRD83.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 6e1dc4a9-ba3a-42e0-1458-08dc312d8b31 X-MS-Exchange-CrossTenant-originalarrivaltime: 19 Feb 2024 09:31:25.7767 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 72f988bf-86f1-41af-91ab-2d7cd011db47 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: jyZonPZm6J3fupPPQbE1NmBhq2pcpsh0aON5qMBNIxj8NqON7/l6K32BL0+Ih+DdFkvupknBcRj+JQzebTgO9w== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DBAPR83MB0421 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 Add common macros for extracting parts of VLAN tag. Fixes: 4e9c73e96e83 ("net/netvsc: add Hyper-V network device") Cc: sthemmin@microsoft.com Cc: stable@dpdk.org Signed-off-by: Alan Elder Reviewed-by: Ferruh Yigit --- v7: * Split patches for lib and driver v6: * Line length can be 100 - un-split lines v5: * Move the VLAN parsing macros to rte_ether.h v4: * Make consistent with FreeBSD code --- .mailmap | 1 + lib/net/rte_ether.h | 14 ++++++++++++++ 2 files changed, 15 insertions(+) -- 2.25.1 diff --git a/.mailmap b/.mailmap index de339562f4..08fce0c472 100644 --- a/.mailmap +++ b/.mailmap @@ -33,6 +33,7 @@ Alain Leon Alan Brady Alan Carew Alan Dewar +Alan Elder Alan Liu Alan Winkowski Alejandro Lucero diff --git a/lib/net/rte_ether.h b/lib/net/rte_ether.h index ce073ea818..75285bdd12 100644 --- a/lib/net/rte_ether.h +++ b/lib/net/rte_ether.h @@ -46,6 +46,20 @@ extern "C" { #define RTE_ETHER_MIN_MTU 68 /**< Minimum MTU for IPv4 packets, see RFC 791. */ +/* VLAN header fields */ +#define RTE_VLAN_DEI_SHIFT 12 +#define RTE_VLAN_PRI_SHIFT 13 +#define RTE_VLAN_PRI_MASK 0xe000 /* Priority Code Point */ +#define RTE_VLAN_DEI_MASK 0x1000 /* Drop Eligible Indicator */ +#define RTE_VLAN_ID_MASK 0x0fff /* VLAN Identifier */ + +#define RTE_VLAN_TCI_ID(vlan_tci) ((vlan_tci) & RTE_VLAN_ID_MASK) +#define RTE_VLAN_TCI_PRI(vlan_tci) (((vlan_tci) & RTE_VLAN_PRI_MASK) >> RTE_VLAN_PRI_SHIFT) +#define RTE_VLAN_TCI_DEI(vlan_tci) (((vlan_tci) & RTE_VLAN_DEI_MASK) >> RTE_VLAN_DEI_SHIFT) +#define RTE_VLAN_TCI_MAKE(id, pri, dei) ((id) | \ + ((pri) << RTE_VLAN_PRI_SHIFT) | \ + ((dei) << RTE_VLAN_DEI_SHIFT)) + /** * Ethernet address: * A universally administered address is uniquely assigned to a device by its From patchwork Mon Feb 19 09:31:39 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alan Elder X-Patchwork-Id: 136872 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 BDC8343B4B; Mon, 19 Feb 2024 10:31:42 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id AD15140685; Mon, 19 Feb 2024 10:31:42 +0100 (CET) Received: from EUR05-DB8-obe.outbound.protection.outlook.com (mail-db8eur05on2134.outbound.protection.outlook.com [40.107.20.134]) by mails.dpdk.org (Postfix) with ESMTP id 853084067E for ; Mon, 19 Feb 2024 10:31:41 +0100 (CET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=XZhCmNT8ULHMZHNKKBV++QGKz4vmESHpu1fwZ89dmYb0o7vD+s5MBTrHp0YS9k7ZdHPhidGJD5JuZ4TyOifK5IoXV7RZht522ws61eYm5DZSNCkvZDmoEJVHPwE78YV65c3DhQ02PExvA7Ugu87VTfIG3A2BdhZ9eltXtFqQ9wu5gol4hNdnoPcbf1/zn0YwFkncoLwKeG7YvQUrXjOkISkWVqB6VK4eXF9/x1nZ/PBjQyXFZApfrgjyqeUO+hkx/OuJ/erQeiAGy/6VsWKSRGRl5F/mb9XdMJDdQDEpfkAmdGlwv2hieSD5m3ail6EzRJOZK8xVIrw+e6Niu3k5Fg== 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=qfyaOztTc2o5uq9hrypsd2Itf+Tv8e0AXEz0+NI3/0s=; b=P2zijpV554nvOjYFGN1tYkYFTuUdRfiODGtdft2K6y0z40FQJQ6yhnSoqsck4ISgSELi+SQZLFXUF/x4apfZYBKw2vUn5mc+1oOD1bJ77HHXbdQ8/CNwAYfxOr8VSb3oHJcSJNN0DrKt3RxJgb8tVzp1CuPXGqr0eIlPe8UrtzoNyxei0lEl1BVqtLCuSptBLI0yWPwe/SViHKUZXmlBbA7gSw//A2cK028z7VK0QZ5E1H9TYeevRfNUcWuusy6MOfMFHlTCFazpB+/1Pp0fZQJgHIn7xb6k/+nZoo1SyaBu8GOrSEihqeEqDLxy9LoVNeP7vKcASTkrVRXp8O16gw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=microsoft.com; dmarc=pass action=none header.from=microsoft.com; dkim=pass header.d=microsoft.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=qfyaOztTc2o5uq9hrypsd2Itf+Tv8e0AXEz0+NI3/0s=; b=VyJJ3V/bAIssoIHC92OQ6TZbFL5465yAI1uf9g/B6q2/lRC71Z74lSy8uUwzUCGpnlRF1HywkoWzRvOv/4ZtPjCyYW5HAFdWbpvyvnnJ/4aHbeZ3Q+C0sZkv1cNalOS0PdPmdVqB/LQZXhqIZAzTtVviA78F7UNDgJTBTuVkmqQ= Received: from PA4PR83MB0526.EURPRD83.prod.outlook.com (2603:10a6:102:26b::17) by DBAPR83MB0421.EURPRD83.prod.outlook.com (2603:10a6:10:190::5) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7339.6; Mon, 19 Feb 2024 09:31:39 +0000 Received: from PA4PR83MB0526.EURPRD83.prod.outlook.com ([fe80::3256:dd6f:ad59:ee34]) by PA4PR83MB0526.EURPRD83.prod.outlook.com ([fe80::3256:dd6f:ad59:ee34%7]) with mapi id 15.20.7339.001; Mon, 19 Feb 2024 09:31:39 +0000 From: Alan Elder To: Ferruh Yigit , Long Li , Andrew Rybchenko CC: "dev@dpdk.org" , stephen Subject: [PATCH v7 2/2] net/netvsc: fix parsing of VLAN metadata Thread-Topic: [PATCH v7 2/2] net/netvsc: fix parsing of VLAN metadata Thread-Index: AQHaYxZw6hmKT4BOwkOfjURn3nVrtQ== Date: Mon, 19 Feb 2024 09:31:39 +0000 Message-ID: References: <8139ddff-aabf-4ee0-a51b-a2a658691624@amd.com> In-Reply-To: <8139ddff-aabf-4ee0-a51b-a2a658691624@amd.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: msip_labels: MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_ActionId=7ae1a5e5-dec8-4ee2-a466-240d3d109cc8; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_ContentBits=0; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Enabled=true; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Method=Standard; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Name=Internal; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_SetDate=2024-02-19T09:07:03Z; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_SiteId=72f988bf-86f1-41af-91ab-2d7cd011db47; authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=microsoft.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: PA4PR83MB0526:EE_|DBAPR83MB0421:EE_ x-ms-office365-filtering-correlation-id: 9afebe47-ee74-4483-2163-08dc312d9350 x-ld-processed: 72f988bf-86f1-41af-91ab-2d7cd011db47,ExtAddr x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: bD3vUSrxJkQEdAewS2fbNRHEKLqGK7detkumumEmsdMLzNwntjljRX486diPFhuZ17p3atj59bzd4SClCaFmFEBwYEQ+TNbIeAw9vfMjK7pWWs8Zu8nC911S+J2qxv4lJg1pcLBTwY0nnYWtbbC1TVxq7FJvLQNbv0EoUnlNVKyTRgFiVO54BarSAZWtOdt9m9U5DAzkP0Hhh9hU6ywSsNh2QmPGJ+dRugtWGXPj4vntzKyogM134xQLiAV918zStTE9/G4IERYaMUYex5EygfOPCojglpP7JaT7W9UyDD4xtQIAp3bJ08FU7NZsmk9BO1KvLDkvILGbbsODg27nt6Vg7cYT7okcw4Z9U51nnwXOx0Q+nAeUsLh/up16MqmAiYjXK6UNdjBO2ewORde/yrifBDO0DdFcta8iz7eIvbceVJuz3KDpv4/eQcMtxBzKF3nn1kVYrTmHZBCsiULQaZyU+XlAlJMnL/9cOfBqssUseoDH9ONJZxppph489g0WRYnBMjVFxvKpFqPW503cz89FhUzBaQzZAEEOTA7nVfbKY6FaDeSRrIRSpe3s3uvriz9o9usa0H+H7RAYNR4OFVc4/EJ/EZbHYTclP+IKwHcjvtGcKN07szwQne6r6yA+ x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:PA4PR83MB0526.EURPRD83.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230031)(39860400002)(366004)(346002)(136003)(376002)(396003)(230922051799003)(451199024)(1800799012)(186009)(64100799003)(6506007)(10290500003)(9686003)(7696005)(478600001)(71200400001)(54906003)(316002)(38070700009)(83380400001)(82950400001)(82960400001)(122000001)(33656002)(86362001)(38100700002)(41300700001)(110136005)(26005)(2906002)(44832011)(55016003)(8990500004)(5660300002)(66946007)(76116006)(66476007)(66556008)(4326008)(52536014)(8676002)(64756008)(66446008)(8936002); DIR:OUT; SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?q?BOGk8CiUse/0mXPKVPqWLTXQjjGd?= =?utf-8?q?IMWm4y8VssAFBtVEWdf3b6PHbWjRmDheCjEVajngCqZnmUahDkO8SNDXQwRpwVc+g?= =?utf-8?q?32LKRiUC/GqyoAeEXquNBjZMAje/S0Ol6+9l/G1nIIFvu2l4RZ9seZG6XNI8wFm6b?= =?utf-8?q?Ow4u+KANf0TNIwyb//MVL/gyPN07CZOCVWNjkYMl4wa1UXhn/932Z+gL7uGBDf7DT?= =?utf-8?q?wEXhxsSh6a2ML7kOexiIC15e3sm5am3EeqrxlWK2yoywtp/9aVmtL5BavmSHIi15Q?= =?utf-8?q?P6iU4qvorldHqbaS+Iz3rg6OWpQNdRePZeGIUTQD53wn35pCnRe91IuCSH6aLZFQM?= =?utf-8?q?u6yLhclWSYzOm1G0xIJBsSsnTQHod4pGCDky49LF73xsSd/f5nJbFklybWm1eEK4H?= =?utf-8?q?aMAsScLCq83Dw5jj93Z0S/aNLOXZndNG8TjZtLD/Ao6xg5t77wYE0meWG6qyx59AR?= =?utf-8?q?OvmESVhyYUfdMgBH26lJxWa/DRdt549BJQRpq28aIZEUhJqkgVoHdjswsiVBu5oZp?= =?utf-8?q?ZW0SvFSj5yi5wOpSrWBTNXpHcZhXEf3TwfGHspcEwiKjXFisy0phd43tO881bs+H0?= =?utf-8?q?rSQ1nEG6LHxygDavhPUIR0p+t1hyy1S0c+wFsGVN5lqp0drCENVSYJ7RtkGUxrJ8m?= =?utf-8?q?R7d0UpiEt851IL+BLACCyXA/DRsNVEwnlRimcDOQ7olBnKL9GRACCBHXCJDITpgB9?= =?utf-8?q?3V14qkfE0x4nouWuWh2lG0h1V9UwaDy1FNJoBFQbqj9wEUvVl1+2fBg6gK4XYTfIc?= =?utf-8?q?eRr2Cu5++aWcfw2+U4MdsMsQYVtGJpp5dRNO4b1S+ytCWM1eW8NnG+8+dlN30BCtj?= =?utf-8?q?B897yFWqHKLYZvlLboKxDZDTE7YjIoo0cnyDNaT6CGaetCGLj4GpemxVlIimjuoio?= =?utf-8?q?q2ZOE3tJiglI6V33/stq+JteVPd8FU4I7ZKOgyoPnDTBUSrhEKNxPZQ14lBrzuka1?= =?utf-8?q?WSuTuL5qXEm8QIreuVLSmCmdnhUw5fiygADqkqkxxmR/idyUteHoS7qtlyAejzart?= =?utf-8?q?N8SCOI8f7jLZgA4cNrUMLRs0PUx0YC8D9Y/bEFG4uyRfzJcQIhwuDyE96fNlKVCqI?= =?utf-8?q?4uzgaWKWk7aQ/jDCMlIE4JlofPlmLHnejbYIG5ltn/bUxaeEJEo9rI8+hYts1GcS7?= =?utf-8?q?fll45XnvjtnELvTLMAK8mX6Tsa/ts1HVFuEES1uXgnFV8COkLA04XuSS/qrRCU8Qn?= =?utf-8?q?xMowMhO493KN1Y0GRM+jqXJ/MrZEBLCHxoWJvVfr1hdAjlD6Kky/PLVOTZemp3HX/?= =?utf-8?q?1Olp6u7vyhc5gZLGmLZNs5GzPHZ059iEyd0akPdJezLoGHBrjIao4/OTe8oYubeD3?= =?utf-8?q?HWFFQyZUsmcS+0WoF5yIhQqZkrsfPVhVKnW0zzN9Anq16gDUfGhYELpC+TzzkfyTP?= =?utf-8?q?N/KDxf8rTSVr/b845Gy0gIfNNOjgPTHNeSjGRRCZGNPcJX8uvPbPxnaSkOPQ2x1ui?= =?utf-8?q?UEJwFO2AKtke6?= MIME-Version: 1.0 X-OriginatorOrg: microsoft.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: PA4PR83MB0526.EURPRD83.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 9afebe47-ee74-4483-2163-08dc312d9350 X-MS-Exchange-CrossTenant-originalarrivaltime: 19 Feb 2024 09:31:39.4271 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 72f988bf-86f1-41af-91ab-2d7cd011db47 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: bYuy58WiKF7FwVDQeuiYotsnwE1nnY7Fm9Ly/kLYgpY6ZLQfjBh9PEnc3noU78eH7r/oEsPz5dqa+YfUa6W8LA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DBAPR83MB0421 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 previous code incorrectly parsed the VLAN ID and priority. If the 16-bits of VLAN ID and priority/CFI on the wire was 0123456789ABCDEF the code parsed it as 456789ABCDEF3012. There were macros defined to handle this conversion but they were not used. Fixes: 4e9c73e96e83 ("net/netvsc: add Hyper-V network device") Cc: sthemmin@microsoft.com Cc: stable@dpdk.org Signed-off-by: Alan Elder Acked-by: Ferruh Yigit --- v7: * Split into two patches, one for lib and one for driver v6: * Line length can be 100 - un-split lines v5: * Move the VLAN parsing macros to rte_ether.h v4: * Make consistent with FreeBSD code --- drivers/net/netvsc/hn_rxtx.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) -- 2.25.1 diff --git a/drivers/net/netvsc/hn_rxtx.c b/drivers/net/netvsc/hn_rxtx.c index e4f5015aa3..9bf1ec5509 100644 --- a/drivers/net/netvsc/hn_rxtx.c +++ b/drivers/net/netvsc/hn_rxtx.c @@ -612,7 +612,9 @@ static void hn_rxpkt(struct hn_rx_queue *rxq, struct hn_rx_bufinfo *rxb, RTE_PTYPE_L4_MASK); if (info->vlan_info != HN_NDIS_VLAN_INFO_INVALID) { - m->vlan_tci = info->vlan_info; + m->vlan_tci = RTE_VLAN_TCI_MAKE(NDIS_VLAN_INFO_ID(info->vlan_info), + NDIS_VLAN_INFO_PRI(info->vlan_info), + NDIS_VLAN_INFO_CFI(info->vlan_info)); m->ol_flags |= RTE_MBUF_F_RX_VLAN_STRIPPED | RTE_MBUF_F_RX_VLAN; /* NDIS always strips tag, put it back if necessary */ @@ -1332,7 +1334,9 @@ static void hn_encap(struct rndis_packet_msg *pkt, if (m->ol_flags & RTE_MBUF_F_TX_VLAN) { pi_data = hn_rndis_pktinfo_append(pkt, NDIS_VLAN_INFO_SIZE, NDIS_PKTINFO_TYPE_VLAN); - *pi_data = m->vlan_tci; + *pi_data = NDIS_VLAN_INFO_MAKE(RTE_VLAN_TCI_ID(m->vlan_tci), + RTE_VLAN_TCI_PRI(m->vlan_tci), + RTE_VLAN_TCI_DEI(m->vlan_tci)); } if (m->ol_flags & RTE_MBUF_F_TX_TCP_SEG) {