summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| | | * testsuite: Skip T15892 in nonmoving_thr_ghcBen Gamari2019-10-221-1/+1
| | | |
| | | * testsuite: bug1010 requires -c, which isn't supported by nonmovingBen Gamari2019-10-221-1/+4
| | | |
| | | * testsuite: Ensure that threaded tests are run in nonmoving_thrBen Gamari2019-10-222-9/+13
| | | |
| | | * testsuite: Nonmoving collector doesn't support -G1Ben Gamari2019-10-221-1/+2
| | | |
| | | * testsuite: Don't run T15892 in nonmoving waysBen Gamari2019-10-221-3/+5
| | | | | | | | | | | | | | | | The nonmoving GC doesn't support `+RTS -G1`, which this test insists on.
| | | * testsuite: Add nonmoving_thr_ghc wayBen Gamari2019-10-221-1/+4
| | | | | | | | | | | | | | | | This uses the nonmoving collector when compiling the testcases.
| | | * testsuite: Add nonmoving_thr wayBen Gamari2019-10-221-3/+6
| | | |
| | * | Unconditionally flush update remembered set during minor GCwip/gc/optimizeBen Gamari2019-10-222-0/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Flush the update remembered set. The goal here is to flush periodically to ensure that we don't end up with a thread who marks their stack on their local update remembered set and doesn't flush until the nonmoving sync period as this would result in a large fraction of the heap being marked during the sync pause.
| | * | NonMoving: Clean mut_listBen Gamari2019-10-221-1/+121
| | | |
| | * | NonMoving: Don't do major GC if one is already runningBen Gamari2019-10-222-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously we would perform a preparatory moving collection, resulting in many things being added to the mark queue. When we finished with this we would realize in nonmovingCollect that there was already a collection running, in which case we would simply not run the nonmoving collector. However, it was very easy to end up in a "treadmilling" situation: all subsequent GC following the first failed major GC would be scheduled as major GCs. Consequently we would continuously feed the concurrent collector with more mark queue entries and it would never finish. This patch aborts the major collection far earlier, meaning that we avoid adding nonmoving objects to the mark queue and allowing the concurrent collector to finish.
| | * | NonMovingMark: Eliminate redundant check_in_nonmoving_heapsBen Gamari2019-10-221-16/+15
| | | |
| | * | NonMoving: Optimise allocator cache behaviorBen Gamari2019-10-222-24/+42
| | | | | | | | | | | | | | | | | | | | Previously we would look at the segment header to determine the block size despite the fact that we already had the block size at hand.
| | * | NonMoving: Prefetch segment headerBen Gamari2019-10-222-2/+8
| | | |
| | * | NonMoving: Pre-fetch during markBen Gamari2019-10-222-1/+55
| | | | | | | | | | | | | | | | This improved overall runtime on nofib's constraints test by nearly 10%.
| | * | NonMoving: Fuse sweep preparation into mark prepBen Gamari2019-10-223-45/+25
| | | |
| | * | NonMoving: Inline nonmovingClearAllBitmapsBen Gamari2019-10-221-25/+10
| | | |
| | * | NonMoving: Prefetch when clearing bitmapsBen Gamari2019-10-221-0/+2
| | | | | | | | | | | | | | | | | | | | Ensure that the bitmap of the segmentt that we will clear next is in cache by the time we reach it.
| | * | rts: Add prefetch macrosBen Gamari2019-10-221-0/+4
| | | |
| | * | NonMoving: Optimize bitmap search during allocationBen Gamari2019-10-221-2/+14
| | | | | | | | | | | | | | | | | | | | Use memchr instead of a open-coded loop. This is nearly twice as fast in a synthetic benchmark.
| | * | NonMovingMark: Optimize representation of mark queueBen Gamari2019-10-222-13/+38
| | | | | | | | | | | | | | | | This shortens MarkQueueEntry by 30% (one word)
| | * | NonMoving: Allocate mark queues in larger block groupsBen Gamari2019-10-222-4/+7
| | | |
| | * | NonMoving: Eliminate integer division in nonmovingBlockCountBen Gamari2019-10-221-4/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Perf showed that the this single div was capturing up to 10% of samples in nonmovingMark. However, the overwhelming majority of cases is looking at small block sizes. These cases we can easily compute explicitly, allowing the compiler to turn the division into a significantly more efficient division-by-constant. While the increase in source code looks scary, this all optimises down to very nice looking assembler. At this point the only remaining hotspots in nonmovingBlockCount are due to memory access.
| | * | More comments for agingwip/gc/agingBen Gamari2019-10-221-5/+91
| | | |
| | * | Disable aging when doing deadlock detection GCBen Gamari2019-10-225-33/+66
| | | |
| | * | Nonmoving: Allow aging and refactor static objects logicBen Gamari2019-10-2210-81/+266
| | |/ | | | | | | | | | | | | | | | | | | This commit does two things: * Allow aging of objects during the preparatory minor GC * Refactor handling of static objects to avoid the use of a hashtable
| * | NonmovingCensus: Emit samples to eventlogwip/gc/instrumentationBen Gamari2019-10-2210-2/+69
| | |
| * | Allow census without live word countBen Gamari2019-10-222-16/+47
| | | | | | | | | | | | | | | Otherwise the census is unsafe when mutators are running due to concurrent mutation.
| * | rts/Eventlog: More descriptive error messageBen Gamari2019-10-221-2/+4
| | |
| * | rts: Introduce non-moving heap censusBen Gamari2019-10-224-0/+111
| | | | | | | | | | | | | | | | | | | | | | | | This introduces a simple census of the non-moving heap (not to be confused with the heap census used by the heap profiler). This collects basic heap usage information (number of allocated and free blocks) which is useful when characterising fragmentation of the nonmoving heap.
| * | rts: Tracing support for nonmoving collection eventsBen Gamari2019-10-227-2/+134
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This introduces a few events to mark key points in the nonmoving garbage collection cycle. These include: * `EVENT_CONC_MARK_BEGIN`, denoting the beginning of a round of marking. This may happen more than once in a single major collection since we the major collector iterates until it hits a fixed point. * `EVENT_CONC_MARK_END`, denoting the end of a round of marking. * `EVENT_CONC_SYNC_BEGIN`, denoting the beginning of the post-mark synchronization phase * `EVENT_CONC_UPD_REM_SET_FLUSH`, indicating that a capability has flushed its update remembered set. * `EVENT_CONC_SYNC_END`, denoting that all mutators have flushed their update remembered sets. * `EVENT_CONC_SWEEP_BEGIN`, denoting the beginning of the sweep portion of the major collection. * `EVENT_CONC_SWEEP_END`, denoting the end of the sweep portion of the major collection.
| * Fix unregisterised buildwip/gc/nonmoving-concurrentBen Gamari2019-10-227-23/+41
| | | | | | | | | | | | This required some fiddling around with the location of forward declarations since the C sources generated by GHC's C backend only includes Stg.h.
| * ThreadPaused: Add barrer on updated thunkBen Gamari2019-10-211-0/+4
| |
| * Nonmoving: Ensure write barrier vanishes in non-threaded RTSBen Gamari2019-10-2112-36/+65
| |
| * Don't cleanup until we've stopped the collectorBen Gamari2019-10-204-3/+21
| | | | | | | | | | | | | | This requires that we break nonmovingExit into two pieces since we need to first stop the collector to relinquish any capabilities, then we need to shutdown the scheduler, then we need to free the nonmoving allocators.
| * rts: Shrink size of STACK's dirty and marking fieldsBen Gamari2019-10-203-6/+25
| |
| * Nonmoving: Disable memory inventory with concurrent collectionBen Gamari2019-10-201-0/+8
| |
| * rts: Implement concurrent collection in the nonmoving collectorBen Gamari2019-10-2034-124/+1290
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This extends the non-moving collector to allow concurrent collection. The full design of the collector implemented here is described in detail in a technical note B. Gamari. "A Concurrent Garbage Collector For the Glasgow Haskell Compiler" (2018) This extension involves the introduction of a capability-local remembered set, known as the /update remembered set/, which tracks objects which may no longer be visible to the collector due to mutation. To maintain this remembered set we introduce a write barrier on mutations which is enabled while a concurrent mark is underway. The update remembered set representation is similar to that of the nonmoving mark queue, being a chunked array of `MarkEntry`s. Each `Capability` maintains a single accumulator chunk, which it flushed when it (a) is filled, or (b) when the nonmoving collector enters its post-mark synchronization phase. While the write barrier touches a significant amount of code it is conceptually straightforward: the mutator must ensure that the referee of any pointer it overwrites is added to the update remembered set. However, there are a few details: * In the case of objects with a dirty flag (e.g. `MVar`s) we can exploit the fact that only the *first* mutation requires a write barrier. * Weak references, as usual, complicate things. In particular, we must ensure that the referee of a weak object is marked if dereferenced by the mutator. For this we (unfortunately) must introduce a read barrier, as described in Note [Concurrent read barrier on deRefWeak#] (in `NonMovingMark.c`). * Stable names are also a bit tricky as described in Note [Sweeping stable names in the concurrent collector] (`NonMovingSweep.c`). We take quite some pains to ensure that the high thread count often seen in parallel Haskell applications doesn't affect pause times. To this end we allow thread stacks to be marked either by the thread itself (when it is executed or stack-underflows) or the concurrent mark thread (if the thread owning the stack is never scheduled). There is a non-trivial handshake to ensure that this happens without racing which is described in Note [StgStack dirtiness flags and concurrent marking]. Co-Authored-by: Ömer Sinan Ağacan <omer@well-typed.com>
| * rts: Mark binder as constwip/gc/nonmoving-nonconcurrentBen Gamari2019-10-201-1/+1
| |
| * testsuite: Add nonmoving WAYBen Gamari2019-10-201-2/+5
| | | | | | | | | | This simply runs the compile_and_run tests with `-xn`, enabling the nonmoving oldest generation.
| * rts: Non-concurrent mark and sweepÖmer Sinan Ağacan2019-10-2024-155/+3644
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This implements the core heap structure and a serial mark/sweep collector which can be used to manage the oldest-generation heap. This is the first step towards a concurrent mark-and-sweep collector aimed at low-latency applications. The full design of the collector implemented here is described in detail in a technical note B. Gamari. "A Concurrent Garbage Collector For the Glasgow Haskell Compiler" (2018) The basic heap structure used in this design is heavily inspired by K. Ueno & A. Ohori. "A fully concurrent garbage collector for functional programs on multicore processors." /ACM SIGPLAN Notices/ Vol. 51. No. 9 (presented by ICFP 2016) This design is intended to allow both marking and sweeping concurrent to execution of a multi-core mutator. Unlike the Ueno design, which requires no global synchronization pauses, the collector introduced here requires a stop-the-world pause at the beginning and end of the mark phase. To avoid heap fragmentation, the allocator consists of a number of fixed-size /sub-allocators/. Each of these sub-allocators allocators into its own set of /segments/, themselves allocated from the block allocator. Each segment is broken into a set of fixed-size allocation blocks (which back allocations) in addition to a bitmap (used to track the liveness of blocks) and some additional metadata (used also used to track liveness). This heap structure enables collection via mark-and-sweep, which can be performed concurrently via a snapshot-at-the-beginning scheme (although concurrent collection is not implemented in this patch). The mark queue is a fairly straightforward chunked-array structure. The representation is a bit more verbose than a typical mark queue to accomodate a combination of two features: * a mark FIFO, which improves the locality of marking, reducing one of the major overheads seen in mark/sweep allocators (see [1] for details) * the selector optimization and indirection shortcutting, which requires that we track where we found each reference to an object in case we need to update the reference at a later point (e.g. when we find that it is an indirection). See Note [Origin references in the nonmoving collector] (in `NonMovingMark.h`) for details. Beyond this the mark/sweep is fairly run-of-the-mill. [1] R. Garner, S.M. Blackburn, D. Frampton. "Effective Prefetch for Mark-Sweep Garbage Collection." ISMM 2007. Co-Authored-By: Ben Gamari <ben@well-typed.com>
| * rts: Introduce debug flag for non-moving GCBen Gamari2019-10-202-15/+19
| |
| * rts: Introduce flag to enable the nonmoving old generationBen Gamari2019-10-203-0/+43
| | | | | | | | This flag will enable the use of a non-moving oldest generation.
| * rts/Scav: Expose scavenging functionsÖmer Sinan Ağacan2019-10-202-11/+31
| | | | | | | | | | | | | | To keep the non-moving collector nicely separated from the moving collector its scavenging phase will live in another file, `NonMovingScav.c`. However, it will need to use these functions so let's expose them.
| * rts: Disable aggregate-return warnings from gccBen Gamari2019-10-201-0/+2
| | | | | | | | | | This warning is a bit of a relic; there is little reason to avoid aggregate return values in 2019.
| * rts/StableName: Expose FOR_EACH_STABLE_NAME, freeSnEntry, SNT_sizeÖmer Sinan Ağacan2019-10-202-24/+27
| | | | | | | | | | These will be needed when we implement sweeping in the nonmoving collector.
| *-. Merge branches 'wip/gc/sync-without-capability' and ↵wip/gc/preparationBen Gamari2019-10-205-61/+307
| |\ \ | | | | | | | | | | | | 'wip/gc/aligned-block-allocation' into wip/gc/preparation
| | | * rts/BlockAlloc: Allow aligned allocation requestswip/gc/aligned-block-allocationÖmer Sinan Ağacan2019-10-183-38/+234
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This implements support for block group allocations which are aligned to an integral number of blocks. This will be used by the nonmoving garbage collector, which uses the block allocator to allocate the segments which back its heap. These segments are a fixed number of blocks in size, with each segment being aligned to the segment size boundary. This allows us to easily find the segment metadata stored at the beginning of the segment.
| | * | rts/Schedule: Allow synchronization without holding a capabilitywip/gc/sync-without-capabilityBen Gamari2019-10-182-23/+73
| | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The concurrent mark-and-sweep will be performed by a GHC task which will not hold a capability. This is necessary to avoid a concurrent mark from interfering with minor generation collections. However, the major collector must synchronize with the mutators at the end of marking to flush their update remembered sets. This patch extends the `requestSync` mechanism used to synchronize garbage collectors to allow synchronization without holding a capability. This change is fairly straightforward as the capability was previously only required for two reasons: 1. to ensure that we don't try to re-acquire a capability that we the sync requestor already holds. 2. to provide a way to suspend and later resume the sync request if there is already a sync pending. When synchronizing without holding a capability we needn't worry about consideration (1) at all. (2) is slightly trickier and may happen, for instance, when a capability requests a minor collection and shortly thereafter the non-moving mark thread requests a post-mark synchronization. In this case we need to ensure that the non-moving mark thread suspends his request until after the minor GC has concluded to avoid dead-locking. For this we introduce a condition variable, `sync_finished_cond`, which a non-capability-bearing requestor will wait on and which is signalled after a synchronization or GC has finished.
| * | rts: Fix macro parenthesisationwip/gc/misc-rtsBen Gamari2019-10-181-1/+1
| | |
| * | rts/GC: Refactor gcCAFsBen Gamari2019-10-181-13/+8
| | |