Author: njn Date: 2008-02-10 22:36:55 +0000 (Sun, 10 Feb 2008) New Revision: 7391
Log: >From Bart van Assche: A long time ago pthreads-functions were instrumented in the drd-patch via modifications of coregrind/vg_preloaded.c. A.o. detached threads were recognized by intercepting pthread_create(). This intercept was active for all tools, so it made sense to add a unit test for it (none/tests/pth_detached.c). This intercept has been moved from coregrind/vg_preloaded.c to exp-drd/drd_preloaded.c, so this unit test does no longer make sense for nulgrind. This test may be removed. Removed: trunk/none/tests/pth_detached.c trunk/none/tests/pth_detached.stderr.exp trunk/none/tests/pth_detached.stdout.exp trunk/none/tests/pth_detached.vgtest Modified: trunk/none/tests/Makefile.am Modified: trunk/none/tests/Makefile.am =================================================================== --- trunk/none/tests/Makefile.am 2008-02-10 22:19:58 UTC (rev 7390) +++ trunk/none/tests/Makefile.am 2008-02-10 22:36:55 UTC (rev 7391) @@ -95,8 +95,6 @@ pth_rwlock.stderr.exp pth_rwlock.vgtest \ pth_stackalign.stderr.exp \ pth_stackalign.stdout.exp pth_stackalign.vgtest \ - pth_detached.stderr.exp \ - pth_detached.stdout.exp pth_detached.vgtest \ rcrl.stderr.exp rcrl.stdout.exp rcrl.vgtest \ readline1.stderr.exp readline1.stdout.exp \ readline1.vgtest \ @@ -148,7 +146,6 @@ pth_atfork1 pth_blockedsig pth_cancel1 pth_cancel2 pth_cvsimple \ pth_empty pth_exit pth_exit2 pth_mutexspeed pth_once pth_rwlock \ pth_stackalign \ - pth_detached \ rcrl readline1 res_search resolv \ rlimit_nofile selfrun sem semlimit sha1_test \ shortpush shorts stackgrowth sigstackgrowth susphello \ @@ -178,7 +175,6 @@ pth_once_LDADD = -lpthread pth_rwlock_LDADD = -lpthread pth_stackalign_LDADD = -lpthread -pth_detached_LDADD = -lpthread if VGO_AIX5 res_search_LDADD = -lpthread else Deleted: trunk/none/tests/pth_detached.c =================================================================== --- trunk/none/tests/pth_detached.c 2008-02-10 22:19:58 UTC (rev 7390) +++ trunk/none/tests/pth_detached.c 2008-02-10 22:36:55 UTC (rev 7391) @@ -1,90 +0,0 @@ -/* Test whether detached threads are handled properly. - Contributed by Bart Van Assche ([EMAIL PROTECTED]). -*/ - -#include <assert.h> -#include <pthread.h> -#include <stdio.h> -#include <stdlib.h> -#include <unistd.h> - -static int s_finished_count = 0; -static pthread_mutex_t s_mutex; - -void increment_finished_count() -{ - pthread_mutex_lock(&s_mutex); - s_finished_count++; - pthread_mutex_unlock(&s_mutex); -} - -int get_finished_count() -{ - int result; - pthread_mutex_lock(&s_mutex); - result = s_finished_count; - pthread_mutex_unlock(&s_mutex); - return result; -} - -static void* thread_func1(void* arg) -{ - write(STDOUT_FILENO, ".", 1); - increment_finished_count(); - return 0; -} - -static void* thread_func2(void* arg) -{ - pthread_detach(pthread_self()); - write(STDOUT_FILENO, ".", 1); - increment_finished_count(); - return 0; -} - -int main(int argc, char** argv) -{ - const int count1 = argc > 1 ? atoi(argv[1]) : 100; - const int count2 = argc > 2 ? atoi(argv[2]) : 100; - int i; - int detachstate; - pthread_attr_t attr; - - pthread_mutex_init(&s_mutex, 0); - - pthread_attr_init(&attr); - pthread_attr_setdetachstate(&attr, PTHREAD_CREATE_DETACHED); - assert(pthread_attr_getdetachstate(&attr, &detachstate) == 0); - assert(detachstate == PTHREAD_CREATE_DETACHED); - pthread_attr_setstacksize(&attr, 16384); - // Create count1 detached threads by setting the "detached" property via - // thread attributes. - for (i = 0; i < count1; i++) - { - pthread_t thread; - pthread_create(&thread, &attr, thread_func1, 0); - } - // Create count2 detached threads by letting the threads detach themselves. - pthread_attr_setdetachstate(&attr, PTHREAD_CREATE_JOINABLE); - assert(pthread_attr_getdetachstate(&attr, &detachstate) == 0); - assert(detachstate == PTHREAD_CREATE_JOINABLE); - for (i = 0; i < count2; i++) - { - pthread_t thread; - pthread_create(&thread, &attr, thread_func2, 0); - } - pthread_attr_destroy(&attr); - - // Wait until all detached threads have written their output to stdout. - while (get_finished_count() < count1 + count2) - { - struct timespec delay = { 0, 1 * 1000 * 1000 }; - nanosleep(&delay, 0); - } - - printf("\n"); - - pthread_mutex_destroy(&s_mutex); - - return 0; -} Deleted: trunk/none/tests/pth_detached.stderr.exp =================================================================== --- trunk/none/tests/pth_detached.stderr.exp 2008-02-10 22:19:58 UTC (rev 7390) +++ trunk/none/tests/pth_detached.stderr.exp 2008-02-10 22:36:55 UTC (rev 7391) @@ -1,2 +0,0 @@ - - Deleted: trunk/none/tests/pth_detached.stdout.exp =================================================================== --- trunk/none/tests/pth_detached.stdout.exp 2008-02-10 22:19:58 UTC (rev 7390) +++ trunk/none/tests/pth_detached.stdout.exp 2008-02-10 22:36:55 UTC (rev 7391) @@ -1 +0,0 @@ -........................................................................................................................................................................................................ Deleted: trunk/none/tests/pth_detached.vgtest =================================================================== --- trunk/none/tests/pth_detached.vgtest 2008-02-10 22:19:58 UTC (rev 7390) +++ trunk/none/tests/pth_detached.vgtest 2008-02-10 22:36:55 UTC (rev 7391) @@ -1 +0,0 @@ -prog: pth_detached ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ Valgrind-developers mailing list Valgrind-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/valgrind-developers