List comments

GET /api/patches/282/comments/
HTTP 200 OK
Allow: GET, HEAD, OPTIONS
Content-Type: application/json
Vary: Accept

[
    {
        "id": 631,
        "web_url": "http://patches.dpdk.org/comment/631/",
        "msgid": "<CADNuJVpHwbR2DsRi6_CzcQvZFo+kdeNqyx5WiPe488WsLMRfRw@mail.gmail.com>",
        "date": "2014-09-02T18:20:12",
        "subject": "Re: [dpdk-dev] [PATCH v2 09/17] i40e: clean log messages",
        "submitter": {
            "id": 61,
            "url": "http://patches.dpdk.org/api/people/61/",
            "name": "Jay Rolette",
            "email": "rolette@infiniteio.com"
        },
        "content": "On Mon, Sep 1, 2014 at 5:24 AM, David Marchand <david.marchand@6wind.com>\nwrote:\n\n> Clean log messages:\n> - remove leading \\n in some messages,\n> - remove trailing \\n in some messages,\n> - split multi lines messages,\n> - replace some PMD_INIT_LOG(DEBUG, \"some_func\") with PMD_INIT_FUNC_TRACE().\n>\n> Signed-off-by: David Marchand <david.marchand@6wind.com>\n> ---\n>  lib/librte_pmd_i40e/i40e_ethdev.c    |  418\n> +++++++++++++++++-----------------\n>  lib/librte_pmd_i40e/i40e_ethdev_vf.c |  166 +++++++-------\n>  lib/librte_pmd_i40e/i40e_pf.c        |   75 +++---\n>  lib/librte_pmd_i40e/i40e_rxtx.c      |  118 +++++-----\n>  4 files changed, 385 insertions(+), 392 deletions(-)\n>\n> diff --git a/lib/librte_pmd_i40e/i40e_ethdev.c\n> b/lib/librte_pmd_i40e/i40e_ethdev.c\n> index 352beb1..1a5b55d 100644\n> --- a/lib/librte_pmd_i40e/i40e_ethdev.c\n> +++ b/lib/librte_pmd_i40e/i40e_ethdev.c\n> @@ -371,7 +371,7 @@ eth_i40e_dev_init(__rte_unused struct eth_driver\n> *eth_drv,\n>         hw->hw_addr = (uint8_t *)(pci_dev->mem_resource[0].addr);\n>         if (!hw->hw_addr) {\n>                 PMD_INIT_LOG(ERR, \"Hardware is not available, \"\n> -                                       \"as address is NULL\\n\");\n> +                            \"as address is NULL\");\n>                 return -ENODEV;\n>         }\n>\n> @@ -395,7 +395,8 @@ eth_i40e_dev_init(__rte_unused struct eth_driver\n> *eth_drv,\n>         /* Initialize the shared code (base driver) */\n>         ret = i40e_init_shared_code(hw);\n>         if (ret) {\n> -               PMD_INIT_LOG(ERR, \"Failed to init shared code (base\n> driver): %d\", ret);\n> +               PMD_INIT_LOG(ERR, \"Failed to init shared code (base\n> driver):\"\n> +                            \"%d\", ret);\n>                 return ret;\n>         }\n>\n> @@ -406,8 +407,7 @@ eth_i40e_dev_init(__rte_unused struct eth_driver\n> *eth_drv,\n>                 PMD_INIT_LOG(ERR, \"Failed to init adminq: %d\", ret);\n>                 return -EIO;\n>         }\n> -       PMD_INIT_LOG(INFO, \"FW %d.%d API %d.%d NVM \"\n> -                       \"%02d.%02d.%02d eetrack %04x\\n\",\n> +       PMD_INIT_LOG(INFO, \"FW %d.%d API %d.%d NVM %02d.%02d.%02d eetrack\n> %04x\",\n>                         hw->aq.fw_maj_ver, hw->aq.fw_min_ver,\n>                         hw->aq.api_maj_ver, hw->aq.api_min_ver,\n>                         ((hw->nvm.version >> 12) & 0xf),\n> @@ -417,7 +417,7 @@ eth_i40e_dev_init(__rte_unused struct eth_driver\n> *eth_drv,\n>         /* Disable LLDP */\n>         ret = i40e_aq_stop_lldp(hw, true, NULL);\n>         if (ret != I40E_SUCCESS) /* Its failure can be ignored */\n> -               PMD_INIT_LOG(INFO, \"Failed to stop lldp\\n\");\n> +               PMD_INIT_LOG(INFO, \"Failed to stop lldp\");\n>\n>         /* Clear PXE mode */\n>         i40e_clear_pxe_mode(hw);\n> @@ -439,13 +439,13 @@ eth_i40e_dev_init(__rte_unused struct eth_driver\n> *eth_drv,\n>         /* Initialize the queue management */\n>         ret = i40e_res_pool_init(&pf->qp_pool, 0, hw->func_caps.num_tx_qp);\n>         if (ret < 0) {\n> -               PMD_INIT_LOG(ERR, \"Failed to init queue pool\\n\");\n> +               PMD_INIT_LOG(ERR, \"Failed to init queue pool\");\n>                 goto err_qp_pool_init;\n>         }\n>         ret = i40e_res_pool_init(&pf->msix_pool, 1,\n>                                 hw->func_caps.num_msix_vectors - 1);\n>         if (ret < 0) {\n> -               PMD_INIT_LOG(ERR, \"Failed to init MSIX pool\\n\");\n> +               PMD_INIT_LOG(ERR, \"Failed to init MSIX pool\");\n>                 goto err_msix_pool_init;\n>         }\n>\n> @@ -499,8 +499,8 @@ eth_i40e_dev_init(__rte_unused struct eth_driver\n> *eth_drv,\n>         /* Should be after VSI initialized */\n>         dev->data->mac_addrs = rte_zmalloc(\"i40e\", len, 0);\n>         if (!dev->data->mac_addrs) {\n> -               PMD_INIT_LOG(ERR, \"Failed to allocated memory \"\n> -                                       \"for storing mac address\");\n> +               PMD_INIT_LOG(ERR, \"Failed to allocated memory for storing \"\n> +                            \"mac address\");\n>                 goto err_get_mac_addr;\n>         }\n>         ether_addr_copy((struct ether_addr *)hw->mac.perm_addr,\n> @@ -723,9 +723,9 @@ i40e_phy_conf_link(struct i40e_hw *hw, uint8_t\n> abilities, uint8_t force_speed)\n>         phy_conf.eeer = phy_ab.eeer_val;\n>         phy_conf.low_power_ctrl = phy_ab.d3_lpan;\n>\n> -       PMD_DRV_LOG(DEBUG, \"\\n\\tCurrent: abilities %x, link_speed %x\\n\"\n> -                   \"\\tConfig:  abilities %x, link_speed %x\",\n> -                   phy_ab.abilities, phy_ab.link_speed,\n> +       PMD_DRV_LOG(DEBUG, \"\\tCurrent: abilities %x, link_speed %x\",\n> +                   phy_ab.abilities, phy_ab.link_speed);\n> +       PMD_DRV_LOG(DEBUG, \"\\tConfig:  abilities %x, link_speed %x\",\n>                     phy_conf.abilities, phy_conf.link_speed);\n>\n>         status = i40e_aq_set_phy_config(hw, &phy_conf, NULL);\n> @@ -763,7 +763,7 @@ i40e_dev_start(struct rte_eth_dev *dev)\n>\n>         if ((dev->data->dev_conf.link_duplex != ETH_LINK_AUTONEG_DUPLEX) &&\n>                 (dev->data->dev_conf.link_duplex != ETH_LINK_FULL_DUPLEX))\n> {\n> -               PMD_INIT_LOG(ERR, \"Invalid link_duplex (%hu) for port\n> %hhu\\n\",\n> +               PMD_INIT_LOG(ERR, \"Invalid link_duplex (%hu) for port\n> %hhu\",\n>                                 dev->data->dev_conf.link_duplex,\n>                                 dev->data->port_id);\n>                 return -EINVAL;\n> @@ -772,7 +772,7 @@ i40e_dev_start(struct rte_eth_dev *dev)\n>         /* Initialize VSI */\n>         ret = i40e_vsi_init(vsi);\n>         if (ret != I40E_SUCCESS) {\n> -               PMD_DRV_LOG(ERR, \"Failed to init VSI\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to init VSI\");\n>                 goto err_up;\n>         }\n>\n> @@ -783,7 +783,7 @@ i40e_dev_start(struct rte_eth_dev *dev)\n>         /* Enable all queues which have been configured */\n>         ret = i40e_vsi_switch_queues(vsi, TRUE);\n>         if (ret != I40E_SUCCESS) {\n> -               PMD_DRV_LOG(ERR, \"Failed to enable VSI\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to enable VSI\");\n>                 goto err_up;\n>         }\n>\n> @@ -791,13 +791,13 @@ i40e_dev_start(struct rte_eth_dev *dev)\n>         if ((vsi->type == I40E_VSI_MAIN) || (vsi->type == I40E_VSI_VMDQ2))\n> {\n>                 ret = i40e_aq_set_vsi_broadcast(hw, vsi->seid, true, NULL);\n>                 if (ret != I40E_SUCCESS)\n> -                       PMD_DRV_LOG(INFO, \"fail to set vsi broadcast\\n\");\n> +                       PMD_DRV_LOG(INFO, \"fail to set vsi broadcast\");\n>         }\n>\n>         /* Apply link configure */\n>         ret = i40e_apply_link_speed(dev);\n>         if (I40E_SUCCESS != ret) {\n> -               PMD_DRV_LOG(ERR, \"Fail to apply link setting\\n\");\n> +               PMD_DRV_LOG(ERR, \"Fail to apply link setting\");\n>                 goto err_up;\n>         }\n>\n> @@ -872,12 +872,12 @@ i40e_dev_promiscuous_enable(struct rte_eth_dev *dev)\n>         status = i40e_aq_set_vsi_unicast_promiscuous(hw, vsi->seid,\n>                                                         true, NULL);\n>         if (status != I40E_SUCCESS)\n> -               PMD_DRV_LOG(ERR, \"Failed to enable unicast promiscuous\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to enable unicast promiscuous\");\n>\n>         status = i40e_aq_set_vsi_multicast_promiscuous(hw, vsi->seid,\n>                                                         TRUE, NULL);\n>         if (status != I40E_SUCCESS)\n> -               PMD_DRV_LOG(ERR, \"Failed to enable multicast\n> promiscuous\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to enable multicast promiscuous\");\n>\n>  }\n>\n> @@ -892,12 +892,12 @@ i40e_dev_promiscuous_disable(struct rte_eth_dev *dev)\n>         status = i40e_aq_set_vsi_unicast_promiscuous(hw, vsi->seid,\n>                                                         false, NULL);\n>         if (status != I40E_SUCCESS)\n> -               PMD_DRV_LOG(ERR, \"Failed to disable unicast\n> promiscuous\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to disable unicast promiscuous\");\n>\n>         status = i40e_aq_set_vsi_multicast_promiscuous(hw, vsi->seid,\n>                                                         false, NULL);\n>         if (status != I40E_SUCCESS)\n> -               PMD_DRV_LOG(ERR, \"Failed to disable multicast\n> promiscuous\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to disable multicast\n> promiscuous\");\n>  }\n>\n>  static void\n> @@ -910,7 +910,7 @@ i40e_dev_allmulticast_enable(struct rte_eth_dev *dev)\n>\n>         ret = i40e_aq_set_vsi_multicast_promiscuous(hw, vsi->seid, TRUE,\n> NULL);\n>         if (ret != I40E_SUCCESS)\n> -               PMD_DRV_LOG(ERR, \"Failed to enable multicast\n> promiscuous\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to enable multicast promiscuous\");\n>  }\n>\n>  static void\n> @@ -927,7 +927,7 @@ i40e_dev_allmulticast_disable(struct rte_eth_dev *dev)\n>         ret = i40e_aq_set_vsi_multicast_promiscuous(hw,\n>                                 vsi->seid, FALSE, NULL);\n>         if (ret != I40E_SUCCESS)\n> -               PMD_DRV_LOG(ERR, \"Failed to disable multicast\n> promiscuous\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to disable multicast\n> promiscuous\");\n>  }\n>\n>  /*\n> @@ -972,7 +972,7 @@ i40e_dev_link_update(struct rte_eth_dev *dev,\n>         if (status != I40E_SUCCESS) {\n>                 link.link_speed = ETH_LINK_SPEED_100;\n>                 link.link_duplex = ETH_LINK_FULL_DUPLEX;\n> -               PMD_DRV_LOG(ERR, \"Failed to get link info\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to get link info\");\n>                 goto out;\n>         }\n>\n> @@ -1059,22 +1059,22 @@ i40e_update_vsi_stats(struct i40e_vsi *vsi)\n>                             &oes->tx_errors, &nes->tx_errors);\n>         vsi->offset_loaded = true;\n>\n> -       PMD_DRV_LOG(DEBUG, \"***************** VSI[%u] stats start\n> *******************\\n\",\n> +       PMD_DRV_LOG(DEBUG, \"***************** VSI[%u] stats start\n> *******************\",\n>                     vsi->vsi_id);\n> -       PMD_DRV_LOG(DEBUG, \"rx_bytes:            %lu\\n\", nes->rx_bytes);\n> -       PMD_DRV_LOG(DEBUG, \"rx_unicast:          %lu\\n\", nes->rx_unicast);\n> -       PMD_DRV_LOG(DEBUG, \"rx_multicast:        %lu\\n\",\n> nes->rx_multicast);\n> -       PMD_DRV_LOG(DEBUG, \"rx_broadcast:        %lu\\n\",\n> nes->rx_broadcast);\n> -       PMD_DRV_LOG(DEBUG, \"rx_discards:         %lu\\n\", nes->rx_discards);\n> -       PMD_DRV_LOG(DEBUG, \"rx_unknown_protocol: %lu\\n\",\n> +       PMD_DRV_LOG(DEBUG, \"rx_bytes:            %lu\", nes->rx_bytes);\n> +       PMD_DRV_LOG(DEBUG, \"rx_unicast:          %lu\", nes->rx_unicast);\n> +       PMD_DRV_LOG(DEBUG, \"rx_multicast:        %lu\", nes->rx_multicast);\n> +       PMD_DRV_LOG(DEBUG, \"rx_broadcast:        %lu\", nes->rx_broadcast);\n> +       PMD_DRV_LOG(DEBUG, \"rx_discards:         %lu\", nes->rx_discards);\n> +       PMD_DRV_LOG(DEBUG, \"rx_unknown_protocol: %lu\",\n>                     nes->rx_unknown_protocol);\n> -       PMD_DRV_LOG(DEBUG, \"tx_bytes:            %lu\\n\", nes->tx_bytes);\n> -       PMD_DRV_LOG(DEBUG, \"tx_unicast:          %lu\\n\", nes->tx_unicast);\n> -       PMD_DRV_LOG(DEBUG, \"tx_multicast:        %lu\\n\",\n> nes->tx_multicast);\n> -       PMD_DRV_LOG(DEBUG, \"tx_broadcast:        %lu\\n\",\n> nes->tx_broadcast);\n> -       PMD_DRV_LOG(DEBUG, \"tx_discards:         %lu\\n\", nes->tx_discards);\n> -       PMD_DRV_LOG(DEBUG, \"tx_errors:           %lu\\n\", nes->tx_errors);\n> -       PMD_DRV_LOG(DEBUG, \"***************** VSI[%u] stats end\n> *******************\\n\",\n> +       PMD_DRV_LOG(DEBUG, \"tx_bytes:            %lu\", nes->tx_bytes);\n> +       PMD_DRV_LOG(DEBUG, \"tx_unicast:          %lu\", nes->tx_unicast);\n> +       PMD_DRV_LOG(DEBUG, \"tx_multicast:        %lu\", nes->tx_multicast);\n> +       PMD_DRV_LOG(DEBUG, \"tx_broadcast:        %lu\", nes->tx_broadcast);\n> +       PMD_DRV_LOG(DEBUG, \"tx_discards:         %lu\", nes->tx_discards);\n> +       PMD_DRV_LOG(DEBUG, \"tx_errors:           %lu\", nes->tx_errors);\n> +       PMD_DRV_LOG(DEBUG, \"***************** VSI[%u] stats end\n> *******************\",\n>                     vsi->vsi_id);\n>  }\n>\n> @@ -1276,74 +1276,74 @@ i40e_dev_stats_get(struct rte_eth_dev *dev, struct\n> rte_eth_stats *stats)\n>         if (pf->main_vsi)\n>                 i40e_update_vsi_stats(pf->main_vsi);\n>\n> -       PMD_DRV_LOG(DEBUG, \"***************** PF stats start\n> *******************\\n\");\n> -       PMD_DRV_LOG(DEBUG, \"rx_bytes:            %lu\\n\", ns->eth.rx_bytes);\n> -       PMD_DRV_LOG(DEBUG, \"rx_unicast:          %lu\\n\",\n> ns->eth.rx_unicast);\n> -       PMD_DRV_LOG(DEBUG, \"rx_multicast:        %lu\\n\",\n> ns->eth.rx_multicast);\n> -       PMD_DRV_LOG(DEBUG, \"rx_broadcast:        %lu\\n\",\n> ns->eth.rx_broadcast);\n> -       PMD_DRV_LOG(DEBUG, \"rx_discards:         %lu\\n\",\n> ns->eth.rx_discards);\n> -       PMD_DRV_LOG(DEBUG, \"rx_unknown_protocol: %lu\\n\",\n> +       PMD_DRV_LOG(DEBUG, \"***************** PF stats start\n> *******************\");\n> +       PMD_DRV_LOG(DEBUG, \"rx_bytes:            %lu\", ns->eth.rx_bytes);\n> +       PMD_DRV_LOG(DEBUG, \"rx_unicast:          %lu\", ns->eth.rx_unicast);\n> +       PMD_DRV_LOG(DEBUG, \"rx_multicast:        %lu\",\n> ns->eth.rx_multicast);\n> +       PMD_DRV_LOG(DEBUG, \"rx_broadcast:        %lu\",\n> ns->eth.rx_broadcast);\n> +       PMD_DRV_LOG(DEBUG, \"rx_discards:         %lu\",\n> ns->eth.rx_discards);\n> +       PMD_DRV_LOG(DEBUG, \"rx_unknown_protocol: %lu\",\n>                     ns->eth.rx_unknown_protocol);\n> -       PMD_DRV_LOG(DEBUG, \"tx_bytes:            %lu\\n\", ns->eth.tx_bytes);\n> -       PMD_DRV_LOG(DEBUG, \"tx_unicast:          %lu\\n\",\n> ns->eth.tx_unicast);\n> -       PMD_DRV_LOG(DEBUG, \"tx_multicast:        %lu\\n\",\n> ns->eth.tx_multicast);\n> -       PMD_DRV_LOG(DEBUG, \"tx_broadcast:        %lu\\n\",\n> ns->eth.tx_broadcast);\n> -       PMD_DRV_LOG(DEBUG, \"tx_discards:         %lu\\n\",\n> ns->eth.tx_discards);\n> -       PMD_DRV_LOG(DEBUG, \"tx_errors:           %lu\\n\",\n> ns->eth.tx_errors);\n> -\n> -       PMD_DRV_LOG(DEBUG, \"tx_dropped_link_down:     %lu\\n\",\n> +       PMD_DRV_LOG(DEBUG, \"tx_bytes:            %lu\", ns->eth.tx_bytes);\n> +       PMD_DRV_LOG(DEBUG, \"tx_unicast:          %lu\", ns->eth.tx_unicast);\n> +       PMD_DRV_LOG(DEBUG, \"tx_multicast:        %lu\",\n> ns->eth.tx_multicast);\n> +       PMD_DRV_LOG(DEBUG, \"tx_broadcast:        %lu\",\n> ns->eth.tx_broadcast);\n> +       PMD_DRV_LOG(DEBUG, \"tx_discards:         %lu\",\n> ns->eth.tx_discards);\n> +       PMD_DRV_LOG(DEBUG, \"tx_errors:           %lu\", ns->eth.tx_errors);\n> +\n> +       PMD_DRV_LOG(DEBUG, \"tx_dropped_link_down:     %lu\",\n>                     ns->tx_dropped_link_down);\n> -       PMD_DRV_LOG(DEBUG, \"crc_errors:               %lu\\n\",\n> ns->crc_errors);\n> -       PMD_DRV_LOG(DEBUG, \"illegal_bytes:            %lu\\n\",\n> +       PMD_DRV_LOG(DEBUG, \"crc_errors:               %lu\",\n> ns->crc_errors);\n> +       PMD_DRV_LOG(DEBUG, \"illegal_bytes:            %lu\",\n>                     ns->illegal_bytes);\n> -       PMD_DRV_LOG(DEBUG, \"error_bytes:              %lu\\n\",\n> ns->error_bytes);\n> -       PMD_DRV_LOG(DEBUG, \"mac_local_faults:         %lu\\n\",\n> +       PMD_DRV_LOG(DEBUG, \"error_bytes:              %lu\",\n> ns->error_bytes);\n> +       PMD_DRV_LOG(DEBUG, \"mac_local_faults:         %lu\",\n>                     ns->mac_local_faults);\n> -       PMD_DRV_LOG(DEBUG, \"mac_remote_faults:        %lu\\n\",\n> +       PMD_DRV_LOG(DEBUG, \"mac_remote_faults:        %lu\",\n>                     ns->mac_remote_faults);\n> -       PMD_DRV_LOG(DEBUG, \"rx_length_errors:         %lu\\n\",\n> +       PMD_DRV_LOG(DEBUG, \"rx_length_errors:         %lu\",\n>                     ns->rx_length_errors);\n> -       PMD_DRV_LOG(DEBUG, \"link_xon_rx:              %lu\\n\",\n> ns->link_xon_rx);\n> -       PMD_DRV_LOG(DEBUG, \"link_xoff_rx:             %lu\\n\",\n> ns->link_xoff_rx);\n> +       PMD_DRV_LOG(DEBUG, \"link_xon_rx:              %lu\",\n> ns->link_xon_rx);\n> +       PMD_DRV_LOG(DEBUG, \"link_xoff_rx:             %lu\",\n> ns->link_xoff_rx);\n>         for (i = 0; i < 8; i++) {\n> -               PMD_DRV_LOG(DEBUG, \"priority_xon_rx[%d]:      %lu\\n\",\n> +               PMD_DRV_LOG(DEBUG, \"priority_xon_rx[%d]:      %lu\",\n>                                 i, ns->priority_xon_rx[i]);\n> -               PMD_DRV_LOG(DEBUG, \"priority_xoff_rx[%d]:     %lu\\n\",\n> +               PMD_DRV_LOG(DEBUG, \"priority_xoff_rx[%d]:     %lu\",\n>                                 i, ns->priority_xoff_rx[i]);\n>         }\n> -       PMD_DRV_LOG(DEBUG, \"link_xon_tx:              %lu\\n\",\n> ns->link_xon_tx);\n> -       PMD_DRV_LOG(DEBUG, \"link_xoff_tx:             %lu\\n\",\n> ns->link_xoff_tx);\n> +       PMD_DRV_LOG(DEBUG, \"link_xon_tx:              %lu\",\n> ns->link_xon_tx);\n> +       PMD_DRV_LOG(DEBUG, \"link_xoff_tx:             %lu\",\n> ns->link_xoff_tx);\n>         for (i = 0; i < 8; i++) {\n> -               PMD_DRV_LOG(DEBUG, \"priority_xon_tx[%d]:      %lu\\n\",\n> +               PMD_DRV_LOG(DEBUG, \"priority_xon_tx[%d]:      %lu\",\n>                                 i, ns->priority_xon_tx[i]);\n> -               PMD_DRV_LOG(DEBUG, \"priority_xoff_tx[%d]:     %lu\\n\",\n> +               PMD_DRV_LOG(DEBUG, \"priority_xoff_tx[%d]:     %lu\",\n>                                 i, ns->priority_xoff_tx[i]);\n> -               PMD_DRV_LOG(DEBUG, \"priority_xon_2_xoff[%d]:  %lu\\n\",\n> +               PMD_DRV_LOG(DEBUG, \"priority_xon_2_xoff[%d]:  %lu\",\n>                                 i, ns->priority_xon_2_xoff[i]);\n>         }\n> -       PMD_DRV_LOG(DEBUG, \"rx_size_64:               %lu\\n\",\n> ns->rx_size_64);\n> -       PMD_DRV_LOG(DEBUG, \"rx_size_127:              %lu\\n\",\n> ns->rx_size_127);\n> -       PMD_DRV_LOG(DEBUG, \"rx_size_255:              %lu\\n\",\n> ns->rx_size_255);\n> -       PMD_DRV_LOG(DEBUG, \"rx_size_511:              %lu\\n\",\n> ns->rx_size_511);\n> -       PMD_DRV_LOG(DEBUG, \"rx_size_1023:             %lu\\n\",\n> ns->rx_size_1023);\n> -       PMD_DRV_LOG(DEBUG, \"rx_size_1522:             %lu\\n\",\n> ns->rx_size_1522);\n> -       PMD_DRV_LOG(DEBUG, \"rx_size_big:              %lu\\n\",\n> ns->rx_size_big);\n> -       PMD_DRV_LOG(DEBUG, \"rx_undersize:             %lu\\n\",\n> ns->rx_undersize);\n> -       PMD_DRV_LOG(DEBUG, \"rx_fragments:             %lu\\n\",\n> ns->rx_fragments);\n> -       PMD_DRV_LOG(DEBUG, \"rx_oversize:              %lu\\n\",\n> ns->rx_oversize);\n> -       PMD_DRV_LOG(DEBUG, \"rx_jabber:                %lu\\n\",\n> ns->rx_jabber);\n> -       PMD_DRV_LOG(DEBUG, \"tx_size_64:               %lu\\n\",\n> ns->tx_size_64);\n> -       PMD_DRV_LOG(DEBUG, \"tx_size_127:              %lu\\n\",\n> ns->tx_size_127);\n> -       PMD_DRV_LOG(DEBUG, \"tx_size_255:              %lu\\n\",\n> ns->tx_size_255);\n> -       PMD_DRV_LOG(DEBUG, \"tx_size_511:              %lu\\n\",\n> ns->tx_size_511);\n> -       PMD_DRV_LOG(DEBUG, \"tx_size_1023:             %lu\\n\",\n> ns->tx_size_1023);\n> -       PMD_DRV_LOG(DEBUG, \"tx_size_1522:             %lu\\n\",\n> ns->tx_size_1522);\n> -       PMD_DRV_LOG(DEBUG, \"tx_size_big:              %lu\\n\",\n> ns->tx_size_big);\n> -       PMD_DRV_LOG(DEBUG, \"mac_short_packet_dropped: %lu\\n\",\n> +       PMD_DRV_LOG(DEBUG, \"rx_size_64:               %lu\",\n> ns->rx_size_64);\n> +       PMD_DRV_LOG(DEBUG, \"rx_size_127:              %lu\",\n> ns->rx_size_127);\n> +       PMD_DRV_LOG(DEBUG, \"rx_size_255:              %lu\",\n> ns->rx_size_255);\n> +       PMD_DRV_LOG(DEBUG, \"rx_size_511:              %lu\",\n> ns->rx_size_511);\n> +       PMD_DRV_LOG(DEBUG, \"rx_size_1023:             %lu\",\n> ns->rx_size_1023);\n> +       PMD_DRV_LOG(DEBUG, \"rx_size_1522:             %lu\",\n> ns->rx_size_1522);\n> +       PMD_DRV_LOG(DEBUG, \"rx_size_big:              %lu\",\n> ns->rx_size_big);\n> +       PMD_DRV_LOG(DEBUG, \"rx_undersize:             %lu\",\n> ns->rx_undersize);\n> +       PMD_DRV_LOG(DEBUG, \"rx_fragments:             %lu\",\n> ns->rx_fragments);\n> +       PMD_DRV_LOG(DEBUG, \"rx_oversize:              %lu\",\n> ns->rx_oversize);\n> +       PMD_DRV_LOG(DEBUG, \"rx_jabber:                %lu\", ns->rx_jabber);\n> +       PMD_DRV_LOG(DEBUG, \"tx_size_64:               %lu\",\n> ns->tx_size_64);\n> +       PMD_DRV_LOG(DEBUG, \"tx_size_127:              %lu\",\n> ns->tx_size_127);\n> +       PMD_DRV_LOG(DEBUG, \"tx_size_255:              %lu\",\n> ns->tx_size_255);\n> +       PMD_DRV_LOG(DEBUG, \"tx_size_511:              %lu\",\n> ns->tx_size_511);\n> +       PMD_DRV_LOG(DEBUG, \"tx_size_1023:             %lu\",\n> ns->tx_size_1023);\n> +       PMD_DRV_LOG(DEBUG, \"tx_size_1522:             %lu\",\n> ns->tx_size_1522);\n> +       PMD_DRV_LOG(DEBUG, \"tx_size_big:              %lu\",\n> ns->tx_size_big);\n> +       PMD_DRV_LOG(DEBUG, \"mac_short_packet_dropped: %lu\",\n>                         ns->mac_short_packet_dropped);\n> -       PMD_DRV_LOG(DEBUG, \"checksum_error:           %lu\\n\",\n> +       PMD_DRV_LOG(DEBUG, \"checksum_error:           %lu\",\n>                     ns->checksum_error);\n> -       PMD_DRV_LOG(DEBUG, \"***************** PF stats end\n> ********************\\n\");\n> +       PMD_DRV_LOG(DEBUG, \"***************** PF stats end\n> ********************\");\n>  }\n>\n>  /* Reset the statistics */\n> @@ -1520,12 +1520,12 @@ i40e_macaddr_add(struct rte_eth_dev *dev,\n>         int ret;\n>\n>         if (!is_valid_assigned_ether_addr(mac_addr)) {\n> -               PMD_DRV_LOG(ERR, \"Invalid ethernet address\\n\");\n> +               PMD_DRV_LOG(ERR, \"Invalid ethernet address\");\n>                 return;\n>         }\n>\n>         if (is_same_ether_addr(mac_addr, &(pf->dev_addr))) {\n> -               PMD_DRV_LOG(INFO, \"Ignore adding permanent mac address\\n\");\n> +               PMD_DRV_LOG(INFO, \"Ignore adding permanent mac address\");\n>                 return;\n>         }\n>\n> @@ -1533,7 +1533,7 @@ i40e_macaddr_add(struct rte_eth_dev *dev,\n>         ret = i40e_aq_mac_address_write(hw, I40E_AQC_WRITE_TYPE_LAA_ONLY,\n>                                         mac_addr->addr_bytes, NULL);\n>         if (ret != I40E_SUCCESS) {\n> -               PMD_DRV_LOG(ERR, \"Failed to write mac address\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to write mac address\");\n>                 return;\n>         }\n>\n> @@ -1543,7 +1543,7 @@ i40e_macaddr_add(struct rte_eth_dev *dev,\n>\n>         ret = i40e_vsi_add_mac(vsi, mac_addr);\n>         if (ret != I40E_SUCCESS) {\n> -               PMD_DRV_LOG(ERR, \"Failed to add MACVLAN filter\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to add MACVLAN filter\");\n>                 return;\n>         }\n>\n> @@ -1573,7 +1573,7 @@ i40e_macaddr_remove(struct rte_eth_dev *dev,\n> uint32_t index)\n>         ret = i40e_aq_mac_address_write(hw, I40E_AQC_WRITE_TYPE_LAA_ONLY,\n>                                         hw->mac.perm_addr, NULL);\n>         if (ret != I40E_SUCCESS) {\n> -               PMD_DRV_LOG(ERR, \"Failed to write mac address\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to write mac address\");\n>                 return;\n>         }\n>\n> @@ -1794,7 +1794,7 @@ i40e_get_cap(struct i40e_hw *hw)\n>                                                 I40E_MAX_CAP_ELE_NUM;\n>         buf = rte_zmalloc(\"i40e\", len, 0);\n>         if (!buf) {\n> -               PMD_DRV_LOG(ERR, \"Failed to allocate memory\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to allocate memory\");\n>                 return I40E_ERR_NO_MEMORY;\n>         }\n>\n> @@ -1802,7 +1802,7 @@ i40e_get_cap(struct i40e_hw *hw)\n>         ret = i40e_aq_discover_capabilities(hw, buf, len, &size,\n>                         i40e_aqc_opc_list_func_capabilities, NULL);\n>         if (ret != I40E_SUCCESS)\n> -               PMD_DRV_LOG(ERR, \"Failed to discover capabilities\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to discover capabilities\");\n>\n>         /* Free the temporary buffer after being used */\n>         rte_free(buf);\n> @@ -1819,13 +1819,13 @@ i40e_pf_parameter_init(struct rte_eth_dev *dev)\n>\n>         /* First check if FW support SRIOV */\n>         if (dev->pci_dev->max_vfs && !hw->func_caps.sr_iov_1_1) {\n> -               PMD_INIT_LOG(ERR, \"HW configuration doesn't support\n> SRIOV\\n\");\n> +               PMD_INIT_LOG(ERR, \"HW configuration doesn't support\n> SRIOV\");\n>                 return -EINVAL;\n>         }\n>\n>         pf->flags = I40E_FLAG_HEADER_SPLIT_DISABLED;\n>         pf->max_num_vsi = RTE_MIN(hw->func_caps.num_vsis,\n> I40E_MAX_NUM_VSIS);\n> -       PMD_INIT_LOG(INFO, \"Max supported VSIs:%u\\n\", pf->max_num_vsi);\n> +       PMD_INIT_LOG(INFO, \"Max supported VSIs:%u\", pf->max_num_vsi);\n>         /* Allocate queues for pf */\n>         if (hw->func_caps.rss) {\n>                 pf->flags |= I40E_FLAG_RSS;\n> @@ -1837,27 +1837,27 @@ i40e_pf_parameter_init(struct rte_eth_dev *dev)\n>         sum_queues = pf->lan_nb_qps;\n>         /* Default VSI is not counted in */\n>         sum_vsis = 0;\n> -       PMD_INIT_LOG(INFO, \"PF queue pairs:%u\\n\", pf->lan_nb_qps);\n> +       PMD_INIT_LOG(INFO, \"PF queue pairs:%u\", pf->lan_nb_qps);\n>\n>         if (hw->func_caps.sr_iov_1_1 && dev->pci_dev->max_vfs) {\n>                 pf->flags |= I40E_FLAG_SRIOV;\n>                 pf->vf_nb_qps = RTE_LIBRTE_I40E_QUEUE_NUM_PER_VF;\n>                 if (dev->pci_dev->max_vfs > hw->func_caps.num_vfs) {\n>                         PMD_INIT_LOG(ERR, \"Config VF number %u, \"\n> -                               \"max supported %u.\\n\",\n> dev->pci_dev->max_vfs,\n> +                               \"max supported %u.\", dev->pci_dev->max_vfs,\n>                                                 hw->func_caps.num_vfs);\n>                         return -EINVAL;\n>                 }\n>                 if (pf->vf_nb_qps > I40E_MAX_QP_NUM_PER_VF) {\n>                         PMD_INIT_LOG(ERR, \"FVL VF queue %u, \"\n> -                               \"max support %u queues.\\n\", pf->vf_nb_qps,\n> +                               \"max support %u queues.\", pf->vf_nb_qps,\n>                                                 I40E_MAX_QP_NUM_PER_VF);\n>                         return -EINVAL;\n>                 }\n>                 pf->vf_num = dev->pci_dev->max_vfs;\n>                 sum_queues += pf->vf_nb_qps * pf->vf_num;\n>                 sum_vsis   += pf->vf_num;\n> -               PMD_INIT_LOG(INFO, \"Max VF num:%u each has queue\n> pairs:%u\\n\",\n> +               PMD_INIT_LOG(INFO, \"Max VF num:%u each has queue pairs:%u\",\n>                                                 pf->vf_num, pf->vf_nb_qps);\n>         } else\n>                 pf->vf_num = 0;\n> @@ -1867,7 +1867,7 @@ i40e_pf_parameter_init(struct rte_eth_dev *dev)\n>                 pf->vmdq_nb_qps = I40E_DEFAULT_QP_NUM_VMDQ;\n>                 sum_queues += pf->vmdq_nb_qps;\n>                 sum_vsis += 1;\n> -               PMD_INIT_LOG(INFO, \"VMDQ queue pairs:%u\\n\",\n> pf->vmdq_nb_qps);\n> +               PMD_INIT_LOG(INFO, \"VMDQ queue pairs:%u\", pf->vmdq_nb_qps);\n>         }\n>\n>         if (hw->func_caps.fd) {\n> @@ -1881,17 +1881,18 @@ i40e_pf_parameter_init(struct rte_eth_dev *dev)\n>\n>         if (sum_vsis > pf->max_num_vsi ||\n>                 sum_queues > hw->func_caps.num_rx_qp) {\n> -               PMD_INIT_LOG(ERR, \"VSI/QUEUE setting can't be\n> satisfied\\n\");\n> -               PMD_INIT_LOG(ERR, \"Max VSIs: %u, asked:%u\\n\",\n> +               PMD_INIT_LOG(ERR, \"VSI/QUEUE setting can't be satisfied\");\n> +               PMD_INIT_LOG(ERR, \"Max VSIs: %u, asked:%u\",\n>                                 pf->max_num_vsi, sum_vsis);\n> -               PMD_INIT_LOG(ERR, \"Total queue pairs:%u, asked:%u\\n\",\n> +               PMD_INIT_LOG(ERR, \"Total queue pairs:%u, asked:%u\",\n>                                 hw->func_caps.num_rx_qp, sum_queues);\n>                 return -EINVAL;\n>         }\n>\n> -       /* Each VSI occupy 1 MSIX interrupt at least, plus IRQ0 for misc\n> intr cause */\n> +       /* Each VSI occupy 1 MSIX interrupt at least, plus IRQ0 for misc\n> intr\n> +        * cause */\n>         if (sum_vsis > hw->func_caps.num_msix_vectors - 1) {\n> -               PMD_INIT_LOG(ERR, \"Too many VSIs(%u), MSIX intr(%u) not\n> enough\\n\",\n> +               PMD_INIT_LOG(ERR, \"Too many VSIs(%u), MSIX intr(%u) not\n> enough\",\n>                                 sum_vsis, hw->func_caps.num_msix_vectors);\n>                 return -EINVAL;\n>         }\n> @@ -1910,7 +1911,7 @@ i40e_pf_get_switch_config(struct i40e_pf *pf)\n>         switch_config = (struct i40e_aqc_get_switch_config_resp *)\\\n>                         rte_zmalloc(\"i40e\", I40E_AQ_LARGE_BUF, 0);\n>         if (!switch_config) {\n> -               PMD_DRV_LOG(ERR, \"Failed to allocated memory\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to allocated memory\");\n>                 return -ENOMEM;\n>         }\n>\n> @@ -1918,12 +1919,12 @@ i40e_pf_get_switch_config(struct i40e_pf *pf)\n>         ret = i40e_aq_get_switch_config(hw, switch_config,\n>                 I40E_AQ_LARGE_BUF, &start_seid, NULL);\n>         if (ret != I40E_SUCCESS) {\n> -               PMD_DRV_LOG(ERR, \"Failed to get switch configurations\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to get switch configurations\");\n>                 goto fail;\n>         }\n>         num_reported =\n> rte_le_to_cpu_16(switch_config->header.num_reported);\n>         if (num_reported != 1) { /* The number should be 1 */\n> -               PMD_DRV_LOG(ERR, \"Wrong number of switch config\n> reported\\n\");\n> +               PMD_DRV_LOG(ERR, \"Wrong number of switch config reported\");\n>                 goto fail;\n>         }\n>\n> @@ -1933,7 +1934,7 @@ i40e_pf_get_switch_config(struct i40e_pf *pf)\n>                 pf->mac_seid = rte_le_to_cpu_16(element->uplink_seid);\n>                 pf->main_vsi_seid = rte_le_to_cpu_16(element->seid);\n>         } else\n> -               PMD_DRV_LOG(INFO, \"Unknown element type\\n\");\n> +               PMD_DRV_LOG(INFO, \"Unknown element type\");\n>\n>  fail:\n>         rte_free(switch_config);\n> @@ -1952,8 +1953,7 @@ i40e_res_pool_init (struct i40e_res_pool_info *pool,\n> uint32_t base,\n>\n>         entry = rte_zmalloc(\"i40e\", sizeof(*entry), 0);\n>         if (entry == NULL) {\n> -               PMD_DRV_LOG(ERR, \"Failed to allocate memory for \"\n> -                                               \"resource pool\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to allocate memory for resource\n> pool\");\n>                 return -ENOMEM;\n>         }\n>\n> @@ -2006,7 +2006,7 @@ i40e_res_pool_free(struct i40e_res_pool_info *pool,\n>         int insert;\n>\n>         if (pool == NULL) {\n> -               PMD_DRV_LOG(ERR, \"Invalid parameter\\n\");\n> +               PMD_DRV_LOG(ERR, \"Invalid parameter\");\n>                 return -EINVAL;\n>         }\n>\n> @@ -2022,7 +2022,7 @@ i40e_res_pool_free(struct i40e_res_pool_info *pool,\n>\n>         /* Not find, return */\n>         if (valid_entry == NULL) {\n> -               PMD_DRV_LOG(ERR, \"Failed to find entry\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to find entry\");\n>                 return -EINVAL;\n>         }\n>\n> @@ -2091,12 +2091,12 @@ i40e_res_pool_alloc(struct i40e_res_pool_info\n> *pool,\n>         struct pool_entry *entry, *valid_entry;\n>\n>         if (pool == NULL || num == 0) {\n> -               PMD_DRV_LOG(ERR, \"Invalid parameter\\n\");\n> +               PMD_DRV_LOG(ERR, \"Invalid parameter\");\n>                 return -EINVAL;\n>         }\n>\n>         if (pool->num_free < num) {\n> -               PMD_DRV_LOG(ERR, \"No resource. ask:%u, available:%u\\n\",\n> +               PMD_DRV_LOG(ERR, \"No resource. ask:%u, available:%u\",\n>                                 num, pool->num_free);\n>                 return -ENOMEM;\n>         }\n> @@ -2117,7 +2117,7 @@ i40e_res_pool_alloc(struct i40e_res_pool_info *pool,\n>\n>         /* Not find one to satisfy the request, return */\n>         if (valid_entry == NULL) {\n> -               PMD_DRV_LOG(ERR, \"No valid entry found\\n\");\n> +               PMD_DRV_LOG(ERR, \"No valid entry found\");\n>                 return -ENOMEM;\n>         }\n>         /**\n> @@ -2135,7 +2135,7 @@ i40e_res_pool_alloc(struct i40e_res_pool_info *pool,\n>                 entry = rte_zmalloc(\"res_pool\", sizeof(*entry), 0);\n>                 if (entry == NULL) {\n>                         PMD_DRV_LOG(ERR, \"Failed to allocate memory for \"\n> -                                       \"resource pool\\n\");\n> +                                   \"resource pool\");\n>                         return -ENOMEM;\n>                 }\n>                 entry->base = valid_entry->base;\n> @@ -2170,15 +2170,14 @@ validate_tcmap_parameter(struct i40e_vsi *vsi,\n> uint8_t enabled_tcmap)\n>\n>         /* If DCB is not supported, only default TC is supported */\n>         if (!hw->func_caps.dcb && enabled_tcmap != I40E_DEFAULT_TCMAP) {\n> -               PMD_DRV_LOG(ERR, \"DCB is not enabled, \"\n> -                               \"only TC0 is supported\\n\");\n> +               PMD_DRV_LOG(ERR, \"DCB is not enabled, only TC0 is\n> supported\");\n>                 return -EINVAL;\n>         }\n>\n>         if (!bitmap_is_subset(hw->func_caps.enabled_tcmap, enabled_tcmap))\n> {\n>                 PMD_DRV_LOG(ERR, \"Enabled TC map 0x%x not applicable to \"\n> -                       \"HW support 0x%x\\n\", hw->func_caps.enabled_tcmap,\n> -                                                       enabled_tcmap);\n> +                           \"HW support 0x%x\", hw->func_caps.enabled_tcmap,\n> +                           enabled_tcmap);\n>                 return -EINVAL;\n>         }\n>         return I40E_SUCCESS;\n> @@ -2194,7 +2193,7 @@ i40e_vsi_vlan_pvid_set(struct i40e_vsi *vsi,\n>         int ret;\n>\n>         if (vsi == NULL || info == NULL) {\n> -               PMD_DRV_LOG(ERR, \"invalid parameters\\n\");\n> +               PMD_DRV_LOG(ERR, \"invalid parameters\");\n>                 return I40E_ERR_PARAM;\n>         }\n>\n> @@ -2226,7 +2225,7 @@ i40e_vsi_vlan_pvid_set(struct i40e_vsi *vsi,\n>         hw = I40E_VSI_TO_HW(vsi);\n>         ret = i40e_aq_update_vsi_params(hw, &ctxt, NULL);\n>         if (ret != I40E_SUCCESS)\n> -               PMD_DRV_LOG(ERR, \"Failed to update VSI params\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to update VSI params\");\n>\n>         return ret;\n>  }\n> @@ -2243,7 +2242,7 @@ i40e_vsi_update_tc_bandwidth(struct i40e_vsi *vsi,\n> uint8_t enabled_tcmap)\n>                 return ret;\n>\n>         if (!vsi->seid) {\n> -               PMD_DRV_LOG(ERR, \"seid not valid\\n\");\n> +               PMD_DRV_LOG(ERR, \"seid not valid\");\n>                 return -EINVAL;\n>         }\n>\n> @@ -2255,7 +2254,7 @@ i40e_vsi_update_tc_bandwidth(struct i40e_vsi *vsi,\n> uint8_t enabled_tcmap)\n>\n>         ret = i40e_aq_config_vsi_tc_bw(hw, vsi->seid, &tc_bw_data, NULL);\n>         if (ret != I40E_SUCCESS) {\n> -               PMD_DRV_LOG(ERR, \"Failed to configure TC BW\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to configure TC BW\");\n>                 return ret;\n>         }\n>\n> @@ -2333,7 +2332,7 @@ i40e_veb_release(struct i40e_veb *veb)\n>                 return -EINVAL;\n>\n>         if (!TAILQ_EMPTY(&veb->head)) {\n> -               PMD_DRV_LOG(ERR, \"VEB still has VSI attached, can't\n> remove\\n\");\n> +               PMD_DRV_LOG(ERR, \"VEB still has VSI attached, can't\n> remove\");\n>                 return -EACCES;\n>         }\n>\n> @@ -2357,14 +2356,14 @@ i40e_veb_setup(struct i40e_pf *pf, struct i40e_vsi\n> *vsi)\n>\n>         if (NULL == pf || vsi == NULL) {\n>                 PMD_DRV_LOG(ERR, \"veb setup failed, \"\n> -                       \"associated VSI shouldn't null\\n\");\n> +                       \"associated VSI shouldn't null\");\n>                 return NULL;\n>         }\n>         hw = I40E_PF_TO_HW(pf);\n>\n>         veb = rte_zmalloc(\"i40e_veb\", sizeof(struct i40e_veb), 0);\n>         if (!veb) {\n> -               PMD_DRV_LOG(ERR, \"Failed to allocate memory for veb\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to allocate memory for veb\");\n>                 goto fail;\n>         }\n>\n> @@ -2376,8 +2375,8 @@ i40e_veb_setup(struct i40e_pf *pf, struct i40e_vsi\n> *vsi)\n>                 I40E_DEFAULT_TCMAP, false, false, &veb->seid, NULL);\n>\n>         if (ret != I40E_SUCCESS) {\n> -               PMD_DRV_LOG(ERR, \"Add veb failed, aq_err: %d\\n\",\n> -                                       hw->aq.asq_last_status);\n> +               PMD_DRV_LOG(ERR, \"Add veb failed, aq_err: %d\",\n> +                           hw->aq.asq_last_status);\n>                 goto fail;\n>         }\n>\n> @@ -2385,8 +2384,8 @@ i40e_veb_setup(struct i40e_pf *pf, struct i40e_vsi\n> *vsi)\n>         ret = i40e_aq_get_veb_parameters(hw, veb->seid, NULL, NULL,\n>                                 &veb->stats_idx, NULL, NULL, NULL);\n>         if (ret != I40E_SUCCESS) {\n> -               PMD_DRV_LOG(ERR, \"Get veb statics index failed, aq_err:\n> %d\\n\",\n> -                                               hw->aq.asq_last_status);\n> +               PMD_DRV_LOG(ERR, \"Get veb statics index failed, aq_err:\n> %d\",\n> +                           hw->aq.asq_last_status);\n>                 goto fail;\n>         }\n>\n> @@ -2433,7 +2432,7 @@ i40e_vsi_release(struct i40e_vsi *vsi)\n>         if (vsi->type != I40E_VSI_MAIN) {\n>                 /* Remove vsi from parent's sibling list */\n>                 if (vsi->parent_vsi == NULL || vsi->parent_vsi->veb ==\n> NULL) {\n> -                       PMD_DRV_LOG(ERR, \"VSI's parent VSI is NULL\\n\");\n> +                       PMD_DRV_LOG(ERR, \"VSI's parent VSI is NULL\");\n>                         return I40E_ERR_PARAM;\n>                 }\n>                 TAILQ_REMOVE(&vsi->parent_vsi->veb->head,\n> @@ -2442,7 +2441,7 @@ i40e_vsi_release(struct i40e_vsi *vsi)\n>                 /* Remove all switch element of the VSI */\n>                 ret = i40e_aq_delete_element(hw, vsi->seid, NULL);\n>                 if (ret != I40E_SUCCESS)\n> -                       PMD_DRV_LOG(ERR, \"Failed to delete element\\n\");\n> +                       PMD_DRV_LOG(ERR, \"Failed to delete element\");\n>         }\n>         i40e_res_pool_free(&pf->qp_pool, vsi->base_queue);\n>\n> @@ -2473,11 +2472,11 @@ i40e_update_default_filter_setting(struct i40e_vsi\n> *vsi)\n>                 struct i40e_mac_filter *f;\n>\n>                 PMD_DRV_LOG(WARNING, \"Cannot remove the default \"\n> -                                               \"macvlan filter\\n\");\n> +                           \"macvlan filter\");\n>                 /* It needs to add the permanent mac into mac list */\n>                 f = rte_zmalloc(\"macv_filter\", sizeof(*f), 0);\n>                 if (f == NULL) {\n> -                       PMD_DRV_LOG(ERR, \"failed to allocate memory\\n\");\n> +                       PMD_DRV_LOG(ERR, \"failed to allocate memory\");\n>                         return I40E_ERR_NO_MEMORY;\n>                 }\n>                 (void)rte_memcpy(&f->macaddr.addr_bytes, hw->mac.perm_addr,\n> @@ -2504,7 +2503,7 @@ i40e_vsi_dump_bw_config(struct i40e_vsi *vsi)\n>         ret = i40e_aq_query_vsi_bw_config(hw, vsi->seid, &bw_config, NULL);\n>         if (ret != I40E_SUCCESS) {\n>                 PMD_DRV_LOG(ERR, \"VSI failed to get bandwidth \"\n> -                       \"configuration %u\\n\", hw->aq.asq_last_status);\n> +                           \"configuration %u\", hw->aq.asq_last_status);\n>                 return ret;\n>         }\n>\n> @@ -2513,21 +2512,21 @@ i40e_vsi_dump_bw_config(struct i40e_vsi *vsi)\n>                                         &ets_sla_config, NULL);\n>         if (ret != I40E_SUCCESS) {\n>                 PMD_DRV_LOG(ERR, \"VSI failed to get TC bandwdith \"\n> -                       \"configuration %u\\n\", hw->aq.asq_last_status);\n> +                           \"configuration %u\", hw->aq.asq_last_status);\n>                 return ret;\n>         }\n>\n>         /* Not store the info yet, just print out */\n> -       PMD_DRV_LOG(INFO, \"VSI bw limit:%u\\n\", bw_config.port_bw_limit);\n> -       PMD_DRV_LOG(INFO, \"VSI max_bw:%u\\n\", bw_config.max_bw);\n> +       PMD_DRV_LOG(INFO, \"VSI bw limit:%u\", bw_config.port_bw_limit);\n> +       PMD_DRV_LOG(INFO, \"VSI max_bw:%u\", bw_config.max_bw);\n>         for (i = 0; i < I40E_MAX_TRAFFIC_CLASS; i++) {\n> -               PMD_DRV_LOG(INFO, \"\\tVSI TC%u:share credits %u\\n\", i,\n> -                                       ets_sla_config.share_credits[i]);\n> -               PMD_DRV_LOG(INFO, \"\\tVSI TC%u:credits %u\\n\", i,\n> -                       rte_le_to_cpu_16(ets_sla_config.credits[i]));\n> +               PMD_DRV_LOG(INFO, \"\\tVSI TC%u:share credits %u\", i,\n> +                           ets_sla_config.share_credits[i]);\n> +               PMD_DRV_LOG(INFO, \"\\tVSI TC%u:credits %u\", i,\n> +                           rte_le_to_cpu_16(ets_sla_config.credits[i]));\n>                 PMD_DRV_LOG(INFO, \"\\tVSI TC%u: max credits: %u\", i,\n> -                       rte_le_to_cpu_16(ets_sla_config.credits[i / 4]) >>\n> -                                                               (i * 4));\n> +                           rte_le_to_cpu_16(ets_sla_config.credits[i /\n> 4]) >>\n> +                           (i * 4));\n>         }\n>\n>         return 0;\n> @@ -2549,13 +2548,13 @@ i40e_vsi_setup(struct i40e_pf *pf,\n>\n>         if (type != I40E_VSI_MAIN && uplink_vsi == NULL) {\n>                 PMD_DRV_LOG(ERR, \"VSI setup failed, \"\n> -                       \"VSI link shouldn't be NULL\\n\");\n> +                           \"VSI link shouldn't be NULL\");\n>                 return NULL;\n>         }\n>\n>         if (type == I40E_VSI_MAIN && uplink_vsi != NULL) {\n>                 PMD_DRV_LOG(ERR, \"VSI setup failed, MAIN VSI \"\n> -                               \"uplink VSI should be NULL\\n\");\n> +                           \"uplink VSI should be NULL\");\n>                 return NULL;\n>         }\n>\n> @@ -2564,14 +2563,14 @@ i40e_vsi_setup(struct i40e_pf *pf,\n>                 uplink_vsi->veb = i40e_veb_setup(pf, uplink_vsi);\n>\n>                 if (NULL == uplink_vsi->veb) {\n> -                       PMD_DRV_LOG(ERR, \"VEB setup failed\\n\");\n> +                       PMD_DRV_LOG(ERR, \"VEB setup failed\");\n>                         return NULL;\n>                 }\n>         }\n>\n>         vsi = rte_zmalloc(\"i40e_vsi\", sizeof(struct i40e_vsi), 0);\n>         if (!vsi) {\n> -               PMD_DRV_LOG(ERR, \"Failed to allocate memory for vsi\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to allocate memory for vsi\");\n>                 return NULL;\n>         }\n>         TAILQ_INIT(&vsi->mac_list);\n> @@ -2594,7 +2593,7 @@ i40e_vsi_setup(struct i40e_pf *pf,\n>         ret = i40e_res_pool_alloc(&pf->qp_pool, vsi->nb_qps);\n>         if (ret < 0) {\n>                 PMD_DRV_LOG(ERR, \"VSI %d allocate queue failed %d\",\n> -                               vsi->seid, ret);\n> +                           vsi->seid, ret);\n>                 goto fail_mem;\n>         }\n>         vsi->base_queue = ret;\n> @@ -2603,7 +2602,8 @@ i40e_vsi_setup(struct i40e_pf *pf,\n>         if (type != I40E_VSI_SRIOV) {\n>                 ret = i40e_res_pool_alloc(&pf->msix_pool, 1);\n>                 if (ret < 0) {\n> -                       PMD_DRV_LOG(ERR, \"VSI %d get heap failed %d\",\n> vsi->seid, ret);\n> +                       PMD_DRV_LOG(ERR, \"VSI %d get heap failed %d\",\n> +                                   vsi->seid, ret);\n>                         goto fail_queue_alloc;\n>                 }\n>                 vsi->msix_intr = ret;\n> @@ -2629,7 +2629,7 @@ i40e_vsi_setup(struct i40e_pf *pf,\n>                 ctxt.vf_num = 0;\n>                 ret = i40e_aq_get_vsi_params(hw, &ctxt, NULL);\n>                 if (ret != I40E_SUCCESS) {\n> -                       PMD_DRV_LOG(ERR, \"Failed to get VSI params\\n\");\n> +                       PMD_DRV_LOG(ERR, \"Failed to get VSI params\");\n>                         goto fail_msix_alloc;\n>                 }\n>                 (void)rte_memcpy(&vsi->info, &ctxt.info,\n> @@ -2640,7 +2640,7 @@ i40e_vsi_setup(struct i40e_pf *pf,\n>                 /* Configure tc, enabled TC0 only */\n>                 if (i40e_vsi_update_tc_bandwidth(vsi, I40E_DEFAULT_TCMAP)\n> !=\n>                         I40E_SUCCESS) {\n> -                       PMD_DRV_LOG(ERR, \"Failed to update TC\n> bandwidth\\n\");\n> +                       PMD_DRV_LOG(ERR, \"Failed to update TC bandwidth\");\n>                         goto fail_msix_alloc;\n>                 }\n>\n> @@ -2656,7 +2656,7 @@ i40e_vsi_setup(struct i40e_pf *pf,\n>                                                 I40E_DEFAULT_TCMAP);\n>                 if (ret != I40E_SUCCESS) {\n>                         PMD_DRV_LOG(ERR, \"Failed to configure \"\n> -                                       \"TC queue mapping\\n\");\n> +                                   \"TC queue mapping\");\n>                         goto fail_msix_alloc;\n>                 }\n>                 ctxt.seid = vsi->seid;\n> @@ -2667,7 +2667,7 @@ i40e_vsi_setup(struct i40e_pf *pf,\n>                 /* Update VSI parameters */\n>                 ret = i40e_aq_update_vsi_params(hw, &ctxt, NULL);\n>                 if (ret != I40E_SUCCESS) {\n> -                       PMD_DRV_LOG(ERR, \"Failed to update VSI params\\n\");\n> +                       PMD_DRV_LOG(ERR, \"Failed to update VSI params\");\n>                         goto fail_msix_alloc;\n>                 }\n>\n> @@ -2719,7 +2719,7 @@ i40e_vsi_setup(struct i40e_pf *pf,\n>                                                 I40E_DEFAULT_TCMAP);\n>                 if (ret != I40E_SUCCESS) {\n>                         PMD_DRV_LOG(ERR, \"Failed to configure \"\n> -                                       \"TC queue mapping\\n\");\n> +                                   \"TC queue mapping\");\n>                         goto fail_msix_alloc;\n>                 }\n>                 ctxt.info.up_enable_bits = I40E_DEFAULT_TCMAP;\n> @@ -2731,15 +2731,15 @@ i40e_vsi_setup(struct i40e_pf *pf,\n>                  */\n>         }\n>         else {\n> -               PMD_DRV_LOG(ERR, \"VSI: Not support other type VSI yet\\n\");\n> +               PMD_DRV_LOG(ERR, \"VSI: Not support other type VSI yet\");\n>                 goto fail_msix_alloc;\n>         }\n>\n>         if (vsi->type != I40E_VSI_MAIN) {\n>                 ret = i40e_aq_add_vsi(hw, &ctxt, NULL);\n>                 if (ret) {\n> -                       PMD_DRV_LOG(ERR, \"add vsi failed, aq_err=%d\\n\",\n> -                                hw->aq.asq_last_status);\n> +                       PMD_DRV_LOG(ERR, \"add vsi failed, aq_err=%d\",\n> +                                   hw->aq.asq_last_status);\n>                         goto fail_msix_alloc;\n>                 }\n>                 memcpy(&vsi->info, &ctxt.info, sizeof(ctxt.info));\n> @@ -2754,7 +2754,7 @@ i40e_vsi_setup(struct i40e_pf *pf,\n>         /* MAC/VLAN configuration */\n>         ret = i40e_vsi_add_mac(vsi, &broadcast);\n>         if (ret != I40E_SUCCESS) {\n> -               PMD_DRV_LOG(ERR, \"Failed to add MACVLAN filter\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to add MACVLAN filter\");\n>                 goto fail_msix_alloc;\n>         }\n>\n> @@ -2806,8 +2806,8 @@ i40e_vsi_config_vlan_stripping(struct i40e_vsi *vsi,\n> bool on)\n>         (void)rte_memcpy(&ctxt.info, &vsi->info, sizeof(vsi->info));\n>         ret = i40e_aq_update_vsi_params(hw, &ctxt, NULL);\n>         if (ret)\n> -               PMD_DRV_LOG(INFO, \"Update VSI failed to %s vlan\n> stripping\\n\",\n> -                                               on ? \"enable\" : \"disable\");\n> +               PMD_DRV_LOG(INFO, \"Update VSI failed to %s vlan stripping\",\n> +                           on ? \"enable\" : \"disable\");\n>\n>         return ret;\n>  }\n> @@ -2827,7 +2827,7 @@ i40e_dev_init_vlan(struct rte_eth_dev *dev)\n>         ret = i40e_vlan_pvid_set(dev, data->dev_conf.txmode.pvid,\n>                                 data->dev_conf.txmode.hw_vlan_insert_pvid);\n>         if (ret)\n> -               PMD_DRV_LOG(INFO, \"Failed to update VSI params\\n\");\n> +               PMD_DRV_LOG(INFO, \"Failed to update VSI params\");\n>\n>         return ret;\n>  }\n> @@ -2852,13 +2852,13 @@ i40e_update_flow_control(struct i40e_hw *hw)\n>         memset(&link_status, 0, sizeof(link_status));\n>         ret = i40e_aq_get_link_info(hw, FALSE, &link_status, NULL);\n>         if (ret != I40E_SUCCESS) {\n> -               PMD_DRV_LOG(ERR, \"Failed to get link status\n> information\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to get link status information\");\n>                 goto write_reg; /* Disable flow control */\n>         }\n>\n>         an_info = hw->phy.link_info.an_info;\n>         if (!(an_info & I40E_AQ_AN_COMPLETED)) {\n> -               PMD_DRV_LOG(INFO, \"Link auto negotiation not completed\\n\");\n> +               PMD_DRV_LOG(INFO, \"Link auto negotiation not completed\");\n>                 ret = I40E_ERR_NOT_READY;\n>                 goto write_reg; /* Disable flow control */\n>         }\n> @@ -2936,7 +2936,7 @@ i40e_pf_setup(struct i40e_pf *pf)\n>         ret = i40e_set_filter_control(hw, &settings);\n>         if (ret)\n>                 PMD_INIT_LOG(WARNING, \"setup_pf_filter_control failed: %d\",\n> -                                                               ret);\n> +                            ret);\n>\n>         /* Update flow control according to the auto negotiation */\n>         i40e_update_flow_control(hw);\n> @@ -2996,8 +2996,8 @@ i40e_switch_tx_queue(struct i40e_hw *hw, uint16_t\n> q_idx, bool on)\n>         }\n>         /* Check if it is timeout */\n>         if (j >= I40E_CHK_Q_ENA_COUNT) {\n> -               PMD_DRV_LOG(ERR, \"Failed to %s tx queue[%u]\\n\",\n> -                       (on ? \"enable\" : \"disable\"), q_idx);\n> +               PMD_DRV_LOG(ERR, \"Failed to %s tx queue[%u]\",\n> +                           (on ? \"enable\" : \"disable\"), q_idx);\n>                 return I40E_ERR_TIMEOUT;\n>         }\n>\n> @@ -3075,8 +3075,8 @@ i40e_switch_rx_queue(struct i40e_hw *hw, uint16_t\n> q_idx, bool on)\n>\n>         /* Check if it is timeout */\n>         if (j >= I40E_CHK_Q_ENA_COUNT) {\n> -               PMD_DRV_LOG(ERR, \"Failed to %s rx queue[%u]\\n\",\n> -                       (on ? \"enable\" : \"disable\"), q_idx);\n> +               PMD_DRV_LOG(ERR, \"Failed to %s rx queue[%u]\",\n> +                           (on ? \"enable\" : \"disable\"), q_idx);\n>                 return I40E_ERR_TIMEOUT;\n>         }\n>\n> @@ -3119,7 +3119,7 @@ i40e_vsi_switch_queues(struct i40e_vsi *vsi, bool on)\n>                 /* enable rx queues before enabling tx queues */\n>                 ret = i40e_vsi_switch_rx_queues(vsi, on);\n>                 if (ret) {\n> -                       PMD_DRV_LOG(ERR, \"Failed to switch rx queues\\n\");\n> +                       PMD_DRV_LOG(ERR, \"Failed to switch rx queues\");\n>                         return ret;\n>                 }\n>                 ret = i40e_vsi_switch_tx_queues(vsi, on);\n> @@ -3127,7 +3127,7 @@ i40e_vsi_switch_queues(struct i40e_vsi *vsi, bool on)\n>                 /* Stop tx queues before stopping rx queues */\n>                 ret = i40e_vsi_switch_tx_queues(vsi, on);\n>                 if (ret) {\n> -                       PMD_DRV_LOG(ERR, \"Failed to switch tx queues\\n\");\n> +                       PMD_DRV_LOG(ERR, \"Failed to switch tx queues\");\n>                         return ret;\n>                 }\n>                 ret = i40e_vsi_switch_rx_queues(vsi, on);\n> @@ -3168,7 +3168,7 @@ i40e_vsi_rx_init(struct i40e_vsi *vsi)\n>                 ret = i40e_rx_queue_init(data->rx_queues[i]);\n>                 if (ret != I40E_SUCCESS) {\n>                         PMD_DRV_LOG(ERR, \"Failed to do RX queue \"\n> -                                       \"initialization\\n\");\n> +                                   \"initialization\");\n>                         break;\n>                 }\n>         }\n> @@ -3184,12 +3184,12 @@ i40e_vsi_init(struct i40e_vsi *vsi)\n>\n>         err = i40e_vsi_tx_init(vsi);\n>         if (err) {\n> -               PMD_DRV_LOG(ERR, \"Failed to do vsi TX initialization\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to do vsi TX initialization\");\n>                 return err;\n>         }\n>         err = i40e_vsi_rx_init(vsi);\n>         if (err) {\n> -               PMD_DRV_LOG(ERR, \"Failed to do vsi RX initialization\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to do vsi RX initialization\");\n>                 return err;\n>         }\n>\n> @@ -3318,14 +3318,14 @@ i40e_dev_handle_vfr_event(struct rte_eth_dev *dev)\n>                         /* Clear the event first */\n>                         I40E_WRITE_REG(hw, I40E_GLGEN_VFLRSTAT(index),\n>                                                         (0x1 << offset));\n> -                       PMD_DRV_LOG(INFO, \"VF %u reset occured\\n\",\n> abs_vf_id);\n> +                       PMD_DRV_LOG(INFO, \"VF %u reset occured\",\n> abs_vf_id);\n>                         /**\n>                          * Only notify a VF reset event occured,\n>                          * don't trigger another SW reset\n>                          */\n>                         ret = i40e_pf_host_vf_reset(&pf->vfs[i], 0);\n>                         if (ret != I40E_SUCCESS)\n> -                               PMD_DRV_LOG(ERR, \"Failed to do VF\n> reset\\n\");\n> +                               PMD_DRV_LOG(ERR, \"Failed to do VF reset\");\n>                 }\n>         }\n>  }\n> @@ -3341,7 +3341,7 @@ i40e_dev_handle_aq_msg(struct rte_eth_dev *dev)\n>         info.msg_size = I40E_AQ_BUF_SZ;\n>         info.msg_buf = rte_zmalloc(\"msg_buffer\", I40E_AQ_BUF_SZ, 0);\n>         if (!info.msg_buf) {\n> -               PMD_DRV_LOG(ERR, \"Failed to allocate mem\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to allocate mem\");\n>                 return;\n>         }\n>\n> @@ -3351,7 +3351,7 @@ i40e_dev_handle_aq_msg(struct rte_eth_dev *dev)\n>\n>                 if (ret != I40E_SUCCESS) {\n>                         PMD_DRV_LOG(INFO, \"Failed to read msg from AdminQ,\n> \"\n> -                               \"aq_err: %u\\n\", hw->aq.asq_last_status);\n> +                                   \"aq_err: %u\", hw->aq.asq_last_status);\n>                         break;\n>                 }\n>                 opcode = rte_le_to_cpu_16(info.desc.opcode);\n> @@ -3367,8 +3367,8 @@ i40e_dev_handle_aq_msg(struct rte_eth_dev *dev)\n>                                         info.msg_size);\n>                         break;\n>                 default:\n> -                       PMD_DRV_LOG(ERR, \"Request %u is not supported\n> yet\\n\",\n> -                               opcode);\n> +                       PMD_DRV_LOG(ERR, \"Request %u is not supported yet\",\n> +                                   opcode);\n>                         break;\n>                 }\n>                 /* Reset the buffer after processing one */\n> @@ -3405,38 +3405,38 @@ i40e_dev_interrupt_handler(__rte_unused struct\n> rte_intr_handle *handle,\n>         /* Shared IRQ case, return */\n>         if (!(cause & I40E_PFINT_ICR0_INTEVENT_MASK)) {\n>                 PMD_DRV_LOG(INFO, \"Port%d INT0:share IRQ case, \"\n> -                       \"no INT event to process\\n\", hw->pf_id);\n> +                           \"no INT event to process\", hw->pf_id);\n>                 goto done;\n>         }\n>\n>         if (cause & I40E_PFINT_ICR0_LINK_STAT_CHANGE_MASK) {\n> -               PMD_DRV_LOG(INFO, \"INT:Link status changed\\n\");\n> +               PMD_DRV_LOG(INFO, \"INT:Link status changed\");\n>                 i40e_dev_link_update(dev, 0);\n>         }\n>\n>         if (cause & I40E_PFINT_ICR0_ECC_ERR_MASK)\n> -               PMD_DRV_LOG(INFO, \"INT:Unrecoverable ECC Error\\n\");\n> +               PMD_DRV_LOG(INFO, \"INT:Unrecoverable ECC Error\");\n>\n>         if (cause & I40E_PFINT_ICR0_MAL_DETECT_MASK)\n> -               PMD_DRV_LOG(INFO, \"INT:Malicious programming detected\\n\");\n> +               PMD_DRV_LOG(INFO, \"INT:Malicious programming detected\");\n>\n>         if (cause & I40E_PFINT_ICR0_GRST_MASK)\n> -               PMD_DRV_LOG(INFO, \"INT:Global Resets Requested\\n\");\n> +               PMD_DRV_LOG(INFO, \"INT:Global Resets Requested\");\n>\n>         if (cause & I40E_PFINT_ICR0_PCI_EXCEPTION_MASK)\n> -               PMD_DRV_LOG(INFO, \"INT:PCI EXCEPTION occured\\n\");\n> +               PMD_DRV_LOG(INFO, \"INT:PCI EXCEPTION occured\");\n>\n>         if (cause & I40E_PFINT_ICR0_HMC_ERR_MASK)\n> -               PMD_DRV_LOG(INFO, \"INT:HMC error occured\\n\");\n> +               PMD_DRV_LOG(INFO, \"INT:HMC error occured\");\n>\n>         /* Add processing func to deal with VF reset vent */\n>         if (cause & I40E_PFINT_ICR0_VFLR_MASK) {\n> -               PMD_DRV_LOG(INFO, \"INT:VF reset detected\\n\");\n> +               PMD_DRV_LOG(INFO, \"INT:VF reset detected\");\n>                 i40e_dev_handle_vfr_event(dev);\n>         }\n>         /* Find admin queue event */\n>         if (cause & I40E_PFINT_ICR0_ADMINQ_MASK) {\n> -               PMD_DRV_LOG(INFO, \"INT:ADMINQ event\\n\");\n> +               PMD_DRV_LOG(INFO, \"INT:ADMINQ event\");\n>                 i40e_dev_handle_aq_msg(dev);\n>         }\n>\n> @@ -3466,7 +3466,7 @@ i40e_add_macvlan_filters(struct i40e_vsi *vsi,\n>\n>         req_list = rte_zmalloc(\"macvlan_add\", ele_buff_size, 0);\n>         if (req_list == NULL) {\n> -               PMD_DRV_LOG(ERR, \"Fail to allocate memory\\n\");\n> +               PMD_DRV_LOG(ERR, \"Fail to allocate memory\");\n>                 return I40E_ERR_NO_MEMORY;\n>         }\n>\n> @@ -3488,7 +3488,7 @@ i40e_add_macvlan_filters(struct i40e_vsi *vsi,\n>                 ret = i40e_aq_add_macvlan(hw, vsi->seid, req_list,\n>                                                 actual_num, NULL);\n>                 if (ret != I40E_SUCCESS) {\n> -                       PMD_DRV_LOG(ERR, \"Failed to add macvlan filter\\n\");\n> +                       PMD_DRV_LOG(ERR, \"Failed to add macvlan filter\");\n>                         goto DONE;\n>                 }\n>                 num += actual_num;\n> @@ -3518,7 +3518,7 @@ i40e_remove_macvlan_filters(struct i40e_vsi *vsi,\n>\n>         req_list = rte_zmalloc(\"macvlan_remove\", ele_buff_size, 0);\n>         if (req_list == NULL) {\n> -               PMD_DRV_LOG(ERR, \"Fail to allocate memory\\n\");\n> +               PMD_DRV_LOG(ERR, \"Fail to allocate memory\");\n>                 return I40E_ERR_NO_MEMORY;\n>         }\n>\n> @@ -3538,7 +3538,7 @@ i40e_remove_macvlan_filters(struct i40e_vsi *vsi,\n>                 ret = i40e_aq_remove_macvlan(hw, vsi->seid, req_list,\n>                                                 actual_num, NULL);\n>                 if (ret != I40E_SUCCESS) {\n> -                       PMD_DRV_LOG(ERR, \"Failed to remove macvlan\n> filter\\n\");\n> +                       PMD_DRV_LOG(ERR, \"Failed to remove macvlan\n> filter\");\n>                         goto DONE;\n>                 }\n>                 num += actual_num;\n> @@ -3626,7 +3626,7 @@ i40e_find_all_vlan_for_mac(struct i40e_vsi *vsi,\n>                                 if (vsi->vfta[j] & (1 << k)) {\n>                                         if (i > num - 1) {\n>                                                 PMD_DRV_LOG(ERR, \"vlan\n> number \"\n> -                                                               \"not\n> match\\n\");\n> +                                                           \"not match\");\n>                                                 return I40E_ERR_PARAM;\n>                                         }\n>                                         (void)rte_memcpy(&mv_f[i].macaddr,\n> @@ -3655,7 +3655,7 @@ i40e_find_all_mac_for_vlan(struct i40e_vsi *vsi,\n>\n>         TAILQ_FOREACH(f, &vsi->mac_list, next) {\n>                 if (i > num - 1) {\n> -                       PMD_DRV_LOG(ERR, \"buffer number not match\\n\");\n> +                       PMD_DRV_LOG(ERR, \"buffer number not match\");\n>                         return I40E_ERR_PARAM;\n>                 }\n>                 (void)rte_memcpy(&mv_f[i].macaddr, &f->macaddr,\n> ETH_ADDR_LEN);\n> @@ -3685,7 +3685,7 @@ i40e_vsi_remove_all_macvlan_filter(struct i40e_vsi\n> *vsi)\n>\n>         mv_f = rte_zmalloc(\"macvlan_data\", num * sizeof(*mv_f), 0);\n>         if (mv_f == NULL) {\n> -               PMD_DRV_LOG(ERR, \"failed to allocate memory\\n\");\n> +               PMD_DRV_LOG(ERR, \"failed to allocate memory\");\n>                 return I40E_ERR_NO_MEMORY;\n>         }\n>\n> @@ -3731,14 +3731,14 @@ i40e_vsi_add_vlan(struct i40e_vsi *vsi, uint16_t\n> vlan)\n>         mac_num = vsi->mac_num;\n>\n>         if (mac_num == 0) {\n> -               PMD_DRV_LOG(ERR, \"Error! VSI doesn't have a mac addr\\n\");\n> +               PMD_DRV_LOG(ERR, \"Error! VSI doesn't have a mac addr\");\n>                 return I40E_ERR_PARAM;\n>         }\n>\n>         mv_f = rte_zmalloc(\"macvlan_data\", mac_num * sizeof(*mv_f), 0);\n>\n>         if (mv_f == NULL) {\n> -               PMD_DRV_LOG(ERR, \"failed to allocate memory\\n\");\n> +               PMD_DRV_LOG(ERR, \"failed to allocate memory\");\n>                 return I40E_ERR_NO_MEMORY;\n>         }\n>\n> @@ -3782,14 +3782,14 @@ i40e_vsi_delete_vlan(struct i40e_vsi *vsi,\n> uint16_t vlan)\n>         mac_num = vsi->mac_num;\n>\n>         if (mac_num == 0) {\n> -               PMD_DRV_LOG(ERR, \"Error! VSI doesn't have a mac addr\\n\");\n> +               PMD_DRV_LOG(ERR, \"Error! VSI doesn't have a mac addr\");\n>                 return I40E_ERR_PARAM;\n>         }\n>\n>         mv_f = rte_zmalloc(\"macvlan_data\", mac_num * sizeof(*mv_f), 0);\n>\n>         if (mv_f == NULL) {\n> -               PMD_DRV_LOG(ERR, \"failed to allocate memory\\n\");\n> +               PMD_DRV_LOG(ERR, \"failed to allocate memory\");\n>                 return I40E_ERR_NO_MEMORY;\n>         }\n>\n> @@ -3849,7 +3849,7 @@ i40e_vsi_add_mac(struct i40e_vsi *vsi, struct\n> ether_addr *addr)\n>\n>         mv_f = rte_zmalloc(\"macvlan_data\", vlan_num * sizeof(*mv_f), 0);\n>         if (mv_f == NULL) {\n> -               PMD_DRV_LOG(ERR, \"failed to allocate memory\\n\");\n> +               PMD_DRV_LOG(ERR, \"failed to allocate memory\");\n>                 return I40E_ERR_NO_MEMORY;\n>         }\n>\n> @@ -3864,7 +3864,7 @@ i40e_vsi_add_mac(struct i40e_vsi *vsi, struct\n> ether_addr *addr)\n>         /* Add the mac addr into mac list */\n>         f = rte_zmalloc(\"macv_filter\", sizeof(*f), 0);\n>         if (f == NULL) {\n> -               PMD_DRV_LOG(ERR, \"failed to allocate memory\\n\");\n> +               PMD_DRV_LOG(ERR, \"failed to allocate memory\");\n>                 ret = I40E_ERR_NO_MEMORY;\n>                 goto DONE;\n>         }\n> @@ -3894,12 +3894,12 @@ i40e_vsi_delete_mac(struct i40e_vsi *vsi, struct\n> ether_addr *addr)\n>\n>         vlan_num = vsi->vlan_num;\n>         if (vlan_num == 0) {\n> -               PMD_DRV_LOG(ERR, \"VLAN number shouldn't be 0\\n\");\n> +               PMD_DRV_LOG(ERR, \"VLAN number shouldn't be 0\");\n>                 return I40E_ERR_PARAM;\n>         }\n>         mv_f = rte_zmalloc(\"macvlan_data\", vlan_num * sizeof(*mv_f), 0);\n>         if (mv_f == NULL) {\n> -               PMD_DRV_LOG(ERR, \"failed to allocate memory\\n\");\n> +               PMD_DRV_LOG(ERR, \"failed to allocate memory\");\n>                 return I40E_ERR_NO_MEMORY;\n>         }\n>\n> diff --git a/lib/librte_pmd_i40e/i40e_ethdev_vf.c\n> b/lib/librte_pmd_i40e/i40e_ethdev_vf.c\n> index 9c1ae94..8c89465 100644\n> --- a/lib/librte_pmd_i40e/i40e_ethdev_vf.c\n> +++ b/lib/librte_pmd_i40e/i40e_ethdev_vf.c\n> @@ -205,23 +205,22 @@ i40evf_parse_pfmsg(struct i40e_vf *vf,\n>                         vf->link_up =\n>                                 vpe->event_data.link_event.link_status;\n>                         vf->pend_msg |= PFMSG_LINK_CHANGE;\n> -                       PMD_DRV_LOG(INFO, \"Link status update:%s\\n\",\n> -                                       vf->link_up ? \"up\" : \"down\");\n> +                       PMD_DRV_LOG(INFO, \"Link status update:%s\",\n> +                                   vf->link_up ? \"up\" : \"down\");\n>                         break;\n>                 case I40E_VIRTCHNL_EVENT_RESET_IMPENDING:\n>                         vf->vf_reset = true;\n>                         vf->pend_msg |= PFMSG_RESET_IMPENDING;\n> -                       PMD_DRV_LOG(INFO, \"vf is reseting\\n\");\n> +                       PMD_DRV_LOG(INFO, \"vf is reseting\");\n>                         break;\n>                 case I40E_VIRTCHNL_EVENT_PF_DRIVER_CLOSE:\n>                         vf->dev_closed = true;\n>                         vf->pend_msg |= PFMSG_DRIVER_CLOSE;\n> -                       PMD_DRV_LOG(INFO, \"PF driver closed\\n\");\n> +                       PMD_DRV_LOG(INFO, \"PF driver closed\");\n>                         break;\n>                 default:\n> -                       PMD_DRV_LOG(ERR,\n> -                               \"%s: Unknown event %d from pf\\n\",\n> -                               __func__, vpe->event);\n> +                       PMD_DRV_LOG(ERR, \"%s: Unknown event %d from pf\",\n> +                                   __func__, vpe->event);\n>                 }\n>         } else {\n>                 /* async reply msg on command issued by vf previously */\n> @@ -315,7 +314,7 @@ _atomic_set_cmd(struct i40e_vf *vf, enum\n> i40e_virtchnl_ops ops)\n>                         I40E_VIRTCHNL_OP_UNKNOWN, ops);\n>\n>         if (!ret)\n> -               PMD_DRV_LOG(ERR, \"There is incomplete cmd %d\\n\",\n> vf->pend_cmd);\n> +               PMD_DRV_LOG(ERR, \"There is incomplete cmd %d\",\n> vf->pend_cmd);\n>\n>         return !ret;\n>  }\n> @@ -339,7 +338,7 @@ i40evf_execute_vf_cmd(struct rte_eth_dev *dev, struct\n> vf_cmd_info *args)\n>         err = i40e_aq_send_msg_to_pf(hw, args->ops, I40E_SUCCESS,\n>                      args->in_args, args->in_args_size, NULL);\n>         if (err) {\n> -               PMD_DRV_LOG(ERR, \"fail to send cmd %d\\n\", args->ops);\n> +               PMD_DRV_LOG(ERR, \"fail to send cmd %d\", args->ops);\n>                 return err;\n>         }\n>\n> @@ -348,10 +347,10 @@ i40evf_execute_vf_cmd(struct rte_eth_dev *dev,\n> struct vf_cmd_info *args)\n>         if (!err && args->ops == info.ops)\n>                 _clear_cmd(vf);\n>         else if (err)\n> -               PMD_DRV_LOG(ERR, \"Failed to read message from AdminQ\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to read message from AdminQ\");\n>         else if (args->ops != info.ops)\n> -               PMD_DRV_LOG(ERR, \"command mismatch, expect %u, get %u\\n\",\n> -                               args->ops, info.ops);\n> +               PMD_DRV_LOG(ERR, \"command mismatch, expect %u, get %u\",\n> +                           args->ops, info.ops);\n>\n>         return (err | info.result);\n>  }\n> @@ -378,7 +377,7 @@ i40evf_check_api_version(struct rte_eth_dev *dev)\n>\n>         err = i40evf_execute_vf_cmd(dev, &args);\n>         if (err) {\n> -               PMD_INIT_LOG(ERR, \"fail to execute command OP_VERSION\\n\");\n> +               PMD_INIT_LOG(ERR, \"fail to execute command OP_VERSION\");\n>                 return err;\n>         }\n>\n> @@ -386,14 +385,14 @@ i40evf_check_api_version(struct rte_eth_dev *dev)\n>         /* We are talking with DPDK host */\n>         if (pver->major == I40E_DPDK_VERSION_MAJOR) {\n>                 vf->host_is_dpdk = TRUE;\n> -               PMD_DRV_LOG(INFO, \"Detect PF host is DPDK app\\n\");\n> +               PMD_DRV_LOG(INFO, \"Detect PF host is DPDK app\");\n>         }\n>         /* It's linux host driver */\n>         else if ((pver->major != version.major) ||\n>             (pver->minor != version.minor)) {\n>                 PMD_INIT_LOG(ERR, \"pf/vf API version mismatch. \"\n> -                       \"(%u.%u)-(%u.%u)\\n\", pver->major, pver->minor,\n> -                                       version.major, version.minor);\n> +                            \"(%u.%u)-(%u.%u)\", pver->major, pver->minor,\n> +                            version.major, version.minor);\n>                 return -1;\n>         }\n>\n> @@ -418,8 +417,7 @@ i40evf_get_vf_resource(struct rte_eth_dev *dev)\n>         err = i40evf_execute_vf_cmd(dev, &args);\n>\n>         if (err) {\n> -               PMD_DRV_LOG(ERR, \"fail to execute command \"\n> -                                       \"OP_GET_VF_RESOURCE\\n\");\n> +               PMD_DRV_LOG(ERR, \"fail to execute command\n> OP_GET_VF_RESOURCE\");\n>                 return err;\n>         }\n>\n> @@ -462,7 +460,7 @@ i40evf_config_promisc(struct rte_eth_dev *dev,\n>\n>         if (err)\n>                 PMD_DRV_LOG(ERR, \"fail to execute command \"\n> -                               \"CONFIG_PROMISCUOUS_MODE\\n\");\n> +                           \"CONFIG_PROMISCUOUS_MODE\");\n>         return err;\n>  }\n>\n> @@ -487,7 +485,7 @@ i40evf_config_vlan_offload(struct rte_eth_dev *dev,\n>\n>         err = i40evf_execute_vf_cmd(dev, &args);\n>         if (err)\n> -               PMD_DRV_LOG(ERR, \"fail to execute command\n> CFG_VLAN_OFFLOAD\\n\");\n> +               PMD_DRV_LOG(ERR, \"fail to execute command\n> CFG_VLAN_OFFLOAD\");\n>\n>         return err;\n>  }\n> @@ -502,7 +500,7 @@ i40evf_config_vlan_pvid(struct rte_eth_dev *dev,\n>         struct i40e_virtchnl_pvid_info tpid_info;\n>\n>         if (dev == NULL || info == NULL) {\n> -               PMD_DRV_LOG(ERR, \"invalid parameters\\n\");\n> +               PMD_DRV_LOG(ERR, \"invalid parameters\");\n>                 return I40E_ERR_PARAM;\n>         }\n>\n> @@ -518,7 +516,7 @@ i40evf_config_vlan_pvid(struct rte_eth_dev *dev,\n>\n>         err = i40evf_execute_vf_cmd(dev, &args);\n>         if (err)\n> -               PMD_DRV_LOG(ERR, \"fail to execute command\n> CFG_VLAN_PVID\\n\");\n> +               PMD_DRV_LOG(ERR, \"fail to execute command CFG_VLAN_PVID\");\n>\n>         return err;\n>  }\n> @@ -542,7 +540,7 @@ i40evf_configure_queues(struct rte_eth_dev *dev)\n>         len = sizeof(*queue_info) + sizeof(*queue_cfg) * nb_qpairs;\n>         queue_info = rte_zmalloc(\"queue_info\", len, 0);\n>         if (queue_info == NULL) {\n> -               PMD_INIT_LOG(ERR, \"failed alloc memory for queue_info\\n\");\n> +               PMD_INIT_LOG(ERR, \"failed alloc memory for queue_info\");\n>                 return -1;\n>         }\n>         queue_info->vsi_id = vf->vsi_res->vsi_id;\n> @@ -595,7 +593,7 @@ i40evf_configure_queues(struct rte_eth_dev *dev)\n>         err = i40evf_execute_vf_cmd(dev, &args);\n>         if (err)\n>                 PMD_DRV_LOG(ERR, \"fail to execute command \"\n> -                               \"OP_CONFIG_VSI_QUEUES\\n\");\n> +                           \"OP_CONFIG_VSI_QUEUES\");\n>         rte_free(queue_info);\n>\n>         return err;\n> @@ -630,7 +628,7 @@ i40evf_config_irq_map(struct rte_eth_dev *dev)\n>         args.out_size = I40E_AQ_BUF_SZ;\n>         err = i40evf_execute_vf_cmd(dev, &args);\n>         if (err)\n> -               PMD_DRV_LOG(ERR, \"fail to execute command\n> OP_ENABLE_QUEUES\\n\");\n> +               PMD_DRV_LOG(ERR, \"fail to execute command\n> OP_ENABLE_QUEUES\");\n>\n>         return err;\n>  }\n> @@ -661,8 +659,8 @@ i40evf_switch_queue(struct rte_eth_dev *dev, bool\n> isrx, uint16_t qid,\n>         args.out_size = I40E_AQ_BUF_SZ;\n>         err = i40evf_execute_vf_cmd(dev, &args);\n>         if (err)\n> -               PMD_DRV_LOG(ERR, \"fail to switch %s %u %s\\n\", isrx ? \"RX\"\n> : \"TX\",\n> -                       qid, on ? \"on\" : \"off\");\n> +               PMD_DRV_LOG(ERR, \"fail to switch %s %u %s\", isrx ? \"RX\" :\n> \"TX\",\n> +                           qid, on ? \"on\" : \"off\");\n>\n>         return err;\n>  }\n> @@ -680,8 +678,7 @@ i40evf_start_queues(struct rte_eth_dev *dev)\n>                 if (rxq->start_rx_per_q)\n>                         continue;\n>                 if (i40evf_dev_rx_queue_start(dev, i) != 0) {\n> -                       PMD_DRV_LOG(ERR, \"Fail to start queue %u\\n\",\n> -                               i);\n> +                       PMD_DRV_LOG(ERR, \"Fail to start queue %u\", i);\n>                         return -1;\n>                 }\n>         }\n> @@ -691,8 +688,7 @@ i40evf_start_queues(struct rte_eth_dev *dev)\n>                 if (txq->start_tx_per_q)\n>                         continue;\n>                 if (i40evf_dev_tx_queue_start(dev, i) != 0) {\n> -                       PMD_DRV_LOG(ERR, \"Fail to start queue %u\\n\",\n> -                               i);\n> +                       PMD_DRV_LOG(ERR, \"Fail to start queue %u\", i);\n>                         return -1;\n>                 }\n>         }\n> @@ -708,8 +704,7 @@ i40evf_stop_queues(struct rte_eth_dev *dev)\n>         /* Stop TX queues first */\n>         for (i = 0; i < dev->data->nb_tx_queues; i++) {\n>                 if (i40evf_dev_tx_queue_stop(dev, i) != 0) {\n> -                       PMD_DRV_LOG(ERR, \"Fail to start queue %u\\n\",\n> -                               i);\n> +                       PMD_DRV_LOG(ERR, \"Fail to start queue %u\", i);\n>                         return -1;\n>                 }\n>         }\n> @@ -717,8 +712,7 @@ i40evf_stop_queues(struct rte_eth_dev *dev)\n>         /* Then stop RX queues */\n>         for (i = 0; i < dev->data->nb_rx_queues; i++) {\n>                 if (i40evf_dev_rx_queue_stop(dev, i) != 0) {\n> -                       PMD_DRV_LOG(ERR, \"Fail to start queue %u\\n\",\n> -                               i);\n> +                       PMD_DRV_LOG(ERR, \"Fail to start queue %u\", i);\n>                         return -1;\n>                 }\n>         }\n> @@ -737,10 +731,10 @@ i40evf_add_mac_addr(struct rte_eth_dev *dev, struct\n> ether_addr *addr)\n>         struct vf_cmd_info args;\n>\n>         if (i40e_validate_mac_addr(addr->addr_bytes) != I40E_SUCCESS) {\n> -               PMD_DRV_LOG(ERR, \"Invalid mac:%x:%x:%x:%x:%x:%x\\n\",\n> -                       addr->addr_bytes[0], addr->addr_bytes[1],\n> -                       addr->addr_bytes[2], addr->addr_bytes[3],\n> -                       addr->addr_bytes[4], addr->addr_bytes[5]);\n> +               PMD_DRV_LOG(ERR, \"Invalid mac:%x:%x:%x:%x:%x:%x\",\n> +                           addr->addr_bytes[0], addr->addr_bytes[1],\n> +                           addr->addr_bytes[2], addr->addr_bytes[3],\n> +                           addr->addr_bytes[4], addr->addr_bytes[5]);\n>                 return -1;\n>         }\n>\n> @@ -758,7 +752,7 @@ i40evf_add_mac_addr(struct rte_eth_dev *dev, struct\n> ether_addr *addr)\n>         err = i40evf_execute_vf_cmd(dev, &args);\n>         if (err)\n>                 PMD_DRV_LOG(ERR, \"fail to execute command \"\n> -                               \"OP_ADD_ETHER_ADDRESS\\n\");\n> +                           \"OP_ADD_ETHER_ADDRESS\");\n>\n>         return err;\n>  }\n> @@ -774,10 +768,10 @@ i40evf_del_mac_addr(struct rte_eth_dev *dev, struct\n> ether_addr *addr)\n>         struct vf_cmd_info args;\n>\n>         if (i40e_validate_mac_addr(addr->addr_bytes) != I40E_SUCCESS) {\n> -               PMD_DRV_LOG(ERR, \"Invalid mac:%x-%x-%x-%x-%x-%x\\n\",\n> -                       addr->addr_bytes[0], addr->addr_bytes[1],\n> -                       addr->addr_bytes[2], addr->addr_bytes[3],\n> -                       addr->addr_bytes[4], addr->addr_bytes[5]);\n> +               PMD_DRV_LOG(ERR, \"Invalid mac:%x-%x-%x-%x-%x-%x\",\n> +                           addr->addr_bytes[0], addr->addr_bytes[1],\n> +                           addr->addr_bytes[2], addr->addr_bytes[3],\n> +                           addr->addr_bytes[4], addr->addr_bytes[5]);\n>                 return -1;\n>         }\n>\n> @@ -795,7 +789,7 @@ i40evf_del_mac_addr(struct rte_eth_dev *dev, struct\n> ether_addr *addr)\n>         err = i40evf_execute_vf_cmd(dev, &args);\n>         if (err)\n>                 PMD_DRV_LOG(ERR, \"fail to execute command \"\n> -                               \"OP_DEL_ETHER_ADDRESS\\n\");\n> +                           \"OP_DEL_ETHER_ADDRESS\");\n>\n>         return err;\n>  }\n> @@ -819,7 +813,7 @@ i40evf_get_statics(struct rte_eth_dev *dev, struct\n> rte_eth_stats *stats)\n>\n>         err = i40evf_execute_vf_cmd(dev, &args);\n>         if (err) {\n> -               PMD_DRV_LOG(ERR, \"fail to execute command OP_GET_STATS\\n\");\n> +               PMD_DRV_LOG(ERR, \"fail to execute command OP_GET_STATS\");\n>                 return err;\n>         }\n>         pstats = (struct i40e_eth_stats *)args.out_buffer;\n> @@ -857,7 +851,7 @@ i40evf_add_vlan(struct rte_eth_dev *dev, uint16_t\n> vlanid)\n>         args.out_size = I40E_AQ_BUF_SZ;\n>         err = i40evf_execute_vf_cmd(dev, &args);\n>         if (err)\n> -               PMD_DRV_LOG(ERR, \"fail to execute command OP_ADD_VLAN\\n\");\n> +               PMD_DRV_LOG(ERR, \"fail to execute command OP_ADD_VLAN\");\n>\n>         return err;\n>  }\n> @@ -884,7 +878,7 @@ i40evf_del_vlan(struct rte_eth_dev *dev, uint16_t\n> vlanid)\n>         args.out_size = I40E_AQ_BUF_SZ;\n>         err = i40evf_execute_vf_cmd(dev, &args);\n>         if (err)\n> -               PMD_DRV_LOG(ERR, \"fail to execute command OP_DEL_VLAN\\n\");\n> +               PMD_DRV_LOG(ERR, \"fail to execute command OP_DEL_VLAN\");\n>\n>         return err;\n>  }\n> @@ -903,7 +897,7 @@ i40evf_get_link_status(struct rte_eth_dev *dev, struct\n> rte_eth_link *link)\n>         args.out_size = I40E_AQ_BUF_SZ;\n>         err = i40evf_execute_vf_cmd(dev, &args);\n>         if (err) {\n> -               PMD_DRV_LOG(ERR, \"fail to execute command\n> OP_GET_LINK_STAT\\n\");\n> +               PMD_DRV_LOG(ERR, \"fail to execute command\n> OP_GET_LINK_STAT\");\n>                 return err;\n>         }\n>\n> @@ -939,7 +933,7 @@ i40evf_reset_vf(struct i40e_hw *hw)\n>         int i, reset;\n>\n>         if (i40e_vf_reset(hw) != I40E_SUCCESS) {\n> -               PMD_INIT_LOG(ERR, \"Reset VF NIC failed\\n\");\n> +               PMD_INIT_LOG(ERR, \"Reset VF NIC failed\");\n>                 return -1;\n>         }\n>         /**\n> @@ -964,7 +958,7 @@ i40evf_reset_vf(struct i40e_hw *hw)\n>         }\n>\n>         if (i >= MAX_RESET_WAIT_CNT) {\n> -               PMD_INIT_LOG(ERR, \"Reset VF NIC failed\\n\");\n> +               PMD_INIT_LOG(ERR, \"Reset VF NIC failed\");\n>                 return -1;\n>         }\n>\n> @@ -980,49 +974,49 @@ i40evf_init_vf(struct rte_eth_dev *dev)\n>\n>         err = i40evf_set_mac_type(hw);\n>         if (err) {\n> -               PMD_INIT_LOG(ERR, \"set_mac_type failed: %d\\n\", err);\n> +               PMD_INIT_LOG(ERR, \"set_mac_type failed: %d\", err);\n>                 goto err;\n>         }\n>\n>         i40e_init_adminq_parameter(hw);\n>         err = i40e_init_adminq(hw);\n>         if (err) {\n> -               PMD_INIT_LOG(ERR, \"init_adminq failed: %d\\n\", err);\n> +               PMD_INIT_LOG(ERR, \"init_adminq failed: %d\", err);\n>                 goto err;\n>         }\n>\n>\n>         /* Reset VF and wait until it's complete */\n>         if (i40evf_reset_vf(hw)) {\n> -               PMD_INIT_LOG(ERR, \"reset NIC failed\\n\");\n> +               PMD_INIT_LOG(ERR, \"reset NIC failed\");\n>                 goto err_aq;\n>         }\n>\n>         /* VF reset, shutdown admin queue and initialize again */\n>         if (i40e_shutdown_adminq(hw) != I40E_SUCCESS) {\n> -               PMD_INIT_LOG(ERR, \"i40e_shutdown_adminq failed\\n\");\n> +               PMD_INIT_LOG(ERR, \"i40e_shutdown_adminq failed\");\n>                 return -1;\n>         }\n>\n>         i40e_init_adminq_parameter(hw);\n>         if (i40e_init_adminq(hw) != I40E_SUCCESS) {\n> -               PMD_INIT_LOG(ERR, \"init_adminq failed\\n\");\n> +               PMD_INIT_LOG(ERR, \"init_adminq failed\");\n>                 return -1;\n>         }\n>         if (i40evf_check_api_version(dev) != 0) {\n> -               PMD_INIT_LOG(ERR, \"check_api version failed\\n\");\n> +               PMD_INIT_LOG(ERR, \"check_api version failed\");\n>                 goto err_aq;\n>         }\n>         bufsz = sizeof(struct i40e_virtchnl_vf_resource) +\n>                 (I40E_MAX_VF_VSI * sizeof(struct\n> i40e_virtchnl_vsi_resource));\n>         vf->vf_res = rte_zmalloc(\"vf_res\", bufsz, 0);\n>         if (!vf->vf_res) {\n> -               PMD_INIT_LOG(ERR, \"unable to allocate vf_res memory\\n\");\n> +               PMD_INIT_LOG(ERR, \"unable to allocate vf_res memory\");\n>                         goto err_aq;\n>         }\n>\n>         if (i40evf_get_vf_resource(dev) != 0) {\n> -               PMD_INIT_LOG(ERR, \"i40evf_get_vf_config failed\\n\");\n> +               PMD_INIT_LOG(ERR, \"i40evf_get_vf_config failed\");\n>                 goto err_alloc;\n>         }\n>\n> @@ -1033,7 +1027,7 @@ i40evf_init_vf(struct rte_eth_dev *dev)\n>         }\n>\n>         if (!vf->vsi_res) {\n> -               PMD_INIT_LOG(ERR, \"no LAN VSI found\\n\");\n> +               PMD_INIT_LOG(ERR, \"no LAN VSI found\");\n>                 goto err_alloc;\n>         }\n>\n> @@ -1092,7 +1086,7 @@ i40evf_dev_init(__rte_unused struct eth_driver\n> *eth_drv,\n>         hw->hw_addr = (void *)eth_dev->pci_dev->mem_resource[0].addr;\n>\n>         if(i40evf_init_vf(eth_dev) != 0) {\n> -               PMD_INIT_LOG(ERR, \"Init vf failed\\n\");\n> +               PMD_INIT_LOG(ERR, \"Init vf failed\");\n>                 return -1;\n>         }\n>\n> @@ -1101,7 +1095,7 @@ i40evf_dev_init(__rte_unused struct eth_driver\n> *eth_drv,\n>                                         ETHER_ADDR_LEN, 0);\n>         if (eth_dev->data->mac_addrs == NULL) {\n>                 PMD_INIT_LOG(ERR, \"Failed to allocate %d bytes needed to \"\n> -                               \"store MAC addresses\", ETHER_ADDR_LEN);\n> +                            \"store MAC addresses\", ETHER_ADDR_LEN);\n>                 return -ENOMEM;\n>         }\n>         ether_addr_copy((struct ether_addr *)hw->mac.addr,\n> @@ -1229,7 +1223,7 @@ i40evf_dev_rx_queue_start(struct rte_eth_dev *dev,\n> uint16_t rx_queue_id)\n>\n>                 err = i40e_alloc_rx_queue_mbufs(rxq);\n>                 if (err) {\n> -                       PMD_DRV_LOG(ERR, \"Failed to allocate RX queue\n> mbuf\\n\");\n> +                       PMD_DRV_LOG(ERR, \"Failed to allocate RX queue\n> mbuf\");\n>                         return err;\n>                 }\n>\n> @@ -1243,8 +1237,8 @@ i40evf_dev_rx_queue_start(struct rte_eth_dev *dev,\n> uint16_t rx_queue_id)\n>                 err = i40evf_switch_queue(dev, TRUE, rx_queue_id, TRUE);\n>\n>                 if (err)\n> -                       PMD_DRV_LOG(ERR, \"Failed to switch RX queue %u\n> on\\n\",\n> -                               rx_queue_id);\n> +                       PMD_DRV_LOG(ERR, \"Failed to switch RX queue %u on\",\n> +                                   rx_queue_id);\n>         }\n>\n>         return err;\n> @@ -1262,8 +1256,8 @@ i40evf_dev_rx_queue_stop(struct rte_eth_dev *dev,\n> uint16_t rx_queue_id)\n>                 err = i40evf_switch_queue(dev, TRUE, rx_queue_id, FALSE);\n>\n>                 if (err) {\n> -                       PMD_DRV_LOG(ERR, \"Failed to switch RX queue %u\n> off\\n\",\n> -                               rx_queue_id);\n> +                       PMD_DRV_LOG(ERR, \"Failed to switch RX queue %u\n> off\",\n> +                                   rx_queue_id);\n>                         return err;\n>                 }\n>\n> @@ -1287,8 +1281,8 @@ i40evf_dev_tx_queue_start(struct rte_eth_dev *dev,\n> uint16_t tx_queue_id)\n>                 err = i40evf_switch_queue(dev, FALSE, tx_queue_id, TRUE);\n>\n>                 if (err)\n> -                       PMD_DRV_LOG(ERR, \"Failed to switch TX queue %u\n> on\\n\",\n> -                               tx_queue_id);\n> +                       PMD_DRV_LOG(ERR, \"Failed to switch TX queue %u on\",\n> +                                   tx_queue_id);\n>         }\n>\n>         return err;\n> @@ -1306,8 +1300,8 @@ i40evf_dev_tx_queue_stop(struct rte_eth_dev *dev,\n> uint16_t tx_queue_id)\n>                 err = i40evf_switch_queue(dev, FALSE, tx_queue_id, FALSE);\n>\n>                 if (err) {\n> -                       PMD_DRV_LOG(ERR, \"Failed to switch TX queue %u\n> of\\n\",\n> -                               tx_queue_id);\n> +                       PMD_DRV_LOG(ERR, \"Failed to switch TX queue %u of\",\n> +                                   tx_queue_id);\n>                         return err;\n>                 }\n>\n> @@ -1384,27 +1378,27 @@ i40evf_dev_start(struct rte_eth_dev *dev)\n>         struct i40e_hw *hw =\n> I40E_DEV_PRIVATE_TO_HW(dev->data->dev_private);\n>         struct ether_addr mac_addr;\n>\n> -       PMD_DRV_LOG(DEBUG, \"i40evf_dev_start\");\n> +       PMD_INIT_FUNC_TRACE();\n>\n>         vf->max_pkt_len = dev->data->dev_conf.rxmode.max_rx_pkt_len;\n>         if (dev->data->dev_conf.rxmode.jumbo_frame == 1) {\n>                 if (vf->max_pkt_len <= ETHER_MAX_LEN ||\n>                         vf->max_pkt_len > I40E_FRAME_SIZE_MAX) {\n>                         PMD_DRV_LOG(ERR, \"maximum packet length must \"\n> -                               \"be larger than %u and smaller than %u,\"\n> -                                       \"as jumbo frame is enabled\\n\",\n> -                                               (uint32_t)ETHER_MAX_LEN,\n> -                                       (uint32_t)I40E_FRAME_SIZE_MAX);\n> +                                   \"be larger than %u and smaller than\n> %u,\"\n> +                                   \"as jumbo frame is enabled\",\n> +                                   (uint32_t)ETHER_MAX_LEN,\n> +                                   (uint32_t)I40E_FRAME_SIZE_MAX);\n>                         return I40E_ERR_CONFIG;\n>                 }\n>         } else {\n>                 if (vf->max_pkt_len < ETHER_MIN_LEN ||\n>                         vf->max_pkt_len > ETHER_MAX_LEN) {\n>                         PMD_DRV_LOG(ERR, \"maximum packet length must be \"\n> -                                       \"larger than %u and smaller than\n> %u, \"\n> -                                       \"as jumbo frame is disabled\\n\",\n> -                                               (uint32_t)ETHER_MIN_LEN,\n> -                                               (uint32_t)ETHER_MAX_LEN);\n> +                                   \"larger than %u and smaller than %u, \"\n> +                                   \"as jumbo frame is disabled\",\n> +                                   (uint32_t)ETHER_MIN_LEN,\n> +                                   (uint32_t)ETHER_MAX_LEN);\n>                         return I40E_ERR_CONFIG;\n>                 }\n>         }\n> @@ -1413,18 +1407,18 @@ i40evf_dev_start(struct rte_eth_dev *dev)\n>                                         dev->data->nb_tx_queues);\n>\n>         if (i40evf_rx_init(dev) != 0){\n> -               PMD_DRV_LOG(ERR, \"failed to do RX init\\n\");\n> +               PMD_DRV_LOG(ERR, \"failed to do RX init\");\n>                 return -1;\n>         }\n>\n>         i40evf_tx_init(dev);\n>\n>         if (i40evf_configure_queues(dev) != 0) {\n> -               PMD_DRV_LOG(ERR, \"configure queues failed\\n\");\n> +               PMD_DRV_LOG(ERR, \"configure queues failed\");\n>                 goto err_queue;\n>         }\n>         if (i40evf_config_irq_map(dev)) {\n> -               PMD_DRV_LOG(ERR, \"config_irq_map failed\\n\");\n> +               PMD_DRV_LOG(ERR, \"config_irq_map failed\");\n>                 goto err_queue;\n>         }\n>\n> @@ -1432,12 +1426,12 @@ i40evf_dev_start(struct rte_eth_dev *dev)\n>         (void)rte_memcpy(mac_addr.addr_bytes, hw->mac.addr,\n>                                 sizeof(mac_addr.addr_bytes));\n>         if (i40evf_add_mac_addr(dev, &mac_addr)) {\n> -               PMD_DRV_LOG(ERR, \"Failed to add mac addr\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to add mac addr\");\n>                 goto err_queue;\n>         }\n>\n>         if (i40evf_start_queues(dev) != 0) {\n> -               PMD_DRV_LOG(ERR, \"enable queues failed\\n\");\n> +               PMD_DRV_LOG(ERR, \"enable queues failed\");\n>                 goto err_mac;\n>         }\n>\n> @@ -1561,7 +1555,7 @@ i40evf_dev_stats_get(struct rte_eth_dev *dev, struct\n> rte_eth_stats *stats)\n>  {\n>         memset(stats, 0, sizeof(*stats));\n>         if (i40evf_get_statics(dev, stats))\n> -               PMD_DRV_LOG(ERR, \"Get statics failed\\n\");\n> +               PMD_DRV_LOG(ERR, \"Get statics failed\");\n>  }\n>\n>  static void\n> diff --git a/lib/librte_pmd_i40e/i40e_pf.c b/lib/librte_pmd_i40e/i40e_pf.c\n> index eec291f..682ff44 100644\n> --- a/lib/librte_pmd_i40e/i40e_pf.c\n> +++ b/lib/librte_pmd_i40e/i40e_pf.c\n> @@ -158,7 +158,7 @@ i40e_pf_host_vf_reset(struct i40e_pf_vf *vf, bool\n> do_hw_reset)\n>         }\n>\n>         if (i >= VFRESET_MAX_WAIT_CNT) {\n> -               PMD_DRV_LOG(ERR, \"VF reset timeout\\n\");\n> +               PMD_DRV_LOG(ERR, \"VF reset timeout\");\n>                 return -ETIMEDOUT;\n>         }\n>\n> @@ -171,7 +171,7 @@ i40e_pf_host_vf_reset(struct i40e_pf_vf *vf, bool\n> do_hw_reset)\n>                 qsel.tx_queues = qsel.rx_queues;\n>                 ret = i40e_pf_host_switch_queues(vf, &qsel, false);\n>                 if (ret != I40E_SUCCESS) {\n> -                       PMD_DRV_LOG(ERR, \"Disable VF queues failed\\n\");\n> +                       PMD_DRV_LOG(ERR, \"Disable VF queues failed\");\n>                         return -EFAULT;\n>                 }\n>\n> @@ -190,7 +190,7 @@ i40e_pf_host_vf_reset(struct i40e_pf_vf *vf, bool\n> do_hw_reset)\n>                 /* remove VSI */\n>                 ret = i40e_vsi_release(vf->vsi);\n>                 if (ret != I40E_SUCCESS) {\n> -                       PMD_DRV_LOG(ERR, \"Release VSI failed\\n\");\n> +                       PMD_DRV_LOG(ERR, \"Release VSI failed\");\n>                         return -EFAULT;\n>                 }\n>         }\n> @@ -209,7 +209,7 @@ i40e_pf_host_vf_reset(struct i40e_pf_vf *vf, bool\n> do_hw_reset)\n>         }\n>\n>         if (i >= VFRESET_MAX_WAIT_CNT) {\n> -               PMD_DRV_LOG(ERR, \"Wait VF PCI transaction end timeout\\n\");\n> +               PMD_DRV_LOG(ERR, \"Wait VF PCI transaction end timeout\");\n>                 return -ETIMEDOUT;\n>         }\n>\n> @@ -225,13 +225,13 @@ i40e_pf_host_vf_reset(struct i40e_pf_vf *vf, bool\n> do_hw_reset)\n>         vf->vsi = i40e_vsi_setup(vf->pf, I40E_VSI_SRIOV,\n>                         vf->pf->main_vsi, vf->vf_idx);\n>         if (vf->vsi == NULL) {\n> -               PMD_DRV_LOG(ERR, \"Add vsi failed\\n\");\n> +               PMD_DRV_LOG(ERR, \"Add vsi failed\");\n>                 return -EFAULT;\n>         }\n>\n>         ret = i40e_pf_vf_queues_mapping(vf);\n>         if (ret != I40E_SUCCESS) {\n> -               PMD_DRV_LOG(ERR, \"queue mapping error\\n\");\n> +               PMD_DRV_LOG(ERR, \"queue mapping error\");\n>                 i40e_vsi_release(vf->vsi);\n>                 return -EFAULT;\n>         }\n> @@ -253,7 +253,7 @@ i40e_pf_host_send_msg_to_vf(struct i40e_pf_vf *vf,\n>         ret = i40e_aq_send_msg_to_vf(hw, abs_vf_id, opcode, retval,\n>                                                 msg, msglen, NULL);\n>         if (ret) {\n> -               PMD_INIT_LOG(ERR, \"Fail to send message to VF, err %u\\n\",\n> +               PMD_INIT_LOG(ERR, \"Fail to send message to VF, err %u\",\n>                              hw->aq.asq_last_status);\n>         }\n>\n> @@ -295,7 +295,7 @@ i40e_pf_host_process_cmd_get_vf_resource(struct\n> i40e_pf_vf *vf)\n>\n>         vf_res = rte_zmalloc(\"i40e_vf_res\", len, 0);\n>         if (vf_res == NULL) {\n> -               PMD_DRV_LOG(ERR, \"failed to allocate mem\\n\");\n> +               PMD_DRV_LOG(ERR, \"failed to allocate mem\");\n>                 ret = I40E_ERR_NO_MEMORY;\n>                 vf_res = NULL;\n>                 len = 0;\n> @@ -421,7 +421,7 @@ i40e_pf_host_process_cmd_config_vsi_queues(struct\n> i40e_pf_vf *vf,\n>\n>         if (msg == NULL || msglen <= sizeof(*qconfig) ||\n>                 qconfig->num_queue_pairs > vsi->nb_qps) {\n> -               PMD_DRV_LOG(ERR, \"vsi_queue_config_info argument wrong\\n\");\n> +               PMD_DRV_LOG(ERR, \"vsi_queue_config_info argument wrong\");\n>                 ret = I40E_ERR_PARAM;\n>                 goto send_msg;\n>         }\n> @@ -466,20 +466,20 @@ i40e_pf_host_process_cmd_config_irq_map(struct\n> i40e_pf_vf *vf,\n>             (struct i40e_virtchnl_irq_map_info *)msg;\n>\n>         if (msg == NULL || msglen < sizeof(struct\n> i40e_virtchnl_irq_map_info)) {\n> -               PMD_DRV_LOG(ERR, \"buffer too short\\n\");\n> +               PMD_DRV_LOG(ERR, \"buffer too short\");\n>                 ret = I40E_ERR_PARAM;\n>                 goto send_msg;\n>         }\n>\n>         /* Assume VF only have 1 vector to bind all queues */\n>         if (irqmap->num_vectors != 1) {\n> -               PMD_DRV_LOG(ERR, \"DKDK host only support 1 vector\\n\");\n> +               PMD_DRV_LOG(ERR, \"DKDK host only support 1 vector\");\n>                 ret = I40E_ERR_PARAM;\n>                 goto send_msg;\n>         }\n>\n>         if (irqmap->vecmap[0].vector_id == 0) {\n> -               PMD_DRV_LOG(ERR, \"DPDK host don't support use IRQ0\\n\");\n> +               PMD_DRV_LOG(ERR, \"DPDK host don't support use IRQ0\");\n>                 ret = I40E_ERR_PARAM;\n>                 goto send_msg;\n>         }\n> @@ -601,7 +601,7 @@ i40e_pf_host_process_cmd_add_ether_address(struct\n> i40e_pf_vf *vf,\n>         struct ether_addr *mac;\n>\n>         if (msg == NULL || msglen <= sizeof(*addr_list)) {\n> -               PMD_DRV_LOG(ERR, \"add_ether_address argument too short\\n\");\n> +               PMD_DRV_LOG(ERR, \"add_ether_address argument too short\");\n>                 ret = I40E_ERR_PARAM;\n>                 goto send_msg;\n>         }\n> @@ -634,7 +634,7 @@ i40e_pf_host_process_cmd_del_ether_address(struct\n> i40e_pf_vf *vf,\n>         struct ether_addr *mac;\n>\n>         if (msg == NULL || msglen <= sizeof(*addr_list)) {\n> -               PMD_DRV_LOG(ERR, \"delete_ether_address argument too\n> short\\n\");\n> +               PMD_DRV_LOG(ERR, \"delete_ether_address argument too\n> short\");\n>                 ret = I40E_ERR_PARAM;\n>                 goto send_msg;\n>         }\n> @@ -667,7 +667,7 @@ i40e_pf_host_process_cmd_add_vlan(struct i40e_pf_vf\n> *vf,\n>         uint16_t *vid;\n>\n>         if (msg == NULL || msglen <= sizeof(*vlan_filter_list)) {\n> -               PMD_DRV_LOG(ERR, \"add_vlan argument too short\\n\");\n> +               PMD_DRV_LOG(ERR, \"add_vlan argument too short\");\n>                 ret = I40E_ERR_PARAM;\n>                 goto send_msg;\n>         }\n> @@ -699,7 +699,7 @@ i40e_pf_host_process_cmd_del_vlan(struct i40e_pf_vf\n> *vf,\n>         uint16_t *vid;\n>\n>         if (msg == NULL || msglen <= sizeof(*vlan_filter_list)) {\n> -               PMD_DRV_LOG(ERR, \"delete_vlan argument too short\\n\");\n> +               PMD_DRV_LOG(ERR, \"delete_vlan argument too short\");\n>                 ret = I40E_ERR_PARAM;\n>                 goto send_msg;\n>         }\n> @@ -796,7 +796,7 @@ i40e_pf_host_process_cmd_cfg_vlan_offload(\n>         ret = i40e_vsi_config_vlan_stripping(vf->vsi,\n>\n> !!offload->enable_vlan_strip);\n>         if (ret != 0)\n> -               PMD_DRV_LOG(ERR, \"Failed to configure vlan stripping\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to configure vlan stripping\");\n>\n>  send_msg:\n>         i40e_pf_host_send_msg_to_vf(vf, I40E_VIRTCHNL_OP_CFG_VLAN_OFFLOAD,\n> @@ -842,13 +842,13 @@ i40e_pf_host_handle_vf_msg(struct rte_eth_dev *dev,\n>         uint16_t vf_id = abs_vf_id - hw->func_caps.vf_base_id;\n>\n>         if (!dev || vf_id > pf->vf_num - 1 || !pf->vfs) {\n> -               PMD_DRV_LOG(ERR, \"invalid argument\\n\");\n> +               PMD_DRV_LOG(ERR, \"invalid argument\");\n>                 return;\n>         }\n>\n>         vf = &pf->vfs[vf_id];\n>         if (!vf->vsi) {\n> -               PMD_DRV_LOG(ERR, \"NO VSI associated with VF found\\n\");\n> +               PMD_DRV_LOG(ERR, \"NO VSI associated with VF found\");\n>                 i40e_pf_host_send_msg_to_vf(vf, opcode,\n>                         I40E_ERR_NO_AVAILABLE_VSI, NULL, 0);\n>                 return;\n> @@ -856,82 +856,81 @@ i40e_pf_host_handle_vf_msg(struct rte_eth_dev *dev,\n>\n>         switch (opcode) {\n>         case I40E_VIRTCHNL_OP_VERSION :\n> -               PMD_DRV_LOG(INFO, \"OP_VERSION received\\n\");\n> +               PMD_DRV_LOG(INFO, \"OP_VERSION received\");\n>                 i40e_pf_host_process_cmd_version(vf);\n>                 break;\n>         case I40E_VIRTCHNL_OP_RESET_VF :\n> -               PMD_DRV_LOG(INFO, \"OP_RESET_VF received\\n\");\n> +               PMD_DRV_LOG(INFO, \"OP_RESET_VF received\");\n>                 i40e_pf_host_process_cmd_reset_vf(vf);\n>                 break;\n>         case I40E_VIRTCHNL_OP_GET_VF_RESOURCES:\n> -               PMD_DRV_LOG(INFO, \"OP_GET_VF_RESOURCES received\\n\");\n> +               PMD_DRV_LOG(INFO, \"OP_GET_VF_RESOURCES received\");\n>                 i40e_pf_host_process_cmd_get_vf_resource(vf);\n>                 break;\n>         case I40E_VIRTCHNL_OP_CONFIG_VSI_QUEUES:\n> -               PMD_DRV_LOG(INFO, \"OP_CONFIG_VSI_QUEUES received\\n\");\n> +               PMD_DRV_LOG(INFO, \"OP_CONFIG_VSI_QUEUES received\");\n>                 i40e_pf_host_process_cmd_config_vsi_queues(vf,\n>                                                 msg, msglen);\n>                 break;\n>         case I40E_VIRTCHNL_OP_CONFIG_IRQ_MAP:\n> -               PMD_DRV_LOG(INFO, \"OP_CONFIG_IRQ_MAP received\\n\");\n> +               PMD_DRV_LOG(INFO, \"OP_CONFIG_IRQ_MAP received\");\n>                 i40e_pf_host_process_cmd_config_irq_map(vf, msg, msglen);\n>                 break;\n>         case I40E_VIRTCHNL_OP_ENABLE_QUEUES:\n> -               PMD_DRV_LOG(INFO, \"OP_ENABLE_QUEUES received\\n\");\n> +               PMD_DRV_LOG(INFO, \"OP_ENABLE_QUEUES received\");\n>                 i40e_pf_host_process_cmd_enable_queues(vf,\n>                                                 msg, msglen);\n>                 break;\n>         case I40E_VIRTCHNL_OP_DISABLE_QUEUES:\n> -               PMD_DRV_LOG(INFO, \"OP_DISABLE_QUEUE received\\n\");\n> +               PMD_DRV_LOG(INFO, \"OP_DISABLE_QUEUE received\");\n>                 i40e_pf_host_process_cmd_disable_queues(vf,\n>                                                 msg, msglen);\n>                 break;\n>         case I40E_VIRTCHNL_OP_ADD_ETHER_ADDRESS:\n> -               PMD_DRV_LOG(INFO, \"OP_ADD_ETHER_ADDRESS received\\n\");\n> +               PMD_DRV_LOG(INFO, \"OP_ADD_ETHER_ADDRESS received\");\n>                 i40e_pf_host_process_cmd_add_ether_address(vf,\n>                                                 msg, msglen);\n>                 break;\n>         case I40E_VIRTCHNL_OP_DEL_ETHER_ADDRESS:\n> -               PMD_DRV_LOG(INFO, \"OP_DEL_ETHER_ADDRESS received\\n\");\n> +               PMD_DRV_LOG(INFO, \"OP_DEL_ETHER_ADDRESS received\");\n>                 i40e_pf_host_process_cmd_del_ether_address(vf,\n>                                                 msg, msglen);\n>                 break;\n>         case I40E_VIRTCHNL_OP_ADD_VLAN:\n> -               PMD_DRV_LOG(INFO, \"OP_ADD_VLAN received\\n\");\n> +               PMD_DRV_LOG(INFO, \"OP_ADD_VLAN received\");\n>                 i40e_pf_host_process_cmd_add_vlan(vf, msg, msglen);\n>                 break;\n>         case I40E_VIRTCHNL_OP_DEL_VLAN:\n> -               PMD_DRV_LOG(INFO, \"OP_DEL_VLAN received\\n\");\n> +               PMD_DRV_LOG(INFO, \"OP_DEL_VLAN received\");\n>                 i40e_pf_host_process_cmd_del_vlan(vf, msg, msglen);\n>                 break;\n>         case I40E_VIRTCHNL_OP_CONFIG_PROMISCUOUS_MODE:\n> -               PMD_DRV_LOG(INFO, \"OP_CONFIG_PROMISCUOUS_MODE received\\n\");\n> +               PMD_DRV_LOG(INFO, \"OP_CONFIG_PROMISCUOUS_MODE received\");\n>                 i40e_pf_host_process_cmd_config_promisc_mode(vf, msg,\n> msglen);\n>                 break;\n>         case I40E_VIRTCHNL_OP_GET_STATS:\n> -               PMD_DRV_LOG(INFO, \"OP_GET_STATS received\\n\");\n> +               PMD_DRV_LOG(INFO, \"OP_GET_STATS received\");\n>                 i40e_pf_host_process_cmd_get_stats(vf);\n>                 break;\n>         case I40E_VIRTCHNL_OP_GET_LINK_STAT:\n> -               PMD_DRV_LOG(INFO, \"OP_GET_LINK_STAT received\\n\");\n> +               PMD_DRV_LOG(INFO, \"OP_GET_LINK_STAT received\");\n>                 i40e_pf_host_process_cmd_get_link_status(vf);\n>                 break;\n>         case I40E_VIRTCHNL_OP_CFG_VLAN_OFFLOAD:\n> -               PMD_DRV_LOG(INFO, \"OP_CFG_VLAN_OFFLOAD received\\n\");\n> +               PMD_DRV_LOG(INFO, \"OP_CFG_VLAN_OFFLOAD received\");\n>                 i40e_pf_host_process_cmd_cfg_vlan_offload(vf, msg, msglen);\n>                 break;\n>         case I40E_VIRTCHNL_OP_CFG_VLAN_PVID:\n> -               PMD_DRV_LOG(INFO, \"OP_CFG_VLAN_PVID received\\n\");\n> +               PMD_DRV_LOG(INFO, \"OP_CFG_VLAN_PVID received\");\n>                 i40e_pf_host_process_cmd_cfg_pvid(vf, msg, msglen);\n>                 break;\n>          /* Don't add command supported below, which will\n>          *  return an error code.\n>          */\n>         case I40E_VIRTCHNL_OP_FCOE:\n> -               PMD_DRV_LOG(ERR, \"OP_FCOE received, not supported\\n\");\n> +               PMD_DRV_LOG(ERR, \"OP_FCOE received, not supported\");\n>         default:\n> -               PMD_DRV_LOG(ERR, \"%u received, not supported\\n\",\n> -                                                       opcode);\n> +               PMD_DRV_LOG(ERR, \"%u received, not supported\", opcode);\n>                 i40e_pf_host_send_msg_to_vf(vf, opcode,\n>                                 I40E_ERR_PARAM, NULL, 0);\n>                 break;\n> diff --git a/lib/librte_pmd_i40e/i40e_rxtx.c\n> b/lib/librte_pmd_i40e/i40e_rxtx.c\n> index 6987200..70fabaa 100644\n> --- a/lib/librte_pmd_i40e/i40e_rxtx.c\n> +++ b/lib/librte_pmd_i40e/i40e_rxtx.c\n> @@ -420,13 +420,13 @@ i40e_txd_enable_checksum(uint32_t ol_flags,\n>                         uint8_t l3_len)\n>  {\n>         if (!l2_len) {\n> -               PMD_DRV_LOG(DEBUG, \"L2 length set to 0\\n\");\n> +               PMD_DRV_LOG(DEBUG, \"L2 length set to 0\");\n>                 return;\n>         }\n>         *td_offset |= (l2_len >> 1) << I40E_TX_DESC_LENGTH_MACLEN_SHIFT;\n>\n>         if (!l3_len) {\n> -               PMD_DRV_LOG(DEBUG, \"L3 length set to 0\\n\");\n> +               PMD_DRV_LOG(DEBUG, \"L3 length set to 0\");\n>                 return;\n>         }\n>\n> @@ -507,7 +507,7 @@ i40e_xmit_cleanup(struct i40e_tx_queue *txq)\n>         if (!(txd[desc_to_clean_to].cmd_type_offset_bsz &\n>                 rte_cpu_to_le_64(I40E_TX_DESC_DTYPE_DESC_DONE))) {\n>                 PMD_TX_FREE_LOG(DEBUG, \"TX descriptor %4u is not done \"\n> -                       \"(port=%d queue=%d)\", desc_to_clean_to,\n> +                               \"(port=%d queue=%d)\", desc_to_clean_to,\n>                                 txq->port_id, txq->queue_id);\n>                 return -1;\n>         }\n> @@ -676,7 +676,7 @@ i40e_rx_alloc_bufs(struct i40e_rx_queue *rxq)\n>         diag = rte_mempool_get_bulk(rxq->mp, (void *)rxep,\n>                                         rxq->rx_free_thresh);\n>         if (unlikely(diag != 0)) {\n> -               PMD_DRV_LOG(ERR, \"Failed to get mbufs in bulk\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to get mbufs in bulk\");\n>                 return -ENOMEM;\n>         }\n>\n> @@ -729,7 +729,7 @@ rx_recv_pkts(void *rx_queue, struct rte_mbuf\n> **rx_pkts, uint16_t nb_pkts)\n>                         uint16_t i, j;\n>\n>                         PMD_RX_LOG(DEBUG, \"Rx mbuf alloc failed for \"\n> -                                       \"port_id=%u, queue_id=%u\\n\",\n> +                                       \"port_id=%u, queue_id=%u\",\n>                                         rxq->port_id, rxq->queue_id);\n>                         rxq->rx_nb_avail = 0;\n>                         rxq->rx_tail = (uint16_t)(rxq->rx_tail - nb_rx);\n> @@ -1441,7 +1441,7 @@ i40e_dev_rx_queue_start(struct rte_eth_dev *dev,\n> uint16_t rx_queue_id)\n>\n>                 err = i40e_alloc_rx_queue_mbufs(rxq);\n>                 if (err) {\n> -                       PMD_DRV_LOG(ERR, \"Failed to allocate RX queue\n> mbuf\\n\");\n> +                       PMD_DRV_LOG(ERR, \"Failed to allocate RX queue\n> mbuf\");\n>                         return err;\n>                 }\n>\n> @@ -1453,8 +1453,8 @@ i40e_dev_rx_queue_start(struct rte_eth_dev *dev,\n> uint16_t rx_queue_id)\n>                 err = i40e_switch_rx_queue(hw, rx_queue_id + q_base, TRUE);\n>\n>                 if (err) {\n> -                       PMD_DRV_LOG(ERR, \"Failed to switch RX queue %u\n> on\\n\",\n> -                               rx_queue_id);\n> +                       PMD_DRV_LOG(ERR, \"Failed to switch RX queue %u on\",\n> +                                   rx_queue_id);\n>\n>                         i40e_rx_queue_release_mbufs(rxq);\n>                         i40e_reset_rx_queue(rxq);\n> @@ -1479,8 +1479,8 @@ i40e_dev_rx_queue_stop(struct rte_eth_dev *dev,\n> uint16_t rx_queue_id)\n>                 err = i40e_switch_rx_queue(hw, rx_queue_id + q_base,\n> FALSE);\n>\n>                 if (err) {\n> -                       PMD_DRV_LOG(ERR, \"Failed to switch RX queue %u\n> off\\n\",\n> -                               rx_queue_id);\n> +                       PMD_DRV_LOG(ERR, \"Failed to switch RX queue %u\n> off\",\n> +                                   rx_queue_id);\n>                         return err;\n>                 }\n>                 i40e_rx_queue_release_mbufs(rxq);\n> @@ -1503,8 +1503,8 @@ i40e_dev_tx_queue_start(struct rte_eth_dev *dev,\n> uint16_t tx_queue_id)\n>         if (tx_queue_id < dev->data->nb_tx_queues) {\n>                 err = i40e_switch_tx_queue(hw, tx_queue_id + q_base, TRUE);\n>                 if (err)\n> -                       PMD_DRV_LOG(ERR, \"Failed to switch TX queue %u\n> on\\n\",\n> -                               tx_queue_id);\n> +                       PMD_DRV_LOG(ERR, \"Failed to switch TX queue %u on\",\n> +                                   tx_queue_id);\n>         }\n>\n>         return err;\n> @@ -1525,8 +1525,8 @@ i40e_dev_tx_queue_stop(struct rte_eth_dev *dev,\n> uint16_t tx_queue_id)\n>                 err = i40e_switch_tx_queue(hw, tx_queue_id + q_base,\n> FALSE);\n>\n>                 if (err) {\n> -                       PMD_DRV_LOG(ERR, \"Failed to switch TX queue %u\n> of\\n\",\n> -                               tx_queue_id);\n> +                       PMD_DRV_LOG(ERR, \"Failed to switch TX queue %u of\",\n> +                                   tx_queue_id);\n>                         return err;\n>                 }\n>\n> @@ -1554,14 +1554,14 @@ i40e_dev_rx_queue_setup(struct rte_eth_dev *dev,\n>\n>         if (!vsi || queue_idx >= vsi->nb_qps) {\n>                 PMD_DRV_LOG(ERR, \"VSI not available or queue \"\n> -                               \"index exceeds the maximum\\n\");\n> +                           \"index exceeds the maximum\");\n>                 return I40E_ERR_PARAM;\n>         }\n>         if (((nb_desc * sizeof(union i40e_rx_desc)) % I40E_ALIGN) != 0 ||\n>                                         (nb_desc > I40E_MAX_RING_DESC) ||\n>                                         (nb_desc < I40E_MIN_RING_DESC)) {\n>                 PMD_DRV_LOG(ERR, \"Number (%u) of receive descriptors is \"\n> -                                               \"invalid\\n\", nb_desc);\n> +                           \"invalid\", nb_desc);\n>                 return I40E_ERR_PARAM;\n>         }\n>\n> @@ -1578,7 +1578,7 @@ i40e_dev_rx_queue_setup(struct rte_eth_dev *dev,\n>                                  socket_id);\n>         if (!rxq) {\n>                 PMD_DRV_LOG(ERR, \"Failed to allocate memory for \"\n> -                                       \"rx queue data structure\\n\");\n> +                           \"rx queue data structure\");\n>                 return (-ENOMEM);\n>         }\n>         rxq->mp = mp;\n> @@ -1603,7 +1603,7 @@ i40e_dev_rx_queue_setup(struct rte_eth_dev *dev,\n>                                         socket_id);\n>         if (!rz) {\n>                 i40e_dev_rx_queue_release(rxq);\n> -               PMD_DRV_LOG(ERR, \"Failed to reserve DMA memory for RX\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to reserve DMA memory for RX\");\n>                 return (-ENOMEM);\n>         }\n>\n> @@ -1632,7 +1632,7 @@ i40e_dev_rx_queue_setup(struct rte_eth_dev *dev,\n>                                    socket_id);\n>         if (!rxq->sw_ring) {\n>                 i40e_dev_rx_queue_release(rxq);\n> -               PMD_DRV_LOG(ERR, \"Failed to allocate memory for SW\n> ring\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to allocate memory for SW ring\");\n>                 return (-ENOMEM);\n>         }\n>\n> @@ -1645,17 +1645,17 @@ i40e_dev_rx_queue_setup(struct rte_eth_dev *dev,\n>         if (!use_def_burst_func && !dev->data->scattered_rx) {\n>  #ifdef RTE_LIBRTE_I40E_RX_ALLOW_BULK_ALLOC\n>                 PMD_INIT_LOG(DEBUG, \"Rx Burst Bulk Alloc Preconditions are\n> \"\n> -                       \"satisfied. Rx Burst Bulk Alloc function will be \"\n> -                                       \"used on port=%d, queue=%d.\\n\",\n> -                                       rxq->port_id, rxq->queue_id);\n> +                            \"satisfied. Rx Burst Bulk Alloc function will\n> be \"\n> +                            \"used on port=%d, queue=%d.\",\n> +                            rxq->port_id, rxq->queue_id);\n>                 dev->rx_pkt_burst = i40e_recv_pkts_bulk_alloc;\n>  #endif /* RTE_LIBRTE_I40E_RX_ALLOW_BULK_ALLOC */\n>         } else {\n>                 PMD_INIT_LOG(DEBUG, \"Rx Burst Bulk Alloc Preconditions are\n> \"\n> -                               \"not satisfied, Scattered Rx is requested,\n> \"\n> -                               \"or RTE_LIBRTE_I40E_RX_ALLOW_BULK_ALLOC is\n> \"\n> -                                       \"not enabled on port=%d,\n> queue=%d.\\n\",\n> -                                               rxq->port_id,\n> rxq->queue_id);\n> +                            \"not satisfied, Scattered Rx is requested, \"\n> +                            \"or RTE_LIBRTE_I40E_RX_ALLOW_BULK_ALLOC is \"\n> +                            \"not enabled on port=%d, queue=%d.\",\n> +                            rxq->port_id, rxq->queue_id);\n>         }\n>\n>         return 0;\n> @@ -1667,7 +1667,7 @@ i40e_dev_rx_queue_release(void *rxq)\n>         struct i40e_rx_queue *q = (struct i40e_rx_queue *)rxq;\n>\n>         if (!q) {\n> -               PMD_DRV_LOG(DEBUG, \"Pointer to rxq is NULL\\n\");\n> +               PMD_DRV_LOG(DEBUG, \"Pointer to rxq is NULL\");\n>                 return;\n>         }\n>\n> @@ -1685,7 +1685,7 @@ i40e_dev_rx_queue_count(struct rte_eth_dev *dev,\n> uint16_t rx_queue_id)\n>         uint16_t desc = 0;\n>\n>         if (unlikely(rx_queue_id >= dev->data->nb_rx_queues)) {\n> -               PMD_DRV_LOG(ERR, \"Invalid RX queue id %u\\n\", rx_queue_id);\n> +               PMD_DRV_LOG(ERR, \"Invalid RX queue id %u\", rx_queue_id);\n>                 return 0;\n>         }\n>\n> @@ -1719,7 +1719,7 @@ i40e_dev_rx_descriptor_done(void *rx_queue, uint16_t\n> offset)\n>         int ret;\n>\n>         if (unlikely(offset >= rxq->nb_rx_desc)) {\n> -               PMD_DRV_LOG(ERR, \"Invalid RX queue id %u\\n\", offset);\n> +               PMD_DRV_LOG(ERR, \"Invalid RX queue id %u\", offset);\n>                 return 0;\n>         }\n>\n> @@ -1751,7 +1751,7 @@ i40e_dev_tx_queue_setup(struct rte_eth_dev *dev,\n>\n>         if (!vsi || queue_idx >= vsi->nb_qps) {\n>                 PMD_DRV_LOG(ERR, \"VSI is NULL, or queue index (%u) \"\n> -                               \"exceeds the maximum\\n\", queue_idx);\n> +                           \"exceeds the maximum\", queue_idx);\n>                 return I40E_ERR_PARAM;\n>         }\n>\n> @@ -1759,7 +1759,7 @@ i40e_dev_tx_queue_setup(struct rte_eth_dev *dev,\n>                                         (nb_desc > I40E_MAX_RING_DESC) ||\n>                                         (nb_desc < I40E_MIN_RING_DESC)) {\n>                 PMD_DRV_LOG(ERR, \"Number (%u) of transmit descriptors is \"\n> -                                                \"invalid\\n\", nb_desc);\n> +                           \"invalid\", nb_desc);\n>                 return I40E_ERR_PARAM;\n>         }\n>\n> @@ -1790,7 +1790,7 @@ i40e_dev_tx_queue_setup(struct rte_eth_dev *dev,\n>         if (tx_rs_thresh >= (nb_desc - 2)) {\n>                 PMD_INIT_LOG(ERR, \"tx_rs_thresh must be less than the \"\n>                              \"number of TX descriptors minus 2. \"\n> -                            \"(tx_rs_thresh=%u port=%d queue=%d)\\n\",\n> +                            \"(tx_rs_thresh=%u port=%d queue=%d)\",\n>                              (unsigned int)tx_rs_thresh,\n>                              (int)dev->data->port_id,\n>                              (int)queue_idx);\n> @@ -1800,7 +1800,7 @@ i40e_dev_tx_queue_setup(struct rte_eth_dev *dev,\n>                 PMD_INIT_LOG(ERR, \"tx_rs_thresh must be less than the \"\n>                              \"tx_free_thresh must be less than the \"\n>                              \"number of TX descriptors minus 3. \"\n> -                            \"(tx_free_thresh=%u port=%d queue=%d)\\n\",\n> +                            \"(tx_free_thresh=%u port=%d queue=%d)\",\n>                              (unsigned int)tx_free_thresh,\n>                              (int)dev->data->port_id,\n>                              (int)queue_idx);\n> @@ -1809,7 +1809,7 @@ i40e_dev_tx_queue_setup(struct rte_eth_dev *dev,\n>         if (tx_rs_thresh > tx_free_thresh) {\n>                 PMD_INIT_LOG(ERR, \"tx_rs_thresh must be less than or \"\n>                              \"equal to tx_free_thresh. (tx_free_thresh=%u\"\n> -                            \" tx_rs_thresh=%u port=%d queue=%d)\\n\",\n> +                            \" tx_rs_thresh=%u port=%d queue=%d)\",\n>                              (unsigned int)tx_free_thresh,\n>                              (unsigned int)tx_rs_thresh,\n>                              (int)dev->data->port_id,\n> @@ -1819,7 +1819,7 @@ i40e_dev_tx_queue_setup(struct rte_eth_dev *dev,\n>         if ((nb_desc % tx_rs_thresh) != 0) {\n>                 PMD_INIT_LOG(ERR, \"tx_rs_thresh must be a divisor of the \"\n>                              \"number of TX descriptors. (tx_rs_thresh=%u\"\n> -                            \" port=%d queue=%d)\\n\",\n> +                            \" port=%d queue=%d)\",\n>                              (unsigned int)tx_rs_thresh,\n>                              (int)dev->data->port_id,\n>                              (int)queue_idx);\n> @@ -1828,7 +1828,7 @@ i40e_dev_tx_queue_setup(struct rte_eth_dev *dev,\n>         if ((tx_rs_thresh > 1) && (tx_conf->tx_thresh.wthresh != 0)) {\n>                 PMD_INIT_LOG(ERR, \"TX WTHRESH must be set to 0 if \"\n>                              \"tx_rs_thresh is greater than 1. \"\n> -                            \"(tx_rs_thresh=%u port=%d queue=%d)\\n\",\n> +                            \"(tx_rs_thresh=%u port=%d queue=%d)\",\n>                              (unsigned int)tx_rs_thresh,\n>                              (int)dev->data->port_id,\n>                              (int)queue_idx);\n> @@ -1848,7 +1848,7 @@ i40e_dev_tx_queue_setup(struct rte_eth_dev *dev,\n>                                   socket_id);\n>         if (!txq) {\n>                 PMD_DRV_LOG(ERR, \"Failed to allocate memory for \"\n> -                                       \"tx queue structure\\n\");\n> +                           \"tx queue structure\");\n>                 return (-ENOMEM);\n>         }\n>\n> @@ -1862,7 +1862,7 @@ i40e_dev_tx_queue_setup(struct rte_eth_dev *dev,\n>                                         socket_id);\n>         if (!tz) {\n>                 i40e_dev_tx_queue_release(txq);\n> -               PMD_DRV_LOG(ERR, \"Failed to reserve DMA memory for TX\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to reserve DMA memory for TX\");\n>                 return (-ENOMEM);\n>         }\n>\n> @@ -1894,7 +1894,7 @@ i40e_dev_tx_queue_setup(struct rte_eth_dev *dev,\n>                                    socket_id);\n>         if (!txq->sw_ring) {\n>                 i40e_dev_tx_queue_release(txq);\n> -               PMD_DRV_LOG(ERR, \"Failed to allocate memory for SW TX\n> ring\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to allocate memory for SW TX\n> ring\");\n>                 return (-ENOMEM);\n>         }\n>\n> @@ -1905,10 +1905,10 @@ i40e_dev_tx_queue_setup(struct rte_eth_dev *dev,\n>         /* Use a simple TX queue without offloads or multi segs if\n> possible */\n>         if (((txq->txq_flags & I40E_SIMPLE_FLAGS) == I40E_SIMPLE_FLAGS) &&\n>                                 (txq->tx_rs_thresh >= I40E_TX_MAX_BURST)) {\n> -               PMD_INIT_LOG(INFO, \"Using simple tx path\\n\");\n> +               PMD_INIT_LOG(INFO, \"Using simple tx path\");\n>                 dev->tx_pkt_burst = i40e_xmit_pkts_simple;\n>         } else {\n> -               PMD_INIT_LOG(INFO, \"Using full-featured tx path\\n\");\n> +               PMD_INIT_LOG(INFO, \"Using full-featured tx path\");\n>                 dev->tx_pkt_burst = i40e_xmit_pkts;\n>         }\n>\n> @@ -1921,7 +1921,7 @@ i40e_dev_tx_queue_release(void *txq)\n>         struct i40e_tx_queue *q = (struct i40e_tx_queue *)txq;\n>\n>         if (!q) {\n> -               PMD_DRV_LOG(DEBUG, \"Pointer to TX queue is NULL\\n\");\n> +               PMD_DRV_LOG(DEBUG, \"Pointer to TX queue is NULL\");\n>                 return;\n>         }\n>\n> @@ -1962,7 +1962,7 @@ i40e_rx_queue_release_mbufs(struct i40e_rx_queue\n> *rxq)\n>         uint16_t i;\n>\n>         if (!rxq || !rxq->sw_ring) {\n> -               PMD_DRV_LOG(DEBUG, \"Pointer to rxq or sw_ring is NULL\\n\");\n> +               PMD_DRV_LOG(DEBUG, \"Pointer to rxq or sw_ring is NULL\");\n>                 return;\n>         }\n>\n> @@ -2022,7 +2022,7 @@ i40e_tx_queue_release_mbufs(struct i40e_tx_queue\n> *txq)\n>         uint16_t i;\n>\n>         if (!txq || !txq->sw_ring) {\n> -               PMD_DRV_LOG(DEBUG, \"Pointer to rxq or sw_ring is NULL\\n\");\n> +               PMD_DRV_LOG(DEBUG, \"Pointer to rxq or sw_ring is NULL\");\n>                 return;\n>         }\n>\n> @@ -2041,7 +2041,7 @@ i40e_reset_tx_queue(struct i40e_tx_queue *txq)\n>         uint16_t i, prev, size;\n>\n>         if (!txq) {\n> -               PMD_DRV_LOG(DEBUG, \"Pointer to txq is NULL\\n\");\n> +               PMD_DRV_LOG(DEBUG, \"Pointer to txq is NULL\");\n>                 return;\n>         }\n>\n> @@ -2092,13 +2092,13 @@ i40e_tx_queue_init(struct i40e_tx_queue *txq)\n>\n>         err = i40e_clear_lan_tx_queue_context(hw, pf_q);\n>         if (err != I40E_SUCCESS) {\n> -               PMD_DRV_LOG(ERR, \"Failure of clean lan tx queue\n> context\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failure of clean lan tx queue context\");\n>                 return err;\n>         }\n>\n>         err = i40e_set_lan_tx_queue_context(hw, pf_q, &tx_ctx);\n>         if (err != I40E_SUCCESS) {\n> -               PMD_DRV_LOG(ERR, \"Failure of set lan tx queue context\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failure of set lan tx queue context\");\n>                 return err;\n>         }\n>\n> @@ -2126,7 +2126,7 @@ i40e_alloc_rx_queue_mbufs(struct i40e_rx_queue *rxq)\n>                 struct rte_mbuf *mbuf = rte_rxmbuf_alloc(rxq->mp);\n>\n>                 if (unlikely(!mbuf)) {\n> -                       PMD_DRV_LOG(ERR, \"Failed to allocate mbuf for\n> RX\\n\");\n> +                       PMD_DRV_LOG(ERR, \"Failed to allocate mbuf for RX\");\n>                         return -ENOMEM;\n>                 }\n>\n> @@ -2194,20 +2194,20 @@ i40e_rx_queue_config(struct i40e_rx_queue *rxq)\n>                 if (rxq->max_pkt_len <= ETHER_MAX_LEN ||\n>                         rxq->max_pkt_len > I40E_FRAME_SIZE_MAX) {\n>                         PMD_DRV_LOG(ERR, \"maximum packet length must \"\n> -                               \"be larger than %u and smaller than %u,\"\n> -                                       \"as jumbo frame is enabled\\n\",\n> -                                               (uint32_t)ETHER_MAX_LEN,\n> -                                       (uint32_t)I40E_FRAME_SIZE_MAX);\n> +                                   \"be larger than %u and smaller than\n> %u,\"\n> +                                   \"as jumbo frame is enabled\",\n> +                                   (uint32_t)ETHER_MAX_LEN,\n> +                                   (uint32_t)I40E_FRAME_SIZE_MAX);\n>                         return I40E_ERR_CONFIG;\n>                 }\n>         } else {\n>                 if (rxq->max_pkt_len < ETHER_MIN_LEN ||\n>                         rxq->max_pkt_len > ETHER_MAX_LEN) {\n>                         PMD_DRV_LOG(ERR, \"maximum packet length must be \"\n> -                                       \"larger than %u and smaller than\n> %u, \"\n> -                                       \"as jumbo frame is disabled\\n\",\n> -                                               (uint32_t)ETHER_MIN_LEN,\n> -                                               (uint32_t)ETHER_MAX_LEN);\n> +                                   \"larger than %u and smaller than %u, \"\n> +                                   \"as jumbo frame is disabled\",\n> +                                   (uint32_t)ETHER_MIN_LEN,\n> +                                   (uint32_t)ETHER_MAX_LEN);\n>                         return I40E_ERR_CONFIG;\n>                 }\n>         }\n> @@ -2230,7 +2230,7 @@ i40e_rx_queue_init(struct i40e_rx_queue *rxq)\n>\n>         err = i40e_rx_queue_config(rxq);\n>         if (err < 0) {\n> -               PMD_DRV_LOG(ERR, \"Failed to config RX queue\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to config RX queue\");\n>                 return err;\n>         }\n>\n> @@ -2262,12 +2262,12 @@ i40e_rx_queue_init(struct i40e_rx_queue *rxq)\n>\n>         err = i40e_clear_lan_rx_queue_context(hw, pf_q);\n>         if (err != I40E_SUCCESS) {\n> -               PMD_DRV_LOG(ERR, \"Failed to clear LAN RX queue context\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to clear LAN RX queue context\");\n>                 return err;\n>         }\n>         err = i40e_set_lan_rx_queue_context(hw, pf_q, &rx_ctx);\n>         if (err != I40E_SUCCESS) {\n> -               PMD_DRV_LOG(ERR, \"Failed to set LAN RX queue context\\n\");\n> +               PMD_DRV_LOG(ERR, \"Failed to set LAN RX queue context\");\n>                 return err;\n>         }\n>\n> --\n> 1.7.10.4\n>\n>\nReviewed-By: Jay Rolette <rolette@infiniteio.com>",
        "headers": {
            "Return-Path": "<rolette@infiniteio.com>",
            "MIME-Version": "1.0",
            "X-Mailman-Version": "2.1.15",
            "Date": "Tue, 2 Sep 2014 13:20:12 -0500",
            "X-List-Received-Date": "Tue, 02 Sep 2014 18:15:41 -0000",
            "References": "<1409567080-27083-1-git-send-email-david.marchand@6wind.com>\n\t<1409567080-27083-10-git-send-email-david.marchand@6wind.com>",
            "Content-Type": "text/plain; charset=UTF-8",
            "X-BeenThere": "dev@dpdk.org",
            "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20130820;\n\th=x-gm-message-state:mime-version:in-reply-to:references:date\n\t:message-id:subject:from:to:cc:content-type;\n\tbh=y0JStazB74Qow9HU0bonycaaYElUSDoMDKu2DsFxkNk=;\n\tb=h/JAphkojIUPDhrHdnG/LEo1/7bgfEJTFXVEEgh5o79qfRK+GdH6HXe1gP8YvZnsPz\n\tq8UjXU6NLy3ZotmVyv5LHCfDmTdVbRfPjdk6CjFkgL7977QeaqSiRPd5uSDsuPVavnDN\n\tyOKUApOnN7qsPpoM9btn0V6e9Kwv7xgTfnoGYLUapqc8te357Gwg86pyRiqhW22WeQDe\n\tskDbXTjUAjDAZuq7TzZu7H4zfHjWvuJyJ9mJ9RwgYVyHXDkRD1JMSL3ImBwz2c7QquHw\n\t8+LKuvP/sTHuos123GWGG+IFYKKzv0gUJTtJG8UO2y5lPZTI3Vx18gN0lAOHqFBbm3Ia\n\tcOJw==",
            "Received": [
                "from mail-yk0-f179.google.com (mail-yk0-f179.google.com\n\t[209.85.160.179]) by dpdk.org (Postfix) with ESMTP id C3ED55941\n\tfor <dev@dpdk.org>; Tue,  2 Sep 2014 20:15:40 +0200 (CEST)",
                "by mail-yk0-f179.google.com with SMTP id 142so4336139ykq.38\n\tfor <dev@dpdk.org>; Tue, 02 Sep 2014 11:20:13 -0700 (PDT)",
                "by 10.170.96.213 with HTTP; Tue, 2 Sep 2014 11:20:12 -0700 (PDT)"
            ],
            "Subject": "Re: [dpdk-dev] [PATCH v2 09/17] i40e: clean log messages",
            "X-Content-Filtered-By": "Mailman/MimeDel 2.1.15",
            "X-Received": "by 10.236.7.197 with SMTP id 45mr5368560yhp.113.1409682013072;\n\tTue, 02 Sep 2014 11:20:13 -0700 (PDT)",
            "List-Help": "<mailto:dev-request@dpdk.org?subject=help>",
            "X-Mailman-Approved-At": "Tue, 02 Sep 2014 22:52:54 +0200",
            "Message-ID": "<CADNuJVpHwbR2DsRi6_CzcQvZFo+kdeNqyx5WiPe488WsLMRfRw@mail.gmail.com>",
            "Precedence": "list",
            "X-Gm-Message-State": "ALoCoQl/QM07hlgCWCK0mJfz/PUAVVVyvJ2KoUuWIIhF22yUy9rPv1OoILWKaGeoq2fu5tmSCCwT",
            "From": "Jay Rolette <rolette@infiniteio.com>",
            "List-Archive": "<http://dpdk.org/ml/archives/dev/>",
            "List-Subscribe": "<http://dpdk.org/ml/listinfo/dev>,\n\t<mailto:dev-request@dpdk.org?subject=subscribe>",
            "Cc": "\"dev@dpdk.org\" <dev@dpdk.org>",
            "List-Id": "patches and discussions about DPDK <dev.dpdk.org>",
            "In-Reply-To": "<1409567080-27083-10-git-send-email-david.marchand@6wind.com>",
            "List-Unsubscribe": "<http://dpdk.org/ml/options/dev>,\n\t<mailto:dev-request@dpdk.org?subject=unsubscribe>",
            "List-Post": "<mailto:dev@dpdk.org>",
            "To": "David Marchand <david.marchand@6wind.com>"
        }
    }
]