Module Name:    src
Committed By:   kre
Date:           Wed Mar 13 08:50:12 UTC 2019

Modified Files:
        src/tests/lib/libc/sys: t_mlock.c

Log Message:
Add a bunch of fprintf(stderr,...) diagnostics to try and help determine
what is going wrong which is causing this test to crash the kernel.


To generate a diff of this commit:
cvs rdiff -u -r1.6 -r1.7 src/tests/lib/libc/sys/t_mlock.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/tests/lib/libc/sys/t_mlock.c
diff -u src/tests/lib/libc/sys/t_mlock.c:1.6 src/tests/lib/libc/sys/t_mlock.c:1.7
--- src/tests/lib/libc/sys/t_mlock.c:1.6	Tue Aug  9 12:02:44 2016
+++ src/tests/lib/libc/sys/t_mlock.c	Wed Mar 13 08:50:12 2019
@@ -1,4 +1,4 @@
-/* $NetBSD: t_mlock.c,v 1.6 2016/08/09 12:02:44 kre Exp $ */
+/* $NetBSD: t_mlock.c,v 1.7 2019/03/13 08:50:12 kre Exp $ */
 
 /*-
  * Copyright (c) 2012 The NetBSD Foundation, Inc.
@@ -29,7 +29,7 @@
  * POSSIBILITY OF SUCH DAMAGE.
  */
 #include <sys/cdefs.h>
-__RCSID("$NetBSD: t_mlock.c,v 1.6 2016/08/09 12:02:44 kre Exp $");
+__RCSID("$NetBSD: t_mlock.c,v 1.7 2019/03/13 08:50:12 kre Exp $");
 
 #include <sys/mman.h>
 #include <sys/resource.h>
@@ -41,6 +41,7 @@ __RCSID("$NetBSD: t_mlock.c,v 1.6 2016/0
 #include <stdint.h>
 #include <stdio.h>
 #include <stdlib.h>
+#include <string.h>
 #include <unistd.h>
 
 static long page = 0;
@@ -55,16 +56,26 @@ ATF_TC_HEAD(mlock_clip, tc)
 ATF_TC_BODY(mlock_clip, tc)
 {
 	void *buf;
+	int err1, err2;
 
 	buf = malloc(page);
 	ATF_REQUIRE(buf != NULL);
+	fprintf(stderr, "mlock_clip: buf = %p (page=%ld)\n", buf, page);
 
 	if (page < 1024)
 		atf_tc_skip("page size too small");
 
 	for (size_t i = page; i >= 1; i = i - 1024) {
-		(void)mlock(buf, page - i);
-		(void)munlock(buf, page - i);
+		err1 = mlock(buf, page - i);
+		if (err1 != 0)
+			fprintf(stderr, "mlock_clip: page=%ld i=%zu,"
+			    " mlock(%p, %ld): %s\n", page, i, buf, page - i,
+			    strerror(errno));
+		err2 = munlock(buf, page - i);
+		if (err2 != 0)
+			fprintf(stderr, "mlock_clip: page=%ld i=%zu,"
+			    " munlock(%p, %ld): %s (mlock %s)\n", page, i,
+			    buf, page - i, strerror(errno), err1?"failed":"ok");
 	}
 
 	free(buf);
@@ -81,6 +92,7 @@ ATF_TC_BODY(mlock_err, tc)
 {
 	void *invalid_ptr;
 	void *buf;
+	int mlock_err, munlock_err;
 
 	/*
 	 * Any bad address must return ENOMEM (for lock & unlock)
@@ -105,6 +117,7 @@ ATF_TC_BODY(mlock_err, tc)
 
 	buf = malloc(page);
 	ATF_REQUIRE(buf != NULL);
+	fprintf(stderr, "mlock_err: buf = %p (page=%ld)\n", buf, page);
 
 	/*
 	 * unlocking memory that is not locked is an error...
@@ -116,8 +129,18 @@ ATF_TC_BODY(mlock_err, tc)
 	/*
 	 * These are permitted to fail (EINVAL) but do not on NetBSD
 	 */
-	ATF_REQUIRE(mlock((void *)(((uintptr_t)buf) + page/3), page/5) == 0);
-	ATF_REQUIRE(munlock((void *)(((uintptr_t)buf) + page/3), page/5) == 0);
+	mlock_err = mlock((void *)(((uintptr_t)buf) + page/3), page/5);
+	if (mlock_err != 0)
+	    fprintf(stderr, "mlock_err: mlock(%p, %ld): %d [%d] %s\n",
+		(void *)(((uintptr_t)buf) + page/3), page/5, mlock_err,
+		errno, strerror(errno));
+	ATF_REQUIRE(mlock_err == 0);
+	munlock_err= munlock((void *)(((uintptr_t)buf) + page/3), page/5);
+	if (munlock_err != 0)
+	    fprintf(stderr, "mlock_err: munlock(%p, %ld): %d [%d] %s\n",
+		(void *)(((uintptr_t)buf) + page/3), page/5, munlock_err,
+		errno, strerror(errno));
+	ATF_REQUIRE(munlock_err == 0);
 
 	(void)free(buf);
 
@@ -150,6 +173,7 @@ ATF_TC_BODY(mlock_limits, tc)
 
 	buf = malloc(page);
 	ATF_REQUIRE(buf != NULL);
+	fprintf(stderr, "mlock_limits: buf = %p (page=%ld)\n", buf, page);
 
 	pid = fork();
 	ATF_REQUIRE(pid >= 0);
@@ -161,7 +185,7 @@ ATF_TC_BODY(mlock_limits, tc)
 			res.rlim_cur = i - 1;
 			res.rlim_max = i - 1;
 
-			(void)fprintf(stderr, "trying to lock %zd bytes "
+			(void)fprintf(stderr, "trying to lock %zu bytes "
 			    "with %zu byte limit\n", i, (size_t)res.rlim_cur);
 
 			if (setrlimit(RLIMIT_MEMLOCK, &res) != 0)
@@ -169,7 +193,9 @@ ATF_TC_BODY(mlock_limits, tc)
 
 			errno = 0;
 
-			if (mlock(buf, i) != -1 || errno != EAGAIN) {
+			if ((sta = mlock(buf, i)) != -1 || errno != EAGAIN) {
+				fprintf(stderr, "mlock(%p, %zu): %d [%d] %s\n",
+				    buf, i, sta, errno, strerror(errno));
 				(void)munlock(buf, i);
 				_exit(EXIT_FAILURE);
 			}
@@ -203,20 +229,39 @@ ATF_TC_BODY(mlock_mmap, tc)
 	 */
 	buf = mmap(NULL, page, PROT_READ | PROT_WRITE, flags, -1, 0);
 
+	if (buf == MAP_FAILED)
+		fprintf(stderr,
+		    "mlock_mmap: mmap(NULL, %ld, %#x, %#x, -1, 0): MAP_FAILED"
+		    " [%d] %s\n", page, PROT_READ | PROT_WRITE, flags, errno,
+		    strerror(errno));
+
 	ATF_REQUIRE(buf != MAP_FAILED);
+
+	fprintf(stderr, "mlock_mmap: buf=%p, page=%ld\n", buf, page);
+
 	ATF_REQUIRE(mlock(buf, page) == 0);
 	ATF_REQUIRE(munlock(buf, page) == 0);
 	ATF_REQUIRE(munmap(buf, page) == 0);
 	ATF_REQUIRE(munlock(buf, page) != 0);
 
+	fprintf(stderr, "mlock_mmap: first test succeeded\n");
+
 	/*
 	 * But it should be impossible to mlock(2) a PROT_NONE mapping.
 	 */
 	buf = mmap(NULL, page, PROT_NONE, flags, -1, 0);
 
+	if (buf == MAP_FAILED)
+		fprintf(stderr,
+		    "mlock_mmap: mmap(NULL, %ld, %#x, %#x, -1, 0): MAP_FAILED"
+		    " [%d] %s\n", page, PROT_NONE, flags, errno,
+		    strerror(errno));
+
 	ATF_REQUIRE(buf != MAP_FAILED);
 	ATF_REQUIRE(mlock(buf, page) != 0);
 	ATF_REQUIRE(munmap(buf, page) == 0);
+
+	fprintf(stderr, "mlock_mmap: second test succeeded\n");
 }
 
 ATF_TC(mlock_nested);
@@ -230,14 +275,26 @@ ATF_TC_BODY(mlock_nested, tc)
 {
 	const size_t maxiter = 100;
 	void *buf;
+	int err;
 
 	buf = malloc(page);
 	ATF_REQUIRE(buf != NULL);
+	fprintf(stderr, "mlock_nested: buf = %p (page=%ld)\n", buf, page);
 
-	for (size_t i = 0; i < maxiter; i++)
-		ATF_REQUIRE(mlock(buf, page) == 0);
+	for (size_t i = 0; i < maxiter; i++) {
+		err = mlock(buf, page);
+		if (err != 0)
+		    fprintf(stderr,
+		    "mlock_nested: i=%zu (of %zu) mlock(%p, %ld): %d [%d] %s\n",
+			i, maxiter, buf, page, err, errno, strerror(errno));
+		ATF_REQUIRE(err == 0);
+	}
 
-	ATF_REQUIRE(munlock(buf, page) == 0);
+	err = munlock(buf, page);
+	if (err != 0)
+		fprintf(stderr, "mlock_nested: munlock(%p, %ld): %d [%d] %s\n",
+		    buf, page, err, errno, strerror(errno));
+	ATF_REQUIRE(err == 0);
 	free(buf);
 }
 
@@ -245,6 +302,7 @@ ATF_TP_ADD_TCS(tp)
 {
 
 	page = sysconf(_SC_PAGESIZE);
+	fprintf(stderr, "t_mlock: pagesize %ld\n", page);
 	ATF_REQUIRE(page >= 0);
 
 	ATF_TP_ADD_TC(tp, mlock_clip);

Reply via email to