From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wi0-f171.google.com (mail-wi0-f171.google.com [209.85.212.171]) by dpdk.org (Postfix) with ESMTP id 6A0CF682E for ; Tue, 26 Aug 2014 16:05:29 +0200 (CEST) Received: by mail-wi0-f171.google.com with SMTP id hi2so4263700wib.10 for ; Tue, 26 Aug 2014 07:09:29 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:subject:date:message-id; bh=Q4al9YE9fhk07RKWt377mkFFOw+f1mOkze6FIlX4lgk=; b=D18C9IVVYZAVge6/eyr+WWkLdEjK4k6Dx7rMQ3q5GsHRdMuSW1at2Ay704xN8cnWWW SXjI/b2aj8DUG6WKfDb0WpMRCjPnLatJU9sUrMoLPHPON/i0Dr7UC8EwhbOiDS0uxroh qSvUXqIxwnqD7Er/xqiIwSMWzddySaz5dcV53N+0KImsBQ9QM7IN3s+4H152T5pBG6oj +a5vslbGjYeK69yZh6CymX/mNrwxRFm/XLsYbW/PnBtzb6sIK3ygoeS8JxEvEY+q4EbB r5dEHk56j5LIEm7qZ6nrYOEIrA58aOkvHZFanr0h8QsvNRqwbvDvB/WdW/1c76b/LX6l gq6g== X-Gm-Message-State: ALoCoQmC+mHexa29QDB5QT+e4kNksylcW155Axyw9gYPZ+MnQnFPEAASxd0VKb4vKEse3vW1cnr/ X-Received: by 10.180.21.101 with SMTP id u5mr21845766wie.68.1409062169424; Tue, 26 Aug 2014 07:09:29 -0700 (PDT) Received: from alcyon.dev.6wind.com (guy78-3-82-239-227-177.fbx.proxad.net. [82.239.227.177]) by mx.google.com with ESMTPSA id cy9sm12543569wib.18.2014.08.26.07.09.27 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 26 Aug 2014 07:09:28 -0700 (PDT) From: David Marchand To: dev@dpdk.org Date: Tue, 26 Aug 2014 16:09:10 +0200 Message-Id: <1409062162-19575-1-git-send-email-david.marchand@6wind.com> X-Mailer: git-send-email 1.7.10.4 Subject: [dpdk-dev] [PATCH 00/11] cleanup logs in main PMDs X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 26 Aug 2014 14:05:29 -0000 Here is a patchset that reworks the log macro in e1000, ixgbe and i40e PMDs. The idea behind this is to make it easier to debug some init failures and to be sure of the datapath selected in these PMDs (rx / tx handlers selection). The PMDs changes involve adding more debug messages in the default build. A new eal option has been added to set the default log level, so that you can render the eal a little less noisy. I did not change the default log level for now, as some eal log messages are marked as DEBUG while being interesting (from my point of view). I suppose we can change the default log level later once the eal has been cleaned up. -- David Marchand David Marchand (11): ixgbe: clean log messages ixgbe: always log init messages ixgbe: add a message when forcing scatter mode ixgbe: add log messages when rx bulk mode is not usable i40e: clean log messages i40e: always log init messages i40e: add log messages when rx bulk mode is not usable e1000: clean log messages e1000: always log init messages e1000: add a message when forcing scatter mode eal: set log level from command line lib/librte_eal/bsdapp/eal/eal.c | 43 +++++++++ .../bsdapp/eal/include/eal_internal_cfg.h | 1 + lib/librte_eal/linuxapp/eal/eal.c | 43 +++++++++ .../linuxapp/eal/include/eal_internal_cfg.h | 1 + lib/librte_pmd_e1000/e1000_logs.h | 17 ++-- lib/librte_pmd_e1000/em_ethdev.c | 47 +++++----- lib/librte_pmd_e1000/em_rxtx.c | 39 ++++---- lib/librte_pmd_e1000/igb_ethdev.c | 75 +++++++-------- lib/librte_pmd_e1000/igb_rxtx.c | 23 ++++- lib/librte_pmd_i40e/i40e/i40e_osdep.h | 2 +- lib/librte_pmd_i40e/i40e_ethdev.c | 43 ++++----- lib/librte_pmd_i40e/i40e_ethdev_vf.c | 4 +- lib/librte_pmd_i40e/i40e_logs.h | 17 ++-- lib/librte_pmd_i40e/i40e_pf.c | 4 +- lib/librte_pmd_i40e/i40e_rxtx.c | 35 +++++-- lib/librte_pmd_ixgbe/ixgbe_ethdev.c | 99 ++++++++++---------- lib/librte_pmd_ixgbe/ixgbe_fdir.c | 12 +-- lib/librte_pmd_ixgbe/ixgbe_logs.h | 17 ++-- lib/librte_pmd_ixgbe/ixgbe_rxtx.c | 51 +++++++--- 19 files changed, 360 insertions(+), 213 deletions(-) -- 1.7.10.4