[Tarantool-patches] [PATCH luajit] memprof: report stack resizing as internal event

Sergey Ostanevich sergos at tarantool.org
Mon Mar 29 19:01:06 MSK 2021


Hi!

Had a good time reading all your comments, thanks!

As for the reason I still not blessed this - I got good 
explanation why stack resizing should be guarded for the whole
part of the code it was in original patch, so

LGTM.

Sergos

> On 29 Mar 2021, at 11:35, Igor Munkin <imun at tarantool.org> wrote:
> 
> On 29.03.21, Igor Munkin wrote:
>> Sergey,
>> 
>> On 29.03.21, Sergey Kaplun wrote:
>>> Igor,
>>> 
>> 
>> <snipped>
>> 
>>>>>> 
>>>>>> "Feel free to prove the opposite"(c) :)
>>>>>> 
>>>>>> The current naming is much better than the previous one, but I still
>>>>>> propose to save the original on and save G(L) into a new variable.
>>>>> 
>>>>> Ignoring for now.
>>>> 
>>>> Well, if you just need my LGTM, you got it. I'll even push this patch by
>>>> myself. But try to hear my arguments even if they look like "bred deda".
>>> 
>>> This is not about LGTM, but about a miscommunication, I hope we will
>>> understand each other's point of view and come to a consensus.
>> 
>> I see no changes on the branch, hence we won't. Anyway, you have the
>> necessary amount of LGTMs, so I'll push your patch to the trunk later.
> 
> Oops, seems you have only mine (at least I see no explicit LGTM from
> Sergos in the thread). Will not rush then.
> 
>> 
>>> 
>> 
>> <snipped>
>> 
>>> 
>>> -- 
>>> Best regards,
>>> Sergey Kaplun
>> 
>> -- 
>> Best regards,
>> IM
> 
> -- 
> Best regards,
> IM

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.tarantool.org/pipermail/tarantool-patches/attachments/20210329/80c8c526/attachment.htm>


More information about the Tarantool-patches mailing list