On Mon, 13 Apr 2020 14:30:27 +0800
alvinx.zh...@intel.com wrote:

> +     if (dev->data->mac_addrs == NULL) {
> +             PMD_INIT_LOG(ERR, "Failed to allocate %d bytes needed to "
> +                             "store MAC addresses", RTE_ETHER_ADDR_LEN);

Stylistic note.
Please don't break messages across multiple lines, it makes it harder to
read, and also harder for users to search for error messages.

If you are concerned about checkpatch, checkpatch allows long lines for
messages.

                PMD_INIT_LOG(ERR,
                             "Failed allocation for MAC address");

Reply via email to