Changeset 0aae87a6 in mainline

Timestamp:

2012-11-09T19:33:11Z (11 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

67b152e

Parents:

3b8a990

Message:

Unify the default stack size used by userspace threads and fibrils.

• The size is stored by the kernel into sysinfo.
• stack_size_get() reads this value from sysinfo and caches it locally.

Files:

• kernel/generic/src/main/kinit.c (modified) (1 diff)
• uspace/lib/c/Makefile (modified) (1 diff)
• uspace/lib/c/generic/fibril.c (modified) (3 diffs)
• uspace/lib/c/generic/stack.c (added)
• uspace/lib/c/generic/thread.c (modified) (2 diffs)
• uspace/lib/c/include/stack.h (added)

kernel/generic/src/main/kinit.c

@@ -172 +172 @@
 #endif /* CONFIG_KCONSOLE */
         
+        /*
+         * Store the default stack size in sysinfo so that uspace can create
+         * stack with this default size.
+         */
+        sysinfo_set_item_val("default.stack_size", NULL, STACK_SIZE);
+        
         interrupts_enable();
         

uspace/lib/c/Makefile

@@ -140 +140 @@
         generic/net/socket_client.c \
         generic/net/socket_parse.c \
+        generic/stack.c \
         generic/stacktrace.c \
         generic/arg_parse.c \

uspace/lib/c/generic/fibril.c

@@ -37 +37 @@
 #include <fibril.h>
 #include <thread.h>
+#include <stack.h>
 #include <tls.h>
 #include <malloc.h>
@@ -271 +272 @@
                 return 0;
         
-        fibril->stack = as_area_create((void *) -1,
-            FIBRIL_INITIAL_STACK_PAGES_NO * getpagesize(),
+        size_t stack_size = stack_size_get();
+        fibril->stack = as_area_create((void *) -1, stack_size,
             AS_AREA_READ | AS_AREA_WRITE | AS_AREA_CACHEABLE | AS_AREA_GUARD |
             AS_AREA_LATE_RESERVE);
@@ -285 +286 @@
         context_save(&fibril->ctx);
         context_set(&fibril->ctx, FADDR(fibril_main), fibril->stack,
-            FIBRIL_INITIAL_STACK_PAGES_NO * getpagesize(), fibril->tcb);
+            stack_size, fibril->tcb);
 
         return (fid_t) fibril;

uspace/lib/c/generic/thread.c

@@ -39 +39 @@
 #include <abi/proc/uarg.h>
 #include <fibril.h>
+#include <stack.h>
 #include <str.h>
 #include <async.h>
@@ -101 +102 @@
                 return ENOMEM;
         
-        size_t stack_size = getpagesize() * THREAD_INITIAL_STACK_PAGES;
+        size_t stack_size = stack_size_get();
         void *stack = as_area_create(AS_AREA_ANY, stack_size,
             AS_AREA_READ | AS_AREA_WRITE | AS_AREA_CACHEABLE | AS_AREA_GUARD |