-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
DWARF: Disable GC around dl_iterate_phdr in freebsd #10259
Conversation
What about #10084 (comment)? |
I think that would require iterating the structure twice, to count the required memory. Disabling the GC seems a less error prone approach to me. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe add a TODO comment? This is merely a workaround and we should keep looking for a proper fix with LLVM probably, right?
@straight-shoota mmm, no. There is no llvm issue here. We are preventing a deadlock in freebsd. |
Okay, if that's all. Sry, I couldn't look into it more closely, just rememberd your initial hunch it might be caused by LLVM. |
ChangeLog: https://github.com/crystal-lang/crystal/blob/0.36.1/CHANGELOG.md Should fix this issue: crystal-lang/crystal#10259 PR: 253111 Submitted by: rlwestlund@gmail.com Approved by: greg@unrelenting.technology (maintainer) MFH: 2021Q1 (runtime fix) git-svn-id: svn+ssh://svn.freebsd.org/ports/head@565009 35697150-7ecd-e111-bb59-0022644237b5
ChangeLog: https://github.com/crystal-lang/crystal/blob/0.36.1/CHANGELOG.md Should fix this issue: crystal-lang/crystal#10259 PR: 253111 Submitted by: rlwestlund@gmail.com Approved by: greg@unrelenting.technology (maintainer) MFH: 2021Q1 (runtime fix)
lang/crystal: Update to 0.36.1 ChangeLog: https://github.com/crystal-lang/crystal/blob/0.36.1/CHANGELOG.md Should fix this issue: crystal-lang/crystal#10259 PR: 253111 Submitted by: rlwestlund@gmail.com Approved by: greg@unrelenting.technology (maintainer)
ChangeLog: https://github.com/crystal-lang/crystal/blob/0.36.1/CHANGELOG.md Should fix this issue: crystal-lang/crystal#10259 PR: 253111 Submitted by: rlwestlund@gmail.com Approved by: greg@unrelenting.technology (maintainer) MFH: 2021Q1 (runtime fix) git-svn-id: svn+ssh://svn.freebsd.org/ports/head@565009 35697150-7ecd-e111-bb59-0022644237b5
ChangeLog: https://github.com/crystal-lang/crystal/blob/0.36.1/CHANGELOG.md Should fix this issue: crystal-lang/crystal#10259 PR: 253111 Submitted by: rlwestlund@gmail.com Approved by: greg@unrelenting.technology (maintainer) MFH: 2021Q1 (runtime fix)
lang/crystal: Update to 0.36.1 ChangeLog: https://github.com/crystal-lang/crystal/blob/0.36.1/CHANGELOG.md Should fix this issue: crystal-lang/crystal#10259 PR: 253111 Submitted by: rlwestlund@gmail.com Approved by: greg@unrelenting.technology (maintainer)
ChangeLog: https://github.com/crystal-lang/crystal/blob/0.36.1/CHANGELOG.md Should fix this issue: crystal-lang/crystal#10259 PR: 253111 Submitted by: rlwestlund@gmail.com Approved by: greg@unrelenting.technology (maintainer) MFH: 2021Q1 (runtime fix)
Fixes #10084. Prevents deadlock as shown in #10084 (comment)
I don't know if it's a good idea to disable it in every platform or in every bsd platform.