[PATCH v3 03/16] efd: replace RTE_LOGTYPE_EFD with local type

[fengchengwen]

On 2023/2/10 9:07, Stephen Hemminger wrote:
> Replace all uses of global logtype with a local log type.
> Do not break message formats across source lines.
> 
> Signed-off-by: Stephen Hemminger <stephen at networkplumber.org>
> ---

...

>  };
>  EAL_REGISTER_TAILQ(rte_efd_tailq);
>  
> +RTE_LOG_REGISTER_DEFAULT(efd_logtype, INFO);
> +
> +#define EFD_LOG(level, fmt, args...) \
> +	rte_log(RTE_LOG_ ## level, efd_logtype, "%s(): " fmt "\n", __func__, ##args)
> +
>  /** Internal permutation array used to shuffle bins into pseudorandom groups */
>  const uint32_t efd_bin_to_group[EFD_CHUNK_NUM_BIN_TO_GROUP_SETS][EFD_CHUNK_NUM_BINS] = {
>  	{
> @@ -509,13 +514,12 @@ rte_efd_create(const char *name, uint32_t max_num_rules, uint32_t key_len,
>  	efd_list = RTE_TAILQ_CAST(rte_efd_tailq.head, rte_efd_list);
>  
>  	if (online_cpu_socket_bitmask == 0) {
> -		RTE_LOG(ERR, EFD, "At least one CPU socket must be enabled "
> -				"in the bitmask\n");
> +		EFD_LOG(ERR, "At least one CPU socket must be enabled in the bitmask");

Hi Stephen,

Original output will be: EFD: At least one CPU ...
and new output will be: rte_efd_create(): At least one CPU ...

The new output don't have EFD (module or lib name), logs of the EFD module cannot be easily
filtered, althought new have function name, but will may not contains module name.

Suggest add module name in EFD_LOG:
#define EFD_LOG(level, fmt, args...) \
	rte_log(RTE_LOG_ ## level, efd_logtype, "EFD: %s(): " fmt "\n", __func__, ##args)

Thanks.

>  		return NULL;
>  	}
>  

...