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

segfault using gadfly #5401

Closed
JeffBezanson opened this issue Jan 15, 2014 · 3 comments
Closed

segfault using gadfly #5401

JeffBezanson opened this issue Jan 15, 2014 · 3 comments
Labels
bug Indicates an unexpected problem or unintended behavior

Comments

@JeffBezanson
Copy link
Member

I get a hard-to-debug crash when trying to call plot with gadfly in IJulia. Valgrind says:

vex amd64->IR: unhandled instruction bytes: 0xCD 0xCD 0xCD 0xE3 0x37 0x0 0x0 0x0
vex amd64->IR:   REX=0 REX.W=0 REX.R=0 REX.X=0 REX.B=0
vex amd64->IR:   VEX=0 VEX.L=0 VEX.nVVVV=0x0 ESC=NONE
vex amd64->IR:   PFX.66=0 PFX.F2=0 PFX.F3=0
==20307== valgrind: Unrecognised instruction at address 0x15e0513d.
==20307==    at 0x15E0513D: ???
==20307==    by 0x4F147F8: jl_apply (julia.h:1047)
==20307==    by 0x4F17BDC: jl_trampoline (builtins.c:753)
==20307==    by 0x4F0B61D: jl_apply (julia.h:1047)
==20307==    by 0x4F0FC31: jl_apply_generic (gf.c:1378)
==20307==    by 0x15DEBBD8: ???
==20307==    by 0x15DE7F36: ???
==20307==    by 0x15DE7E9A: ???
==20307==    by 0x4F147F8: jl_apply (julia.h:1047)
==20307==    by 0x4F17BDC: jl_trampoline (builtins.c:753)
==20307==    by 0x4F0B61D: jl_apply (julia.h:1047)
==20307==    by 0x4F0FC31: jl_apply_generic (gf.c:1378)

Julia commit 202668d
Gadfly commit GiovineItalia/Gadfly.jl@8cc09b1
@JeffBezanson
Copy link
Member Author

Another modest data point: the crash goes away if I turn off the LLVM optimizer.

@dcjones
Copy link
Contributor

dcjones commented Jan 16, 2014

When I bisect julia, I get 5c62b41 as the first bad commit.

@dcjones
Copy link
Contributor

dcjones commented Jan 16, 2014

And sure enough re-commenting out that part on master fixes the segfault.

tanmaykm pushed a commit to tanmaykm/julia that referenced this issue Jan 16, 2014
@JeffBezanson @tanmaykm
fix JuliaLang#5401
5f1e051 
type inference was being too greedy by returning `None` for no matching
methods. it's safer to be able to handle methods that might not be
defined yet. we might be able to roll back this change when JuliaLang#265 is
fixed.

also fix a bug in ... with keyword arguments that caused GiovineItalia/Gadfly.jl#157
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Indicates an unexpected problem or unintended behavior
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dcjones @JeffBezanson