Re: [PATCH v3 1/5] x86: initialize secondary CPU only if master CPU will wait for it

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



* Igor Mammedov <imammedo@xxxxxxxxxx> wrote:

>  	/*
> +	 * wait for ACK from master CPU before continuing
> +	 * with AP initialization
> +	 */
> +	cpumask_set_cpu(cpu, cpu_initialized_mask);
> +	while (!cpumask_test_cpu(cpu, cpu_callout_mask))
> +		cpu_relax();

> +	/*
> +	 * wait for ACK from master CPU before continuing
> +	 * with AP initialization
> +	 */
> +	cpumask_set_cpu(cpu, cpu_initialized_mask);
> +	while (!cpumask_test_cpu(cpu, cpu_callout_mask))
> +		cpu_relax();

That repetitive pattern could be stuck into a properly named helper 
inline function.

(Also, before the cpumask_set_cpu() we should probably do a WARN_ON() 
if the bit is already set.)

Thanks,

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




[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux