Sender: =?UTF-8?Q?Timo_Ter=C3=A4s?= <timo.teras@gmail.com>
Date: Tue, 23 Feb 2016 17:50:28 +0200
From: Timo Teras <timo.teras@iki.fi>
To: Marc Vertes <marc.vertes@ugrid.net>
Cc: alpine-aports@lists.alpinelinux.org
Subject: Re: [alpine-aports] [PATCH] testing/mongodb: fix bug #5117 where
 mongo fails at start
Message-ID: <20160223175029.23bcadb2@vostro>
In-Reply-To: <E7A49593-05F0-4188-921C-7A4B1868D4FC@ugrid.net>
References: <1456083383-9196-1-git-send-email-marc.vertes@ugrid.net>
	<20160223150743.33e55efa@vostro.util.wtbts.net>
	<9D775D86-8A4D-49ED-8512-EBD572ECF6D6@ugrid.net>
	<20160223170446.483be803@vostro.util.wtbts.net>
	<E7A49593-05F0-4188-921C-7A4B1868D4FC@ugrid.net>
Precedence: list
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit

On Tue, 23 Feb 2016 16:35:58 +0100
Marc Vertes <marc.vertes@ugrid.net> wrote:

> No success. I think that the base address returned by
> pthread_attr_getstack(&attr, &base, &size) is the problem. Increasing
> the size (like with getrlimit) doesn't change the test in
> StackLocator.available() which triggers abort: it checks the
> available space against this base address (&localaddr - base > quota).
> 
> Changing size does nothing.
> 
> I also noticed that pthread_getattr_np and pthread_attr_getstack
> where not performed from the main thread (gettid != getpid)

Right. ulimit should be used only for main thread. Not the created
threads with libc allocated stack. That needs a fix.

pthread_attr_getstack should return the lowest addressable byte of the
stack. So indeed, if we change the size, we need to adjust base
accordingly on system with downward growing stack. Something like:

--- mongodb-src-r3.2.1.orig/src/mongo/platform/stack_locator_pthread_getattr_np.cpp
+++ mongodb-src-r3.2.1/src/mongo/platform/stack_locator_pthread_getattr_np.cpp
@@ -31,6 +31,7 @@
 #include "mongo/platform/stack_locator.h"
 
 #include <pthread.h>
+#include <sys/resource.h>
 
 #include "mongo/util/assert_util.h"
 #include "mongo/util/scopeguard.h"
@@ -52,6 +53,13 @@
     invariant(result == 0);
     invariant(base != nullptr);
     invariant(size != 0);
+
+    struct rlimit rl;
+
+    base += size;
+    invariant(getrlimit(RLIMIT_STACK, &rl) == 0);
+    size = rl.rlim_cur ? : 2 * 1024 * 1024;
+    base -= size;
 
     // TODO: Assumes a downward growing stack. Note here that
     // getstack returns the stack *base*, being the bottom of the



---
Unsubscribe:  alpine-aports+unsubscribe@lists.alpinelinux.org
Help:         alpine-aports+help@lists.alpinelinux.org
---