X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=ext%2Fthreads%2Fthreads.xs;h=5caedbe65e8fc553dccd38fe7d289c27566873d8;hb=fd58862f51cd3f5fdbd6e93549721f19252499bc;hp=82786e2c974381bfbef226d368128e68c15bd894;hpb=e6e315b903395964ca2f901936c1bbae16d42f5e;p=p5sagit%2Fp5-mst-13.2.git diff --git a/ext/threads/threads.xs b/ext/threads/threads.xs index 82786e2..5caedbe 100755 --- a/ext/threads/threads.xs +++ b/ext/threads/threads.xs @@ -1,21 +1,15 @@ - #include "threads.h" - - - - - /* - Starts executing the thread. Needs to clean up memory a tad better. -*/ + * Starts executing the thread. Needs to clean up memory a tad better. + */ #ifdef WIN32 THREAD_RET_TYPE Perl_thread_run(LPVOID arg) { - ithread* thread = (ithread*) arg; #else -void Perl_thread_run(ithread* thread) { +void* Perl_thread_run(void * arg) { #endif + ithread* thread = (ithread*) arg; SV* thread_tid_ptr; SV* thread_ptr; dTHXa(thread->interp); @@ -29,8 +23,8 @@ void Perl_thread_run(ithread* thread) { SHAREDSvLOCK(threads); SHAREDSvEDIT(threads); - thread_tid_ptr = Perl_newSViv(PL_sharedsv_space, (IV) thread->thr); - thread_ptr = Perl_newSViv(PL_sharedsv_space, (IV) thread); + thread_tid_ptr = Perl_newSVuv(PL_sharedsv_space, PTR2UV(thread->thr)); + thread_ptr = Perl_newSVuv(PL_sharedsv_space, PTR2UV(thread)); hv_store_ent((HV*)SHAREDSvGET(threads), thread_tid_ptr, thread_ptr,0); SvREFCNT_dec(thread_tid_ptr); SHAREDSvRELEASE(threads); @@ -61,9 +55,8 @@ void Perl_thread_run(ithread* thread) { } - - MUTEX_LOCK(&thread->mutex); + PerlIO_flush((PerlIO*)NULL); perl_destruct(thread->interp); perl_free(thread->interp); if(thread->detached == 1) { @@ -74,70 +67,66 @@ void Perl_thread_run(ithread* thread) { } #ifdef WIN32 return (DWORD)0; +#else + return 0; #endif } - - /* - iThread->create(); -*/ + * iThread->create(); + */ SV* Perl_thread_create(char* class, SV* init_function, SV* params) { ithread* thread = malloc(sizeof(ithread)); SV* obj_ref; SV* obj; SV* temp_store; - I32 result; PerlInterpreter *current_perl; MUTEX_LOCK(&create_mutex); obj_ref = newSViv(0); obj = newSVrv(obj_ref, class); - sv_setiv(obj, (IV)thread); - SvREADONLY_on(obj); + sv_setiv(obj, (IV)thread); + SvREADONLY_on(obj); - - current_perl = PERL_GET_CONTEXT; + current_perl = PERL_GET_CONTEXT; /* - here we put the values of params and function to call onto namespace, this is so perl will properly clone them when we call perl_clone. - */ - - + * here we put the values of params and function to call onto + * namespace, this is so perl will properly clone them when we + * call perl_clone. + */ - temp_store = Perl_get_sv(current_perl, "threads::paramtempstore", TRUE | GV_ADDMULTI); + temp_store = Perl_get_sv(current_perl, "threads::paramtempstore", + TRUE | GV_ADDMULTI); Perl_sv_setsv(current_perl, temp_store,params); params = NULL; temp_store = NULL; - temp_store = Perl_get_sv(current_perl, "threads::calltempstore", TRUE | GV_ADDMULTI); + temp_store = Perl_get_sv(current_perl, "threads::calltempstore", + TRUE | GV_ADDMULTI); Perl_sv_setsv(current_perl,temp_store, init_function); init_function = NULL; temp_store = NULL; - #ifdef WIN32 - thread->interp = perl_clone(current_perl,4); + thread->interp = perl_clone(current_perl, 4); #else - thread->interp = perl_clone(current_perl,0); + thread->interp = perl_clone(current_perl, 0); #endif - thread->init_function = newSVsv(Perl_get_sv(thread->interp, "threads::calltempstore",FALSE)); - thread->params = newSVsv(Perl_get_sv(thread->interp, "threads::paramtempstore",FALSE)); - - - - + thread->init_function = newSVsv(Perl_get_sv(thread->interp, + "threads::calltempstore",FALSE)); + thread->params = newSVsv(Perl_get_sv(thread->interp, + "threads::paramtempstore",FALSE)); /* - And here we make sure we clean up the data we put in the namespace of iThread, both in the new and the calling inteprreter - */ - - + * And here we make sure we clean up the data we put in the + * namespace of iThread, both in the new and the calling + * inteprreter */ - temp_store = Perl_get_sv(thread->interp,"threads::paramtempstore",FALSE); + temp_store = Perl_get_sv(thread->interp, "threads::paramtempstore",FALSE); Perl_sv_setsv(thread->interp,temp_store, &PL_sv_undef); temp_store = Perl_get_sv(thread->interp,"threads::calltempstore",FALSE); @@ -151,11 +140,7 @@ SV* Perl_thread_create(char* class, SV* init_function, SV* params) { temp_store = Perl_get_sv(current_perl,"threads::calltempstore",FALSE); Perl_sv_setsv(current_perl, temp_store, &PL_sv_undef); - /* lets init the thread */ - - - - + /* let's init the thread */ MUTEX_INIT(&thread->mutex); thread->tid = tid_counter++; @@ -168,18 +153,16 @@ SV* Perl_thread_create(char* class, SV* init_function, SV* params) { (LPVOID)thread, 0, &thread->thr); #else - pthread_create( &thread->thr, NULL, (void *) Perl_thread_run, thread); + pthread_create( &thread->thr, (pthread_attr_t*)NULL, Perl_thread_run, thread); #endif MUTEX_UNLOCK(&create_mutex); - - - return obj_ref; + return obj_ref; } /* - returns the id of the thread -*/ + * returns the id of the thread + */ I32 Perl_thread_tid (SV* obj) { ithread* thread; if(!SvROK(obj)) { @@ -196,27 +179,26 @@ SV* Perl_thread_self (char* class) { dTHX; SV* obj_ref; SV* obj; - SV* thread_tid_ptr; - SV* thread_ptr; - HE* thread_entry; - PerlInterpreter *old_context = PERL_GET_CONTEXT; - - + SV* thread_tid_ptr; + SV* thread_ptr; + HE* thread_entry; SHAREDSvLOCK(threads); SHAREDSvEDIT(threads); #ifdef WIN32 - thread_tid_ptr = Perl_newSViv(PL_sharedsv_space, (IV) GetCurrentThreadId()); + thread_tid_ptr = Perl_newSVuv(PL_sharedsv_space, + (UV) GetCurrentThreadId()); #else - thread_tid_ptr = Perl_newSViv(PL_sharedsv_space, (IV) pthread_self()); + thread_tid_ptr = Perl_newSVuv(PL_sharedsv_space, + PTR2UV(pthread_self())); #endif - thread_entry = Perl_hv_fetch_ent(PL_sharedsv_space,(HV*) SHAREDSvGET(threads), thread_tid_ptr, 0,0); + thread_entry = Perl_hv_fetch_ent(PL_sharedsv_space, + (HV*) SHAREDSvGET(threads), + thread_tid_ptr, 0,0); thread_ptr = HeVAL(thread_entry); SvREFCNT_dec(thread_tid_ptr); SHAREDSvRELEASE(threads); SHAREDSvUNLOCK(threads); - - obj_ref = newSViv(0); obj = newSVrv(obj_ref, class); @@ -226,9 +208,8 @@ SV* Perl_thread_self (char* class) { } /* - joins the thread - this code needs to take the returnvalue from the call_sv and send it back -*/ + * joins the thread this code needs to take the returnvalue from the + * call_sv and send it back */ void Perl_thread_join(SV* obj) { ithread* thread = (ithread*)SvIV(SvRV(obj)); @@ -241,11 +222,8 @@ void Perl_thread_join(SV* obj) { #endif } - -/* - detaches a thread - needs to better clean up memory -*/ +/* detaches a thread + * needs to better clean up memory */ void Perl_thread_detach(SV* obj) { ithread* thread = (ithread*)SvIV(SvRV(obj)); @@ -257,8 +235,6 @@ void Perl_thread_detach(SV* obj) { MUTEX_UNLOCK(&thread->mutex); } - - void Perl_thread_DESTROY (SV* obj) { ithread* thread = (ithread*)SvIV(SvRV(obj)); @@ -266,7 +242,6 @@ void Perl_thread_DESTROY (SV* obj) { thread->count--; MUTEX_UNLOCK(&thread->mutex); Perl_thread_destruct(thread); - } void Perl_thread_destruct (ithread* thread) { @@ -281,14 +256,13 @@ void Perl_thread_destruct (ithread* thread) { /*printf("proper destruction!\n");*/ } - MODULE = threads PACKAGE = threads BOOT: Perl_sharedsv_init(aTHX); PL_perl_destruct_level = 2; threads = Perl_sharedsv_new(aTHX); SHAREDSvEDIT(threads); - ((HV*) SHAREDSvGET(threads)) = newHV(); + SHAREDSvGET(threads) = (SV *)newHV(); SHAREDSvRELEASE(threads); { @@ -308,17 +282,15 @@ BOOT: #else thread->thr = pthread_self(); #endif - thread_tid_ptr = Perl_newSViv(PL_sharedsv_space, (IV) thread->thr); - thread_ptr = Perl_newSViv(PL_sharedsv_space, (IV) thread); SHAREDSvEDIT(threads); + thread_tid_ptr = Perl_newSVuv(PL_sharedsv_space, PTR2UV(thread->thr)); + thread_ptr = Perl_newSVuv(PL_sharedsv_space, PTR2UV(thread)); hv_store_ent((HV*) SHAREDSvGET(threads), thread_tid_ptr, thread_ptr,0); - SHAREDSvRELEASE(threads); SvREFCNT_dec(thread_tid_ptr); + SHAREDSvRELEASE(threads); } MUTEX_INIT(&create_mutex); - - PROTOTYPES: DISABLE SV * @@ -385,10 +357,6 @@ detach (obj) /* must have used dXSARGS; list context implied */ return; /* assume stack size is correct */ - - - - void DESTROY (obj) SV * obj @@ -405,5 +373,3 @@ DESTROY (obj) /* must have used dXSARGS; list context implied */ return; /* assume stack size is correct */ - -