Remove old memory tests
authorBarret Rhoden <brho@cs.berkeley.edu>
Tue, 22 Nov 2016 20:41:09 +0000 (15:41 -0500)
committerBarret Rhoden <brho@cs.berkeley.edu>
Tue, 29 Nov 2016 16:27:40 +0000 (11:27 -0500)
They are unused and just print a bunch of stuff that no one looks at.
Also, I want to remove the printing functions in lieu of better debugging
diagnostics.

Signed-off-by: Barret Rhoden <brho@cs.berkeley.edu>
kern/include/slab.h
kern/src/ktest/pb_ktests.c
kern/src/slab.c

index 6291f2a..705befe 100644 (file)
@@ -144,7 +144,3 @@ void __kmem_cache_create(struct kmem_cache *kc, const char *name,
                          struct arena *source,
                          void (*ctor)(void *, size_t),
                          void (*dtor)(void *, size_t));
-
-/* Debug */
-void print_kmem_cache(struct kmem_cache *kc);
-void print_kmem_slab(struct kmem_slab *slab);
index 3be4a78..640a575 100644 (file)
@@ -568,69 +568,6 @@ bool test_kernel_messages(void)
        return true;
 }
 #endif // CONFIG_X86
-static void test_single_cache(int iters, size_t size, int align, int flags,
-                              void (*ctor)(void *, size_t),
-                              void (*dtor)(void *, size_t))
-{
-       struct kmem_cache *test_cache;
-       void *objects[iters];
-       test_cache = kmem_cache_create("test_cache", size, align, flags,
-                                      NULL, ctor, dtor);
-       printk("Testing Kmem Cache:\n");
-       print_kmem_cache(test_cache);
-       for (int i = 0; i < iters; i++) {
-               objects[i] = kmem_cache_alloc(test_cache, 0);
-               printk("Buffer %d addr = %p\n", i, objects[i]);
-       }
-       for (int i = 0; i < iters; i++) {
-               kmem_cache_free(test_cache, objects[i]);
-       }
-       kmem_cache_destroy(test_cache);
-       printk("\n\n\n\n");
-}
-
-void a_ctor(void *buf, size_t size)
-{
-       printk("constructin tests\n");
-}
-void a_dtor(void *buf, size_t size)
-{
-       printk("destructin tests\n");
-}
-
-// TODO: Make test_single_cache return something, and then add assertions here.
-bool test_slab(void)
-{
-       test_single_cache(10, 128, 512, 0, 0, 0);
-       test_single_cache(10, 128, 4, 0, a_ctor, a_dtor);
-       test_single_cache(10, 1024, 16, 0, 0, 0);
-
-       return true;
-}
-
-// TODO: Add assertions.
-bool test_kmalloc(void)
-{
-       printk("Testing Kmalloc\n");
-       void *bufs[NUM_KMALLOC_CACHES + 1];
-       size_t size;
-       for (int i = 0; i < NUM_KMALLOC_CACHES + 1; i++){
-               size = (KMALLOC_SMALLEST << i) - sizeof(struct kmalloc_tag);
-               bufs[i] = kmalloc(size, 0);
-               printk("Size %d, Addr = %p\n", size, bufs[i]);
-       }
-       for (int i = 0; i < NUM_KMALLOC_CACHES; i++) {
-               printk("Freeing buffer %d\n", i);
-               kfree(bufs[i]);
-       }
-       printk("Testing a large kmalloc\n");
-       size = (KMALLOC_LARGEST << 2);
-       bufs[0] = kmalloc(size, 0);
-       printk("Size %d, Addr = %p\n", size, bufs[0]);
-       kfree(bufs[0]);
-
-       return true;
-}
 
 static size_t test_hash_fn_col(void *k)
 {
@@ -2270,8 +2207,6 @@ static struct ktest ktests[] = {
        KTEST_REG(bitmasks,           CONFIG_TEST_bitmasks),
        KTEST_REG(checklists,         CONFIG_TEST_checklists),
        KTEST_REG(smp_call_functions, CONFIG_TEST_smp_call_functions),
-       KTEST_REG(slab,               CONFIG_TEST_slab),
-       KTEST_REG(kmalloc,            CONFIG_TEST_kmalloc),
        KTEST_REG(hashtable,          CONFIG_TEST_hashtable),
        KTEST_REG(circular_buffer,    CONFIG_TEST_circular_buffer),
        KTEST_REG(bcq,                CONFIG_TEST_bcq),
index b38e4d8..b26b7c5 100644 (file)
@@ -752,20 +752,3 @@ void kmem_cache_reap(struct kmem_cache *cp)
        }
        spin_unlock_irqsave(&cp->cache_lock);
 }
-
-void print_kmem_cache(struct kmem_cache *cp)
-{
-       spin_lock_irqsave(&cp->cache_lock);
-       printk("\nPrinting kmem_cache:\n---------------------\n");
-       printk("Name: %s\n", cp->name);
-       printk("Objsize (incl align): %d\n", cp->obj_size);
-       printk("Align: %d\n", cp->align);
-       printk("Flags: 0x%08x\n", cp->flags);
-       printk("Constructor: %p\n", cp->ctor);
-       printk("Destructor: %p\n", cp->dtor);
-       printk("Slab Full: %p\n", cp->full_slab_list);
-       printk("Slab Partial: %p\n", cp->partial_slab_list);
-       printk("Slab Empty: %p\n", cp->empty_slab_list);
-       printk("Current Allocations: %d\n", cp->nr_cur_alloc);
-       spin_unlock_irqsave(&cp->cache_lock);
-}