X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/9a74a4d9c970bf6b49d2c15d284ca190451d5ef6..b3ca3a517af21780e88cc89869b16927d1fea68a:/src/simix/smx_context.c diff --git a/src/simix/smx_context.c b/src/simix/smx_context.c index 0bfad29665..f222f62f28 100644 --- a/src/simix/smx_context.c +++ b/src/simix/smx_context.c @@ -14,7 +14,20 @@ #include "simgrid/sg_config.h" #include "internal_config.h" #include "simgrid/modelchecker.h" + + +#ifdef _WIN32 +#include +#else #include +#endif + +#ifdef __MINGW32__ +#define _aligned_malloc __mingw_aligned_malloc +#define _aligned_free __mingw_aligned_free +#endif //MINGW + + #ifdef HAVE_VALGRIND_VALGRIND_H # include @@ -111,7 +124,21 @@ void *SIMIX_context_stack_new(void) { void *stack; + /* FIXME: current code for stack overflow protection assumes that stacks are + * growing downward (PTH_STACKGROWTH == -1). Protected pages need to be put + * after the stack when PTH_STACKGROWTH == 1. */ + if (smx_context_guard_size > 0 && !MC_is_active()) { + +#if defined(_XBT_WIN32) || (PTH_STACKGROWTH != -1) + static int warned_once = 0; + if (!warned_once) { + XBT_WARN("Stack overflow protection is known to be broken on your system. Either you're on Windows or PTH_STACKGROWTH != -1 (current value is %d).", + PTH_STACKGROWTH); + warned_once = 1; + } +#endif + size_t size = smx_context_stack_size + smx_context_guard_size; #ifdef HAVE_MC /* Cannot use posix_memalign when HAVE_MC. Align stack by hand, and save the @@ -119,14 +146,19 @@ void *SIMIX_context_stack_new(void) char *alloc = xbt_malloc0(size + xbt_pagesize); stack = alloc - ((uintptr_t)alloc & (xbt_pagesize - 1)) + xbt_pagesize; *((void **)stack - 1) = alloc; -#else +#elif !defined(_XBT_WIN32) if (posix_memalign(&stack, xbt_pagesize, size) != 0) xbt_die("Failed to allocate stack."); +#else + stack = _aligned_malloc(size, xbt_pagesize); #endif + +#ifndef _XBT_WIN32 if (mprotect(stack, smx_context_guard_size, PROT_NONE) == -1) { XBT_WARN("Failed to protect stack: %s", strerror(errno)); /* That's not fatal, pursue anyway. */ } +#endif stack = (char *)stack + smx_context_guard_size; } else { stack = xbt_malloc0(smx_context_stack_size); @@ -154,6 +186,7 @@ void SIMIX_context_stack_delete(void *stack) VALGRIND_STACK_DEREGISTER(valgrind_stack_id); #endif +#ifndef WIN32 if (smx_context_guard_size > 0 && !MC_is_active()) { stack = (char *)stack - smx_context_guard_size; if (mprotect(stack, smx_context_guard_size, @@ -166,6 +199,8 @@ void SIMIX_context_stack_delete(void *stack) stack = *((void **)stack - 1); #endif } +#endif + xbt_free(stack); }