Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/139217/?format=api
{ "id": 139217, "url": "https://patches.dpdk.org/api/patches/139217/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/patch/20240409172811.27866-1-npratte@iol.unh.edu/", "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": "<20240409172811.27866-1-npratte@iol.unh.edu>", "list_archive_url": "https://inbox.dpdk.org/dev/20240409172811.27866-1-npratte@iol.unh.edu", "date": "2024-04-09T17:28:11", "name": "[v2] dts: Change hugepage runtime config to 2MB Exclusively", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": false, "hash": "68b4d5524f148411b4890f3efa2b0a94e79dd15a", "submitter": { "id": 3281, "url": "https://patches.dpdk.org/api/people/3281/?format=api", "name": "Nicholas Pratte", "email": "npratte@iol.unh.edu" }, "delegate": { "id": 1, "url": "https://patches.dpdk.org/api/users/1/?format=api", "username": "tmonjalo", "first_name": "Thomas", "last_name": "Monjalon", "email": "thomas@monjalon.net" }, "mbox": "https://patches.dpdk.org/project/dpdk/patch/20240409172811.27866-1-npratte@iol.unh.edu/mbox/", "series": [ { "id": 31713, "url": "https://patches.dpdk.org/api/series/31713/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/list/?series=31713", "date": "2024-04-09T17:28:11", "name": "[v2] dts: Change hugepage runtime config to 2MB Exclusively", "version": 2, "mbox": "https://patches.dpdk.org/series/31713/mbox/" } ], "comments": "https://patches.dpdk.org/api/patches/139217/comments/", "check": "success", "checks": "https://patches.dpdk.org/api/patches/139217/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 416D443E2C;\n\tTue, 9 Apr 2024 19:29:51 +0200 (CEST)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 12F82402EB;\n\tTue, 9 Apr 2024 19:29:51 +0200 (CEST)", "from mail-oa1-f47.google.com (mail-oa1-f47.google.com\n [209.85.160.47]) by mails.dpdk.org (Postfix) with ESMTP id 88B2B402C7\n for <dev@dpdk.org>; Tue, 9 Apr 2024 19:29:49 +0200 (CEST)", "by mail-oa1-f47.google.com with SMTP id\n 586e51a60fabf-22ec8db3803so443574fac.1\n for <dev@dpdk.org>; Tue, 09 Apr 2024 10:29:49 -0700 (PDT)", "from d121028.iol.unh.edu ([2606:4100:3880:1271:e2f8:4ec3:8bf3:864c])\n by smtp.gmail.com with ESMTPSA id\n pi34-20020a05620a37a200b0078bdce0acecsm4299919qkn.80.2024.04.09.10.29.47\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Tue, 09 Apr 2024 10:29:47 -0700 (PDT)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=iol.unh.edu; s=unh-iol; t=1712683789; x=1713288589; darn=dpdk.org;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:from:to:cc:subject:date\n :message-id:reply-to;\n bh=2LkRJl0nzbaWyEjd/4HkfnZ6RCSoaNSZX7Bs9QnHEMc=;\n b=S0KZi6bL6Urxcqx28GTFdYSmjKxA7EaLeh5UWLVi5oD5IqAa71HzL6+CyqViYPeZeJ\n fQlcegMV9KPAI7iAlp0gXOLufq1QfHdBIpkfQPPAitZCftoe0no/adSdGlBRBUmqJXQw\n 7VCqVxHDtplEI6pgM7AG2Q+eDzGxoEPPU1ncg=", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20230601; t=1712683789; x=1713288589;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc\n :subject:date:message-id:reply-to;\n bh=2LkRJl0nzbaWyEjd/4HkfnZ6RCSoaNSZX7Bs9QnHEMc=;\n b=tBZDuM7WRQqxAD2soBAHOEoqGPog+BItwyxkEF9YmYhDd2WET161FHKwzTxp4dkrvQ\n 3+B9N4rX52G0VVYZfH7KyfKO+Xn0nnCPIm7uQUGpZ3B584h5Vd0y/d0MPOHHnkFhSMKe\n eVjT5CMwtZs0jbOH/uxjd26S4iRN6Ver5/VVniqAFdEwbl8QeAQL5UAVQX9+VjhK0PFt\n Samd9kmIlafqG0Kxew6wPvR2oRo8nZsW4AEeAGAwxRdRfD4LvQKTkk4fzOl2t/5OGtGs\n bKHN3LosoSEmv8Yno6lni0mLpsEy6yvpmZsFs/2Bga8qiDwB3P9RDdHzGP8X79BzY9Px\n 1AYw==", "X-Gm-Message-State": "AOJu0Yy1o/xIWhChYZ78O3mksv/Szl9xJ4rm7MuVjRQWvp81i6EPMDWo\n Bb/xQzdMHq2V2DnyaYVaoXSgAiT3DAS+520AsNt6ZY6BH5f7ot/dNY7rTBBlv6g=", "X-Google-Smtp-Source": "\n AGHT+IF1T4qIlabz4u62Isc5AuzI2nriXQaWefr92t/1E7qHlG0cmvjfO6PPINyRaXOtbQsaUWbISg==", "X-Received": "by 2002:a05:6871:e01c:b0:22e:88bd:6b26 with SMTP id\n by28-20020a056871e01c00b0022e88bd6b26mr237159oac.3.1712683788728;\n Tue, 09 Apr 2024 10:29:48 -0700 (PDT)", "From": "Nicholas Pratte <npratte@iol.unh.edu>", "To": "paul.szczepanek@arm.com, bruce.richardson@intel.com,\n yoan.picchi@foss.arm.com, juraj.linkes@pantheon.tech, jspewock@iol.unh.edu,\n thomas@monjalon.net, mb@smartsharesystems.com, wathsala.vithanage@arm.com,\n probb@iol.unh.edu, Honnappa.Nagarahalli@arm.com", "Cc": "dev@dpdk.org,\n\tNicholas Pratte <npratte@iol.unh.edu>", "Subject": "[PATCH v2] dts: Change hugepage runtime config to 2MB Exclusively", "Date": "Tue, 9 Apr 2024 13:28:11 -0400", "Message-ID": "<20240409172811.27866-1-npratte@iol.unh.edu>", "X-Mailer": "git-send-email 2.44.0", "In-Reply-To": "<20240404153106.19047-1-npratte@iol.unh.edu>", "References": "<20240404153106.19047-1-npratte@iol.unh.edu>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "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": "The previous implementation configures and allocates hugepage sizes\nbased on a system default. This can lead to two problems: overallocation of\nhugepages (which may crash the remote host), and configuration of hugepage\nsizes that are not recommended during runtime. This new implementation\nallows only 2MB hugepage allocation during runtime; any other unique\nhugepage size must be configured by the end-user for initializing DTS.\n\nIf the amount of 2MB hugepages requested exceeds the amount of 2MB\nhugepages already configured on the system, then the system will remount\nhugepages to cover the difference. If the amount of hugepages requested is\neither less than or equal to the amount already configured on the system,\nthen nothing is done.\n\nBugzilla ID: 1370\nSigned-off-by: Nicholas Pratte <npratte@iol.unh.edu>\nReviewed-by: Jeremy Spewock <jspewock@iol.unh.edu>\n---\n dts/conf.yaml | 4 ++--\n dts/framework/config/__init__.py | 4 ++--\n dts/framework/config/conf_yaml_schema.json | 6 +++---\n dts/framework/config/types.py | 2 +-\n dts/framework/testbed_model/linux_session.py | 15 ++++++---------\n 5 files changed, 14 insertions(+), 17 deletions(-)", "diff": "diff --git a/dts/conf.yaml b/dts/conf.yaml\nindex 8068345dd5..56c3ae6f4c 100644\n--- a/dts/conf.yaml\n+++ b/dts/conf.yaml\n@@ -35,7 +35,7 @@ nodes:\n lcores: \"\" # use all the available logical cores\n use_first_core: false # tells DPDK to use any physical core\n memory_channels: 4 # tells DPDK to use 4 memory channels\n- hugepages: # optional; if removed, will use system hugepage configuration\n+ hugepages_2mb: # optional; if removed, will use system hugepage configuration\n amount: 256\n force_first_numa: false\n ports:\n@@ -71,7 +71,7 @@ nodes:\n os_driver: rdma\n peer_node: \"SUT 1\"\n peer_pci: \"0000:00:08.1\"\n- hugepages: # optional; if removed, will use system hugepage configuration\n+ hugepages_2mb: # optional; if removed, will use system hugepage configuration\n amount: 256\n force_first_numa: false\n traffic_generator:\ndiff --git a/dts/framework/config/__init__.py b/dts/framework/config/__init__.py\nindex 4cb5c74059..b6f820e39e 100644\n--- a/dts/framework/config/__init__.py\n+++ b/dts/framework/config/__init__.py\n@@ -255,8 +255,8 @@ def from_dict(\n Either an SUT or TG configuration instance.\n \"\"\"\n hugepage_config = None\n- if \"hugepages\" in d:\n- hugepage_config_dict = d[\"hugepages\"]\n+ if \"hugepages_2mb\" in d:\n+ hugepage_config_dict = d[\"hugepages_2mb\"]\n if \"force_first_numa\" not in hugepage_config_dict:\n hugepage_config_dict[\"force_first_numa\"] = False\n hugepage_config = HugepageConfiguration(**hugepage_config_dict)\ndiff --git a/dts/framework/config/conf_yaml_schema.json b/dts/framework/config/conf_yaml_schema.json\nindex 4731f4511d..f4d7199523 100644\n--- a/dts/framework/config/conf_yaml_schema.json\n+++ b/dts/framework/config/conf_yaml_schema.json\n@@ -146,7 +146,7 @@\n \"compiler\"\n ]\n },\n- \"hugepages\": {\n+ \"hugepages_2mb\": {\n \"type\": \"object\",\n \"description\": \"Optional hugepage configuration. If not specified, hugepages won't be configured and DTS will use system configuration.\",\n \"properties\": {\n@@ -253,8 +253,8 @@\n \"type\": \"integer\",\n \"description\": \"How many memory channels to use. Optional, defaults to 1.\"\n },\n- \"hugepages\": {\n- \"$ref\": \"#/definitions/hugepages\"\n+ \"hugepages_2mb\": {\n+ \"$ref\": \"#/definitions/hugepages_2mb\"\n },\n \"ports\": {\n \"type\": \"array\",\ndiff --git a/dts/framework/config/types.py b/dts/framework/config/types.py\nindex 1927910d88..016e0c3dbd 100644\n--- a/dts/framework/config/types.py\n+++ b/dts/framework/config/types.py\n@@ -46,7 +46,7 @@ class NodeConfigDict(TypedDict):\n \"\"\"Allowed keys and values.\"\"\"\n \n #:\n- hugepages: HugepageConfigurationDict\n+ hugepages_2mb: HugepageConfigurationDict\n #:\n name: str\n #:\ndiff --git a/dts/framework/testbed_model/linux_session.py b/dts/framework/testbed_model/linux_session.py\nindex 5d24030c3d..37f5eacb21 100644\n--- a/dts/framework/testbed_model/linux_session.py\n+++ b/dts/framework/testbed_model/linux_session.py\n@@ -15,7 +15,7 @@\n \n from typing_extensions import NotRequired\n \n-from framework.exception import RemoteCommandExecutionError\n+from framework.exception import ConfigurationError, RemoteCommandExecutionError\n from framework.utils import expand_range\n \n from .cpu import LogicalCore\n@@ -87,25 +87,22 @@ def get_dpdk_file_prefix(self, dpdk_prefix: str) -> str:\n def setup_hugepages(self, hugepage_count: int, force_first_numa: bool) -> None:\n \"\"\"Overrides :meth:`~.os_session.OSSession.setup_hugepages`.\"\"\"\n self._logger.info(\"Getting Hugepage information.\")\n- hugepage_size = self._get_hugepage_size()\n+ if \"hugepages-2048kB\" not in self.send_command(\"ls /sys/kernel/mm/hugepages\").stdout:\n+ raise ConfigurationError(\"2MB hugepages not supported by operating system\")\n hugepages_total = self._get_hugepages_total()\n self._numa_nodes = self._get_numa_nodes()\n \n- if force_first_numa or hugepages_total != hugepage_count:\n+ if force_first_numa or hugepages_total < hugepage_count:\n # when forcing numa, we need to clear existing hugepages regardless\n # of size, so they can be moved to the first numa node\n- self._configure_huge_pages(hugepage_count, hugepage_size, force_first_numa)\n+ self._configure_huge_pages(hugepage_count, 2048, force_first_numa)\n else:\n self._logger.info(\"Hugepages already configured.\")\n self._mount_huge_pages()\n \n- def _get_hugepage_size(self) -> int:\n- hugepage_size = self.send_command(\"awk '/Hugepagesize/ {print $2}' /proc/meminfo\").stdout\n- return int(hugepage_size)\n-\n def _get_hugepages_total(self) -> int:\n hugepages_total = self.send_command(\n- \"awk '/HugePages_Total/ { print $2 }' /proc/meminfo\"\n+ \"cat /sys/kernel/mm/hugepages/hugepages-2048kB/nr_hugepages\"\n ).stdout\n return int(hugepages_total)\n \n", "prefixes": [ "v2" ] }