The distributor header file includes the mbuf header file, but it does not need to do so as it only uses pointers to the struct rte_mbuf type, and does not use any of the mbuf internals, nor any of the mbuf functions or macros. Therefore the inclusion is unnecessary, and can be replaced by a forward declaration of the mbuf type.
Signed-off-by: Bruce Richardson <bruce.richardson at intel.com> --- app/test/test_distributor.c | 2 ++ app/test/test_distributor_perf.c | 3 +++ lib/librte_distributor/rte_distributor.h | 3 +-- 3 files changed, 6 insertions(+), 2 deletions(-) diff --git a/app/test/test_distributor.c b/app/test/test_distributor.c index 9e8c06d..c8c0942 100644 --- a/app/test/test_distributor.c +++ b/app/test/test_distributor.c @@ -37,6 +37,8 @@ #include <string.h> #include <rte_cycles.h> #include <rte_errno.h> +#include <rte_mempool.h> +#include <rte_mbuf.h> #include <rte_distributor.h> #define ITER_POWER 20 /* log 2 of how many iterations we do when timing. */ diff --git a/app/test/test_distributor_perf.c b/app/test/test_distributor_perf.c index 31431bb..234a37e 100644 --- a/app/test/test_distributor_perf.c +++ b/app/test/test_distributor_perf.c @@ -35,7 +35,10 @@ #include <unistd.h> #include <string.h> +#include <rte_mempool.h> #include <rte_cycles.h> +#include <rte_common.h> +#include <rte_mbuf.h> #include <rte_distributor.h> #define ITER_POWER 20 /* log 2 of how many iterations we do when timing. */ diff --git a/lib/librte_distributor/rte_distributor.h b/lib/librte_distributor/rte_distributor.h index cc1d559..7d36bc8 100644 --- a/lib/librte_distributor/rte_distributor.h +++ b/lib/librte_distributor/rte_distributor.h @@ -46,11 +46,10 @@ extern "C" { #endif -#include <rte_mbuf.h> - #define RTE_DISTRIBUTOR_NAMESIZE 32 /**< Length of name for instance */ struct rte_distributor; +struct rte_mbuf; /** * Function to create a new distributor instance -- 2.1.0