Page MenuHomeFreeBSD

ossl: Move arm_arch.h to a common subdirectory
ClosedPublic

Authored by markj on Dec 1 2023, 3:36 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, May 3, 7:00 PM
Unknown Object (File)
Thu, May 2, 11:11 PM
Unknown Object (File)
Thu, May 2, 11:10 PM
Unknown Object (File)
Thu, May 2, 11:06 PM
Unknown Object (File)
Mon, Apr 29, 4:07 AM
Unknown Object (File)
Sun, Apr 28, 10:48 PM
Unknown Object (File)
Thu, Apr 18, 4:38 PM
Unknown Object (File)
Sun, Apr 7, 1:31 PM
Subscribers

Details

Summary

OpenSSL itself keeps only a single copy of this header. Do the same in
sys/crypto/openssl to avoid the extra maintenance burden. This requires
adjusting the include paths for generated asm files.

No functional change intended.

Reported by: jrtc27

Test Plan

buildworld+buildkernel for arm and arm64

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

markj requested review of this revision.Dec 1 2023, 3:36 PM

Do we need to update make buildasm in secure/lib/libcrypto? It updates arm_arch.h from the imported OpenSSL.

Fix the arm64 buildasm target, have buildasm update arm_arch.h for arm as well

Do we need to update make buildasm in secure/lib/libcrypto? It updates arm_arch.h from the imported OpenSSL.

Yes, thanks.

I would perhaps be tempted to leave the header in arm/ and have aarch64 include it from there. It is true that upstream doesn't have it in a subdirectory, but upstream doesn't use platform-specific subdirectories for its assembly files at all.

This revision is now accepted and ready to land.Dec 1 2023, 8:51 PM
In D42866#977839, @jhb wrote:

I would perhaps be tempted to leave the header in arm/ and have aarch64 include it from there. It is true that upstream doesn't have it in a subdirectory, but upstream doesn't use platform-specific subdirectories for its assembly files at all.

I don't have a strong feeling either way. One weak argument for doing it this way: when it comes time to remove 32-bit arm support from the tree, whenever that is, the person doing it won't have to fixup ossl/aarch64.

This revision was automatically updated to reflect the committed changes.