source: libcfa/src/concurrency/coroutine.cfa @ 1c0a3a4

Last change on this file since 1c0a3a4 was cce0807, checked in by caparson <caparson@…>, 13 months ago

Fixed non-local ehm memory leak that occurs when a coroutine is deleted while suspended in a catchresume handler

  • Property mode set to 100644
File size: 13.6 KB
Line 
1//
2// Cforall Version 1.0.0 Copyright (C) 2016 University of Waterloo
3//
4// The contents of this file are covered under the licence agreement in the
5// file "LICENCE" distributed with Cforall.
6//
7// coroutine.c --
8//
9// Author           : Thierry Delisle
10// Created On       : Mon Nov 28 12:27:26 2016
11// Last Modified By : Peter A. Buhr
12// Last Modified On : Mon Sep 18 21:47:12 2023
13// Update Count     : 25
14//
15
16#define __cforall_thread__
17
18#include "coroutine.hfa"
19
20#include <stddef.h>
21#include <malloc.h>
22#include <errno.h>
23#include <string.h>
24#include <unistd.h>
25#include <sys/mman.h>                                                                   // mprotect
26#include <unwind.h>
27
28#include "kernel/private.hfa"
29#include "exception.hfa"
30#include "exception.h"
31#include "math.hfa"
32
33#define CFA_COROUTINE_USE_MMAP 0
34
35#define __CFA_INVOKE_PRIVATE__
36#include "invoke.h"
37
38extern "C" {
39        void _CtxCoroutine_Unwind(struct _Unwind_Exception * storage, struct coroutine$ *) __attribute__ ((__noreturn__));
40        static void _CtxCoroutine_UnwindCleanup(_Unwind_Reason_Code, struct _Unwind_Exception *) __attribute__ ((__noreturn__));
41        static void _CtxCoroutine_UnwindCleanup(_Unwind_Reason_Code, struct _Unwind_Exception *) {
42                abort();
43        }
44
45        extern void CtxRet( struct __stack_context_t * to ) asm ("CtxRet") __attribute__ ((__noreturn__));
46}
47
48//-----------------------------------------------------------------------------
49forall(T &)
50void copy(CoroutineCancelled(T) * dst, CoroutineCancelled(T) * src) libcfa_public {
51        dst->virtual_table = src->virtual_table;
52        dst->the_coroutine = src->the_coroutine;
53        dst->the_exception = src->the_exception;
54}
55
56forall(T &)
57const char * msg(CoroutineCancelled(T) *) libcfa_public {
58        return "CoroutineCancelled(...)";
59}
60
61// This code should not be inlined. It is the error path on resume.
62forall(T & | is_coroutine(T))
63void __cfaehm_cancelled_coroutine(
64                T & cor, coroutine$ * desc, EHM_DEFAULT_VTABLE(CoroutineCancelled(T)) ) libcfa_public {
65        verify( desc->cancellation );
66        desc->state = Cancelled;
67        exception_t * except = __cfaehm_cancellation_exception( desc->cancellation );
68
69        // TODO: Remove explitate vtable set once trac#186 is fixed.
70        CoroutineCancelled(T) except;
71        except.virtual_table = &_default_vtable;
72        except.the_coroutine = &cor;
73        except.the_exception = except;
74        // Why does this need a cast?
75        throwResume (CoroutineCancelled(T) &)except;
76
77        except->virtual_table->free( except );
78        free( desc->cancellation );
79        desc->cancellation = 0p;
80}
81
82// helper for popping from coroutine's ehm buffer
83inline nonlocal_exception * pop_ehm_head( coroutine$ * this ) {
84    lock( this->ehm_state.buffer_lock __cfaabi_dbg_ctx2 );
85    nonlocal_exception * nl_ex = pop_head( this->ehm_state.ehm_buffer );
86    unlock( this->ehm_state.buffer_lock );
87    return nl_ex;
88}
89
90//-----------------------------------------------------------------------------
91// Global state variables
92
93// minimum feasible stack size in bytes
94static const size_t MinStackSize = 1000;
95extern size_t __page_size;                              // architecture pagesize HACK, should go in proper runtime singleton
96extern int __map_prot;
97
98void __stack_prepare( __stack_info_t * this, size_t create_size );
99static void __stack_clean  ( __stack_info_t * this );
100
101//-----------------------------------------------------------------------------
102// Coroutine ctors and dtors
103void ?{}( __stack_info_t & this, void * storage, size_t storageSize ) {
104        this.storage   = (__stack_t *)storage;
105
106        // Did we get a piece of storage ?
107        if (this.storage || storageSize != 0) {
108                // We either got a piece of storage or the user asked for a specific size
109                // Immediately create the stack
110                // (This is slightly unintuitive that non-default sized coroutines create are eagerly created
111                // but it avoids that all coroutines carry an unnecessary size)
112                verify( storageSize != 0 );
113                __stack_prepare( &this, storageSize );
114        }
115}
116
117void ^?{}(__stack_info_t & this) {
118        bool userStack = ((intptr_t)this.storage & 0x1) != 0;
119        if ( ! userStack && this.storage ) {
120                __stack_clean( &this );
121        }
122}
123
124void ?{}( coroutine$ & this, const char name[], void * storage, size_t storageSize ) libcfa_public with( this ) {
125        (this.context){0p, 0p};
126        (this.stack){storage, storageSize};
127        this.name = name;
128        state = Start;
129        starter = 0p;
130        last = 0p;
131        cancellation = 0p;
132    ehm_state.ehm_buffer{};
133    ehm_state.buffer_lock{};
134    ehm_state.ehm_enabled = false;
135}
136
137void ^?{}(coroutine$& this) libcfa_public {
138    // handle any leftover pending non-local exceptions
139    nonlocal_exception * nl_ex = pop_ehm_head( &this );
140    unsigned unhandled_ex = 0;
141   
142    // if any leftover exceptions handle
143    while ( nl_ex != 0p ){
144        unhandled_ex++;
145        free( nl_ex->the_exception );
146        free( nl_ex );
147        nl_ex = pop_ehm_head( &this );
148    }
149
150    #ifdef __CFA_DEBUG__
151    if ( unhandled_ex > 0 )
152        printf( "Warning: Coroutine %p exited with %u pending nonlocal exceptions.\n", &this, unhandled_ex );
153    #endif
154
155        if(this.state != Halted && this.state != Start && this.state != Primed) {
156                coroutine$ * src = active_coroutine();
157                coroutine$ * dst = &this;
158
159                struct _Unwind_Exception storage;
160                storage.exception_class = -1;
161                storage.exception_cleanup = _CtxCoroutine_UnwindCleanup;
162                this.cancellation = &storage;
163                this.last = src;
164
165                // not resuming self ?
166                if ( src == dst ) {
167                        abort( "Attempt by coroutine %.256s (%p) to terminate itself.\n", src->name, src );
168                }
169
170                $ctx_switch( src, dst );
171        }
172}
173
174// Part of the Public API
175// Not inline since only ever called once per coroutine
176forall(T & | is_coroutine(T) | { EHM_DEFAULT_VTABLE(CoroutineCancelled(T)); })
177void prime(T& cor) libcfa_public {
178        coroutine$* this = get_coroutine(cor);
179        assert(this->state == Start);
180
181        this->state = Primed;
182        resume(cor);
183}
184
185static [void *, size_t] __stack_alloc( size_t storageSize ) {
186        const size_t stack_data_size = libCeiling( sizeof(__stack_t), 16 ); // minimum alignment
187        assert(__page_size != 0l);
188        size_t size = libCeiling( storageSize, 16 ) + stack_data_size;
189        size = ceiling(size, __page_size);
190
191        // If we are running debug, we also need to allocate a guardpage to catch stack overflows.
192        void * storage;
193        #if CFA_COROUTINE_USE_MMAP
194                storage = mmap(0p, size + __page_size, PROT_EXEC | PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANONYMOUS, 0, 0);
195                if(storage == ((void*)-1)) {
196                        abort( "coroutine stack creation : internal error, mmap failure, error(%d) %s.", errno, strerror( errno ) );
197                }
198                if ( mprotect( storage, __page_size, PROT_NONE ) == -1 ) {
199                        abort( "coroutine stack creation : internal error, mprotect failure, error(%d) %s.", errno, strerror( errno ) );
200                } // if
201                storage = (void *)(((intptr_t)storage) + __page_size);
202        #else
203                __cfaabi_dbg_debug_do(
204                        storage = memalign( __page_size, size + __page_size );
205                );
206                __cfaabi_dbg_no_debug_do(
207                        storage = (void*)malloc(size);
208                );
209
210                __cfaabi_dbg_debug_do(
211                        if ( mprotect( storage, __page_size, PROT_NONE ) == -1 ) {
212                                abort( "__stack_alloc : internal error, mprotect failure, error(%d) %s.", (int)errno, strerror( (int)errno ) );
213                        }
214                        storage = (void *)(((intptr_t)storage) + __page_size);
215                );
216        #endif
217        __cfaabi_dbg_print_safe("Kernel : Created stack %p of size %zu\n", storage, size);
218
219        verify( ((intptr_t)storage & (libAlign() - 1)) == 0ul );
220        return [storage, size];
221}
222
223static void __stack_clean  ( __stack_info_t * this ) {
224        void * storage = this->storage->limit;
225
226        #if CFA_COROUTINE_USE_MMAP
227                size_t size = ((intptr_t)this->storage->base) - ((intptr_t)this->storage->limit) + sizeof(__stack_t);
228                storage = (void *)(((intptr_t)storage) - __page_size);
229                if(munmap(storage, size + __page_size) == -1) {
230                        abort( "coroutine stack destruction : internal error, munmap failure, error(%d) %s.", errno, strerror( errno ) );
231                }
232        #else
233                __cfaabi_dbg_debug_do(
234                        storage = (char*)(storage) - __page_size;
235                        if ( mprotect( storage, __page_size, __map_prot ) == -1 ) {
236                                abort( "(coStack_t *)%p.^?{}() : internal error, mprotect failure, error(%d) %s.", &this, errno, strerror( errno ) );
237                        }
238                );
239
240                free( storage );
241        #endif
242        __cfaabi_dbg_print_safe("Kernel : Deleting stack %p\n", storage);
243}
244
245void __stack_prepare( __stack_info_t * this, size_t create_size ) libcfa_public {
246        const size_t stack_data_size = libCeiling( sizeof(__stack_t), 16 ); // minimum alignment
247        bool userStack;
248        void * storage;
249        size_t size;
250        if ( !this->storage ) {
251                userStack = false;
252                [storage, size] = __stack_alloc( create_size );
253        } else {
254                userStack = true;
255                __cfaabi_dbg_print_safe("Kernel : stack obj %p using user stack %p(%zd bytes)\n", this, this->storage, (intptr_t)this->storage->limit - (intptr_t)this->storage->base);
256
257                // The stack must be aligned, advance the pointer to the next align data
258                storage = (void*)libCeiling( (intptr_t)this->storage, libAlign());
259
260                // The size needs to be shrinked to fit all the extra data structure and be aligned
261                ptrdiff_t diff = (intptr_t)storage - (intptr_t)this->storage;
262                size = libFloor(create_size - stack_data_size - diff, libAlign());
263        } // if
264        assertf( size >= MinStackSize, "Stack size %zd provides less than minimum of %zd bytes for a stack.", size, MinStackSize );
265
266        this->storage = (__stack_t *)((intptr_t)storage + size - sizeof(__stack_t));
267        this->storage->limit = storage;
268        this->storage->base  = (void*)((intptr_t)storage + size - sizeof(__stack_t));
269        this->storage->exception_context.top_resume = 0p;
270        this->storage->exception_context.current_exception = 0p;
271        __attribute__((may_alias)) intptr_t * istorage = (intptr_t*)&this->storage;
272        *istorage |= userStack ? 0x1 : 0x0;
273}
274
275// We need to call suspend from invoke.c, so we expose this wrapper that
276// is not inline (We can't inline Cforall in C)
277extern "C" {
278        void __cfactx_cor_leave( struct coroutine$ * src ) {
279                coroutine$ * starter = src->cancellation != 0 ? src->last : src->starter;
280
281                src->state = Halted;
282
283                assertf( starter != 0,
284                        "Attempt to suspend/leave coroutine \"%.256s\" (%p) that has never been resumed.\n"
285                        "Possible cause is a suspend executed in a member called by a coroutine user rather than by the coroutine main.",
286                        src->name, src );
287                assertf( starter->state != Halted,
288                        "Attempt by coroutine \"%.256s\" (%p) to suspend/leave back to terminated coroutine \"%.256s\" (%p).\n"
289                        "Possible cause is terminated coroutine's main routine has already returned.",
290                        src->name, src, starter->name, starter );
291
292                $ctx_switch( src, starter );
293        }
294
295        struct coroutine$ * __cfactx_cor_finish(void) {
296                struct coroutine$ * cor = active_coroutine();
297
298                // get the active thread once
299                thread$ * athrd = active_thread();
300
301                /* paranoid */ verify( athrd->corctx_flag );
302                athrd->corctx_flag = false;
303
304                if(cor->state == Primed) {
305                        __cfactx_suspend();
306                }
307
308                cor->state = Active;
309
310                return cor;
311        }
312}
313
314
315////////////////////////////////////////////////////////////////////////////////////////////////////
316// non local ehm routines
317
318void defaultResumeAtHandler( exception_t * except ) {
319    __cfaehm_allocate_exception( except );
320    __cfaehm_begin_unwind( (void(*)(exception_t *))defaultTerminationHandler );
321}
322
323struct ehm_cleanup {
324        exception_t * ex;
325};
326
327void ^?{}( ehm_cleanup & this ) { free( this.ex ); }
328
329bool poll( coroutine$ * cor ) libcfa_public {
330    nonlocal_exception * nl_ex = pop_ehm_head( cor );
331
332    // if no exceptions return false
333    if ( nl_ex == 0p ) return false;
334   
335    // otherwise loop and throwResume all pending exceptions
336    while ( nl_ex != 0p ){
337                ehm_cleanup ex_holder{ nl_ex->the_exception };
338        free( nl_ex );
339        __cfaehm_throw_resume( ex_holder.ex , defaultResumeAtHandler );
340       
341        nl_ex = pop_ehm_head( cor );
342    }
343   
344    return true;
345}
346
347bool poll() libcfa_public { return poll( active_coroutine() ); }
348void enable_ehm() libcfa_public { active_coroutine()->ehm_state.ehm_enabled = true; }
349void disable_ehm() libcfa_public { active_coroutine()->ehm_state.ehm_enabled = false; }
350bool checked_poll() libcfa_public { return active_coroutine()->ehm_state.ehm_enabled ? poll( active_coroutine() ) : false; }
351coroutine$ * resumer() libcfa_public { return active_coroutine()->last; }
352coroutine$ * first_resumer() libcfa_public { return active_coroutine()->starter; }
353
354// user facing ehm operations
355forall(T & | is_coroutine(T)) {
356    // enable/disable non-local exceptions
357    void enable_ehm( T & cor ) libcfa_public { get_coroutine( cor )->ehm_state.ehm_enabled = true; }
358    void disable_ehm( T & cor ) libcfa_public { get_coroutine( cor )->ehm_state.ehm_enabled = false; }
359
360    // poll for non-local exceptions
361    bool poll( T & cor ) libcfa_public { return poll( get_coroutine( cor ) ); }
362
363    // poll iff nonlocal ehm is enabled
364    bool checked_poll( T & cor ) libcfa_public { return get_coroutine( cor )->ehm_state.ehm_enabled ? poll( cor ) : false; }
365
366    coroutine$ * resumer( T & cor ) libcfa_public { return get_coroutine( cor )->last; }
367    coroutine$ * first_resumer( T & cor ) libcfa_public { return get_coroutine( cor )->starter; }
368}
369
370// resume non local exception at receiver (i.e. enqueue in ehm buffer)
371forall(exceptT *, T & | ehm_resume_at( exceptT, T ))
372void resumeAt( T & receiver, exceptT & ex ) libcfa_public {
373    coroutine$ * cor = get_coroutine( receiver );
374    nonlocal_exception * nl_ex = alloc();
375    exceptT * ex_copy = alloc();
376    memcpy( ex_copy, &ex, sizeof(exceptT) );
377    (*nl_ex){ (exception_t *)ex_copy };
378    lock( cor->ehm_state.buffer_lock __cfaabi_dbg_ctx2 );
379    append( cor->ehm_state.ehm_buffer, nl_ex );
380    unlock( cor->ehm_state.buffer_lock );
381}
382
383forall(exceptT * | { void $throwResume(exceptT &); })
384void resumeAt( coroutine$ * receiver, exceptT & ex ) libcfa_public {
385    nonlocal_exception * nl_ex = alloc();
386    exceptT * ex_copy = alloc();
387    memcpy( ex_copy, &ex, sizeof(exceptT) );
388    (*nl_ex){ (exception_t *)ex_copy };
389    lock( receiver->ehm_state.buffer_lock __cfaabi_dbg_ctx2 );
390    append( receiver->ehm_state.ehm_buffer, nl_ex );
391    unlock( receiver->ehm_state.buffer_lock );
392}
393
394// Local Variables: //
395// mode: c //
396// tab-width: 4 //
397// End: //
Note: See TracBrowser for help on using the repository browser.