Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/tests/modules Fix flaky check for the /dev/kcov device in t_...



details:   https://anonhg.NetBSD.org/src/rev/0ec86b552128
branches:  trunk
changeset: 840358:0ec86b552128
user:      kamil <kamil%NetBSD.org@localhost>
date:      Thu Apr 04 10:53:20 2019 +0000

description:
Fix flaky check for the /dev/kcov device in t_kcov

Add a top-level check for the KCOV device, instead of deferring it ot each
thread. The thread-based solution was flaky.

PR kern/54064 by Andreas Gustafsson

diffstat:

 tests/modules/t_kcov.c |  8 ++++++++
 1 files changed, 8 insertions(+), 0 deletions(-)

diffs (18 lines):

diff -r e61ab2b6f814 -r 0ec86b552128 tests/modules/t_kcov.c
--- a/tests/modules/t_kcov.c    Thu Apr 04 08:16:24 2019 +0000
+++ b/tests/modules/t_kcov.c    Thu Apr 04 10:53:20 2019 +0000
@@ -514,6 +514,14 @@
 {
        pthread_t thread[32];
        size_t i;
+       int fd;
+
+       /*
+        * Check if /dev/kcov is available, if not bail out.
+        * Verifying it on a per-thread basis is flaky.
+        */
+       fd = open_kcov();
+       ATF_REQUIRE(close(fd) == 0);
 
        ATF_REQUIRE(__arraycount(thread) >= N);
 



Home | Main Index | Thread Index | Old Index