Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/119041/?format=api
http://patchwork.dpdk.org/api/patches/119041/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/patch/20221018072240.128791-1-kim.tae.kyung@navercorp.com/", "project": { "id": 1, "url": "http://patchwork.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": "<20221018072240.128791-1-kim.tae.kyung@navercorp.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20221018072240.128791-1-kim.tae.kyung@navercorp.com", "date": "2022-10-18T07:22:40", "name": "vdpa/ifc: fix update_datapath error handling", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "9116b7cb4a74a63f5650458282fec6b891ff8ff8", "submitter": { "id": 2877, "url": "http://patchwork.dpdk.org/api/people/2877/?format=api", "name": "Taekyung Kim", "email": "kim.tae.kyung@navercorp.com" }, "delegate": { "id": 2642, "url": "http://patchwork.dpdk.org/api/users/2642/?format=api", "username": "mcoquelin", "first_name": "Maxime", "last_name": "Coquelin", "email": "maxime.coquelin@redhat.com" }, "mbox": "http://patchwork.dpdk.org/project/dpdk/patch/20221018072240.128791-1-kim.tae.kyung@navercorp.com/mbox/", "series": [ { "id": 25398, "url": "http://patchwork.dpdk.org/api/series/25398/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/list/?series=25398", "date": "2022-10-18T07:22:40", "name": "vdpa/ifc: fix update_datapath error handling", "version": 1, "mbox": "http://patchwork.dpdk.org/series/25398/mbox/" } ], "comments": "http://patchwork.dpdk.org/api/patches/119041/comments/", "check": "success", "checks": "http://patchwork.dpdk.org/api/patches/119041/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 D9DDCA0542;\n\tMon, 24 Oct 2022 17:34:48 +0200 (CEST)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 319AB42C07;\n\tMon, 24 Oct 2022 17:33:20 +0200 (CEST)", "from cvbackendsmtp001.nmdf.navercorp.com\n (cvbackendsmtp001.nmdf.navercorp.com [125.209.246.147])\n by mails.dpdk.org (Postfix) with ESMTP id 6C8CF40143\n for <dev@dpdk.org>; Tue, 18 Oct 2022 09:22:50 +0200 (CEST)", "from dev-tkkim-git-send-email-ncl.nfra.io ([10.113.149.62])\n by cvbackendsmtp001.nmdf.navercorp.com with ESMTP id FRn0LCCAQS+mzfTXHen+vQ\n for <multiple recipients>; Tue, 18 Oct 2022 07:22:47 -0000" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=navercorp.com;\n s=s20171120; t=1666077767;\n bh=03W03VsQtV0pZ55Aq6ydQ1M+1bmcwzVv4I2UzOwOv/8=;\n h=From:To:Subject:Date:Message-Id:From:Subject:Feedback-ID:\n X-Works-Security;\n b=a4hbIxrUnWLkmTGHLcpkdvP3elph1IbRubJXmmEzk2fmALacAKm2rz6cVMlMelMfn\n jd0TueUTRL27BDtZeCL/kxMAnCBOLDVq2zuXo7zdv1Cbq3Zl3BJ6Khr4KBM1TwNoVT\n fQAnhhew4chMQf+TbndJt8/r/gRQ4B5tKmRyBNbxSmumcDPwv+0KUDnDyEGVQH3Mte\n PLE6ucBUGskvgLspWNtXNgRZDAY/1NHBMZSB1GAAZKMUzWhZ0n2CF6f60nBFWAoOui\n VSSUcndsuWVz0Kmevs46GAuOt2h7vvcrYzE8hR4XXQ16rncnUG7YATBSA+Q+KUUiaj\n EAlm8GzP7c33g==", "X-Session-ID": "FRn0LCCAQS+mzfTXHen+vQ", "X-Works-Smtp-Source": "XXKdKAMrFoJCpNmXFoK9WHF0", "From": "Taekyung Kim <kim.tae.kyung@navercorp.com>", "To": "dev@dpdk.org", "Cc": "Maxime Coquelin <maxime.coquelin@redhat.com>,\n Taekyung Kim <kim.tae.kyung@navercorp.com>", "Subject": "[PATCH] vdpa/ifc: fix update_datapath error handling", "Date": "Tue, 18 Oct 2022 16:22:40 +0900", "Message-Id": "<20221018072240.128791-1-kim.tae.kyung@navercorp.com>", "X-Mailer": "git-send-email 2.31.1", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "X-Mailman-Approved-At": "Mon, 24 Oct 2022 17:33:00 +0200", "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": "Stop and return the error code if update_datapath fails.\nupdate_datapath prepares resources for the vdpa device.\nThe driver should not perform any further actions\nif update_datapath returns an error.\n\nSigned-off-by: Taekyung Kim <kim.tae.kyung@navercorp.com>\n---\n drivers/vdpa/ifc/ifcvf_vdpa.c | 20 ++++++++++++++++----\n 1 file changed, 16 insertions(+), 4 deletions(-)", "diff": "diff --git a/drivers/vdpa/ifc/ifcvf_vdpa.c b/drivers/vdpa/ifc/ifcvf_vdpa.c\nindex d5ac583589..795967e998 100644\n--- a/drivers/vdpa/ifc/ifcvf_vdpa.c\n+++ b/drivers/vdpa/ifc/ifcvf_vdpa.c\n@@ -1063,7 +1063,10 @@ ifcvf_dev_config(int vid)\n \tinternal = list->internal;\n \tinternal->vid = vid;\n \trte_atomic32_set(&internal->dev_attached, 1);\n-\tupdate_datapath(internal);\n+\tif (update_datapath(internal) < 0) {\n+\t\tDRV_LOG(ERR, \"failed to update datapath: %p\", vdev);\n+\t\treturn -1;\n+\t}\n \n \tif (rte_vhost_host_notifier_ctrl(vid, RTE_VHOST_QUEUE_ALL, true) != 0)\n \t\tDRV_LOG(NOTICE, \"vDPA (%s): software relay is used.\",\n@@ -1105,7 +1108,10 @@ ifcvf_dev_close(int vid)\n \t\tinternal->sw_fallback_running = false;\n \t} else {\n \t\trte_atomic32_set(&internal->dev_attached, 0);\n-\t\tupdate_datapath(internal);\n+\t\tif (update_datapath(internal) < 0) {\n+\t\t\tDRV_LOG(ERR, \"failed to update datapath: %p\", vdev);\n+\t\t\treturn -1;\n+\t\t}\n \t}\n \n \tinternal->configured = 0;\n@@ -1632,7 +1638,10 @@ ifcvf_pci_probe(struct rte_pci_driver *pci_drv __rte_unused,\n \tpthread_mutex_unlock(&internal_list_lock);\n \n \trte_atomic32_set(&internal->started, 1);\n-\tupdate_datapath(internal);\n+\tif (update_datapath(internal) < 0) {\n+\t\tDRV_LOG(ERR, \"failed to update datapath: %s\", pci_dev->name);\n+\t\treturn -1;\n+\t}\n \n \trte_kvargs_free(kvlist);\n \treturn 0;\n@@ -1661,7 +1670,10 @@ ifcvf_pci_remove(struct rte_pci_device *pci_dev)\n \n \tinternal = list->internal;\n \trte_atomic32_set(&internal->started, 0);\n-\tupdate_datapath(internal);\n+\tif (update_datapath(internal) < 0) {\n+\t\tDRV_LOG(ERR, \"failed to update datapath: %s\", pci_dev->name);\n+\t\treturn -1;\n+\t}\n \n \trte_pci_unmap_device(internal->pdev);\n \trte_vfio_container_destroy(internal->vfio_container_fd);\n", "prefixes": [] }{ "id": 119041, "url": "