Skip to content

Comments

Fix detection of ARMv7 and ARM64 CPU features on FreeBSD#17079

Closed
allanjude wants to merge 1 commit intoopenssl:masterfrom
allanjude:freebsd_arm_fix
Closed

Fix detection of ARMv7 and ARM64 CPU features on FreeBSD#17079
allanjude wants to merge 1 commit intoopenssl:masterfrom
allanjude:freebsd_arm_fix

Conversation

@allanjude
Copy link
Contributor

OpenSSL assumes AT_HWCAP = 16 (as on Linux), but on FreeBSD AT_HWCAP = 25

OpenSSL calls elf_auxv_info() with AT_CANARY which returns ENOENT
resulting in all ARM acceleration features being disabled.

CLA: trivial

Checklist

OpenSSL assumes AT_HWCAP = 16 (as on Linux), but on FreeBSD AT_HWCAP = 25

OpenSSL calls elf_auxv_info() with AT_CANARY which returns ENOENT
resulting in all ARM acceleration features being disabled.

CLA: trivial
@allanjude
Copy link
Contributor Author

Would it be preferable to instead do something like:

#ifndef AT_HWCAP
#  define AT_HWCAP 16
#endif

And then use AT_HWCAP to avoid the FreeBSD specific ifdefs?

@t8m
Copy link
Member

t8m commented Nov 19, 2021

Would it be preferable to instead do something like:

#ifndef AT_HWCAP
#  define AT_HWCAP 16
#endif

And then use AT_HWCAP to avoid the FreeBSD specific ifdefs?

Yeah, that would look better to me.

@allanjude
Copy link
Contributor Author

I've opened a different pull request for the alternative version, I agree it is cleaner: #17082

@t8m
Copy link
Member

t8m commented Nov 22, 2021

Closing in favor of #17082

@t8m t8m closed this Nov 22, 2021
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