Changeset aae2869 in mainline for kernel/generic/src/time/clock.c

Timestamp:

2023-02-04T16:19:21Z (2 years ago)

Author:

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

Branches:

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

Children:

011c79a

Parents:

8addb24a

git-author:

Jiří Zárevúcky <zarevucky.jiri@…> (2022-08-16 11:03:29)

git-committer:

Jiří Zárevúcky <zarevucky.jiri@…> (2023-02-04 16:19:21)

Message:

Replace THREAD→ticks with CPU→preempt_deadline

This removes a bit of unnecessary locking in clock().

File:

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

kernel/generic/src/time/clock.c

@@ -184 +184 @@
 
         if (THREAD) {
-                uint64_t ticks;
-
                 irq_spinlock_lock(&CPU->lock, false);
                 CPU->needs_relink += 1 + missed_clock_ticks;
                 irq_spinlock_unlock(&CPU->lock, false);
 
-                irq_spinlock_lock(&THREAD->lock, false);
-                if ((ticks = THREAD->ticks)) {
-                        if (ticks >= 1 + missed_clock_ticks)
-                                THREAD->ticks -= 1 + missed_clock_ticks;
-                        else
-                                THREAD->ticks = 0;
-                }
-                irq_spinlock_unlock(&THREAD->lock, false);
-
-                if (ticks == 0 && PREEMPTION_ENABLED) {
+                if (current_clock_tick >= CPU->preempt_deadline && PREEMPTION_ENABLED) {
                         scheduler();
 #ifdef CONFIG_UDEBUG