mirror of
git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2025-08-05 16:54:27 +00:00

Inject fault while probing drm_kunit_helpers.ko, if one of kunit_next_attr_filter(), kunit_filter_glob_tests() and kunit_filter_attr_tests() fails, parsed_filters, parsed_glob.suite_glob/test_glob alloced in kunit_parse_glob_filter() is leaked. And the filtered_suite->test_cases alloced in kunit_filter_glob_tests() or kunit_filter_attr_tests() may also be leaked. unreferenced object 0xff110001067e4800 (size 1024): comm "kunit_try_catch", pid 96, jiffies 4294671796 (age 763.547s) hex dump (first 32 bytes): 73 75 69 74 65 32 00 00 00 00 00 00 00 00 00 00 suite2.......... 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................ backtrace: [<00000000116e8eba>] __kmalloc_node_track_caller+0x4e/0x140 [<00000000e2f9cce9>] kmemdup+0x2c/0x60 [<000000002a36710b>] kunit_filter_suites+0x3e4/0xa50 [<0000000045779fb9>] filter_suites_test+0x1b7/0x440 [<00000000cd1104a7>] kunit_try_run_case+0x119/0x270 [<00000000c654c917>] kunit_generic_run_threadfn_adapter+0x4e/0xa0 [<00000000d195ac13>] kthread+0x2c7/0x3c0 [<00000000b79c1ee9>] ret_from_fork+0x2c/0x70 [<000000001167f7e6>] ret_from_fork_asm+0x1b/0x30 unreferenced object 0xff11000105d79b00 (size 192): comm "kunit_try_catch", pid 96, jiffies 4294671796 (age 763.547s) hex dump (first 32 bytes): f0 e1 5a 88 ff ff ff ff 60 59 bb 8a ff ff ff ff ..Z.....`Y...... 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................ backtrace: [<000000000d6e4891>] __kmalloc+0x4d/0x140 [<000000006afe50bd>] kunit_filter_suites+0x424/0xa50 [<0000000045779fb9>] filter_suites_test+0x1b7/0x440 [<00000000cd1104a7>] kunit_try_run_case+0x119/0x270 [<00000000c654c917>] kunit_generic_run_threadfn_adapter+0x4e/0xa0 [<00000000d195ac13>] kthread+0x2c7/0x3c0 [<00000000b79c1ee9>] ret_from_fork+0x2c/0x70 [<000000001167f7e6>] ret_from_fork_asm+0x1b/0x30 unreferenced object 0xff110001067e6000 (size 1024): comm "kunit_try_catch", pid 98, jiffies 4294671798 (age 763.545s) hex dump (first 32 bytes): 73 75 69 74 65 32 00 00 00 00 00 00 00 00 00 00 suite2.......... 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................ backtrace: [<00000000116e8eba>] __kmalloc_node_track_caller+0x4e/0x140 [<00000000e2f9cce9>] kmemdup+0x2c/0x60 [<000000002a36710b>] kunit_filter_suites+0x3e4/0xa50 [<00000000f452f130>] filter_suites_test_glob_test+0x1b7/0x660 [<00000000cd1104a7>] kunit_try_run_case+0x119/0x270 [<00000000c654c917>] kunit_generic_run_threadfn_adapter+0x4e/0xa0 [<00000000d195ac13>] kthread+0x2c7/0x3c0 [<00000000b79c1ee9>] ret_from_fork+0x2c/0x70 [<000000001167f7e6>] ret_from_fork_asm+0x1b/0x30 unreferenced object 0xff11000103f3a800 (size 96): comm "kunit_try_catch", pid 98, jiffies 4294671798 (age 763.545s) hex dump (first 32 bytes): f0 e1 5a 88 ff ff ff ff 40 39 bb 8a ff ff ff ff ..Z.....@9...... 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................ backtrace: [<000000000d6e4891>] __kmalloc+0x4d/0x140 [<000000006afe50bd>] kunit_filter_suites+0x424/0xa50 [<00000000f452f130>] filter_suites_test_glob_test+0x1b7/0x660 [<00000000cd1104a7>] kunit_try_run_case+0x119/0x270 [<00000000c654c917>] kunit_generic_run_threadfn_adapter+0x4e/0xa0 [<00000000d195ac13>] kthread+0x2c7/0x3c0 [<00000000b79c1ee9>] ret_from_fork+0x2c/0x70 [<000000001167f7e6>] ret_from_fork_asm+0x1b/0x30 unreferenced object 0xff11000101a72ac0 (size 16): comm "kunit_try_catch", pid 104, jiffies 4294671814 (age 763.529s) hex dump (first 16 bytes): 00 00 00 00 00 00 00 00 e0 2a a7 01 01 00 11 ff .........*...... backtrace: [<000000000d6e4891>] __kmalloc+0x4d/0x140 [<00000000c7b724e7>] kunit_filter_suites+0x108/0xa50 [<00000000bad5427d>] filter_attr_test+0x1e9/0x6a0 [<00000000cd1104a7>] kunit_try_run_case+0x119/0x270 [<00000000c654c917>] kunit_generic_run_threadfn_adapter+0x4e/0xa0 [<00000000d195ac13>] kthread+0x2c7/0x3c0 [<00000000b79c1ee9>] ret_from_fork+0x2c/0x70 [<000000001167f7e6>] ret_from_fork_asm+0x1b/0x30 unreferenced object 0xff11000103caf880 (size 32): comm "kunit_try_catch", pid 104, jiffies 4294671814 (age 763.547s) hex dump (first 32 bytes): 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................ 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................ backtrace: [<000000000d6e4891>] __kmalloc+0x4d/0x140 [<00000000c47b0f75>] kunit_filter_suites+0x189/0xa50 [<00000000bad5427d>] filter_attr_test+0x1e9/0x6a0 [<00000000cd1104a7>] kunit_try_run_case+0x119/0x270 [<00000000c654c917>] kunit_generic_run_threadfn_adapter+0x4e/0xa0 [<00000000d195ac13>] kthread+0x2c7/0x3c0 [<00000000b79c1ee9>] ret_from_fork+0x2c/0x70 [<000000001167f7e6>] ret_from_fork_asm+0x1b/0x30 unreferenced object 0xff11000101a72ae0 (size 16): comm "kunit_try_catch", pid 106, jiffies 4294671823 (age 763.538s) hex dump (first 16 bytes): 00 00 00 00 00 00 00 00 00 2b a7 01 01 00 11 ff .........+...... backtrace: [<000000000d6e4891>] __kmalloc+0x4d/0x140 [<00000000c7b724e7>] kunit_filter_suites+0x108/0xa50 [<0000000096255c51>] filter_attr_empty_test+0x1b0/0x310 [<00000000cd1104a7>] kunit_try_run_case+0x119/0x270 [<00000000c654c917>] kunit_generic_run_threadfn_adapter+0x4e/0xa0 [<00000000d195ac13>] kthread+0x2c7/0x3c0 [<00000000b79c1ee9>] ret_from_fork+0x2c/0x70 [<000000001167f7e6>] ret_from_fork_asm+0x1b/0x30 unreferenced object 0xff11000103caf9c0 (size 32): comm "kunit_try_catch", pid 106, jiffies 4294671823 (age 763.538s) hex dump (first 32 bytes): 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................ 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................ backtrace: [<000000000d6e4891>] __kmalloc+0x4d/0x140 [<00000000c47b0f75>] kunit_filter_suites+0x189/0xa50 [<0000000096255c51>] filter_attr_empty_test+0x1b0/0x310 [<00000000cd1104a7>] kunit_try_run_case+0x119/0x270 [<00000000c654c917>] kunit_generic_run_threadfn_adapter+0x4e/0xa0 [<00000000d195ac13>] kthread+0x2c7/0x3c0 [<00000000b79c1ee9>] ret_from_fork+0x2c/0x70 [<000000001167f7e6>] ret_from_fork_asm+0x1b/0x30 unreferenced object 0xff11000101a72b00 (size 16): comm "kunit_try_catch", pid 108, jiffies 4294671832 (age 763.529s) hex dump (first 16 bytes): 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................ backtrace: [<000000000d6e4891>] __kmalloc+0x4d/0x140 [<00000000c47b0f75>] kunit_filter_suites+0x189/0xa50 [<00000000881258cc>] filter_attr_skip_test+0x148/0x770 [<00000000cd1104a7>] kunit_try_run_case+0x119/0x270 [<00000000c654c917>] kunit_generic_run_threadfn_adapter+0x4e/0xa0 [<00000000d195ac13>] kthread+0x2c7/0x3c0 [<00000000b79c1ee9>] ret_from_fork+0x2c/0x70 [<000000001167f7e6>] ret_from_fork_asm+0x1b/0x30 Fixes:5d31f71efc
("kunit: add kunit.filter_glob cmdline option to filter suites") Fixes:529534e8cb
("kunit: Add ability to filter attributes") Signed-off-by: Ruan Jinjie <ruanjinjie@huawei.com> Reviewed-by: Rae Moar <rmoar@google.com> Reviewed-by: David Gow <davidgow@google.com> Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
302 lines
8 KiB
C
302 lines
8 KiB
C
// SPDX-License-Identifier: GPL-2.0
|
|
|
|
#include <linux/reboot.h>
|
|
#include <kunit/test.h>
|
|
#include <kunit/attributes.h>
|
|
#include <linux/glob.h>
|
|
#include <linux/moduleparam.h>
|
|
|
|
/*
|
|
* These symbols point to the .kunit_test_suites section and are defined in
|
|
* include/asm-generic/vmlinux.lds.h, and consequently must be extern.
|
|
*/
|
|
extern struct kunit_suite * const __kunit_suites_start[];
|
|
extern struct kunit_suite * const __kunit_suites_end[];
|
|
|
|
#if IS_BUILTIN(CONFIG_KUNIT)
|
|
|
|
static char *filter_glob_param;
|
|
static char *action_param;
|
|
static char *filter_param;
|
|
static char *filter_action_param;
|
|
|
|
module_param_named(filter_glob, filter_glob_param, charp, 0);
|
|
MODULE_PARM_DESC(filter_glob,
|
|
"Filter which KUnit test suites/tests run at boot-time, e.g. list* or list*.*del_test");
|
|
module_param_named(action, action_param, charp, 0);
|
|
MODULE_PARM_DESC(action,
|
|
"Changes KUnit executor behavior, valid values are:\n"
|
|
"<none>: run the tests like normal\n"
|
|
"'list' to list test names instead of running them.\n"
|
|
"'list_attr' to list test names and attributes instead of running them.\n");
|
|
module_param_named(filter, filter_param, charp, 0);
|
|
MODULE_PARM_DESC(filter,
|
|
"Filter which KUnit test suites/tests run at boot-time using attributes, e.g. speed>slow");
|
|
module_param_named(filter_action, filter_action_param, charp, 0);
|
|
MODULE_PARM_DESC(filter_action,
|
|
"Changes behavior of filtered tests using attributes, valid values are:\n"
|
|
"<none>: do not run filtered tests as normal\n"
|
|
"'skip': skip all filtered tests instead so tests will appear in output\n");
|
|
|
|
/* glob_match() needs NULL terminated strings, so we need a copy of filter_glob_param. */
|
|
struct kunit_glob_filter {
|
|
char *suite_glob;
|
|
char *test_glob;
|
|
};
|
|
|
|
/* Split "suite_glob.test_glob" into two. Assumes filter_glob is not empty. */
|
|
static void kunit_parse_glob_filter(struct kunit_glob_filter *parsed,
|
|
const char *filter_glob)
|
|
{
|
|
const int len = strlen(filter_glob);
|
|
const char *period = strchr(filter_glob, '.');
|
|
|
|
if (!period) {
|
|
parsed->suite_glob = kzalloc(len + 1, GFP_KERNEL);
|
|
parsed->test_glob = NULL;
|
|
strcpy(parsed->suite_glob, filter_glob);
|
|
return;
|
|
}
|
|
|
|
parsed->suite_glob = kzalloc(period - filter_glob + 1, GFP_KERNEL);
|
|
parsed->test_glob = kzalloc(len - (period - filter_glob) + 1, GFP_KERNEL);
|
|
|
|
strncpy(parsed->suite_glob, filter_glob, period - filter_glob);
|
|
strncpy(parsed->test_glob, period + 1, len - (period - filter_glob));
|
|
}
|
|
|
|
/* Create a copy of suite with only tests that match test_glob. */
|
|
static struct kunit_suite *
|
|
kunit_filter_glob_tests(const struct kunit_suite *const suite, const char *test_glob)
|
|
{
|
|
int n = 0;
|
|
struct kunit_case *filtered, *test_case;
|
|
struct kunit_suite *copy;
|
|
|
|
kunit_suite_for_each_test_case(suite, test_case) {
|
|
if (!test_glob || glob_match(test_glob, test_case->name))
|
|
++n;
|
|
}
|
|
|
|
if (n == 0)
|
|
return NULL;
|
|
|
|
copy = kmemdup(suite, sizeof(*copy), GFP_KERNEL);
|
|
if (!copy)
|
|
return ERR_PTR(-ENOMEM);
|
|
|
|
filtered = kcalloc(n + 1, sizeof(*filtered), GFP_KERNEL);
|
|
if (!filtered) {
|
|
kfree(copy);
|
|
return ERR_PTR(-ENOMEM);
|
|
}
|
|
|
|
n = 0;
|
|
kunit_suite_for_each_test_case(suite, test_case) {
|
|
if (!test_glob || glob_match(test_glob, test_case->name))
|
|
filtered[n++] = *test_case;
|
|
}
|
|
|
|
copy->test_cases = filtered;
|
|
return copy;
|
|
}
|
|
|
|
static char *kunit_shutdown;
|
|
core_param(kunit_shutdown, kunit_shutdown, charp, 0644);
|
|
|
|
/* Stores an array of suites, end points one past the end */
|
|
struct suite_set {
|
|
struct kunit_suite * const *start;
|
|
struct kunit_suite * const *end;
|
|
};
|
|
|
|
static void kunit_free_suite_set(struct suite_set suite_set)
|
|
{
|
|
struct kunit_suite * const *suites;
|
|
|
|
for (suites = suite_set.start; suites < suite_set.end; suites++)
|
|
kfree(*suites);
|
|
kfree(suite_set.start);
|
|
}
|
|
|
|
static struct suite_set kunit_filter_suites(const struct suite_set *suite_set,
|
|
const char *filter_glob,
|
|
char *filters,
|
|
char *filter_action,
|
|
int *err)
|
|
{
|
|
int i, j, k;
|
|
int filter_count = 0;
|
|
struct kunit_suite **copy, *filtered_suite, *new_filtered_suite;
|
|
struct suite_set filtered;
|
|
struct kunit_glob_filter parsed_glob;
|
|
struct kunit_attr_filter *parsed_filters;
|
|
|
|
const size_t max = suite_set->end - suite_set->start;
|
|
|
|
copy = kmalloc_array(max, sizeof(*filtered.start), GFP_KERNEL);
|
|
filtered.start = copy;
|
|
if (!copy) { /* won't be able to run anything, return an empty set */
|
|
filtered.end = copy;
|
|
return filtered;
|
|
}
|
|
|
|
if (filter_glob)
|
|
kunit_parse_glob_filter(&parsed_glob, filter_glob);
|
|
|
|
/* Parse attribute filters */
|
|
if (filters) {
|
|
filter_count = kunit_get_filter_count(filters);
|
|
parsed_filters = kcalloc(filter_count + 1, sizeof(*parsed_filters), GFP_KERNEL);
|
|
for (j = 0; j < filter_count; j++)
|
|
parsed_filters[j] = kunit_next_attr_filter(&filters, err);
|
|
if (*err)
|
|
goto err;
|
|
}
|
|
|
|
for (i = 0; &suite_set->start[i] != suite_set->end; i++) {
|
|
filtered_suite = suite_set->start[i];
|
|
if (filter_glob) {
|
|
if (!glob_match(parsed_glob.suite_glob, filtered_suite->name))
|
|
continue;
|
|
filtered_suite = kunit_filter_glob_tests(filtered_suite,
|
|
parsed_glob.test_glob);
|
|
if (IS_ERR(filtered_suite)) {
|
|
*err = PTR_ERR(filtered_suite);
|
|
goto err;
|
|
}
|
|
}
|
|
if (filter_count) {
|
|
for (k = 0; k < filter_count; k++) {
|
|
new_filtered_suite = kunit_filter_attr_tests(filtered_suite,
|
|
parsed_filters[k], filter_action, err);
|
|
|
|
/* Free previous copy of suite */
|
|
if (k > 0 || filter_glob) {
|
|
kfree(filtered_suite->test_cases);
|
|
kfree(filtered_suite);
|
|
}
|
|
|
|
filtered_suite = new_filtered_suite;
|
|
|
|
if (*err)
|
|
goto err;
|
|
if (IS_ERR(filtered_suite)) {
|
|
*err = PTR_ERR(filtered_suite);
|
|
goto err;
|
|
}
|
|
if (!filtered_suite)
|
|
break;
|
|
}
|
|
}
|
|
|
|
if (!filtered_suite)
|
|
continue;
|
|
|
|
*copy++ = filtered_suite;
|
|
}
|
|
filtered.end = copy;
|
|
|
|
err:
|
|
if (*err)
|
|
kfree(copy);
|
|
|
|
if (filter_glob) {
|
|
kfree(parsed_glob.suite_glob);
|
|
kfree(parsed_glob.test_glob);
|
|
}
|
|
|
|
if (filter_count)
|
|
kfree(parsed_filters);
|
|
|
|
return filtered;
|
|
}
|
|
|
|
static void kunit_handle_shutdown(void)
|
|
{
|
|
if (!kunit_shutdown)
|
|
return;
|
|
|
|
if (!strcmp(kunit_shutdown, "poweroff"))
|
|
kernel_power_off();
|
|
else if (!strcmp(kunit_shutdown, "halt"))
|
|
kernel_halt();
|
|
else if (!strcmp(kunit_shutdown, "reboot"))
|
|
kernel_restart(NULL);
|
|
|
|
}
|
|
|
|
static void kunit_exec_run_tests(struct suite_set *suite_set)
|
|
{
|
|
size_t num_suites = suite_set->end - suite_set->start;
|
|
|
|
pr_info("KTAP version 1\n");
|
|
pr_info("1..%zu\n", num_suites);
|
|
|
|
__kunit_test_suites_init(suite_set->start, num_suites);
|
|
}
|
|
|
|
static void kunit_exec_list_tests(struct suite_set *suite_set, bool include_attr)
|
|
{
|
|
struct kunit_suite * const *suites;
|
|
struct kunit_case *test_case;
|
|
|
|
/* Hack: print a ktap header so kunit.py can find the start of KUnit output. */
|
|
pr_info("KTAP version 1\n");
|
|
|
|
for (suites = suite_set->start; suites < suite_set->end; suites++) {
|
|
/* Print suite name and suite attributes */
|
|
pr_info("%s\n", (*suites)->name);
|
|
if (include_attr)
|
|
kunit_print_attr((void *)(*suites), false, 0);
|
|
|
|
/* Print test case name and attributes in suite */
|
|
kunit_suite_for_each_test_case((*suites), test_case) {
|
|
pr_info("%s.%s\n", (*suites)->name, test_case->name);
|
|
if (include_attr)
|
|
kunit_print_attr((void *)test_case, true, 0);
|
|
}
|
|
}
|
|
}
|
|
|
|
int kunit_run_all_tests(void)
|
|
{
|
|
struct suite_set suite_set = {__kunit_suites_start, __kunit_suites_end};
|
|
int err = 0;
|
|
if (!kunit_enabled()) {
|
|
pr_info("kunit: disabled\n");
|
|
goto out;
|
|
}
|
|
|
|
if (filter_glob_param || filter_param) {
|
|
suite_set = kunit_filter_suites(&suite_set, filter_glob_param,
|
|
filter_param, filter_action_param, &err);
|
|
if (err) {
|
|
pr_err("kunit executor: error filtering suites: %d\n", err);
|
|
goto out;
|
|
}
|
|
}
|
|
|
|
if (!action_param)
|
|
kunit_exec_run_tests(&suite_set);
|
|
else if (strcmp(action_param, "list") == 0)
|
|
kunit_exec_list_tests(&suite_set, false);
|
|
else if (strcmp(action_param, "list_attr") == 0)
|
|
kunit_exec_list_tests(&suite_set, true);
|
|
else
|
|
pr_err("kunit executor: unknown action '%s'\n", action_param);
|
|
|
|
if (filter_glob_param || filter_param) { /* a copy was made of each suite */
|
|
kunit_free_suite_set(suite_set);
|
|
}
|
|
|
|
out:
|
|
kunit_handle_shutdown();
|
|
return err;
|
|
}
|
|
|
|
#if IS_BUILTIN(CONFIG_KUNIT_TEST)
|
|
#include "executor_test.c"
|
|
#endif
|
|
|
|
#endif /* IS_BUILTIN(CONFIG_KUNIT) */
|