Leszek, you were right. The breakage went away once we defined 
V8_31BIT_SMIS_ON_64BIT_ARCH in our embedder code.

As an aside, it would be great if V8's public headers were automatically 
aligned with the V8 build. These mismatches can be difficult to diagnose.

Anyway, we're now testing this configuration for the memory leak. Thanks 
again.

On Friday, February 14, 2020 at 11:44:26 AM UTC-5, [email protected] 
wrote:
>
> >That sounds like a build config mismatch, or something similar -- 31-bit 
> Smis shouldn't cause major breakage, but will if e.g. one header has the 
> build flag enabled and another doesn't.
>
> Thanks, Leszek; we'll double-check our findings.
>
> On Friday, February 14, 2020 at 9:46:49 AM UTC-5, Leszek Swirski wrote:
>>
>> That sounds like a build config mismatch, or something similar -- 31-bit 
>> Smis shouldn't cause major breakage, but will if e.g. one header has the 
>> build flag enabled and another doesn't.
>>
>> On Fri, Feb 14, 2020 at 3:43 PM <[email protected]> wrote:
>>
>>> Sorry, here's a clarification. This is what we're seeing with 8.0:
>>>
>>> Pointer Compression ON, 31-bit Smis ON -> Memory leak
>>> Pointer Compression ON, 31-bit Smis OFF -> Memory leak
>>> Pointer Compression OFF, 31-bit Smis ON -> Major breakage
>>> Pointer Compression OFF, 31-bit Smis OFF -> All good
>>>
>>> On Friday, February 14, 2020 at 9:33:10 AM UTC-5, Santiago Aboy Solanes 
>>> wrote:
>>>>
>>>> Do you mean that v8_enable_pointer_compression = *false* and 
>>>> v8_enable_31bit_smis_on_64bit_arch = *true* "yielded a V8 build that 
>>>> broke a majority of our test suite, including this memory leak test"?
>>>> If that's the case, this is not a pointer compression failure.
>>>>
>>>> Pointer Compression needs 31 bit smis, but 31 bit smis doesn't need 
>>>> Pointer Compression.
>>>>
>>>> On Fri, Feb 14, 2020 at 2:29 PM <[email protected]> wrote:
>>>>
>>>>> That particular combination yielded a V8 build that broke a majority 
>>>>> of our test suite, including this memory leak test.
>>>>>
>>>>> We didn't spend much time investigating, but the build utterly broke 
>>>>> interception (e.g., bad data in FunctionCallbackInfo), so we abandoned it.
>>>>>
>>>>> On Friday, February 14, 2020 at 5:13:35 AM UTC-5, Santiago Aboy 
>>>>> Solanes wrote:
>>>>>>
>>>>>> Quick question: Have you tried with v8_enable_pointer_compression = 
>>>>>> false and v8_enable_31bit_smis_on_64bit_arch = true?
>>>>>>
>>>>>> On Fri, Feb 14, 2020 at 7:54 AM Yang Guo <[email protected]> wrote:
>>>>>>
>>>>>>> +Ulan Degenbaev +Igor Sheludko 
>>>>>>>
>>>>>>> On Thu, Feb 13, 2020 at 8:36 PM <[email protected]> wrote:
>>>>>>>
>>>>>>>> Greetings!
>>>>>>>>
>>>>>>>> Our project is a V8 embedding library for .NET on Windows. We have 
>>>>>>>> a test that spins up a context, runs SunSpider, and destroys the 
>>>>>>>> isolate. 
>>>>>>>> We run it in a loop to check for memory leaks.
>>>>>>>>
>>>>>>>> This test showed a dramatic leak with V8 8.0 that wasn't there with 
>>>>>>>> 7.9. Our tools found nothing suspicious in the managed and native 
>>>>>>>> heaps, 
>>>>>>>> but the process's private memory size kept increasing.
>>>>>>>>
>>>>>>>> Based on some quick research and a little guesswork, we rebuilt 
>>>>>>>> with v8_enable_pointer_compression and 
>>>>>>>> v8_enable_31bit_smis_on_64bit_arch 
>>>>>>>> both set to false. To our surprise this completely eliminated the leak!
>>>>>>>>
>>>>>>>> We'd like to report this issue to the V8 team, but we have no 
>>>>>>>> simple native repro code. Can someone suggest a way to gather some 
>>>>>>>> useful 
>>>>>>>> diagnostics?
>>>>>>>>
>>>>>>>> Thanks!
>>>>>>>>
>>>>>>>> -- 
>>>>>>>> -- 
>>>>>>>> v8-dev mailing list
>>>>>>>> [email protected]
>>>>>>>> http://groups.google.com/group/v8-dev
>>>>>>>> --- 
>>>>>>>> You received this message because you are subscribed to the Google 
>>>>>>>> Groups "v8-dev" group.
>>>>>>>> To unsubscribe from this group and stop receiving emails from it, 
>>>>>>>> send an email to [email protected].
>>>>>>>> To view this discussion on the web visit 
>>>>>>>> https://groups.google.com/d/msgid/v8-dev/6325eb1a-2a8e-431d-a7ee-5e7f0a41f046%40googlegroups.com
>>>>>>>>  
>>>>>>>> <https://groups.google.com/d/msgid/v8-dev/6325eb1a-2a8e-431d-a7ee-5e7f0a41f046%40googlegroups.com?utm_medium=email&utm_source=footer>
>>>>>>>> .
>>>>>>>>
>>>>>>> -- 
>>>>>>> -- 
>>>>>>> v8-dev mailing list
>>>>>>> [email protected]
>>>>>>> http://groups.google.com/group/v8-dev
>>>>>>> --- 
>>>>>>> You received this message because you are subscribed to the Google 
>>>>>>> Groups "v8-dev" group.
>>>>>>> To unsubscribe from this group and stop receiving emails from it, 
>>>>>>> send an email to [email protected].
>>>>>>> To view this discussion on the web visit 
>>>>>>> https://groups.google.com/d/msgid/v8-dev/CAFSTc_iWyBOpP5xEEM6EZk_pLZhyeGoK0WrgM-L8M9%3D6xenyUQ%40mail.gmail.com
>>>>>>>  
>>>>>>> <https://groups.google.com/d/msgid/v8-dev/CAFSTc_iWyBOpP5xEEM6EZk_pLZhyeGoK0WrgM-L8M9%3D6xenyUQ%40mail.gmail.com?utm_medium=email&utm_source=footer>
>>>>>>> .
>>>>>>>
>>>>>> -- 
>>>>> -- 
>>>>> v8-dev mailing list
>>>>> [email protected]
>>>>> http://groups.google.com/group/v8-dev
>>>>> --- 
>>>>> You received this message because you are subscribed to the Google 
>>>>> Groups "v8-dev" group.
>>>>> To unsubscribe from this group and stop receiving emails from it, send 
>>>>> an email to [email protected].
>>>>> To view this discussion on the web visit 
>>>>> https://groups.google.com/d/msgid/v8-dev/2388a332-c093-4d27-9c4b-67bd21a07064%40googlegroups.com
>>>>>  
>>>>> <https://groups.google.com/d/msgid/v8-dev/2388a332-c093-4d27-9c4b-67bd21a07064%40googlegroups.com?utm_medium=email&utm_source=footer>
>>>>> .
>>>>>
>>>> -- 
>>> -- 
>>> v8-dev mailing list
>>> [email protected]
>>> http://groups.google.com/group/v8-dev
>>> --- 
>>> You received this message because you are subscribed to the Google 
>>> Groups "v8-dev" group.
>>> To unsubscribe from this group and stop receiving emails from it, send 
>>> an email to [email protected].
>>> To view this discussion on the web visit 
>>> https://groups.google.com/d/msgid/v8-dev/e5e207c5-880d-4137-8efa-68d1ad4c9d62%40googlegroups.com
>>>  
>>> <https://groups.google.com/d/msgid/v8-dev/e5e207c5-880d-4137-8efa-68d1ad4c9d62%40googlegroups.com?utm_medium=email&utm_source=footer>
>>> .
>>>
>>

-- 
-- 
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- 
You received this message because you are subscribed to the Google Groups 
"v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/v8-dev/22b51632-8549-4cd5-a17b-02b7fe77a8f6%40googlegroups.com.

Reply via email to