On 2023/2/9 8:04, Stephen Hemminger wrote:
> On Wed, 8 Feb 2023 08:24:47 +0000
> Chengwen Feng <fengcheng...@huawei.com> wrote:
>
>> +RTE_LOG_REGISTER_DEFAULT(rte_memarea_logtype, INFO);
>> +#define RTE_MEMAREA_LOG(level, ...) \
>> + rte_log(RTE_LOG_ ## level, rte_memarea_logtype, RTE_FMT("memarea: " \
>> + RTE_FMT_HEAD(__VA_ARGS__,) "\n", RTE_FMT_TAIL(__VA_ARGS__,)))
>> +
>
> For me it is easier to read as:
> #define RTE_MEMAREA_LOG(level, fmt, args...) \
> rte_log(RTE_LOG_ ## level, rte_memarea_logtype, \
> "memarea: " fmt "\n", ## args)
>
>
> A common convention is to use "%s: " and __func__.
All fix in v14, thanks.
> .
>
