Re: [PATCH] x86/microcode: Allow early loading without initrd

2015-04-30 Thread Henrique de Moraes Holschuh
On Sun, Apr 26, 2015, at 13:16, Borislav Petkov wrote: > * the early loader was done with initrd in mind and it was/still is its > main source for microcode blobs early in the boot. So if we want to > make it not-mandatory, then the driver needs to be reorganized so that > builtin blobs and initrd

Re: [PATCH] x86/microcode: Allow early loading without initrd

2015-04-27 Thread Borislav Petkov
On Sun, Apr 26, 2015 at 07:27:56PM +0200, Alexander Hirsch wrote: > I wasn't to happy about the nested ifdefs either, but given my > inexperience in the kernel code I didn't want to push things around > too much. Right, so this is the current situation: The intel early loader is being aggressively

Re: [PATCH] x86/microcode: Allow early loading without initrd

2015-04-26 Thread Alexander Hirsch
On Sun, 26 Apr 2015 18:16:09 +0200 Borislav Petkov wrote: > > Yeah, it is removed but it adds a bunch of ugly ifdeffery which makes > the code even more unreadable than it is :-\ > I wasn't to happy about the nested ifdefs either, but given my inexperience in the kernel code I didn't want to

Re: [PATCH] x86/microcode: Allow early loading without initrd

2015-04-26 Thread Alexander Hirsch
On Sun, 26 Apr 2015 17:31:13 +0200 Paul Bolle wrote: > #ifndef CONFIG_BLK_DEV_INITRD? Indeed, thank you. I fixed it. --- From: Alexander 'z33ky' Hirsch <1ze...@gmail.com> Date: Sun, 26 Apr 2015 15:18:18 +0200 Subject: [PATCH] x86, microcode: Allow early loading without initrd Microcode can b

Re: [PATCH] x86/microcode: Allow early loading without initrd

2015-04-26 Thread Borislav Petkov
On Sun, Apr 26, 2015 at 05:03:14PM +0200, Alexander Hirsch wrote: > Microcode can be baked into the kernel image via CONFIG_EXTRA_FIRMWARE > and the early loader supports that, but still depended on > BLK_DEV_INITRD. > This dependency is removed. Yeah, it is removed but it adds a bunch of ugly ifd

Re: [PATCH] x86/microcode: Allow early loading without initrd

2015-04-26 Thread Paul Bolle
On Sun, 2015-04-26 at 17:03 +0200, Alexander Hirsch wrote: > --- a/arch/x86/Kconfig > +++ b/arch/x86/Kconfig > config MICROCODE_EARLY > bool "Early load microcode" > - depends on MICROCODE=y && BLK_DEV_INITRD > + depends on MICROCODE=y > select MICROCODE_INTEL_EARLY if MICROC