On Thu, 25 Jan 2018 17:00:43 +0000 Andrew Rybchenko <arybche...@solarflare.com> wrote:
> diff --git a/drivers/net/sfc/efsys.h b/drivers/net/sfc/efsys.h > index c7a54c3..8dd225e 100644 > --- a/drivers/net/sfc/efsys.h > +++ b/drivers/net/sfc/efsys.h > @@ -26,6 +26,7 @@ > #include <rte_io.h> > > #include "sfc_debug.h" > +#include "sfc_log.h" > > #ifdef __cplusplus > extern "C" { > @@ -721,7 +722,7 @@ typedef uint64_t efsys_stat_t; > #define EFSYS_ERR(_esip, _code, _dword0, _dword1) \ > do { \ > (void)(_esip); \ > - RTE_LOG(ERR, PMD, "FATAL ERROR #%u (0x%08x%08x)\n", \ > + SFC_GENERIC_LOG(ERR, "FATAL ERROR #%u (0x%08x%08x)", \ > (_code), (_dword0), (_dword1)); \ > _NOTE(CONSTANTCONDITION); \ > } while (B_FALSE) Off topic, but why is this header file having C++ wrapper? It is driver private, and driver is always built with C.