From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <v.shpilevoy@tarantool.org>
Received: from smtp60.i.mail.ru (smtp60.i.mail.ru [217.69.128.40])
 (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
 (No client certificate requested)
 by dev.tarantool.org (Postfix) with ESMTPS id 18149469719
 for <tarantool-patches@dev.tarantool.org>;
 Fri, 14 Feb 2020 02:26:07 +0300 (MSK)
References: <20200213205618.7982-1-gorcunov@gmail.com>
From: Vladislav Shpilevoy <v.shpilevoy@tarantool.org>
Message-ID: <1bec5998-d6de-e549-3883-ae21ab071ce3@tarantool.org>
Date: Fri, 14 Feb 2020 00:26:05 +0100
MIME-Version: 1.0
In-Reply-To: <20200213205618.7982-1-gorcunov@gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Subject: Re: [Tarantool-patches] [PATCH v7 0/2] fiber: Handle stack
	madvise/mprotect errors
List-Id: Tarantool development patches <tarantool-patches.dev.tarantool.org>
List-Unsubscribe: <https://lists.tarantool.org/mailman/options/tarantool-patches>, 
 <mailto:tarantool-patches-request@dev.tarantool.org?subject=unsubscribe>
List-Archive: <https://lists.tarantool.org/pipermail/tarantool-patches/>
List-Post: <mailto:tarantool-patches@dev.tarantool.org>
List-Help: <mailto:tarantool-patches-request@dev.tarantool.org?subject=help>
List-Subscribe: <https://lists.tarantool.org/mailman/listinfo/tarantool-patches>, 
 <mailto:tarantool-patches-request@dev.tarantool.org?subject=subscribe>
To: Cyrill Gorcunov <gorcunov@gmail.com>, tml <tarantool-patches@dev.tarantool.org>

Thanks for the fixes!

Please, put branch and issue link here next time.
It is really hard to find that branch just by commit
title, or by searching for older versions of the
branch in email history. Especially taking into
account that there are 7 versions of mprotect branches.

Btw, you can force push into one branch, it is ok, and
is even more convenient. At least from reviewer's point
of view.

On 13/02/2020 21:56, Cyrill Gorcunov wrote:
> Changes in v7:
>  - address Vlad's comments
>  - add test for slab leak when mprotect fails on exit path
> 
> Still the beautifying of errinj is send as a separate patch
> so diff is pretty big for now.
> 
> Cyrill Gorcunov (2):
>   fiber: set diagnostics at madvise/mprotect failure
>   fiber: leak slab if unable to bring prots back
> 
>  src/lib/core/errinj.h        |   2 +
>  src/lib/core/fiber.c         |  93 +++++++++++++++++++-----
>  test/box/errinj.result       | 136 ++++++++++++++++++-----------------
>  test/unit/CMakeLists.txt     |   4 ++
>  test/unit/fiber_stack.c      | 102 ++++++++++++++++++++++++++
>  test/unit/fiber_stack.result |  10 +++
>  test/unit/suite.ini          |   2 +-
>  7 files changed, 264 insertions(+), 85 deletions(-)
>  create mode 100644 test/unit/fiber_stack.c
>  create mode 100644 test/unit/fiber_stack.result
>