ixgbe: fix build with bypass and debug enabled
authorThomas Monjalon <thomas.monjalon@6wind.com>
Mon, 1 Dec 2014 17:11:02 +0000 (18:11 +0100)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Tue, 2 Dec 2014 11:28:11 +0000 (12:28 +0100)
Since commit aae1047905621 ("use the right debug macro"),
DEBUGOUT was replaced by PMD_DRV_LOG which requires at least
2 arguments. But the level argument was missing.

Commit 7a10de5e27 fixed the logs but not the macros FUNC_PTR_OR_*
which are not preprocessed if RTE_LIBRTE_IXGBE_DEBUG_DRIVER is disabled.

Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
Acked-by: David Marchand <david.marchand@6wind.com>
lib/librte_pmd_ixgbe/ixgbe_bypass.c

index 15203a4..832f415 100644 (file)
@@ -42,7 +42,7 @@
 /* Macros to check for invlaid function pointers. */
 #define        FUNC_PTR_OR_ERR_RET(func, retval) do {              \
        if ((func) == NULL) {                               \
-               PMD_DRV_LOG("%s:%d function not supported", \
+               PMD_DRV_LOG(ERR, "%s:%d function not supported", \
                            __func__, __LINE__);            \
                return retval;                            \
        }                                                   \
@@ -50,7 +50,7 @@
 
 #define        FUNC_PTR_OR_RET(func) do {                          \
        if ((func) == NULL) {                               \
-               PMD_DRV_LOG("%s:%d function not supported", \
+               PMD_DRV_LOG(ERR, "%s:%d function not supported", \
                            __func__, __LINE__);            \
                return;                                     \
        }                                                   \