summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorbunnei <bunneidev@gmail.com>2020-12-05 09:17:18 +0100
committerbunnei <bunneidev@gmail.com>2020-12-06 09:27:13 +0100
commit9b492430bb349b98ac4768ddde044a63976fe146 (patch)
treea8e135d90fafa99fb422de21563eb78767fc2ff3
parenthle: kernel: KScopedSchedulerLockAndSleep: Various style fixes based on code review feedback. (diff)
downloadyuzu-9b492430bb349b98ac4768ddde044a63976fe146.tar
yuzu-9b492430bb349b98ac4768ddde044a63976fe146.tar.gz
yuzu-9b492430bb349b98ac4768ddde044a63976fe146.tar.bz2
yuzu-9b492430bb349b98ac4768ddde044a63976fe146.tar.lz
yuzu-9b492430bb349b98ac4768ddde044a63976fe146.tar.xz
yuzu-9b492430bb349b98ac4768ddde044a63976fe146.tar.zst
yuzu-9b492430bb349b98ac4768ddde044a63976fe146.zip
-rw-r--r--src/core/hle/kernel/thread.h47
1 files changed, 25 insertions, 22 deletions
diff --git a/src/core/hle/kernel/thread.h b/src/core/hle/kernel/thread.h
index f1aa358a4..11ef29888 100644
--- a/src/core/hle/kernel/thread.h
+++ b/src/core/hle/kernel/thread.h
@@ -4,6 +4,7 @@
#pragma once
+#include <array>
#include <functional>
#include <string>
#include <utility>
@@ -346,10 +347,11 @@ public:
void SetStatus(ThreadStatus new_status);
- constexpr s64 GetLastScheduledTick() const {
+ s64 GetLastScheduledTick() const {
return this->last_scheduled_tick;
}
- constexpr void SetLastScheduledTick(s64 tick) {
+
+ void SetLastScheduledTick(s64 tick) {
this->last_scheduled_tick = tick;
}
@@ -481,7 +483,7 @@ public:
return ideal_core;
}
- constexpr const KAffinityMask& GetAffinityMask() const {
+ const KAffinityMask& GetAffinityMask() const {
return affinity_mask;
}
@@ -490,10 +492,11 @@ public:
/// Sleeps this thread for the given amount of nanoseconds.
ResultCode Sleep(s64 nanoseconds);
- constexpr s64 GetYieldScheduleCount() const {
+ s64 GetYieldScheduleCount() const {
return this->schedule_count;
}
- constexpr void SetYieldScheduleCount(s64 count) {
+
+ void SetYieldScheduleCount(s64 count) {
this->schedule_count = count;
}
@@ -570,14 +573,9 @@ public:
return has_exited;
}
- struct QueueEntry {
- private:
- Thread* prev;
- Thread* next;
-
+ class QueueEntry {
public:
- constexpr QueueEntry() : prev(nullptr), next(nullptr) { /* ... */
- }
+ constexpr QueueEntry() = default;
constexpr void Initialize() {
this->prev = nullptr;
@@ -590,18 +588,23 @@ public:
constexpr Thread* GetNext() const {
return this->next;
}
- constexpr void SetPrev(Thread* t) {
- this->prev = t;
+ constexpr void SetPrev(Thread* thread) {
+ this->prev = thread;
}
- constexpr void SetNext(Thread* t) {
- this->next = t;
+ constexpr void SetNext(Thread* thread) {
+ this->next = thread;
}
+
+ private:
+ Thread* prev{};
+ Thread* next{};
};
- constexpr QueueEntry& GetPriorityQueueEntry(s32 core) {
+ QueueEntry& GetPriorityQueueEntry(s32 core) {
return this->per_core_priority_queue_entry[core];
}
- constexpr const QueueEntry& GetPriorityQueueEntry(s32 core) const {
+
+ const QueueEntry& GetPriorityQueueEntry(s32 core) const {
return this->per_core_priority_queue_entry[core];
}
@@ -619,9 +622,6 @@ public:
disable_count--;
}
- ThreadStatus status = ThreadStatus::Dormant;
- u32 scheduling_state = 0;
-
private:
friend class GlobalSchedulerContext;
friend class KScheduler;
@@ -638,6 +638,9 @@ private:
ThreadContext64 context_64{};
std::shared_ptr<Common::Fiber> host_context{};
+ ThreadStatus status = ThreadStatus::Dormant;
+ u32 scheduling_state = 0;
+
u64 thread_id = 0;
VAddr entry_point = 0;
@@ -701,7 +704,7 @@ private:
KScheduler* scheduler = nullptr;
- QueueEntry per_core_priority_queue_entry[Core::Hardware::NUM_CPU_CORES]{};
+ std::array<QueueEntry, Core::Hardware::NUM_CPU_CORES> per_core_priority_queue_entry{};
u32 ideal_core{0xFFFFFFFF};
KAffinityMask affinity_mask{};