Skip to content

New crash in subarray test on 64-bit #15508

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

Closed
nalimilan opened this issue Mar 14, 2016 · 7 comments
Closed

New crash in subarray test on 64-bit #15508

nalimilan opened this issue Mar 14, 2016 · 7 comments
Labels
test This change adds or pertains to unit tests

Comments

@nalimilan
Copy link
Member

One commit in the range fc469b6...64dbecf (i.e. #15444 by @vtjnash -- again, sorry! :-) introduced a crash in the subarray tests when building the RPM nightlies on 64-bit (but not 32-bit). This is with system LLVM 3.8 plus the Julia patch.

signal (11): Segmentation fault
while loading /builddir/build/BUILD/julia/test/subarray.jl, in expression starting on line 309
[inline] at ./essentials.jl:25
reindex at ./subarray.jl:130
unknown function (ip: 0x7f03698fb260)
[inline] at /builddir/build/BUILD/julia/src/julia_internal.h:69
jl_call_method_internal at /builddir/build/BUILD/julia/src/gf.c:1848
[inline] at ./subarray.jl:91
sub at ./subarray.jl:193
[inline] at /builddir/build/BUILD/julia/src/julia_internal.h:69
jl_call_method_internal at /builddir/build/BUILD/julia/src/gf.c:1848
jl_f__apply at /builddir/build/BUILD/julia/src/builtins.c:512
runtests at /builddir/build/BUILD/julia/test/subarray.jl:221
[inline] at /builddir/build/BUILD/julia/src/julia_internal.h:69
jl_call_method_internal at /builddir/build/BUILD/julia/src/gf.c:1848
[inline] at ./int.jl:28
runviews at /builddir/build/BUILD/julia/test/subarray.jl:249
unknown function (ip: 0x7f0369b236b3)
[inline] at /builddir/build/BUILD/julia/src/julia_internal.h:69
jl_call_method_internal at /builddir/build/BUILD/julia/src/gf.c:1848
[inline] at /builddir/build/BUILD/julia/test/subarray.jl:329
anonymous at ./<no file>:4294967295
unknown function (ip: 0x7f0373dcb420)
[inline] at /builddir/build/BUILD/julia/src/julia_internal.h:69
jl_call_method_internal at /builddir/build/BUILD/julia/src/toplevel.c:552
jl_parse_eval_all at /builddir/build/BUILD/julia/src/ast.c:784
[inline] at ./essentials.jl:79
include_string at ./loading.jl:371
unknown function (ip: 0x7f03b06a3194)
[inline] at /builddir/build/BUILD/julia/src/julia_internal.h:69
jl_call_method_internal at /builddir/build/BUILD/julia/src/gf.c:1848
include_from_node1 at ./loading.jl:420
unknown function (ip: 0x7f05b2593b75)
[inline] at /builddir/build/BUILD/julia/src/julia_internal.h:69
jl_call_method_internal at /builddir/build/BUILD/julia/src/gf.c:1848
[inline] at ./util.jl:180
runtests at /builddir/build/BUILD/julia/test/testdefs.jl:7
#16 at /builddir/build/BUILD/julia/test/runtests.jl:36
unknown function (ip: 0x7f03b0681c33)
[inline] at /builddir/build/BUILD/julia/src/julia_internal.h:69
jl_call_method_internal at /builddir/build/BUILD/julia/src/gf.c:1848
jl_f__apply at /builddir/build/BUILD/julia/src/builtins.c:512
run_work_thunk at ./multi.jl:744
[inline] at ./multi.jl:1040
#255 at ./task.jl:59
[inline] at /builddir/build/BUILD/julia/src/julia_internal.h:69
jl_call_method_internal at /builddir/build/BUILD/julia/src/gf.c:1848
start_task at /builddir/build/BUILD/julia/src/task.c:245
unknown function (ip: (nil))
Allocations: 844523643 (Pool: 844164628; Big: 359015); GC: 2060
    From worker 3:       * subarray             Worker 3 terminated.

https://copr-be.cloud.fedoraproject.org/results/nalimilan/julia-nightlies/fedora-23-x86_64/00168226-julia/build.log.gz

@nalimilan nalimilan added the test This change adds or pertains to unit tests label Mar 14, 2016
@tkelman
Copy link
Contributor

tkelman commented Mar 15, 2016

Which Julia patch for 3.8? With reindex showing up in the backtrace this looks similar to what I was seeing on win64 in #15417

@nalimilan
Copy link
Member Author

I only had D17165, didn't notice the new one. I'm currently retrying with the threads patch.

@tkelman
Copy link
Contributor

tkelman commented Mar 15, 2016

The patch that turns off thread-dependent code in LLVM shouldn't really be necessary anywhere other than Windows (if it is disabling code that we otherwise could use elsewhere, I can make it more selectively applied), just saying that the cause might be similar and possibly unrelated to any patches we're carrying.

@nalimilan
Copy link
Member Author

OK, I canceled the build then. I didn't mean that the patch was in cause, I rather wanted to exclude the possibility that the crash would be due to its absence.

Note that it succeeds with LLVM 3.7.1.

@KristofferC
Copy link
Member

Is this still relevant?

@nalimilan
Copy link
Member Author

Looks like it's been fixed either in Julia or in LLVM 3.8.1.

@tkelman
Copy link
Contributor

tkelman commented Jan 22, 2017

probably fixed by #16503

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test This change adds or pertains to unit tests
Projects
None yet
Development

No branches or pull requests

3 participants