Commit 7cfcd985 authored by Jason Wessel's avatar Jason Wessel

kgdb: 1000 loops for the single step test in kgdbts

The single step test is not terribly costly and it should be able to
pass at 1000 loops successfully in under 1 second.  A non-kgdb timing
regression was found using this test, but it did not occur frequently
because by default the test was only executed a single time.

This patch changes the default for the single step test to 1000
iterations and allows for individual configuration of the single step
test to further exercise the kgdb subsystem when needed.
Signed-off-by: default avatarJason Wessel <jason.wessel@windriver.com>
parent 001fddf5
...@@ -47,6 +47,7 @@ ...@@ -47,6 +47,7 @@
* to test the HW NMI watchdog * to test the HW NMI watchdog
* F## = Break at do_fork for ## iterations * F## = Break at do_fork for ## iterations
* S## = Break at sys_open for ## iterations * S## = Break at sys_open for ## iterations
* I## = Run the single step test ## iterations
* *
* NOTE: that the do_fork and sys_open tests are mutually exclusive. * NOTE: that the do_fork and sys_open tests are mutually exclusive.
* *
...@@ -875,7 +876,9 @@ static void kgdbts_run_tests(void) ...@@ -875,7 +876,9 @@ static void kgdbts_run_tests(void)
char *ptr; char *ptr;
int fork_test = 0; int fork_test = 0;
int do_sys_open_test = 0; int do_sys_open_test = 0;
int sstep_test = 1000;
int nmi_sleep = 0; int nmi_sleep = 0;
int i;
ptr = strstr(config, "F"); ptr = strstr(config, "F");
if (ptr) if (ptr)
...@@ -886,6 +889,9 @@ static void kgdbts_run_tests(void) ...@@ -886,6 +889,9 @@ static void kgdbts_run_tests(void)
ptr = strstr(config, "N"); ptr = strstr(config, "N");
if (ptr) if (ptr)
nmi_sleep = simple_strtol(ptr+1, NULL, 10); nmi_sleep = simple_strtol(ptr+1, NULL, 10);
ptr = strstr(config, "I");
if (ptr)
sstep_test = simple_strtol(ptr+1, NULL, 10);
/* required internal KGDB tests */ /* required internal KGDB tests */
v1printk("kgdbts:RUN plant and detach test\n"); v1printk("kgdbts:RUN plant and detach test\n");
...@@ -894,8 +900,13 @@ static void kgdbts_run_tests(void) ...@@ -894,8 +900,13 @@ static void kgdbts_run_tests(void)
run_breakpoint_test(0); run_breakpoint_test(0);
v1printk("kgdbts:RUN bad memory access test\n"); v1printk("kgdbts:RUN bad memory access test\n");
run_bad_read_test(); run_bad_read_test();
v1printk("kgdbts:RUN singlestep breakpoint test\n"); v1printk("kgdbts:RUN singlestep test %i iterations\n", sstep_test);
run_singlestep_break_test(); for (i = 0; i < sstep_test; i++) {
run_singlestep_break_test();
if (i % 100 == 0)
v1printk("kgdbts:RUN singlestep [%i/%i]\n",
i, sstep_test);
}
/* ===Optional tests=== */ /* ===Optional tests=== */
......
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