Re: [patch] Regression in 2.6.19-rc microcode driver

On Mon, 06 Nov 2006 15:15:38 +0100
Arjan van de Ven <[email protected]> wrote:

> Hi,
> 
> if the microcode driver is built in (rather than module) there are some,
> ehm, interesting effects happening due to the new "call out to
> userspace" behavior that is introduced.. and which runs too early. The
> result is a boot hang; which is really nasty.
> 
> The patch below is a minimally safe patch to fix this regression for
> 2.6.19 by just not requesting actual microcode updates during early
> boot. (That is a good idea in general anyway)
> 
> The "real" fix is a lot more complex given the entire cpu hotplug
> scenario (during cpu hotplug you normally need to load the microcode as
> well); but the interactions for that are just really messy at this
> point; this fix at least makes it work and avoids a full detangle of
> hotplug.
> 
> Signed-off-by: Arjan van de Ven <[email protected]>
> 
> --- linux-2.6.18/arch/i386/kernel/microcode.c.org	2006-11-06 14:50:37.000000000 +0100
> +++ linux-2.6.18/arch/i386/kernel/microcode.c	2006-11-06 14:52:30.000000000 +0100
> @@ -577,7 +577,7 @@ static void microcode_init_cpu(int cpu)
>  	set_cpus_allowed(current, cpumask_of_cpu(cpu));
>  	mutex_lock(&microcode_mutex);
>  	collect_cpu_info(cpu);
> -	if (uci->valid)
> +	if (uci->valid && system_state==SYSTEM_RUNNING)
>  		cpu_request_microcode(cpu);
>  	mutex_unlock(&microcode_mutex);
>  	set_cpus_allowed(current, old);

Can we fix this by switching to late_initcall() or something like that?
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• Follow-Ups:
  • Re: [patch] Regression in 2.6.19-rc microcode driver
    • From: Arjan van de Ven <[email protected]>
  • Re: [patch] Regression in 2.6.19-rc microcode driver
    • From: Arjan van de Ven <[email protected]>

• References:
  • [patch] Regression in 2.6.19-rc microcode driver
    • From: Arjan van de Ven <[email protected]>

• Prev by Date: Re: [patch] Regression in 2.6.19-rc microcode driver
• Next by Date: Re: acpiphp makes noise on every lid close/open
• Previous by thread: [patch] Regression in 2.6.19-rc microcode driver
• Next by thread: Re: [patch] Regression in 2.6.19-rc microcode driver
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]