Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/511032/?format=api
{ "id": 511032, "url": "http://patchwork.ozlabs.org/api/patches/511032/?format=api", "web_url": "http://patchwork.ozlabs.org/project/intel-wired-lan/patch/1440616461-110869-12-git-send-email-catherine.sullivan@intel.com/", "project": { "id": 46, "url": "http://patchwork.ozlabs.org/api/projects/46/?format=api", "name": "Intel Wired Ethernet development", "link_name": "intel-wired-lan", "list_id": "intel-wired-lan.osuosl.org", "list_email": "intel-wired-lan@osuosl.org", "web_url": "", "scm_url": "", "webscm_url": "", "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<1440616461-110869-12-git-send-email-catherine.sullivan@intel.com>", "list_archive_url": null, "date": "2015-08-26T19:14:17", "name": "[next,S11,11/15] i40e/i40evf: give up the __func__", "commit_ref": null, "pull_url": null, "state": "accepted", "archived": false, "hash": "83fc0552f4eb4bf45a3f1052f43d4350e513cc5b", "submitter": { "id": 13931, "url": "http://patchwork.ozlabs.org/api/people/13931/?format=api", "name": "Catherine Sullivan", "email": "catherine.sullivan@intel.com" }, "delegate": { "id": 68, "url": "http://patchwork.ozlabs.org/api/users/68/?format=api", "username": "jtkirshe", "first_name": "Jeff", "last_name": "Kirsher", "email": "jeffrey.t.kirsher@intel.com" }, "mbox": "http://patchwork.ozlabs.org/project/intel-wired-lan/patch/1440616461-110869-12-git-send-email-catherine.sullivan@intel.com/mbox/", "series": [], "comments": "http://patchwork.ozlabs.org/api/patches/511032/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/511032/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<intel-wired-lan-bounces@lists.osuosl.org>", "X-Original-To": [ "incoming@patchwork.ozlabs.org", "intel-wired-lan@lists.osuosl.org" ], "Delivered-To": [ "patchwork-incoming@bilbo.ozlabs.org", "intel-wired-lan@lists.osuosl.org" ], "Received": [ "from fraxinus.osuosl.org (smtp4.osuosl.org [140.211.166.137])\n\tby ozlabs.org (Postfix) with ESMTP id 06AFD14012C\n\tfor <incoming@patchwork.ozlabs.org>;\n\tThu, 27 Aug 2015 05:13:57 +1000 (AEST)", "from localhost (localhost [127.0.0.1])\n\tby fraxinus.osuosl.org (Postfix) with ESMTP id 55532A2D4B;\n\tWed, 26 Aug 2015 19:13:57 +0000 (UTC)", "from fraxinus.osuosl.org ([127.0.0.1])\n\tby localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024)\n\twith ESMTP id I2WMkbsZ30QZ; Wed, 26 Aug 2015 19:13:56 +0000 (UTC)", "from ash.osuosl.org (ash.osuosl.org [140.211.166.34])\n\tby fraxinus.osuosl.org (Postfix) with ESMTP id 29D6CA2D55;\n\tWed, 26 Aug 2015 19:13:56 +0000 (UTC)", "from hemlock.osuosl.org (smtp2.osuosl.org [140.211.166.133])\n\tby ash.osuosl.org (Postfix) with ESMTP id 0626C1C1550\n\tfor <intel-wired-lan@lists.osuosl.org>;\n\tWed, 26 Aug 2015 19:13:54 +0000 (UTC)", "from localhost (localhost [127.0.0.1])\n\tby hemlock.osuosl.org (Postfix) with ESMTP id 01F4194F5F\n\tfor <intel-wired-lan@lists.osuosl.org>;\n\tWed, 26 Aug 2015 19:13:54 +0000 (UTC)", "from hemlock.osuosl.org ([127.0.0.1])\n\tby localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024)\n\twith ESMTP id xmUqTKZueLmU for <intel-wired-lan@lists.osuosl.org>;\n\tWed, 26 Aug 2015 19:13:53 +0000 (UTC)", "from mga14.intel.com (mga14.intel.com [192.55.52.115])\n\tby hemlock.osuosl.org (Postfix) with ESMTP id 0009894F37\n\tfor <intel-wired-lan@lists.osuosl.org>;\n\tWed, 26 Aug 2015 19:13:52 +0000 (UTC)", "from orsmga003.jf.intel.com ([10.7.209.27])\n\tby fmsmga103.fm.intel.com with ESMTP; 26 Aug 2015 12:13:52 -0700", "from catheri1-tigger.jf.intel.com ([134.134.176.92])\n\tby orsmga003.jf.intel.com with ESMTP; 26 Aug 2015 12:13:52 -0700" ], "X-Virus-Scanned": [ "amavisd-new at osuosl.org", "amavisd-new at osuosl.org" ], "X-Greylist": "domain auto-whitelisted by SQLgrey-1.7.6", "X-ExtLoop1": "1", "X-IronPort-AV": "E=Sophos;i=\"5.17,418,1437462000\"; d=\"scan'208\";a=\"632786468\"", "From": "Catherine Sullivan <catherine.sullivan@intel.com>", "To": "intel-wired-lan@lists.osuosl.org", "Date": "Wed, 26 Aug 2015 15:14:17 -0400", "Message-Id": "<1440616461-110869-12-git-send-email-catherine.sullivan@intel.com>", "X-Mailer": "git-send-email 1.9.3", "In-Reply-To": "<1440616461-110869-1-git-send-email-catherine.sullivan@intel.com>", "References": "<1440616461-110869-1-git-send-email-catherine.sullivan@intel.com>", "Subject": "[Intel-wired-lan] [next PATCH S11 11/15] i40e/i40evf: give up the\n\t__func__", "X-BeenThere": "intel-wired-lan@lists.osuosl.org", "X-Mailman-Version": "2.1.18-1", "Precedence": "list", "List-Id": "Intel Wired Ethernet Linux Kernel Driver Development\n\t<intel-wired-lan.lists.osuosl.org>", "List-Unsubscribe": "<http://lists.osuosl.org/mailman/options/intel-wired-lan>, \n\t<mailto:intel-wired-lan-request@lists.osuosl.org?subject=unsubscribe>", "List-Archive": "<http://lists.osuosl.org/pipermail/intel-wired-lan/>", "List-Post": "<mailto:intel-wired-lan@lists.osuosl.org>", "List-Help": "<mailto:intel-wired-lan-request@lists.osuosl.org?subject=help>", "List-Subscribe": "<http://lists.osuosl.org/mailman/listinfo/intel-wired-lan>, \n\t<mailto:intel-wired-lan-request@lists.osuosl.org?subject=subscribe>", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=\"us-ascii\"", "Content-Transfer-Encoding": "7bit", "Errors-To": "intel-wired-lan-bounces@lists.osuosl.org", "Sender": "\"Intel-wired-lan\" <intel-wired-lan-bounces@lists.osuosl.org>" }, "content": "From: Shannon Nelson <shannon.nelson@intel.com>\n\nDuring early development, we added the function name to all of the error\nstrings to make debugging simpler. Now that we've released the driver,\nour users should have more comprehensible error messages. So tear the\nroof off and give up the __func__. Ow.\n\nSigned-off-by: Shannon Nelson <shannon.nelson@intel.com>\nSigned-off-by: Mitch Williams <mitch.a.williams@intel.com>\nChange-ID: I7e1766252c7a032b9af6520da6aff536bdfd533c\n---\n drivers/net/ethernet/intel/i40e/i40e_dcb_nl.c | 5 +-\n drivers/net/ethernet/intel/i40e/i40e_main.c | 40 ++++++---------\n drivers/net/ethernet/intel/i40evf/i40evf_main.c | 9 ++--\n .../net/ethernet/intel/i40evf/i40evf_virtchnl.c | 59 ++++++++++------------\n 4 files changed, 47 insertions(+), 66 deletions(-)", "diff": "diff --git a/drivers/net/ethernet/intel/i40e/i40e_dcb_nl.c b/drivers/net/ethernet/intel/i40e/i40e_dcb_nl.c\nindex 1c51f73..dbadad7 100644\n--- a/drivers/net/ethernet/intel/i40e/i40e_dcb_nl.c\n+++ b/drivers/net/ethernet/intel/i40e/i40e_dcb_nl.c\n@@ -240,9 +240,8 @@ static void i40e_dcbnl_del_app(struct i40e_pf *pf,\n \t\tif (pf->vsi[v] && pf->vsi[v]->netdev) {\n \t\t\terr = i40e_dcbnl_vsi_del_app(pf->vsi[v], app);\n \t\t\tif (err)\n-\t\t\t\tdev_info(&pf->pdev->dev, \"%s: Failed deleting app for VSI seid=%d err=%d sel=%d proto=0x%x prio=%d\\n\",\n-\t\t\t\t\t __func__, pf->vsi[v]->seid,\n-\t\t\t\t\t err, app->selector,\n+\t\t\t\tdev_info(&pf->pdev->dev, \"Failed deleting app for VSI seid=%d err=%d sel=%d proto=0x%x prio=%d\\n\",\n+\t\t\t\t\t pf->vsi[v]->seid, err, app->selector,\n \t\t\t\t\t app->protocolid, app->priority);\n \t\t}\n \t}\ndiff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c\nindex f7da41b..6af199a 100644\n--- a/drivers/net/ethernet/intel/i40e/i40e_main.c\n+++ b/drivers/net/ethernet/intel/i40e/i40e_main.c\n@@ -3049,8 +3049,7 @@ static int i40e_vsi_request_irq_msix(struct i40e_vsi *vsi, char *basename)\n \t\t\t\t q_vector);\n \t\tif (err) {\n \t\t\tdev_info(&pf->pdev->dev,\n-\t\t\t\t \"%s: request_irq failed, error: %d\\n\",\n-\t\t\t\t __func__, err);\n+\t\t\t\t \"MSIX request_irq failed, error: %d\\n\", err);\n \t\t\tgoto free_queue_irqs;\n \t\t}\n \t\t/* assign the mask for this irq */\n@@ -3574,9 +3573,8 @@ static int i40e_vsi_control_tx(struct i40e_vsi *vsi, bool enable)\n \t\tret = i40e_pf_txq_wait(pf, pf_q, enable);\n \t\tif (ret) {\n \t\t\tdev_info(&pf->pdev->dev,\n-\t\t\t\t \"%s: VSI seid %d Tx ring %d %sable timeout\\n\",\n-\t\t\t\t __func__, vsi->seid, pf_q,\n-\t\t\t\t (enable ? \"en\" : \"dis\"));\n+\t\t\t\t \"VSI seid %d Tx ring %d %sable timeout\\n\",\n+\t\t\t\t vsi->seid, pf_q, (enable ? \"en\" : \"dis\"));\n \t\t\tbreak;\n \t\t}\n \t}\n@@ -3652,9 +3650,8 @@ static int i40e_vsi_control_rx(struct i40e_vsi *vsi, bool enable)\n \t\tret = i40e_pf_rxq_wait(pf, pf_q, enable);\n \t\tif (ret) {\n \t\t\tdev_info(&pf->pdev->dev,\n-\t\t\t\t \"%s: VSI seid %d Rx ring %d %sable timeout\\n\",\n-\t\t\t\t __func__, vsi->seid, pf_q,\n-\t\t\t\t (enable ? \"en\" : \"dis\"));\n+\t\t\t\t \"VSI seid %d Rx ring %d %sable timeout\\n\",\n+\t\t\t\t vsi->seid, pf_q, (enable ? \"en\" : \"dis\"));\n \t\t\tbreak;\n \t\t}\n \t}\n@@ -3948,8 +3945,7 @@ static void i40e_quiesce_vsi(struct i40e_vsi *vsi)\n \tif ((test_bit(__I40E_PORT_TX_SUSPENDED, &vsi->back->state)) &&\n \t vsi->type == I40E_VSI_FCOE) {\n \t\tdev_dbg(&vsi->back->pdev->dev,\n-\t\t\t\"%s: VSI seid %d skipping FCoE VSI disable\\n\",\n-\t\t\t __func__, vsi->seid);\n+\t\t\t \"VSI seid %d skipping FCoE VSI disable\\n\", vsi->seid);\n \t\treturn;\n \t}\n \n@@ -4023,8 +4019,8 @@ static int i40e_vsi_wait_txq_disabled(struct i40e_vsi *vsi)\n \t\tret = i40e_pf_txq_wait(pf, pf_q, false);\n \t\tif (ret) {\n \t\t\tdev_info(&pf->pdev->dev,\n-\t\t\t\t \"%s: VSI seid %d Tx ring %d disable timeout\\n\",\n-\t\t\t\t __func__, vsi->seid, pf_q);\n+\t\t\t\t \"VSI seid %d Tx ring %d disable timeout\\n\",\n+\t\t\t\t vsi->seid, pf_q);\n \t\t\treturn ret;\n \t\t}\n \t}\n@@ -5213,8 +5209,7 @@ bool i40e_dcb_need_reconfig(struct i40e_pf *pf,\n \t\tdev_dbg(&pf->pdev->dev, \"APP Table change detected.\\n\");\n \t}\n \n-\tdev_dbg(&pf->pdev->dev, \"%s: need_reconfig=%d\\n\", __func__,\n-\t\tneed_reconfig);\n+\tdev_dbg(&pf->pdev->dev, \"dcb need_reconfig=%d\\n\", need_reconfig);\n \treturn need_reconfig;\n }\n \n@@ -5241,16 +5236,14 @@ static int i40e_handle_lldp_event(struct i40e_pf *pf,\n \t/* Ignore if event is not for Nearest Bridge */\n \ttype = ((mib->type >> I40E_AQ_LLDP_BRIDGE_TYPE_SHIFT)\n \t\t& I40E_AQ_LLDP_BRIDGE_TYPE_MASK);\n-\tdev_dbg(&pf->pdev->dev,\n-\t\t\"%s: LLDP event mib bridge type 0x%x\\n\", __func__, type);\n+\tdev_dbg(&pf->pdev->dev, \"LLDP event mib bridge type 0x%x\\n\", type);\n \tif (type != I40E_AQ_LLDP_BRIDGE_TYPE_NEAREST_BRIDGE)\n \t\treturn ret;\n \n \t/* Check MIB Type and return if event for Remote MIB update */\n \ttype = mib->type & I40E_AQ_LLDP_MIB_TYPE_MASK;\n \tdev_dbg(&pf->pdev->dev,\n-\t\t\"%s: LLDP event mib type %s\\n\", __func__,\n-\t\ttype ? \"remote\" : \"local\");\n+\t\t\"LLDP event mib type %s\\n\", type ? \"remote\" : \"local\");\n \tif (type == I40E_AQ_LLDP_MIB_REMOTE) {\n \t\t/* Update the remote cached instance and return */\n \t\tret = i40e_aq_get_dcb_config(hw, I40E_AQ_LLDP_MIB_REMOTE,\n@@ -8983,8 +8976,7 @@ struct i40e_vsi *i40e_vsi_setup(struct i40e_pf *pf, u8 type,\n \t\tif (veb) {\n \t\t\tif (vsi->seid != pf->vsi[pf->lan_vsi]->seid) {\n \t\t\t\tdev_info(&vsi->back->pdev->dev,\n-\t\t\t\t\t \"%s: New VSI creation error, uplink seid of LAN VSI expected.\\n\",\n-\t\t\t\t\t __func__);\n+\t\t\t\t\t \"New VSI creation error, uplink seid of LAN VSI expected.\\n\");\n \t\t\t\treturn NULL;\n \t\t\t}\n \t\t\t/* We come up by default in VEPA mode if SRIOV is not\n@@ -10428,7 +10420,7 @@ static pci_ers_result_t i40e_pci_error_slot_reset(struct pci_dev *pdev)\n \tint err;\n \tu32 reg;\n \n-\tdev_info(&pdev->dev, \"%s\\n\", __func__);\n+\tdev_dbg(&pdev->dev, \"%s\\n\", __func__);\n \tif (pci_enable_device_mem(pdev)) {\n \t\tdev_info(&pdev->dev,\n \t\t\t \"Cannot re-enable PCI device after reset.\\n\");\n@@ -10468,7 +10460,7 @@ static void i40e_pci_error_resume(struct pci_dev *pdev)\n {\n \tstruct i40e_pf *pf = pci_get_drvdata(pdev);\n \n-\tdev_info(&pdev->dev, \"%s\\n\", __func__);\n+\tdev_dbg(&pdev->dev, \"%s\\n\", __func__);\n \tif (test_bit(__I40E_SUSPENDED, &pf->state))\n \t\treturn;\n \n@@ -10560,9 +10552,7 @@ static int i40e_resume(struct pci_dev *pdev)\n \n \terr = pci_enable_device_mem(pdev);\n \tif (err) {\n-\t\tdev_err(&pdev->dev,\n-\t\t\t\"%s: Cannot enable PCI device from suspend\\n\",\n-\t\t\t__func__);\n+\t\tdev_err(&pdev->dev, \"Cannot enable PCI device from suspend\\n\");\n \t\treturn err;\n \t}\n \tpci_set_master(pdev);\ndiff --git a/drivers/net/ethernet/intel/i40evf/i40evf_main.c b/drivers/net/ethernet/intel/i40evf/i40evf_main.c\nindex d5d099b9..3fc5606 100644\n--- a/drivers/net/ethernet/intel/i40evf/i40evf_main.c\n+++ b/drivers/net/ethernet/intel/i40evf/i40evf_main.c\n@@ -411,8 +411,7 @@ i40evf_request_traffic_irqs(struct i40evf_adapter *adapter, char *basename)\n \t\t\tq_vector);\n \t\tif (err) {\n \t\t\tdev_info(&adapter->pdev->dev,\n-\t\t\t\t \"%s: request_irq failed, error: %d\\n\",\n-\t\t\t\t__func__, err);\n+\t\t\t\t \"Request_irq failed, error: %d\\n\", err);\n \t\t\tgoto free_queue_irqs;\n \t\t}\n \t\t/* assign the mask for this irq */\n@@ -1775,8 +1774,7 @@ static int i40evf_setup_all_tx_resources(struct i40evf_adapter *adapter)\n \t\tif (!err)\n \t\t\tcontinue;\n \t\tdev_err(&adapter->pdev->dev,\n-\t\t\t\"%s: Allocation for Tx Queue %u failed\\n\",\n-\t\t\t__func__, i);\n+\t\t\t\"Allocation for Tx Queue %u failed\\n\", i);\n \t\tbreak;\n \t}\n \n@@ -1803,8 +1801,7 @@ static int i40evf_setup_all_rx_resources(struct i40evf_adapter *adapter)\n \t\tif (!err)\n \t\t\tcontinue;\n \t\tdev_err(&adapter->pdev->dev,\n-\t\t\t\"%s: Allocation for Rx Queue %u failed\\n\",\n-\t\t\t__func__, i);\n+\t\t\t\"Allocation for Rx Queue %u failed\\n\", i);\n \t\tbreak;\n \t}\n \treturn err;\ndiff --git a/drivers/net/ethernet/intel/i40evf/i40evf_virtchnl.c b/drivers/net/ethernet/intel/i40evf/i40evf_virtchnl.c\nindex d4eb1a5..317e240 100644\n--- a/drivers/net/ethernet/intel/i40evf/i40evf_virtchnl.c\n+++ b/drivers/net/ethernet/intel/i40evf/i40evf_virtchnl.c\n@@ -234,8 +234,8 @@ void i40evf_configure_queues(struct i40evf_adapter *adapter)\n \n \tif (adapter->current_op != I40E_VIRTCHNL_OP_UNKNOWN) {\n \t\t/* bail because we already have a command pending */\n-\t\tdev_err(&adapter->pdev->dev, \"%s: command %d pending\\n\",\n-\t\t\t__func__, adapter->current_op);\n+\t\tdev_err(&adapter->pdev->dev, \"Cannot configure queues, command %d pending\\n\",\n+\t\t\tadapter->current_op);\n \t\treturn;\n \t}\n \tadapter->current_op = I40E_VIRTCHNL_OP_CONFIG_VSI_QUEUES;\n@@ -288,8 +288,8 @@ void i40evf_enable_queues(struct i40evf_adapter *adapter)\n \n \tif (adapter->current_op != I40E_VIRTCHNL_OP_UNKNOWN) {\n \t\t/* bail because we already have a command pending */\n-\t\tdev_err(&adapter->pdev->dev, \"%s: command %d pending\\n\",\n-\t\t\t__func__, adapter->current_op);\n+\t\tdev_err(&adapter->pdev->dev, \"Cannot enable queues, command %d pending\\n\",\n+\t\t\tadapter->current_op);\n \t\treturn;\n \t}\n \tadapter->current_op = I40E_VIRTCHNL_OP_ENABLE_QUEUES;\n@@ -313,8 +313,8 @@ void i40evf_disable_queues(struct i40evf_adapter *adapter)\n \n \tif (adapter->current_op != I40E_VIRTCHNL_OP_UNKNOWN) {\n \t\t/* bail because we already have a command pending */\n-\t\tdev_err(&adapter->pdev->dev, \"%s: command %d pending\\n\",\n-\t\t\t__func__, adapter->current_op);\n+\t\tdev_err(&adapter->pdev->dev, \"Cannot disable queues, command %d pending\\n\",\n+\t\t\tadapter->current_op);\n \t\treturn;\n \t}\n \tadapter->current_op = I40E_VIRTCHNL_OP_DISABLE_QUEUES;\n@@ -341,8 +341,8 @@ void i40evf_map_queues(struct i40evf_adapter *adapter)\n \n \tif (adapter->current_op != I40E_VIRTCHNL_OP_UNKNOWN) {\n \t\t/* bail because we already have a command pending */\n-\t\tdev_err(&adapter->pdev->dev, \"%s: command %d pending\\n\",\n-\t\t\t__func__, adapter->current_op);\n+\t\tdev_err(&adapter->pdev->dev, \"Cannot map queues to vectors, command %d pending\\n\",\n+\t\t\tadapter->current_op);\n \t\treturn;\n \t}\n \tadapter->current_op = I40E_VIRTCHNL_OP_CONFIG_IRQ_MAP;\n@@ -393,8 +393,8 @@ void i40evf_add_ether_addrs(struct i40evf_adapter *adapter)\n \n \tif (adapter->current_op != I40E_VIRTCHNL_OP_UNKNOWN) {\n \t\t/* bail because we already have a command pending */\n-\t\tdev_err(&adapter->pdev->dev, \"%s: command %d pending\\n\",\n-\t\t\t__func__, adapter->current_op);\n+\t\tdev_err(&adapter->pdev->dev, \"Cannot add filters, command %d pending\\n\",\n+\t\t\tadapter->current_op);\n \t\treturn;\n \t}\n \tlist_for_each_entry(f, &adapter->mac_filter_list, list) {\n@@ -410,8 +410,7 @@ void i40evf_add_ether_addrs(struct i40evf_adapter *adapter)\n \tlen = sizeof(struct i40e_virtchnl_ether_addr_list) +\n \t (count * sizeof(struct i40e_virtchnl_ether_addr));\n \tif (len > I40EVF_MAX_AQ_BUF_SIZE) {\n-\t\tdev_warn(&adapter->pdev->dev, \"%s: Too many MAC address changes in one request\\n\",\n-\t\t\t __func__);\n+\t\tdev_warn(&adapter->pdev->dev, \"Too many add MAC changes in one request\\n\");\n \t\tcount = (I40EVF_MAX_AQ_BUF_SIZE -\n \t\t\t sizeof(struct i40e_virtchnl_ether_addr_list)) /\n \t\t\tsizeof(struct i40e_virtchnl_ether_addr);\n@@ -453,8 +452,8 @@ void i40evf_del_ether_addrs(struct i40evf_adapter *adapter)\n \n \tif (adapter->current_op != I40E_VIRTCHNL_OP_UNKNOWN) {\n \t\t/* bail because we already have a command pending */\n-\t\tdev_err(&adapter->pdev->dev, \"%s: command %d pending\\n\",\n-\t\t\t__func__, adapter->current_op);\n+\t\tdev_err(&adapter->pdev->dev, \"Cannot remove filters, command %d pending\\n\",\n+\t\t\tadapter->current_op);\n \t\treturn;\n \t}\n \tlist_for_each_entry(f, &adapter->mac_filter_list, list) {\n@@ -470,8 +469,7 @@ void i40evf_del_ether_addrs(struct i40evf_adapter *adapter)\n \tlen = sizeof(struct i40e_virtchnl_ether_addr_list) +\n \t (count * sizeof(struct i40e_virtchnl_ether_addr));\n \tif (len > I40EVF_MAX_AQ_BUF_SIZE) {\n-\t\tdev_warn(&adapter->pdev->dev, \"%s: Too many MAC address changes in one request\\n\",\n-\t\t\t __func__);\n+\t\tdev_warn(&adapter->pdev->dev, \"Too many delete MAC changes in one request\\n\");\n \t\tcount = (I40EVF_MAX_AQ_BUF_SIZE -\n \t\t\t sizeof(struct i40e_virtchnl_ether_addr_list)) /\n \t\t\tsizeof(struct i40e_virtchnl_ether_addr);\n@@ -513,8 +511,8 @@ void i40evf_add_vlans(struct i40evf_adapter *adapter)\n \n \tif (adapter->current_op != I40E_VIRTCHNL_OP_UNKNOWN) {\n \t\t/* bail because we already have a command pending */\n-\t\tdev_err(&adapter->pdev->dev, \"%s: command %d pending\\n\",\n-\t\t\t__func__, adapter->current_op);\n+\t\tdev_err(&adapter->pdev->dev, \"Cannot add VLANs, command %d pending\\n\",\n+\t\t\tadapter->current_op);\n \t\treturn;\n \t}\n \n@@ -531,8 +529,7 @@ void i40evf_add_vlans(struct i40evf_adapter *adapter)\n \tlen = sizeof(struct i40e_virtchnl_vlan_filter_list) +\n \t (count * sizeof(u16));\n \tif (len > I40EVF_MAX_AQ_BUF_SIZE) {\n-\t\tdev_warn(&adapter->pdev->dev, \"%s: Too many VLAN changes in one request\\n\",\n-\t\t\t __func__);\n+\t\tdev_warn(&adapter->pdev->dev, \"Too many add VLAN changes in one request\\n\");\n \t\tcount = (I40EVF_MAX_AQ_BUF_SIZE -\n \t\t\t sizeof(struct i40e_virtchnl_vlan_filter_list)) /\n \t\t\tsizeof(u16);\n@@ -572,8 +569,8 @@ void i40evf_del_vlans(struct i40evf_adapter *adapter)\n \n \tif (adapter->current_op != I40E_VIRTCHNL_OP_UNKNOWN) {\n \t\t/* bail because we already have a command pending */\n-\t\tdev_err(&adapter->pdev->dev, \"%s: command %d pending\\n\",\n-\t\t\t__func__, adapter->current_op);\n+\t\tdev_err(&adapter->pdev->dev, \"Cannot remove VLANs, command %d pending\\n\",\n+\t\t\tadapter->current_op);\n \t\treturn;\n \t}\n \n@@ -590,8 +587,7 @@ void i40evf_del_vlans(struct i40evf_adapter *adapter)\n \tlen = sizeof(struct i40e_virtchnl_vlan_filter_list) +\n \t (count * sizeof(u16));\n \tif (len > I40EVF_MAX_AQ_BUF_SIZE) {\n-\t\tdev_warn(&adapter->pdev->dev, \"%s: Too many VLAN changes in one request\\n\",\n-\t\t\t __func__);\n+\t\tdev_warn(&adapter->pdev->dev, \"Too many delete VLAN changes in one request\\n\");\n \t\tcount = (I40EVF_MAX_AQ_BUF_SIZE -\n \t\t\t sizeof(struct i40e_virtchnl_vlan_filter_list)) /\n \t\t\tsizeof(u16);\n@@ -629,8 +625,8 @@ void i40evf_set_promiscuous(struct i40evf_adapter *adapter, int flags)\n \n \tif (adapter->current_op != I40E_VIRTCHNL_OP_UNKNOWN) {\n \t\t/* bail because we already have a command pending */\n-\t\tdev_err(&adapter->pdev->dev, \"%s: command %d pending\\n\",\n-\t\t\t__func__, adapter->current_op);\n+\t\tdev_err(&adapter->pdev->dev, \"Cannot set promiscuous mode, command %d pending\\n\",\n+\t\t\tadapter->current_op);\n \t\treturn;\n \t}\n \tadapter->current_op = I40E_VIRTCHNL_OP_CONFIG_PROMISCUOUS_MODE;\n@@ -720,17 +716,16 @@ void i40evf_virtchnl_completion(struct i40evf_adapter *adapter,\n \t\t\t}\n \t\t\tbreak;\n \t\tdefault:\n-\t\t\tdev_err(&adapter->pdev->dev,\n-\t\t\t\t\"%s: Unknown event %d from pf\\n\",\n-\t\t\t\t__func__, vpe->event);\n+\t\t\tdev_err(&adapter->pdev->dev, \"Unknown event %d from PF\\n\",\n+\t\t\t\tvpe->event);\n \t\t\tbreak;\n \t\t}\n \t\treturn;\n \t}\n \tif (v_retval) {\n-\t\tdev_err(&adapter->pdev->dev, \"%s: PF returned error %d (%s) to our request %d\\n\",\n-\t\t\t__func__, v_retval,\n-\t\t\ti40evf_stat_str(&adapter->hw, v_retval), v_opcode);\n+\t\tdev_err(&adapter->pdev->dev, \"PF returned error %d (%s) to our request %d\\n\",\n+\t\t\tv_retval, i40evf_stat_str(&adapter->hw, v_retval),\n+\t\t\tv_opcode);\n \t}\n \tswitch (v_opcode) {\n \tcase I40E_VIRTCHNL_OP_GET_STATS: {\n", "prefixes": [ "next", "S11", "11/15" ] }