Commit b08e84da authored by Aleksandr Nogikh's avatar Aleksandr Nogikh Committed by Jakub Kicinski

kernel: make kcov_common_handle consider the current context

kcov_common_handle is a method that is used to obtain a "default" KCOV
remote handle of the current process. The handle can later be passed
to kcov_remote_start in order to collect coverage for the processing
that is initiated by one process, but done in another. For details see
Documentation/dev-tools/kcov.rst and comments in kernel/kcov.c.

Presently, if kcov_common_handle is called in an IRQ context, it will
return a handle for the interrupted process. This may lead to
unreliable and incorrect coverage collection.

Adjust the behavior of kcov_common_handle in the following way. If it
is called in a task context, return the common handle for the
currently running task. Otherwise, return 0.
Signed-off-by: default avatarAleksandr Nogikh <nogikh@google.com>
Reviewed-by: default avatarAndrey Konovalov <andreyknvl@google.com>
Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 0e8c266c
...@@ -1023,6 +1023,8 @@ EXPORT_SYMBOL(kcov_remote_stop); ...@@ -1023,6 +1023,8 @@ EXPORT_SYMBOL(kcov_remote_stop);
/* See the comment before kcov_remote_start() for usage details. */ /* See the comment before kcov_remote_start() for usage details. */
u64 kcov_common_handle(void) u64 kcov_common_handle(void)
{ {
if (!in_task())
return 0;
return current->kcov_handle; return current->kcov_handle;
} }
EXPORT_SYMBOL(kcov_common_handle); EXPORT_SYMBOL(kcov_common_handle);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment