[dpdk-dev] [PATCH v2 02/22] event/dlb2: add dynamic logging

[Jerin Jacob]

On Sat, Oct 17, 2020 at 11:50 PM Timothy McDaniel
<timothy.mcdaniel at intel.com> wrote:
>
> This commit adds base support for dynamic logging.
> The default log level is NOTICE. Dynamic logging
> is used exclusively throughout this patchset.
>
> Signed-off-by: Timothy McDaniel <timothy.mcdaniel at intel.com>
> Reviewed-by: Gage Eads <gage.eads at intel.com>
> ---
>  drivers/event/dlb2/dlb2_log.h | 25 +++++++++++++++++++++++++
>  1 file changed, 25 insertions(+)
>  create mode 100644 drivers/event/dlb2/dlb2_log.h
>
> diff --git a/drivers/event/dlb2/dlb2_log.h b/drivers/event/dlb2/dlb2_log.h
> new file mode 100644
> index 0000000..dc1481e
> --- /dev/null
> +++ b/drivers/event/dlb2/dlb2_log.h
> @@ -0,0 +1,25 @@
> +/* SPDX-License-Identifier: BSD-3-Clause
> + * Copyright(c) 2016-2020 Intel Corporation
> + */
> +
> +#ifndef _DLB2_EVDEV_LOG_H_
> +#define _DLB2_EVDEV_LOG_H_
> +
> +extern int eventdev_dlb2_log_level;

Move rte_log_register() calling part to this patch. Without
registration this patch
is incomplete.


> +/* Dynamic logging */
> +#define DLB2_LOG_IMPL(level, fmt, args...) \
> +       rte_log(RTE_LOG_ ## level, eventdev_dlb2_log_level, "%s" fmt "\n", \
> +               __func__, ##args)
> +
> +#define DLB2_LOG_INFO(fmt, args...) \
> +       DLB2_LOG_IMPL(INFO, fmt, ## args)
> +
> +#define DLB2_LOG_ERR(fmt, args...) \
> +       DLB2_LOG_IMPL(ERR, fmt, ## args)
> +
> +/* remove debug logs at compile time unless actually debugging */
> +#define DLB2_LOG_DBG(fmt, args...) \
> +       RTE_LOG_DP(DEBUG, PMD, fmt, ## args)
> +
> +#endif /* _DLB2_EVDEV_LOG_H_ */
> --
> 2.6.4
>