diff mbox series

[v2,02/22] event/dlb2: add dynamic logging

Message ID	1602958879-8558-3-git-send-email-timothy.mcdaniel@intel.com (mailing list archive)
State	Superseded, archived
Delegated to:	Jerin Jacob
Headers	IronPort-SDR: q3uCppFe+/1Aw5YRbQAwaWAbk8WdBLTRn4uqdmwx9O43riyi8NsU1SSl3z3rj35PZzw/XoAJig LepIjGBZZwLg== IronPort-SDR: a0D/AKnXHrbP2Y0iCcLkPXsqSCgf1UqyoDylIgwba8RTMKDif8NMoiw4oQTS45ZRkfonCygOLW miiSodxIJMXw== From: Timothy McDaniel <timothy.mcdaniel@intel.com> To: Cc: dev@dpdk.org, erik.g.carrillo@intel.com, gage.eads@intel.com, harry.van.haaren@intel.com, jerinj@marvell.com Date: Sat, 17 Oct 2020 13:20:59 -0500 Message-Id: <1602958879-8558-3-git-send-email-timothy.mcdaniel@intel.com> In-Reply-To: <1602958879-8558-1-git-send-email-timothy.mcdaniel@intel.com> References: <1599855987-25976-2-git-send-email-timothy.mcdaniel@intel.com> <1602958879-8558-1-git-send-email-timothy.mcdaniel@intel.com> Subject: [dpdk-dev] [PATCH v2 02/22] event/dlb2: add dynamic logging Precedence: list Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org>
Series	Add DLB2 PMD \| [v2,00/22] Add DLB2 PMD [v2,01/22] event/dlb2: add documentation and meson build infrastructure [v2,02/22] event/dlb2: add dynamic logging [v2,03/22] event/dlb2: add private data structures and constants [v2,04/22] event/dlb2: add definitions shared with LKM or shared code [v2,05/22] event/dlb2: add inline functions [v2,06/22] event/dlb2: add probe [v2,07/22] event/dlb2: add xstats [v2,08/22] event/dlb2: add infos get and configure [v2,09/22] event/dlb2: add queue and port default conf [v2,10/22] event/dlb2: add queue setup [v2,11/22] event/dlb2: add port setup [v2,12/22] event/dlb2: add port link [v2,13/22] event/dlb2: add port unlink and port unlinks in progress [v2,14/22] event/dlb2: add eventdev start [v2,15/22] event/dlb2: add enqueue and its burst variants [v2,16/22] event/dlb2: add dequeue and its burst variants [v2,17/22] event/dlb2: add eventdev stop and close [v2,18/22] event/dlb2: add PMD's token pop public interface [v2,19/22] event/dlb2: add PMD self-tests [v2,20/22] event/dlb2: add queue and port release [v2,21/22] event/dlb2: add timeout ticks entry point [v2,22/22] doc: add new DLB2 eventdev driver to relnotes

Checks

Context	Check	Description
ci/checkpatch	success	coding style OK

Commit Message

Timothy McDaniel Oct. 17, 2020, 6:20 p.m. UTC

  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@intel.com>
Reviewed-by: Gage Eads <gage.eads@intel.com>
---
 drivers/event/dlb2/dlb2_log.h | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)
 create mode 100644 drivers/event/dlb2/dlb2_log.h

Comments

Jerin Jacob Oct. 18, 2020, 8:57 a.m. UTC | #1

On Sat, Oct 17, 2020 at 11:50 PM Timothy McDaniel
<timothy.mcdaniel@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@intel.com>
> Reviewed-by: Gage Eads <gage.eads@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
>

Timothy McDaniel Oct. 20, 2020, 2:08 p.m. UTC | #2

> -----Original Message-----
> From: Jerin Jacob <jerinjacobk@gmail.com>
> Sent: Sunday, October 18, 2020 3:57 AM
> To: McDaniel, Timothy <timothy.mcdaniel@intel.com>
> Cc: dpdk-dev <dev@dpdk.org>; Carrillo, Erik G <Erik.G.Carrillo@intel.com>;
> Eads, Gage <gage.eads@intel.com>; Van Haaren, Harry
> <harry.van.haaren@intel.com>; Jerin Jacob <jerinj@marvell.com>
> Subject: Re: [dpdk-dev] [PATCH v2 02/22] event/dlb2: add dynamic logging
> 
> On Sat, Oct 17, 2020 at 11:50 PM Timothy McDaniel
> <timothy.mcdaniel@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@intel.com>
> > Reviewed-by: Gage Eads <gage.eads@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.
> 

will do

diff mbox series

Patch

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;
+
+/* 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_ */