From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1755021AbcAHMso (ORCPT <rfc822;w@1wt.eu>);
	Fri, 8 Jan 2016 07:48:44 -0500
Received: from mx2.suse.de ([195.135.220.15]:36258 "EHLO mx2.suse.de"
	rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
	id S1751351AbcAHMsm (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
	Fri, 8 Jan 2016 07:48:42 -0500
Subject: Re: x86/microcode update on systems without INITRD
To: Borislav Petkov <bp@suse.de>
References: <20151120082716.GA4028@pd.tnic>
 <alpine.LSU.2.11.1601071308020.23854@er-systems.de>
 <20160107121841.GB16472@pd.tnic>
 <alpine.LSU.2.11.1601071330090.23854@er-systems.de>
 <20160108105900.GA14673@pd.tnic> <yw1xio34z5d0.fsf@unicorn.mansr.com>
 <20160108113641.GE14673@pd.tnic> <yw1xegdsz42z.fsf@unicorn.mansr.com>
 <20160108120812.GF14673@pd.tnic> <20160108121601.GC320@x4>
 <20160108122719.GG14673@pd.tnic>
Cc: Markus Trippelsdorf <markus@trippelsdorf.de>,
        =?UTF-8?B?TcOlbnMgUnVsbGfDpXJk?= <mans@mansr.com>,
        Thomas Voegtle <tv@lio96.de>, linux-kernel@vger.kernel.org
From: Michal Marek <mmarek@suse.cz>
Message-ID: <568FB028.1030706@suse.cz>
Date: Fri, 8 Jan 2016 13:48:40 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101
 Thunderbird/38.3.0
MIME-Version: 1.0
In-Reply-To: <20160108122719.GG14673@pd.tnic>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 2016-01-08 13:27, Borislav Petkov wrote:
> On Fri, Jan 08, 2016 at 01:16:01PM +0100, Markus Trippelsdorf wrote:
>> But you take the choice away from people like me, who don't need initrd
>> at all. BLK_DEV_INITRD is a superfluous dependency in this case, because
>> microcode update works perfectly well without it.
> 
> Damn, you have a valid point too. I need to think about it...
> 
> ... well, the only thing I can think of right now is to remove the
> dependency on BLK_DEV_INITRD completely - I believe this is what you
> proposed initially Markus - and add a BIG FAT SUGGESTION to Kconfig
> saying that people should strive for enabling early microcode loading if
> possible.
> 
> This is the only way I see we can handle "make oldconfig" without
> BLK_DEV_INITRD and microcode built-in into the kernel cases relatively
> fair.
> 
> Hmmm.
> 
> Or I could hack "oldconfig" to issue that warning... Lemme think about
> it and see how much Michal would scream at me for it. :-)

You can add a conditional comment like this

comment "WARNING: Early microcode loader requires initramfs support"
	depends on MICROCODE && !BLK_DEV_INITRD

and hope that somebody reads it.

Michal