-
Notifications
You must be signed in to change notification settings - Fork 130
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
cpufeatures: aarch64 support (Linux and macOS/M4) #393
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,12 +1,21 @@ | ||
[package] | ||
name = "cpufeatures" | ||
version = "0.1.0" # Also update html_root_url in lib.rs when bumping this | ||
description = """ | ||
Lightweight and efficient no-std compatible alternative to the | ||
is_x86_feature_detected! macro | ||
""" | ||
authors = ["RustCrypto Developers"] | ||
license = "MIT OR Apache-2.0" | ||
description = "Lightweight and efficient no-std compatible alternative to the is_x86_feature_detected macro" | ||
documentation = "https://docs.rs/cpufeatures" | ||
repository = "https://github.com/RustCrypto/utils" | ||
keywords = ["cpuid", "target-feature"] | ||
categories = ["no-std"] | ||
edition = "2018" | ||
readme = "README.md" | ||
|
||
[target.aarch64-apple-darwin.dependencies] | ||
libc = "0.2" | ||
|
||
[target.'cfg(all(target_arch = "aarch64", target_os = "linux"))'.dependencies] | ||
libc = "0.2" |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,112 @@ | ||
//! ARM64 CPU feature detection support. | ||
//! | ||
//! Unfortunately ARM instructions to detect CPU features cannot be called from | ||
//! unprivileged userspace code, so this implementation relies on OS-specific | ||
//! APIs for feature detection. | ||
|
||
#[macro_export] | ||
#[doc(hidden)] | ||
macro_rules! __unless_target_features { | ||
($($tf:tt),+ => $body:expr ) => { | ||
{ | ||
#[cfg(not(all($(target_feature=$tf,)*)))] | ||
$body | ||
|
||
#[cfg(all($(target_feature=$tf,)*))] | ||
true | ||
} | ||
}; | ||
} | ||
|
||
#[cfg(target_os = "linux")] | ||
#[macro_export] | ||
#[doc(hidden)] | ||
macro_rules! __detect_target_features { | ||
($($tf:tt),+) => {{ | ||
let hwcaps = unsafe { libc::getauxval(libc::AT_HWCAP) }; | ||
$($crate::check!(hwcaps, $tf) & )+ true | ||
}}; | ||
} | ||
|
||
#[cfg(target_os = "macos")] | ||
#[macro_export] | ||
#[doc(hidden)] | ||
macro_rules! __detect_target_features { | ||
($($tf:tt),+) => {{ | ||
$($crate::check!($tf) & )+ true | ||
}}; | ||
} | ||
|
||
/// Linux `expand_check_macro` | ||
#[cfg(target_os = "linux")] | ||
macro_rules! __expand_check_macro { | ||
($(($name:tt, $hwcap:expr)),* $(,)?) => { | ||
#[macro_export] | ||
#[doc(hidden)] | ||
macro_rules! check { | ||
$( | ||
($hwcaps:expr, $name) => { (($hwcaps & libc::$hwcap) != 0) }; | ||
)* | ||
} | ||
}; | ||
} | ||
|
||
/// Linux `expand_check_macro` | ||
#[cfg(target_os = "linux")] | ||
__expand_check_macro! { | ||
("aes", HWCAP_AES), // Enable AES support. | ||
("sha2", HWCAP_SHA2), // Enable SHA1 and SHA256 support. | ||
("sha3", HWCAP_SHA3), // Enable SHA512 and SHA3 support. | ||
Comment on lines
+57
to
+59
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Note: the comments here are taken directly from the output of:
The output for these target features is the same on an I'm a bit confused because the target features aren't isomorphic to either the Linux ARM64 ELF hwcaps or the macOS sysctl flags. Per the Linux ARM64 ELF hwcaps docs: https://www.kernel.org/doc/html/latest/arm64/elf_hwcaps.html
That would suggest that the SHA-1 and SHA-3 hwcaps are independent of the others, and that SHA-2 support implies SHA-512 support. macOS/M4 likewise has separate
...but the rustc target-features suggest that |
||
} | ||
|
||
/// macOS `check!` macro. | ||
/// | ||
/// NOTE: several of these instructions (e.g. `aes`, `sha2`) can be assumed to | ||
/// be present on all Apple ARM64 hardware. | ||
/// | ||
/// Newer CPU instructions now have nodes within sysctl's `hw.optional` | ||
/// namespace, however the ones that do not can safely be assumed to be | ||
/// present on all Apple ARM64 devices, now and for the foreseeable future. | ||
/// | ||
/// See discussion on this issue for more information: | ||
/// <https://github.com/RustCrypto/utils/issues/378> | ||
#[cfg(target_os = "macos")] | ||
#[macro_export] | ||
#[doc(hidden)] | ||
macro_rules! check { | ||
("aes") => { | ||
true | ||
}; | ||
("sha2") => { | ||
true | ||
}; | ||
("sha3") => { | ||
unsafe { $crate::aarch64::sysctlbyname(b"hw.optional.armv8_2_sha3\0") } | ||
}; | ||
} | ||
|
||
/// macOS helper function for calling `sysctlbyname`. | ||
#[cfg(target_os = "macos")] | ||
pub unsafe fn sysctlbyname(name: &[u8]) -> bool { | ||
assert_eq!( | ||
name.last().cloned(), | ||
Some(0), | ||
"name is not NUL terminated: {:?}", | ||
name | ||
); | ||
|
||
let mut value: u32 = 0; | ||
let mut size = core::mem::size_of::<u32>(); | ||
|
||
let rc = libc::sysctlbyname( | ||
name.as_ptr() as *const i8, | ||
&mut value as *mut _ as *mut libc::c_void, | ||
&mut size, | ||
core::ptr::null_mut(), | ||
0, | ||
); | ||
|
||
assert_eq!(size, 4, "unexpected sysctlbyname(3) result size"); | ||
assert_eq!(rc, 0, "sysctlbyname returned error code: {}", rc); | ||
value != 0 | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Played around with a few different syntaxes for this macro and this is the best one I could come up with.
Open to suggestions for alternatives.