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

Randomize segment name generated for SharedArray. #10877

Merged
merged 1 commit into from
Apr 19, 2015

Conversation

twadleigh
Copy link
Contributor

Compensates for the lack of an analog of shm_unlink in Windows.

Addresses #10773.

@ViralBShah
Copy link
Member

Cc: @amitmurthy

@amitmurthy
Copy link
Contributor

On OSX, I get the error

while loading parallel.jl, in expression starting on line 30
    From worker 3:       * examples             in  13.07 seconds
ERROR: LoadError: LoadError: SystemError: shm_open() failed for /jl-073248-HwtDQWQkm64lrHd4tut6: File name too long
 in anonymous at task.jl:1388

I'll test it with a shorter name and identify the right length for which we can make it work.

@amitmurthy
Copy link
Contributor

Can you update the PR with

shm_seg_name = @sprintf("/jl%06u%s", getpid() % 10^6, randstring(20)) ?

I just removed the -s and it now passes on OSX.

Compensates for the lack of an analog of `shm_unlink` in Windows.

Addresses JuliaLang#10773.
@twadleigh
Copy link
Contributor Author

Done. I thought I had made it short enough. Thanks for checking.

amitmurthy added a commit that referenced this pull request Apr 19, 2015
Randomize segment name generated for `SharedArray`.
@amitmurthy amitmurthy merged commit 5a8ae0a into JuliaLang:master Apr 19, 2015
@nalimilan
Copy link
Member

Isn't the comment # On OSX, the shm_seg_name length must be < 32 characters wrong (or imprecise) then?

@amitmurthy
Copy link
Contributor

Yes. the comment needs to be changed. Seems like < 30 is the right answer.

On Sun, Apr 19, 2015 at 8:42 PM, Milan Bouchet-Valat <
notifications@github.com> wrote:

Isn't the comment # On OSX, the shm_seg_name length must be < 32
characters wrong (or imprecise) then?


Reply to this email directly or view it on GitHub
#10877 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants