Re: [PATCH 2/2] microcode update driver rewrite

[Arjan van de Ven <arjan@infradead.org>]

On Wed, 2006-05-24 at 21:05 -0700, Greg KH wrote:
> On Thu, May 25, 2006 at 11:50:49AM +0800, Shaohua Li wrote:
> > This is the rewrite of microcode update driver. Changes:
> > 1. trim the code
> 
> Hm, but the code is now bigger.

... because of the compat stuff. Maybe that should be split into a
separate file after some time.


> Don't use request_firmware, use request_firmware_nowait() instead.  That
> way you never stall.  You only want to update the firmware when
> userspace tells you to, not for every boot like I think this will do.

this isn't persistent firmware, it's microcode. That is volatile, to the
point that if you reboot (but that effect is also achieved at cpu S3 or
do cpu hotplug) you need to reload it. These types of events are
something the kernel knows about in general, but userspace.. not so.
(esp since you generally want to load microcode as early as possible)

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