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 43232CF11FE for ; Thu, 10 Oct 2024 15:24:47 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id BB5C26B0082; Thu, 10 Oct 2024 11:24:46 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id B65D96B0083; Thu, 10 Oct 2024 11:24:46 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id A2D8E6B0089; Thu, 10 Oct 2024 11:24:46 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0010.hostedemail.com [216.40.44.10]) by kanga.kvack.org (Postfix) with ESMTP id 86C116B0082 for ; Thu, 10 Oct 2024 11:24:46 -0400 (EDT) Received: from smtpin13.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay04.hostedemail.com (Postfix) with ESMTP id 25D401A0767 for ; Thu, 10 Oct 2024 15:24:40 +0000 (UTC) X-FDA: 82658064972.13.BE1F99E Received: from nyc.source.kernel.org (nyc.source.kernel.org [147.75.193.91]) by imf16.hostedemail.com (Postfix) with ESMTP id 5135518000A for ; Thu, 10 Oct 2024 15:24:43 +0000 (UTC) Authentication-Results: imf16.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=ekagPdIZ; spf=pass (imf16.hostedemail.com: domain of rppt@kernel.org designates 147.75.193.91 as permitted sender) smtp.mailfrom=rppt@kernel.org; dmarc=pass (policy=quarantine) header.from=kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1728573748; 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=c62i3F029wIUU6xtQe3DQyljasrsetXeDj04xF8k0V4=; b=Fa7UzLfI8S/BKqnNz2FunYbC05BhcfvCtuqXC0pThVHdIhJQ9q+Wy5FSHthOXXYHLNTz4w YaMODcVJ22glTr3sKKxRkqgVeH4TBiS3WapQpI8zbcBJvXsRNwMcFLopl1zwf/176zmBeF rlDWHTwQHYB5RO/9rUoof8HeLEiSD7I= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1728573748; a=rsa-sha256; cv=none; b=5FtyLqpbNFGaH7VLC0qe4mvSfFiNwNup3MejuSzFXmOSVQGt62ip0J9eH/vNL4iW+A4ro+ E70oMRiy/u0MWAfxSVjb/yBNSKGVNn7/6EH48tjVNM1Ab4kuUN0brORpx6tdSP/3r+aqa0 6uGWO8oJIf1KsrhjpwGDiQf5x/tzTws= ARC-Authentication-Results: i=1; imf16.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=ekagPdIZ; spf=pass (imf16.hostedemail.com: domain of rppt@kernel.org designates 147.75.193.91 as permitted sender) smtp.mailfrom=rppt@kernel.org; dmarc=pass (policy=quarantine) header.from=kernel.org Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by nyc.source.kernel.org (Postfix) with ESMTP id CE93BA4347C; Thu, 10 Oct 2024 15:24:34 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 94D86C4CEC5; Thu, 10 Oct 2024 15:24:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1728573883; bh=gnTehr7qrLMKw/pUpKhTMaCY3tpDIt6F6aFobT96HoY=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=ekagPdIZxqT4SWWRuS6VjAHAzc6oWKEW/4hsSTRm4pLygcR/K/5nYo1MRhIB1W8Bi WhUInbNKfRaSZHzNuMbNo/gh/Dm11IzVkhHlBcuZvhkOJMQBELbJP+QR/GYDeVHSby +1ODj4I12HqiH5y8g34fzvrSEzf2u/HDRwYAX/nNnTn9iTEhJxcxooLVqAwIKD6FZv b3fbK7YHQj8tX05PEVkLu2GJn3pxXz8GTYW1p2sF1xOis1mXNPjG1oTG9dYByuOZVe q0nYNSfpcIxt1MrmJaX80Vmf5bsmJaexf7/nT75nIm1VGT6H7a1RxSwQLd/ELhYVan 63H6TXss7XvhQ== Date: Thu, 10 Oct 2024 18:20:53 +0300 From: Mike Rapoport To: Sergey Senozhatsky Cc: Andrew Morton , Andreas Larsson , Andy Lutomirski , Ard Biesheuvel , Arnd Bergmann , Borislav Petkov , Brian Cain , Catalin Marinas , Christoph Hellwig , Christophe Leroy , Dave Hansen , Dinh Nguyen , Geert Uytterhoeven , Guo Ren , Helge Deller , Huacai Chen , Ingo Molnar , Johannes Berg , John Paul Adrian Glaubitz , Kent Overstreet , "Liam R. Howlett" , Luis Chamberlain , Mark Rutland , Masami Hiramatsu , Matt Turner , Max Filippov , Michael Ellerman , Michal Simek , Oleg Nesterov , Palmer Dabbelt , Peter Zijlstra , Richard Weinberger , Russell King , Song Liu , Stafford Horne , Steven Rostedt , Thomas Bogendoerfer , Thomas Gleixner , Uladzislau Rezki , Vineet Gupta , Will Deacon , bpf@vger.kernel.org, linux-alpha@vger.kernel.org, linux-arch@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-csky@vger.kernel.org, linux-hexagon@vger.kernel.org, linux-kernel@vger.kernel.org, linux-m68k@lists.linux-m68k.org, linux-mips@vger.kernel.org, linux-mm@kvack.org, linux-modules@vger.kernel.org, linux-openrisc@vger.kernel.org, linux-parisc@vger.kernel.org, linux-riscv@lists.infradead.org, linux-sh@vger.kernel.org, linux-snps-arc@lists.infradead.org, linux-trace-kernel@vger.kernel.org, linux-um@lists.infradead.org, linuxppc-dev@lists.ozlabs.org, loongarch@lists.linux.dev, sparclinux@vger.kernel.org, x86@kernel.org Subject: Re: Bisected: [PATCH v5 8/8] x86/module: enable ROX caches for module text Message-ID: References: <20241009180816.83591-1-rppt@kernel.org> <20241009180816.83591-9-rppt@kernel.org> <20241010083033.GA1279924@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20241010083033.GA1279924@google.com> X-Rspamd-Queue-Id: 5135518000A X-Stat-Signature: awg67a73yc6guzgb9hjoyw96s31m43zt X-Rspamd-Server: rspam09 X-Rspam-User: X-HE-Tag: 1728573883-851242 X-HE-Meta: U2FsdGVkX1+JbvJg3Yeo/8ApwGAdBD6C066+/CNPYlCBnOTqAMQrcYkQbb7T5IdLvb9gAIBkdW11Ejyj4SvwGytZA2wrzjUUBk8cAwsNNufwABtq2EIbogYp2mgEO1VaC5Xe9jFBKoJEYQZP/nFD0hl7vex494HY/Q/We7MT3exmILYMlYrD+4J9CAWxHyZ5g0F0Svj6QmtibHQT9ReXWSd2UmTOwe0c3lvLN0zmHiEZBwazS6Kb/MOgEfuP8jlp7sEfo+gj8ExapXgo3vHxgDravTn/o6Qm16zHJNMNWaLynWHbGxvfllOtRQ774Fd8kA9HSXFfAGtFCSggSKvhentrM0vHPCF7eDI7/dm0Y5mawdJnvWRqmXYiuoLIr7Zh54YuxPkc9RF0rq4IO61RL28L8bn6ood0Wlr+I6T/dn90daLzM4NjFkHHSwSu78RwzwmGu0yLvh3TdmEgpIrU4wPNufhCM8013y+8x3/2+ORc2TdreW/Aa/3UVAk9iXKrVWUr24B3+waBynyt3q543L5Eqei/frcGaGBZCXUJe1dTKvg7LK5cza9U7bHPm4sABIjlz3XQWxzcnboTr8JPl3/8xCjzsC7dzpwBs0zobVtsbdPXHnG45RbJNgjK5BAeXY+Rk7CfqaRjtquxI1Jbq7Tfvqe1pEslq3cWFoc0urox6WwSYGqc0T+o8wc8s+BUp5b8laNcQAfqzpbOkRl9J/h4buIl/69EtpHO5eBzOmimBDcqz2MDNnqOH4Razk5JaOtAt7Fxz4xWmv9eInwbx+P/50FcEqAnK3w/Gpug646EoPF9LIRM4Tqu7y7neMeg5L1Xg7xOLg9vZ9cBPOVGyG4utS2hJzvuhalHUADC2L8Bbg4KxGzN/8XFXhHt+mURlB60P3dDS7+j7RZOtqykEUkA6l1FlBqA+8pHg3MkASMbtnpVXfei1GVkachG1xOeey8m/dn5lGpM3XJEspZ a9EmrAUS U7IhitvrKoM/DRZ4BKZIn7hRTasDL/+J1gVT2gsHsr4Gr5qPwKZEASd0muHfXfg7/Tb5oAb+HHtH5Tp6ldv2gBIAKv+ES3b0HF/J9JO0mBuGgE0cgeXsfjyTiI+KyKfHWd0XoCBP6bg2uGBz5fsw4zqLDrgLCU3/LiCd/+/iXVDaPau4vTCLbQsRpqkQs6ASVWd1agLfaBwDWM7sxfF4/XA4MMA0CF73oCuwDk/MBrxaam60pk1QvBBA5LK3LcRhY6/dSR4GDKFDjB8/YcaQAcpROkA== 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: On Thu, Oct 10, 2024 at 05:30:33PM +0900, Sergey Senozhatsky wrote: > On (24/10/09 21:08), Mike Rapoport wrote: > > From: "Mike Rapoport (Microsoft)" > > > > Enable execmem's cache of PMD_SIZE'ed pages mapped as ROX for module > > text allocations. > > > > With this modprobe disappoints kmemleak > > [ 12.700128] kmemleak: Found object by alias at 0xffffffffa000a000 > [ 12.702179] CPU: 5 UID: 0 PID: 410 Comm: modprobe Tainted: G N 6.12.0-rc2+ #760 > [ 12.704656] Tainted: [N]=TEST > [ 12.705526] Call Trace: > [ 12.706250] > [ 12.706888] dump_stack_lvl+0x3e/0xdb > [ 12.707961] __find_and_get_object+0x100/0x110 > [ 12.709256] kmemleak_no_scan+0x2e/0xb0 > [ 12.710354] kmemleak_load_module+0xad/0xe0 > [ 12.711557] load_module+0x2391/0x45a0 > [ 12.712507] __se_sys_finit_module+0x4e0/0x7a0 > [ 12.713599] do_syscall_64+0x54/0xf0 > [ 12.714477] ? irqentry_exit_to_user_mode+0x33/0x100 > [ 12.715696] entry_SYSCALL_64_after_hwframe+0x4b/0x53 > [ 12.716931] RIP: 0033:0x7fc7af51f059 > [ 12.717816] Code: 08 89 e8 5b 5d c3 66 2e 0f 1f 84 00 00 00 00 00 90 48 89 f8 48 89 f7 48 89 d6 48 89 ca 4d 89 c2 4d 89 c8 4c 8b 4c 24 08 0f 05 <48> 3d 01 f0 ff ff 73 01 c3 48 8b 0d 8f 1d 0d 00 f7 d8 64 89 01 48 > [ 12.722324] RSP: 002b:00007ffc1d0b0c18 EFLAGS: 00000246 ORIG_RAX: 0000000000000139 > [ 12.724173] RAX: ffffffffffffffda RBX: 00005618a9439b20 RCX: 00007fc7af51f059 > [ 12.725884] RDX: 0000000000000000 RSI: 000056187aea098b RDI: 0000000000000003 > [ 12.727617] RBP: 0000000000000000 R08: 0000000000000060 R09: 00005618a943af60 > [ 12.729361] R10: 0000000000000038 R11: 0000000000000246 R12: 000056187aea098b > [ 12.731101] R13: 0000000000040000 R14: 00005618a9439ac0 R15: 0000000000000000 > [ 12.732814] Below is a quick fix, I'll revisit module - kmemleak interaction in v6 diff --git a/kernel/module/debug_kmemleak.c b/kernel/module/debug_kmemleak.c index b4cc03842d70..df873dad049d 100644 --- a/kernel/module/debug_kmemleak.c +++ b/kernel/module/debug_kmemleak.c @@ -14,7 +14,8 @@ void kmemleak_load_module(const struct module *mod, { /* only scan writable, non-executable sections */ for_each_mod_mem_type(type) { - if (type != MOD_DATA && type != MOD_INIT_DATA) + if (type != MOD_DATA && type != MOD_INIT_DATA && + !mod->mem[type].is_rox) kmemleak_no_scan(mod->mem[type].base); } } -- Sincerely yours, Mike.