Re: [Tarantool-patches] [PATCH v8 2/2] fiber: leak slab if unable to bring prots back

[Alexander Turenko <alexander.turenko@tarantool.org>]

Guys, please don't remove me from CC in discussions, where I
participated.

On Fri, Feb 14, 2020 at 05:31:35PM +0300, Cyrill Gorcunov wrote:
> On Fri, Feb 14, 2020 at 05:29:19PM +0300, Konstantin Osipov wrote:
> > > 
> > > As far as I understand we gonna drop guard pages for release
> > > builds only, for debug builds it will be with us, no?
> > 
> > Panic in debug is ok.
> 
> Thanks for clarification, Kostya! I'll rework the series once time permit.

We using guard page for a long time (f3155daa 'Add a guard page to the
end fiber stack', 2015) and reconsidering the approach as well as
reimplementation may took significant time, may be stuck due to some
reason or be rejected (for debug builds or at all). It also requires
laborious performance testing. I propose to don't block this bugfix.

For panic on debug: it is okay, but not good, when we able to operate
further. In some rare cases it may even affect customers, when we're
working with them to catch a problem on a debug build and it spins on
some part of production servers under load for a while.

When we detect that we made something wrong, assertion fail looks good.
When we detect that a system resource is exhausted and we able to
proceed with this situation w/o data corruption, it is better to do the
same thing in both release and debug builds.

WBR, Alexander Turenko.