From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: [PATCH] ixgbe: fix build with bypass and debug enabled Date: Mon, 1 Dec 2014 18:40:59 +0100 Message-ID: <1417455659-24854-1-git-send-email-thomas.monjalon@6wind.com> To: dev-VfR2kkLFssw@public.gmane.org Return-path: List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" 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 --- 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 15203a4..832f415 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.1.3