Re: [PATCH] arch/x86/kernel/cpu/microcode/intel: don't store initrd's start
From: Nicolai Stange
Date: Mon Jul 25 2016 - 13:44:19 EST
Borislav Petkov <bp@xxxxxxxxx> writes:
> Yeah, we should tie this to CONFIG_RANDOMIZE_MEMORY. IOW, here's another
> version, I'll do some more hammering on it tomorrow.
Boots fine and updates microcode (as always, on top of next-20160722).
> It should take care of the builtin case too as there we have the
> microcode in the kernel text mapping which is already relocated when we
> go search for microcode blobs so no need for adjusting start then.
Now that you've got basically the same #ifdefery and if(->valid)'s in
your final result as I did, may I ask whether you would still consider
it as being simpler? In particular, what's the point of having that
->start if it's redundant and has to be corrected in a more or less
hackish way anyway?
Thanks,
Nicolai