From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from proxy.6wind.com (host.76.145.23.62.rev.coltfrance.com [62.23.145.76]) by dpdk.org (Postfix) with ESMTP id 712842E8A for ; Tue, 4 Apr 2017 18:44:47 +0200 (CEST) Received: from glumotte.dev.6wind.com (unknown [10.16.0.195]) by proxy.6wind.com (Postfix) with ESMTP id C5AD624A1A; Tue, 4 Apr 2017 18:44:41 +0200 (CEST) From: Olivier Matz To: dev@dpdk.org Cc: david.marchand@6wind.com, bruce.richardson@intel.com, thomas.monjalon@6wind.com, keith.wiles@intel.com, stephen@networkplumber.org Date: Tue, 4 Apr 2017 18:40:32 +0200 Message-Id: <20170404164040.24132-1-olivier.matz@6wind.com> X-Mailer: git-send-email 2.11.0 In-Reply-To: <20170329155323.4760-1-olivier.matz@6wind.com> References: <20170329155323.4760-1-olivier.matz@6wind.com> Subject: [dpdk-dev] [PATCH v3 0/8] eal: dynamic logs X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 04 Apr 2017 16:44:47 -0000 The objective of this patchset is to introduce a framework to support dynamic log types in EAL. It also provides one example of use (in i40e). Features: - log types are identified by a string - at registration, a uniq identifier is associated to a log type - each log type can have its level changed dynamically - extend command line parameters to set the log level of a specific type, or logs matching a regular expression - keep compat with other legacy types (eal, malloc, ring, user*, etc... keep their hardcoded log type value) Next step is to adapt drivers, libs and apps to use this new API. At the end, we can expect that all non-dataplane logs are moved to be dynamic, so we can enable/disable them at runtime, without recompiling. Many debug options can probably be removed from configuration: $ git grep DEBUG config/common_base | wc -l 89 v2 -> v3: - fix compilation of intermediate patches - replace references to 17.02 by 17.05 (doc and .map) v1 -> v2: - fix issues reported by Stephen - add entry in release note and deprecation notice - rebase on top of master RFC -> v1: - rebase on top of current master - fix eal help alignment - remove unused i40e compilation options Olivier Matz (8): eal: support dynamic log types eal: dump registered log types eal: change several log levels matching a regexp eal: change specific log levels at startup eal: deprecate log functions app/test: new command to dump log types app/testpmd: new command to dump log types net/i40e: use dynamic log type for control logs app/test-pmd/cmdline.c | 5 +- config/common_base | 2 - doc/guides/contributing/coding_style.rst | 30 ++-- doc/guides/rel_notes/deprecation.rst | 8 + doc/guides/rel_notes/release_17_05.rst | 5 + drivers/net/i40e/i40e_ethdev.c | 18 +- drivers/net/i40e/i40e_fdir.c | 4 - drivers/net/i40e/i40e_logs.h | 17 +- examples/quota_watermark/qw/main.c | 2 +- examples/quota_watermark/qwctl/qwctl.c | 2 +- lib/librte_eal/bsdapp/eal/eal.c | 4 +- lib/librte_eal/bsdapp/eal/rte_eal_version.map | 12 ++ lib/librte_eal/common/eal_common_log.c | 216 +++++++++++++++++++++++- lib/librte_eal/common/eal_common_options.c | 49 ++++-- lib/librte_eal/common/include/rte_log.h | 76 ++++++++- lib/librte_eal/linuxapp/eal/eal.c | 4 +- lib/librte_eal/linuxapp/eal/rte_eal_version.map | 12 ++ test/test/commands.c | 4 +- test/test/test_logs.c | 12 +- 19 files changed, 417 insertions(+), 65 deletions(-) -- 2.11.0