[Tarantool-patches] [PATCH] fiber: set diag error on madvise, mprotect fails
Oleg Babin
olegrok at tarantool.org
Sat Jan 11 10:03:33 MSK 2020
Hello, thanks for your patch!
Our documentation doesn't contain any mentions that fiber.create can fail.
Could you please file an issue (or may be doc bot request) with
description in what cases user
can expect that fiber.create will fail. Usually users don't use pcall
for fiber.create and this fact can
be quite unexpected for them.
On 10/01/2020 17:27, Cyrill Gorcunov wrote:
> Currently we use say_x to log if error happened.
> Strictly speaking when madvise is used to relax
> pressue on mm subsystem since we started to use
> bigger stacks (due to libreadline increased memory
> usage).
>
> Same time mprotect usage is more sensitive -- we
> setup guard pages with its help and when failing
> it is definitely an error which we should not only
> mention in logger but provide a caller diag error.
> Here is an example from memcached
>
--
Oleg Babin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.tarantool.org/pipermail/tarantool-patches/attachments/20200111/3ac4fc67/attachment.html>
More information about the Tarantool-patches
mailing list