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.

Sign up for GitHub

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

Jump to bottom

Missing GC root somewhere #36649

Closed
Keno opened this issue Jul 14, 2020 · 0 comments · Fixed by #36650
Closed

Missing GC root somewhere #36649

Keno opened this issue Jul 14, 2020 · 0 comments · Fixed by #36650

Comments

@Keno
Copy link
Member

Keno commented Jul 14, 2020

We have a missing GC root somewhere, as seen on CI https://build.julialang.org/#/builders/77/builds/1618/steps/8/logs/stdio

Please submit a bug report with steps to reproduce this fault, and any error messages that follow (in their entirety). Thanks.
Exception: EXCEPTION_ACCESS_VIOLATION at 0x70532ab4 -- gc_try_setmark at /cygdrive/d/buildbot/worker/package_win32/build/src\gc.c:1643 [inlined]
gc_mark_scan_obj8 at /cygdrive/d/buildbot/worker/package_win32/build/src\gc.c:1837 [inlined]
gc_mark_loop at /cygdrive/d/buildbot/worker/package_win32/build/src\gc.c:2118
in expression starting at loading.jl:1267
gc_try_setmark at /cygdrive/d/buildbot/worker/package_win32/build/src\gc.c:1643 [inlined]
gc_mark_scan_obj8 at /cygdrive/d/buildbot/worker/package_win32/build/src\gc.c:1837 [inlined]
gc_mark_loop at /cygdrive/d/buildbot/worker/package_win32/build/src\gc.c:2118
_jl_gc_collect at /cygdrive/d/buildbot/worker/package_win32/build/src\gc.c:2903
jl_gc_collect at /cygdrive/d/buildbot/worker/package_win32/build/src\gc.c:3109
maybe_collect at /cygdrive/d/buildbot/worker/package_win32/build/src\gc.c:828 [inlined]
jl_gc_pool_alloc at /cygdrive/d/buildbot/worker/package_win32/build/src\gc.c:1143 [inlined]
jl_gc_alloc_ at /cygdrive/d/buildbot/worker/package_win32/build/src\julia_internal.h:277 [inlined]
jl_gc_alloc at /cygdrive/d/buildbot/worker/package_win32/build/src\gc.c:3151
jl_alloc_svec_uninit at /cygdrive/d/buildbot/worker/package_win32/build/src\simplevector.c:60
inst_datatype_inner at /cygdrive/d/buildbot/worker/package_win32/build/src\jltypes.c:1396
inst_datatype_env at /cygdrive/d/buildbot/worker/package_win32/build/src\jltypes.c:895 [inlined]
inst_datatype_env at /cygdrive/d/buildbot/worker/package_win32/build/src\jltypes.c:899 [inlined]
inst_datatype_env at /cygdrive/d/buildbot/worker/package_win32/build/src\jltypes.c:899 [inlined]
inst_datatype_env at /cygdrive/d/buildbot/worker/package_win32/build/src\jltypes.c:899 [inlined]
Keno added a commit that referenced this issue Jul 14, 2020
@Keno
Move instance initialization into jl_uninitialized_datatype
a926005 
Otherwise it's illegal to hold onto this object without having
set ->instance, which is just asking for crashes. Fixes #36649.
@Keno Keno mentioned this issue Jul 14, 2020
Merged
Keno referenced this issue Jul 14, 2020
@vtjnash
Merge pull request #36612 from JuliaLang/jb/bigexprs
110765a 
some improvements for lowering of large expressions
Keno added a commit that referenced this issue Jul 14, 2020
@Keno
Move instance initialization into jl_uninitialized_datatype
d778a3d 
Otherwise it's illegal to hold onto this object without having
set ->instance, which is just asking for crashes. Fixes #36649.
JeffBezanson pushed a commit that referenced this issue Jul 21, 2020
@Keno @JeffBezanson
Move instance initialization into jl_uninitialized_datatype
8ac1a3a 
Otherwise it's illegal to hold onto this object without having
set ->instance, which is just asking for crashes. Fixes #36649.

(cherry picked from commit d778a3d)
simeonschaub pushed a commit to simeonschaub/julia that referenced this issue Aug 11, 2020
@Keno @simeonschaub
Move instance initialization into jl_uninitialized_datatype
fc166f1 
Otherwise it's illegal to hold onto this object without having
set ->instance, which is just asking for crashes. Fixes JuliaLang#36649.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix GC corruption during NamedTuple instantiation JuliaLang/julia
1 participant
@Keno