Changeset 0366d09d in mainline for kernel/generic/src/proc/scheduler.c

Timestamp:

2023-02-07T16:49:43Z (15 months ago)

Author:

Jiří Zárevúcky <zarevucky.jiri@…>

Branches:

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

Children:

78acbc72

Parents:

1eaead4

git-author:

Jiří Zárevúcky <zarevucky.jiri@…> (2022-08-14 14:08:42)

git-committer:

Jiří Zárevúcky <zarevucky.jiri@…> (2023-02-07 16:49:43)

Message:

Avoid separate allocation for FPU context

With _Alignas, we can encode alignment in the type itself and
include it directly as a member of thread_t, thus removing
the need for a separate allocation.

The alignment requirement is never more than 16 bytes,
so this adds only minimal extra padding to the structure.

File:

• kernel/generic/src/proc/scheduler.c (modified) (4 diffs)

kernel/generic/src/proc/scheduler.c

@@ -97 +97 @@
         fpu_enable();
         if (THREAD->fpu_context_exists)
-                fpu_context_restore(THREAD->saved_fpu_context);
+                fpu_context_restore(&THREAD->fpu_context);
         else {
                 fpu_init();
@@ -140 +140 @@
         if (CPU->fpu_owner != NULL) {
                 irq_spinlock_lock(&CPU->fpu_owner->lock, false);
-                fpu_context_save(CPU->fpu_owner->saved_fpu_context);
+                fpu_context_save(&CPU->fpu_owner->fpu_context);
 
                 /* Don't prevent migration */
@@ -150 +150 @@
         irq_spinlock_lock(&THREAD->lock, false);
         if (THREAD->fpu_context_exists) {
-                fpu_context_restore(THREAD->saved_fpu_context);
+                fpu_context_restore(&THREAD->fpu_context);
         } else {
                 fpu_init();
@@ -325 +325 @@
 
 #if (defined CONFIG_FPU) && (!defined CONFIG_FPU_LAZY)
-                fpu_context_save(THREAD->saved_fpu_context);
+                fpu_context_save(&THREAD->fpu_context);
 #endif
                 if (!context_save(&THREAD->saved_context)) {