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

Strongswan: enable rdrand and aes-ni only on X86 #21548

Merged
merged 1 commit into from
Jan 2, 2017

Conversation

FPtje
Copy link
Contributor

@FPtje FPtje commented Dec 31, 2016

Motivation for this change

Strongswan fails to compile on armv7l-linux with --enable-aesni and --enable-rdrand enabled. Errors are thrown about impossible constraints in asm (--enable-rdrand) or about gcc getting unknown command line parameters about aes (--enable-aesni). The options only makes sense on X86_64 processors.

The rdrand plugin is designed for Ivy Bridge processors:

High quality / high performance random source using the Intel rdrand instruction found on Ivy Bridge processors

The aes-ni plugin also only exists on X86 processors (which have the AES instruction set)

Tested with a local override. The change triggers a (successful) rebuild on my X86_64 system (purely because of the changed order of options). On armv7-linux this change fixes build errors.

See for a similar issue:
https://wiki.strongswan.org/issues/337

Strongswan fails to compile on armv7l-linux with `--enable-aesni` and  `--enable-rdrand` enabled. Errors are thrown about impossible constraints in asm (`--enable-rdrand`) or about gcc getting unknown command line parameters about aes (`--enable-aesni`). The options only makes sense on X86_64 processors.

The rdrand plugin is designed for Ivy Bridge processors:

> High quality / high performance random source using the Intel rdrand instruction found on Ivy Bridge processors

The aes-ni plugin also only exists on X86 processors (which have the AES instruction set)

Tested with a local override. The change triggers a (successful) rebuild on my X86_64 system. On armv7-linux this change fixes build errors.

See: 
https://wiki.strongswan.org/issues/337
@mention-bot
Copy link

@FPtje, thanks for your PR! By analyzing the history of the files in this pull request, we identified @ip1981, @zimbatm and @vcunat to be potential reviewers.

@7c6f434c 7c6f434c merged commit c882915 into NixOS:master Jan 2, 2017
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.

None yet

4 participants