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 58027C27C4F for ; Thu, 13 Jun 2024 06:12:06 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 8E94D6B009C; Thu, 13 Jun 2024 02:12:05 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 8986A6B009D; Thu, 13 Jun 2024 02:12:05 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 738936B009E; Thu, 13 Jun 2024 02:12:05 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0015.hostedemail.com [216.40.44.15]) by kanga.kvack.org (Postfix) with ESMTP id 5477A6B009C for ; Thu, 13 Jun 2024 02:12:05 -0400 (EDT) Received: from smtpin29.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay10.hostedemail.com (Postfix) with ESMTP id E824FC0644 for ; Thu, 13 Jun 2024 06:12:04 +0000 (UTC) X-FDA: 82224844968.29.AC8E3CD Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf28.hostedemail.com (Postfix) with ESMTP id 0351DC0013 for ; Thu, 13 Jun 2024 06:12:02 +0000 (UTC) Authentication-Results: imf28.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=aLMc62CM; dmarc=pass (policy=none) header.from=kernel.org; spf=pass (imf28.hostedemail.com: domain of ardb@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=ardb@kernel.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1718259122; a=rsa-sha256; cv=none; b=nnLCOqDsxYsWFadMR9+UhL4x/w2OcFULZzQh5LRUJ/pie/7TItKuFFLp9zkQfNo89M1vxA /OrJ3YfpiUdFqukm33E6DgXyfYgWHsNYHN+HSmOO+513aLnY5750eVT+NAVIryYGfxLT2J KpesUOvEP+hSkdWlfYFs2X/jJXvcih8= ARC-Authentication-Results: i=1; imf28.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=aLMc62CM; dmarc=pass (policy=none) header.from=kernel.org; spf=pass (imf28.hostedemail.com: domain of ardb@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=ardb@kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1718259122; 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=JVTLncKH0WmALmNqFmhP++DIy50apOifR/KTBZlyNTQ=; b=wjzOuFDv+DmQBItQJQJZud5OfLM9337pPWuYPkf9Vy1W1sN7verbUyOhDAWB9Lb7ZMksJJ CNPnQOtup0HqQanAqhw+a8qoby//RlgEGhyEwKk5JOJaFcWEZGcGKaOkPvJAE+tfAksGBn 4BeRfwrU17LcYBbGJ7SYbhFKH35QsJc= Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 1569C61883 for ; Thu, 13 Jun 2024 06:12:02 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 97F25C4AF4D for ; Thu, 13 Jun 2024 06:12:01 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1718259121; bh=2tfIxnfDqiV8m2eUOgIRXcgva0U3ORhh129cB7f/07Y=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=aLMc62CMcYbhlI0BxA1bmCKIFpC7d6uUNzAaYgoMqzg59/gCWcohnthbnkHm3+PuL zVtwTkOMcMOvC7TikDQsC/2VSZMAFRluA1HBDadue43vDev0PzLlvojOLhhkNE1YCg ex0oFd7VBGibaU6QGkqbCRvkNW6/ksv7xrRJ5lEsQ83WEsxJKkJWb4mTGWG3N/K9QA Javtf6isZCNRW3fAseXH79Ym7TvCN4tEredwd9vy0ftIPHqUwvY2N3dddOOUW95Rbv XoB1GELyN67C3bG8sUp7ZHz7I499XFVQKJJ4mMRS5J7xQEgFSJU9bmkPkIsS8G4zhH lkCItH0cnFioQ== Received: by mail-lf1-f53.google.com with SMTP id 2adb3069b0e04-52c7f7fdd24so937537e87.1 for ; Wed, 12 Jun 2024 23:12:01 -0700 (PDT) X-Forwarded-Encrypted: i=1; AJvYcCWqwz9pPExRAmkWC534SfOlx2qMJLVIAodqqNWFicS/TSyr4dLRPU9ZgeFSzvR9m6BXt2pJLMFy5HORtmP9D6TrFds= X-Gm-Message-State: AOJu0YweB+dGNwLyNKc2JK3+mSw+pt9VKwLTV5EDDIo9J2AeM9UF26vO i92hGhX9xraVCBr5O3Nu8DlfqxdXzR0atCMPUDwQN9d4KiZ7ebk+eEa9gZtUnakssqluWyP4VCp 2orkQWWiJMKXHgXNQOVHNFyLkJz4= X-Google-Smtp-Source: AGHT+IFdLbYZojyhqvDEXOP8w6OnQtiDQDSpYwdN44bj4F4g15wPRxlf8c6AOb6GWfv8yK5ap3XFShBJ/euTC9+lXSo= X-Received: by 2002:ac2:4a82:0:b0:52b:cb05:ed46 with SMTP id 2adb3069b0e04-52c9a3c79a1mr2099464e87.21.1718259119855; Wed, 12 Jun 2024 23:11:59 -0700 (PDT) MIME-Version: 1.0 References: <20240611144911.327227285@goodmis.org> <20240611144949.703297941@goodmis.org> <202406121145.8860502D7@keescook> <20240612145228.5bf426e0@rorschach.local.home> In-Reply-To: From: Ard Biesheuvel Date: Thu, 13 Jun 2024 08:11:48 +0200 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v3 2/2] pstore/ramoops: Add ramoops.mem_name= command line option To: Mike Rapoport Cc: Steven Rostedt , Kees Cook , linux-kernel@vger.kernel.org, linux-trace-kernel@vger.kernel.org, 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 , 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 Content-Type: text/plain; charset="UTF-8" X-Rspamd-Queue-Id: 0351DC0013 X-Rspam-User: X-Rspamd-Server: rspam05 X-Stat-Signature: bcc66gzf5p3nmdy7s9f1pn1w86fmin53 X-HE-Tag: 1718259122-566037 X-HE-Meta: U2FsdGVkX1/Oo1AEBHi+dqurmks2P80VODGAwAevaUqCnGO8f0vFphYAxZuVqu4e77K7Woh3cD2PWE0R7XQ158ZrqjxatxQR7fA/xiPydv38rHT+/KrwWickohgLDkaDULZqmVgbFyqsqaw7SxsIkDzgaEsfi7riLpwqeSFrsw99eDkt3OTHhsTH59c56s+sEoSl6KxCJHmu6bDv6x7NjBOIEnbczZ7TjGKOHjjaZncTK7AOVgzHTeqknqNKucenr0E9yzWfpk6OMvwdohx8+Y04nnBRm2phzyeJuy7XQro3edqvSdlFy5CZdBFfpgLhEV8Y/QOuRnBYDe8B/lvSsiw4CqZUkg0ao10M8rGnFm0S/HpfXT6yBieJA49ZDRyvn5z2+wJIpyIg70d2RWuLQVX8aLw/EK17qUebhlrWu69B5HwzS27nM7wJBobtxrKJjrl7shm9dvLYkmi5OvKIHGH9axMSoqiwgDTChNr4iQsXJFu8SM/AeI21nWhaWQDX0eBZo3WCU2kJxdVhqwaqpjNRzsfrgHY6rbbPLsDTccub1NzLzRCZVo+FB+rSNrPZTIuNwwV+jfTyTT+2Gsdrtzg96qXzYr5Eir+zpWn9aA3EFqp/Z+GXaVKc5Om0TtzMsdU9D2VlIM1tU7G3eGJikcMu+M0ULG5YeY0fzwGrhLAM2UCXK8HVAN1I8RwfdrRp/2HKlMaoNSWLAGuQd0k1Y6VW6+0kRT0sbjz209wp0T6Zxj2oiWupHsYQ7pCm8ZXo1/0DwuSyQ+ch94Qeol9HmVuYD1MdqVgd6g+40q72ZNBTCEugxEFHU0wC2FCs08OpUEstHPM9+4g3EWSU+tfelBpIIRYOWYSdeZlJmACeMLXsqe+mFzdQ8YUZ+8z7f40ZoAMQmkyg7CjqsLkmqzbyn/SSfaEVV+725AI3632LNzZzRfYJjjzkWyWH+arOWOfFWtrLEw9tBEq1sQjWOa8 48Bkw5jq bW0hGVusxoHcAjhRtE9sjA8qsluldROBG+z+ly2RbHSaZs8bBExDh8K0AKummSxvRgnIBOHtFdjee4fGdgIg4y+ZtucETw2aGlG5Xgt7B7o52q5u1fZwgSf8AmlLvTquHBOth5Lkzl4CJyl1Iew+UZ9VccCKX2XKbhgcfANO0j8Ee7kACaJo6g6yIUTEqQtq2S9errPb+EI5V09RcWKxh1KO4f9CaHbUqxamUUkf2/r4hJ2Jqsf5ThlZrOHl7Aq0UIKOkcobbJLZwq3Xj6ke1OVQC9Y1Pacl2rQyarlvlWiRWipxCpu2m1HSkdkPLB3KZGqrdV8s2D5pLlbiRqZ8Z3Kao0S1Sx+kCrPtDSZ8x3+QqcsO7jQJsNGgouo/7OJqGd0pHJXm5ziJRUqt93iAiUokQjGkk7ZT+DG7PSDX3CI+Ep4o= 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, 13 Jun 2024 at 08:04, Mike Rapoport wrote: > > On Wed, Jun 12, 2024 at 02:52:28PM -0400, Steven Rostedt wrote: > > On Wed, 12 Jun 2024 11:45:57 -0700 > > Kees Cook wrote: > > > > > On Tue, Jun 11, 2024 at 10:49:13AM -0400, Steven Rostedt wrote: > > > > From: "Steven Rostedt (Google)" > > > > > > > > Add a method to find a region specified by reserve_mem=nn:align:name for > > > > ramoops. Adding a kernel command line parameter: > > > > > > > > reserve_mem=12M:4096:oops ramoops.mem_name=oops > > > > > > > > Will use the size and location defined by the memmap parameter where it > > > > finds the memory and labels it "oops". The "oops" in the ramoops option > > > > is used to search for it. > > > > > > > > This allows for arbitrary RAM to be used for ramoops if it is known that > > > > the memory is not cleared on kernel crashes or soft reboots. > > > > > > > > Signed-off-by: Steven Rostedt (Google) > > > > > > Acked-by: Kees Cook > > > > > > Let me know if this should go via the pstore tree, if you'd rather carry > > > it? > > > > > > > I'm going to send out another version to cover some more comments that > > were made, and perhaps it's best if Mike Rapoport takes it through his > > tree. > > I've added one more comment to v5, with that fixed I can take this. > So how is this supposed to work wrt to the rigid 'no user visible regressions' rule, given that this whole thing is a best effort thing to begin with. This needs at least a huge disclaimer that this rule does not apply, and if this works today, there is no guarantee that it will keep working on newer kernels. Otherwise, you will be making the job of the people who work on the boot code significantly more difficult. And even then, I wonder whether Linus and #regzcop are going to honour such a disclaimer. So this belongs downstream, unless some guarantees can be provided that this functionality is exempt from the usual regression policies.