X-Git-Url: https://git.ucc.asn.au/?a=blobdiff_plain;f=AcessNative%2Facesskernel_src%2Fthreads.c;h=ecc9f3315768f356dedb2444489e5c06c09d21e9;hb=9d234f984c149fcc6b1f5aab21a0ca227c060e32;hp=c390a031e9e72d8350fa85ca46f96134714f746a;hpb=b519c413b534ab831a0fcb6228b8a49d4a491279;p=tpg%2Facess2.git

diff --git a/AcessNative/acesskernel_src/threads.c b/AcessNative/acesskernel_src/threads.c
index c390a031..ecc9f331 100644
--- a/AcessNative/acesskernel_src/threads.c
+++ b/AcessNative/acesskernel_src/threads.c
@@ -5,28 +5,16 @@
  * threads.c
  * - Thread and process handling
  */
-#define _SIGNAL_H_	// Stop the acess signal.h being used
-#define _HEAP_H_	// Stop heap.h being imported (collides with stdlib heap)
-#define _VFS_EXT_H	// Stop vfs_ext.h being imported (collides with fd_set)
 
-#define off_t	_acess_off_t
 #include <arch.h>
-#undef NULL	// Remove acess definition
 #include <acess.h>
 #include <mutex.h>
-#include <semaphore.h>
+#include "../../KernelLand/Kernel/include/semaphore.h"
+#include <rwlock.h>
 #include <events.h>
 #include <threads_int.h>
-
-#undef CLONE_VM	// Such a hack
-#undef off_t	
-
-// - Native headers
-#include <unistd.h>
-#include <sys/types.h>
-#include <stdint.h>
-#include "/usr/include/signal.h"
-#include <SDL/SDL.h>
+#include <limits.h>
+#include "include/threads_glue.h"
 
 #define THREAD_EVENT_WAKEUP	0x80000000
 
@@ -34,14 +22,6 @@
 void	VFS_CloneHandleList(int PID);
 
 // === STRUCTURES ===
-#if 0
-typedef struct sState
-{
-	void	*CurState;
-	Uint	SP, BP, IP;
-}	tState;
-#endif
-
 // === PROTOTYPES ===
  int	Threads_Wake(tThread *Thread);
 
@@ -118,7 +98,7 @@ tThread *Threads_CloneTCB(tThread *TemplateThread)
 	ret->TID = giThreads_NextThreadID ++;
 	
 	ret->ThreadName = strdup(TemplateThread->ThreadName);
-	ret->EventSem = SDL_CreateSemaphore(0);
+	Threads_Glue_SemInit( &ret->EventSem, 0 );
 	
 	ret->WaitingThreads = NULL;
 	ret->WaitingThreadsEnd = NULL;
@@ -219,7 +199,7 @@ tTID Threads_WaitTID(int TID, int *Status)
 void Threads_Sleep(void)
 {
 	// TODO: Add to a sleeping queue
-	pause();
+	//pause();
 }
 
 void Threads_Yield(void)
@@ -240,7 +220,7 @@ void Threads_Exit(int TID, int Status)
 	{
 		// Wait for the thread to be waited upon
 		while( gpCurrentThread->WaitingThreads == NULL )
-			SDL_Delay(10);
+			Threads_Glue_Yield();
 	}
 	#endif
 	
@@ -251,7 +231,7 @@ void Threads_Exit(int TID, int Status)
 		Threads_Wake(toWake);
 		
 		while(gpCurrentThread->WaitingThreads == toWake)
-			SDL_Delay(10);
+			Threads_Glue_Yield();
 	}
 }
 
@@ -296,17 +276,13 @@ int Threads_Fork(void)
 // --------------------------------------------------------------------
 int Mutex_Acquire(tMutex *Mutex)
 {
-	if(!Mutex->Protector.IsValid) {
-		pthread_mutex_init( &Mutex->Protector.Mutex, NULL );
-		Mutex->Protector.IsValid = 1;
-	}
-	pthread_mutex_lock( &Mutex->Protector.Mutex );
+	Threads_Glue_AcquireMutex(&Mutex->Protector.Mutex);
 	return 0;
 }
 
 void Mutex_Release(tMutex *Mutex)
 {
-	pthread_mutex_unlock( &Mutex->Protector.Mutex );
+	Threads_Glue_ReleaseMutex(&Mutex->Protector.Mutex);
 }
 
 // --------------------------------------------------------------------
@@ -316,21 +292,17 @@ void Semaphore_Init(tSemaphore *Sem, int InitValue, int MaxValue, const char *Mo
 {
 	memset(Sem, 0, sizeof(tSemaphore));
 	// HACK: Use `Sem->Protector` as space for the semaphore pointer
-	*(void**)(&Sem->Protector) = SDL_CreateSemaphore(InitValue);
+	Threads_Glue_SemInit( &Sem->Protector.Mutex, InitValue );
 }
 
 int Semaphore_Wait(tSemaphore *Sem, int MaxToTake)
 {
-	SDL_SemWait( *(void**)(&Sem->Protector) );
-	return 1;
+	return Threads_Glue_SemWait( Sem->Protector.Mutex, MaxToTake );
 }
 
 int Semaphore_Signal(tSemaphore *Sem, int AmmountToAdd)
 {
-	 int	i;
-	for( i = 0; i < AmmountToAdd; i ++ )
-		SDL_SemPost( *(void**)(&Sem->Protector) );
-	return AmmountToAdd;
+	return Threads_Glue_SemSignal( Sem->Protector.Mutex, AmmountToAdd );
 }
 
 // --------------------------------------------------------------------
@@ -340,27 +312,33 @@ Uint32 Threads_WaitEvents(Uint32 Mask)
 {
 	Uint32	rv;
 
-	Log_Debug("Threads", "Mask = %x, ->Events = %x", Mask, gpCurrentThread->Events);	
+	//Log_Debug("Threads", "Mask = %x, ->Events = %x", Mask, gpCurrentThread->Events);	
 
 	gpCurrentThread->WaitMask = Mask;
 	if( !(gpCurrentThread->Events & Mask) )
 	{
-		SDL_SemWait( gpCurrentThread->EventSem );
+		if( Threads_Glue_SemWait(gpCurrentThread->EventSem, INT_MAX) == -1 ) {
+			Log_Warning("Threads", "Wait on eventsem of %p, %p failed",
+				gpCurrentThread, gpCurrentThread->EventSem);
+		}
+		//Log_Debug("Threads", "Woken from nap (%i here)", SDL_SemValue(gpCurrentThread->EventSem));
 	}
 	rv = gpCurrentThread->Events & Mask;
 	gpCurrentThread->Events &= ~Mask;
 	gpCurrentThread->WaitMask = -1;
-	
+
+	//Log_Debug("Threads", "- rv = %x", rv);
+
 	return rv;
 }
 
 void Threads_PostEvent(tThread *Thread, Uint32 Events)
 {
 	Thread->Events |= Events;
-	Log_Debug("Threads", "Trigger event %x (->Events = %p)", Events, Thread->Events);
+	Log_Debug("Threads", "Trigger event %x (->Events = %p) on %p", Events, Thread->Events, Thread);
 	
-	if( Thread->WaitMask & Events ) {
-		SDL_SemPost( Thread->EventSem );
+	if( Events == 0 || Thread->WaitMask & Events ) {
+		Threads_Glue_SemSignal( Thread->EventSem, 1 );
 //		Log_Debug("Threads", "Waking %p(%i %s)", Thread, Thread->TID, Thread->ThreadName);
 	}
 }