summaryrefslogtreecommitdiffstats
path: root/src/core/hle/kernel
diff options
context:
space:
mode:
authorbunnei <bunneidev@gmail.com>2015-01-16 01:29:46 +0100
committerbunnei <bunneidev@gmail.com>2015-01-22 00:43:50 +0100
commite4a5d8ad4f708c9674c9865eb872e3c081d9a8c8 (patch)
tree5604f380890c841fa850de11f84239c9cf7c7191 /src/core/hle/kernel
parentThread: Keep track of multiple wait objects. (diff)
downloadyuzu-e4a5d8ad4f708c9674c9865eb872e3c081d9a8c8.tar
yuzu-e4a5d8ad4f708c9674c9865eb872e3c081d9a8c8.tar.gz
yuzu-e4a5d8ad4f708c9674c9865eb872e3c081d9a8c8.tar.bz2
yuzu-e4a5d8ad4f708c9674c9865eb872e3c081d9a8c8.tar.lz
yuzu-e4a5d8ad4f708c9674c9865eb872e3c081d9a8c8.tar.xz
yuzu-e4a5d8ad4f708c9674c9865eb872e3c081d9a8c8.tar.zst
yuzu-e4a5d8ad4f708c9674c9865eb872e3c081d9a8c8.zip
Diffstat (limited to 'src/core/hle/kernel')
-rw-r--r--src/core/hle/kernel/event.cpp58
-rw-r--r--src/core/hle/kernel/event.h9
2 files changed, 13 insertions, 54 deletions
diff --git a/src/core/hle/kernel/event.cpp b/src/core/hle/kernel/event.cpp
index 9dd3d0f5d..540199e03 100644
--- a/src/core/hle/kernel/event.cpp
+++ b/src/core/hle/kernel/event.cpp
@@ -25,66 +25,36 @@ public:
ResetType intitial_reset_type; ///< ResetType specified at Event initialization
ResetType reset_type; ///< Current ResetType
- bool locked; ///< Event signal wait
+ bool signaled; ///< Whether the event has already been signaled
std::string name; ///< Name of event (optional)
ResultVal<bool> WaitSynchronization() override {
- bool wait = locked;
- if (locked) {
+ bool wait = !signaled;
+ if (wait) {
AddWaitingThread(GetCurrentThread());
Kernel::WaitCurrentThread(WAITTYPE_EVENT, this);
}
- if (reset_type != RESETTYPE_STICKY) {
- locked = true;
- }
return MakeResult<bool>(wait);
}
};
-/**
- * Changes whether an event is locked or not
- * @param handle Handle to event to change
- * @param locked Boolean locked value to set event
- * @return Result of operation, 0 on success, otherwise error code
- */
-ResultCode SetEventLocked(const Handle handle, const bool locked) {
- Event* evt = g_handle_table.Get<Event>(handle).get();
- if (evt == nullptr) return InvalidHandle(ErrorModule::Kernel);
-
- evt->locked = locked;
-
- return RESULT_SUCCESS;
-}
-
-/**
- * Signals an event
- * @param handle Handle to event to signal
- * @return Result of operation, 0 on success, otherwise error code
- */
ResultCode SignalEvent(const Handle handle) {
Event* evt = g_handle_table.Get<Event>(handle).get();
- if (evt == nullptr) return InvalidHandle(ErrorModule::Kernel);
+ if (evt == nullptr)
+ return InvalidHandle(ErrorModule::Kernel);
- // If any thread is signalled awake by this event, assume the event was "caught" and reset
- // the event. This will result in the next thread waiting on the event to block. Otherwise,
- // the event will not be reset, and the next thread to call WaitSynchronization on it will
- // not block. Not sure if this is correct behavior, but it seems to work.
- // TODO(bunnei): Test how this works on hardware
- evt->locked = evt->ResumeAllWaitingThreads();
+ evt->signaled = true;
+ evt->ReleaseAllWaitingThreads();
return RESULT_SUCCESS;
}
-/**
- * Clears an event
- * @param handle Handle to event to clear
- * @return Result of operation, 0 on success, otherwise error code
- */
ResultCode ClearEvent(Handle handle) {
Event* evt = g_handle_table.Get<Event>(handle).get();
- if (evt == nullptr) return InvalidHandle(ErrorModule::Kernel);
+ if (evt == nullptr)
+ return InvalidHandle(ErrorModule::Kernel);
- evt->locked = true;
+ evt->signaled = false;
return RESULT_SUCCESS;
}
@@ -102,19 +72,13 @@ Event* CreateEvent(Handle& handle, const ResetType reset_type, const std::string
// TOOD(yuriks): Fix error reporting
handle = Kernel::g_handle_table.Create(evt).ValueOr(INVALID_HANDLE);
- evt->locked = true;
+ evt->signaled = false;
evt->reset_type = evt->intitial_reset_type = reset_type;
evt->name = name;
return evt;
}
-/**
- * Creates an event
- * @param reset_type ResetType describing how to create event
- * @param name Optional name of event
- * @return Handle to newly created Event object
- */
Handle CreateEvent(const ResetType reset_type, const std::string& name) {
Handle handle;
Event* evt = CreateEvent(handle, reset_type, name);
diff --git a/src/core/hle/kernel/event.h b/src/core/hle/kernel/event.h
index b1b9d4b7b..c08b12ee1 100644
--- a/src/core/hle/kernel/event.h
+++ b/src/core/hle/kernel/event.h
@@ -12,21 +12,16 @@
namespace Kernel {
/**
- * Changes whether an event is locked or not
- * @param handle Handle to event to change
- * @param locked Boolean locked value to set event
- */
-ResultCode SetEventLocked(const Handle handle, const bool locked);
-
-/**
* Signals an event
* @param handle Handle to event to signal
+ * @return Result of operation, 0 on success, otherwise error code
*/
ResultCode SignalEvent(const Handle handle);
/**
* Clears an event
* @param handle Handle to event to clear
+ * @return Result of operation, 0 on success, otherwise error code
*/
ResultCode ClearEvent(Handle handle);