From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1162194AbbKTIdx (ORCPT ); Fri, 20 Nov 2015 03:33:53 -0500 Received: from ud10.udmedia.de ([194.117.254.50]:33882 "EHLO mail.ud10.udmedia.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1162179AbbKTIdv (ORCPT ); Fri, 20 Nov 2015 03:33:51 -0500 Date: Fri, 20 Nov 2015 09:33:48 +0100 From: Markus Trippelsdorf To: Borislav Petkov Cc: linux-kernel@vger.kernel.org Subject: Re: x86/microcode update on systems without INITRD Message-ID: <20151120083348.GB321@x4> References: <20151119214301.GA318@x4> <20151119215543.GA4926@pd.tnic> <20151119225828.GB4926@pd.tnic> <20151120071920.GA321@x4> <20151120082716.GA4028@pd.tnic> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20151120082716.GA4028@pd.tnic> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2015.11.20 at 09:27 +0100, Borislav Petkov wrote: > 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. Well, BLK_DEV_INITRD is not set in my config and it works just fine... -- Markus