diff options
Diffstat (limited to 'src/core/hle/kernel/kernel.cpp')
| -rw-r--r-- | src/core/hle/kernel/kernel.cpp | 23 |
1 files changed, 10 insertions, 13 deletions
diff --git a/src/core/hle/kernel/kernel.cpp b/src/core/hle/kernel/kernel.cpp index 7d40dcbee..d3684896f 100644 --- a/src/core/hle/kernel/kernel.cpp +++ b/src/core/hle/kernel/kernel.cpp | |||
| @@ -14,7 +14,7 @@ | |||
| 14 | 14 | ||
| 15 | namespace Kernel { | 15 | namespace Kernel { |
| 16 | 16 | ||
| 17 | Thread* g_main_thread = nullptr; | 17 | SharedPtr<Thread> g_main_thread = nullptr; |
| 18 | HandleTable g_handle_table; | 18 | HandleTable g_handle_table; |
| 19 | u64 g_program_id = 0; | 19 | u64 g_program_id = 0; |
| 20 | 20 | ||
| @@ -23,7 +23,7 @@ HandleTable::HandleTable() { | |||
| 23 | Clear(); | 23 | Clear(); |
| 24 | } | 24 | } |
| 25 | 25 | ||
| 26 | ResultVal<Handle> HandleTable::Create(Object* obj) { | 26 | ResultVal<Handle> HandleTable::Create(SharedPtr<Object> obj) { |
| 27 | _dbg_assert_(Kernel, obj != nullptr); | 27 | _dbg_assert_(Kernel, obj != nullptr); |
| 28 | 28 | ||
| 29 | u16 slot = next_free_slot; | 29 | u16 slot = next_free_slot; |
| @@ -39,23 +39,23 @@ ResultVal<Handle> HandleTable::Create(Object* obj) { | |||
| 39 | // CTR-OS doesn't use generation 0, so skip straight to 1. | 39 | // CTR-OS doesn't use generation 0, so skip straight to 1. |
| 40 | if (next_generation >= (1 << 15)) next_generation = 1; | 40 | if (next_generation >= (1 << 15)) next_generation = 1; |
| 41 | 41 | ||
| 42 | generations[slot] = generation; | ||
| 43 | intrusive_ptr_add_ref(obj); | ||
| 44 | objects[slot] = obj; | ||
| 45 | |||
| 46 | Handle handle = generation | (slot << 15); | 42 | Handle handle = generation | (slot << 15); |
| 47 | if (obj->handle == INVALID_HANDLE) | 43 | if (obj->handle == INVALID_HANDLE) |
| 48 | obj->handle = handle; | 44 | obj->handle = handle; |
| 45 | |||
| 46 | generations[slot] = generation; | ||
| 47 | objects[slot] = std::move(obj); | ||
| 48 | |||
| 49 | return MakeResult<Handle>(handle); | 49 | return MakeResult<Handle>(handle); |
| 50 | } | 50 | } |
| 51 | 51 | ||
| 52 | ResultVal<Handle> HandleTable::Duplicate(Handle handle) { | 52 | ResultVal<Handle> HandleTable::Duplicate(Handle handle) { |
| 53 | Object* object = GetGeneric(handle); | 53 | SharedPtr<Object> object = GetGeneric(handle); |
| 54 | if (object == nullptr) { | 54 | if (object == nullptr) { |
| 55 | LOG_ERROR(Kernel, "Tried to duplicate invalid handle: %08X", handle); | 55 | LOG_ERROR(Kernel, "Tried to duplicate invalid handle: %08X", handle); |
| 56 | return ERR_INVALID_HANDLE; | 56 | return ERR_INVALID_HANDLE; |
| 57 | } | 57 | } |
| 58 | return Create(object); | 58 | return Create(std::move(object)); |
| 59 | } | 59 | } |
| 60 | 60 | ||
| 61 | ResultCode HandleTable::Close(Handle handle) { | 61 | ResultCode HandleTable::Close(Handle handle) { |
| @@ -65,7 +65,6 @@ ResultCode HandleTable::Close(Handle handle) { | |||
| 65 | size_t slot = GetSlot(handle); | 65 | size_t slot = GetSlot(handle); |
| 66 | u16 generation = GetGeneration(handle); | 66 | u16 generation = GetGeneration(handle); |
| 67 | 67 | ||
| 68 | intrusive_ptr_release(objects[slot]); | ||
| 69 | objects[slot] = nullptr; | 68 | objects[slot] = nullptr; |
| 70 | 69 | ||
| 71 | generations[generation] = next_free_slot; | 70 | generations[generation] = next_free_slot; |
| @@ -80,7 +79,7 @@ bool HandleTable::IsValid(Handle handle) const { | |||
| 80 | return slot < MAX_COUNT && objects[slot] != nullptr && generations[slot] == generation; | 79 | return slot < MAX_COUNT && objects[slot] != nullptr && generations[slot] == generation; |
| 81 | } | 80 | } |
| 82 | 81 | ||
| 83 | Object* HandleTable::GetGeneric(Handle handle) const { | 82 | SharedPtr<Object> HandleTable::GetGeneric(Handle handle) const { |
| 84 | if (handle == CurrentThread) { | 83 | if (handle == CurrentThread) { |
| 85 | return GetCurrentThread(); | 84 | return GetCurrentThread(); |
| 86 | } else if (handle == CurrentProcess) { | 85 | } else if (handle == CurrentProcess) { |
| @@ -97,8 +96,6 @@ Object* HandleTable::GetGeneric(Handle handle) const { | |||
| 97 | void HandleTable::Clear() { | 96 | void HandleTable::Clear() { |
| 98 | for (size_t i = 0; i < MAX_COUNT; ++i) { | 97 | for (size_t i = 0; i < MAX_COUNT; ++i) { |
| 99 | generations[i] = i + 1; | 98 | generations[i] = i + 1; |
| 100 | if (objects[i] != nullptr) | ||
| 101 | intrusive_ptr_release(objects[i]); | ||
| 102 | objects[i] = nullptr; | 99 | objects[i] = nullptr; |
| 103 | } | 100 | } |
| 104 | next_free_slot = 0; | 101 | next_free_slot = 0; |
| @@ -126,7 +123,7 @@ bool LoadExec(u32 entry_point) { | |||
| 126 | Core::g_app_core->SetPC(entry_point); | 123 | Core::g_app_core->SetPC(entry_point); |
| 127 | 124 | ||
| 128 | // 0x30 is the typical main thread priority I've seen used so far | 125 | // 0x30 is the typical main thread priority I've seen used so far |
| 129 | g_main_thread = Kernel::SetupMainThread(0x30); | 126 | g_main_thread = Kernel::SetupMainThread(0x30, Kernel::DEFAULT_STACK_SIZE); |
| 130 | // Setup the idle thread | 127 | // Setup the idle thread |
| 131 | Kernel::SetupIdleThread(); | 128 | Kernel::SetupIdleThread(); |
| 132 | 129 | ||