[dpdk-dev] gro: fix missing includes

Message ID 20170804101227.3698-1-thomas@monjalon.net (mailing list archive)
State Accepted, archived
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Thomas Monjalon Aug. 4, 2017, 10:12 a.m. UTC
  The GRO header file depends on stdint and mbuf.

Spotted with devtools/check-includes.sh

Fixes: e996506a1c07 ("lib/gro: add Generic Receive Offload API framework")

Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
---
 lib/librte_gro/rte_gro.h | 3 +++
 1 file changed, 3 insertions(+)
  

Comments

John McNamara Aug. 4, 2017, 12:30 p.m. UTC | #1
> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Thomas Monjalon
> Sent: Friday, August 4, 2017 11:12 AM
> To: Hu, Jiayu <jiayu.hu@intel.com>
> Cc: dev@dpdk.org
> Subject: [dpdk-dev] [PATCH] gro: fix missing includes
> 
> The GRO header file depends on stdint and mbuf.
> 
> Spotted with devtools/check-includes.sh
> 
> Fixes: e996506a1c07 ("lib/gro: add Generic Receive Offload API framework")
> 
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>


Acked-by: John McNamara <john.mcnamara@intel.com>
  
Thomas Monjalon Aug. 4, 2017, 11:19 p.m. UTC | #2
> > The GRO header file depends on stdint and mbuf.
> > 
> > Spotted with devtools/check-includes.sh
> > 
> > Fixes: e996506a1c07 ("lib/gro: add Generic Receive Offload API framework")
> > 
> > Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> 
> Acked-by: John McNamara <john.mcnamara@intel.com>

Applied
  

Patch

diff --git a/lib/librte_gro/rte_gro.h b/lib/librte_gro/rte_gro.h
index 69721a9c0..d57e0c5f9 100644
--- a/lib/librte_gro/rte_gro.h
+++ b/lib/librte_gro/rte_gro.h
@@ -38,6 +38,9 @@ 
  * Interface to GRO library
  */
 
+#include <stdint.h>
+#include <rte_mbuf.h>
+
 #ifdef __cplusplus
 extern "C" {
 #endif