From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-9.8 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS, USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2FAFFC433E0 for ; Mon, 6 Jul 2020 17:21:27 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id E106A206CD for ; Mon, 6 Jul 2020 17:21:26 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="WVD7dRWj" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org E106A206CD Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 88CEA6B000D; Mon, 6 Jul 2020 13:21:26 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 8639D6B000E; Mon, 6 Jul 2020 13:21:26 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 7543E6B0010; Mon, 6 Jul 2020 13:21:26 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0225.hostedemail.com [216.40.44.225]) by kanga.kvack.org (Postfix) with ESMTP id 6027C6B000D for ; Mon, 6 Jul 2020 13:21:26 -0400 (EDT) Received: from smtpin14.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay05.hostedemail.com (Postfix) with ESMTP id 146E8181AC9CB for ; Mon, 6 Jul 2020 17:21:26 +0000 (UTC) X-FDA: 77008317372.14.look01_4e0450226ead Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin14.hostedemail.com (Postfix) with ESMTP id D557318229837 for ; Mon, 6 Jul 2020 17:21:25 +0000 (UTC) X-HE-Tag: look01_4e0450226ead X-Filterd-Recvd-Size: 7906 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by imf48.hostedemail.com (Postfix) with ESMTP for ; Mon, 6 Jul 2020 17:21:25 +0000 (UTC) Received: from aquarius.haifa.ibm.com (nesher1.haifa.il.ibm.com [195.110.40.7]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 063D120722; Mon, 6 Jul 2020 17:21:19 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1594056084; bh=KWxR6mRU/GFOXIHUDBV7Fo+VmYibiF7rALXmdUu7XHQ=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=WVD7dRWjMXMMZFvE4ZL1idqQBVWov9lTmPqkY4r/8GpnlrqsGrRYbvvh6+GDkqUmC wMdUhTIqKbrJyUHQuvEk7Ti6pcMUF9eBvhKlDQh98freJVZrrPomDkfF7GmYbpXZqz V6Ziz0964HsWhgTNtGNx2C1nuae+TOz44EGUTESQ= From: Mike Rapoport To: linux-kernel@vger.kernel.org Cc: Alan Cox , Andrew Morton , Andy Lutomirski , Christopher Lameter , Dave Hansen , Idan Yaniv , James Bottomley , "Kirill A. Shutemov" , Matthew Wilcox , Peter Zijlstra , "Reshetova, Elena" , Thomas Gleixner , Tycho Andersen , linux-api@vger.kernel.org, linux-mm@kvack.org, Mike Rapoport Subject: [RFC PATCH v2 5/5] mm: secretmem: add ability to reserve memory at boot Date: Mon, 6 Jul 2020 20:20:51 +0300 Message-Id: <20200706172051.19465-6-rppt@kernel.org> X-Mailer: git-send-email 2.26.2 In-Reply-To: <20200706172051.19465-1-rppt@kernel.org> References: <20200706172051.19465-1-rppt@kernel.org> MIME-Version: 1.0 X-Rspamd-Queue-Id: D557318229837 X-Spamd-Result: default: False [0.00 / 100.00] X-Rspamd-Server: rspam01 Content-Transfer-Encoding: quoted-printable X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: From: Mike Rapoport Taking pages out from the direct map and bringing them back may create undesired fragmentation and usage of the smaller pages in the direct mapping of the physical memory. This can be avoided if a significantly large area of the physical memory would be reserved for secretmem purposes at boot time. Add ability to reserve physical memory for secretmem at boot time using "secretmem" kernel parameter and then use that reserved memory as a globa= l pool for secret memory needs. Signed-off-by: Mike Rapoport --- mm/secretmem.c | 145 ++++++++++++++++++++++++++++++++++++++++++++++--- 1 file changed, 137 insertions(+), 8 deletions(-) diff --git a/mm/secretmem.c b/mm/secretmem.c index c6fcf6d76951..d50ac9ea1844 100644 --- a/mm/secretmem.c +++ b/mm/secretmem.c @@ -6,6 +6,8 @@ #include #include #include +#include +#include #include #include #include @@ -28,6 +30,39 @@ struct secretmem_ctx { unsigned int mode; }; =20 +struct secretmem_pool { + struct gen_pool *pool; + unsigned long reserved_size; + void *reserved; +}; + +static struct secretmem_pool secretmem_pool; + +static struct page *secretmem_alloc_huge_page(gfp_t gfp) +{ + struct gen_pool *pool =3D secretmem_pool.pool; + unsigned long addr =3D 0; + struct page *page =3D NULL; + + if (pool) { + if (gen_pool_avail(pool) < HPAGE_PMD_SIZE) + return NULL; + + addr =3D gen_pool_alloc(pool, HPAGE_PMD_SIZE); + if (!addr) + return NULL; + + page =3D virt_to_page(addr); + } else { + page =3D alloc_pages(gfp, HPAGE_PMD_ORDER); + + if (page) + split_page(page, HPAGE_PMD_ORDER); + } + + return page; +} + static int secretmem_pool_increase(struct secretmem_ctx *ctx, gfp_t gfp) { unsigned long nr_pages =3D (1 << HPAGE_PMD_ORDER); @@ -36,12 +71,11 @@ static int secretmem_pool_increase(struct secretmem_c= tx *ctx, gfp_t gfp) struct page *page; int err; =20 - page =3D alloc_pages(gfp, HPAGE_PMD_ORDER); + page =3D secretmem_alloc_huge_page(gfp); if (!page) return -ENOMEM; =20 addr =3D (unsigned long)page_address(page); - split_page(page, HPAGE_PMD_ORDER); =20 err =3D gen_pool_add(pool, addr, HPAGE_PMD_SIZE, NUMA_NO_NODE); if (err) { @@ -250,11 +284,23 @@ struct file *secretmem_file_create(const char *name= , unsigned int flags) return file; } =20 -static void secretmem_cleanup_chunk(struct gen_pool *pool, - struct gen_pool_chunk *chunk, void *data) +static void secretmem_recycle_range(unsigned long start, unsigned long e= nd) +{ + unsigned long addr; + + for (addr =3D start; addr < end; addr +=3D PAGE_SIZE) { + struct page *page =3D virt_to_page(addr); + struct lruvec *lruvec =3D mem_cgroup_page_lruvec(page, page_pgdat(page= )); + + __ClearPageLRU(page); + del_page_from_lru_list(page, lruvec, page_off_lru(page)); + } + + gen_pool_free(secretmem_pool.pool, start, HPAGE_PMD_SIZE); +} + +static void secretmem_release_range(unsigned long start, unsigned long e= nd) { - unsigned long start =3D chunk->start_addr; - unsigned long end =3D chunk->end_addr; unsigned long nr_pages, addr; =20 nr_pages =3D (end - start + 1) / PAGE_SIZE; @@ -264,6 +310,18 @@ static void secretmem_cleanup_chunk(struct gen_pool = *pool, put_page(virt_to_page(addr)); } =20 +static void secretmem_cleanup_chunk(struct gen_pool *pool, + struct gen_pool_chunk *chunk, void *data) +{ + unsigned long start =3D chunk->start_addr; + unsigned long end =3D chunk->end_addr; + + if (secretmem_pool.pool) + secretmem_recycle_range(start, end); + else + secretmem_release_range(start, end); +} + static void secretmem_cleanup_pool(struct secretmem_ctx *ctx) { struct gen_pool *pool =3D ctx->pool; @@ -303,14 +361,85 @@ static struct file_system_type secretmem_fs =3D { .kill_sb =3D kill_anon_super, }; =20 +static int secretmem_reserved_mem_init(void) +{ + struct gen_pool *pool; + struct page *page; + void *addr; + int err; + + if (!secretmem_pool.reserved) + return 0; + + pool =3D gen_pool_create(PMD_SHIFT, NUMA_NO_NODE); + if (!pool) + return -ENOMEM; + + err =3D gen_pool_add(pool, (unsigned long)secretmem_pool.reserved, + secretmem_pool.reserved_size, NUMA_NO_NODE); + if (err) + goto err_destroy_pool; + + for (addr =3D secretmem_pool.reserved; + addr < secretmem_pool.reserved + secretmem_pool.reserved_size; + addr +=3D PAGE_SIZE) { + page =3D virt_to_page(addr); + __ClearPageReserved(page); + set_page_count(page, 1); + } + + secretmem_pool.pool =3D pool; + page =3D virt_to_page(secretmem_pool.reserved); + __kernel_map_pages(page, secretmem_pool.reserved_size / PAGE_SIZE, 0); + return 0; + +err_destroy_pool: + gen_pool_destroy(pool); + return err; +} + static int secretmem_init(void) { - int ret =3D 0; + int ret; + + ret =3D secretmem_reserved_mem_init(); + if (ret) + return ret; =20 secretmem_mnt =3D kern_mount(&secretmem_fs); - if (IS_ERR(secretmem_mnt)) + if (IS_ERR(secretmem_mnt)) { + gen_pool_destroy(secretmem_pool.pool); ret =3D PTR_ERR(secretmem_mnt); + } =20 return ret; } fs_initcall(secretmem_init); + +static int __init secretmem_setup(char *str) +{ + phys_addr_t align =3D HPAGE_PMD_SIZE; + unsigned long reserved_size; + void *reserved; + + reserved_size =3D memparse(str, NULL); + if (!reserved_size) + return 0; + + if (reserved_size * 2 > HPAGE_PUD_SIZE) + align =3D HPAGE_PUD_SIZE; + + reserved =3D memblock_alloc(reserved_size, align); + if (!reserved) { + pr_err("failed to reserve %zu bytes\n", secretmem_pool.reserved_size); + return 0; + } + + secretmem_pool.reserved_size =3D reserved_size; + secretmem_pool.reserved =3D reserved; + + pr_info("reserved %zuM\n", reserved_size >> 20); + + return 1; +} +__setup("secretmem=3D", secretmem_setup); --=20 2.26.2