On Tue, Jan 29, 2019 at 03:26:18AM -0700, Jan Beulich wrote: >>>> Chao Gao <chao....@intel.com> 01/28/19 8:06 AM >>> >>This check has been done in microcode_sanity_check(). Needn't do it >>again in get_matching_microcode(). > >But while there are two call sites of get_matching_microcode() only >one is preceded by a call to microcode_sanity_check(). There's also >no visible connection between cpu_request_microcode() (which frees >the blob it has validated) and microcode_resume_match() (and in >particular the blob passed into there). What am I missing?
microcode_resume_match() takes a cached blob; the blob should have passed the sanity check. I will explain it in the commit message. Thanks Chao _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
