On Thu, Nov 03, 2022 at 12:33:50PM +0100, Borislav Petkov wrote: > On Fri, Oct 21, 2022 at 01:34:06PM -0700, Jithu Joseph wrote: > > IFS test image carries the same microcode header as regular Intel > > microcode blobs. Microcode blobs use header version of 1, > > whereas IFS test images will use header version of 2. > > > > microcode_sanity_check() can be used by IFS driver to perform > > sanity check of the IFS test images too. > > So I'm very sceptical about such reuse. The generic parts of the microcode is completely common between IFS and CPU microcode. The parts that differ are in the meta-data. CPU microcode has stayed the same for over 15 years roughly. Changing formats isn't something that's done lightly especially since there is cost of enabling in all OS's. Both are in the same class of microcode, consumed by the CPU. Inside Intel the format change is also pretty strict that they have several levels of approval within the HW teams and also needs review and approval from SW teams. What probably is best is that we should document that the external formats are shared in the header and probably in the shared routines. Changing names as you suggested to be more generic is in the right track. > Then the testing burden won't be an issue and there won't be any > potential cross-breakages. Even if its only a few hundred lines of code, it seems odd to simply duplicate for the sake of fear of testing.
