From: Nadav Har'El <n...@scylladb.com>
Committer: Nadav Har'El <n...@scylladb.com>
Branch: master

tests/tst-kill: make compilable on Linux as well

Stop using OSv-specific features (namely debug()) in this test, so we
can also run it on Linux.

This test currently hangs on Linux, because the test assumes that signal()
sets system calls to not restart - while on Linux, they do.

Signed-off-by: Nadav Har'El <n...@scylladb.com>
Message-Id: <20170306121300.19582-1-...@scylladb.com>

---
diff --git a/tests/tst-kill.cc b/tests/tst-kill.cc
--- a/tests/tst-kill.cc
+++ b/tests/tst-kill.cc
@@ -10,22 +10,23 @@
 #include <sys/types.h>
 #include <signal.h>
 #include <sys/socket.h>
-
-#include <osv/debug.hh>
+#include <stdio.h>
+#include <unistd.h>
+#include <errno.h>

 int tests = 0, fails = 0;

 static void report(bool ok, const char* msg)
 {
     ++tests;
     fails += !ok;
-    debug("%s: %s\n", (ok ? "PASS" : "FAIL"), msg);
+    printf("%s: %s\n", (ok ? "PASS" : "FAIL"), msg);
 }

 int global = 0;

 void handler1(int sig) {
-    debug("handler1 called, sig=%d\n", sig);
+    printf("handler1 called, sig=%d, global=%d\n", sig, global);
     global = 1;
 }

@@ -144,12 +145,14 @@ int main(int ac, char** av)
report(r == 0 && oldact.sa_handler == SIG_DFL, "with SA_RESETHAND, signal handler is reset");


-    debug("SUMMARY: %d tests, %d failures\n", tests, fails);
+    printf("SUMMARY: %d tests, %d failures\n", tests, fails);

// At this point, handler1 might still be running, and if we return this
     // module, including handler1, might be unmapped. So sleep to make sure
     // that handler1 is done.
     sleep(1);
+
+    return fails == 0 ? 0 : 1;
 }


--
You received this message because you are subscribed to the Google Groups "OSv 
Development" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to osv-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to