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

adding mimalloc breaks running on Windows 10 #657

Closed
steel-ne opened this issue Jul 22, 2020 · 14 comments
Closed

adding mimalloc breaks running on Windows 10 #657

steel-ne opened this issue Jul 22, 2020 · 14 comments
Labels

Comments

@steel-ne
Copy link

System information

SolveSpace version: 3.0~cbac182e
next commit is broken

Operating system: Windows 10

Expected behavior

solvespace runs and shows GUI on windows 10

Actual behavior

solvespace is started and shows no GUI, task is present in task manager

Additional information

commit cbac182 is good
commit c4ca4be is bad

@whitequark whitequark added the bug label Jul 22, 2020
@whitequark
Copy link
Contributor

Can't reproduce. Could you provide more details?

@nabijaczleweli
Copy link
Contributor

nabijaczleweli commented Jul 22, 2020 via email

@whitequark
Copy link
Contributor

Works fine for me under Wine and a Win10 VM as well.

@nabijaczleweli
Copy link
Contributor

nabijaczleweli commented Jul 22, 2020 via email

@steel-ne
Copy link
Author

I've downloaded the last snapshot build from https://ci.appveyor.com/project/whitequark/solvespace/build/artifacts
Doubleclick .exe file, nothing happened, but in the Task manager it appears as "SolveSpace, a parametric 2d/3d CAD (32 bit)"

So I start Git bisect, building on Windows with Visual studio 2019

Windows 10 Enterprise v1909 build 18363.959

It happens on HP notebook too (build 18363.778)

I have domain logons on both devices with some restrictions (can't install/run some programs, etc).

@whitequark
Copy link
Contributor

building on Windows with Visual studio 2019

Since you have Visual Studio, can you attach a debugger and determine the backtrace for your hang?

@steel-ne
Copy link
Author

steel-ne commented Jul 22, 2020

I am not so familiar with VS, here is call stack of thread main:

 	ntdll.dll!_NtWaitForSingleObject@12�()	Unknown
 	ntdll.dll!LdrpDrainWorkQueue()	Unknown
 	ntdll.dll!_LdrUnloadDll@4�()	Unknown
 	KernelBase.dll!FreeLibrary()	Unknown
 	ig9icd32.dll!686e7843()	Unknown
 	ig9icd32.dll![Frames below may be incorrect and/or missing, no symbols loaded for ig9icd32.dll]	Unknown
 	ig9icd32.dll!688950b0()	Unknown
 	ig9icd32.dll!6870fe96()	Unknown
 	ig9icd32.dll!686cf19e()	Unknown
 	opengl32.dll!6935d1df()	Unknown

>	solvespace.exe!rx::DisplayWGL::terminate() Line 374	C++
 	solvespace.exe!egl::Display::terminate() Line 463	C++
 	solvespace.exe!egl::Terminate(void * dpy) Line 94	C++
 	solvespace.exe!eglTerminate(void * dpy) Line 118	C++
 	solvespace.exe!SolveSpace::Platform::WindowImplWin32::WindowImplWin32(SolveSpace::Platform::Window::Kind kind, std::shared_ptr<SolveSpace::Platform::WindowImplWin32> parentWindow) Line 647	C++
 	[External Code]	
 	solvespace.exe!SolveSpace::Platform::CreateWindow(SolveSpace::Platform::Window::Kind kind, std::shared_ptr<SolveSpace::Platform::Window> parentWindow) Line 1380	C++
 	solvespace.exe!SolveSpace::GraphicsWindow::Init() Line 413	C++
 	solvespace.exe!SolveSpace::SolveSpaceUI::AfterNewFile() Line 437	C++
 	solvespace.exe!SolveSpace::SolveSpaceUI::Init() Line 141	C++
 	solvespace.exe!main(int argc, char * * argv) Line 19	C++
 	solvespace.exe!WinMain(HINSTANCE__ * hInstance, HINSTANCE__ * hPrevInstance, char * lpCmdLine, int nCmdShow) Line 40	C++
 	[External Code]	

@whitequark
Copy link
Contributor

Thanks, that's useful.

@steel-ne
Copy link
Author

here is the callstack with "External code" unwrapped:

 	ntdll.dll!_NtWaitForSingleObject@12�()	Unknown
 	ntdll.dll!LdrpDrainWorkQueue()	Unknown
 	ntdll.dll!_LdrUnloadDll@4�()	Unknown
 	KernelBase.dll!FreeLibrary()	Unknown
 	ig9icd32.dll!686e7843()	Unknown
 	ig9icd32.dll![Frames below may be incorrect and/or missing, no symbols loaded for ig9icd32.dll]	Unknown
 	ig9icd32.dll!688950b0()	Unknown
 	ig9icd32.dll!6870fe96()	Unknown
 	ig9icd32.dll!686cf19e()	Unknown
 	opengl32.dll!6935d1df()	Unknown
>	solvespace.exe!rx::DisplayWGL::terminate() Line 374	C++
 	solvespace.exe!egl::Display::terminate() Line 463	C++
 	solvespace.exe!egl::Terminate(void * dpy) Line 94	C++
 	solvespace.exe!eglTerminate(void * dpy) Line 118	C++
 	solvespace.exe!SolveSpace::Platform::WindowImplWin32::WindowImplWin32(SolveSpace::Platform::Window::Kind kind, std::shared_ptr<SolveSpace::Platform::WindowImplWin32> parentWindow) Line 647	C++
 	solvespace.exe!std::_Ref_count_obj<SolveSpace::Platform::WindowImplWin32>::_Ref_count_obj<SolveSpace::Platform::WindowImplWin32><enum SolveSpace::Platform::Window::Kind &,std::shared_ptr<SolveSpace::Platform::WindowImplWin32>>(SolveSpace::Platform::Window::Kind & <_Args_0>, std::shared_ptr<SolveSpace::Platform::WindowImplWin32> && <_Args_1>) Line 901	C++
 	solvespace.exe!std::make_shared<SolveSpace::Platform::WindowImplWin32,enum SolveSpace::Platform::Window::Kind &,std::shared_ptr<SolveSpace::Platform::WindowImplWin32>>(SolveSpace::Platform::Window::Kind & <_Args_0>, std::shared_ptr<SolveSpace::Platform::WindowImplWin32> && <_Args_1>) Line 971	C++
 	solvespace.exe!SolveSpace::Platform::CreateWindow(SolveSpace::Platform::Window::Kind kind, std::shared_ptr<SolveSpace::Platform::Window> parentWindow) Line 1380	C++
 	solvespace.exe!SolveSpace::GraphicsWindow::Init() Line 413	C++
 	solvespace.exe!SolveSpace::SolveSpaceUI::AfterNewFile() Line 437	C++
 	solvespace.exe!SolveSpace::SolveSpaceUI::Init() Line 141	C++
 	solvespace.exe!main(int argc, char * * argv) Line 19	C++
 	solvespace.exe!WinMain(HINSTANCE__ * hInstance, HINSTANCE__ * hPrevInstance, char * lpCmdLine, int nCmdShow) Line 40	C++
 	solvespace.exe!invoke_main() Line 99	C++
 	solvespace.exe!__scrt_common_main_seh() Line 253	C++
 	solvespace.exe!__scrt_common_main() Line 296	C++
 	solvespace.exe!WinMainCRTStartup() Line 17	C++
 	kernel32.dll!@BaseThreadInitThunk@12�()	Unknown
 	ntdll.dll!__RtlUserThreadStart()	Unknown
 	ntdll.dll!__RtlUserThreadStart@8�()	Unknown

@whitequark
Copy link
Contributor

@nabijaczleweli I have a suspicion of what's happening here. It looks like it hangs in FreeLibrary. This likely happens because a mimalloc heap is created (because of the way C++ thread locals work) for every thread, including those in the Intel integrated graphics driver, and for some reason this causes a deadlock. I think we can fix it by lazily creating a heap.

@whitequark
Copy link
Contributor

@steel-ne Does this help?

diff --git a/src/platform/platform.cpp b/src/platform/platform.cpp
index 934220ca..a0809821 100644
--- a/src/platform/platform.cpp
+++ b/src/platform/platform.cpp
@@ -684,20 +684,21 @@ void DebugPrint(const char *fmt, ...) {
 //-----------------------------------------------------------------------------
 
 struct MimallocHeap {
-    mi_heap_t *heap = mi_heap_new();
-
-    MimallocHeap() {
-        ssassert(heap != NULL, "out of memory");
-    }
+    mi_heap_t *heap = NULL;
 
     ~MimallocHeap() {
-        mi_heap_destroy(heap);
+        if(heap != NULL)
+            mi_heap_destroy(heap);
     }
 };
 
 static thread_local MimallocHeap TempArena;
 
 void *AllocTemporary(size_t size) {
+    if(TempArena.heap == NULL) {
+        TempArena.heap = mi_heap_new();
+        ssassert(TempArena.heap != NULL, "out of memory");
+    }
     void *ptr = mi_heap_zalloc(TempArena.heap, size);
     ssassert(ptr != NULL, "out of memory");
     return ptr;

@steel-ne
Copy link
Author

Yes, it works!

@nabijaczleweli
Copy link
Contributor

Interestingly, none of the laptops nor my desktop currently have dedicated GPUs in them; just driver weirdness, I guess?

@whitequark
Copy link
Contributor

just driver weirdness, I guess?

Yeah. I think it actually oughtn't crash even the way we wrote it, it might be actually a bug in the driver. But it's good to be defensive here because drivers are full of bugs :)

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

No branches or pull requests

3 participants