Cover Detail
Show a cover letter.
GET /api/covers/86896/?format=api
https://patches.dpdk.org/api/covers/86896/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/cover/20210119212507.1043636-1-maxime.coquelin@redhat.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": "<20210119212507.1043636-1-maxime.coquelin@redhat.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20210119212507.1043636-1-maxime.coquelin@redhat.com", "date": "2021-01-19T21:24:23", "name": "[v2,00/44] net/virtio: Virtio PMD rework", "submitter": { "id": 512, "url": "https://patches.dpdk.org/api/people/512/?format=api", "name": "Maxime Coquelin", "email": "maxime.coquelin@redhat.com" }, "mbox": "https://patches.dpdk.org/project/dpdk/cover/20210119212507.1043636-1-maxime.coquelin@redhat.com/mbox/", "series": [ { "id": 14852, "url": "https://patches.dpdk.org/api/series/14852/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/list/?series=14852", "date": "2021-01-19T21:24:23", "name": "net/virtio: Virtio PMD rework", "version": 2, "mbox": "https://patches.dpdk.org/series/14852/mbox/" } ], "comments": "https://patches.dpdk.org/api/covers/86896/comments/", "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 851C5A0A05;\n\tTue, 19 Jan 2021 22:25:52 +0100 (CET)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 07D7C140DD3;\n\tTue, 19 Jan 2021 22:25:52 +0100 (CET)", "from us-smtp-delivery-124.mimecast.com\n (us-smtp-delivery-124.mimecast.com [216.205.24.124])\n by mails.dpdk.org (Postfix) with ESMTP id 0E331140DCF\n for <dev@dpdk.org>; Tue, 19 Jan 2021 22:25:49 +0100 (CET)", "from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com\n [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id\n us-mta-199-WBIMorrfPzmQW7c-Rn62mQ-1; Tue, 19 Jan 2021 16:25:47 -0500", "from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com\n [10.5.11.14])\n (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))\n (No client certificate requested)\n by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 61CD8800D62;\n Tue, 19 Jan 2021 21:25:46 +0000 (UTC)", "from max-t490s.redhat.com (unknown [10.36.110.29])\n by smtp.corp.redhat.com (Postfix) with ESMTP id 78C2A5D9E2;\n Tue, 19 Jan 2021 21:25:38 +0000 (UTC)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com;\n s=mimecast20190719; t=1611091549;\n h=from:from:reply-to:subject:subject:date:date:message-id:message-id:\n to:to:cc:cc:mime-version:mime-version:content-type:content-type:\n content-transfer-encoding:content-transfer-encoding;\n bh=1nx9CJR83OSp44cqQAtfjdu6NNmYj8G6weKs/nPNWMg=;\n b=WJcO+sodro061fWm4/Nbb8hC4oNfhzLfSQ2dzjrn9D6Hu8BjmxAT4D0lPVI2zvSSZWct87\n Z0jBxrS05h9BId+mRhEnFpaP8sw6OFH9Y0Lvz4CVPegSHb7A+OICHcI9LBiMpcXzPDodxL\n 3APZcNhrSMebys1wJds7G4gu/wxZ0KY=", "X-MC-Unique": "WBIMorrfPzmQW7c-Rn62mQ-1", "From": "Maxime Coquelin <maxime.coquelin@redhat.com>", "To": "dev@dpdk.org, chenbo.xia@intel.com, olivier.matz@6wind.com,\n amorenoz@redhat.com, david.marchand@redhat.com", "Cc": "Maxime Coquelin <maxime.coquelin@redhat.com>", "Date": "Tue, 19 Jan 2021 22:24:23 +0100", "Message-Id": "<20210119212507.1043636-1-maxime.coquelin@redhat.com>", "MIME-Version": "1.0", "X-Scanned-By": "MIMEDefang 2.79 on 10.5.11.14", "Authentication-Results": "relay.mimecast.com;\n auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=maxime.coquelin@redhat.com", "X-Mimecast-Spam-Score": "0", "X-Mimecast-Originator": "redhat.com", "Content-Transfer-Encoding": "8bit", "Content-Type": "text/plain; charset=\"US-ASCII\"", "Subject": "[dpdk-dev] [PATCH v2 00/44] net/virtio: Virtio PMD rework", "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", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "Thanks Chenbo, David & Adrian for your review contribution to\nthe V1. This V2 should implement all the changes you requested,\nand fix a few regressions caught while testing.\n\nThis series significantly rework Virtio PMD to improve\nthe Virtio-user PMD and its backends integration.\n\nFirst part of the series removes the dependency of\nVirtio-user ethdev on Virtio PCI, by creating generic\nfiles, adding per-bus meta data, ...\n\nMain (if not single) functionnal change of this first\npart is to remove the hack for Virtio-user to work in\nIOVA as PA mode, this hack being very fragile.\n\nSecond part of the series reworks Virtio-user internal,\nby reworking the requests handling so that vDPA and Kernel\nbackends no more hack into being Vhost-user backend. It\nimplies implementing new ops for all the request types.\nAlso, all the backend specific actions are moved from the\nvirtio_user_dev.c and virtio_user_ethdev.c to their\nbackend files.\n\nOnly functionnal change in this second part is making the\nVhost-user server mode blocking at init time, as long as\na client is not connected. The goal of this change is to\nmake the Vhost-user support much more robust, as without\nblocking, the driver has to assume features that are going\nto be supported by the client, which is very fragile and\nerror prone. As a side-effect, it also simplifies the\nlogic nin several place of the virtio-user PMD.\n\nMain changes in v2:\n===================\n- Introduce vdev driver flag for drivers to require IOVA VA mode\n- Rebase on top of -rc1 changes\n- Fix regressions introduced in V1 (vhost-kernel broken, vhost-user reconnect...)\n- Various minor issues & typos fixed\n- Fix status feature issue introduced in v20.11, only reproduceable now that server\n mode is made blocking\n- Improve failure handling in Virtio-user\n- Improve logging\n\nTesting coverage (All passed)\n=============================\n- Virtio-pci PMD\n * Virtio PMD in guest with Vhost-user backend in host\n * Virtio PMD in guest with Vhost-kernel backend in host\n- Virtio-user PMD with Vhost-user backend\n * Vhost-user PMD server <-> Virtio-user client PMD IO loopback\n * Vhost-user PMD client <-> Virtio-user server PMD IO loopback\n * Vhost-user PMD client <-> Virtio-user server PMD reconnect\n- Virtio-user PMD with Vhost-kernel backend\n * iperf test case\n * Txonly testpmd\n- Virtio-user PMD with Vhost-vDPA backend\n * vdpa-sim (IO loopback)\n * CX-6 DX Kernel vDPA (Tx only)\n\nMaxime Coquelin (44):\n bus/vdev: add helper to get vdev from eth dev\n bus/vdev: add driver IOVA VA mode requirement\n net/virtio: fix getting old status on reconnect\n net/virtio: introduce Virtio bus type\n net/virtio: refactor virtio-user device\n net/virtio: introduce PCI device metadata\n net/virtio: move PCI device init in dedicated file\n net/virtio: move PCI specific dev init to PCI ethdev init\n net/virtio: move MSIX detection to PCI ethdev\n net/virtio: force IOVA as VA mode for Virtio-user\n net/virtio: store PCI type in Virtio device metadata\n net/virtio: add callback for device closing\n net/virtio: validate features at bus level\n net/virtio: remove bus type enum\n net/virtio: move PCI-specific fields to PCI device\n net/virtio: pack virtio HW struct\n net/virtio: move legacy IO to Virtio PCI\n net/virtio: introduce generic virtio header\n net/virtio: move features definition to generic header\n net/virtio: move virtqueue defines in generic header\n net/virtio: move config definitions to generic header\n net/virtio: make interrupt handling more generic\n net/virtio: move vring alignment to generic header\n net/virtio: remove last PCI refs in non-PCI code\n net/virtio: make Vhost-user req sender consistent\n net/virtio: add Virtio-user ops to set owner\n net/virtio: add Virtio-user features ops\n net/virtio: add Virtio-user protocol features ops\n net/virtio: add Virtio-user memory tables ops\n net/virtio: add Virtio-user vring setting ops\n net/virtio: add Virtio-user vring file ops\n net/virtio: add Virtio-user vring address ops\n net/virtio: add Virtio-user status ops\n net/virtio: remove useless request ops\n net/virtio: improve Virtio-user errors handling\n net/virtio: move Vhost-user reqs to Vhost-user backend\n net/virtio: make server mode blocking\n net/virtio: move protocol features to Vhost-user\n net/virtio: introduce backend data\n net/virtio: move Vhost-user specifics to its backend\n net/virtio: move Vhost-kernel data to its backend\n net/virtio: move Vhost-vDPA data to its backend\n net/virtio: improve Vhost-user error logging\n net/virtio: handle Virtio-user setup failure properly\n\n drivers/bus/vdev/rte_bus_vdev.h | 6 +\n drivers/bus/vdev/vdev.c | 31 +\n drivers/net/virtio/meson.build | 6 +-\n drivers/net/virtio/virtio.c | 71 ++\n drivers/net/virtio/virtio.h | 246 +++++\n drivers/net/virtio/virtio_ethdev.c | 457 +++------\n drivers/net/virtio/virtio_ethdev.h | 6 +-\n drivers/net/virtio/virtio_pci.c | 449 +++++----\n drivers/net/virtio/virtio_pci.h | 286 +-----\n drivers/net/virtio/virtio_pci_ethdev.c | 226 +++++\n drivers/net/virtio/virtio_ring.h | 2 +-\n drivers/net/virtio/virtio_rxtx.c | 90 +-\n drivers/net/virtio/virtio_rxtx_packed.h | 10 +-\n drivers/net/virtio/virtio_rxtx_packed_avx.h | 10 +-\n drivers/net/virtio/virtio_rxtx_packed_neon.h | 10 +-\n drivers/net/virtio/virtio_rxtx_simple.h | 3 +-\n drivers/net/virtio/virtio_user/vhost.h | 79 +-\n drivers/net/virtio/virtio_user/vhost_kernel.c | 461 ++++++---\n .../net/virtio/virtio_user/vhost_kernel_tap.c | 25 +-\n .../net/virtio/virtio_user/vhost_kernel_tap.h | 1 +\n drivers/net/virtio/virtio_user/vhost_user.c | 896 ++++++++++++++----\n drivers/net/virtio/virtio_user/vhost_vdpa.c | 323 +++++--\n .../net/virtio/virtio_user/virtio_user_dev.c | 514 +++++-----\n .../net/virtio/virtio_user/virtio_user_dev.h | 20 +-\n drivers/net/virtio/virtio_user_ethdev.c | 301 +-----\n drivers/net/virtio/virtqueue.c | 6 +-\n drivers/net/virtio/virtqueue.h | 45 +-\n 27 files changed, 2704 insertions(+), 1876 deletions(-)\n create mode 100644 drivers/net/virtio/virtio.c\n create mode 100644 drivers/net/virtio/virtio.h\n create mode 100644 drivers/net/virtio/virtio_pci_ethdev.c" }{ "id": 86896, "url": "