Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/105532/?format=api
{ "id": 105532, "url": "http://patchwork.dpdk.org/api/patches/105532/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/patch/20211230143744.3550098-4-dkozlyuk@nvidia.com/", "project": { "id": 1, "url": "http://patchwork.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": "<20211230143744.3550098-4-dkozlyuk@nvidia.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20211230143744.3550098-4-dkozlyuk@nvidia.com", "date": "2021-12-30T14:37:41", "name": "[RFC,3/6] eal: refactor --huge-unlink storage", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "387684a182689d93a164cceea1ac9628cbf491f9", "submitter": { "id": 2248, "url": "http://patchwork.dpdk.org/api/people/2248/?format=api", "name": "Dmitry Kozlyuk", "email": "dkozlyuk@nvidia.com" }, "delegate": { "id": 24651, "url": "http://patchwork.dpdk.org/api/users/24651/?format=api", "username": "dmarchand", "first_name": "David", "last_name": "Marchand", "email": "david.marchand@redhat.com" }, "mbox": "http://patchwork.dpdk.org/project/dpdk/patch/20211230143744.3550098-4-dkozlyuk@nvidia.com/mbox/", "series": [ { "id": 21042, "url": "http://patchwork.dpdk.org/api/series/21042/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/list/?series=21042", "date": "2021-12-30T14:37:38", "name": "Fast restart with many hugepages", "version": 1, "mbox": "http://patchwork.dpdk.org/series/21042/mbox/" } ], "comments": "http://patchwork.dpdk.org/api/patches/105532/comments/", "check": "success", "checks": "http://patchwork.dpdk.org/api/patches/105532/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 EAAA3A04A5;\n\tThu, 30 Dec 2021 15:38:28 +0100 (CET)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id C5BBF4115D;\n\tThu, 30 Dec 2021 15:38:15 +0100 (CET)", "from NAM10-MW2-obe.outbound.protection.outlook.com\n (mail-mw2nam10on2055.outbound.protection.outlook.com [40.107.94.55])\n by mails.dpdk.org (Postfix) with ESMTP id 9BC0641148\n for <dev@dpdk.org>; Thu, 30 Dec 2021 15:38:12 +0100 (CET)", "from CO2PR18CA0063.namprd18.prod.outlook.com (2603:10b6:104:2::31)\n by DM6PR12MB4985.namprd12.prod.outlook.com (2603:10b6:5:16d::21) with\n Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4844.14; Thu, 30 Dec\n 2021 14:38:10 +0000", "from CO1NAM11FT053.eop-nam11.prod.protection.outlook.com\n (2603:10b6:104:2:cafe::ef) by CO2PR18CA0063.outlook.office365.com\n (2603:10b6:104:2::31) with Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4823.19 via Frontend\n Transport; Thu, 30 Dec 2021 14:38:10 +0000", "from mail.nvidia.com (12.22.5.236) by\n CO1NAM11FT053.mail.protection.outlook.com (10.13.175.63) with Microsoft SMTP\n Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id\n 15.20.4844.14 via Frontend Transport; Thu, 30 Dec 2021 14:38:10 +0000", "from rnnvmail201.nvidia.com (10.129.68.8) by DRHQMAIL109.nvidia.com\n (10.27.9.19) with Microsoft SMTP Server (TLS) id 15.0.1497.18;\n Thu, 30 Dec 2021 14:38:08 +0000", "from nvidia.com (172.20.187.6) by rnnvmail201.nvidia.com\n (10.129.68.8) with Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.2.986.9; Thu, 30 Dec 2021\n 06:38:04 -0800" ], "ARC-Seal": "i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none;\n b=EAB1dbAIdR2MMx4ZCbP/uNBlP7t/8/MPAYTZ+5AKWewM65hwdpg9SJBmZUwf5l9rf4i09CbrPr96UnmIwqwHfDVOfslCGLiU6V4XsPWs2ymqLi4gy3FQn1XrXkYihsSFQ+hLa3WO0FLdc4llZCYoFD3Vfye74eFi0J5ufgjRlFitB0i0NCWRABVAZHdW001f7xlo8zX+z5t2b8BuTBW3ChNnb88vUsg8QIm6UBDLdX8bi7Vqo09OFzk5qsniLHiz5H85Y1303LF9wPYEqOdKQ7YMLw8qijuJncyVUFtSEf3Zx8jkYrLE+NekYPon4SQAOfPchFx4nkrydApjBckM+Q==", "ARC-Message-Signature": "i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com;\n s=arcselector9901;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1;\n bh=MotN3DXJjnc90k7ER7XOP0TEwbKuHQSsDOkUzmyIzf0=;\n b=D2sMQFutSte/BN5GqkK9m7sMUXaAd3XL5o5P/r/0i/NMBvuzutYakSLPoyUTV4maKlmacmF5h6slA/CkCa++xrVODm1ZxVLRCIXqWzD1HXGfZsdiDTsm5ovsJuIFl8Jg5uUxN6ePxZu7pnJftXEeFra0uUSQCYBa4GzupdMNiMNG8IEashD0NFicoAPvsAb6rEoah6b9VLo9BY3EjEZlz/xrh9VAoz3XeYdUiHyvmQKqXgxU1dslmoyDRn3MeuGiqha2fEHwnYmJztARAX8bggkQncWHBGByQP/extrCWiR4orvocipqKeQ2VPmuv31m9AFKyr1G1bIHMKwXtx3Iwg==", "ARC-Authentication-Results": "i=1; mx.microsoft.com 1; spf=pass (sender ip is\n 12.22.5.236) smtp.rcpttodomain=intel.com smtp.mailfrom=nvidia.com; dmarc=pass\n (p=reject sp=reject pct=100) action=none header.from=nvidia.com; dkim=none\n (message not signed); arc=none", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com;\n s=selector2;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;\n bh=MotN3DXJjnc90k7ER7XOP0TEwbKuHQSsDOkUzmyIzf0=;\n b=hSNkMXu0Zlqyd6SNNb6waOS4JFkpkDRlwhlgCwg3m/GUwsVBx4PNsYeHA+DWGJhYPwbt61jY6UnsvsvEjMfJ4rXno1SXZ15bWWZsTxEoFf8acc75c9lULg4MlabRqY/SJfbI++qRXKij/ADy16v0VmPmVAnQTv2/ghzAC70+XYyW8FBOhzDiy+CoGra9BTXWsVYrvAurkdHD3cR+lLWSm6LQ0f760p3rrVIkJAO6Ig+ZVPq2Ci50uYmQUJhQIxw0W4bKPVFrFy8qos0n3Dj1UYfrDiGNLONWvh6giKv4BHPsV+1M+8uyK2WNF5bPWiC7wvlhBQdLTAo5XXH8T/ja2A==", "X-MS-Exchange-Authentication-Results": "spf=pass (sender IP is 12.22.5.236)\n smtp.mailfrom=nvidia.com; dkim=none (message not signed)\n header.d=none;dmarc=pass action=none header.from=nvidia.com;", "Received-SPF": "Pass (protection.outlook.com: domain of nvidia.com designates\n 12.22.5.236 as permitted sender) receiver=protection.outlook.com;\n client-ip=12.22.5.236; helo=mail.nvidia.com;", "From": "Dmitry Kozlyuk <dkozlyuk@nvidia.com>", "To": "<dev@dpdk.org>", "CC": "Anatoly Burakov <anatoly.burakov@intel.com>", "Subject": "[RFC PATCH 3/6] eal: refactor --huge-unlink storage", "Date": "Thu, 30 Dec 2021 16:37:41 +0200", "Message-ID": "<20211230143744.3550098-4-dkozlyuk@nvidia.com>", "X-Mailer": "git-send-email 2.25.1", "In-Reply-To": "<20211230143744.3550098-1-dkozlyuk@nvidia.com>", "References": "<20211230143744.3550098-1-dkozlyuk@nvidia.com>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Content-Type": "text/plain", "X-Originating-IP": "[172.20.187.6]", "X-ClientProxiedBy": "HQMAIL105.nvidia.com (172.20.187.12) To\n rnnvmail201.nvidia.com (10.129.68.8)", "X-EOPAttributedMessage": "0", "X-MS-PublicTrafficType": "Email", "X-MS-Office365-Filtering-Correlation-Id": "de36673d-a378-47ea-3fbc-08d9cba20070", "X-MS-TrafficTypeDiagnostic": "DM6PR12MB4985:EE_", "X-Microsoft-Antispam-PRVS": "\n <DM6PR12MB4985FAA3DABB394F616E4B2AB9459@DM6PR12MB4985.namprd12.prod.outlook.com>", "X-MS-Oob-TLC-OOBClassifiers": "OLM:1850;", "X-MS-Exchange-SenderADCheck": "1", "X-MS-Exchange-AntiSpam-Relay": "0", "X-Microsoft-Antispam": "BCL:0;", "X-Microsoft-Antispam-Message-Info": "\n bDutFIMqN5CI1ZE1F9tXual/T+b3nZpObpC9vMlJ7YtEOYm7/BJJal9JAVUSJm2cM0SAKOJ3FeGmNJu8BongHOe53UvtAxoagZQsY1fUIcmtu4h3XM6v7n7gpPsSEcL1KGwA6zQDGk7jg5CxNou6i7Q2siDgost9R8Vh3WTVAFoeT6MNXdbeHdzuzOwKtFPQveeliVpa7QU9GX9fqJ0wFFWntGNkiNbqwr35Xq2txcfy217z1owvBStWkzvT7lYbA97Bb8UlyaEYQGnmUYceoKupd0ZXvGSX16O1iPZMgTz8FCQaFpZsGT36qkeuewvfjiP96GH3Z+eTOa//nXI/SvXdY9wDq2hLq/A8dxF8zOXLoynj/5VBBPlQwQPOQT3Av46mBcB90b2Ux2QISROdzY+HPEWrzjvmlYmVeEtmCxeMvzUH5ez1V9js0yBCFnelO4O+9iZno4GYfk34gPJ/d2BQe03XN8nDrW4oy+DYloHq4HbwL5g38obnopDbxvtp/r28IQmGVNCz/dCOBqdGzKbfmPcLbWEybSFg9tRbw6miBcHsB0hJue7scP//WciNLTIq6QsqlQGWAMyLYzaL1+PkCvUc2dJWIzoAV8SWXCd9YpuRE8ZEREUCJnN3YBRt8lG428m/+/qedll6uo6bQq6EB7CgathKgMv8rTXwOaznTOrxSOTVc8lLbe0A7fojFUDLtDOZtU+mBVcQUFoZnqvZYHQLX3B/wg0G32ZAt/8LohJ5QB6fLuvdpp6QBJsyHQwcOT3OHegqEu8lr/FkCrOETg32qye7Urws8xYaMqNJcllNT6Qmkx6oxOS/b0vqpeIFePYzZkAE6D2CmBjobA==", "X-Forefront-Antispam-Report": "CIP:12.22.5.236; CTRY:US; LANG:en; SCL:1; SRV:;\n IPV:CAL; SFV:NSPM; H:mail.nvidia.com; PTR:InfoNoRecords; CAT:NONE;\n SFS:(4636009)(46966006)(40470700002)(36840700001)(40460700001)(16526019)(186003)(1076003)(426003)(70206006)(4326008)(7696005)(6916009)(55016003)(8936002)(82310400004)(356005)(83380400001)(6666004)(86362001)(70586007)(336012)(2616005)(2906002)(8676002)(81166007)(316002)(26005)(36860700001)(36756003)(508600001)(47076005)(6286002)(5660300002)(14583001)(36900700001);\n DIR:OUT; SFP:1101;", "X-OriginatorOrg": "Nvidia.com", "X-MS-Exchange-CrossTenant-OriginalArrivalTime": "30 Dec 2021 14:38:10.0882 (UTC)", "X-MS-Exchange-CrossTenant-Network-Message-Id": "\n de36673d-a378-47ea-3fbc-08d9cba20070", "X-MS-Exchange-CrossTenant-Id": "43083d15-7273-40c1-b7db-39efd9ccc17a", "X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp": "\n TenantId=43083d15-7273-40c1-b7db-39efd9ccc17a; Ip=[12.22.5.236];\n Helo=[mail.nvidia.com]", "X-MS-Exchange-CrossTenant-AuthSource": "\n CO1NAM11FT053.eop-nam11.prod.protection.outlook.com", "X-MS-Exchange-CrossTenant-AuthAs": "Anonymous", "X-MS-Exchange-CrossTenant-FromEntityHeader": "HybridOnPrem", "X-MS-Exchange-Transport-CrossTenantHeadersStamped": "DM6PR12MB4985", "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": "In preparation to extend --huge-unlink option semantics\nrefactor how it is stored in the internal configuration.\nIt makes future changes more isolated.\n\nSigned-off-by: Dmitry Kozlyuk <dkozlyuk@nvidia.com>\n---\n lib/eal/common/eal_common_options.c | 9 +++++----\n lib/eal/common/eal_internal_cfg.h | 8 +++++++-\n lib/eal/linux/eal_memalloc.c | 7 ++++---\n lib/eal/linux/eal_memory.c | 2 +-\n 4 files changed, 17 insertions(+), 9 deletions(-)", "diff": "diff --git a/lib/eal/common/eal_common_options.c b/lib/eal/common/eal_common_options.c\nindex 1cfdd75f3b..7520ebda8e 100644\n--- a/lib/eal/common/eal_common_options.c\n+++ b/lib/eal/common/eal_common_options.c\n@@ -1737,7 +1737,7 @@ eal_parse_common_option(int opt, const char *optarg,\n \n \t/* long options */\n \tcase OPT_HUGE_UNLINK_NUM:\n-\t\tconf->hugepage_unlink = 1;\n+\t\tconf->hugepage_file.unlink_before_mapping = true;\n \t\tbreak;\n \n \tcase OPT_NO_HUGE_NUM:\n@@ -1766,7 +1766,7 @@ eal_parse_common_option(int opt, const char *optarg,\n \t\tconf->in_memory = 1;\n \t\t/* in-memory is a superset of noshconf and huge-unlink */\n \t\tconf->no_shconf = 1;\n-\t\tconf->hugepage_unlink = 1;\n+\t\tconf->hugepage_file.unlink_before_mapping = true;\n \t\tbreak;\n \n \tcase OPT_PROC_TYPE_NUM:\n@@ -2050,7 +2050,8 @@ eal_check_common_options(struct internal_config *internal_cfg)\n \t\t\t\"be specified together with --\"OPT_NO_HUGE\"\\n\");\n \t\treturn -1;\n \t}\n-\tif (internal_cfg->no_hugetlbfs && internal_cfg->hugepage_unlink &&\n+\tif (internal_cfg->no_hugetlbfs &&\n+\t\t\tinternal_cfg->hugepage_file.unlink_before_mapping &&\n \t\t\t!internal_cfg->in_memory) {\n \t\tRTE_LOG(ERR, EAL, \"Option --\"OPT_HUGE_UNLINK\" cannot \"\n \t\t\t\"be specified together with --\"OPT_NO_HUGE\"\\n\");\n@@ -2061,7 +2062,7 @@ eal_check_common_options(struct internal_config *internal_cfg)\n \t\t\t\" is only supported in non-legacy memory mode\\n\");\n \t}\n \tif (internal_cfg->single_file_segments &&\n-\t\t\tinternal_cfg->hugepage_unlink &&\n+\t\t\tinternal_cfg->hugepage_file.unlink_before_mapping &&\n \t\t\t!internal_cfg->in_memory) {\n \t\tRTE_LOG(ERR, EAL, \"Option --\"OPT_SINGLE_FILE_SEGMENTS\" is \"\n \t\t\t\"not compatible with --\"OPT_HUGE_UNLINK\"\\n\");\ndiff --git a/lib/eal/common/eal_internal_cfg.h b/lib/eal/common/eal_internal_cfg.h\nindex d6c0470eb8..b5e6942578 100644\n--- a/lib/eal/common/eal_internal_cfg.h\n+++ b/lib/eal/common/eal_internal_cfg.h\n@@ -40,6 +40,12 @@ struct simd_bitwidth {\n \tuint16_t bitwidth; /**< bitwidth value */\n };\n \n+/** Hugepage backing files discipline. */\n+struct hugepage_file_discipline {\n+\t/** Unlink files before mapping them to leave no trace in hugetlbfs. */\n+\tbool unlink_before_mapping;\n+};\n+\n /**\n * internal configuration\n */\n@@ -48,7 +54,7 @@ struct internal_config {\n \tvolatile unsigned force_nchannel; /**< force number of channels */\n \tvolatile unsigned force_nrank; /**< force number of ranks */\n \tvolatile unsigned no_hugetlbfs; /**< true to disable hugetlbfs */\n-\tunsigned hugepage_unlink; /**< true to unlink backing files */\n+\tstruct hugepage_file_discipline hugepage_file;\n \tvolatile unsigned no_pci; /**< true to disable PCI */\n \tvolatile unsigned no_hpet; /**< true to disable HPET */\n \tvolatile unsigned vmware_tsc_map; /**< true to use VMware TSC mapping\ndiff --git a/lib/eal/linux/eal_memalloc.c b/lib/eal/linux/eal_memalloc.c\nindex 337f2bc739..abbe605e49 100644\n--- a/lib/eal/linux/eal_memalloc.c\n+++ b/lib/eal/linux/eal_memalloc.c\n@@ -564,7 +564,7 @@ alloc_seg(struct rte_memseg *ms, void *addr, int socket_id,\n \t\t\t\t\t__func__, strerror(errno));\n \t\t\t\tgoto resized;\n \t\t\t}\n-\t\t\tif (internal_conf->hugepage_unlink &&\n+\t\t\tif (internal_conf->hugepage_file.unlink_before_mapping &&\n \t\t\t\t\t!internal_conf->in_memory) {\n \t\t\t\tif (unlink(path)) {\n \t\t\t\t\tRTE_LOG(DEBUG, EAL, \"%s(): unlink() failed: %s\\n\",\n@@ -697,7 +697,7 @@ alloc_seg(struct rte_memseg *ms, void *addr, int socket_id,\n \t\t\tclose_hugefile(fd, path, list_idx);\n \t} else {\n \t\t/* only remove file if we can take out a write lock */\n-\t\tif (internal_conf->hugepage_unlink == 0 &&\n+\t\tif (!internal_conf->hugepage_file.unlink_before_mapping &&\n \t\t\t\tinternal_conf->in_memory == 0 &&\n \t\t\t\tlock(fd, LOCK_EX) == 1)\n \t\t\tunlink(path);\n@@ -756,7 +756,8 @@ free_seg(struct rte_memseg *ms, struct hugepage_info *hi,\n \t\t/* if we're able to take out a write lock, we're the last one\n \t\t * holding onto this page.\n \t\t */\n-\t\tif (!internal_conf->in_memory && !internal_conf->hugepage_unlink) {\n+\t\tif (!internal_conf->in_memory &&\n+\t\t\t\tinternal_conf->hugepage_file.unlink_before_mapping) {\n \t\t\tret = lock(fd, LOCK_EX);\n \t\t\tif (ret >= 0) {\n \t\t\t\t/* no one else is using this page */\ndiff --git a/lib/eal/linux/eal_memory.c b/lib/eal/linux/eal_memory.c\nindex 03a4f2dd2d..83eec078a4 100644\n--- a/lib/eal/linux/eal_memory.c\n+++ b/lib/eal/linux/eal_memory.c\n@@ -1428,7 +1428,7 @@ eal_legacy_hugepage_init(void)\n \t}\n \n \t/* free the hugepage backing files */\n-\tif (internal_conf->hugepage_unlink &&\n+\tif (internal_conf->hugepage_file.unlink_before_mapping &&\n \t\tunlink_hugepage_files(tmp_hp, internal_conf->num_hugepage_sizes) < 0) {\n \t\tRTE_LOG(ERR, EAL, \"Unlinking hugepage files failed!\\n\");\n \t\tgoto fail;\n", "prefixes": [ "RFC", "3/6" ] }