From patchwork Mon Jun 3 23:33:31 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Steven Rostedt X-Patchwork-Id: 13684473 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id CF4BEC25B75 for ; Mon, 3 Jun 2024 23:35:27 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 4DCBE6B009C; Mon, 3 Jun 2024 19:35:27 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 461126B00A0; Mon, 3 Jun 2024 19:35:27 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 301736B009F; Mon, 3 Jun 2024 19:35:27 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id 151836B009B for ; Mon, 3 Jun 2024 19:35:27 -0400 (EDT) Received: from smtpin08.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id CCC521C106B for ; Mon, 3 Jun 2024 23:35:26 +0000 (UTC) X-FDA: 82191186252.08.EEE012D Received: from sin.source.kernel.org (sin.source.kernel.org [145.40.73.55]) by imf19.hostedemail.com (Postfix) with ESMTP id 889361A0006 for ; Mon, 3 Jun 2024 23:35:24 +0000 (UTC) Authentication-Results: imf19.hostedemail.com; dkim=none; dmarc=none; spf=pass (imf19.hostedemail.com: domain of "SRS0=/hzC=NF=goodmis.org=rostedt@kernel.org" designates 145.40.73.55 as permitted sender) smtp.mailfrom="SRS0=/hzC=NF=goodmis.org=rostedt@kernel.org" ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1717457725; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding:in-reply-to: references:references; bh=pkJubAnRN4tiBHGuDpqI6NmNsrLlhwWE7ztLdFn9uwo=; b=Bb9wG7C4wmeuoFfo+ZAEkJol5AhLDjm0DcoRId1sRlftojc2VqlSzVtN+r/h7ruv8uNiR2 bWZwLoBVjE/MD6m7NH9aLw2cYV89bQThwPukI4kM7cR/k4iTkpLQDAbO2YNwJqyL/QOljh LllV6OZPzfw7NpC7E9qGW1NbDhzC8y0= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1717457725; a=rsa-sha256; cv=none; b=j4VSwKCLVj9lptPUf2ZzhaItpE9ngzwdoCMk+O5uZ3ulaB8JvxwMdE+S0KsAN0wLbR+uxD H3h9WwDxKr7H2IuykKAUJ2WTuQ5RJ2neoTVeJdnMPWdNkKZoNbHzmHXeoQxwQ4teLWvYwg 6+hGh2HIV9ky6XApCrzrFhouAfwWB/c= ARC-Authentication-Results: i=1; imf19.hostedemail.com; dkim=none; dmarc=none; spf=pass (imf19.hostedemail.com: domain of "SRS0=/hzC=NF=goodmis.org=rostedt@kernel.org" designates 145.40.73.55 as permitted sender) smtp.mailfrom="SRS0=/hzC=NF=goodmis.org=rostedt@kernel.org" Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sin.source.kernel.org (Postfix) with ESMTP id DBDDBCE0F7D; Mon, 3 Jun 2024 23:35:20 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 27D25C32781; Mon, 3 Jun 2024 23:35:20 +0000 (UTC) Received: from rostedt by gandalf with local (Exim 4.97) (envelope-from ) id 1sEHE7-00000009cvT-2ViM; Mon, 03 Jun 2024 19:36:31 -0400 Message-ID: <20240603233631.452433539@goodmis.org> User-Agent: quilt/0.68 Date: Mon, 03 Jun 2024 19:33:31 -0400 From: Steven Rostedt To: linux-kernel@vger.kernel.org, linux-trace-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton , "Liam R. Howlett" , Vlastimil Babka , Lorenzo Stoakes , linux-mm@kvack.org, Thomas Gleixner , Ingo Molnar , Borislav Petkov , Dave Hansen , x86@kernel.org, "H. Peter Anvin" , Peter Zijlstra , Kees Cook , Tony Luck , "Guilherme G. Piccoli" , linux-hardening@vger.kernel.org, Guenter Roeck , Ross Zwisler , wklin@google.com, Vineeth Remanan Pillai , Joel Fernandes , Suleiman Souhlal , Linus Torvalds , Catalin Marinas , Will Deacon , Mike Rapoport Subject: [PATCH 1/2] mm/memblock: Add "reserve_mem" to reserved named memory at boot up References: <20240603233330.801075898@goodmis.org> MIME-Version: 1.0 X-Stat-Signature: pq7ur96uns575a94631wt1d18wzx4pc1 X-Rspamd-Queue-Id: 889361A0006 X-Rspam-User: X-Rspamd-Server: rspam01 X-HE-Tag: 1717457724-641225 X-HE-Meta: U2FsdGVkX1/tSIwkNFs33hAtkxn4BXT4GnsqxTQg6lMIdwz1lS6KH8ul01o+atsdmnH45okv+UmNcpuvROwMv/oLTIFuNv+sv/AQWCdFK48H6ZyVotuN/3D7cpkTqNewe/ltjtmOYno6n3Rdry8CQt7oKRoo9QDgWlz93ouj6Or6bg6/gcXrVlhAJZ6Qt0RVRwFqZjD0QGAyR5in3uqhwa/8yRKH804Ebi+cipVSqRqHEP8+l2beoNRFrpMHXWmDhOYgaSlzbKWigYXBn4YeOvc8aDIbD5TJKWpfiZCwdGp6BlAHz78qj/7kbBPGZa0KL1vPBMmDIfxf7V7BpmzcXy5rhRxe5UM8jW5OdaIaeD+qauAHlf+J5kvUmhqBa+cGb0wphMBQDp6Wl5/yC0xa3LsoE7+fVWzVA1lvOI5ya191FzveKuIpSvyV3+c+aUsuFaK1Yy4LZcfSHy4VHVDIsap0FEkWH6L0Q7dXfonQ2wO0oS5omdiME5ldTZ3HxVSA1O+91WVXepQExY7cOICuu+G+ptGq8+dNFo8kvPHH2JDSKXtOmLtra8Hp2C3Vhf/YuHeGJ9tOSeyXnucFB83WU8XuxR9foBMyI11o+cm7+Wr9kaj9IhOpllqwErk+irMv3PlgzUBwsRqCeXdHH86kyg/lhSf20mXgSpY1bR93ns63XkUmgTdwbXQIM+fMlWwDVPv/9CQ6VMBI/2iBKea2qu2H5rdpfZS3rT9aZGFgYU4QURwx+IOucas/H4uIPlebKA6zKh5jLRrTKmFJ+4uCEDV/6t5e6NHLp9jmsF07QVr7nj7HCck5Zu33C5K8S0R3I3gYUMXwDKxdPwu50BYXEktmyi0LkvbtJbRlxHYfRY3AtOkjJFI/cRpqQhJPPkrSzl5tS6ClWEv0QCmUotfAX0YUL0G2XuGW077QlplQ1kHrsYlJxf7dbtYa97kZmDwAadngDu7tpEJ9OCrn9+y 9VjD4S7s TKm8mZ3bUMzXi4EgzBnXxZLh4dBw5cj4579iIxxoxs1nusbBHW1Cf5v9DlmptvePN17XSVwzIS6vFClpvpRHFnOxSvG5Utnn+RckYECGGSyDGURe4OcrE2YWbJuTuttXxec89kVZBp7HKC83ELPK+a+DomyrLLHcsB5KX4FnRtz3D893TTM8u1IX1zz2H1KgJHMH9DQlL6hbLEH0oD08LwWr3yAUF1ftg2OFJF/e3824wiDG5B/JT6Ge94u8XboS1O8wRw40YT0q7p6Ik2I2HnUir49DqgPQfqzTOtU8vexIj0YrlNNbHGrox/O+JxP/72nWYopQ0ivWA9zQIB+phxxI26KDuZvH8N7hoSZJLTZxcCPW20Jnw31ATVVP0A8Qt/YMZlVk60yu5UnFuzPHS5+mGM6Rk60bkKr7306/XLIsW6WHLyAWDtLCHTWVSf+NGOOYCDj2oeVKKUbc= 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: List-Subscribe: List-Unsubscribe: From: "Steven Rostedt (Google)" In order to allow for requesting a memory region that can be used for things like pstore on multiple machines where the memory layout is not the same, add a new option to the kernel command line called "reserve_mem". The format is: reserve_mem=nn:align:name Where it will find nn amount of memory at the given alignment of align. The name field is to allow another subsystem to retrieve where the memory was found. For example: reserve_mem=12M:4096:oops ramoops.mem_name=oops Where ramoops.mem_name will tell ramoops that memory was reserved for it via the reserve_mem option and it can find it by calling: if (reserve_mem_find_by_name("oops", &start, &size)) { // start holds the start address and size holds the size given Link: https://lore.kernel.org/all/ZjJVnZUX3NZiGW6q@kernel.org/ Suggested-by: Mike Rapoport Signed-off-by: Steven Rostedt (Google) --- include/linux/mm.h | 2 + mm/memblock.c | 97 ++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 99 insertions(+) diff --git a/include/linux/mm.h b/include/linux/mm.h index 9849dfda44d4..b4455cc02f2c 100644 --- a/include/linux/mm.h +++ b/include/linux/mm.h @@ -4263,4 +4263,6 @@ static inline bool pfn_is_unaccepted_memory(unsigned long pfn) void vma_pgtable_walk_begin(struct vm_area_struct *vma); void vma_pgtable_walk_end(struct vm_area_struct *vma); +int reserve_mem_find_by_name(const char *name, unsigned long *start, unsigned long *size); + #endif /* _LINUX_MM_H */ diff --git a/mm/memblock.c b/mm/memblock.c index d09136e040d3..a8bf0ee9e2b4 100644 --- a/mm/memblock.c +++ b/mm/memblock.c @@ -2244,6 +2244,103 @@ void __init memblock_free_all(void) totalram_pages_add(pages); } +/* Keep a table to reserve named memory */ +#define RESERVE_MEM_MAX_ENTRIES 8 +#define RESERVE_MEM_NAME_SIZE 16 +struct reserve_mem_table { + char name[RESERVE_MEM_NAME_SIZE]; + unsigned long start; + unsigned long size; +}; +static struct reserve_mem_table reserved_mem_table[RESERVE_MEM_MAX_ENTRIES]; +static int reserved_mem_count; + +/* Add wildcard region with a lookup name */ +static int __init reserved_mem_add(unsigned long start, unsigned long size, + const char *name) +{ + struct reserve_mem_table *map; + + if (!name || !name[0] || strlen(name) >= RESERVE_MEM_NAME_SIZE) + return -EINVAL; + + if (reserved_mem_count >= RESERVE_MEM_MAX_ENTRIES) + return -1; + + map = &reserved_mem_table[reserved_mem_count++]; + map->start = start; + map->size = size; + strscpy(map->name, name); + return 0; +} + +/** + * reserve_mem_find_by_name - Find reserved memory region with a given name + * @name: The name that is attached to a reserved memory region + * @start: If found, holds the start address + * @size: If found, holds the size of the address. + * + * Returns: 1 if found or 0 if not found. + */ +int reserve_mem_find_by_name(const char *name, unsigned long *start, unsigned long *size) +{ + struct reserve_mem_table *map; + int i; + + for (i = 0; i < reserved_mem_count; i++) { + map = &reserved_mem_table[i]; + if (!map->size) + continue; + if (strcmp(name, map->name) == 0) { + *start = map->start; + *size = map->size; + return 1; + } + } + return 0; +} + +/* + * Parse early_reserve_mem=nn:align:name + */ +static int __init reserve_mem(char *p) +{ + phys_addr_t start, size, align; + char *oldp; + int err; + + if (!p) + return -EINVAL; + + oldp = p; + size = memparse(p, &p); + if (p == oldp) + return -EINVAL; + + if (*p != ':') + return -EINVAL; + + align = memparse(p+1, &p); + if (*p != ':') + return -EINVAL; + + start = memblock_phys_alloc(size, align); + if (!start) + return -ENOMEM; + + p++; + err = reserved_mem_add(start, size, p); + if (err) { + memblock_phys_free(start, size); + return err; + } + + p += strlen(p); + + return *p == '\0' ? 0: -EINVAL; +} +__setup("reserve_mem=", reserve_mem); + #if defined(CONFIG_DEBUG_FS) && defined(CONFIG_ARCH_KEEP_MEMBLOCK) static const char * const flagname[] = { [ilog2(MEMBLOCK_HOTPLUG)] = "HOTPLUG",