mikros/rust
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

android: use posix_memalign for aligned allocations

Browse Source
This commit is contained in:
Ralf Jung 2024-05-18 12:49:01 +02:00
parent 36c0a6d40f
commit dde1134c6d
1 changed files with 1 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
library/std/src/sys/pal/unix/alloc.rs
View File

@ -59,8 +59,8 @@ unsafe fn realloc(&self, ptr: *mut u8, layout: Layout, new_size: usize) -> *mut
} }
cfg_if::cfg_if! { cfg_if::cfg_if! {
// We use posix_memalign wherever possible, but not all targets have that function.
if #[cfg(any( if #[cfg(any(
target_os = "android",
target_os = "redox", target_os = "redox",
target_os = "espidf", target_os = "espidf",
target_os = "horizon", target_os = "horizon",
@ -68,23 +68,6 @@ unsafe fn realloc(&self, ptr: *mut u8, layout: Layout, new_size: usize) -> *mut
))] { ))] {
#[inline] #[inline]
unsafe fn aligned_malloc(layout: &Layout) -> *mut u8 { unsafe fn aligned_malloc(layout: &Layout) -> *mut u8 {
// On android we currently target API level 9 which unfortunately
// doesn't have the `posix_memalign` API used below. Instead we use
// `memalign`, but this unfortunately has the property on some systems
// where the memory returned cannot be deallocated by `free`!
//
// Upon closer inspection, however, this appears to work just fine with
// Android, so for this platform we should be fine to call `memalign`
// (which is present in API level 9). Some helpful references could
// possibly be chromium using memalign [1], attempts at documenting that
// memalign + free is ok [2] [3], or the current source of chromium
// which still uses memalign on android [4].
//
// [1]: https://codereview.chromium.org/10796020/
// [2]: https://code.google.com/p/android/issues/detail?id=35391
// [3]: https://bugs.chromium.org/p/chromium/issues/detail?id=138579
// [4]: https://chromium.googlesource.com/chromium/src/base/+/master/
// /memory/aligned_memory.cc
libc::memalign(layout.align(), layout.size()) as *mut u8 libc::memalign(layout.align(), layout.size()) as *mut u8
} }
} else { } else {