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 <ferruh.yi...@intel.com> > Acked-by: Hyong Youb Kim <hyon...@cisco.com> > Acked-by: Hemant Agrawal <hemant.agra...@nxp.com> > --- > Cc: David Marchand <david.march...@redhat.com> > Cc: Thomas Monjalon <tho...@monjalon.net> > Cc: Stephen Hemminger <step...@networkplumber.org> > > 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 <aostrus...@marvell.com>
