Cover Detail
Show a cover letter.
GET /api/covers/72798/?format=api
{ "id": 72798, "url": "https://patches.dpdk.org/api/covers/72798/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/cover/20200702083237.1215652-1-amorenoz@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": "<20200702083237.1215652-1-amorenoz@redhat.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20200702083237.1215652-1-amorenoz@redhat.com", "date": "2020-07-02T08:32:29", "name": "[(v20.08),v2,0/8] vhost: improve Vhost/vDPA device init", "submitter": { "id": 1397, "url": "https://patches.dpdk.org/api/people/1397/?format=api", "name": "Adrian Moreno", "email": "amorenoz@redhat.com" }, "mbox": "https://patches.dpdk.org/project/dpdk/cover/20200702083237.1215652-1-amorenoz@redhat.com/mbox/", "series": [ { "id": 10732, "url": "https://patches.dpdk.org/api/series/10732/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/list/?series=10732", "date": "2020-07-02T08:32:30", "name": "vhost: improve Vhost/vDPA device init", "version": 2, "mbox": "https://patches.dpdk.org/series/10732/mbox/" } ], "comments": "https://patches.dpdk.org/api/covers/72798/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 A536AA0524;\n\tThu, 2 Jul 2020 10:33:11 +0200 (CEST)", "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id E2C351D65A;\n\tThu, 2 Jul 2020 10:33:04 +0200 (CEST)", "from us-smtp-delivery-1.mimecast.com (us-smtp-1.mimecast.com\n [207.211.31.81]) by dpdk.org (Postfix) with ESMTP id 198DA1D626\n for <dev@dpdk.org>; Thu, 2 Jul 2020 10:33:03 +0200 (CEST)", "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-141-B5NRSoLIP-e6-tP50V5qQg-1; Thu, 02 Jul 2020 04:32:55 -0400", "from smtp.corp.redhat.com (int-mx07.intmail.prod.int.phx2.redhat.com\n [10.5.11.22])\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 AE310BFC1;\n Thu, 2 Jul 2020 08:32:53 +0000 (UTC)", "from amorenoz.users.ipa.redhat.com (ovpn-112-91.ams2.redhat.com\n [10.36.112.91])\n by smtp.corp.redhat.com (Postfix) with ESMTP id 374B11002397;\n Thu, 2 Jul 2020 08:32:48 +0000 (UTC)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com;\n s=mimecast20190719; t=1593678783;\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=/dueM3O+wy0rrSsV0pcB/hHWFO8JbuFMAi/S09ae3bQ=;\n b=itcJcSZsdiTjzQ/cc4CdoB3Uzdb+C6pxjNHA4kkpKOy7Hj1dh9RRLLik3ngbmCsQ1BgZaC\n 7dadg9a78Gz39BF80HeG9yH4Wy5UZl8r2HqLlkeSlzSCDw0UaEdEBUH3JgoQCMeDFw2Xxq\n 0cci2mIta8LHCMD2IWI2E3LDWCDeHSg=", "X-MC-Unique": "B5NRSoLIP-e6-tP50V5qQg-1", "From": "Adrian Moreno <amorenoz@redhat.com>", "To": "dev@dpdk.org, xiaolong.ye@intel.com, shahafs@mellanox.com,\n matan@mellanox.com, maxime.coquelin@redhat.com, xiao.w.wang@intel.com,\n viacheslavo@mellanox.com", "Cc": "jasowang@redhat.com, lulu@redhat.com, Adrian Moreno <amorenoz@redhat.com>", "Date": "Thu, 2 Jul 2020 10:32:29 +0200", "Message-Id": "<20200702083237.1215652-1-amorenoz@redhat.com>", "MIME-Version": "1.0", "X-Scanned-By": "MIMEDefang 2.84 on 10.5.11.22", "Authentication-Results": "relay.mimecast.com;\n auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=amorenoz@redhat.com", "X-Mimecast-Spam-Score": "0", "X-Mimecast-Originator": "redhat.com", "Content-Type": "text/plain; charset=US-ASCII", "Content-Transfer-Encoding": "8bit", "Subject": "[dpdk-dev] [PATCH (v20.08) v2 0/8] vhost: improve Vhost/vDPA device\n\tinit", "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": "The goal of this series is to make the Vhost/vDPA device init more\nrobust by adding support to a new protocol feature and two new messages.\n\nVHOST_USER_SET_STATUS is received by the backend when the driver updates\nthe virtio device status register.\n\nFor now this series only deals with the DRIVER_OK bit, which indicates\nthat the driver is done with the device initialization. So, if the\nfeature is negotiated, make it a requirement to consider the device\nready.\n\nFor example, such information helps the Vhost backend to know when it\ncan call the dev_conf vDPA callback.\n\nVIRTIO_USER_GET_STATUS is received by the backend to report the status\nof the virtio device as per the virtio specification.\n\nOne use of this message is for the frontend to read the status after\nthe feature negotiation. If the device has cleared the FEATURE_OK bit,\nthe driver should interpret that the device has rejected the features\nand should fail accordingly.\n\nBefore adding support for these new requests, some clean-ups and\nrefactoring are done as preliminary steps to make the code more easily\nreadable. Some vDPA APIs are also made mandatory.\n\nNote that the VHOST_USER_PROTOCOL_F_STATUS protocol feature requires\nfrontend support. For Qemu it has been posted [1] , reviewed and acked.\nHowever, it's still waiting for the nex merge windows, so this series\nshould not be merged until the former is.\n\nPosting it earler so it can be reviewed, specially after the recent\nseries developed by Matan which also affects the vhost ready status.\n\n[1] https://patchwork.ozlabs.org/project/qemu-devel/patch/20200514073332.1434576-1-maxime.coquelin@redhat.com/\n\n---\nChanges from v1:\n- Rebased on top of virtio-next:\n - Dropped [PATCH 9/9] vhost: only use vDPA config workaround if needed\n since the latest changes have made the workaround uncessary.\n - Dropped [PATCH 3/9] vdpa/ifc: add support to vDPA queue enable\n Callbacks are now expected to reconfigure the file descriptors.\n- Updated the message ids as per latest version of qemu spec:\n- Added VHOST_USER_GET_STATUS support\n- [Chenbo Xia] Add validate_msg_fds to set_staus message handler\n\n\nAdrian Moreno (1):\n vhost: add support for virtio get status message\n\nMaxime Coquelin (7):\n vhost: fix virtio ready flag check\n vhost: refactor Virtio ready check\n vhost: make some vDPA callbacks mandatory\n vhost: check vDPA configuration succeed\n vhost: add support for virtio status\n vdpa/ifc: enable status protocol feature\n vdpa/mlx5: enable status protocol feature\n\n drivers/vdpa/ifc/ifcvf_vdpa.c | 3 +-\n drivers/vdpa/mlx5/mlx5_vdpa.c | 3 +-\n lib/librte_vhost/rte_vdpa_dev.h | 14 ++--\n lib/librte_vhost/rte_vhost.h | 4 ++\n lib/librte_vhost/socket.c | 6 +-\n lib/librte_vhost/vdpa.c | 10 +++\n lib/librte_vhost/vhost.c | 3 +-\n lib/librte_vhost/vhost.h | 11 +++\n lib/librte_vhost/vhost_user.c | 117 +++++++++++++++++++++++++++-----\n lib/librte_vhost/vhost_user.h | 7 +-\n 10 files changed, 148 insertions(+), 30 deletions(-)" }