Re: [Patch V1 2/7] x86/microcode/intel: Remove retries on early microcode load
From: Ashok Raj
Date: Fri Dec 02 2022 - 20:54:15 EST
On Fri, Dec 02, 2022 at 03:53:52PM -0800, Mehta, Sohil wrote:
> On 11/29/2022 1:08 PM, Ashok Raj wrote:
> > -
> > - if (apply_microcode_early(&uci, true)) {
> > - /* Mixed-silicon system? Try to refetch the proper patch: */
> > - *iup = NULL;
> > -
> > - goto reget;
> > - }
> > + apply_microcode_early(&uci, true);
>
> After this change, none of the callers of apply_microcode_early() check the
> return code.
>
> In future, do we expect callers to care about the return code? The rest
> patches in this series don't seem to suggest so. Also, the expected error
> printing happens in the function itself.
>
> Should the return type for apply_microcode_early() be changed to void (in a
> follow-up patch)?
Good idea.. But I think its early, the return code could be used for
something useful. I have some additional cleanup patches that I need to
fixup and we could use this for real.
For e.g. early loading failures are now reported by each vendor, if we can
consolidate this, we could do it more at a core level, but I'm worried it
might be too much change right now, and this can wait its turn.
Cheers,
Ashok