]> granicus.if.org Git - zfs/commitdiff
Switch ztest mmap(2) ASSERTs to VERIFYs
authorRichard Yao <ryao@gentoo.org>
Thu, 26 Dec 2013 20:51:35 +0000 (15:51 -0500)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Wed, 12 Mar 2014 16:05:00 +0000 (09:05 -0700)
This is just a small bit of cleanup to ensure ztest fails early
on systems where mmap(2) is not functioning.  For the automated
testing which is the primary consumer of ztest there is no
functional change because debugging is always enabled.

Signed-off-by: Richard Yao <ryao@gentoo.org>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes #2177

cmd/ztest/ztest.c

index d04b4dea09c835b981698017f6a8057b4d46a2ae..d392a625e9aee71e29985b89ae740ce14700d92d 100644 (file)
@@ -6086,7 +6086,7 @@ setup_hdr(void)
 
        hdr = (void *)mmap(0, P2ROUNDUP(sizeof (*hdr), getpagesize()),
            PROT_READ | PROT_WRITE, MAP_SHARED, ztest_fd_data, 0);
-       ASSERT(hdr != MAP_FAILED);
+       VERIFY3P(hdr, !=, MAP_FAILED);
 
        VERIFY3U(0, ==, ftruncate(ztest_fd_data, sizeof (ztest_shared_hdr_t)));
 
@@ -6113,14 +6113,14 @@ setup_data(void)
 
        hdr = (void *)mmap(0, P2ROUNDUP(sizeof (*hdr), getpagesize()),
            PROT_READ, MAP_SHARED, ztest_fd_data, 0);
-       ASSERT(hdr != MAP_FAILED);
+       VERIFY3P(hdr, !=, MAP_FAILED);
 
        size = shared_data_size(hdr);
 
        (void) munmap((caddr_t)hdr, P2ROUNDUP(sizeof (*hdr), getpagesize()));
        hdr = ztest_shared_hdr = (void *)mmap(0, P2ROUNDUP(size, getpagesize()),
            PROT_READ | PROT_WRITE, MAP_SHARED, ztest_fd_data, 0);
-       ASSERT(hdr != MAP_FAILED);
+       VERIFY3P(hdr, !=, MAP_FAILED);
        buf = (uint8_t *)hdr;
 
        offset = hdr->zh_hdr_size;