From patchwork Wed Apr 25 15:45:51 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 38946 X-Patchwork-Delegate: ferruh.yigit@amd.com Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id ADDDD8E73; Wed, 25 Apr 2018 17:46:09 +0200 (CEST) Received: from mail-pf0-f194.google.com (mail-pf0-f194.google.com [209.85.192.194]) by dpdk.org (Postfix) with ESMTP id 53D518E1C for ; Wed, 25 Apr 2018 17:46:01 +0200 (CEST) Received: by mail-pf0-f194.google.com with SMTP id l27so15560994pfk.12 for ; Wed, 25 Apr 2018 08:46:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=C9dH+TXCVXTQZeru9TsuTV7yZ1NtFDujRllKjzUlwjI=; b=h2epplOWEDZWor47ZNfF9I22yxZmkt1CG9fJSip//uQX9NbQxzPKAgo4ZyoJ0YGBo6 lOSrF9JrKJ2iYfSopJw5IFnSyWq62xNUTD8CjE5OPNcxzUdimG6s8xWvRG+JP9c7Jcwe HCWoMvFsciHjNykY6WzK7bOFRoEwHO5KPNQDte2uU9ocezyha7HZPGqGlADti+bZDmPN tp51utYE+g24F7QilU3ECvJrVjSAV2ddT/c90z4aA2d/wrCoHB8TIsjXscFmbr7aWtd+ Nt6CyCDIYIzePSqu3eW5CwrcYIhqFkx0AevG2+5RRsAyxnkLKd4VVlP5j9K1zDQWWcLA nWww== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=C9dH+TXCVXTQZeru9TsuTV7yZ1NtFDujRllKjzUlwjI=; b=Xul9GEi0kZJ/HHoeYFHwOmKyFLfnmNUwSyo7Hz+iqyHeZhG9YFsCdMsXxXGv+oJ48U vU4PJpe/Qq6aXXKanqLXE6UEz/QmwXSzYCvxRwJ0EvojzkirPjk3K1mzO1O/33TULZIm treI9eRdt3IgiHPJ6Q50kzpOXpSxMOr/EvMZbRDI529jblEq3STXtMaWSsfNPl+dRgT/ d4L1EuK3DiN/fElFW2N5iRCTMNpMJoy2znoSUdsELwBelxvkrqPrLZHWnA+h853yGyPu PDpDM7ds21zddA0T9aInTvBhi9O7BS/+jxfqRKGCYagdqWLCkp19DvO5vBVUfW3uVOO/ pbww== X-Gm-Message-State: ALQs6tDyNZpU1ZMcKDGHo5PzGaN9nqW30pAbMP04muV2U4vW5B+S6iD9 VCbbEmYDAXGQ8K3w8N9PeFty2y3kl5c= X-Google-Smtp-Source: AIpwx4+mVSWPsgQ6rkCbMJDg06XzSLyluhgzUZ+Jkzpr3mUVSMEJ9O+eifjleAI/sBpsSlwR9VktYg== X-Received: by 2002:a17:902:362:: with SMTP id 89-v6mr30328115pld.270.1524671160329; Wed, 25 Apr 2018 08:46:00 -0700 (PDT) Received: from xeon-e3.lan (204-195-71-95.wavecable.com. [204.195.71.95]) by smtp.gmail.com with ESMTPSA id b89sm8115152pfd.85.2018.04.25.08.45.58 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 25 Apr 2018 08:45:59 -0700 (PDT) From: Stephen Hemminger To: alejandro.lucero@netronome.com Cc: dev@dpdk.org, Stephen Hemminger Date: Wed, 25 Apr 2018 08:45:51 -0700 Message-Id: <20180425154551.6795-6-stephen@networkplumber.org> X-Mailer: git-send-email 2.17.0 In-Reply-To: <20180425154551.6795-1-stephen@networkplumber.org> References: <20180425154551.6795-1-stephen@networkplumber.org> Subject: [dpdk-dev] [PATCH 5/5] net/nfp: use dynamic logging everywhere X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Drivers should only log with their assigned logtype, not with the generic PMD log type. Signed-off-by: Stephen Hemminger --- drivers/net/nfp/nfp_net.c | 132 +++++++++++++++++++------------------- 1 file changed, 66 insertions(+), 66 deletions(-) diff --git a/drivers/net/nfp/nfp_net.c b/drivers/net/nfp/nfp_net.c index c8b0fff9df14..e092ab20296f 100644 --- a/drivers/net/nfp/nfp_net.c +++ b/drivers/net/nfp/nfp_net.c @@ -1423,15 +1423,15 @@ nfp_net_dev_link_status_print(struct rte_eth_dev *dev) rte_eth_linkstatus_get(dev, &link); if (link.link_status) - RTE_LOG(INFO, PMD, "Port %d: Link Up - speed %u Mbps - %s\n", - dev->data->port_id, link.link_speed, - link.link_duplex == ETH_LINK_FULL_DUPLEX - ? "full-duplex" : "half-duplex"); + PMD_DRV_LOG(INFO, "Port %d: Link Up - speed %u Mbps - %s", + dev->data->port_id, link.link_speed, + link.link_duplex == ETH_LINK_FULL_DUPLEX + ? "full-duplex" : "half-duplex"); else - RTE_LOG(INFO, PMD, " Port %d: Link Down\n", - dev->data->port_id); + PMD_DRV_LOG(INFO, " Port %d: Link Down", + dev->data->port_id); - RTE_LOG(INFO, PMD, "PCI Address: %04d:%02d:%02d:%d\n", + PMD_DRV_LOG(INFO, "PCI Address: %04d:%02d:%02d:%d", pci_dev->addr.domain, pci_dev->addr.bus, pci_dev->addr.devid, pci_dev->addr.function); } @@ -1491,7 +1491,7 @@ nfp_net_dev_interrupt_handler(void *param) if (rte_eal_alarm_set(timeout * 1000, nfp_net_dev_interrupt_delayed_handler, (void *)dev) < 0) { - RTE_LOG(ERR, PMD, "Error setting alarm"); + PMD_INIT_LOG(ERR, "Error setting alarm"); /* Unmasking */ nfp_net_irq_unmask(dev); } @@ -1578,7 +1578,7 @@ nfp_net_rx_queue_setup(struct rte_eth_dev *dev, if (((nb_desc * sizeof(struct nfp_net_rx_desc)) % 128) != 0 || (nb_desc > NFP_NET_MAX_RX_DESC) || (nb_desc < NFP_NET_MIN_RX_DESC)) { - RTE_LOG(ERR, PMD, "Wrong nb_desc value\n"); + PMD_DRV_LOG(ERR, "Wrong nb_desc value"); return -EINVAL; } @@ -1586,10 +1586,10 @@ nfp_net_rx_queue_setup(struct rte_eth_dev *dev, rxmode = &dev_conf->rxmode; if (rx_conf->offloads != rxmode->offloads) { - RTE_LOG(ERR, PMD, "queue %u rx offloads not as port offloads\n", + PMD_DRV_LOG(ERR, "queue %u rx offloads not as port offloads", queue_idx); - RTE_LOG(ERR, PMD, "\tport: %" PRIx64 "\n", rxmode->offloads); - RTE_LOG(ERR, PMD, "\tqueue: %" PRIx64 "\n", rx_conf->offloads); + PMD_DRV_LOG(ERR, "\tport: %" PRIx64 "", rxmode->offloads); + PMD_DRV_LOG(ERR, "\tqueue: %" PRIx64 "", rx_conf->offloads); return -EINVAL; } @@ -1642,7 +1642,7 @@ nfp_net_rx_queue_setup(struct rte_eth_dev *dev, socket_id); if (tz == NULL) { - RTE_LOG(ERR, PMD, "Error allocatig rx dma\n"); + PMD_DRV_LOG(ERR, "Error allocatig rx dma"); nfp_net_rx_queue_release(rxq); return -ENOMEM; } @@ -1693,7 +1693,7 @@ nfp_net_rx_fill_freelist(struct nfp_net_rxq *rxq) struct rte_mbuf *mbuf = rte_pktmbuf_alloc(rxq->mem_pool); if (mbuf == NULL) { - RTE_LOG(ERR, PMD, "RX mbuf alloc failed queue_id=%u\n", + PMD_DRV_LOG(ERR, "RX mbuf alloc failed queue_id=%u", (unsigned)rxq->qidx); return -ENOMEM; } @@ -1740,7 +1740,7 @@ nfp_net_tx_queue_setup(struct rte_eth_dev *dev, uint16_t queue_idx, if (((nb_desc * sizeof(struct nfp_net_tx_desc)) % 128) != 0 || (nb_desc > NFP_NET_MAX_TX_DESC) || (nb_desc < NFP_NET_MIN_TX_DESC)) { - RTE_LOG(ERR, PMD, "Wrong nb_desc value\n"); + PMD_DRV_LOG(ERR, "Wrong nb_desc value"); return -EINVAL; } @@ -1748,7 +1748,7 @@ nfp_net_tx_queue_setup(struct rte_eth_dev *dev, uint16_t queue_idx, txmode = &dev_conf->txmode; if (tx_conf->offloads != txmode->offloads) { - RTE_LOG(ERR, PMD, "queue %u tx offloads not as port offloads", + PMD_DRV_LOG(ERR, "queue %u tx offloads not as port offloads", queue_idx); return -EINVAL; } @@ -1758,10 +1758,10 @@ nfp_net_tx_queue_setup(struct rte_eth_dev *dev, uint16_t queue_idx, DEFAULT_TX_FREE_THRESH); if (tx_free_thresh > (nb_desc)) { - RTE_LOG(ERR, PMD, + PMD_DRV_LOG(ERR, "tx_free_thresh must be less than the number of TX " "descriptors. (tx_free_thresh=%u port=%d " - "queue=%d)\n", (unsigned int)tx_free_thresh, + "queue=%d)", (unsigned int)tx_free_thresh, dev->data->port_id, (int)queue_idx); return -(EINVAL); } @@ -1781,7 +1781,7 @@ nfp_net_tx_queue_setup(struct rte_eth_dev *dev, uint16_t queue_idx, txq = rte_zmalloc_socket("ethdev TX queue", sizeof(struct nfp_net_txq), RTE_CACHE_LINE_SIZE, socket_id); if (txq == NULL) { - RTE_LOG(ERR, PMD, "Error allocating tx dma\n"); + PMD_DRV_LOG(ERR, "Error allocating tx dma"); return -ENOMEM; } @@ -1795,7 +1795,7 @@ nfp_net_tx_queue_setup(struct rte_eth_dev *dev, uint16_t queue_idx, NFP_NET_MAX_TX_DESC, NFP_MEMZONE_ALIGN, socket_id); if (tz == NULL) { - RTE_LOG(ERR, PMD, "Error allocating tx dma\n"); + PMD_DRV_LOG(ERR, "Error allocating tx dma"); nfp_net_tx_queue_release(txq); return -ENOMEM; } @@ -2420,7 +2420,7 @@ nfp_net_vlan_offload_set(struct rte_eth_dev *dev, int mask) if ((mask & ETH_VLAN_FILTER_OFFLOAD) || (mask & ETH_VLAN_EXTEND_OFFLOAD)) - RTE_LOG(INFO, PMD, "No support for ETH_VLAN_FILTER_OFFLOAD or" + PMD_DRV_LOG(INFO, "No support for ETH_VLAN_FILTER_OFFLOAD or" " ETH_VLAN_EXTEND_OFFLOAD"); /* Enable vlan strip if it is not configured yet */ @@ -2457,9 +2457,9 @@ nfp_net_rss_reta_write(struct rte_eth_dev *dev, NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private); if (reta_size != NFP_NET_CFG_RSS_ITBL_SZ) { - RTE_LOG(ERR, PMD, "The size of hash lookup table configured " + PMD_DRV_LOG(ERR, "The size of hash lookup table configured " "(%d) doesn't match the number hardware can supported " - "(%d)\n", reta_size, NFP_NET_CFG_RSS_ITBL_SZ); + "(%d)", reta_size, NFP_NET_CFG_RSS_ITBL_SZ); return -EINVAL; } @@ -2538,9 +2538,9 @@ nfp_net_reta_query(struct rte_eth_dev *dev, return -EINVAL; if (reta_size != NFP_NET_CFG_RSS_ITBL_SZ) { - RTE_LOG(ERR, PMD, "The size of hash lookup table configured " + PMD_DRV_LOG(ERR, "The size of hash lookup table configured " "(%d) doesn't match the number hardware can supported " - "(%d)\n", reta_size, NFP_NET_CFG_RSS_ITBL_SZ); + "(%d)", reta_size, NFP_NET_CFG_RSS_ITBL_SZ); return -EINVAL; } @@ -2626,14 +2626,14 @@ nfp_net_rss_hash_update(struct rte_eth_dev *dev, /* Checking if RSS is enabled */ if (!(hw->ctrl & NFP_NET_CFG_CTRL_RSS)) { if (rss_hf != 0) { /* Enable RSS? */ - RTE_LOG(ERR, PMD, "RSS unsupported\n"); + PMD_DRV_LOG(ERR, "RSS unsupported"); return -EINVAL; } return 0; /* Nothing to do */ } if (rss_conf->rss_key_len > NFP_NET_CFG_RSS_KEY_SZ) { - RTE_LOG(ERR, PMD, "hash key too long\n"); + PMD_DRV_LOG(ERR, "hash key too long"); return -EINVAL; } @@ -2705,7 +2705,7 @@ nfp_net_rss_config_default(struct rte_eth_dev *dev) uint16_t queue; int i, j, ret; - RTE_LOG(INFO, PMD, "setting default RSS conf for %u queues\n", + PMD_DRV_LOG(INFO, "setting default RSS conf for %u queues", rx_queues); nfp_reta_conf[0].mask = ~0x0; @@ -2725,7 +2725,7 @@ nfp_net_rss_config_default(struct rte_eth_dev *dev) dev_conf = &dev->data->dev_conf; if (!dev_conf) { - RTE_LOG(INFO, PMD, "wrong rss conf"); + PMD_DRV_LOG(INFO, "wrong rss conf"); return -EINVAL; } rss_conf = dev_conf->rx_adv_conf.rss_conf; @@ -2816,11 +2816,11 @@ nfp_net_init(struct rte_eth_dev *eth_dev) (pci_dev->id.device_id == PCI_DEVICE_ID_NFP6000_PF_NIC)) { port = get_pf_port_number(eth_dev->data->name); if (port < 0 || port > 7) { - RTE_LOG(ERR, PMD, "Port value is wrong\n"); + PMD_DRV_LOG(ERR, "Port value is wrong"); return -ENODEV; } - PMD_INIT_LOG(DEBUG, "Working with PF port value %d\n", port); + PMD_INIT_LOG(DEBUG, "Working with PF port value %d", port); /* This points to port 0 private data */ hwport0 = NFP_NET_DEV_PRIVATE_TO_HW(eth_dev->data->dev_private); @@ -2854,8 +2854,8 @@ nfp_net_init(struct rte_eth_dev *eth_dev) hw->ctrl_bar = (uint8_t *)pci_dev->mem_resource[0].addr; if (hw->ctrl_bar == NULL) { - RTE_LOG(ERR, PMD, - "hw->ctrl_bar is NULL. BAR0 not configured\n"); + PMD_DRV_LOG(ERR, + "hw->ctrl_bar is NULL. BAR0 not configured"); return -ENODEV; } @@ -2864,11 +2864,11 @@ nfp_net_init(struct rte_eth_dev *eth_dev) hw->total_ports * 32768, &hw->ctrl_area); if (!hw->ctrl_bar) { - printf("nfp_rtsym_map fails for _pf0_net_ctrl_bar\n"); + printf("nfp_rtsym_map fails for _pf0_net_ctrl_bar"); return -EIO; } - PMD_INIT_LOG(DEBUG, "ctrl bar: %p\n", hw->ctrl_bar); + PMD_INIT_LOG(DEBUG, "ctrl bar: %p", hw->ctrl_bar); } if (port > 0) { @@ -2880,7 +2880,7 @@ nfp_net_init(struct rte_eth_dev *eth_dev) (port * NFP_PF_CSR_SLICE_SIZE); } - PMD_INIT_LOG(DEBUG, "ctrl bar: %p\n", hw->ctrl_bar); + PMD_INIT_LOG(DEBUG, "ctrl bar: %p", hw->ctrl_bar); hw->max_rx_queues = nn_cfg_readl(hw, NFP_NET_CFG_MAX_RXRINGS); hw->max_tx_queues = nn_cfg_readl(hw, NFP_NET_CFG_MAX_TXRINGS); @@ -2896,13 +2896,13 @@ nfp_net_init(struct rte_eth_dev *eth_dev) rx_bar_off = start_q * NFP_QCP_QUEUE_ADDR_SZ; break; default: - RTE_LOG(ERR, PMD, "nfp_net: no device ID matching\n"); + PMD_DRV_LOG(ERR, "nfp_net: no device ID matching"); err = -ENODEV; goto dev_err_ctrl_map; } - PMD_INIT_LOG(DEBUG, "tx_bar_off: 0x%" PRIx64 "\n", tx_bar_off); - PMD_INIT_LOG(DEBUG, "rx_bar_off: 0x%" PRIx64 "\n", rx_bar_off); + PMD_INIT_LOG(DEBUG, "tx_bar_off: 0x%" PRIx64 "", tx_bar_off); + PMD_INIT_LOG(DEBUG, "rx_bar_off: 0x%" PRIx64 "", rx_bar_off); if (hw->is_pf && port == 0) { /* configure access to tx/rx vNIC BARs */ @@ -2912,12 +2912,12 @@ nfp_net_init(struct rte_eth_dev *eth_dev) &hw->hwqueues_area); if (!hwport0->hw_queues) { - printf("nfp_rtsym_map fails for net.qc\n"); + printf("nfp_rtsym_map fails for net.qc"); err = -EIO; goto dev_err_ctrl_map; } - PMD_INIT_LOG(DEBUG, "tx/rx bar address: 0x%p\n", + PMD_INIT_LOG(DEBUG, "tx/rx bar address: 0x%p", hwport0->hw_queues); } @@ -2998,7 +2998,7 @@ nfp_net_init(struct rte_eth_dev *eth_dev) } if (!is_valid_assigned_ether_addr((struct ether_addr *)&hw->mac_addr)) { - PMD_INIT_LOG(INFO, "Using random mac address for port %d\n", + PMD_INIT_LOG(INFO, "Using random mac address for port %d", port); /* Using random mac addresses for VFs */ eth_random_addr(&hw->mac_addr[0]); @@ -3124,7 +3124,7 @@ nfp_fw_upload(struct rte_pci_device *dev, struct nfp_nsp *nsp, char *card) sprintf(fw_name, "%s/%s.nffw", DEFAULT_FW_PATH, serial); - RTE_LOG(DEBUG, PMD, "Trying with fw file: %s\n", fw_name); + PMD_DRV_LOG(DEBUG, "Trying with fw file: %s", fw_name); fw_f = open(fw_name, O_RDONLY); if (fw_f > 0) goto read_fw; @@ -3132,34 +3132,34 @@ nfp_fw_upload(struct rte_pci_device *dev, struct nfp_nsp *nsp, char *card) /* Then try the PCI name */ sprintf(fw_name, "%s/pci-%s.nffw", DEFAULT_FW_PATH, dev->device.name); - RTE_LOG(DEBUG, PMD, "Trying with fw file: %s\n", fw_name); + PMD_DRV_LOG(DEBUG, "Trying with fw file: %s", fw_name); fw_f = open(fw_name, O_RDONLY); if (fw_f > 0) goto read_fw; /* Finally try the card type and media */ sprintf(fw_name, "%s/%s", DEFAULT_FW_PATH, card); - RTE_LOG(DEBUG, PMD, "Trying with fw file: %s\n", fw_name); + PMD_DRV_LOG(DEBUG, "Trying with fw file: %s", fw_name); fw_f = open(fw_name, O_RDONLY); if (fw_f < 0) { - RTE_LOG(INFO, PMD, "Firmware file %s not found.", fw_name); + PMD_DRV_LOG(INFO, "Firmware file %s not found.", fw_name); return -ENOENT; } read_fw: if (fstat(fw_f, &file_stat) < 0) { - RTE_LOG(INFO, PMD, "Firmware file %s size is unknown", fw_name); + PMD_DRV_LOG(INFO, "Firmware file %s size is unknown", fw_name); close(fw_f); return -ENOENT; } fsize = file_stat.st_size; - RTE_LOG(INFO, PMD, "Firmware file found at %s with size: %" PRIu64 "\n", + PMD_DRV_LOG(INFO, "Firmware file found at %s with size: %" PRIu64 "", fw_name, (uint64_t)fsize); fw_buf = malloc((size_t)fsize); if (!fw_buf) { - RTE_LOG(INFO, PMD, "malloc failed for fw buffer"); + PMD_DRV_LOG(INFO, "malloc failed for fw buffer"); close(fw_f); return -ENOMEM; } @@ -3167,7 +3167,7 @@ nfp_fw_upload(struct rte_pci_device *dev, struct nfp_nsp *nsp, char *card) bytes = read(fw_f, fw_buf, fsize); if (bytes != fsize) { - RTE_LOG(INFO, PMD, "Reading fw to buffer failed.\n" + PMD_DRV_LOG(INFO, "Reading fw to buffer failed." "Just %" PRIu64 " of %" PRIu64 " bytes read", (uint64_t)bytes, (uint64_t)fsize); free(fw_buf); @@ -3175,9 +3175,9 @@ nfp_fw_upload(struct rte_pci_device *dev, struct nfp_nsp *nsp, char *card) return -EIO; } - RTE_LOG(INFO, PMD, "Uploading the firmware ..."); + PMD_DRV_LOG(INFO, "Uploading the firmware ..."); nfp_nsp_load_fw(nsp, fw_buf, bytes); - RTE_LOG(INFO, PMD, "Done"); + PMD_DRV_LOG(INFO, "Done"); free(fw_buf); close(fw_f); @@ -3197,29 +3197,29 @@ nfp_fw_setup(struct rte_pci_device *dev, struct nfp_cpp *cpp, nfp_fw_model = nfp_hwinfo_lookup(hwinfo, "assembly.partno"); if (nfp_fw_model) { - RTE_LOG(INFO, PMD, "firmware model found: %s\n", nfp_fw_model); + PMD_DRV_LOG(INFO, "firmware model found: %s", nfp_fw_model); } else { - RTE_LOG(ERR, PMD, "firmware model NOT found\n"); + PMD_DRV_LOG(ERR, "firmware model NOT found"); return -EIO; } if (nfp_eth_table->count == 0 || nfp_eth_table->count > 8) { - RTE_LOG(ERR, PMD, "NFP ethernet table reports wrong ports: %u\n", + PMD_DRV_LOG(ERR, "NFP ethernet table reports wrong ports: %u", nfp_eth_table->count); return -EIO; } - RTE_LOG(INFO, PMD, "NFP ethernet port table reports %u ports\n", + PMD_DRV_LOG(INFO, "NFP ethernet port table reports %u ports", nfp_eth_table->count); - RTE_LOG(INFO, PMD, "Port speed: %u\n", nfp_eth_table->ports[0].speed); + PMD_DRV_LOG(INFO, "Port speed: %u", nfp_eth_table->ports[0].speed); sprintf(card_desc, "nic_%s_%dx%d.nffw", nfp_fw_model, nfp_eth_table->count, nfp_eth_table->ports[0].speed / 1000); nsp = nfp_nsp_open(cpp); if (!nsp) { - RTE_LOG(ERR, PMD, "NFP error when obtaining NSP handle\n"); + PMD_DRV_LOG(ERR, "NFP error when obtaining NSP handle"); return -EIO; } @@ -3248,25 +3248,25 @@ static int nfp_pf_pci_probe(struct rte_pci_driver *pci_drv __rte_unused, cpp = nfp_cpp_from_device_name(dev->device.name); if (!cpp) { - RTE_LOG(ERR, PMD, "A CPP handle can not be obtained"); + PMD_DRV_LOG(ERR, "A CPP handle can not be obtained"); ret = -EIO; goto error; } hwinfo = nfp_hwinfo_read(cpp); if (!hwinfo) { - RTE_LOG(ERR, PMD, "Error reading hwinfo table"); + PMD_DRV_LOG(ERR, "Error reading hwinfo table"); return -EIO; } nfp_eth_table = nfp_eth_read_ports(cpp); if (!nfp_eth_table) { - RTE_LOG(ERR, PMD, "Error reading NFP ethernet table\n"); + PMD_DRV_LOG(ERR, "Error reading NFP ethernet table"); return -EIO; } if (nfp_fw_setup(dev, cpp, nfp_eth_table, hwinfo)) { - RTE_LOG(INFO, PMD, "Error when uploading firmware\n"); + PMD_DRV_LOG(INFO, "Error when uploading firmware"); ret = -EIO; goto error; } @@ -3274,7 +3274,7 @@ static int nfp_pf_pci_probe(struct rte_pci_driver *pci_drv __rte_unused, /* Now the symbol table should be there */ sym_tbl = nfp_rtsym_table_read(cpp); if (!sym_tbl) { - RTE_LOG(ERR, PMD, "Something is wrong with the firmware" + PMD_DRV_LOG(ERR, "Something is wrong with the firmware" " symbol table"); ret = -EIO; goto error; @@ -3282,14 +3282,14 @@ static int nfp_pf_pci_probe(struct rte_pci_driver *pci_drv __rte_unused, total_ports = nfp_rtsym_read_le(sym_tbl, "nfd_cfg_pf0_num_ports", &err); if (total_ports != (int)nfp_eth_table->count) { - RTE_LOG(ERR, PMD, "Inconsistent number of ports\n"); + PMD_DRV_LOG(ERR, "Inconsistent number of ports"); ret = -EIO; goto error; } - PMD_INIT_LOG(INFO, "Total pf ports: %d\n", total_ports); + PMD_INIT_LOG(INFO, "Total pf ports: %d", total_ports); if (total_ports <= 0 || total_ports > 8) { - RTE_LOG(ERR, PMD, "nfd_cfg_pf0_num_ports symbol with wrong value"); + PMD_DRV_LOG(ERR, "nfd_cfg_pf0_num_ports symbol with wrong value"); ret = -ENODEV; goto error; }