Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752402AbbFYIsf (ORCPT ); Thu, 25 Jun 2015 04:48:35 -0400 Received: from mx1.redhat.com ([209.132.183.28]:43962 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751448AbbFYIs3 (ORCPT ); Thu, 25 Jun 2015 04:48:29 -0400 Date: Thu, 25 Jun 2015 16:48:18 +0800 From: Dave Young To: Vivek Goyal Cc: "Eric W. Biederman" , Josh Boyer , "Theodore Ts'o" , Petr Tesarik , kexec , "Linux-Kernel@Vger. Kernel. Org" , David Howells Subject: Re: kexec_load(2) bypasses signature verification Message-ID: <20150625084818.GB20085@dhcp-128-31.nay.redhat.com> References: <87zj3zigug.fsf@x220.int.ebiederm.org> <20150616202757.GB14943@redhat.com> <87y4jjglvu.fsf@x220.int.ebiederm.org> <20150617014737.GB30214@redhat.com> <20150618011629.GA8718@dhcp-128-32.nay.redhat.com> <20150618020209.GA8390@dhcp-128-32.nay.redhat.com> <20150618133044.GA1040@redhat.com> <20150619062140.GH4636@dhcp-128-32.nay.redhat.com> <20150619081816.GK4636@dhcp-128-32.nay.redhat.com> <20150619130933.GB13714@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150619130933.GB13714@redhat.com> User-Agent: Mutt/1.5.22.1-rc1 (2013-10-16) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1611 Lines: 37 On 06/19/15 at 09:09am, Vivek Goyal wrote: > On Fri, Jun 19, 2015 at 04:18:16PM +0800, Dave Young wrote: > > > > If we want to disable unsigned kernel loading at compile time, then we > > > > really need to work on decoupling CONFIG_KEXEC and CONFIG_FILE_KEXEC. > > > > Introducing another config option is not the way forward, IMHO. > > > > > > Yes, let's do it in this way since everyone is fine with it. > > > > I will work on a patch if nobody else have interest or no time on it. > > Thanks Dave. Will be good if you can get this done. Vivek, I worked out some draft patches here: https://github.com/daveyoung/linux/commits/kexec-syscall-cleanup Basiclly I split kexec_file first, then add CONFIG_KEXEC_CORE kconfig option then split kexec_load code from general code. There's a lot of #ifdef CONFIG_KEXEC in kernel source, because CONFIG_KEXEC can be disabled so I changed all kernel general and x86 #ifdef to use CONFIG_KEXEC_CORE if necessary. For other arches dependent code with #ifdef I did not change anything other than the new Kconfig option. It will works because only x86 support KEXEC_FILE. Please take a look if you have time, if this is not what you want please let me know. I will have no time this week, only did building test, will do more test next week, if everything is ok I can send out the patches to list for review. Thanks Dave -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/