Andy Lutomirski <l...@kernel.org> writes:

> On Fri, May 15, 2020 at 5:11 PM Thomas Gleixner <t...@linutronix.de> wrote:
>
> I think something's missing here.  With this patch applied, don't we
> get to exc_debug_kernel() -> handle_debug() without doing
> idtentry_enter() or equivalent?  And that can even enable IRQs.
>
> Maybe exc_debug_kernel() should wrap handle_debug() in some
> appropriate _enter() / _exit() pair?

I'm the one who is missing something here, i.e. the connection of this
patch to #DB. exc_debug_kernel() still looks like this:

        nmi_enter_notrace();
        handle_debug(regs, dr6, false);
        nmi_exit_notrace();

Confused.

Reply via email to