Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/127472/?format=api
{ "id": 127472, "url": "https://patches.dpdk.org/api/patches/127472/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/patch/20230525162551.70359-4-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": "<20230525162551.70359-4-maxime.coquelin@redhat.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20230525162551.70359-4-maxime.coquelin@redhat.com", "date": "2023-05-25T16:25:26", "name": "[v3,03/28] vhost: fix IOTLB entries overlap check with previous entry", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "af025b2e2d12d0606e1075cecf5d29e5a2403e21", "submitter": { "id": 512, "url": "https://patches.dpdk.org/api/people/512/?format=api", "name": "Maxime Coquelin", "email": "maxime.coquelin@redhat.com" }, "delegate": { "id": 2642, "url": "https://patches.dpdk.org/api/users/2642/?format=api", "username": "mcoquelin", "first_name": "Maxime", "last_name": "Coquelin", "email": "maxime.coquelin@redhat.com" }, "mbox": "https://patches.dpdk.org/project/dpdk/patch/20230525162551.70359-4-maxime.coquelin@redhat.com/mbox/", "series": [ { "id": 28187, "url": "https://patches.dpdk.org/api/series/28187/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/list/?series=28187", "date": "2023-05-25T16:25:23", "name": "Add VDUSE support to Vhost library", "version": 3, "mbox": "https://patches.dpdk.org/series/28187/mbox/" } ], "comments": "https://patches.dpdk.org/api/patches/127472/comments/", "check": "warning", "checks": "https://patches.dpdk.org/api/patches/127472/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 E08F842B9D;\n\tThu, 25 May 2023 18:26:29 +0200 (CEST)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 8ED3542D59;\n\tThu, 25 May 2023 18:26:27 +0200 (CEST)", "from us-smtp-delivery-124.mimecast.com\n (us-smtp-delivery-124.mimecast.com [170.10.129.124])\n by mails.dpdk.org (Postfix) with ESMTP id 2F0B640EDF\n for <dev@dpdk.org>; Thu, 25 May 2023 18:26:26 +0200 (CEST)", "from mimecast-mx02.redhat.com (mx3-rdu2.redhat.com\n [66.187.233.73]) by relay.mimecast.com with ESMTP with STARTTLS\n (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id\n us-mta-231-X33gCSIqN9CjOOixuO6ZtA-1; Thu, 25 May 2023 12:26:21 -0400", "from smtp.corp.redhat.com (int-mx07.intmail.prod.int.rdu2.redhat.com\n [10.11.54.7])\n (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))\n (No client certificate requested)\n by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 097503800E94;\n Thu, 25 May 2023 16:26:21 +0000 (UTC)", "from max-t490s.redhat.com (unknown [10.39.208.23])\n by smtp.corp.redhat.com (Postfix) with ESMTP id 67249140E95D;\n Thu, 25 May 2023 16:26:17 +0000 (UTC)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com;\n s=mimecast20190719; t=1685031985;\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 in-reply-to:in-reply-to:references:references;\n bh=kdk7A4dPJ3aSyRCNhnDx2xvj/ieo1OEJTR+LhLYwHNo=;\n b=brJ1OORujkf9rXP82exgvLfPCFVQxK+pTzWXJXZo6GR68OFNFg86eMBig3KyTU0s3ffq6D\n hBOCqKn6qIQZKkuu/2yJjdu3VR5Gunn+w32W5p9g19uVvZHKzfeG1E+MBoAk/UpMyv3rEh\n Fz98aO5b37wCPVw/5ixPTKBjzJ61eE8=", "X-MC-Unique": "X33gCSIqN9CjOOixuO6ZtA-1", "From": "Maxime Coquelin <maxime.coquelin@redhat.com>", "To": "dev@dpdk.org, chenbo.xia@intel.com, david.marchand@redhat.com,\n mkp@redhat.com, fbl@redhat.com, jasowang@redhat.com,\n cunming.liang@intel.com, xieyongji@bytedance.com, echaudro@redhat.com,\n eperezma@redhat.com, amorenoz@redhat.com, lulu@redhat.com", "Cc": "Maxime Coquelin <maxime.coquelin@redhat.com>,\n\tstable@dpdk.org", "Subject": "[PATCH v3 03/28] vhost: fix IOTLB entries overlap check with previous\n entry", "Date": "Thu, 25 May 2023 18:25:26 +0200", "Message-Id": "<20230525162551.70359-4-maxime.coquelin@redhat.com>", "In-Reply-To": "<20230525162551.70359-1-maxime.coquelin@redhat.com>", "References": "<20230525162551.70359-1-maxime.coquelin@redhat.com>", "MIME-Version": "1.0", "X-Scanned-By": "MIMEDefang 3.1 on 10.11.54.7", "X-Mimecast-Spam-Score": "0", "X-Mimecast-Originator": "redhat.com", "Content-Transfer-Encoding": "8bit", "Content-Type": "text/plain; charset=\"US-ASCII\"; x-default=true", "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": "Commit 22b6d0ac691a (\"vhost: fix madvise IOTLB entries pages overlap check\")\nfixed the check to ensure the entry to be removed does not\noverlap with the next one in the IOTLB cache before marking\nit as DONTDUMP with madvise(). This is not enough, because\nthe same issue is present when comparing with the previous\nentry in the cache, where the end address of the previous\nentry should be used, not the start one.\n\nFixes: dea092d0addb (\"vhost: fix madvise arguments alignment\")\nCc: stable@dpdk.org\n\nSigned-off-by: Maxime Coquelin <maxime.coquelin@redhat.com>\nAcked-by: Mike Pattrick <mkp@redhat.com>\nReviewed-by: Chenbo Xia <chenbo.xia@intel.com>\n---\n lib/vhost/iotlb.c | 8 ++++----\n 1 file changed, 4 insertions(+), 4 deletions(-)", "diff": "diff --git a/lib/vhost/iotlb.c b/lib/vhost/iotlb.c\nindex 3f45bc6061..870c8acb88 100644\n--- a/lib/vhost/iotlb.c\n+++ b/lib/vhost/iotlb.c\n@@ -178,8 +178,8 @@ vhost_user_iotlb_cache_random_evict(struct virtio_net *dev, struct vhost_virtque\n \t\t\tmask = ~(alignment - 1);\n \n \t\t\t/* Don't disable coredump if the previous node is in the same page */\n-\t\t\tif (prev_node == NULL ||\n-\t\t\t\t\t(node->uaddr & mask) != (prev_node->uaddr & mask)) {\n+\t\t\tif (prev_node == NULL || (node->uaddr & mask) !=\n+\t\t\t\t\t((prev_node->uaddr + prev_node->size - 1) & mask)) {\n \t\t\t\tnext_node = RTE_TAILQ_NEXT(node, next);\n \t\t\t\t/* Don't disable coredump if the next node is in the same page */\n \t\t\t\tif (next_node == NULL || ((node->uaddr + node->size - 1) & mask) !=\n@@ -283,8 +283,8 @@ vhost_user_iotlb_cache_remove(struct virtio_net *dev, struct vhost_virtqueue *vq\n \t\t\tmask = ~(alignment-1);\n \n \t\t\t/* Don't disable coredump if the previous node is in the same page */\n-\t\t\tif (prev_node == NULL ||\n-\t\t\t\t\t(node->uaddr & mask) != (prev_node->uaddr & mask)) {\n+\t\t\tif (prev_node == NULL || (node->uaddr & mask) !=\n+\t\t\t\t\t((prev_node->uaddr + prev_node->size - 1) & mask)) {\n \t\t\t\tnext_node = RTE_TAILQ_NEXT(node, next);\n \t\t\t\t/* Don't disable coredump if the next node is in the same page */\n \t\t\t\tif (next_node == NULL || ((node->uaddr + node->size - 1) & mask) !=\n", "prefixes": [ "v3", "03/28" ] }