On 3/13/20 3:51 PM, Ferruh Yigit wrote: > This is a helper function in case components would like to do more work > than just logging a message based on log level, like for example > collecting some stats if the log type is DEBUG etc.. > > A few existing relevant usage converted to this new API. > > Signed-off-by: Ferruh Yigit <[email protected]> > Acked-by: Hyong Youb Kim <[email protected]> > Acked-by: Hemant Agrawal <[email protected]> > --- > Cc: David Marchand <[email protected]> > Cc: Thomas Monjalon <[email protected]> > Cc: Stephen Hemminger <[email protected]> > > v2: > * Convert API return type to 'bool'. Removed custom definitions from > 'ionic' PMD for this. > > v3: > * Update function comment > * Use new function in `rte_vlog()` > --- > drivers/bus/fslmc/fslmc_bus.c | 7 +------ > drivers/common/qat/qat_logs.c | 7 ++----- > drivers/net/enic/enic_fm_flow.c | 2 +- > drivers/net/mlx5/mlx5_mr.c | 2 +- > lib/librte_eal/common/eal_common_log.c | 22 +++++++++++++++++--- > lib/librte_eal/common/include/rte_log.h | 14 +++++++++++++ > lib/librte_eal/rte_eal_version.map | 3 +++ > lib/librte_flow_classify/rte_flow_classify.c | 7 ++----- > 8 files changed, 43 insertions(+), 21 deletions(-)
Ferruh I like this and would like use it for the next round of my patches. Is there any timeline for that? With regards Andrzej Ostruszka Reviewed-by: Andrzej Ostruszka <[email protected]>
