[PR PATCH] intel-ucode: use iucode-tool with earlyfw option

[PR PATCH] intel-ucode: use iucode-tool with earlyfw option

[dkwo]

[-- Attachment #1: Type: text/plain, Size: 463 bytes --]

There is a new pull request by dkwo against master on the void-packages repository

https://github.com/dkwo/void-packages ucode
https://github.com/void-linux/void-packages/pull/48234

intel-ucode: use iucode-tool with earlyfw option
- I tested the changes in this PR: yes, by `cat ucode old-init > new-init`
- I built this PR locally for my native architecture, (x86_64)

A patch file from https://github.com/void-linux/void-packages/pull/48234.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-ucode-48234.patch --]
[-- Type: text/x-diff, Size: 1478 bytes --]

From 2b6797b1afde370b1fb0ea0be0bd34c1fb6e4768 Mon Sep 17 00:00:00 2001
From: dkwo <nicolopiazzalunga@gmail.com>
Date: Mon, 15 Jan 2024 09:53:13 +0100
Subject: [PATCH] intel-ucode: use iucode-tool with earlyfw option

---
 srcpkgs/intel-ucode/template | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/srcpkgs/intel-ucode/template b/srcpkgs/intel-ucode/template
index eb4060e3201ca..0b24bbd79f8b4 100644
--- a/srcpkgs/intel-ucode/template
+++ b/srcpkgs/intel-ucode/template
@@ -1,8 +1,9 @@
 # Template file for 'intel-ucode'
 pkgname=intel-ucode
 version=20231114
-revision=1
+revision=2
 archs="i686* x86_64*"
+hostmakedepends="iucode-tool"
 short_desc="Microcode update files for Intel CPUs"
 maintainer="Orphaned <orphan@voidlinux.org>"
 license="custom: Proprietary"
@@ -12,10 +13,15 @@ distfiles="https://github.com/intel/Intel-Linux-Processor-Microcode-Data-Files/a
 checksum=cee26f311f7e2c039dd48cd30f995183bde9b98fb4c3039800e2ddaf5c090e55
 repository=nonfree
 
+do_build() {
+	rm -f intel-ucode{,-with-caveats}/list
+	iucode_tool -v --write-earlyfw=intel-ucode.img intel-ucode{,-with-caveats}/
+}
+
 do_install() {
-	vmkdir usr/lib/firmware/intel-ucode
-	vcopy "intel-ucode/*" usr/lib/firmware/intel-ucode
+	vinstall intel-ucode.img 0644 usr/lib/firmware/intel-ucode
+	vlicense license
+
 	vmkdir usr/lib/dracut/dracut.conf.d
 	echo "early_microcode=yes" >> ${DESTDIR}/usr/lib/dracut/dracut.conf.d/intel_ucode.conf
-	vlicense license
 }

Re: [PR PATCH] [Closed]: intel-ucode: use iucode-tool with earlyfw option

[dkwo]

[-- Attachment #1: Type: text/plain, Size: 320 bytes --]

There's a closed pull request on the void-packages repository

intel-ucode: use iucode-tool with earlyfw option
https://github.com/void-linux/void-packages/pull/48234

Description:
- I tested the changes in this PR: yes, by `cat ucode old-init > new-init`
- I built this PR locally for my native architecture, (x86_64)

Re: intel-ucode: use iucode-tool with earlyfw option

[dkwo]

[-- Attachment #1: Type: text/plain, Size: 431 bytes --]

New comment by dkwo on void-packages repository

https://github.com/void-linux/void-packages/pull/48234#issuecomment-1894387310

Comment:
got it. i agree that microcode should not be put into /boot (which i did not), and that it should not be precompiled with iucode-tool (as i tried to do, and also e.g. alpine does).

btw, the above works with booster and it gives me `[    0.572185] microcode: Updated early from: 0x000000f0`