[dpdk-dev] [EXT] [PATCH v1 04/17] eventdev: replace library debug flag with global one

Thomas Monjalon thomas at monjalon.net
Sat Apr 18 15:22:51 CEST 2020


18/04/2020 12:05, Jerin Jacob:
> On Sat, Apr 18, 2020 at 3:23 PM Thomas Monjalon <thomas at monjalon.net> wrote:
> >
> > 18/04/2020 11:41, Jerin Jacob Kollanukkaran:
> > > From: Lukasz Wojciechowski <l.wojciechow at partner.samsung.com>
> > > > Use global debug flag RTE_DEBUG instead of RTE_LIBRTE_EVENTDEV_DEBUG.
> > > > The old define is completely removed from source code and config.
> > >
> > >
> > > May I know the rational for this change?
> >
> > You know the rationale, it was discussed several times in Technical Board:
> 
> If I already know the rationale, Why should I ask in ml?

I meant you knew but you probably did not connect previous discussions
with this patch. That's why I explain below, no problem.

> > we are limiting the number of compilation options, and meson config does
> > not propose them.
> >
> > > Other subsystem like ethdev still have CONFIG_RTE_LIBRTE_ETHDEV_DEBUG=n.
> >
> > No, look at the other patches in the series.
> >
> > > If we decide to make RTE_DEBUG then please submit the patch for all subsystem
> > > across the DPDK.
> >
> > Yes, this is what Lukasz did.
> 
> I missed the series. I just thought it is a single email due to email
> filters. My bad.





More information about the dev mailing list