[v2,04/19] net: fix missing header include

Message ID 20210115111052.16437-5-bruce.richardson@intel.com (mailing list archive)
State Accepted, archived
Delegated to: David Marchand
Headers
Series ensure headers have correct includes |

Checks

Context Check Description
ci/checkpatch success coding style OK

Commit Message

Bruce Richardson Jan. 15, 2021, 11:10 a.m. UTC
  The Geneve protocol header file is missing the rte_byteorder.h header.

Fixes: ea0e711b8ae0 ("app/testpmd: add GENEVE parsing")
Cc: stable@dpdk.org

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
 lib/librte_net/rte_geneve.h | 1 +
 1 file changed, 1 insertion(+)
  

Comments

Ophir Munk Jan. 16, 2021, 12:10 p.m. UTC | #1
Acked-by: Ophir Munk <ophirmu@nvidia.com>

> -----Original Message-----
> From: Bruce Richardson <bruce.richardson@intel.com>
> Sent: Friday, January 15, 2021 1:11 PM
> To: dev@dpdk.org
> Cc: david.marchand@redhat.com; Bruce Richardson
> <bruce.richardson@intel.com>; stable@dpdk.org; Olivier Matz
> <olivier.matz@6wind.com>; Ophir Munk <ophirmu@mellanox.com>; Ferruh
> Yigit <ferruh.yigit@intel.com>
> Subject: [PATCH v2 04/19] net: fix missing header include
> 
> The Geneve protocol header file is missing the rte_byteorder.h header.
> 
> Fixes: ea0e711b8ae0 ("app/testpmd: add GENEVE parsing")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> ---
>  lib/librte_net/rte_geneve.h | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/lib/librte_net/rte_geneve.h b/lib/librte_net/rte_geneve.h index
> bb67724c31..7c3d477dcb 100644
> --- a/lib/librte_net/rte_geneve.h
> +++ b/lib/librte_net/rte_geneve.h
> @@ -11,6 +11,7 @@
>   * GENEVE-related definitions
>   */
>  #include <stdint.h>
> +#include <rte_byteorder.h>
> 
>  #ifdef __cplusplus
>  extern "C" {
> --
> 2.27.0
  

Patch

diff --git a/lib/librte_net/rte_geneve.h b/lib/librte_net/rte_geneve.h
index bb67724c31..7c3d477dcb 100644
--- a/lib/librte_net/rte_geneve.h
+++ b/lib/librte_net/rte_geneve.h
@@ -11,6 +11,7 @@ 
  * GENEVE-related definitions
  */
 #include <stdint.h>
+#include <rte_byteorder.h>
 
 #ifdef __cplusplus
 extern "C" {