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 7829AD1CA09 for ; Mon, 4 Nov 2024 23:27:51 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 868E16B0088; Mon, 4 Nov 2024 18:27:50 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 818FB6B008C; Mon, 4 Nov 2024 18:27:50 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 6B9576B0092; Mon, 4 Nov 2024 18:27:50 -0500 (EST) 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 4B5806B0088 for ; Mon, 4 Nov 2024 18:27:50 -0500 (EST) Received: from smtpin11.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id A2E07120CA6 for ; Mon, 4 Nov 2024 23:27:49 +0000 (UTC) X-FDA: 82750001670.11.EF63532 Received: from nyc.source.kernel.org (nyc.source.kernel.org [147.75.193.91]) by imf28.hostedemail.com (Postfix) with ESMTP id 15C25C0017 for ; Mon, 4 Nov 2024 23:27:14 +0000 (UTC) Authentication-Results: imf28.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=uD6aQVKZ; spf=pass (imf28.hostedemail.com: domain of nathan@kernel.org designates 147.75.193.91 as permitted sender) smtp.mailfrom=nathan@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=1730762703; 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=Q2NDL0EIuphmCCwbl0Vy6MrgKt/ZIc9Yf0NK3jfYrbk=; b=0v5naklIFcxhf7ugEcCD4jDWEckmOMvA2gBqcgSNDuoXRCdaooVyf3Ed59MXbj3mgjBHOt I5lRnVZbdk3e3F+/ARdTb0vxkF1yvpODwORdw5WVGgdEJmf/EDcoYb8/3vsYT9mhx58A6V EMTihPGfexH73MM36G0S31JL5mwbcqU= ARC-Authentication-Results: i=1; imf28.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=uD6aQVKZ; spf=pass (imf28.hostedemail.com: domain of nathan@kernel.org designates 147.75.193.91 as permitted sender) smtp.mailfrom=nathan@kernel.org; dmarc=pass (policy=quarantine) header.from=kernel.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1730762703; a=rsa-sha256; cv=none; b=3nMuqfzCDb7ldGPyty3KdR2DTRi6JUJcNb0PSNTkwDrx1mD3SmM11sa8NGjYn8bgKcsqyL gOc7w1PYiseleY6cz80rSgvQXJPdDRoHTbbGq8kuCVXFkNjjFmeXjRHlHIDEBoBALoyqgK hQJCPrQp95B6stFJe2VknNxv9fXhJhI= Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by nyc.source.kernel.org (Postfix) with ESMTP id 8521EA41B3D; Mon, 4 Nov 2024 23:25:51 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id BB3A7C4CECE; Mon, 4 Nov 2024 23:27:42 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1730762866; bh=G7tF/ZaqXBtDJLMTb97Oc+SktZk//Kx3oRVjwm2Gkec=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=uD6aQVKZ8xYi+PTTwYupZ6PGdQoZjGxor3CJ6T/KZlWjFxurKitUGg4OsLqeVl22R oTMAQXmWqXP0EO6JaBIYZ9VEqoL35XGUitRTwgi+bJtDhjSL6UuAIcLW7XMfz2tbvE x3MFVpG1yHfJRXfsS6NaRprs6A1rIqEf/OZ8jX2sy+DwUFdsMetTULRZ3iOQoxlOsn 5JoGBx6K0BrtfF9Z0+hxFGtNMWymLR+Dppngi0NxeJnTpYgOWeDN7FHowhaRXk3khA u6nha7LupF4lzkb6ryYEoHDIxLmpv0DY+XiF9diWn7xWUzJjMMZ/7hbbF3UF0ISQXS B++nhrf0RE3MQ== Date: Mon, 4 Nov 2024 16:27:41 -0700 From: Nathan Chancellor To: Mike Rapoport Cc: Andrew Morton , Luis Chamberlain , 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" , 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 , Suren Baghdasaryan , 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: [PATCH v7 6/8] x86/module: prepare module loading for ROX allocations of text Message-ID: <20241104232741.GA3843610@thelio-3990X> References: <20241023162711.2579610-1-rppt@kernel.org> <20241023162711.2579610-7-rppt@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20241023162711.2579610-7-rppt@kernel.org> X-Rspamd-Server: rspam09 X-Rspamd-Queue-Id: 15C25C0017 X-Stat-Signature: 5w8jjtwat7yhp7z73mpzfoyxiy4umw94 X-Rspam-User: X-HE-Tag: 1730762834-357350 X-HE-Meta: U2FsdGVkX1+GJyCT0M9P7YS8EdYQN5frcpa1Mt/TA7PIy3ZbBqCok6b+wH3EuTiDwZZfaWgwbNjXSo7uLVglGzmyVloYnJTRbZv1A+dVNwvfldIEu0H6prr765waTpWvpbx0F0t+dkshAIQRCdNH4G13jA/50PzIhShH08w6Ib8Q1SQT4bVyeSp3M61Nq6U6UmpODPiYrCyI9YMJlIHDBUvqFw4DGIm/WLD8Qf5YPeNeJ1ossokJeNdZLlSX/cUKjSQgSbTGcp3kId42uw/Gz3P/edAoYwg05DtgEHXNFHDavC1cZvf46UCs0hRDAl+ObPET9RONAYRBpJjlOw1JWOMPVgHUQFa+nX/8Cf/v2bn37nj/hIHMuOEQYG/9F8CXrm8vOKcbX/J7kC2/keaA0zqSW8o+uInMswZTTTlLjshv+SvLdhzObiGSUt+veEE3q4/gAh2eAhVw7eVVAmCPODtnyKuJLOyuKzKb3jaqGUoOZUNbDiXRhK1dymo6xEPKj5eZXsmHTpaAysVquBXtEHlSJCmiidfydV+hIAEaemSaBLhwQZ/8O2G/7cWO5pa3fMRawOkaSCUKMIcVkCstGga6LSPsGQ5n2geAWWnVE106jR9KJcbPPZDP+uS66lMkV2mu2AyWF0fMxcPkIkG8iBMashBUeLFDTe6B/WjOJoffIe9fPDq2CVWQ4+g0UV9a+kY8/oTxBHYFI1HETrGxlXq6mpzC2O/edK7Oyv44DdyNAGCq3kovY78BmzHhdy5wsTKHwonskf0PuvTIalkEuNv373PUUy6wPUrGrn0Tr0n3YOikz4BsbaRzT43NIp+tr9jXM8yYoo7SZgkGQcjxtlBHNsD0te/3qMo52lBvQHfY9m4laSdqrjaeO7zCfmUl13madEdfWtTdTNPUHwPjq6wK39Lh/xkZg8kDX+gcbGcZquFD04W1YBgxFKP3kGHMjx/OvIdzFWAHL/W6bqW hi/xHagP giZyZA9fqDmJvqIdg+SJOc+33xHMrlOtZ3Jd4QW3T2C4/KglXsh3qf2kHb/m94vOxKlo3sjgomZpDzuMJXfmfty36k4gG/Xpv8K8pyaoDc7I9/QKQMinrT39+ApwprvN1Cc2CDAN82l2EUAB1LYGBQfWRpWp6NiOc3bO5NfMMGmR+zWq3UZPeA9NIQVD4uJ5oxHqQBHmqhJWAkIThn046mUICeJNB6dqSh8uiYnZOIfJSi6hryEO2qIowhBxLkQdQE5HpolCZFhLyhwWnKAg+Cmk9XrrpQSZETnRZIovwO3uCsn+7qRNKE+UF76bjnlUgYMwe 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: Hi Mike, On Wed, Oct 23, 2024 at 07:27:09PM +0300, Mike Rapoport wrote: > From: "Mike Rapoport (Microsoft)" > > When module text memory will be allocated with ROX permissions, the > memory at the actual address where the module will live will contain > invalid instructions and there will be a writable copy that contains the > actual module code. > > Update relocations and alternatives patching to deal with it. > > Signed-off-by: Mike Rapoport (Microsoft) > Tested-by: kdevops Hopefully the last time you have to hear from me, as I am only experiencing issues with only one of my test machines at this point and it is my only machine that supports IBT, so it seems to point to something specific with the IBT part of the FineIBT support. I notice either a boot hang or an almost immediate reboot (triple fault?). I guess this is how I missed reporting this earlier, as my machine was falling back to the default distribution kernel after the restart and I did not notice I was not actually testing a -next kernel. Checking out the version of this change that is in next-20241104, commit 7ca6ed09db62 ("x86/module: prepare module loading for ROX allocations of text"), it boots with either 'cfi=off' or 'cfi=kcfi' but it exhibits the issues noted above with 'cfi=fineibt'. At the immediate parent, commit b575d981092f ("arch: introduce set_direct_map_valid_noflush()"), all three combinations boot fine. $ uname -r; tr ' ' '\n'