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

Adding arm32v7 architecture support. #16

Merged
merged 2 commits into from
Dec 29, 2017
Merged

Adding arm32v7 architecture support. #16

merged 2 commits into from
Dec 29, 2017

Conversation

ninjarobot
Copy link
Contributor

This adds a Dockerfile for arm32v7 support. It differs from the amd64 build as it removes the rocksdb addin since there are no packages for it on the Debian arm32 images. It also removes the hdt addin due to an issue building that addin.

@JanWielemaker aside from those packages, are there any other tests or concerns you have with running on arm32? I tested loading the remaining addins as well as some basic prolog and it seems to work, but you mentioned there were some edge cases from time to time.

@JanWielemaker
Copy link
Member

I don't think the current version has arm32 specific issues. Keri and I are running rather extreme stress tests on amd64 as well as aarch64 (arm 64). This exposed a number of race conditions in atom and clause garbage collection. Depending on progress I expect a new stable and development in the coming week. It might be wise to push this through with the next releases.

@ninjarobot
Copy link
Contributor Author

@JanWielemaker I see the 7.7.7 update is available. Do you think this is a good candidate for adding an arm32 image?

@JanWielemaker
Copy link
Member

Yes, I think 7.7.7 is pretty much ok on ARM. Let us try :)

@ninjarobot
Copy link
Contributor Author

I updated to 7.7.7 and tested on amd64 and arm32. I'm going to merge and then update the official images.

@ninjarobot ninjarobot merged commit 10cea35 into master Dec 29, 2017
@JanWielemaker
Copy link
Member

Thanks! Fingers crossed :)

@JanWielemaker JanWielemaker deleted the arm_support branch October 27, 2020 14:04
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.

2 participants