From: Eugeny Parshutin <eugeny.parshutin@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.parshutin@linux.intel.com> --- 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
From: Eugeny Parshutin <eugeny.parshutin@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.parshutin@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
Hi Eugeny,
> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of
> eugeny.parshutin@linux.intel.com
> Sent: Wednesday, April 1, 2020 8:26
> To: Thomas Monjalon <thomas@monjalon.net>; ferruh.yigit@intel.com;
> john.mcnamara@intel.com
> Cc: dev@dpdk.org; eugeny.parshutin@linux.intel.com;
> roman.p.khatko@intel.com
> Subject: [dpdk-dev] [PATCH v2] ethdev: fix for compiler warning when vtune
> profiling option is enabled
>
> From: Eugeny Parshutin <eugeny.parshutin@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.parshutin@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