> I think I'd go ahead and ACK this unless Tony has some comments. I'm not > happy about the two pr_emerg calls based on the conditional.
As written the patch has the nice property of not making any changes to the console output (except to eliminate the possibility of interleaved output that the original code had). Whereas making the change you suggest would end up with a useless "{}" in the usermode case (or worse if %pSR did something with m->ip .. which is most probably not NULL ... it should be a valid user mode instruction pointer when m->cs != KERNEL_CS). So Acked-by: Tony Luck <tony.l...@intel.com> -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/