From: Cyrill Gorcunov <gorcunov@gmail.com> To: tml <tarantool-patches@freelists.org> Cc: Vladimir Davydov <vdavydov.dev@gmail.com>, Cyrill Gorcunov <gorcunov@gmail.com> Subject: [PATCH 1/3] lib/core/fiber: Increase default stack size Date: Wed, 13 Mar 2019 01:47:19 +0300 [thread overview] Message-ID: <20190312224721.8053-2-gorcunov@gmail.com> (raw) In-Reply-To: <20190312224721.8053-1-gorcunov@gmail.com> The default 64K stack size used for years become too small for modern distors (Fedora 29 and etc) where third party libraries (such as ncurses) started to use 64K for own buffers and we get SIGSGV early without reaching interactive console phase. Thus we increase default size up to 512K which should fit for common case. Later we will make this value configurable to address arbitrary stack sizes without a need to rebuild the whole code. Note the values are switched to 4K page granularity for sake of future modifications -- we gonna manipulate pages to relax rss usage if OS allows. Closes #3418 --- src/lib/core/fiber.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/lib/core/fiber.c b/src/lib/core/fiber.c index abd6c6b11..f16ac873f 100644 --- a/src/lib/core/fiber.c +++ b/src/lib/core/fiber.c @@ -93,9 +93,9 @@ static int stack_direction; enum { /* The minimum allowable fiber stack size in bytes */ - FIBER_STACK_SIZE_MINIMAL = 16384, + FIBER_STACK_SIZE_MINIMAL = 4 << 12, /* Default fiber stack size in bytes */ - FIBER_STACK_SIZE_DEFAULT = 65536 + FIBER_STACK_SIZE_DEFAULT = 128 << 12 }; /** Default fiber attributes */ -- 2.20.1
next prev parent reply other threads:[~2019-03-12 22:47 UTC|newest] Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-03-12 22:47 [PATCH v5 0/2] " Cyrill Gorcunov 2019-03-12 22:47 ` Cyrill Gorcunov [this message] 2019-03-13 13:42 ` [PATCH 1/3] " Vladimir Davydov 2019-03-13 13:53 ` Cyrill Gorcunov 2019-03-13 14:13 ` Vladimir Davydov 2019-03-12 22:47 ` [PATCH 2/3] lib/core/fiber: Mark stack as unneeded on creation Cyrill Gorcunov 2019-03-12 22:47 ` [PATCH 3/3] lib/core/fiber: Put watermarks into stack to track its usage Cyrill Gorcunov 2019-03-13 13:52 ` Vladimir Davydov 2019-03-13 14:00 ` Cyrill Gorcunov 2019-03-13 14:17 ` Vladimir Davydov 2019-03-13 16:15 ` Cyrill Gorcunov
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20190312224721.8053-2-gorcunov@gmail.com \ --to=gorcunov@gmail.com \ --cc=tarantool-patches@freelists.org \ --cc=vdavydov.dev@gmail.com \ --subject='Re: [PATCH 1/3] lib/core/fiber: Increase default stack size' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox