Commit 77a46c01 authored by Marc Vef's avatar Marc Vef
Browse files

Version bump for Margo that fixes excess memory consumption + client fix 

See https://xgitlab.cels.anl.gov/sds/margo/issues/40 for reference of the issue.
parent a25ed265

ifs/scripts/dl_dep.sh

+1 −1
Original line number Diff line number Diff line
@@ -180,7 +180,7 @@ clonedeps "argobots" "git clone -b dev-get-dev-basic https://github.com/carns/ar 
# get Argobots-snoozer

clonedeps "abt-snoozer" "git clone https://xgitlab.cels.anl.gov/sds/abt-snoozer.git" "3d9240eda290bfb89f08a5673cebd888194a4bd7"

# get Margo

clonedeps "margo" "git clone https://xgitlab.cels.anl.gov/sds/margo.git" "3f9fe3a13392af1ba6df1b3d3bb16503da6b627d"

clonedeps "margo" "git clone https://xgitlab.cels.anl.gov/sds/margo.git" "72eec057314a4251d8658e03a18240275992e1ce"

# get rocksdb

clonedeps "rocksdb" "git clone https://github.com/facebook/rocksdb" "tags/v5.10.2"

ifs/src/preload/preload.cpp

+8 −0
Original line number Diff line number Diff line
@@ -64,6 +64,14 @@ bool init_ld_argobots() { 
        ld_logger->error("{}() ABT_snoozer_xstream_self_set()  (client)", __func__);

        return false;

    }

    /*

     * Single producer (progress function) and multiple consumers are causing an excess memory consumption

     * in some Argobots version. It does only show if an ES with a pool is created.

     * Although this is probably not an issue on the consumer process, we set reduce the Argobots stack size here,

     * just in case we change the client process in the future.

     * See for reference: https://xgitlab.cels.anl.gov/sds/margo/issues/40

     */

    putenv(const_cast<char*>("ABT_MEM_MAX_NUM_STACKS=8"));

    ld_logger->debug("{}() Argobots initialization successful.", __func__);

    return true;

}