[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