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 9F44AEB64D9 for ; Sat, 17 Jun 2023 06:15:35 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 0FD3E6B0072; Sat, 17 Jun 2023 02:15:35 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 0AC6C6B0075; Sat, 17 Jun 2023 02:15:35 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id EB6226B0078; Sat, 17 Jun 2023 02:15:34 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id DBEDB6B0072 for ; Sat, 17 Jun 2023 02:15:34 -0400 (EDT) Received: from smtpin04.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id A414A80BC7 for ; Sat, 17 Jun 2023 06:15:34 +0000 (UTC) X-FDA: 80911228188.04.6B21F90 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf16.hostedemail.com (Postfix) with ESMTP id D52CD180009 for ; Sat, 17 Jun 2023 06:15:32 +0000 (UTC) Authentication-Results: imf16.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=dZPxNNOZ; dmarc=pass (policy=none) header.from=kernel.org; spf=pass (imf16.hostedemail.com: domain of rppt@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=rppt@kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1686982533; 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:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=26BtaAc63CHnWpK0yZHDvnGi2yFfIbXKK8brxlskGug=; b=gFL6QRWON1YNPAgtqWJTnFI+2drl7p4tBBZCBa5RMJmOuNHhlz9pDrezMIotNuDSTNiBPT 3GTWgOwsItMiZyNk3cPO11sBsRIriq3d36SNictqWRPZFgB1AID5Z8ski4Mh1C4Vj2kHnQ 6Xnj6ojvl+F5pnHpO5q0umoixSHGIiI= ARC-Authentication-Results: i=1; imf16.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=dZPxNNOZ; dmarc=pass (policy=none) header.from=kernel.org; spf=pass (imf16.hostedemail.com: domain of rppt@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=rppt@kernel.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1686982533; a=rsa-sha256; cv=none; b=dhopJb8TiGan581SOhfUlQ/kXZhsYQfbyvnrRXGDiSujHwWor/4tNCMEze2mXOYhSVQmfg RxZTz9hnR83HiGgUpfI8YzuMQcmu293yjf3Me22voMciYKpIjRuR6EIuOC0Xp8v82v0yNa y3SIPEyjnPeqD69Tq137M6yBM7b0IZM= Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id F39F460A54; Sat, 17 Jun 2023 06:15:31 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 87CEDC433C8; Sat, 17 Jun 2023 06:15:19 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1686982531; bh=FmBCPGQr9kc4FZ3SJahzQDpcowAg0zz2/CXRHAPmuKU=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=dZPxNNOZ7Pj7Xyf/2II866o9WGyGm1IrJ+s0YaBcPXU6Zpw9n8L8eiTjVcgJY+2Vs kld2AEWURToZP5cMWq1Ovy1Zcdh+nXcQ07YLmVsYdmCnhWaIZH4rzkh5ACAnqBAq9Q wzQ0gANPefS1BRw6XI2qxQxNWzMTUvfRDmLMmiCAO67igHmVxwDCWFEarRwd0xHul6 0oDC+Um1uswFwi7/PIIIM9hS3jTKdm7q0/KfQYWawgyRH6/0DjKNXWaG0NeXVYm4Y8 x6nV8BirqJ1tjKuPw4r0wnIidEjuy3LK+walcl5GcB9OnmJlHxiCHLuLAiHeiVPkg6 78arIfpTAXAlw== Date: Sat, 17 Jun 2023 09:14:44 +0300 From: Mike Rapoport To: Song Liu Cc: linux-kernel@vger.kernel.org, Andrew Morton , Catalin Marinas , Christophe Leroy , "David S. Miller" , Dinh Nguyen , Heiko Carstens , Helge Deller , Huacai Chen , Kent Overstreet , Luis Chamberlain , Mark Rutland , Michael Ellerman , Nadav Amit , "Naveen N. Rao" , Palmer Dabbelt , Puranjay Mohan , Rick Edgecombe , Russell King , Steven Rostedt , Thomas Bogendoerfer , Thomas Gleixner , Will Deacon , bpf@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mips@vger.kernel.org, linux-mm@kvack.org, linux-modules@vger.kernel.org, linux-parisc@vger.kernel.org, linux-riscv@lists.infradead.org, linux-s390@vger.kernel.org, linux-trace-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, loongarch@lists.linux.dev, netdev@vger.kernel.org, sparclinux@vger.kernel.org, x86@kernel.org Subject: Re: [PATCH v2 04/12] mm/execmem, arch: convert remaining overrides of module_alloc to execmem Message-ID: <20230617061444.GP52412@kernel.org> References: <20230616085038.4121892-1-rppt@kernel.org> <20230616085038.4121892-5-rppt@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-Rspamd-Queue-Id: D52CD180009 X-Rspam-User: X-Rspamd-Server: rspam05 X-Stat-Signature: uw8oajfj6dj54fuckmjho5ek14zim3uw X-HE-Tag: 1686982532-718873 X-HE-Meta: U2FsdGVkX19wInRdPQP8hREWvsVS8m0Eicz3z6CiyTsJmA7ed4yZsabPCA57zmTcIgLET4a97cgj+DoRdmBRZToEEvzSJDIkFimA0GMFEchNgSftUARKmhRswMFivduQB1tCRaQRupXCUk7nDy1P63xabjNUqCGMcOScMEKwwBD2wWT1jR3pezlWlGerg6fexs7hwxQhwHtTOXVjHPF38PwszZ4yCyd6hsOtPzFUY0SWoN/L5Dt924amALU9mQR5NrtFGjJVis7risg2pzejSG1ttE60mwFnmrTLi4TiuvrXrKXBT0ivBymf8tgh13sqT1aVNc3nCYxdNbbHr/HrphxxIGIPoxGqCsXWuOShTnu7qHfsRL7/vAgqNUmXI6v1mXyw4AN1r3/4RzzBlwSpe73JaeAIFZxtscdAyU3VHDSyt9jgOfi0BN5GhnQUENfzln2HN1kMuOt+zhevGCvVzGj0dJLBB/A7rYtSeDKuWgSMeH0r0ZnPgKaxXyiKz6Ti+LmbLJ/5A96tPAVntQIj47fzHekJ0vPynUL1M5L8YCKnIMqtxGOOMzgZLvhcrJoZdXbL6JhAlNKmFdNhb+wPzKMQV6qRJcLQ8uSespnJ+1X4BaWJqpdDzm9wZM36wgsZPN05p3NgEF8ZEGjlXRfWaSlExSXXGUlduAanqCH2kxfkN/hcmDr7/Qbgiknd0I5cd64xsuwL8p4mAT3w0Rr7xzT+/SZTTVIgkuvr2k9qz0uiFOQe3nO2/1srItjEAW0QZYpKoH+9N5VX/R2T3+Qr5DsdLa8X4IUSb0E7u4ML+Fo1dg13DA6QLDIFsA8AsQsEyzTpHPv1Dp1n0OIImAUbUz4sxFzaSEFrm8BWZ71S92QiVYjhRMqS3CXvpn35Mb2RVcX+QEl4KjbxaWGKhVGzte7Z9Tq9N7w2+pYnUrwjFoe+J0tdGyhOibH7zMNWZUcpyxrbTQSphLWOEhs4axT +Ca3EjZl HY1iMXhD38J6n82U75ckibTdLt8nvf0dJtM4PIkxh2YNKf8etG1CI/urenC5Aii42rCpjDELJOZc+RH1dssf/CBHydkRXWFB6CQ9XRi5knOlbuSXKJ8luiMqZyRjR4drcAo67jRo7slNP09ABlKY8AQ9veGegxDNahbPgKRfDPdUC+8dL2JZ4JkvzcN7I/7Pn0AU3hXq1vkEj24TzfUHETO00igshWHjLuU1GV3nId3DekDvp8o2uwMezz3jrFmFKT7iZ8pmrq66P9aG/R0LELzLW305YPoaQdHjVXjj3Ulj2OqOzEnIwDEi9NhNJO1I3RHGN99ilpfpzgOY/Sxc0vcX+ltSVH7Fm3btQ+DGnhJl1jfJo+WWkSqXlP3p0RpX3t8GDwcXdDgu7HCj88g1p8tyiSt4xHJLAmZPC2q7Rha/udMxdPqn9CSwpHfHTmlwdgX4+xP4/zA/5myeyvRBQOyHH/PqFIx4TFAdNzN1TOfIM5bD7pVzFXBGggQGfrzELcEvz 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 Fri, Jun 16, 2023 at 11:53:54AM -0700, Song Liu wrote: > On Fri, Jun 16, 2023 at 1:51 AM Mike Rapoport wrote: > [...] > > diff --git a/arch/arm64/kernel/module.c b/arch/arm64/kernel/module.c > > index 5af4975caeb5..c3d999f3a3dd 100644 > > --- a/arch/arm64/kernel/module.c > > +++ b/arch/arm64/kernel/module.c > > @@ -17,56 +17,50 @@ > > #include > > #include > > #include > > +#include > > #include > > #include > > #include > > #include > > > > -void *module_alloc(unsigned long size) > > +static struct execmem_params execmem_params = { > > + .modules = { > > + .flags = EXECMEM_KASAN_SHADOW, > > + .text = { > > + .alignment = MODULE_ALIGN, > > + }, > > + }, > > +}; > > + > > +struct execmem_params __init *execmem_arch_params(void) > > { > > u64 module_alloc_end = module_alloc_base + MODULES_VSIZE; > > - gfp_t gfp_mask = GFP_KERNEL; > > - void *p; > > - > > - /* Silence the initial allocation */ > > - if (IS_ENABLED(CONFIG_ARM64_MODULE_PLTS)) > > - gfp_mask |= __GFP_NOWARN; > > > > - if (IS_ENABLED(CONFIG_KASAN_GENERIC) || > > - IS_ENABLED(CONFIG_KASAN_SW_TAGS)) > > - /* don't exceed the static module region - see below */ > > - module_alloc_end = MODULES_END; > > + execmem_params.modules.text.pgprot = PAGE_KERNEL; > > + execmem_params.modules.text.start = module_alloc_base; > > I think I mentioned this earlier. For arm64 with CONFIG_RANDOMIZE_BASE, > module_alloc_base is not yet set when execmem_arch_params() is > called. So we will need some extra logic for this. Right, this is taken care of in a later patch, but it actually belongs here. Good catch! > Thanks, > Song > > > > + execmem_params.modules.text.end = module_alloc_end; > > > > - p = __vmalloc_node_range(size, MODULE_ALIGN, module_alloc_base, > > - module_alloc_end, gfp_mask, PAGE_KERNEL, VM_DEFER_KMEMLEAK, > > - NUMA_NO_NODE, __builtin_return_address(0)); > > - > > - if (!p && IS_ENABLED(CONFIG_ARM64_MODULE_PLTS) && > > + /* > > + * KASAN without KASAN_VMALLOC can only deal with module > > + * allocations being served from the reserved module region, > > + * since the remainder of the vmalloc region is already > > + * backed by zero shadow pages, and punching holes into it > > + * is non-trivial. Since the module region is not randomized > > + * when KASAN is enabled without KASAN_VMALLOC, it is even > > + * less likely that the module region gets exhausted, so we > > + * can simply omit this fallback in that case. > > + */ > > + if (IS_ENABLED(CONFIG_ARM64_MODULE_PLTS) && > > (IS_ENABLED(CONFIG_KASAN_VMALLOC) || > > (!IS_ENABLED(CONFIG_KASAN_GENERIC) && > > - !IS_ENABLED(CONFIG_KASAN_SW_TAGS)))) > > - /* > > - * KASAN without KASAN_VMALLOC can only deal with module > > - * allocations being served from the reserved module region, > > - * since the remainder of the vmalloc region is already > > - * backed by zero shadow pages, and punching holes into it > > - * is non-trivial. Since the module region is not randomized > > - * when KASAN is enabled without KASAN_VMALLOC, it is even > > - * less likely that the module region gets exhausted, so we > > - * can simply omit this fallback in that case. > > - */ > > - p = __vmalloc_node_range(size, MODULE_ALIGN, module_alloc_base, > > - module_alloc_base + SZ_2G, GFP_KERNEL, > > - PAGE_KERNEL, 0, NUMA_NO_NODE, > > - __builtin_return_address(0)); > > - > > - if (p && (kasan_alloc_module_shadow(p, size, gfp_mask) < 0)) { > > - vfree(p); > > - return NULL; > > + !IS_ENABLED(CONFIG_KASAN_SW_TAGS)))) { > > + unsigned long end = module_alloc_base + SZ_2G; > > + > > + execmem_params.modules.text.fallback_start = module_alloc_base; > > + execmem_params.modules.text.fallback_end = end; > > } > > > > - /* Memory is intended to be executable, reset the pointer tag. */ > > - return kasan_reset_tag(p); > > + return &execmem_params; > > } > > > > enum aarch64_reloc_op { -- Sincerely yours, Mike.