Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/136362/?format=api
{ "id": 136362, "url": "http://patches.dpdk.org/api/patches/136362/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20240203041230.285599-8-stephen@networkplumber.org/", "project": { "id": 1, "url": "http://patches.dpdk.org/api/projects/1/?format=api", "name": "DPDK", "link_name": "dpdk", "list_id": "dev.dpdk.org", "list_email": "dev@dpdk.org", "web_url": "http://core.dpdk.org", "scm_url": "git://dpdk.org/dpdk", "webscm_url": "http://git.dpdk.org/dpdk", "list_archive_url": "https://inbox.dpdk.org/dev", "list_archive_url_format": "https://inbox.dpdk.org/dev/{}", "commit_url_format": "" }, "msgid": "<20240203041230.285599-8-stephen@networkplumber.org>", "list_archive_url": "https://inbox.dpdk.org/dev/20240203041230.285599-8-stephen@networkplumber.org", "date": "2024-02-03T04:11:40", "name": "[v7,07/19] net/nfb: use dynamic logtype", "commit_ref": null, "pull_url": null, "state": "accepted", "archived": true, "hash": "89d0c39a8c1a1ec140352df7e886d76f391b66f9", "submitter": { "id": 27, "url": "http://patches.dpdk.org/api/people/27/?format=api", "name": "Stephen Hemminger", "email": "stephen@networkplumber.org" }, "delegate": { "id": 24651, "url": "http://patches.dpdk.org/api/users/24651/?format=api", "username": "dmarchand", "first_name": "David", "last_name": "Marchand", "email": "david.marchand@redhat.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/patch/20240203041230.285599-8-stephen@networkplumber.org/mbox/", "series": [ { "id": 30996, "url": "http://patches.dpdk.org/api/series/30996/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=30996", "date": "2024-02-03T04:11:33", "name": "Replace uses of RTE_LOGTYPE_PMD", "version": 7, "mbox": "http://patches.dpdk.org/series/30996/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/136362/comments/", "check": "warning", "checks": "http://patches.dpdk.org/api/patches/136362/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@inbox.dpdk.org", "Delivered-To": "patchwork@inbox.dpdk.org", "Received": [ "from mails.dpdk.org (mails.dpdk.org [217.70.189.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id DAD1C43A57;\n\tSat, 3 Feb 2024 05:13:40 +0100 (CET)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id DFB6142E95;\n\tSat, 3 Feb 2024 05:13:32 +0100 (CET)", "from mail-pl1-f169.google.com (mail-pl1-f169.google.com\n [209.85.214.169])\n by mails.dpdk.org (Postfix) with ESMTP id 292C74064E\n for <dev@dpdk.org>; Sat, 3 Feb 2024 05:13:29 +0100 (CET)", "by mail-pl1-f169.google.com with SMTP id\n d9443c01a7336-1d71cb97937so26124435ad.3\n for <dev@dpdk.org>; Fri, 02 Feb 2024 20:13:29 -0800 (PST)", "from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141])\n by smtp.gmail.com with ESMTPSA id\n mf11-20020a170902fc8b00b001d94a3f398esm2361800plb.141.2024.02.02.20.13.27\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Fri, 02 Feb 2024 20:13:28 -0800 (PST)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1706933608;\n x=1707538408; darn=dpdk.org;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:from:to:cc:subject:date\n :message-id:reply-to;\n bh=p1CL05txfCQAMKHdoVGLoD22ggOCmji7+06/aHMlspA=;\n b=wC5I3pq7gAumFI8anAlLfqgRDSHj6AKD0EGo71mTTscZ81i6XBi6DgaVViQakj2QA+\n lQAJpZmwltm9vjzbvSGrpXoYe3VyUBpLF/VCGzpJwc/gMg/wglPlolqD28nEMSH8gaaF\n Ki7H8Z+slFx5rmIFmC6PMNcRwMoQ3wmCRQd2lm5EDCVS5muqHLETQoZt5TwRIEUypB/i\n IB4068+DPBobjQc7hNHjJN/Lsc9RzBleoemn1pOsD41Ji9BuQaY6pVoazadiCTI255lV\n ifJpon3e2hsme8gX8QLCDdCa02xZBvtczyD7Z6wkUjLTPEhgF6pyNFCMDO+lKKxE55v9\n aVnA==", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20230601; t=1706933608; x=1707538408;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc\n :subject:date:message-id:reply-to;\n bh=p1CL05txfCQAMKHdoVGLoD22ggOCmji7+06/aHMlspA=;\n b=qoWmGOI6ipduki0EKnS51wdoRSsd+VEE/oYrtmxNd+cbBmn5f32a2XWbB8WPdefIr3\n KJt2MXtTurdfENDwIWTQm2UGTj9SyPos4Oq2cEKwzz9di7R20AXZpCx7BnHrJydQe/WK\n 0KEkEQxz56ObZQJ56mJaH2tkgTgpl0cMhUzERTjPG9m4cXqEK2Y12aJqravEkIMLSeBt\n C50XmEkSJClC2F1Lu8rOKK6mWzY4jMJfhMtubIMe9Ef+YP386p1j0cdzlUlgH6gd9qDs\n SqkMquIqeWiKcF5jMDVeywxzgg7BdBvfTHlFRrxWiUgCvZjDDJLteHXi4VMJj33GQqkG\n D0BQ==", "X-Gm-Message-State": "AOJu0YzQbgBhMoMDP7eCiLQVSMBzDXrvW+t+GQE1DeNKhOa6y9QI3Grl\n BbXsGVe5vm+eYEyf7qSZzqSRNr+Wg5z0mPhBdiqYhKtQC8UwCLvzbXN6tGT35Q5Nqzw84n6v0xs\n uYro=", "X-Google-Smtp-Source": "\n AGHT+IGfxNoheoQKiqGcgaAPRSmvYowm0I3LQirtSZv/oputtre91Vk4pGFZB7W28BK2F+po8VL4hA==", "X-Received": "by 2002:a17:903:11cd:b0:1d8:fb03:a01e with SMTP id\n q13-20020a17090311cd00b001d8fb03a01emr12695609plh.32.1706933608349;\n Fri, 02 Feb 2024 20:13:28 -0800 (PST)", "X-Forwarded-Encrypted": "i=0;\n AJvYcCX00RaAWcLU88qy4sF++34zytdxunbo/GbPKwhMXFbtM6Ly4QuzaoUXVUsJrX+zO5+SSnvj7czdiBSVQz8YZHDiqQ6/9+MHaxgj/Z8C8sT1AUwQN2k3", "From": "Stephen Hemminger <stephen@networkplumber.org>", "To": "dev@dpdk.org", "Cc": "Stephen Hemminger <stephen@networkplumber.org>,\n Martin Spinler <spinler@cesnet.cz>, Rastislav Cernay <cernay@netcope.com>", "Subject": "[PATCH v7 07/19] net/nfb: use dynamic logtype", "Date": "Fri, 2 Feb 2024 20:11:40 -0800", "Message-ID": "<20240203041230.285599-8-stephen@networkplumber.org>", "X-Mailer": "git-send-email 2.43.0", "In-Reply-To": "<20240203041230.285599-1-stephen@networkplumber.org>", "References": "<20231213014408.612051-1-stephen@networkplumber.org>\n <20240203041230.285599-1-stephen@networkplumber.org>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dev>,\n <mailto:dev-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://mails.dpdk.org/archives/dev/>", "List-Post": "<mailto:dev@dpdk.org>", "List-Help": "<mailto:dev-request@dpdk.org?subject=help>", "List-Subscribe": "<https://mails.dpdk.org/listinfo/dev>,\n <mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org" }, "content": "All drivers should be using dynamic logtype.\n\nFixes: 6435f9a0ac22 (\"net/nfb: add new netcope driver\")\nSigned-off-by: Stephen Hemminger <stephen@networkplumber.org>\n---\n drivers/net/nfb/nfb.h | 5 +++++\n drivers/net/nfb/nfb_ethdev.c | 20 +++++++++-----------\n drivers/net/nfb/nfb_rx.c | 10 +++++-----\n drivers/net/nfb/nfb_rx.h | 2 +-\n drivers/net/nfb/nfb_tx.c | 10 +++++-----\n drivers/net/nfb/nfb_tx.h | 2 +-\n 6 files changed, 26 insertions(+), 23 deletions(-)", "diff": "diff --git a/drivers/net/nfb/nfb.h b/drivers/net/nfb/nfb.h\nindex 7dc5bd29e44c..45226ee3d938 100644\n--- a/drivers/net/nfb/nfb.h\n+++ b/drivers/net/nfb/nfb.h\n@@ -12,6 +12,11 @@\n #include <netcope/rxmac.h>\n #include <netcope/txmac.h>\n \n+extern int nfb_logtype;\n+#define NFB_LOG(level, fmt, args...) \\\n+\trte_log(RTE_LOG_ ## level, nfb_logtype, \"%s(): \" fmt \"\\n\", \\\n+\t\t__func__, ## args)\n+\n #include \"nfb_rx.h\"\n #include \"nfb_tx.h\"\n \ndiff --git a/drivers/net/nfb/nfb_ethdev.c b/drivers/net/nfb/nfb_ethdev.c\nindex defd118bd0ee..da9e4167ea69 100644\n--- a/drivers/net/nfb/nfb_ethdev.c\n+++ b/drivers/net/nfb/nfb_ethdev.c\n@@ -192,8 +192,7 @@ nfb_eth_dev_configure(struct rte_eth_dev *dev __rte_unused)\n \t\t\t\t(&nfb_timestamp_dynfield_offset,\n \t\t\t\t&nfb_timestamp_rx_dynflag);\n \t\tif (ret != 0) {\n-\t\t\tRTE_LOG(ERR, PMD, \"Cannot register Rx timestamp\"\n-\t\t\t\t\t\" field/flag %d\\n\", ret);\n+\t\t\tNFB_LOG(ERR, \"Cannot register Rx timestamp field/flag %d\", ret);\n \t\t\tnfb_close(internals->nfb);\n \t\t\treturn -rte_errno;\n \t\t}\n@@ -520,7 +519,7 @@ nfb_eth_dev_init(struct rte_eth_dev *dev)\n \tstruct rte_ether_addr eth_addr_init;\n \tstruct rte_kvargs *kvlist;\n \n-\tRTE_LOG(INFO, PMD, \"Initializing NFB device (\" PCI_PRI_FMT \")\\n\",\n+\tNFB_LOG(INFO, \"Initializing NFB device (\" PCI_PRI_FMT \")\",\n \t\tpci_addr->domain, pci_addr->bus, pci_addr->devid,\n \t\tpci_addr->function);\n \n@@ -536,7 +535,7 @@ nfb_eth_dev_init(struct rte_eth_dev *dev)\n \t\tkvlist = rte_kvargs_parse(dev->device->devargs->args,\n \t\t\t\t\t\tVALID_KEYS);\n \t\tif (kvlist == NULL) {\n-\t\t\tRTE_LOG(ERR, PMD, \"Failed to parse device arguments %s\",\n+\t\t\tNFB_LOG(ERR, \"Failed to parse device arguments %s\",\n \t\t\t\tdev->device->devargs->args);\n \t\t\trte_kvargs_free(kvlist);\n \t\t\treturn -EINVAL;\n@@ -551,14 +550,14 @@ nfb_eth_dev_init(struct rte_eth_dev *dev)\n \t */\n \tinternals->nfb = nfb_open(internals->nfb_dev);\n \tif (internals->nfb == NULL) {\n-\t\tRTE_LOG(ERR, PMD, \"nfb_open(): failed to open %s\",\n+\t\tNFB_LOG(ERR, \"nfb_open(): failed to open %s\",\n \t\t\tinternals->nfb_dev);\n \t\treturn -EINVAL;\n \t}\n \tdata->nb_rx_queues = ndp_get_rx_queue_available_count(internals->nfb);\n \tdata->nb_tx_queues = ndp_get_tx_queue_available_count(internals->nfb);\n \n-\tRTE_LOG(INFO, PMD, \"Available NDP queues RX: %u TX: %u\\n\",\n+\tNFB_LOG(INFO, \"Available NDP queues RX: %u TX: %u\",\n \t\tdata->nb_rx_queues, data->nb_tx_queues);\n \n \tnfb_nc_rxmac_init(internals->nfb,\n@@ -583,7 +582,7 @@ nfb_eth_dev_init(struct rte_eth_dev *dev)\n \tdata->mac_addrs = rte_zmalloc(data->name,\n \t\tsizeof(struct rte_ether_addr) * mac_count, RTE_CACHE_LINE_SIZE);\n \tif (data->mac_addrs == NULL) {\n-\t\tRTE_LOG(ERR, PMD, \"Could not alloc space for MAC address!\\n\");\n+\t\tNFB_LOG(ERR, \"Could not alloc space for MAC address\");\n \t\tnfb_close(internals->nfb);\n \t\treturn -EINVAL;\n \t}\n@@ -601,8 +600,7 @@ nfb_eth_dev_init(struct rte_eth_dev *dev)\n \n \tdev->data->dev_flags |= RTE_ETH_DEV_AUTOFILL_QUEUE_XSTATS;\n \n-\tRTE_LOG(INFO, PMD, \"NFB device (\"\n-\t\tPCI_PRI_FMT \") successfully initialized\\n\",\n+\tNFB_LOG(INFO, \"NFB device (\" PCI_PRI_FMT \") successfully initialized\",\n \t\tpci_addr->domain, pci_addr->bus, pci_addr->devid,\n \t\tpci_addr->function);\n \n@@ -626,8 +624,7 @@ nfb_eth_dev_uninit(struct rte_eth_dev *dev)\n \n \tnfb_eth_dev_close(dev);\n \n-\tRTE_LOG(INFO, PMD, \"NFB device (\"\n-\t\tPCI_PRI_FMT \") successfully uninitialized\\n\",\n+\tNFB_LOG(INFO, \"NFB device (\" PCI_PRI_FMT \") successfully uninitialized\",\n \t\tpci_addr->domain, pci_addr->bus, pci_addr->devid,\n \t\tpci_addr->function);\n \n@@ -690,3 +687,4 @@ static struct rte_pci_driver nfb_eth_driver = {\n RTE_PMD_REGISTER_PCI(RTE_NFB_DRIVER_NAME, nfb_eth_driver);\n RTE_PMD_REGISTER_PCI_TABLE(RTE_NFB_DRIVER_NAME, nfb_pci_id_table);\n RTE_PMD_REGISTER_KMOD_DEP(RTE_NFB_DRIVER_NAME, \"* nfb\");\n+RTE_LOG_REGISTER_DEFAULT(nfb_logtype, NOTICE);\ndiff --git a/drivers/net/nfb/nfb_rx.c b/drivers/net/nfb/nfb_rx.c\nindex 8a9b232305f2..f72afafe8fb9 100644\n--- a/drivers/net/nfb/nfb_rx.c\n+++ b/drivers/net/nfb/nfb_rx.c\n@@ -6,8 +6,8 @@\n \n #include <rte_kvargs.h>\n \n-#include \"nfb_rx.h\"\n #include \"nfb.h\"\n+#include \"nfb_rx.h\"\n \n uint64_t nfb_timestamp_rx_dynflag;\n int nfb_timestamp_dynfield_offset = -1;\n@@ -19,7 +19,7 @@ nfb_eth_rx_queue_start(struct rte_eth_dev *dev, uint16_t rxq_id)\n \tint ret;\n \n \tif (rxq->queue == NULL) {\n-\t\tRTE_LOG(ERR, PMD, \"RX NDP queue is NULL!\\n\");\n+\t\tNFB_LOG(ERR, \"RX NDP queue is NULL\");\n \t\treturn -EINVAL;\n \t}\n \n@@ -40,7 +40,7 @@ nfb_eth_rx_queue_stop(struct rte_eth_dev *dev, uint16_t rxq_id)\n \tint ret;\n \n \tif (rxq->queue == NULL) {\n-\t\tRTE_LOG(ERR, PMD, \"RX NDP queue is NULL!\\n\");\n+\t\tNFB_LOG(ERR, \"RX NDP queue is NULL\");\n \t\treturn -EINVAL;\n \t}\n \n@@ -70,8 +70,8 @@ nfb_eth_rx_queue_setup(struct rte_eth_dev *dev,\n \t\t\tRTE_CACHE_LINE_SIZE, socket_id);\n \n \tif (rxq == NULL) {\n-\t\tRTE_LOG(ERR, PMD, \"rte_zmalloc_socket() failed for rx queue id \"\n-\t\t\t\t\"%\" PRIu16 \"!\\n\", rx_queue_id);\n+\t\tNFB_LOG(ERR, \"rte_zmalloc_socket() failed for rx queue id %\" PRIu16,\n+\t\t\trx_queue_id);\n \t\treturn -ENOMEM;\n \t}\n \ndiff --git a/drivers/net/nfb/nfb_rx.h b/drivers/net/nfb/nfb_rx.h\nindex b618682e1393..2802f17091a0 100644\n--- a/drivers/net/nfb/nfb_rx.h\n+++ b/drivers/net/nfb/nfb_rx.h\n@@ -156,7 +156,7 @@ nfb_eth_ndp_rx(void *queue,\n \tstruct rte_mbuf *mbufs[nb_pkts];\n \n \tif (unlikely(ndp->queue == NULL || nb_pkts == 0)) {\n-\t\tRTE_LOG(ERR, PMD, \"RX invalid arguments!\\n\");\n+\t\tNFB_LOG(ERR, \"RX invalid arguments\");\n \t\treturn 0;\n \t}\n \ndiff --git a/drivers/net/nfb/nfb_tx.c b/drivers/net/nfb/nfb_tx.c\nindex d49fc324e76b..a1318a420580 100644\n--- a/drivers/net/nfb/nfb_tx.c\n+++ b/drivers/net/nfb/nfb_tx.c\n@@ -4,8 +4,8 @@\n * All rights reserved.\n */\n \n-#include \"nfb_tx.h\"\n #include \"nfb.h\"\n+#include \"nfb_tx.h\"\n \n int\n nfb_eth_tx_queue_start(struct rte_eth_dev *dev, uint16_t txq_id)\n@@ -14,7 +14,7 @@ nfb_eth_tx_queue_start(struct rte_eth_dev *dev, uint16_t txq_id)\n \tint ret;\n \n \tif (txq->queue == NULL) {\n-\t\tRTE_LOG(ERR, PMD, \"RX NDP queue is NULL!\\n\");\n+\t\tNFB_LOG(ERR, \"RX NDP queue is NULL\");\n \t\treturn -EINVAL;\n \t}\n \n@@ -35,7 +35,7 @@ nfb_eth_tx_queue_stop(struct rte_eth_dev *dev, uint16_t txq_id)\n \tint ret;\n \n \tif (txq->queue == NULL) {\n-\t\tRTE_LOG(ERR, PMD, \"TX NDP queue is NULL!\\n\");\n+\t\tNFB_LOG(ERR, \"TX NDP queue is NULL\");\n \t\treturn -EINVAL;\n \t}\n \n@@ -62,8 +62,8 @@ nfb_eth_tx_queue_setup(struct rte_eth_dev *dev,\n \t\tRTE_CACHE_LINE_SIZE, socket_id);\n \n \tif (txq == NULL) {\n-\t\tRTE_LOG(ERR, PMD, \"rte_zmalloc_socket() failed for tx queue id \"\n-\t\t\t\"%\" PRIu16 \"!\\n\", tx_queue_id);\n+\t\tNFB_LOG(ERR, \"rte_zmalloc_socket() failed for tx queue id %\" PRIu16,\n+\t\t\ttx_queue_id);\n \t\treturn -ENOMEM;\n \t}\n \ndiff --git a/drivers/net/nfb/nfb_tx.h b/drivers/net/nfb/nfb_tx.h\nindex 910020e9e96f..f107cf914bbd 100644\n--- a/drivers/net/nfb/nfb_tx.h\n+++ b/drivers/net/nfb/nfb_tx.h\n@@ -140,7 +140,7 @@ nfb_eth_ndp_tx(void *queue,\n \t\treturn 0;\n \n \tif (unlikely(ndp->queue == NULL)) {\n-\t\tRTE_LOG(ERR, PMD, \"TX invalid arguments!\\n\");\n+\t\tNFB_LOG(ERR, \"TX invalid arguments\");\n \t\treturn 0;\n \t}\n \n", "prefixes": [ "v7", "07/19" ] }