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

[argon2] many tests fail on s390x, likely due to endianness issues #481

Closed
michel-slm opened this issue Jan 19, 2024 · 5 comments · Fixed by #482
Closed

[argon2] many tests fail on s390x, likely due to endianness issues #481

michel-slm opened this issue Jan 19, 2024 · 5 comments · Fixed by #482

Comments

@michel-slm
Copy link

I'm packaging argon2 in Fedora, and many tests fail on s390x:

https://koji.fedoraproject.org/koji/taskinfo?taskID=112000146
https://kojipkgs.fedoraproject.org/work/tasks/231/112000231/build.log
build.log

Full build.log uploaded since Fedora's build system would garbage collect scratch builds after a couple of weeks.

It succeeds on other architectures - x86_64, aarch64, ppc64le

@tarcieri
Copy link
Member

Reproduced, thanks.

That's unfortunate. Normally we run our crates through cross tests to look for this sort of thing, but they were never configured for argon2.

@newpavlov
Copy link
Member

These methods introduced in #247 are obviously not endianess-portable.

For BE testing it would be nice to use MIRI instead of cross.

cc @Pjottos

@tarcieri
Copy link
Member

These methods introduced in #247 are obviously not endianess-portable.

Aha! Good catch!

For BE testing it would be nice to use MIRI instead of cross.

I don't think that's going to work. The argon2 test suite exercises the full Argon2 algorithm repeatedly with KATs. Miri is going to be way too slow for those to complete in a reasonable timeframe.

@tarcieri
Copy link
Member

Okay, got it fixed locally. Will add some cross config in CI and push up a PR.

FWIW I tried running the tests under Miri. I gave it about 10 minutes to get through one of the nontrivial ones and it couldn't complete that, so I don't think it's going to work out in CI.

cross with powerpc-unknown-linux-gnu as the target seems like the fastest option to me.

@tarcieri
Copy link
Member

PR here: #482

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 a pull request may close this issue.

3 participants