A
lgorithmique
N
umérique
D
istribuée
Public GIT Repository
projects
/
simgrid.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
| inline |
side by side
Memory leak.
[simgrid.git]
/
src
/
simix
/
smx_context.cpp
diff --git
a/src/simix/smx_context.cpp
b/src/simix/smx_context.cpp
index
933ef54
..
bd4a7e3
100644
(file)
--- a/
src/simix/smx_context.cpp
+++ b/
src/simix/smx_context.cpp
@@
-97,6
+97,9
@@
void SIMIX_context_mod_init()
{
xbt_assert(simix_global->context_factory == nullptr);
+ smx_context_stack_size_was_set = not simgrid::config::is_default("contexts/stack-size");
+ smx_context_guard_size_was_set = not simgrid::config::is_default("contexts/guard-size");
+
#if HAVE_THREAD_CONTEXTS && not HAVE_THREAD_LOCAL_STORAGE
/* the __thread storage class is not available on this platform:
* use getspecific/setspecific instead to store the current context in each thread */
@@
-104,7
+107,7
@@
void SIMIX_context_mod_init()
#endif
#if HAVE_SMPI && (defined(__APPLE__) || defined(__NetBSD__))
- std::string priv =
xbt_cfg_get_string
("smpi/privatization");
+ std::string priv =
simgrid::config::get_value<std::string>
("smpi/privatization");
if (context_factory_name == "thread" && (priv == "dlopen" || priv == "yes" || priv == "default" || priv == "1")) {
XBT_WARN("dlopen+thread broken on Apple and BSD. Switching to raw contexts.");
context_factory_name = "raw";
@@
-112,7
+115,7
@@
void SIMIX_context_mod_init()
#endif
#if HAVE_SMPI && defined(__FreeBSD__)
- if (context_factory_name == "thread" &&
xbt_cfg_get_string("smpi/privatization") != "no")
{
+ if (context_factory_name == "thread" &&
simgrid::config::get_value<std::string>("smpi/privatization") != "no")
{
XBT_WARN("mmap broken on FreeBSD, but dlopen+thread broken too. Switching to dlopen+raw contexts.");
context_factory_name = "raw";
}
@@
-165,18
+168,14
@@
void *SIMIX_context_stack_new()
{
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 && not MC_is_active()) {
#if !defined(PTH_STACKGROWTH) || (PTH_STACKGROWTH != -1)
- static int warned_once = 0;
- if (not warned_once) {
-
XBT_WARN("Stack overflow protection is known to be broken on your system. Either stack grows upwards, or it was not even tested properly.
");
- warned_once = 1;
- }
+ xbt_die("Stack overflow protection is known to be broken on your system: you stacks grow upwards (or detection is "
+ "broken). "
+
"Please disable stack guards with --cfg=contexts:guard-size:0
");
+ /* 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. */
#endif
size_t size = smx_context_stack_size + smx_context_guard_size;
@@
-252,21
+251,21
@@
int SIMIX_context_is_parallel() {
/**
* @brief Returns the number of parallel threads used for the user contexts.
- *
\
return the number of threads (1 means no parallelism)
+ *
@
return the number of threads (1 means no parallelism)
*/
int SIMIX_context_get_nthreads() {
return smx_parallel_contexts;
}
/**
- *
\
brief Sets the number of parallel threads to use
+ *
@
brief Sets the number of parallel threads to use
* for the user contexts.
*
* This function should be called before initializing SIMIX.
* A value of 1 means no parallelism (1 thread only).
* If the value is greater than 1, the thread support must be enabled.
*
- *
\
param nb_threads the number of threads to use
+ *
@
param nb_threads the number of threads to use
*/
void SIMIX_context_set_nthreads(int nb_threads) {
if (nb_threads<=0) {
@@
-280,12
+279,12
@@
void SIMIX_context_set_nthreads(int nb_threads) {
}
/**
- *
\
brief Returns the threshold above which user processes are run in parallel.
+ *
@
brief Returns the threshold above which user processes are run in parallel.
*
* If the number of threads is set to 1, there is no parallelism and this
* threshold has no effect.
*
- *
\
return when the number of user processes ready to run is above
+ *
@
return when the number of user processes ready to run is above
* this threshold, they are run in parallel
*/
int SIMIX_context_get_parallel_threshold() {
@@
-293,12
+292,12
@@
int SIMIX_context_get_parallel_threshold() {
}
/**
- *
\
brief Sets the threshold above which user processes are run in parallel.
+ *
@
brief Sets the threshold above which user processes are run in parallel.
*
* If the number of threads is set to 1, there is no parallelism and this
* threshold has no effect.
*
- *
\
param threshold when the number of user processes ready to run is above
+ *
@
param threshold when the number of user processes ready to run is above
* this threshold, they are run in parallel
*/
void SIMIX_context_set_parallel_threshold(int threshold) {
@@
-306,26
+305,26
@@
void SIMIX_context_set_parallel_threshold(int threshold) {
}
/**
- *
\
brief Returns the synchronization mode used when processes are run in
+ *
@
brief Returns the synchronization mode used when processes are run in
* parallel.
- *
\
return how threads are synchronized if processes are run in parallel
+ *
@
return how threads are synchronized if processes are run in parallel
*/
e_xbt_parmap_mode_t SIMIX_context_get_parallel_mode() {
return smx_parallel_synchronization_mode;
}
/**
- *
\
brief Sets the synchronization mode to use when processes are run in
+ *
@
brief Sets the synchronization mode to use when processes are run in
* parallel.
- *
\
param mode how to synchronize threads if processes are run in parallel
+ *
@
param mode how to synchronize threads if processes are run in parallel
*/
void SIMIX_context_set_parallel_mode(e_xbt_parmap_mode_t mode) {
smx_parallel_synchronization_mode = mode;
}
/**
- *
\
brief Returns the current context of this thread.
- *
\
return the current context of this thread
+ *
@
brief Returns the current context of this thread.
+ *
@
return the current context of this thread
*/
smx_context_t SIMIX_context_get_current()
{
@@
-342,8
+341,8
@@
smx_context_t SIMIX_context_get_current()
}
/**
- *
\
brief Sets the current context of this thread.
- *
\
param context the context to set
+ *
@
brief Sets the current context of this thread.
+ *
@
param context the context to set
*/
void SIMIX_context_set_current(smx_context_t context)
{