Changeset 128359eb in mainline for kernel/generic/src/cpu/cpu.c

Timestamp:

2020-06-12T16:46:32Z (4 years ago)

Author:

Martin Decky <martin@…>

Branches:

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

Children:

ffccdff0

Parents:

94e75cf

Message:

Replace get_stack_base() with builtin_frame_address(0)

The usage of an intrinsic function to obtain the current stack pointer
should provide the compuler more room for performance optimizations than
the hand-written (and volatile) inline assembly block.

File:

• kernel/generic/src/cpu/cpu.c (modified) (1 diff)

kernel/generic/src/cpu/cpu.c

@@ -71 +71 @@
                 memsetb(cpus, sizeof(cpu_t) * config.cpu_count, 0);
 
-                // NOTE: All kernel stacks must be aligned to STACK_SIZE,
-                //       see get_stack_base().
-                size_t i;
-                for (i = 0; i < config.cpu_count; i++) {
+                /*
+                 * NOTE: All kernel stacks must be aligned to STACK_SIZE,
+                 *       see CURRENT.
+                 */
+                for (size_t i = 0; i < config.cpu_count; i++) {
                         uintptr_t stack_phys = frame_alloc(STACK_FRAMES,
                             FRAME_LOWMEM | FRAME_ATOMIC, STACK_SIZE - 1);