Hi Jerin,

Please see inline.

Thanks,
Anoob

> -----Original Message-----
> From: Jerin Jacob Kollanukkaran
> Sent: Monday, June 10, 2019 3:43 PM
> To: Anoob Joseph <ano...@marvell.com>; Nikhil Rao
> <nikhil....@intel.com>; Erik Gabriel Carrillo <erik.g.carri...@intel.com>;
> Abhinandan Gujjar <abhinandan.guj...@intel.com>; Bruce Richardson
> <bruce.richard...@intel.com>; Pablo de Lara
> <pablo.de.lara.gua...@intel.com>
> Cc: Anoob Joseph <ano...@marvell.com>; Narayana Prasad Raju Athreya
> <pathr...@marvell.com>; dev@dpdk.org; Lukas Bartosik
> <lbarto...@marvell.com>; Pavan Nikhilesh Bhagavatula
> <pbhagavat...@marvell.com>; Hemant Agrawal
> <hemant.agra...@nxp.com>; Nipun Gupta <nipun.gu...@nxp.com>; Harry
> van Haaren <harry.van.haa...@intel.com>; Mattias Rönnblom
> <mattias.ronnb...@ericsson.com>; Liang Ma <liang.j...@intel.com>
> Subject: RE: [PATCH 15/39] eventdev: add routines for logging eventmode
> helper
> 
> > -----Original Message-----
> > From: Anoob Joseph <ano...@marvell.com>
> > Sent: Monday, June 3, 2019 11:02 PM
> > To: Jerin Jacob Kollanukkaran <jer...@marvell.com>; Nikhil Rao
> > <nikhil....@intel.com>; Erik Gabriel Carrillo
> > <erik.g.carri...@intel.com>; Abhinandan Gujjar
> > <abhinandan.guj...@intel.com>; Bruce Richardson
> > <bruce.richard...@intel.com>; Pablo de Lara
> > <pablo.de.lara.gua...@intel.com>
> > Cc: Anoob Joseph <ano...@marvell.com>; Narayana Prasad Raju Athreya
> > <pathr...@marvell.com>; dev@dpdk.org; Lukas Bartosik
> > <lbarto...@marvell.com>; Pavan Nikhilesh Bhagavatula
> > <pbhagavat...@marvell.com>; Hemant Agrawal
> <hemant.agra...@nxp.com>;
> > Nipun Gupta <nipun.gu...@nxp.com>; Harry van Haaren
> > <harry.van.haa...@intel.com>; Mattias Rönnblom
> > <mattias.ronnb...@ericsson.com>; Liang Ma <liang.j...@intel.com>
> > Subject: [PATCH 15/39] eventdev: add routines for logging eventmode
> > helper
> >
> > Adding routines for logging eventmode helper.
> >
> > Signed-off-by: Anoob Joseph <ano...@marvell.com>
> > Signed-off-by: Lukasz Bartosik <lbarto...@marvell.com>
> > ---
> >  config/common_base                                 |  1 +
> >  lib/librte_eal/common/eal_common_log.c             |  1 +
> >  lib/librte_eal/common/include/rte_log.h            |  1 +
> >  .../rte_eventmode_helper_internal.h                | 29
> > ++++++++++++++++++++++
> >  4 files changed, 32 insertions(+)
> >
> > diff --git a/config/common_base b/config/common_base index
> > 8576973..96c2537 100644
> > --- a/config/common_base
> > +++ b/config/common_base
> > @@ -685,6 +685,7 @@ CONFIG_RTE_LIBRTE_PMD_ZLIB=n  #
> > CONFIG_RTE_LIBRTE_EVENTDEV=y
> CONFIG_RTE_LIBRTE_EVENTDEV_DEBUG=n
> > +CONFIG_RTE_LIBRTE_EVENTMODE_HELPER_DEBUG=n
> >  CONFIG_RTE_EVENT_MAX_DEVS=16
> >  CONFIG_RTE_EVENT_MAX_QUEUES_PER_DEV=64
> >  CONFIG_RTE_EVENT_TIMER_ADAPTER_NUM_MAX=32
> > diff --git a/lib/librte_eal/common/eal_common_log.c
> > b/lib/librte_eal/common/eal_common_log.c
> > index c714a4b..bc34dc2 100644
> > --- a/lib/librte_eal/common/eal_common_log.c
> > +++ b/lib/librte_eal/common/eal_common_log.c
> > @@ -324,6 +324,7 @@ static const struct logtype logtype_strings[] = {
> >     {RTE_LOGTYPE_EFD,        "lib.efd"},
> >     {RTE_LOGTYPE_EVENTDEV,   "lib.eventdev"},
> >     {RTE_LOGTYPE_GSO,        "lib.gso"},
> > +   {RTE_LOGTYPE_EVENTMODE,  "lib.eventmode"},
> 
> EAL log type, typically adds for subsystem. Please reuse EVENTDEV_DEBUG
> Or introduce dynamic debugging.

[Anoob] Will make this change in v2. In that case, this patch can be dropped.

Reply via email to