Re: [PATCH RESEND] x86: kernel/microcode_core.c simple_strtoulcleanup

From: Ingo Molnar
Date: Tue May 08 2012 - 00:00:53 EST



* Henrique de Moraes Holschuh <hmh@xxxxxxxxxx> wrote:

> > Of course, one can iterate over each core in a shell-loop
> > and write into the reload file to reload ucode after having
> > updated the ucode image in /lib/firmware but removing and
> > then modprobing the module is shorter :-)
>
> Can we PLEASE fix it properly by adding a new node (which is
> _not_ per-cpu) that requests the microcode core to refresh all
> cpus? Preferably by invalidating the microcode cache, THEN
> fetching each required microcode just once for the first core
> that needs it, and caching it for use the other cores. You
> can leave the (IMHO mostly useless) per-cpu sysfs nodes alone,
> so as to not break ABI, or deprecate them for an year or
> something.
>
> I am speaking this with my userland maintainer hat. I *do
> NOT* want to rmmod crap in a production server to update
> microcode. And I want to be able to support static-compiled
> microcode.

Seconded. There's also the ability to disable module unloading.

Thanks,

Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/