From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1162090AbbKTI1W (ORCPT ); Fri, 20 Nov 2015 03:27:22 -0500 Received: from mx2.suse.de ([195.135.220.15]:47637 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1162056AbbKTI1T (ORCPT ); Fri, 20 Nov 2015 03:27:19 -0500 Date: Fri, 20 Nov 2015 09:27:16 +0100 From: Borislav Petkov To: Markus Trippelsdorf Cc: linux-kernel@vger.kernel.org Subject: Re: x86/microcode update on systems without INITRD Message-ID: <20151120082716.GA4028@pd.tnic> References: <20151119214301.GA318@x4> <20151119215543.GA4926@pd.tnic> <20151119225828.GB4926@pd.tnic> <20151120071920.GA321@x4> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20151120071920.GA321@x4> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Nov 20, 2015 at 08:19:20AM +0100, Markus Trippelsdorf wrote: > On 2015.11.19 at 23:58 +0100, Borislav Petkov wrote: > > On Thu, Nov 19, 2015 at 10:55:43PM +0100, Borislav Petkov wrote: > > > On Thu, Nov 19, 2015 at 10:43:01PM +0100, Markus Trippelsdorf wrote: > > > > It looks like the ability to update x86/microcode without using an > > > > initrd was removed this merge window. > > > > > > Whoops, that shouldnt've happened. Will debug it tomorrow and provide a > > > fix. > > > > Anyway, the hunk below seems to work in my guest here, I'll run it > > on the rest of the boxes tomorrow. In case you want to give it a try > > before: > > Your patch works fine. Thanks. Thanks for testing. > But of course it needs this additional patch, otherwise the microcode > loader wouldn't build at all: > > diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig > index db3622f22b61..52c6964e24bd 100644 > --- a/arch/x86/Kconfig > +++ b/arch/x86/Kconfig > @@ -1126,7 +1126,6 @@ config MICROCODE > bool "CPU microcode loading support" > default y > depends on CPU_SUP_AMD || CPU_SUP_INTEL > - depends on BLK_DEV_INITRD > select FW_LOADER > ---help--- That dependency is a must as the early loader needs the initrd support. -- Regards/Gruss, Boris. SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton, HRB 21284 (AG Nürnberg) --