RE: [TRIVIAL PATCH 16/26] x86: Convert print_symbol to %pSR

[Luck, Tony]

> 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.luck@xxxxxxxxx>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/