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

Application stops responding upon calling SceneTree.Quit and never really quits #96978

Closed
HKunogi opened this issue Sep 13, 2024 · 3 comments · Fixed by #96959
Closed

Application stops responding upon calling SceneTree.Quit and never really quits #96978

HKunogi opened this issue Sep 13, 2024 · 3 comments · Fixed by #96959

Comments

@HKunogi
Copy link
Contributor

HKunogi commented Sep 13, 2024

Tested versions

  • Reproducible on v4.4.dev.mono.custom_build [74de05a] when using Run on Separate Thread for either or both of the physics (2D/3D)

System information

Windows 11 x64 and Ubuntu 24.04 ARM64

Issue description

When calling the SceneTree.Quit, instead of correctly quitting the application, instead, the application stops responding and does not actually quit, it stays lingering into an non-responsive state, may it be running from console only or with actual UI, as long as "Run on Separate Thread" is ticked on either of the physics (2D and/or 3D).

Steps to reproduce

  • Enable Run on Separate Thread on project settings
  • Simply call SceneTree.Quit

Minimal reproduction project (MRP)

mrp-quitbug.zip

@matheusmdx
Copy link
Contributor

Bisecting points to commit 2d1dd41 from pr #96760 as the culprit

image


We had a similar issue #96931 with a pr to fix: #96959 but only talks about Rendering and Script server so just to make sure, @RandomShaper pr #96959 also works for this case?

@CedNaru
Copy link
Contributor

CedNaru commented Sep 13, 2024

That PR will work on anything that runs on the WorkerThreadPool, which includes the physic servers too.

@clayjohn
Copy link
Member

This is indeed a duplicate of #96931 and will be fixed by #96959

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

Successfully merging a pull request may close this issue.

5 participants