From: Dave Kleikamp <dave.kleikamp@xxxxxxxxxx> Date: Mon, 29 Dec 2014 17:23:10 -0600 > + while (1) ; Like the powerpc implementation, please correctly format this as: while (1) ; However I also have to wonder if we should be doing a hypervisor or firmware CPU stop call in that handler. That's what the XEN x86 implementation does. It should logically do something like: if (tlb_type == hypervisor) { if (ldom_domaining_enabled) sun4v_stop_cpu(cpuid); else prom_stopcpu_cpuid(cpuid); } else smp_call_function(wrapper_around_prom_stopself, NULL, 0); -- To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html