On 04/17/20 10:14, Gary Lin wrote: > On Sat, Feb 29, 2020 at 09:05:45AM -0600, Leo Duran wrote: >> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2556 >> >> This patch uses CPUID signature check to skip reading the PlatformId MSR, >> which is not implemented on AMD processors. >> >> The PlatformId is used for loading microcode patches, which is also not >> supported and AMD-based platforms. To mitigate the PlatformId dependency, >> PcdCpuMicrocodePatchAddress and PcdCpuMicrodePatchRegionSize must be set >> to 0 (default value), in order to bypass microcode loading code paths. >> > I got an error report about the executable bit of MpLib.h while > packaging ovmf. It turned out that this patch accidentally changed the > file mode. It would be nice if the mode can be corrected.
Thanks for the report. I've filed <https://bugzilla.tianocore.org/show_bug.cgi?id=2678> and assigned it to Leo. Leo, can you please submit the patch? Thanks! Laszlo -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#57605): https://edk2.groups.io/g/devel/message/57605 Mute This Topic: https://groups.io/mt/71626798/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-