Cover Detail
Show a cover letter.
GET /api/covers/73228/?format=api
{ "id": 73228, "url": "https://patches.dpdk.org/api/covers/73228/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/cover/20200706112452.1474533-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": "<20200706112452.1474533-1-amorenoz@redhat.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20200706112452.1474533-1-amorenoz@redhat.com", "date": "2020-07-06T11:24:44", "name": "[(v20.08),v3,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/20200706112452.1474533-1-amorenoz@redhat.com/mbox/", "series": [ { "id": 10810, "url": "https://patches.dpdk.org/api/series/10810/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/list/?series=10810", "date": "2020-07-06T11:24:45", "name": "vhost: improve Vhost/vDPA device init", "version": 3, "mbox": "https://patches.dpdk.org/series/10810/mbox/" } ], "comments": "https://patches.dpdk.org/api/covers/73228/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 6BB2DA00C5;\n\tMon, 6 Jul 2020 13:25:26 +0200 (CEST)", "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id 1A98F1D96E;\n\tMon, 6 Jul 2020 13:25:20 +0200 (CEST)", "from us-smtp-1.mimecast.com (us-smtp-delivery-1.mimecast.com\n [207.211.31.120]) by dpdk.org (Postfix) with ESMTP id A54231D93E\n for <dev@dpdk.org>; Mon, 6 Jul 2020 13:25:16 +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-365-8OWhlT4xNK65VW3nrDZ_qA-1; Mon, 06 Jul 2020 07:25:11 -0400", "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 027B6100CCC2;\n Mon, 6 Jul 2020 11:25:10 +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 C161760CC0;\n Mon, 6 Jul 2020 11:25:02 +0000 (UTC)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com;\n s=mimecast20190719; t=1594034716;\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=zmMHcUbUh+bNRFMFvBJ1eC6Em1aLL+qTaJxp65NmCH8=;\n b=RBJREF4E8fT37/RFco0Dq1PbSG+ZUGXW2t2KBQHvqhKlH72nbcahKgiFo6zcOBfqcNsmx1\n 7O8zoi6+1VeO33GtVrWy/REYVyAHUgCyyRqGiADxPT/8IJPxBdPg6HMMQJYFV2o78cCBEe\n 13cuf6UyhlgT6t5TmDXjSJbIxTh5Z4M=", "X-MC-Unique": "8OWhlT4xNK65VW3nrDZ_qA-1", "From": "Adrian Moreno <amorenoz@redhat.com>", "To": "dev@dpdk.org, chenbo.xia@intel.com, xiaolong.ye@intel.com,\n shahafs@mellanox.com, matan@mellanox.com, maxime.coquelin@redhat.com,\n xiao.w.wang@intel.com, viacheslavo@mellanox.com", "Cc": "jasowang@redhat.com, lulu@redhat.com, Adrian Moreno <amorenoz@redhat.com>", "Date": "Mon, 6 Jul 2020 13:24:44 +0200", "Message-Id": "<20200706112452.1474533-1-amorenoz@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=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) v3 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. The patch that adds such support in QEMU has been\nincluded in the latest Pull Request [1] and should be merged in the\ncoming days.\n\n[1] https://lists.gnu.org/archive/html/qemu-devel/2020-07/msg01641.html\n\n---\nChanges from v2:\n- [Chenbo] Return RTE_VHOST_MSG_RESULT_REPLY in get_status handler\n- Code style error\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(-)" }