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

opal/asm/arm64: Fix opal_atomic_compare_exchange_* bug #4566

Merged
merged 1 commit into from
Dec 5, 2017

Conversation

kawashima-fj
Copy link
Member

Only master and aarch64 is affected.
@hjelmn FYI: This is a bug introduced in recent atomic asm change (#4552).

Signed-off-by: KAWASHIMA Takahiro <t-kawashima@jp.fujitsu.com>
@hjelmn
Copy link
Member

hjelmn commented Dec 5, 2017

Not sure why CI didn't catch that error. Was relying on it to test aarch64. Thanks for catching this.

@hjelmn
Copy link
Member

hjelmn commented Dec 5, 2017

:bot:ompi:retest

@hppritcha
Copy link
Member

bot:lanl:retest

@hjelmn
Copy link
Member

hjelmn commented Dec 5, 2017

Is the aarch64 build really hanging? The timeout is 5 minutes but is that long enough?

@hjelmn
Copy link
Member

hjelmn commented Dec 5, 2017

:bot:ompi:retest

@hjelmn hjelmn merged commit ad59b93 into open-mpi:master Dec 5, 2017
@kawashima-fj kawashima-fj deleted the pr/arm64-atomic branch December 13, 2017 01:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants