diff mbox series

[v9,08/10] app/testpmd: fix headers inclusion

Message ID	1619805162-10684-9-git-send-email-jizh@linux.microsoft.com (mailing list archive)
State	Superseded, archived
Delegated to:	Ferruh Yigit
Headers	From: Jie Zhou <jizh@linux.microsoft.com> To: dev@dpdk.org Cc: dmitry.kozliuk@gmail.com, xiaoyun.li@intel.com, roretzla@microsoft.com, talshn@nvidia.com, pallavi.kadam@intel.com, thomas@monjalon.net, bruce.richardson@intel.com, ferruh.yigit@intel.com, konstantin.ananyev@intel.com Date: Fri, 30 Apr 2021 10:52:40 -0700 Message-Id: <1619805162-10684-9-git-send-email-jizh@linux.microsoft.com> In-Reply-To: <1619805162-10684-1-git-send-email-jizh@linux.microsoft.com> References: <1619553721-5220-1-git-send-email-jizh@linux.microsoft.com> <1619805162-10684-1-git-send-email-jizh@linux.microsoft.com> Subject: [dpdk-dev] [PATCH v9 08/10] app/testpmd: fix headers inclusion Precedence: list Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org>
Series	app/testpmd: enable testpmd on Windows \| [v9,00/10] app/testpmd: enable testpmd on Windows [v9,01/10] lib: build libraries that testpmd depends on [v9,02/10] eal/windows: add necessary macros [v9,03/10] eal/windows: add device event stubs [v9,04/10] eal/Windows: add clock_gettime on Windows [v9,05/10] app/testpmd: resolve name collisions [v9,06/10] app/testpmd: fix parse_fec_mode return type [v9,07/10] app/testpmd: replace POSIX specific code [v9,08/10] app/testpmd: fix headers inclusion [v9,09/10] app/testpmd: fix unused function warnings [v9,10/10] app/testpmd: enable building testpmd on Windows

Checks

Context	Check	Description
ci/checkpatch	success	coding style OK

Commit Message

Jie Zhou April 30, 2021, 5:52 p.m. UTC

  - Include rte_os_shim.h in testpmd.h
- Remove redundant headers

Signed-off-by: Jie Zhou <jizh@microsoft.com>
Signed-off-by: Jie Zhou <jizh@linux.microsoft.com>
---
 app/test-pmd/cmdline.c      | 3 ---
 app/test-pmd/cmdline_flow.c | 2 --
 app/test-pmd/parameters.c   | 1 -
 app/test-pmd/testpmd.h      | 1 +
 4 files changed, 1 insertion(+), 6 deletions(-)

Comments

Tal Shnaiderman May 3, 2021, 5:44 p.m. UTC | #1

> Subject: [PATCH v9 08/10] app/testpmd: fix headers inclusion
> 
> External email: Use caution opening links or attachments
> 
> 
> - Include rte_os_shim.h in testpmd.h
> - Remove redundant headers
> 
> Signed-off-by: Jie Zhou <jizh@microsoft.com>
> Signed-off-by: Jie Zhou <jizh@linux.microsoft.com>
> ---
>  app/test-pmd/cmdline.c      | 3 ---
>  app/test-pmd/cmdline_flow.c | 2 --
>  app/test-pmd/parameters.c   | 1 -
>  app/test-pmd/testpmd.h      | 1 +
>  4 files changed, 1 insertion(+), 6 deletions(-)
> 
> diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c index
> f0636ca9b..2fbef3320 100644
> --- a/app/test-pmd/cmdline.c
> +++ b/app/test-pmd/cmdline.c
> @@ -10,9 +10,6 @@
>  #include <string.h>
>  #include <unistd.h>
>  #include <inttypes.h>
> -#include <sys/socket.h>
> -#include <netinet/in.h>
> -
>  #include <sys/queue.h>
> 
>  #include <rte_common.h>
> diff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c
> index 888b9179b..a3cd4773a 100644
> --- a/app/test-pmd/cmdline_flow.c
> +++ b/app/test-pmd/cmdline_flow.c
> @@ -10,8 +10,6 @@
>  #include <errno.h>
>  #include <ctype.h>
>  #include <string.h>
> -#include <arpa/inet.h>
> -#include <sys/socket.h>
> 
>  #include <rte_string_fns.h>
>  #include <rte_common.h>
> diff --git a/app/test-pmd/parameters.c b/app/test-pmd/parameters.c index
> 4c3cbbac3..5e69d2aa8 100644
> --- a/app/test-pmd/parameters.c
> +++ b/app/test-pmd/parameters.c
> @@ -19,7 +19,6 @@
>  #include <stdint.h>
>  #include <unistd.h>
>  #include <inttypes.h>
> -#include <arpa/inet.h>
> 
>  #include <rte_common.h>
>  #include <rte_byteorder.h>
> diff --git a/app/test-pmd/testpmd.h b/app/test-pmd/testpmd.h index
> a4115861b..f96eec71a 100644
> --- a/app/test-pmd/testpmd.h
> +++ b/app/test-pmd/testpmd.h
> @@ -11,6 +11,7 @@
>  #include <rte_bus_pci.h>
>  #include <rte_gro.h>
>  #include <rte_gso.h>
> +#include <rte_os_shim.h>
>  #include <cmdline.h>
>  #include <sys/queue.h>
> 
> --
> 2.30.0.vfs.0.2

Acked-by: Tal Shnaiderman <talshn@nvidia.com>

diff mbox series

Patch

diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c
index f0636ca9b..2fbef3320 100644
--- a/app/test-pmd/cmdline.c
+++ b/app/test-pmd/cmdline.c
@@ -10,9 +10,6 @@ 
 #include <string.h>
 #include <unistd.h>
 #include <inttypes.h>
-#include <sys/socket.h>
-#include <netinet/in.h>
-
 #include <sys/queue.h>
 
 #include <rte_common.h>
diff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c
index 888b9179b..a3cd4773a 100644
--- a/app/test-pmd/cmdline_flow.c
+++ b/app/test-pmd/cmdline_flow.c
@@ -10,8 +10,6 @@ 
 #include <errno.h>
 #include <ctype.h>
 #include <string.h>
-#include <arpa/inet.h>
-#include <sys/socket.h>
 
 #include <rte_string_fns.h>
 #include <rte_common.h>
diff --git a/app/test-pmd/parameters.c b/app/test-pmd/parameters.c
index 4c3cbbac3..5e69d2aa8 100644
--- a/app/test-pmd/parameters.c
+++ b/app/test-pmd/parameters.c
@@ -19,7 +19,6 @@ 
 #include <stdint.h>
 #include <unistd.h>
 #include <inttypes.h>
-#include <arpa/inet.h>
 
 #include <rte_common.h>
 #include <rte_byteorder.h>
diff --git a/app/test-pmd/testpmd.h b/app/test-pmd/testpmd.h
index a4115861b..f96eec71a 100644
--- a/app/test-pmd/testpmd.h
+++ b/app/test-pmd/testpmd.h
@@ -11,6 +11,7 @@ 
 #include <rte_bus_pci.h>
 #include <rte_gro.h>
 #include <rte_gso.h>
+#include <rte_os_shim.h>
 #include <cmdline.h>
 #include <sys/queue.h>