From 66fa48d924d6898c97eb29a1165bbbc0f08c6ab4 Mon Sep 17 00:00:00 2001
From: Justus Winter <4winter@informatik.uni-hamburg.de>
Date: Thu, 2 Jul 2015 16:20:44 +0200
Subject: kern: allocate kernel stacks using the slab allocator

* kern/slab.c (kmem_cache_init): Relax alignment restriction.
* kern/thread.c (stack_cache): New variable.
(stack_alloc): Use the slab allocator.
(stack_collect): Adjust accordingly.
(thread_init): Initialize `stack_cache'.
---
 kern/thread.c | 29 +++++++++++++++++++----------
 1 file changed, 19 insertions(+), 10 deletions(-)

(limited to 'kern/thread.c')

diff --git a/kern/thread.c b/kern/thread.c
index b996559..3586e34 100644
--- a/kern/thread.c
+++ b/kern/thread.c
@@ -164,6 +164,11 @@ boolean_t stack_alloc_try(
 	}
 }
 
+/*
+ * We allocate kernel stacks using the slab allocator.
+ */
+static struct kmem_cache stack_cache;
+
 /*
  *	stack_alloc:
  *
@@ -195,15 +200,10 @@ kern_return_t stack_alloc(
 	(void) splx(s);
 
 	if (stack == 0) {
-		kern_return_t kr;
-		/*
-		 *	Kernel stacks should be naturally aligned,
-		 *	so that it is easy to find the starting/ending
-		 *	addresses of a stack given an address in the middle.
-		 */
-		kr = kmem_alloc_aligned(kmem_map, &stack, KERNEL_STACK_SIZE);
-		if (kr != KERN_SUCCESS)
-			return kr;
+		stack = kmem_cache_alloc(&stack_cache);
+		assert ((stack & (KERNEL_STACK_SIZE-1)) == 0);
+		if (stack == 0)
+			return KERN_RESOURCE_SHORTAGE;
 
 #if	MACH_DEBUG
 		stack_init(stack);
@@ -265,7 +265,7 @@ void stack_collect(void)
 #if	MACH_DEBUG
 		stack_finalize(stack);
 #endif	/* MACH_DEBUG */
-		kmem_free(kmem_map, stack, KERNEL_STACK_SIZE);
+		kmem_cache_free(&stack_cache, stack);
 
 		s = splsched();
 		stack_lock();
@@ -300,6 +300,15 @@ void thread_init(void)
 	kmem_cache_init(&thread_cache, "thread", sizeof(struct thread), 0,
 			NULL, NULL, NULL, 0);
 
+	/*
+	 *	Kernel stacks should be naturally aligned, so that it
+	 *	is easy to find the starting/ending addresses of a
+	 *	stack given an address in the middle.
+	 */
+	kmem_cache_init(&stack_cache, "stack",
+			KERNEL_STACK_SIZE, KERNEL_STACK_SIZE,
+			NULL, NULL, NULL, 0);
+
 	/*
 	 *	Fill in a template thread for fast initialization.
 	 *	[Fields that must be (or are typically) reset at
-- 
cgit v1.2.3