X-Git-Url: http://git.megacz.com/?a=blobdiff_plain;f=ghc%2Frts%2FWeak.c;h=f216f62561d8b4505485ce69b3f1df36c02b5ccb;hb=8115ed091349a17e65f409a70b55514b8d09318d;hp=ea52d906822bf0a4a35e19f70b920c56e3ff3994;hpb=e151ce30350f2775377a5566fa4583726b46cdbf;p=ghc-hetmet.git diff --git a/ghc/rts/Weak.c b/ghc/rts/Weak.c index ea52d90..f216f62 100644 --- a/ghc/rts/Weak.c +++ b/ghc/rts/Weak.c @@ -1,5 +1,5 @@ /* ----------------------------------------------------------------------------- - * $Id: Weak.c,v 1.13 2000/02/25 17:35:11 sewardj Exp $ + * $Id: Weak.c,v 1.27 2003/01/25 15:54:50 wolfgang Exp $ * * (c) The GHC Team, 1998-1999 * @@ -7,12 +7,15 @@ * * ---------------------------------------------------------------------------*/ +#include "PosixSource.h" +#define COMPILING_RTS_MAIN #include "Rts.h" -#include "RtsAPI.h" #include "SchedAPI.h" #include "RtsFlags.h" #include "Weak.h" #include "Storage.h" +#include "Prelude.h" +#include "RtsAPI.h" StgWeak *weak_ptr_list; @@ -31,20 +34,34 @@ finalizeWeakPointersNow(void) { StgWeak *w; +#if defined(RTS_SUPPORTS_THREADS) + rts_lock(); +#endif while ((w = weak_ptr_list)) { weak_ptr_list = w->link; - IF_DEBUG(weak,fprintf(stderr,"Finalising weak pointer at %p -> %p\n", w, w->key)); - w->header.info = &DEAD_WEAK_info; - if (w->finalizer != &NO_FINALIZER_closure) { - rts_evalIO(w->finalizer,NULL); + if (w->header.info != &stg_DEAD_WEAK_info) { + w->header.info = &stg_DEAD_WEAK_info; + IF_DEBUG(weak,fprintf(stderr,"Finalising weak pointer at %p -> %p\n", w, w->key)); + if (w->finalizer != &stg_NO_FINALIZER_closure) { +#if defined(RTS_SUPPORTS_THREADS) + rts_evalIO(w->finalizer,NULL); + rts_unlock(); + rts_lock(); +#else + rts_mainEvalIO(w->finalizer,NULL); +#endif + } } } +#if defined(RTS_SUPPORTS_THREADS) + rts_unlock(); +#endif } /* * scheduleFinalizers() is called on the list of weak pointers found * to be dead after a garbage collection. It overwrites each object - * with DEAD_WEAK, and creates a new thread for the finalizer. + * with DEAD_WEAK, and creates a new thread to run the pending finalizers. * * This function is called just after GC. The weak pointers on the * argument list are those whose keys were found to be not reachable, @@ -52,20 +69,66 @@ finalizeWeakPointersNow(void) * The weak pointer object itself may not be alive - i.e. we may be * looking at either an object in from-space or one in to-space. It * doesn't really matter either way. + * + * Pre-condition: sched_mutex _not_ held. */ void scheduleFinalizers(StgWeak *list) { - StgWeak *w; - StgTSO *t; - - for (w = list; w; w = w->link) { - IF_DEBUG(weak,fprintf(stderr,"Finalising weak pointer at %p -> %p\n", w, w->key)); - if (w->finalizer != &NO_FINALIZER_closure) { - t = createIOThread(RtsFlags.GcFlags.initialStkSize, w->finalizer); - scheduleThread(t); + StgWeak *w; + StgTSO *t; + StgMutArrPtrs *arr; + nat n; + + // count number of finalizers, and kill all the weak pointers first... + n = 0; + for (w = list; w; w = w->link) { + + // Better not be a DEAD_WEAK at this stage; the garbage + // collector removes DEAD_WEAKs from the weak pointer list. + ASSERT(w->header.info != &stg_DEAD_WEAK_info); + + if (w->finalizer != &stg_NO_FINALIZER_closure) { + n++; + } + +#ifdef PROFILING + // A weak pointer is inherently used, so we do not need to call + // LDV_recordDead(). + // + // Furthermore, when PROFILING is turned on, dead weak + // pointers are exactly as large as weak pointers, so there is + // no need to fill the slop, either. See stg_DEAD_WEAK_info + // in StgMiscClosures.hc. +#endif + SET_HDR(w, &stg_DEAD_WEAK_info, w->header.prof.ccs); } - w->header.info = &DEAD_WEAK_info; - } + + // No finalizers to run? + if (n == 0) return; + + IF_DEBUG(weak,fprintf(stderr,"weak: batching %d finalizers\n", n)); + + arr = (StgMutArrPtrs *)allocate(sizeofW(StgMutArrPtrs) + n); + TICK_ALLOC_PRIM(sizeofW(StgMutArrPtrs), n, 0); + SET_HDR(arr, &stg_MUT_ARR_PTRS_FROZEN_info, CCS_SYSTEM); + arr->ptrs = n; + + n = 0; + for (w = list; w; w = w->link) { + if (w->finalizer != &stg_NO_FINALIZER_closure) { + arr->payload[n] = w->finalizer; + n++; + } + } + + t = createIOThread(RtsFlags.GcFlags.initialStkSize, + rts_apply( + rts_apply( + (StgClosure *)runFinalizerBatch_closure, + rts_mkInt(n)), + (StgClosure *)arr) + ); + scheduleThread(t); }