diff options
author | Simon Marlow <simonmarhaskell@gmail.com> | 2008-04-16 23:22:32 +0000 |
---|---|---|
committer | Simon Marlow <simonmarhaskell@gmail.com> | 2008-04-16 23:22:32 +0000 |
commit | f86e7206ea94b48b94fb61007a1c5d55b8c60f45 (patch) | |
tree | f3253ca0a19d51197b252c8a5003620dec42b94f /rts/Sparks.c | |
parent | ae267d04df855051b99218e3712b3f56b8016d56 (diff) | |
download | haskell-f86e7206ea94b48b94fb61007a1c5d55b8c60f45.tar.gz |
Reorganisation to fix problems related to the gct register variable
- GCAux.c contains code not compiled with the gct register enabled,
it is callable from outside the GC
- marking functions are moved to their relevant subsystems, outside
the GC
- mark_root needs to save the gct register, as it is called from
outside the GC
Diffstat (limited to 'rts/Sparks.c')
-rw-r--r-- | rts/Sparks.c | 69 |
1 files changed, 69 insertions, 0 deletions
diff --git a/rts/Sparks.c b/rts/Sparks.c index 9a843fab41..0f429e2c6c 100644 --- a/rts/Sparks.c +++ b/rts/Sparks.c @@ -162,6 +162,74 @@ newSpark (StgRegTable *reg, StgClosure *p) return 1; } +/* ----------------------------------------------------------------------------- + * Mark all nodes pointed to by sparks in the spark queues (for GC) Does an + * implicit slide i.e. after marking all sparks are at the beginning of the + * spark pool and the spark pool only contains sparkable closures + * -------------------------------------------------------------------------- */ + +void +markSparkQueue (evac_fn evac, void *user, Capability *cap) +{ + StgClosure **sparkp, **to_sparkp; + nat n, pruned_sparks; // stats only + StgSparkPool *pool; + + PAR_TICKY_MARK_SPARK_QUEUE_START(); + + n = 0; + pruned_sparks = 0; + + pool = &(cap->r.rSparks); + + ASSERT_SPARK_POOL_INVARIANTS(pool); + +#if defined(PARALLEL_HASKELL) + // stats only + n = 0; + pruned_sparks = 0; +#endif + + sparkp = pool->hd; + to_sparkp = pool->hd; + while (sparkp != pool->tl) { + ASSERT(*sparkp!=NULL); + ASSERT(LOOKS_LIKE_CLOSURE_PTR(((StgClosure *)*sparkp))); + // ToDo?: statistics gathering here (also for GUM!) + if (closure_SHOULD_SPARK(*sparkp)) { + evac(user, sparkp); + *to_sparkp++ = *sparkp; + if (to_sparkp == pool->lim) { + to_sparkp = pool->base; + } + n++; + } else { + pruned_sparks++; + } + sparkp++; + if (sparkp == pool->lim) { + sparkp = pool->base; + } + } + pool->tl = to_sparkp; + + PAR_TICKY_MARK_SPARK_QUEUE_END(n); + +#if defined(PARALLEL_HASKELL) + debugTrace(DEBUG_sched, + "marked %d sparks and pruned %d sparks on [%x]", + n, pruned_sparks, mytid); +#else + debugTrace(DEBUG_sched, + "marked %d sparks and pruned %d sparks", + n, pruned_sparks); +#endif + + debugTrace(DEBUG_sched, + "new spark queue len=%d; (hd=%p; tl=%p)\n", + sparkPoolSize(pool), pool->hd, pool->tl); +} + #else StgInt @@ -171,6 +239,7 @@ newSpark (StgRegTable *reg STG_UNUSED, StgClosure *p STG_UNUSED) return 1; } + #endif /* PARALLEL_HASKELL || THREADED_RTS */ |