Re: [v8-dev] Pointer compression seemingly responsible for large memory leak

2020-02-20 Thread clearscriptlib
Hmm... Our latest findings are quite baffling: * Memory usage levels off after several thousand test iterations, so it may not be a true leak. We've tested up to 12000 iterations, and memory usage never grew beyond ~85MB. Graphing a moving average clearly indicated an asymptotic curve. Still,

Re: [v8-dev] Pointer compression seemingly responsible for large memory leak

2020-02-14 Thread clearscriptlib
Hi Igor, >Could you please open an issue here: https://bugs.chromium.org/p/v8/issues/ Sure thing; we were just hoping to collect some useful diagnostic info, since we don't have simple repro code. >Could you please try it again on today's tip of tree V8 (say

Re: [v8-dev] Pointer compression seemingly responsible for large memory leak

2020-02-14 Thread clearscriptlib
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

Re: [v8-dev] Pointer compression seemingly responsible for large memory leak

2020-02-14 Thread clearscriptlib
>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

Re: [v8-dev] Pointer compression seemingly responsible for large memory leak

2020-02-14 Thread clearscriptlib
We suspected that, but tested it anyway :) On Friday, February 14, 2020 at 9:54:28 AM UTC-5, Santiago Aboy Solanes wrote: > > Adding to that, this configuration > Pointer Compression ON, 31-bit Smis OFF -> Memory leak > is the same as > Pointer Compression ON, 31-bit Smis ON -> Memory leak >

Re: [v8-dev] Pointer compression seemingly responsible for large memory leak

2020-02-14 Thread 'Santiago Aboy Solanes' via v8-dev
Adding to that, this configuration Pointer Compression ON, 31-bit Smis OFF -> Memory leak is the same as Pointer Compression ON, 31-bit Smis ON -> Memory leak since Pointer Compression forces 31 bit smis

Re: [v8-dev] Pointer compression seemingly responsible for large memory leak

2020-02-14 Thread Leszek Swirski
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 wrote: > Sorry, here's a clarification. This is what we're seeing with 8.0: >

Re: [v8-dev] Pointer compression seemingly responsible for large memory leak

2020-02-14 Thread clearscriptlib
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

Re: [v8-dev] Pointer compression seemingly responsible for large memory leak

2020-02-14 Thread 'Santiago Aboy Solanes' via v8-dev
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

Re: [v8-dev] Pointer compression seemingly responsible for large memory leak

2020-02-14 Thread clearscriptlib
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

Re: [v8-dev] Pointer compression seemingly responsible for large memory leak

2020-02-14 Thread 'Igor Sheludko' via v8-dev
Could you please open an issue here: https://bugs.chromium.org/p/v8/issues/ And yet another suggestion. Could you please try it again on today's tip of tree V8 (say 96f6ac9bc6646ae12acad43a1f9ebdaaf7ceea0a)? Does your memory leak tool give any hints about the source of the memory leak? How much

Re: [v8-dev] Pointer compression seemingly responsible for large memory leak

2020-02-14 Thread 'Santiago Aboy Solanes' via v8-dev
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 wrote: > +Ulan Degenbaev +Igor Sheludko > > On Thu, Feb 13, 2020 at 8:36 PM wrote: > >> Greetings! >> >> Our project is a V8

Re: [v8-dev] Pointer compression seemingly responsible for large memory leak

2020-02-13 Thread Yang Guo
+Ulan Degenbaev +Igor Sheludko On Thu, Feb 13, 2020 at 8:36 PM 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