Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Some SysAllocator replacements do not work with memory decommiting on windows (was: system-alloc_unittest tests fail in MSVC2013 under Win8.1 x64) #665

Open
GoogleCodeExporter opened this issue Jul 23, 2015 · 3 comments

Comments

@GoogleCodeExporter
Copy link

Using the latest release of v2.4 I got the following errors running the tests 
compiled in MSVC2013-Upd4 under Windows 8.1-64bit:

C:\Google\gperftools-2.4\x64\Debug>addr2line-pdb.exe
SymLoadModuleEx returned error : 6 for a.out

C:\Google\gperftools-2.4\x64\Debug>system-alloc_unittest.exe
Assertion failed: success == TRUE, file ..\..\src\windows\system-alloc.cc, line 
166
(-> And then the whole shell froze and had to be killed)

Original issue reported on code.google.com by patle...@gmail.com on 14 Jan 2015 at 10:30

@GoogleCodeExporter
Copy link
Author

add2line-pdf.exe is not a test.

But system-alloc_unittest is indeed failing.

This is bug in our handling of custom SysAllocator instances and releasing 
memory back to OS on windows.

You can largely ignore this unless you're need to deal with custom SysAllocator.

Original comment by alkondratenko on 18 Jan 2015 at 2:28

@GoogleCodeExporter
Copy link
Author

Updating ticket title.

So assertion fails because test's SysAllocator returns memory that is backed by 
array. For that memory MEM_DECOMMIT doesn't work which causes assertion.

Original comment by alkondratenko on 18 Jan 2015 at 2:51

  • Changed title: Some SysAllocator replacements do not work with memory decommiting on windows (was: system-alloc_unittest tests fail in MSVC2013 under Win8.1 x64)

@GoogleCodeExporter
Copy link
Author

Issue 681 has been merged into this issue.

Original comment by alkondratenko on 15 Apr 2015 at 3:03

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant