Stanley,

Thanks for reviewing!!

On 01/14/2011 03:38 PM, stanley.miao wrote:

Viresh Kumar wrote:

quoted

<snip>
+
+void __cpuinit platform_secondary_init(unsigned int cpu)
+{
<snip>
+
+	/*
+	 * let the primary processor know we're out of the
+	 * pen, then head off into the C entry point
+	 */
+	pen_release = -1;
+	smp_wmb();
  

Russell has fixed this line. See commit
3705ff6da538aff6dba535e2e9cbcbb9456d0d53 in the latest kernel.

Ya. I have already seen that in latest kernel and have updated it in
V4.

quoted

+
+	/*
+	 * Synchronise with the boot thread.
+	 */
+	spin_lock(&boot_lock);
+	spin_unlock(&boot_lock);
+}
+
<snip>
+void __init smp_prepare_cpus(unsigned int max_cpus)
+{
+	unsigned int ncores = get_core_count();
+	unsigned int cpu = smp_processor_id();
+	int i;
+
+	/* sanity check */
+	if (ncores == 0) {
+		pr_err("Realview: strange CM count of 0? Default to 1\n");
  

RealView ?

Oops!! Anyway this routine is changed now and this error is
also removed.

-- 
viresh