Re: [PATCH v4] scripts/x86/intel: Add a script to update the old microcode list

[Dave Hansen]

On 1/30/26 15:09, Sohil Mehta wrote:
> Dave probably has a use case where he passes multiple directories to the
> script. IIUC, the new version just inserts a unique entry for every FMS
> + pf_mask and skips the repeats. But it doesn't necessarily use the
> latest revision for the duplicates. We probably need an equivalent logic
> for this:
> 
>     if existing is not None:
>         # If the existing one is newer, just move on:
>         if existing['rev'] > rev['rev']:
>             continue
> 
> Dave, opinion?

I can't think of a case where I wanted to use multiple directories for a
single run of the script. The only thing I tend to do is switch between
the distr-provided ucode in /lib/firmware and a local clone of the Intel
microcode git repo. So, one or the other, not both at once.