From c1aa5e44e6fb45211e69c20f2be554ab96aab276 Mon Sep 17 00:00:00 2001 From: Thomas Monjalon Date: Mon, 1 Dec 2014 18:11:02 +0100 Subject: [PATCH] ixgbe: fix build with bypass and debug enabled 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 Acked-by: David Marchand --- lib/librte_pmd_ixgbe/ixgbe_bypass.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/librte_pmd_ixgbe/ixgbe_bypass.c b/lib/librte_pmd_ixgbe/ixgbe_bypass.c index 15203a47ca..832f4156da 100644 --- a/lib/librte_pmd_ixgbe/ixgbe_bypass.c +++ b/lib/librte_pmd_ixgbe/ixgbe_bypass.c @@ -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; \ } \ -- 2.20.1