2015-04-09 10:32, Adrien Mazarguil:
> On Tue, Apr 07, 2015 at 07:40:19PM +0200, Thomas Monjalon wrote:
> > The function name is printed in each enic_ethdev function.
> > Disable it by default with a new build option.
> >
> > Signed-off-by: Thomas Monjalon <thomas.monjalon at 6wind.com>
[...]
> > --- a/lib/librte_pmd_enic/enic_ethdev.c
> > +++ b/lib/librte_pmd_enic/enic_ethdev.c
> > @@ -48,8 +48,12 @@
> > #include "vnic_enet.h"
> > #include "enic.h"
> >
> > +#ifdef RTE_LIBRTE_ENIC_DEBUG
> > #define ENICPMD_FUNC_TRACE() \
> > RTE_LOG(DEBUG, PMD, "ENICPMD trace: %s\n", __func__)
> > +#else
> > +#define ENICPMD_FUNC_TRACE() do {} while (0)
>
> How about defining it as (void)0 instead of an empty do/while block?
>
> Doing so will prevent warnings if this macro happens to be used in an
> expression. RTE_LOG() supports it.
Applied with Adrien's suggestion.
