From mboxrd@z Thu Jan 1 00:00:00 1970 From: "H. Peter Anvin" Subject: Re: [PATCH 0/2] introduce post-init read-only memory Date: Mon, 30 Nov 2015 13:43:14 -0800 Message-ID: <565CC2F2.4020808@zytor.com> References: <1448401114-24650-1-git-send-email-keescook@chromium.org> <565595F5.32536.DB9FE75@pageexec.freemail.hu> <20151126085425.GA29848@gmail.com> <20151127075959.GA24991@gmail.com> <20151129080505.GA23721@gmail.com> <565CBC37.9090106@zytor.com> Reply-To: kernel-hardening@lists.openwall.com Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: List-Post: List-Help: List-Unsubscribe: List-Subscribe: In-Reply-To: To: Kees Cook Cc: Ingo Molnar , Andy Lutomirski , Linus Torvalds , PaX Team , "kernel-hardening@lists.openwall.com" , Mathias Krause , "linux-kernel@vger.kernel.org" , Ingo Molnar , Thomas Gleixner , x86-ml , Arnd Bergmann , Michael Ellerman , linux-arch , Emese Revfy List-Id: linux-arch.vger.kernel.org On 11/30/15 13:33, Kees Cook wrote: >> >> I think what should do is have a debug option which can be set to "rw", >> "log" or "oops"; the latter should probably be the default. > > Can someone write that patch, and then I will include it in the > series? I haven't touched fault handler code, and it would be faster > if someone more familiar with that area did it. :) > The "log" option (the only hard one) can be added as a later enhancement, and probably should be. It shouldn't block the series. -hpa From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from terminus.zytor.com ([198.137.202.10]:60090 "EHLO mail.zytor.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753602AbbK3Vnu (ORCPT ); Mon, 30 Nov 2015 16:43:50 -0500 Subject: Re: [kernel-hardening] [PATCH 0/2] introduce post-init read-only memory References: <1448401114-24650-1-git-send-email-keescook@chromium.org> <565595F5.32536.DB9FE75@pageexec.freemail.hu> <20151126085425.GA29848@gmail.com> <20151127075959.GA24991@gmail.com> <20151129080505.GA23721@gmail.com> <565CBC37.9090106@zytor.com> From: "H. Peter Anvin" Message-ID: <565CC2F2.4020808@zytor.com> Date: Mon, 30 Nov 2015 13:43:14 -0800 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-arch-owner@vger.kernel.org List-ID: To: Kees Cook Cc: Ingo Molnar , Andy Lutomirski , Linus Torvalds , PaX Team , "kernel-hardening@lists.openwall.com" , Mathias Krause , "linux-kernel@vger.kernel.org" , Ingo Molnar , Thomas Gleixner , x86-ml , Arnd Bergmann , Michael Ellerman , linux-arch , Emese Revfy Message-ID: <20151130214314.itqlV_U2S-0eYhkFK71sFcb1WNmwzNW2FbIuWK5hX2c@z> On 11/30/15 13:33, Kees Cook wrote: >> >> I think what should do is have a debug option which can be set to "rw", >> "log" or "oops"; the latter should probably be the default. > > Can someone write that patch, and then I will include it in the > series? I haven't touched fault handler code, and it would be faster > if someone more familiar with that area did it. :) > The "log" option (the only hard one) can be added as a later enhancement, and probably should be. It shouldn't block the series. -hpa