Convert objcache test to assert()
authorJack Miller <jack@codezen.org>
Tue, 5 Apr 2016 21:45:31 +0000 (16:45 -0500)
committerJack Miller <jack@codezen.org>
Tue, 5 Apr 2016 21:45:31 +0000 (16:45 -0500)
mm/objcache.c

index 6aa174f..ef1d002 100644 (file)
@@ -226,8 +226,7 @@ void test_objcache(void)
     struct objcache *a = NULL, *b = NULL;
     int i;
 
-    if(__objcache_pages(&test_cache) != 0)
-        while(1);
+    assert(__objcache_pages(&test_cache) == 0);
 
     /* Test struct is 32 bytes, which means 128 objects per page, one
      * of which is taken up by our bitmap, so we should be able to do 127
@@ -240,54 +239,46 @@ void test_objcache(void)
 
         if (b) {
             /* make sure they're same page */
-            if (PAGE_ALIGN((u64) a) != PAGE_ALIGN((u64) b))
-                while(1);
+            assert(PAGE_ALIGN((u64) a) == PAGE_ALIGN((u64) b));
 
             /* make sure that previous can be freed and re allocated */
             objcache_free(&test_cache, b);
-            if (objcache_get(&test_cache) != b)
-                while(1);
+
+            assert(objcache_get(&test_cache) == b);
         }
 
         /* make sure a is after b, also NULL check */
-        if ((u64) a <= (u64) b)
-            while(1);
+        assert((u64) a > (u64) b);
 
         /* shouldn't have caused expansion */
-        if(__objcache_pages(&test_cache) != 1)
-            while(1);
+        assert(__objcache_pages(&test_cache) == 1);
     }
 
     /* The 128th should cause expansion */
     a = objcache_get(&test_cache);
 
-    if(__objcache_pages(&test_cache) != 2)
-        while(1);
+    assert(__objcache_pages(&test_cache) == 2);
 
     /* 126 more, shouldn't expand */
     for(i = 0; i < 126; i++)
         objcache_get(&test_cache);
 
-    if(__objcache_pages(&test_cache) != 2)
-        while(1);
+    assert(__objcache_pages(&test_cache) == 2);
 
     a = objcache_get(&test_cache);
 
-    if(__objcache_pages(&test_cache) != 3)
-        while(1);
+    assert(__objcache_pages(&test_cache) == 3);
 
     /* Now we have 3 pages, with only a single object on the last page */
 
     /* A free of that single object should cause the cache to contract */
 
     objcache_free(&test_cache, a);
-    if(__objcache_pages(&test_cache) != 2)
-        while(1);
+    assert(__objcache_pages(&test_cache) == 2);
 
     /* Test destroy and clean up after our tests */
     objcache_destroy(&test_cache);
 
-    if(__objcache_pages(&test_cache) != 0)
-        while(1);
+    assert(__objcache_pages(&test_cache) == 0);
 }
 #endif