Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/57801/?format=api
{ "id": 57801, "url": "https://patches.dpdk.org/api/patches/57801/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/patch/20190822055400.30119-14-ajit.khaparde@broadcom.com/", "project": { "id": 1, "url": "https://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": "<20190822055400.30119-14-ajit.khaparde@broadcom.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20190822055400.30119-14-ajit.khaparde@broadcom.com", "date": "2019-08-22T05:54:00", "name": "[13/13] net/bnxt: reduce verbosity of logs", "commit_ref": null, "pull_url": null, "state": "changes-requested", "archived": true, "hash": "c3df1783acb128a18996009ef6ce0a77cb39b315", "submitter": { "id": 501, "url": "https://patches.dpdk.org/api/people/501/?format=api", "name": "Ajit Khaparde", "email": "ajit.khaparde@broadcom.com" }, "delegate": { "id": 319, "url": "https://patches.dpdk.org/api/users/319/?format=api", "username": "fyigit", "first_name": "Ferruh", "last_name": "Yigit", "email": "ferruh.yigit@amd.com" }, "mbox": "https://patches.dpdk.org/project/dpdk/patch/20190822055400.30119-14-ajit.khaparde@broadcom.com/mbox/", "series": [ { "id": 6099, "url": "https://patches.dpdk.org/api/series/6099/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/list/?series=6099", "date": "2019-08-22T05:53:47", "name": "bnxt patchset to support device error recovery", "version": 1, "mbox": "https://patches.dpdk.org/series/6099/mbox/" } ], "comments": "https://patches.dpdk.org/api/patches/57801/comments/", "check": "fail", "checks": "https://patches.dpdk.org/api/patches/57801/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@dpdk.org", "Delivered-To": "patchwork@dpdk.org", "Received": [ "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id A98ED1BF67;\n\tThu, 22 Aug 2019 07:54:31 +0200 (CEST)", "from rnd-relay.smtp.broadcom.com (rnd-relay.smtp.broadcom.com\n\t[192.19.229.170]) by dpdk.org (Postfix) with ESMTP id BB2581BF33\n\tfor <dev@dpdk.org>; Thu, 22 Aug 2019 07:54:09 +0200 (CEST)", "from nis-sj1-27.broadcom.com (nis-sj1-27.lvn.broadcom.net\n\t[10.75.144.136])\n\tby rnd-relay.smtp.broadcom.com (Postfix) with ESMTP id EC00E30C210;\n\tWed, 21 Aug 2019 22:54:05 -0700 (PDT)", "from localhost.localdomain (unknown [10.230.30.225])\n\tby nis-sj1-27.broadcom.com (Postfix) with ESMTP id CCE37AC0790;\n\tWed, 21 Aug 2019 22:54:05 -0700 (PDT)" ], "DKIM-Filter": "OpenDKIM Filter v2.10.3 rnd-relay.smtp.broadcom.com EC00E30C210", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=broadcom.com;\n\ts=dkimrelay; t=1566453246;\n\tbh=/OR0f6Li2sn8gkqQTH+RBg/fglZjCgLRUzP2sgyawbY=;\n\th=From:To:Cc:Subject:Date:In-Reply-To:References:From;\n\tb=YnW2ljlfbzVPU08iGgp0TmZ8kpPxEvT/32LPAy5Xo3AJOo3STcDbPJQvT2YBNxl/Z\n\tUMNsf2gJFd8v3bOace51vYy0gMqgkSkxVZzYP1QL+lS2DBlb6j6htyQgk5WVa7KoDI\n\tPkoyQd3vXXhLH23lkIyItLWUNfsUvpGDSKYcX5ak=", "From": "Ajit Khaparde <ajit.khaparde@broadcom.com>", "To": "dev@dpdk.org", "Cc": "ferruh.yigit@intel.com, Kalesh AP <kalesh-anakkur.purayil@broadcom.com>, \n\tLance Richardson <lance.richardson@broadcom.com>,\n\tSomnath Kotur <somnath.kotur@broadcom.com>", "Date": "Wed, 21 Aug 2019 22:54:00 -0700", "Message-Id": "<20190822055400.30119-14-ajit.khaparde@broadcom.com>", "X-Mailer": "git-send-email 2.20.1 (Apple Git-117)", "In-Reply-To": "<20190822055400.30119-1-ajit.khaparde@broadcom.com>", "References": "<20190822055400.30119-1-ajit.khaparde@broadcom.com>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Subject": "[dpdk-dev] [PATCH 13/13] net/bnxt: reduce verbosity of logs", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.15", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dev>,\n\t<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\t<mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "From: Kalesh AP <kalesh-anakkur.purayil@broadcom.com>\n\nWhen IOMMU is available, EAL picks IOVA as VA as the default IOVA mode.\nThis causes the bnxt driver to log warning messages saying\n\"Memzone physical address same as virtual.\" and \"Using rte_mem_virt2iova()\"\nduring load.\n\nReduce the verbosity of logs to DEBUG.\n\nSigned-off-by: Kalesh AP <kalesh-anakkur.purayil@broadcom.com>\nReviewed-by: Lance Richardson <lance.richardson@broadcom.com>\nReviewed-by: Somnath Kotur <somnath.kotur@broadcom.com>\nReviewed-by: Ajit Khaparde <ajit.khaparde@broadcom.com>\n---\n drivers/net/bnxt/bnxt_ethdev.c | 21 +++++++++------------\n drivers/net/bnxt/bnxt_ring.c | 7 +++----\n drivers/net/bnxt/bnxt_vnic.c | 7 +++----\n 3 files changed, 15 insertions(+), 20 deletions(-)", "diff": "diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c\nindex 095395dae..13f1ff6fb 100644\n--- a/drivers/net/bnxt/bnxt_ethdev.c\n+++ b/drivers/net/bnxt/bnxt_ethdev.c\n@@ -3893,10 +3893,9 @@ static int bnxt_alloc_ctx_mem_blk(__rte_unused struct bnxt *bp,\n \t\tmemset(mz->addr, 0, mz->len);\n \t\tmz_phys_addr = mz->iova;\n \t\tif ((unsigned long)mz->addr == mz_phys_addr) {\n-\t\t\tPMD_DRV_LOG(WARNING,\n-\t\t\t\t\"Memzone physical address same as virtual.\\n\");\n-\t\t\tPMD_DRV_LOG(WARNING,\n-\t\t\t\t \"Using rte_mem_virt2iova()\\n\");\n+\t\t\tPMD_DRV_LOG(DEBUG,\n+\t\t\t\t \"physical address same as virtual\\n\");\n+\t\t\tPMD_DRV_LOG(DEBUG, \"Using rte_mem_virt2iova()\\n\");\n \t\t\tmz_phys_addr = rte_mem_virt2iova(mz->addr);\n \t\t\tif (mz_phys_addr == RTE_BAD_IOVA) {\n \t\t\t\tPMD_DRV_LOG(ERR,\n@@ -3929,10 +3928,9 @@ static int bnxt_alloc_ctx_mem_blk(__rte_unused struct bnxt *bp,\n \tmemset(mz->addr, 0, mz->len);\n \tmz_phys_addr = mz->iova;\n \tif ((unsigned long)mz->addr == mz_phys_addr) {\n-\t\tPMD_DRV_LOG(WARNING,\n+\t\tPMD_DRV_LOG(DEBUG,\n \t\t\t \"Memzone physical address same as virtual.\\n\");\n-\t\tPMD_DRV_LOG(WARNING,\n-\t\t\t \"Using rte_mem_virt2iova()\\n\");\n+\t\tPMD_DRV_LOG(DEBUG, \"Using rte_mem_virt2iova()\\n\");\n \t\tfor (sz = 0; sz < mem_size; sz += BNXT_PAGE_SIZE)\n \t\t\trte_mem_lock_page(((char *)mz->addr) + sz);\n \t\tmz_phys_addr = rte_mem_virt2iova(mz->addr);\n@@ -4120,9 +4118,9 @@ static int bnxt_alloc_stats_mem(struct bnxt *bp)\n \tmemset(mz->addr, 0, mz->len);\n \tmz_phys_addr = mz->iova;\n \tif ((unsigned long)mz->addr == mz_phys_addr) {\n-\t\tPMD_DRV_LOG(WARNING,\n+\t\tPMD_DRV_LOG(DEBUG,\n \t\t\t \"Memzone physical address same as virtual.\\n\");\n-\t\tPMD_DRV_LOG(WARNING,\n+\t\tPMD_DRV_LOG(DEBUG,\n \t\t\t \"Using rte_mem_virt2iova()\\n\");\n \t\tmz_phys_addr = rte_mem_virt2iova(mz->addr);\n \t\tif (mz_phys_addr == RTE_BAD_IOVA) {\n@@ -4158,10 +4156,9 @@ static int bnxt_alloc_stats_mem(struct bnxt *bp)\n \tmemset(mz->addr, 0, mz->len);\n \tmz_phys_addr = mz->iova;\n \tif ((unsigned long)mz->addr == mz_phys_addr) {\n-\t\tPMD_DRV_LOG(WARNING,\n+\t\tPMD_DRV_LOG(DEBUG,\n \t\t\t \"Memzone physical address same as virtual\\n\");\n-\t\tPMD_DRV_LOG(WARNING,\n-\t\t\t \"Using rte_mem_virt2iova()\\n\");\n+\t\tPMD_DRV_LOG(DEBUG, \"Using rte_mem_virt2iova()\\n\");\n \t\tmz_phys_addr = rte_mem_virt2iova(mz->addr);\n \t\tif (mz_phys_addr == RTE_BAD_IOVA) {\n \t\t\tPMD_DRV_LOG(ERR,\ndiff --git a/drivers/net/bnxt/bnxt_ring.c b/drivers/net/bnxt/bnxt_ring.c\nindex f19865c83..2f57e038a 100644\n--- a/drivers/net/bnxt/bnxt_ring.c\n+++ b/drivers/net/bnxt/bnxt_ring.c\n@@ -212,10 +212,9 @@ int bnxt_alloc_rings(struct bnxt *bp, uint16_t qidx,\n \tmz_phys_addr_base = mz->iova;\n \tmz_phys_addr = mz->iova;\n \tif ((unsigned long)mz->addr == mz_phys_addr_base) {\n-\t\tPMD_DRV_LOG(WARNING,\n-\t\t\t\"Memzone physical address same as virtual.\\n\");\n-\t\tPMD_DRV_LOG(WARNING,\n-\t\t\t\"Using rte_mem_virt2iova()\\n\");\n+\t\tPMD_DRV_LOG(DEBUG,\n+\t\t\t \"Memzone physical address same as virtual.\\n\");\n+\t\tPMD_DRV_LOG(DEBUG, \"Using rte_mem_virt2iova()\\n\");\n \t\tfor (sz = 0; sz < total_alloc_len; sz += getpagesize())\n \t\t\trte_mem_lock_page(((char *)mz->addr) + sz);\n \t\tmz_phys_addr_base = rte_mem_virt2iova(mz->addr);\ndiff --git a/drivers/net/bnxt/bnxt_vnic.c b/drivers/net/bnxt/bnxt_vnic.c\nindex 98415633e..9ea99388b 100644\n--- a/drivers/net/bnxt/bnxt_vnic.c\n+++ b/drivers/net/bnxt/bnxt_vnic.c\n@@ -150,10 +150,9 @@ int bnxt_alloc_vnic_attributes(struct bnxt *bp)\n \t}\n \tmz_phys_addr = mz->iova;\n \tif ((unsigned long)mz->addr == mz_phys_addr) {\n-\t\tPMD_DRV_LOG(WARNING,\n-\t\t\t\"Memzone physical address same as virtual.\\n\");\n-\t\tPMD_DRV_LOG(WARNING,\n-\t\t\t\"Using rte_mem_virt2iova()\\n\");\n+\t\tPMD_DRV_LOG(DEBUG,\n+\t\t\t \"Memzone physical address same as virtual.\\n\");\n+\t\tPMD_DRV_LOG(DEBUG, \"Using rte_mem_virt2iova()\\n\");\n \t\tmz_phys_addr = rte_mem_virt2iova(mz->addr);\n \t\tif (mz_phys_addr == RTE_BAD_IOVA) {\n \t\t\tPMD_DRV_LOG(ERR,\n", "prefixes": [ "13/13" ] }