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 0E4162BB0 for ; Wed, 29 Mar 2017 17:53:11 +0200 (CEST) Received: from glumotte.dev.6wind.com (unknown [10.16.0.195]) by proxy.6wind.com (Postfix) with ESMTP id 7259B23F8E; Wed, 29 Mar 2017 17:53:05 +0200 (CEST) From: Olivier Matz To: dev@dpdk.org Cc: thomas.monjalon@6wind.com, david.marchand@6wind.com, bruce.richardson@intel.com, keith.wiles@intel.com, stephen@networkplumber.org Date: Wed, 29 Mar 2017 17:53:15 +0200 Message-Id: <20170329155323.4760-1-olivier.matz@6wind.com> X-Mailer: git-send-email 2.11.0 In-Reply-To: <1489765882-30497-1-git-send-email-olivier.matz@6wind.com> References: <1489765882-30497-1-git-send-email-olivier.matz@6wind.com> Subject: [dpdk-dev] [PATCH v2 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: Wed, 29 Mar 2017 15:53:11 -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 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_02.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 | 6 + 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 | 6 + test/test/commands.c | 4 +- test/test/test_logs.c | 12 +- 19 files changed, 405 insertions(+), 65 deletions(-) -- 2.11.0