From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from wp530.webpack.hosteurope.de (wp530.webpack.hosteurope.de [80.237.130.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 59CE21CA9A for ; Wed, 22 Nov 2023 09:15:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=leemhuis.info Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=leemhuis.info Authentication-Results: smtp.subspace.kernel.org; dkim=none Received: from [2a02:8108:8980:2478:8cde:aa2c:f324:937e]; authenticated by wp530.webpack.hosteurope.de running ExIM with esmtpsa (TLS1.3:ECDHE_RSA_AES_128_GCM_SHA256:128) id 1r5jKg-0000kn-Ox; Wed, 22 Nov 2023 10:15:42 +0100 Message-ID: <0e9cbe6f-ac6c-47f2-b663-a22568799eca@leemhuis.info> Date: Wed, 22 Nov 2023 10:15:42 +0100 Precedence: bulk X-Mailing-List: regressions@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird From: "Linux regression tracking (Thorsten Leemhuis)" Subject: Re: [regression] microcode files missing in initramfs imgages from dracut (was Re: [PATCH] x86: Clean up remaining references to CONFIG_MICROCODE_AMD) Reply-To: Linux regressions mailing list To: Borislav Petkov , Linux regressions mailing list Cc: lukas.bulwahn@gmail.com, dave.hansen@linux.intel.com, hpa@zytor.com, kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org, mingo@redhat.com, tglx@linutronix.de, x86@kernel.org References: <20230825141226.13566-1-lukas.bulwahn@gmail.com> <20231112181036.GBZVEVHIIj/Oos1cx4@fat_crate.local> Content-Language: en-US, de-DE In-Reply-To: <20231112181036.GBZVEVHIIj/Oos1cx4@fat_crate.local> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-bounce-key: webpack.hosteurope.de;regressions@leemhuis.info;1700644546;e4ded815; X-HE-SMSGID: 1r5jKg-0000kn-Ox On 12.11.23 19:10, Borislav Petkov wrote: > On Sun, Nov 12, 2023 at 04:03:32PM +0100, Linux regression tracking (Thorsten Leemhuis) wrote: >> That's because dracut until the recent commit >> https://github.com/dracutdevs/dracut/commit/6c80408c8644a0add1907b0593eb83f90d6247b1 >> looked for CONFIG_MICROCODE_AMD and CONFIG_MICROCODE_INTEL in the config >> file to decide what to include or not. > > They've been told a bunch of times already that grepping .config for > specific symbols is not how one should check whether one should add > microcode blobs to the initrd or not because Kconfig symbols are not an > ABI. Maybe, but you know how Linus sees things like this: what's considered an ABI/API or not is nearly[1] irrelevant - if a change breaks something that used to work then it needs to be fixed. [1] unless you fiddle with things obviously internal; not sure if this case would qualify for him, but somehow I doubt it -- but I might be wrong there. > And looking at that commit, now they're grepping for CONFIG_MICROCODE. > And that'll break again if one day we decide to make the microcode > loader built in unconditionally. > > How to fix this reliably and properly? > > Honestly, I don't have a good idea. If we do something like this: > grep microcode_init System.map > > then that makes "microode_init" ABI and we won't be able to change it > eva. I'd need to do some digging here... Any progress on this? BTW: I see that this could help preventing problems like the current one to happen in the far future. But how would that help the current situation (e.g. users that have an old dracut and updated the kernel without updating dracut)? Ciao, Thorsten (wearing his 'the Linux kernel's regression tracker' hat) -- Everything you wanna know about Linux kernel regression tracking: https://linux-regtracking.leemhuis.info/about/#tldr If I did something stupid, please tell me, as explained on that page. #regzbot poke