Cover Detail
Show a cover letter.
GET /api/covers/85524/?format=api
http://patches.dpdk.org/api/covers/85524/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/cover/20201220211405.313012-1-maxime.coquelin@redhat.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": "<20201220211405.313012-1-maxime.coquelin@redhat.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20201220211405.313012-1-maxime.coquelin@redhat.com", "date": "2020-12-20T21:13:25", "name": "[00/40] net/virtio: Virtio PMD rework", "submitter": { "id": 512, "url": "http://patches.dpdk.org/api/people/512/?format=api", "name": "Maxime Coquelin", "email": "maxime.coquelin@redhat.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/cover/20201220211405.313012-1-maxime.coquelin@redhat.com/mbox/", "series": [ { "id": 14390, "url": "http://patches.dpdk.org/api/series/14390/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=14390", "date": "2020-12-20T21:13:25", "name": "net/virtio: Virtio PMD rework", "version": 1, "mbox": "http://patches.dpdk.org/series/14390/mbox/" } ], "comments": "http://patches.dpdk.org/api/covers/85524/comments/", "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@inbox.dpdk.org", "Delivered-To": "patchwork@inbox.dpdk.org", "Received": [ "from dpdk.org (dpdk.org [92.243.14.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id 0A27FA09FD;\n\tSun, 20 Dec 2020 22:14:27 +0100 (CET)", "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id 8ADDCCACF;\n\tSun, 20 Dec 2020 22:14:25 +0100 (CET)", "from us-smtp-delivery-124.mimecast.com\n (us-smtp-delivery-124.mimecast.com [63.128.21.124])\n by dpdk.org (Postfix) with ESMTP id 27BF6CAB9\n for <dev@dpdk.org>; Sun, 20 Dec 2020 22:14:23 +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-476-anXjvrbsOoa2zub4ZRW5Rg-1; Sun, 20 Dec 2020 16:14:19 -0500", "from smtp.corp.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com\n [10.5.11.12])\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 88C98805EF2;\n Sun, 20 Dec 2020 21:14:18 +0000 (UTC)", "from max-t490s.redhat.com (unknown [10.36.110.43])\n by smtp.corp.redhat.com (Postfix) with ESMTP id B39BB60C43;\n Sun, 20 Dec 2020 21:14:10 +0000 (UTC)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com;\n s=mimecast20190719; t=1608498861;\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=QiDBnTMCoayqsQp7oF4TXRbHkz5QvLMTUTqtYIQfcvQ=;\n b=A1K5sz4xCE351CLMIWGzkhcRMGQECOdOFKQeLN/qDpN1v+dqYAqxwbrK1p3t4DO0LIO8Dr\n yH9ToKeyoZw3FPMPqSfNMeQw7oMDSyHFY4Z0QCq1i1kfMMiaO07MoVmhaEjTqhUcvZALQQ\n JdzKwIWg/Mlj6//h1fH/YWm0zi3r35Q=", "X-MC-Unique": "anXjvrbsOoa2zub4ZRW5Rg-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": "Sun, 20 Dec 2020 22:13:25 +0100", "Message-Id": "<20201220211405.313012-1-maxime.coquelin@redhat.com>", "MIME-Version": "1.0", "X-Scanned-By": "MIMEDefang 2.79 on 10.5.11.12", "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 00/40] net/virtio: Virtio PMD rework", "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 <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": "This series significantly rework Virtio PMD to improve\nthe Virtio-user PMD and its backends integration.\n\nFirst part of the series (first 21 patches) removes the\ndependency of Virtio-user ethdev on Virtio PCI, by\ncreating generic files, 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. Now, the\nuser has to manually pass --iova-mode=va in EAL\nparameters, otherwise vdev probe will fail. In v21.11,\nwhen ABI/API can be changed, I will add vdev driver\nflags so that the Virtio-user PMD can request IOVA as VA\nmode to be used.\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\nPlese note that I haven't tested the last 5 patches yet,\nI will conduct more testing early next week.\n\nMaxime Coquelin (40):\n bus/vdev: add helper to get vdev from eth dev\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\n drivers/bus/vdev/rte_bus_vdev.h | 2 +\n drivers/net/virtio/meson.build | 6 +-\n drivers/net/virtio/virtio.c | 71 ++\n drivers/net/virtio/virtio.h | 247 ++++++\n drivers/net/virtio/virtio_ethdev.c | 441 +++------\n drivers/net/virtio/virtio_ethdev.h | 5 +-\n drivers/net/virtio/virtio_pci.c | 399 +++++----\n drivers/net/virtio/virtio_pci.h | 286 +-----\n drivers/net/virtio/virtio_pci_ethdev.c | 225 +++++\n drivers/net/virtio/virtio_ring.h | 2 +-\n drivers/net/virtio/virtio_rxtx.c | 90 +-\n drivers/net/virtio/virtio_rxtx_packed_avx.c | 18 +-\n drivers/net/virtio/virtio_rxtx_simple.h | 3 +-\n drivers/net/virtio/virtio_user/vhost.h | 80 +-\n drivers/net/virtio/virtio_user/vhost_kernel.c | 435 ++++++---\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 | 835 ++++++++++++++----\n drivers/net/virtio/virtio_user/vhost_vdpa.c | 257 ++++--\n .../net/virtio/virtio_user/virtio_user_dev.c | 490 +++++-----\n .../net/virtio/virtio_user/virtio_user_dev.h | 22 +-\n drivers/net/virtio/virtio_user_ethdev.c | 304 ++-----\n drivers/net/virtio/virtqueue.c | 6 +-\n drivers/net/virtio/virtqueue.h | 41 +-\n 24 files changed, 2481 insertions(+), 1810 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": 85524, "url": "