config: add option to enable asserts
authorXueming Li <xuemingl@mellanox.com>
Thu, 24 Aug 2017 08:23:10 +0000 (16:23 +0800)
committerThomas Monjalon <thomas@monjalon.net>
Mon, 9 Oct 2017 21:15:45 +0000 (23:15 +0200)
commitf38530635781adf11134f57498582a50cc32c623
tree930f33d17807be880140834c9172e8593abfe7d4
parent08ef593a707f58eb4e817b8512da7f3b4163aae2
config: add option to enable asserts

Currently, enabling assertion have to set CONFIG_RTE_LOG_LEVEL to
RTE_LOG_DEBUG. CONFIG_RTE_LOG_LEVEL is the default log level of control
path, RTE_LOG_DP_LEVEL is the log level of data path. It's a little bit
hard to understand literally that assertion is decided by control path
LOG_LEVEL, especially assertion used on data path.

On the other hand, DPDK need an assertion enabling switch w/o impacting
log output level, assuming "--log-level" not specified.

Assertion is an important API to balance DPDK high performance and
robustness. To promote assertion usage, it's valuable to unhide
assertion out of COFNIG_RTE_LOG_LEVEL.

In one word, log is log, assertion is assertion, debug is hot pot :)

Rationale of this patch is to introduce an dedicate switch of
assertion: RTE_ENABLE_ASSERT

Signed-off-by: Xueming Li <xuemingl@mellanox.com>
Acked-by: Gaetan Rivet <gaetan.rivet@6wind.com>
config/common_base
lib/librte_eal/common/include/rte_debug.h