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

Infinite loops on uncaught exceptions #58

Open
cwebber opened this issue Apr 27, 2022 · 5 comments
Open

Infinite loops on uncaught exceptions #58

cwebber opened this issue Apr 27, 2022 · 5 comments
Labels

Comments

@cwebber
Copy link
Collaborator

cwebber commented Apr 27, 2022

In switching guile-goblins from Fibers 1.0 to 1.10, I noticed that when exceptions occured, schedulers suddenly stopped working but also my cpu would shoot up to 100%. That's strange, that shouldn't happen, what gives?

I did a git-bisect against fibers to try to figure out where this started. The issue is introduced in 84addfb. I haven't checked for sure, but I'm pretty certain that what's happening is that where the scheduler previously was catching the exception in the individual task but the scheduler would continue looping, now the scheduler is breaking and not continuing on an individual error. Thus, it isn't feeding anything to the individual fibers, which start spin locking madly.

@cwebber
Copy link
Collaborator Author

cwebber commented Apr 28, 2022

One option is simply to revert 84addfb. Maybe @wingo might not like this; it fixes it, but it re-introduces whatever performance issue the commit was meant to address.

@aconchillo
Copy link
Collaborator

IMO, stability is always preferred over performance.

@wingo
Copy link
Owner

wingo commented Apr 29, 2022

Do you have a test case?

@aconchillo
Copy link
Collaborator

@cwebber Probably completely unrelated, but I'm wondering if this commit c25dcb9 would fix the issue you are seeing. Available in Fibers 1.1.1.

@wingo
Copy link
Owner

wingo commented Jun 8, 2023

Pretty sure this is fixed via #76, which is now in Guile main. Christine can you give a try with Guile from git and see if you are still having issues? Or otherwise close this one if you think it is fixed.

@emixa-d emixa-d added the bug label Sep 4, 2023
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

4 participants