On Fri, Aug 12, 2016 at 7:28 AM, Josh Poimboeuf <[email protected]> wrote: > show_stack_log_lvl() and dump_trace() are already preemption safe: > > - If they're running in interrupt context, preemption is already > disabled and the percpu irq stack pointers can be trusted.
I agree with the patch, but I have a minor nitpick about the description. The "irq stack" is an actual thing, but the relevant stacks here aren't just the irq stack: they're the irq stack, the IST stacks (on 64-bit), and the NMI stack (on 32-bit). The same logic applies, though. --Andy
