Message ID | 20210915162603.16652-1-talshn@nvidia.com (mailing list archive) |
---|---|
State | Superseded, archived |
Delegated to: | akhil goyal |
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 45F76A0C41; Wed, 15 Sep 2021 18:27:05 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 309534068F; Wed, 15 Sep 2021 18:27:05 +0200 (CEST) Received: from NAM11-DM6-obe.outbound.protection.outlook.com (mail-dm6nam11on2045.outbound.protection.outlook.com [40.107.223.45]) by mails.dpdk.org (Postfix) with ESMTP id 85B524014F for <dev@dpdk.org>; Wed, 15 Sep 2021 18:27:03 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=b0mcHco2b3PdfR933/0QCB/1+/iRo2msk61OtniiJA817rnI31XD4OLZvuaT+mqJ8x3ssVR121HQX9m04A/DS+/DIHIMQa88pZuKk/cT0v364fkozmh75s3EbERRk56MVPRAcUEJzZYajdYv2bGAdZJ5INmxQqPkDI3jeiVFv3n3iIdQE+3BT8GRBJzFSQEpntSwB57Qn1/G9tp0Mg42sMfgo1eZ2M20uOaWzvZSX27u5X51SBUi4V0/UhcGEWENn8qb9LWA72QOEXQcEttyqY5Vi7Dgal7swPNUVc1/5C9ZKBC68K5C18LfY/sl7poDjjOKjBn+hdvk5mvelWgvxg== 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; bh=RPCX7/4E5pV8gur2i4im06Vt5svg0wPSrnyIwKEU238=; b=cF2p4AI7Z5hfSNFwaaHXnS3zaEZ/blKKbTE3ti1B0stZQWExel0lnZqOMN907gI5OgdL7QUCFD7BolMjyM8vRXp/N82isy7+A1NaWHaLrhTOfUDUpyu4qvhiP3kn+H2x8fQCan0GPMghTU+HBmjJS3Y2Q06gdHp12De6GIuPs64IHdwT6E8qrHvFg3SuQ/Ls1GVS6MSwi+NHq+HvGh+5bZIQIEbiyWPxg3r130tcnPJdTA6ncM0qmhLdh70CbfsM9tZq7mw9c2yXOGgNxgygPQCXMA4wXBGF7qj8Ro1OgAUhNcuu9OkH00tlHAXIM7/G2wYsSBD96goCa0vBS6q1Gg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 216.228.112.36) smtp.rcpttodomain=dpdk.org smtp.mailfrom=nvidia.com; dmarc=pass (p=quarantine sp=none 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=RPCX7/4E5pV8gur2i4im06Vt5svg0wPSrnyIwKEU238=; b=k/v2QTri5q+Kt/U4ePT3ic/XfqSgnFGx43+GX75nrs6EspG6qebG5/EfQGk+eruK7s/G+SyZ60SCzh4KcahA3ELeD82x4z+WxNmW3I5KiorW8gZPyciRtZRIY4AZ9njqCnG+9YVEXPwuaBNC8UCJfcadKrwvWcwj7Ltljf5DCtBenAnqMlTNYxYQxzf5BcdSeTUadQgBU1SnaFzj+N1UBUUd9y7tM8OTD/2/ZfFTtgABWE92ET5SOeHEcXzetsMW1mjNzZZBdJzb7yjsmE7NbtFVauwQ9VEmliTyLxSURXinehQp9j61nB9IcCoG7Q92xg0TC6j67XD0FqDPfQ0xsw== Received: from DM6PR01CA0008.prod.exchangelabs.com (2603:10b6:5:296::13) by MN2PR12MB4566.namprd12.prod.outlook.com (2603:10b6:208:26a::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4523.14; Wed, 15 Sep 2021 16:27:02 +0000 Received: from DM6NAM11FT023.eop-nam11.prod.protection.outlook.com (2603:10b6:5:296:cafe::d5) by DM6PR01CA0008.outlook.office365.com (2603:10b6:5:296::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4523.14 via Frontend Transport; Wed, 15 Sep 2021 16:27:02 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 216.228.112.36) smtp.mailfrom=nvidia.com; dpdk.org; dkim=none (message not signed) header.d=none;dpdk.org; dmarc=pass action=none header.from=nvidia.com; Received-SPF: Pass (protection.outlook.com: domain of nvidia.com designates 216.228.112.36 as permitted sender) receiver=protection.outlook.com; client-ip=216.228.112.36; helo=mail.nvidia.com; Received: from mail.nvidia.com (216.228.112.36) by DM6NAM11FT023.mail.protection.outlook.com (10.13.173.96) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.20.4523.14 via Frontend Transport; Wed, 15 Sep 2021 16:27:02 +0000 Received: from DRHQMAIL107.nvidia.com (10.27.9.16) by HQMAIL101.nvidia.com (172.20.187.10) with Microsoft SMTP Server (TLS) id 15.0.1497.18; Wed, 15 Sep 2021 16:27:01 +0000 Received: from nvidia.com (172.20.187.5) by DRHQMAIL107.nvidia.com (10.27.9.16) with Microsoft SMTP Server (TLS) id 15.0.1497.18; Wed, 15 Sep 2021 16:26:59 +0000 From: Tal Shnaiderman <talshn@nvidia.com> To: <dev@dpdk.org> CC: <thomas@monjalon.net>, <pallavi.kadam@intel.com>, <dmitry.kozliuk@gmail.com>, <navasile@linux.microsoft.com>, <dmitrym@microsoft.com>, <gakhil@marvell.com>, <declan.doherty@intel.com> Date: Wed, 15 Sep 2021 19:26:02 +0300 Message-ID: <20210915162603.16652-1-talshn@nvidia.com> X-Mailer: git-send-email 2.16.1.windows.4 MIME-Version: 1.0 Content-Type: text/plain X-Originating-IP: [172.20.187.5] X-ClientProxiedBy: HQMAIL111.nvidia.com (172.20.187.18) To DRHQMAIL107.nvidia.com (10.27.9.16) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 33097ec9-438d-41a4-8180-08d97865a5f7 X-MS-TrafficTypeDiagnostic: MN2PR12MB4566: X-LD-Processed: 43083d15-7273-40c1-b7db-39efd9ccc17a,ExtAddr X-Microsoft-Antispam-PRVS: <MN2PR12MB4566D625AD3E46D3158827ADA4DB9@MN2PR12MB4566.namprd12.prod.outlook.com> X-MS-Oob-TLC-OOBClassifiers: OLM:4714; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: F9YFxaKEkKT6LFtuUlgXcn66eXlE1LIu46tH0w3qC1Kjo6pH1D7f4PFsci2WvgQU33xSQTxfQp7+rLUaU8fOqLKT9yccoawDACnHpWXsW9PfJQSTu6zdDz6IF8NGS/5KvDKG2LUOh/3AaQsYtGjZoCMrmPnHd1BvMhG8VZoduhfaPuIW4yWoqSk3hExOedvNlyXmE4ldKCFz8fo+Ghlyotofui1RMLcuVoB3mbIwxaS6F1moonWjoQmNJ5KTcLKseQBVBbmrEH04ZK1FuNpZaNhjZrXO3oexGPeBgEuakNk1tuvjvJEXnIcMPWBO+LT8AGA0pFmcfOUO21ijjQ/Uc/pPeB3JSCUx1OWFrCvykqnhp2Cted6lvyQgZbxLlTZdbRX7KMmfGOiU1pHl5A0jOjDX8MDbUncT72e559M2K3GH0Su3aATmqprQ2NmlKCZc9Mwr47jXULn8N67B8yLNVJRkABLf8+TlcrIrB6ijfr0cEZ6bc+8xqgVMXgCJhVh9KEEIQbgs+KVcQJxP+t9XKb/09RULq6A73FfS4OTkd0yecFmKvSCJEtLD53W3vnqU4GWeXkmYN9F5MI9mK2fZhBHV/lJZ5NDRhXgPirqoYxO0I+TNE/vCFAvAJXcExXMDFvGQET8Mt29GtGR02VwaI+bdn56VZ04FO0VKz0lMKcTlk/ofiy4szOzG6FKBN6zhrguLoEaAJTyN05HcrUAeAg== X-Forefront-Antispam-Report: CIP:216.228.112.36; CTRY:US; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:mail.nvidia.com; PTR:schybrid05.nvidia.com; CAT:NONE; SFS:(4636009)(346002)(136003)(396003)(376002)(39860400002)(36840700001)(46966006)(356005)(2906002)(336012)(36756003)(5660300002)(54906003)(47076005)(426003)(7696005)(70586007)(82740400003)(70206006)(26005)(83380400001)(186003)(8676002)(6286002)(6916009)(1076003)(8936002)(2616005)(36906005)(7636003)(82310400003)(478600001)(316002)(16526019)(4326008)(36860700001)(15650500001)(55016002)(6666004)(86362001); DIR:OUT; SFP:1101; X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Sep 2021 16:27:02.0007 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 33097ec9-438d-41a4-8180-08d97865a5f7 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.36]; Helo=[mail.nvidia.com] X-MS-Exchange-CrossTenant-AuthSource: DM6NAM11FT023.eop-nam11.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR12MB4566 Subject: [dpdk-dev] [PATCH 1/2] security: use the net library for IP structs 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 |
[1/2] security: use the net library for IP structs
|
|
Checks
Context | Check | Description |
---|---|---|
ci/checkpatch | success | coding style OK |
Commit Message
Tal Shnaiderman
Sept. 15, 2021, 4:26 p.m. UTC
Remove the netinet includes and replaces them
with rte_ip.h to support the in_addr/in6_addr structs
on all operating systems.
Signed-off-by: Tal Shnaiderman <talshn@nvidia.com>
---
lib/net/rte_ip.h | 1 +
lib/security/meson.build | 2 +-
lib/security/rte_security.h | 5 +----
3 files changed, 3 insertions(+), 5 deletions(-)
Comments
> Remove the netinet includes and replaces them > with rte_ip.h to support the in_addr/in6_addr structs > on all operating systems. > > Signed-off-by: Tal Shnaiderman <talshn@nvidia.com> > --- > lib/net/rte_ip.h | 1 + > lib/security/meson.build | 2 +- > lib/security/rte_security.h | 5 +---- > 3 files changed, 3 insertions(+), 5 deletions(-) > > diff --git a/lib/net/rte_ip.h b/lib/net/rte_ip.h > index 05948b69b7..bbd8650962 100644 > --- a/lib/net/rte_ip.h > +++ b/lib/net/rte_ip.h > @@ -25,6 +25,7 @@ > #include <netinet/in.h> > #include <arpa/inet.h> > #include <netinet/ip.h> > +#include <netinet/ip6.h> > #endif > > #include <rte_byteorder.h> > diff --git a/lib/security/meson.build b/lib/security/meson.build > index 5679c8b5c2..1034a7a299 100644 > --- a/lib/security/meson.build > +++ b/lib/security/meson.build > @@ -3,4 +3,4 @@ > > sources = files('rte_security.c') > headers = files('rte_security.h', 'rte_security_driver.h') > -deps += ['mempool', 'cryptodev'] > +deps += ['mempool', 'cryptodev', 'net'] It is just a header file inclusion, will it really need to add dependency on 'net'? > diff --git a/lib/security/rte_security.h b/lib/security/rte_security.h > index 2e136d7929..0c1adcee21 100644 > --- a/lib/security/rte_security.h > +++ b/lib/security/rte_security.h > @@ -19,13 +19,10 @@ extern "C" { > > #include <sys/types.h> > > -#include <netinet/in.h> > -#include <netinet/ip.h> > -#include <netinet/ip6.h> > - > #include <rte_compat.h> > #include <rte_common.h> > #include <rte_crypto.h> > +#include <rte_ip.h> > #include <rte_mbuf.h> > #include <rte_mbuf_dyn.h> > #include <rte_memory.h> > -- > 2.16.1.windows.4
> Subject: RE: [EXT] [PATCH 1/2] security: use the net library for IP structs > > External email: Use caution opening links or attachments > > > > Remove the netinet includes and replaces them with rte_ip.h to support > > the in_addr/in6_addr structs on all operating systems. > > > > Signed-off-by: Tal Shnaiderman <talshn@nvidia.com> > > --- > > lib/net/rte_ip.h | 1 + > > lib/security/meson.build | 2 +- > > lib/security/rte_security.h | 5 +---- > > 3 files changed, 3 insertions(+), 5 deletions(-) > > > > diff --git a/lib/net/rte_ip.h b/lib/net/rte_ip.h index > > 05948b69b7..bbd8650962 100644 > > --- a/lib/net/rte_ip.h > > +++ b/lib/net/rte_ip.h > > @@ -25,6 +25,7 @@ > > #include <netinet/in.h> > > #include <arpa/inet.h> > > #include <netinet/ip.h> > > +#include <netinet/ip6.h> > > #endif > > > > #include <rte_byteorder.h> > > diff --git a/lib/security/meson.build b/lib/security/meson.build index > > 5679c8b5c2..1034a7a299 100644 > > --- a/lib/security/meson.build > > +++ b/lib/security/meson.build > > @@ -3,4 +3,4 @@ > > > > sources = files('rte_security.c') > > headers = files('rte_security.h', 'rte_security_driver.h') -deps += > > ['mempool', 'cryptodev'] > > +deps += ['mempool', 'cryptodev', 'net'] > > It is just a header file inclusion, will it really need to add dependency on 'net'? > Without it I'm getting an error of missing file on the new include, I can replace it with an ifdef similar to what rte_ip.h has and loose the dependency: #ifdef RTE_EXEC_ENV_WINDOWS #include <ws2tcpip.h> #else #include <netinet/in.h> #include <netinet/ip.h> #include <netinet/ip6.h> #endif however I think it's better to remove OS includes, unless there is an issue with adding 'net' as dependency. > > diff --git a/lib/security/rte_security.h b/lib/security/rte_security.h > > index 2e136d7929..0c1adcee21 100644 > > --- a/lib/security/rte_security.h > > +++ b/lib/security/rte_security.h > > @@ -19,13 +19,10 @@ extern "C" { > > > > #include <sys/types.h> > > > > -#include <netinet/in.h> > > -#include <netinet/ip.h> > > -#include <netinet/ip6.h> > > - > > #include <rte_compat.h> > > #include <rte_common.h> > > #include <rte_crypto.h> > > +#include <rte_ip.h> > > #include <rte_mbuf.h> > > #include <rte_mbuf_dyn.h> > > #include <rte_memory.h> > > -- > > 2.16.1.windows.4
On Wed, Sep 15, 2021 at 9:27 AM Tal Shnaiderman <talshn@nvidia.com> wrote: > > Remove the netinet includes and replaces them > with rte_ip.h to support the in_addr/in6_addr structs > on all operating systems. > > Signed-off-by: Tal Shnaiderman <talshn@nvidia.com> > --- Acked-by: William Tu <u9012063@gmail.com> I sent a similar patch to enable ixgbe support for windows. http://inbox.dpdk.org/dev/CALDO+SY_YkdV+Dx-dUJx8cyqafL6HdL=09DrOGCUBeonH2bPrA@mail.gmail.com/T/#t
diff --git a/lib/net/rte_ip.h b/lib/net/rte_ip.h index 05948b69b7..bbd8650962 100644 --- a/lib/net/rte_ip.h +++ b/lib/net/rte_ip.h @@ -25,6 +25,7 @@ #include <netinet/in.h> #include <arpa/inet.h> #include <netinet/ip.h> +#include <netinet/ip6.h> #endif #include <rte_byteorder.h> diff --git a/lib/security/meson.build b/lib/security/meson.build index 5679c8b5c2..1034a7a299 100644 --- a/lib/security/meson.build +++ b/lib/security/meson.build @@ -3,4 +3,4 @@ sources = files('rte_security.c') headers = files('rte_security.h', 'rte_security_driver.h') -deps += ['mempool', 'cryptodev'] +deps += ['mempool', 'cryptodev', 'net'] diff --git a/lib/security/rte_security.h b/lib/security/rte_security.h index 2e136d7929..0c1adcee21 100644 --- a/lib/security/rte_security.h +++ b/lib/security/rte_security.h @@ -19,13 +19,10 @@ extern "C" { #include <sys/types.h> -#include <netinet/in.h> -#include <netinet/ip.h> -#include <netinet/ip6.h> - #include <rte_compat.h> #include <rte_common.h> #include <rte_crypto.h> +#include <rte_ip.h> #include <rte_mbuf.h> #include <rte_mbuf_dyn.h> #include <rte_memory.h>