Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/60381/?format=api
{ "id": 60381, "url": "http://patches.dpdk.org/api/patches/60381/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20191002012335.85324-13-ajit.khaparde@broadcom.com/", "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": "<20191002012335.85324-13-ajit.khaparde@broadcom.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20191002012335.85324-13-ajit.khaparde@broadcom.com", "date": "2019-10-02T01:23:32", "name": "[v3,12/15] net/bnxt: reduce verbosity of logs", "commit_ref": null, "pull_url": null, "state": "accepted", "archived": true, "hash": "ab6b1235b01cdbe9af59bc6eff937af0bb2cbcd3", "submitter": { "id": 501, "url": "http://patches.dpdk.org/api/people/501/?format=api", "name": "Ajit Khaparde", "email": "ajit.khaparde@broadcom.com" }, "delegate": { "id": 319, "url": "http://patches.dpdk.org/api/users/319/?format=api", "username": "fyigit", "first_name": "Ferruh", "last_name": "Yigit", "email": "ferruh.yigit@amd.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/patch/20191002012335.85324-13-ajit.khaparde@broadcom.com/mbox/", "series": [ { "id": 6660, "url": "http://patches.dpdk.org/api/series/6660/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=6660", "date": "2019-10-02T01:23:24", "name": "bnxt patchset to support device error recovery", "version": 3, "mbox": "http://patches.dpdk.org/series/6660/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/60381/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/60381/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 8DCD01BEE6;\n\tWed, 2 Oct 2019 03:24:02 +0200 (CEST)", "from rnd-relay.smtp.broadcom.com (unknown [192.19.229.170])\n\tby dpdk.org (Postfix) with ESMTP id 253621B9BF\n\tfor <dev@dpdk.org>; Wed, 2 Oct 2019 03:23:48 +0200 (CEST)", "from mail-irv-17.broadcom.com (mail-irv-17.lvn.broadcom.net\n\t[10.75.242.48])\n\tby rnd-relay.smtp.broadcom.com (Postfix) with ESMTP id B70B630CC45;\n\tTue, 1 Oct 2019 18:22:32 -0700 (PDT)", "from localhost.localdomain (unknown [10.230.30.225])\n\tby mail-irv-17.broadcom.com (Postfix) with ESMTP id 39510140069;\n\tTue, 1 Oct 2019 18:23:42 -0700 (PDT)" ], "DKIM-Filter": "OpenDKIM Filter v2.10.3 rnd-relay.smtp.broadcom.com B70B630CC45", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=broadcom.com;\n\ts=dkimrelay; t=1569979352;\n\tbh=SbFwFFaaNbMR5T7RfuY603xQuRsm5s7zYy7DnCJTUj8=;\n\th=From:To:Cc:Subject:Date:In-Reply-To:References:From;\n\tb=k8QGoxP7vJ7l94xdeRtJR7Wf8PfRT5hXEGYar7VjVFE5Mi9jKKDyWzuVIeZ8xUaqd\n\tLbM0X71wt9DvyyNjGZaO4pFSP7F/iRgdhxBlyC9ExZaOnZVZvTkQ69i74qZ19UMF/Q\n\tD/D7L02xwcXB7OkzNgjom5wMesZCs1hVq6dlmZtE=", "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": "Tue, 1 Oct 2019 18:23:32 -0700", "Message-Id": "<20191002012335.85324-13-ajit.khaparde@broadcom.com>", "X-Mailer": "git-send-email 2.20.1 (Apple Git-117)", "In-Reply-To": "<20191002012335.85324-1-ajit.khaparde@broadcom.com>", "References": "<7c08999f-13f3-5fb6-39a2-557a0884bfde@intel.com>\n\t<20191002012335.85324-1-ajit.khaparde@broadcom.com>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Subject": "[dpdk-dev] [PATCH v3 12/15] 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. Reduced couple of other\nlogs level to DEBUG as well.\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_cpr.c | 4 ++--\n drivers/net/bnxt/bnxt_ethdev.c | 21 +++++++++------------\n drivers/net/bnxt/bnxt_filter.c | 2 +-\n drivers/net/bnxt/bnxt_ring.c | 7 +++----\n drivers/net/bnxt/bnxt_vnic.c | 7 +++----\n 5 files changed, 18 insertions(+), 23 deletions(-)", "diff": "diff --git a/drivers/net/bnxt/bnxt_cpr.c b/drivers/net/bnxt/bnxt_cpr.c\nindex 00ca6bbd1f..4817672ef0 100644\n--- a/drivers/net/bnxt/bnxt_cpr.c\n+++ b/drivers/net/bnxt/bnxt_cpr.c\n@@ -137,7 +137,7 @@ void bnxt_handle_async_event(struct bnxt *bp,\n \t\tbnxt_schedule_fw_health_check(bp);\n \t\tbreak;\n \tdefault:\n-\t\tPMD_DRV_LOG(INFO, \"handle_async_event id = 0x%x\\n\", event_id);\n+\t\tPMD_DRV_LOG(DEBUG, \"handle_async_event id = 0x%x\\n\", event_id);\n \t\tbreak;\n \t}\n }\n@@ -254,7 +254,7 @@ int bnxt_event_hwrm_resp_handler(struct bnxt *bp, struct cmpl_base *cmp)\n \t\tbreak;\n \tdefault:\n \t\t/* Ignore any other events */\n-\t\tPMD_DRV_LOG(INFO, \"Ignoring %02x completion\\n\", CMP_TYPE(cmp));\n+\t\tPMD_DRV_LOG(DEBUG, \"Ignoring %02x completion\\n\", CMP_TYPE(cmp));\n \t\tbreak;\n \t}\n \ndiff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c\nindex fbd00d1c72..7c3ef93253 100644\n--- a/drivers/net/bnxt/bnxt_ethdev.c\n+++ b/drivers/net/bnxt/bnxt_ethdev.c\n@@ -3976,10 +3976,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@@ -4012,10 +4011,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@@ -4203,9 +4201,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@@ -4241,10 +4239,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_filter.c b/drivers/net/bnxt/bnxt_filter.c\nindex 0aed29fb08..34db988181 100644\n--- a/drivers/net/bnxt/bnxt_filter.c\n+++ b/drivers/net/bnxt/bnxt_filter.c\n@@ -119,7 +119,7 @@ void bnxt_free_filter_mem(struct bnxt *bp)\n \t\tfilter = &bp->filter_info[i];\n \t\tif (filter->fw_l2_filter_id != ((uint64_t)-1) &&\n \t\t filter->filter_type == HWRM_CFA_L2_FILTER) {\n-\t\t\tPMD_DRV_LOG(ERR, \"L2 filter is not free\\n\");\n+\t\t\tPMD_DRV_LOG(DEBUG, \"L2 filter is not free\\n\");\n \t\t\t/* Call HWRM to try to free filter again */\n \t\t\trc = bnxt_hwrm_clear_l2_filter(bp, filter);\n \t\t\tif (rc)\ndiff --git a/drivers/net/bnxt/bnxt_ring.c b/drivers/net/bnxt/bnxt_ring.c\nindex f19865c832..2f57e038a0 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 98415633e4..9ea99388b7 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": [ "v3", "12/15" ] }