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

Fix for OSX spawn test #22788

Merged
merged 1 commit into from
Jul 14, 2017
Merged

Fix for OSX spawn test #22788

merged 1 commit into from
Jul 14, 2017

Conversation

quinnj
Copy link
Member

@quinnj quinnj commented Jul 13, 2017

Ran into this several times while running the tests about a million times so far for #22441. I don't think this is related to #17626, but one could always hope, right?

@ararslan ararslan added system:mac Affects only macOS test This change adds or pertains to unit tests labels Jul 13, 2017
@iblislin
Copy link
Member

iblislin commented Jul 13, 2017

Not sure it's related or not, I found a way to make success(`true`) insane.

  1. borrow a MBA from my classmate :p
  2. test.jl
using Base.Test

for i in 1:100000  # just a large number
  println(i)
  @test success(`true`)
end

yes, we need @test to trigger crashing/freezing. no idea why single success works fine.
3. julia test.jl
4. hold your Crtl+t, keep sending SIGINFO
5. sometimes you got segmentation fault; sometimes the program freezed, only kill -9 can stop it.

@vtjnash
Copy link
Member

vtjnash commented Jul 13, 2017

@iblis17 looks unrelated. I ran your code in my debugger and found the issue is that Apple's libunwind does not conform to its specification (its unw_init_local allocates memory, but is documented to not allocate memory)

@iblislin
Copy link
Member

@vtjnash :-o file it as another issue?

@vtjnash
Copy link
Member

vtjnash commented Jul 13, 2017

Should be a radar, but I'm not very active about getting those submitted.

@quinnj quinnj merged commit f8ff488 into master Jul 14, 2017
@quinnj quinnj deleted the jq/osxspawn branch July 14, 2017 03:00
jeffwong pushed a commit to jeffwong/julia that referenced this pull request Jul 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
system:mac Affects only macOS test This change adds or pertains to unit tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants