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 Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id DF75BEB64DD for ; Wed, 12 Jul 2023 15:13:20 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 40DCD6B0072; Wed, 12 Jul 2023 11:13:20 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 3BDFA6B0074; Wed, 12 Jul 2023 11:13:20 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 25ECF6B0075; Wed, 12 Jul 2023 11:13:20 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0013.hostedemail.com [216.40.44.13]) by kanga.kvack.org (Postfix) with ESMTP id 171476B0072 for ; Wed, 12 Jul 2023 11:13:20 -0400 (EDT) Received: from smtpin17.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id DBE83801DD for ; Wed, 12 Jul 2023 15:13:19 +0000 (UTC) X-FDA: 81003303318.17.FD457F8 Received: from out5-smtp.messagingengine.com (out5-smtp.messagingengine.com [66.111.4.29]) by imf21.hostedemail.com (Postfix) with ESMTP id 996951C0029 for ; Wed, 12 Jul 2023 15:13:16 +0000 (UTC) Authentication-Results: imf21.hostedemail.com; dkim=pass header.d=fastmail.com header.s=fm2 header.b=VRcANVnE; dkim=pass header.d=messagingengine.com header.s=fm2 header.b=bVjH8KDB; spf=pass (imf21.hostedemail.com: domain of sorear@fastmail.com designates 66.111.4.29 as permitted sender) smtp.mailfrom=sorear@fastmail.com; dmarc=pass (policy=none) header.from=fastmail.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1689174796; 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:in-reply-to:references:references:dkim-signature; bh=dSgpbN2qqN6BLnaVZrXrgZ301dLlfYvJq2QGVf6hj70=; b=VM9RZMOYtBqdwgpKxXeyQrA1O1h+cpkYX0F+fDmICcueLqPS9yLSTh9ENyg5VMAoMYrApe xo5A33JlD63b3lpnn9PBMhLv9zZ1JISogkBOuVE4fm0PKyzOWp5Md13FGaDvUX1E8mTaTg a4Boy5FleTWwoV5//PXH9xzRnbcyscg= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1689174796; a=rsa-sha256; cv=none; b=pXzL+op/M2jcwNuEDPGpmcpKAcvXuvFieJvPP7jLOVk86OnIfVqRI2ZTQ+xbpACrECc4lX 0chssQyYvTL/c5HGhoBK5uU9tGJy4Vs9LzH04kfjdWvAAWESKTbu6AbNanzYHYIR2gkC15 +EPAsTAVIS3qjbmBGHX+h0x15JmHwsE= ARC-Authentication-Results: i=1; imf21.hostedemail.com; dkim=pass header.d=fastmail.com header.s=fm2 header.b=VRcANVnE; dkim=pass header.d=messagingengine.com header.s=fm2 header.b=bVjH8KDB; spf=pass (imf21.hostedemail.com: domain of sorear@fastmail.com designates 66.111.4.29 as permitted sender) smtp.mailfrom=sorear@fastmail.com; dmarc=pass (policy=none) header.from=fastmail.com Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailout.nyi.internal (Postfix) with ESMTP id 0147A5C00E8; Wed, 12 Jul 2023 11:13:14 -0400 (EDT) Received: from imap50 ([10.202.2.100]) by compute1.internal (MEProxy); Wed, 12 Jul 2023 11:13:14 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fastmail.com; h= cc:cc:content-type:content-type:date:date:from:from:in-reply-to :in-reply-to:message-id:mime-version:references:reply-to:sender :subject:subject:to:to; s=fm2; t=1689174793; x=1689261193; bh=dS gpbN2qqN6BLnaVZrXrgZ301dLlfYvJq2QGVf6hj70=; b=VRcANVnEB2elnO8va5 7qDzlklh+Nx+prfh8PRfHEKkTXshU67ofceeigTZ9rT5UDSk8eEOJEa5v5vGynWY SKxEIc2xWdNcG5Fp7nxQiBenVluT7+pbKVS/PtmVaCYe2THBu+Y3i5LsoC/SsQus 5HDA5bbAM9575RbxSE8TMt7kkjvMXkacW4pPbkC7anwh/1oYeHvbqJYPYHQcD5ZX YfPHfhBsFeOLQO453EUuy/sX6DDHEuYHgE0mMc5qEUHEY4HopZjtdriX+JpjH32P AqQ0wzchbFzP7XzVi9Cjr8qs90QOfpWciNwobxhXsCPlzDOz7WY+a2K/NMerUaAG ygrw== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-type:content-type:date:date :feedback-id:feedback-id:from:from:in-reply-to:in-reply-to :message-id:mime-version:references:reply-to:sender:subject :subject:to:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm2; t=1689174793; x=1689261193; bh=dSgpbN2qqN6BL naVZrXrgZ301dLlfYvJq2QGVf6hj70=; b=bVjH8KDBs5z5dsZkuFwFImLh8Slui QqmG02Oy49twz7BISKEgnY/ZjFeZ+llsc1+VE2JrZ1L2jhxCL9WrSWeW1xa3/jex VrTNsK8otzYHHsCVMwWsxl0tx8NAiJHOerGZMcXSWS4V/wfvURG18KbYEgLVHKap JQ5NmZbnDRjHyJZ7cIwBK21dC7VzH2JPwMZn+0Scp4C53kYawRAtY7kg9GGUHUET 5PEUOurEHtKWOpZmN2cGfUu2rSnFMOGl2C6DecMTnXyu2Tnkkua2tYem8McGi7zS hwkVyyREt8oWnHwPwJHtQ3ieIOF0X6R77vGVOMB9q5aNaNwhZUZztYimg== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedviedrfedvgdekiecutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenuc fjughrpefofgggkfgjfhffhffvvefutgesthdtredtreertdenucfhrhhomhepfdfuthgv fhgrnhcuqfdktfgvrghrfdcuoehsohhrvggrrhesfhgrshhtmhgrihhlrdgtohhmqeenuc ggtffrrghtthgvrhhnpeejueehgedtueetgefhheejjeeigffhieefjeehuddvueegtdfh heevgfeggfektdenucffohhmrghinhepihhnfhhrrgguvggrugdrohhrghenucevlhhush htvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhmpehsohhrvggrrhesfhgr shhtmhgrihhlrdgtohhm X-ME-Proxy: Feedback-ID: i84414492:Fastmail Received: by mailuser.nyi.internal (Postfix, from userid 501) id 0BDAD1700096; Wed, 12 Jul 2023 11:13:12 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface User-Agent: Cyrus-JMAP/3.9.0-alpha0-531-gfdfa13a06d-fm-20230703.001-gfdfa13a0 Mime-Version: 1.0 Message-Id: <31c4024b-3553-4117-a22a-b39259c8fea4@app.fastmail.com> In-Reply-To: <20230711130754.481209-3-gerg@kernel.org> References: <20230711130754.481209-1-gerg@kernel.org> <20230711130754.481209-3-gerg@kernel.org> Date: Wed, 12 Jul 2023 11:12:45 -0400 From: "Stefan O'Rear" To: "Greg Ungerer" , linux-riscv@lists.infradead.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org Cc: "Palmer Dabbelt" , viro@zeniv.linux.org.uk, ebiederm@xmission.com, keescook@chromium.org, brauner@kernel.org Subject: Re: [PATCH v2 2/2] riscv: support the elf-fdpic binfmt loader Content-Type: text/plain X-Rspamd-Queue-Id: 996951C0029 X-Rspam-User: X-Stat-Signature: dwunsecwksskixpajhoeozn8366u5mca X-Rspamd-Server: rspam03 X-HE-Tag: 1689174796-321886 X-HE-Meta: U2FsdGVkX18tffrBo81neXJqYv5vdCGOMb2+kxUhSWkiSjTE2XWnq2aorAyNgDXOvan3gwS8FNYaxOjzV++9uQCWHJkY8cU62loYbtAYu+qjvC0BnV2sM99oe0YU6j7JjG6Pa76SlcXeuV8V9GhBYq+JO7L5XGh4xaaeLaoOZgXm4mH1vL/NvnVpMLyiaYTrgFKGTUA5dx4kdG1rgD0YnIL7XmuaCZT1iU4xstc2peK5dbzE+pj5sRInhcVKn5O8GMr1S0xZEiuUr62KbA8QJIRhrqvjDT6B9nxPbi2rR3IdiN2opb/60FdIsY3lQQXGx9rEKgAjK/YcqLQps5CRO/HdBWN1tSvKoXZDkLXWK50RVI5nk0754F2/YoaPysFMN8wclyk9ZdP4OVxTiczpfw+jddY6fXav7blA7pBtovZHbDLUOX0BTR/DdxRwWNPE2WXTYOhQseB2YJK/TM1PXbVDr0Om3rEyIA8pkw+boeXBxubkt1rzC2IaBK07jgWBx1hvrljQqRDGUAA3CwkZxSFZHVhAm91R5qmPAWAVbUm/WaLCsIOxzlu/kLQYmg3RyNbxL87kQs3HGLa8VGU8ZbZbZUrfKcBMhg4AeG2kYX90t0QJ3hzctegTe9cNc4cb2DCUynCTcjNn0EmVi14qkgJBwhiMO6/sm7LGXQqvzv1/pr6rnMhro77m4mbb7NJ6PKuDP9x38N2gxxmhULP4yR4jRHlo/JcHmivIO3Z3RBPOemF8vC2dr4jQooOr4wG1YOQgE/miz34gVV4VWjRPWrBGGqhM+lKqiAb4o2CQuh9Pd4VRP3zwH+A+F9U6y5/9Fm0yiHPxKkrS0zchcMVsGt9zb892HnkZkFTVkSoeDRoM6PG1T6sRt9aLHePhDYNQnN/9TSMChTfVVINq/h5ony//Ujy5JwkeGf//5BCBtL7xXotpvcXGUv/gMY5x9ukRMqoKPtanzg9Nl2NkH3P +mw1+jJz W63217rEEetfw3DORCv0snGRsDJoyOKlmk0djxwM6wakZ91d57yIIuOGlWok737Db2u4xN1l8qzlklYH3OQH+VqHE6uiO9Gub6fBLHXhTmsCvCFwHIUywLVV8A7HIDtzjmtmvGS//sQpnWbZ6Zu/dsQD9vTzMvNGjhkXDQ88nylfb0IHscmceSIxCUzB4CSjwPGmdBL0FRi+QhtOg14d94y9I3xj4pfY4fNRhulqJuN9XPK9Qs6qyNoObmOnmi13tu9nn12H58OrtZdN8EzQCxOjSEM979dSVyJjNsYCP+1EUNePRTY7gt9aBX1aKikGMh1TuNuLsZhPox56sd2yn8mwNQ6/HfTtJ95Cj1wGOBNveBR0DcfvIpBFvZTyxdkZcKcIIXaEoq044hUE= 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: On Tue, Jul 11, 2023, at 9:07 AM, Greg Ungerer wrote: > Add support for enabling and using the binfmt_elf_fdpic program loader > on RISC-V platforms. The most important change is to setup registers > during program load to pass the mapping addresses to the new process. > > One of the interesting features of the elf-fdpic loader is that it > also allows appropriately compiled ELF format binaries to be loaded on > nommu systems. Appropriate being those compiled with -pie. > > Signed-off-by: Greg Ungerer > --- > v1->v2: rebase onto linux-6.5-rc1 > increment PTRACE_GETFDPIC value to keep it unique > > arch/riscv/include/asm/elf.h | 11 ++++++++++- > arch/riscv/include/asm/mmu.h | 4 ++++ > arch/riscv/include/uapi/asm/ptrace.h | 5 +++++ > fs/Kconfig.binfmt | 2 +- > 4 files changed, 20 insertions(+), 2 deletions(-) > > diff --git a/arch/riscv/include/asm/elf.h b/arch/riscv/include/asm/elf.h > index c24280774caf..c33fe923ef6d 100644 > --- a/arch/riscv/include/asm/elf.h > +++ b/arch/riscv/include/asm/elf.h > @@ -41,6 +41,7 @@ extern bool compat_elf_check_arch(Elf32_Ehdr *hdr); > #define compat_elf_check_arch compat_elf_check_arch > > #define CORE_DUMP_USE_REGSET > +#define ELF_FDPIC_CORE_EFLAGS 0 > #define ELF_EXEC_PAGESIZE (PAGE_SIZE) > > /* > @@ -69,6 +70,13 @@ extern bool compat_elf_check_arch(Elf32_Ehdr *hdr); > #define ELF_HWCAP riscv_get_elf_hwcap() > extern unsigned long elf_hwcap; > > +#define ELF_FDPIC_PLAT_INIT(_r, _exec_map_addr, _interp_map_addr, > dynamic_addr) \ > + do { \ > + (_r)->a1 = _exec_map_addr; \ > + (_r)->a2 = _interp_map_addr; \ > + (_r)->a3 = dynamic_addr; \ > + } while (0) > + This should probably be left empty for now; it will be defined by the ELF FDPIC ABI when that is done, and shouldn't be used by normal ELF binaries. I'd ask if there's a reason it starts at a1 instead of a0, but it seems idiosyncratic on all arches that have full FDPIC support. -s > /* > * This yields a string that ld.so will use to load implementation > * specific libraries for optimization. This is more specific in > @@ -78,7 +86,6 @@ extern unsigned long elf_hwcap; > > #define COMPAT_ELF_PLATFORM (NULL) > > -#ifdef CONFIG_MMU > #define ARCH_DLINFO \ > do { \ > /* \ > @@ -115,6 +122,8 @@ do { \ > else \ > NEW_AUX_ENT(AT_IGNORE, 0); \ > } while (0) > + > +#ifdef CONFIG_MMU > #define ARCH_HAS_SETUP_ADDITIONAL_PAGES > struct linux_binprm; > extern int arch_setup_additional_pages(struct linux_binprm *bprm, > diff --git a/arch/riscv/include/asm/mmu.h b/arch/riscv/include/asm/mmu.h > index 0099dc116168..355504b37f8e 100644 > --- a/arch/riscv/include/asm/mmu.h > +++ b/arch/riscv/include/asm/mmu.h > @@ -20,6 +20,10 @@ typedef struct { > /* A local icache flush is needed before user execution can resume. */ > cpumask_t icache_stale_mask; > #endif > +#ifdef CONFIG_BINFMT_ELF_FDPIC > + unsigned long exec_fdpic_loadmap; > + unsigned long interp_fdpic_loadmap; > +#endif > } mm_context_t; > > void __init create_pgd_mapping(pgd_t *pgdp, uintptr_t va, phys_addr_t > pa, > diff --git a/arch/riscv/include/uapi/asm/ptrace.h > b/arch/riscv/include/uapi/asm/ptrace.h > index e17c550986a6..30f6d6537adc 100644 > --- a/arch/riscv/include/uapi/asm/ptrace.h > +++ b/arch/riscv/include/uapi/asm/ptrace.h > @@ -10,6 +10,11 @@ > > #include > > +#define PTRACE_GETFDPIC 33 > + > +#define PTRACE_GETFDPIC_EXEC 0 > +#define PTRACE_GETFDPIC_INTERP 1 > + > /* > * User-mode register state for core dumps, ptrace, sigcontext > * > diff --git a/fs/Kconfig.binfmt b/fs/Kconfig.binfmt > index 93539aac0e5b..f5693164ca9a 100644 > --- a/fs/Kconfig.binfmt > +++ b/fs/Kconfig.binfmt > @@ -58,7 +58,7 @@ config ARCH_USE_GNU_PROPERTY > config BINFMT_ELF_FDPIC > bool "Kernel support for FDPIC ELF binaries" > default y if !BINFMT_ELF > - depends on ARM || ((M68K || SUPERH || XTENSA) && !MMU) > + depends on ARM || ((M68K || RISCV || SUPERH || XTENSA) && !MMU) > select ELFCORE > help > ELF FDPIC binaries are based on ELF, but allow the individual load > -- > 2.25.1 > > > _______________________________________________ > linux-riscv mailing list > linux-riscv@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-riscv