Hi Eugeny,
> -----Original Message-----
> From: dev <dev-boun...@dpdk.org> On Behalf Of
> eugeny.parshu...@linux.intel.com
> Sent: Wednesday, April 1, 2020 8:26
> To: Thomas Monjalon <tho...@monjalon.net>; ferruh.yi...@intel.com;
> john.mcnam...@intel.com
> Cc: dev@dpdk.org; eugeny.parshu...@linux.intel.com;
> roman.p.kha...@intel.com
> Subject: [dpdk-dev] [PATCH v2] ethdev: fix for compiler warning when vtune
> profiling option is enabled
>
> From: Eugeny Parshutin <eugeny.parshu...@linux.intel.com>
>
> Add the previous prototype for the 'profile_hook_rx_burst_cb' function to
> fix the compiler warning when the DPDK is built with
> 'RTE_ETHDEV_PROFILE_WITH_VTUNE' config option enabled:
> /home/dpdk/lib/librte_ethdev/ethdev_profile.c:17:1: warning:
> no previous prototype for profile_hook_rx_burst_cb [-Wmissing-
> prototypes]
>
> Signed-off-by: Eugeny Parshutin <eugeny.parshu...@linux.intel.com>
> ---
> v2: Fix the commit message
> ---
> lib/librte_ethdev/ethdev_profile.h | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/lib/librte_ethdev/ethdev_profile.h
> b/lib/librte_ethdev/ethdev_profile.h
> index 65031e6f3..b8c3c6848 100644
> --- a/lib/librte_ethdev/ethdev_profile.h
> +++ b/lib/librte_ethdev/ethdev_profile.h
> @@ -24,4 +24,14 @@
> int
> __rte_eth_dev_profile_init(uint16_t port_id, struct rte_eth_dev *dev);
>
> +#ifdef RTE_ETHDEV_PROFILE_WITH_VTUNE
> +
> +uint16_t
> +profile_hook_rx_burst_cb(
> + __rte_unused uint16_t port_id, __rte_unused uint16_t queue_id,
> + __rte_unused struct rte_mbuf *pkts[], uint16_t nb_pkts,
> + __rte_unused uint16_t max_pkts, __rte_unused void
> *user_param);
> +
> +#endif /* RTE_ETHDEV_PROFILE_WITH_VTUNE */
> +
> #endif
> --
> 2.17.1
Could you please fix the check-git-log.sh warnings:
Headline too long:
ethdev: fix for compiler warning when vtune profiling option is enabled
Missing 'Fixes' tag:
ethdev: fix for compiler warning when vtune profiling option is enabled
Regards,
Alex
