Message ID | 20211013171354.27817-1-getelson@nvidia.com (mailing list archive) |
---|---|
Headers |
Return-Path: <dev-bounces@dpdk.org> 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 A1531A0C47; Wed, 13 Oct 2021 19:14:16 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 6B8D4410E2; Wed, 13 Oct 2021 19:14:16 +0200 (CEST) Received: from NAM04-MW2-obe.outbound.protection.outlook.com (mail-mw2nam08on2072.outbound.protection.outlook.com [40.107.101.72]) by mails.dpdk.org (Postfix) with ESMTP id 1AF0F410DA for <dev@dpdk.org>; Wed, 13 Oct 2021 19:14:15 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=GS2QlhhvmDU6TP+gUGhu9Jy0hsWnBnIMwLJIisyC4fjIDlzr/waRD0StBTKZn5bekn7Slm2TFsTI8p9Ce03tN4pPF+PkoLiJnnoAP/QVQRsfJJ8ncb1/NMJCEI7W8cR2HTe0HsX+dJNhhM7DbK+WTUkST+ySSHJxTT/WsGj0G+Q+bwvrGs4RTtpWn8/80Kar1yAo7wGWFt9KNWCoi0EgKyPNZhvuoV12EZWPESeTl+dVCWOUubxWyctPzT5WBQ+JYO56pqdGuswSiBYrQ/ZTOP8FrFnHFBjMwSLrGQ+7JPuNsLe895NRRzQjXGw/nmV7pgDORM4WloGDr4JqQZwWlg== 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=EE303xXlmLf8crtHAvG60y2sOOxGqtyJmSfr+aqkwzw=; b=U52ZRQfZo//CKNIHklojEKyYgFVDwX4sXFM3DjTkA8qB6mf/pRsMmWxr83XHLOWpgoFqbn9sTKQl8pkI7URbbSELat3Q/X/HiCpLydYwf7KYWBxHs/Q7PB/Kc+/BB9vMqGWBJnl7uQsNeoUfqp9Rd/v9ICyO6zQKg18Ur17Rng6OLnSFA7nBZXka7WOrMRQJFr6ZBeShtxTS738z/dFDlB+8C02ZH0pQoDvM1QmHZE0wIev/Zpe3o2OrHGHFJRnCC7E07Qxsy3Sf+cN30GF+KwNMo1drHQVaSUrL0ZQ7vgxkSgb1J+ojs/W8Y82SFG4Rf9NmNd+RdCsfXTHmXZMAMg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 216.228.112.34) smtp.rcpttodomain=monjalon.net smtp.mailfrom=nvidia.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=nvidia.com; dkim=none (message not signed); arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=EE303xXlmLf8crtHAvG60y2sOOxGqtyJmSfr+aqkwzw=; b=fDapqvEMZhPMuUOnDMcngHw6B7WZn/H/VEg6fXnRQUoy3TU0mdGOYd09f8Xd5fm7BsGgY/AZp4OBp18Ve2kOfKP5t0wXWl6e6VV9kW8RqTBeiGS117bbzSLW/kUlBN6sVkkFG8BzYflzb5+Uuw+ZvA08lQ3UxnsCBwQ6wi/OCQHqpFBs/JvvDbQzmxqzerudy01pCDjsPkbypWPISu4OP9nLn9gpxr09wDJhrFwUdQTMZxrJ17QnWFXdmLh/Ub+j0Mh0yc61pPXw8KmDAG5WkZc0dDU6tUYrFcwx2Zpg7hOo6jI7jPdpBeOi6pyT6u1JMJYxKuAGJST6mV0LsYXZ6A== Received: from MWHPR15CA0063.namprd15.prod.outlook.com (2603:10b6:301:4c::25) by BL0PR12MB2420.namprd12.prod.outlook.com (2603:10b6:207:4c::23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4587.19; Wed, 13 Oct 2021 17:14:12 +0000 Received: from CO1NAM11FT044.eop-nam11.prod.protection.outlook.com (2603:10b6:301:4c:cafe::ed) by MWHPR15CA0063.outlook.office365.com (2603:10b6:301:4c::25) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4608.15 via Frontend Transport; Wed, 13 Oct 2021 17:14:12 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 216.228.112.34) smtp.mailfrom=nvidia.com; monjalon.net; dkim=none (message not signed) header.d=none;monjalon.net; dmarc=pass action=none header.from=nvidia.com; Received-SPF: Pass (protection.outlook.com: domain of nvidia.com designates 216.228.112.34 as permitted sender) receiver=protection.outlook.com; client-ip=216.228.112.34; helo=mail.nvidia.com; Received: from mail.nvidia.com (216.228.112.34) by CO1NAM11FT044.mail.protection.outlook.com (10.13.175.188) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.20.4608.15 via Frontend Transport; Wed, 13 Oct 2021 17:14:11 +0000 Received: from nvidia.com (172.20.187.5) by HQMAIL107.nvidia.com (172.20.187.13) with Microsoft SMTP Server (TLS) id 15.0.1497.18; Wed, 13 Oct 2021 17:14:09 +0000 From: Gregory Etelson <getelson@nvidia.com> To: <dev@dpdk.org>, <getelson@nvidia.com> CC: <matan@nvidia.com>, <rasland@nvidia.com>, <olivier.matz@6wind.com>, <thomas@monjalon.net>, <ferruh.yigit@intel.com> Date: Wed, 13 Oct 2021 20:13:52 +0300 Message-ID: <20211013171354.27817-1-getelson@nvidia.com> X-Mailer: git-send-email 2.33.0 In-Reply-To: <20210527152858.13312-1-getelson@nvidia.com> References: <20210527152858.13312-1-getelson@nvidia.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-Originating-IP: [172.20.187.5] X-ClientProxiedBy: HQMAIL111.nvidia.com (172.20.187.18) To HQMAIL107.nvidia.com (172.20.187.13) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: b3c054df-245c-42d1-1efc-08d98e6ce055 X-MS-TrafficTypeDiagnostic: BL0PR12MB2420: X-LD-Processed: 43083d15-7273-40c1-b7db-39efd9ccc17a,ExtAddr X-Microsoft-Antispam-PRVS: <BL0PR12MB24205A46A1FC3936EC8E8F5FA5B79@BL0PR12MB2420.namprd12.prod.outlook.com> X-MS-Oob-TLC-OOBClassifiers: OLM:1728; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 4ccdK/7oAWj+/1gXnUBG//0R4tcv8y1bnQFA6ibRIBoHiWVfIF7XTgEpjoZykN65SqDXpaNtiw1jdWBrGrtgGDoFgVudHd0WFd2gcNwz/N9pmkJfZ8EuxxLryOwJlVRJIlm5VsciUjel4S2aG3ffBqHj08KgI7mXWLHBQuggHf7TIPK7scHWqyUUQ0L+mzgqlfwF4Fv6vsGwxlyVURz8o33sg7PxsbokrgItjW84u92iGYADfOqXhuNL2Bo+TPSXOf23RbNpzrnD0GyiD4aOOpUYmxXNmI0dT3duNTYoKuTFQbxg3u/JAr5pDo5S9SfGohRewaPWb/Survi6OJU/5XjRDaKEk73uWCrxOtxK13QugaHjYEy2SGz1p0uZHTRJ9YKNUf+OuTegnD2IhQTkyzKnjb53KSUZTE8GgjSKwp9hql/zirxAo8j6TDbQGATgXhwYBvag9VPIbTtBof9y1YCyGPCkzAa7qkAZ82+jyfjFCn182izbAofMRAsd3qAdp95PL7nUhSTwyockfvI4nHhKuwh+h0ijbTDmRrqEl9QQNaNqLM4/DQVZtjkj4JAmYZh33dwTeaNc3rZMKvBQuiZT93LlQBltOBrK7gDLBeEtQq83q9E/2xEGDmz0S267H2tiwEtwVWddcCOgsvo6nn14HGwQqEIkbIGXjjqZ+LiQrPAI0658k6KjOPPaIS9H4M/uXETYqqwtQms2D5ztRw== X-Forefront-Antispam-Report: CIP:216.228.112.34; CTRY:US; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:mail.nvidia.com; PTR:schybrid03.nvidia.com; CAT:NONE; SFS:(4636009)(36840700001)(46966006)(36860700001)(86362001)(110136005)(356005)(47076005)(5660300002)(55016002)(54906003)(6286002)(7636003)(7696005)(508600001)(7049001)(4326008)(336012)(83380400001)(26005)(8676002)(2906002)(316002)(82310400003)(6666004)(8936002)(1076003)(426003)(36756003)(186003)(16526019)(2616005)(70206006)(4744005)(70586007); DIR:OUT; SFP:1101; X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Oct 2021 17:14:11.9523 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: b3c054df-245c-42d1-1efc-08d98e6ce055 X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=43083d15-7273-40c1-b7db-39efd9ccc17a; Ip=[216.228.112.34]; Helo=[mail.nvidia.com] X-MS-Exchange-CrossTenant-AuthSource: CO1NAM11FT044.eop-nam11.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL0PR12MB2420 Subject: [dpdk-dev] [PATCH v6 0/2] net: introduce IPv4 ihl and version fields X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions <dev.dpdk.org> List-Unsubscribe: <https://mails.dpdk.org/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://mails.dpdk.org/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <https://mails.dpdk.org/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Series |
net: introduce IPv4 ihl and version fields
|
|
Message
Gregory Etelson
Oct. 13, 2021, 5:13 p.m. UTC
Gregory Etelson (2): net: fix IPv4 change announce net: introduce IPv4 ihl and version fields app/test/test_flow_classify.c | 8 ++++---- doc/guides/rel_notes/deprecation.rst | 6 ------ doc/guides/rel_notes/release_21_11.rst | 3 +++ lib/net/rte_ip.h | 16 +++++++++++++++- 4 files changed, 22 insertions(+), 11 deletions(-)
Comments
On 10/13/2021 6:13 PM, Gregory Etelson wrote: > Gregory Etelson (2): > net: fix IPv4 change announce > net: introduce IPv4 ihl and version fields > Hi Gregory, Can you please change the order of the first and second patch? This way I can get the first one, since it is already acked, before -rc1, and continue reviews for second one, it will be OK since it is a doc patch. Thanks, ferruh
14/10/2021 10:21, Ferruh Yigit: > On 10/13/2021 6:13 PM, Gregory Etelson wrote: > > Gregory Etelson (2): > > net: fix IPv4 change announce > > net: introduce IPv4 ihl and version fields > > > > Hi Gregory, > > Can you please change the order of the first and second patch? > > This way I can get the first one, since it is already acked, before -rc1, > and continue reviews for second one, it will be OK since it is a > doc patch. It makes more sense in this order I think. The first patch is just dropping a deprecation note, I can ack.
Hello Ferruh, > On 10/13/2021 6:13 PM, Gregory Etelson wrote: > > Gregory Etelson (2): > > net: fix IPv4 change announce > > net: introduce IPv4 ihl and version fields > > > > Hi Gregory, > > Can you please change the order of the first and > second patch? > In the existing order, the code patch reflects announced changes. > This way I can get the first one, since it is already > acked, before -rc1, > and continue reviews for second one, it will be > OK since it is a > doc patch. > > Thanks, > ferruh
On 10/14/2021 9:30 AM, Thomas Monjalon wrote: > 14/10/2021 10:21, Ferruh Yigit: >> On 10/13/2021 6:13 PM, Gregory Etelson wrote: >>> Gregory Etelson (2): >>> net: fix IPv4 change announce >>> net: introduce IPv4 ihl and version fields >>> >> >> Hi Gregory, >> >> Can you please change the order of the first and second patch? >> >> This way I can get the first one, since it is already acked, before -rc1, >> and continue reviews for second one, it will be OK since it is a >> doc patch. > > It makes more sense in this order I think. > The first patch is just dropping a deprecation note, I can ack. > It will be same I think, first patch can have implementation and remove the implemented part of the deprecation notice, remaining deprecation notice part can be removed with or without its implementation later. Anyway, this is for operational needs, if the first patch gets enough ack/review timely, not update is required and I can get both patches.
On 10/14/2021 10:29 AM, Gregory Etelson wrote: > Hello Ferruh, > >> On 10/13/2021 6:13 PM, Gregory Etelson wrote: >>> Gregory Etelson (2): >>> net: fix IPv4 change announce >>> net: introduce IPv4 ihl and version fields >>> >> >> Hi Gregory, >> >> Can you please change the order of the first and >> second patch? >> > > In the existing order, the code patch reflects announced changes. > Overall you are not implementing the announced changes fully, but partially. Question is how to manage announced but not implemented part. I am for separating discussion of that part from the code change that already acked. >> This way I can get the first one, since it is already >> acked, before -rc1, >> and continue reviews for second one, it will be >> OK since it is a >> doc patch. >> >> Thanks, >> ferruh