]> git.proxmox.com Git - mirror_ubuntu-focal-kernel.git/commit
ftrace: Handle tracing when switching between context
authorSteven Rostedt (VMware) <rostedt@goodmis.org>
Thu, 29 Oct 2020 23:35:08 +0000 (19:35 -0400)
committerStefan Bader <stefan.bader@canonical.com>
Thu, 10 Dec 2020 11:05:52 +0000 (12:05 +0100)
commit8fdde2d241cf3290412ebdfefd7b3dc0dcba62f0
tree3b8bcc2d8555baa6796835a1c6f7d047313c6a6a
parent4a11208adbd9916adf9f3674fa2f9a532170e3de
ftrace: Handle tracing when switching between context

BugLink: https://bugs.launchpad.net/bugs/1905612
commit 726b3d3f141fba6f841d715fc4d8a4a84f02c02a upstream.

When an interrupt or NMI comes in and switches the context, there's a delay
from when the preempt_count() shows the update. As the preempt_count() is
used to detect recursion having each context have its own bit get set when
tracing starts, and if that bit is already set, it is considered a recursion
and the function exits. But if this happens in that section where context
has changed but preempt_count() has not been updated, this will be
incorrectly flagged as a recursion.

To handle this case, create another bit call TRANSITION and test it if the
current context bit is already set. Flag the call as a recursion if the
TRANSITION bit is already set, and if not, set it and continue. The
TRANSITION bit will be cleared normally on the return of the function that
set it, or if the current context bit is clear, set it and clear the
TRANSITION bit to allow for another transition between the current context
and an even higher one.

Cc: stable@vger.kernel.org
Fixes: edc15cafcbfa3 ("tracing: Avoid unnecessary multiple recursion checks")
Signed-off-by: Steven Rostedt (VMware) <rostedt@goodmis.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Kamal Mostafa <kamal@canonical.com>
Signed-off-by: Kleber Sacilotto de Souza <kleber.souza@canonical.com>
kernel/trace/trace.h
kernel/trace/trace_selftest.c