aboutsummaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
4 daysMerge "Snap for 11933108 from 3e8dd2b88c8da039cf89a60bd03a8da8f1407bbc to ↵sdk-releaseAndroid Build Coastguard Worker
sdk-release" into sdk-release
4 daysSnap for 11933108 from 3e8dd2b88c8da039cf89a60bd03a8da8f1407bbc to sdk-releaseAndroid Build Coastguard Worker
Change-Id: I512a883a6c553ee20ea2d8a10a96268cdd88f684
5 daysMerge "new.cpp: add sized operator delete." into mainElliott Hughes
5 daysMerge "<malloc.h>: warn on unused results for malloc_usable_size()." into mainElliott Hughes
5 daysMerge "Add a README.md to docs/, as a guide to what we have available." into ↵Elliott Hughes
main
5 daysnew.cpp: add sized operator delete.Elliott Hughes
clang-19 and later default to C++14 sized deallocation. Currently we don't have C23's C equivalent of free_sized(), so we just implement these in terms of free() for now. Bug: http://b/344993254 Change-Id: I708710aee3f7a684bfe494a96f7baed014acbe57
5 daysMerge "Snap for 11926842 from b9279b62c2a128ceb4e9d86352a2d61ad1384bbd to ↵Android Build Coastguard Worker
sdk-release" into sdk-release
5 daysSnap for 11926842 from b9279b62c2a128ceb4e9d86352a2d61ad1384bbd to sdk-releaseAndroid Build Coastguard Worker
Change-Id: Ie918ae4e5b1d2c341ca719cee3c10bcb108f17f0
6 daysMerge "Update to v6.9 kernel headers." into mainChristopher Ferris
6 daysMerge "<stdio.h>: warn on some unused results." into mainElliott Hughes
6 daysAdd a README.md to docs/, as a guide to what we have available.Elliott Hughes
I was actually here to add some new documentation, but realized there wasn't really a good place to put it... Change-Id: I8a2fc93e61a89e87aa53dd0beb9dfcc6561687ca
6 days<malloc.h>: warn on unused results for malloc_usable_size().Elliott Hughes
Not a security problem, but definitely a bug if you're calling this and ignoring the result, since it has no side-effects. (All of the more important functions -- realloc() especially -- are already annotated.) Change-Id: I217463518b4716befcc0ed9426648eafbfbbdde4
6 daysMerge "<unistd.h>: explain why getpagesize() is better than getauxval() as ↵Elliott Hughes
well as sysconf()." into main
6 daysUpdate to v6.9 kernel headers.Christopher Ferris
Kernel headers coming from: Git: https://android.googlesource.com/kernel/common/ Branch: android-mainline Tag: android-mainline-6.9 Test: Builds and bionic unit tests pass on raven. Test: Able to log in to an Android GO 32 bit device. Change-Id: If53a38bbb65c3c653ce790fe5c6e51597901a14e
6 daysMerge "setjmp_test: riscv64 does have callee-save fp registers." into mainElliott Hughes
6 daysMerge "getpagesize(): remove "portability" claim." into mainElliott Hughes
6 daysgetpagesize(): remove "portability" claim.Elliott Hughes
This is just POSIX nonsense. Android, iOS/macOS, the BSDs, and glibc all have getpagesize(). Window doesn't, but Windows doesn't have sysconf(3) either, so there's not a strong "portability" argument to be had there! Change-Id: Ieacfbb61c3612045f27c20c3fa4fa12694db2243
6 days<unistd.h>: explain why getpagesize() is better than getauxval() as well as ↵Elliott Hughes
sysconf(). I thought we'd already done this, but apparently not. Bug: http://b/315511637 Change-Id: I0581438a6f551be9750b6679a9f3ad06c83f1e9b
7 dayssetjmp_test: riscv64 does have callee-save fp registers.Elliott Hughes
The setjmp implementation is correct, but this comment was wrong, and we were missing the implementation of the test. This would be slightly simpler if we enabled Zfa in the build system and qemu, but this works for now. Change-Id: Iad32771019584beacd8b35138ac2bf4c0a14c903
9 days<stdio.h>: warn on some unused results.Elliott Hughes
This is fairly conservative, touching only those functions (such as feof()) where it's clearly an error to not use the return value. Also fix a test that was ignoring the return value of feof() (because it was just checking whether the function could take the lock, and genuinely didn't care about the result). Change-Id: If2ade10ae87df45a8b9bfcb24828e460201fa9a1
9 daysMerge "Warn about unused results with bsearch()." into mainElliott Hughes
10 daysWarn about unused results with bsearch().Elliott Hughes
This function has no side-effects, and the return value is the whole point. Change-Id: Ibb9143c6a3c4089bfd6402c1c580ef76ce0f5d27
10 daysMerge "fdtrack: make it clearer that there's more information available." ↵Elliott Hughes
into main
10 daysMerge "pthread_exit(): reduce duplication." into mainElliott Hughes
11 daysfdtrack: make it clearer that there's more information available.Elliott Hughes
A lot of people who see these via pitot or whatever don't even think to look at the real tombstone. Make it explicit that this is what they should do next. Also, since we have the numbers, say exactly what we mean by "most common". Bug: http://b/338882189 Change-Id: Ia21848a40e82cb2486538a7578501e2ec0948e1c
11 daysMerge "Spell "calling convention" out in full." into mainElliott Hughes
11 daysMerge "bionic_allocator: more detailed and consistent error reporting." into ↵Elliott Hughes
main
11 daysMerge "Fix some wcwidth() special cases." into mainElliott Hughes
11 daysMerge "Snap for 11904077 from f6486e0e71cfa89fe20ce6baee06af18642eb5d2 to ↵Android Build Coastguard Worker
sdk-release" into sdk-release
11 daysSnap for 11904077 from f6486e0e71cfa89fe20ce6baee06af18642eb5d2 to sdk-releaseAndroid Build Coastguard Worker
Change-Id: I855279ea321d84a63b6834f6e97611fb1cf70a4b
11 daysbionic_allocator: more detailed and consistent error reporting.Elliott Hughes
I only came to improve the signature mismatch error, but I was then annoyed by the copy & paste of the other checks. get_chunk_size() seems to be deliberately avoiding any checks, though I think that might be a bug, and there should be a get_chunk_size() that _does_ check for most callers, and a get_chunk_size_unchecked() for the <sys/thread_properties.h> stuff that seems to want to only be "best effort" (but does still have _some_ possibility of aborting, in addition to the possibility of segfaulting). Also a bit of "include what you use" after cider complained about all the unused includes in bionic_allocator.h. Bug: https://issuetracker.google.com/341850283 Change-Id: I278b495601353733af516a2d60ed10feb9cef36b
11 daysFix some wcwidth() special cases.Elliott Hughes
Detailed explanation in the code comments. Test: treehugger Change-Id: I2aef2510724c1c622b83e226b51d4c8429b88272
12 daysSpell "calling convention" out in full.Elliott Hughes
We only use this in one other place anyway. Also be explicit about how `__tls_get_addr` and `___tls_get_addr` differ, since I missed that at first! Change-Id: Ica214886c5346f118f063bca26e6dd8d74ee21f4
12 daysMerge "Remove unused Android.mk under bionic" into mainTreehugger Robot
12 daysMerge "<sys/thread_properties.h>: remove stray semicolon in the ↵Treehugger Robot
implementation." into main
12 daysMerge "Fix gettid() in functions that call fork() or equivalent." into mainElliott Hughes
12 daysRemove unused Android.mk under bionickellyhung
Bug: b/311284462 Test: CIs Change-Id: I2586c425616ef59dbf2ba88fe1cbaa75a3bc2d5c
12 daysMerge "Revert^2 "Convert Android.mk under bionic to Android.bp"" into mainTreehugger Robot
13 daysFix gettid() in functions that call fork() or equivalent.Elliott Hughes
Bug: https://github.com/android/ndk/issues/2024 Change-Id: Ia264ce219fac99e8823e60c4a2e0607af8518056
2024-05-24pthread_exit(): reduce duplication.Elliott Hughes
Strictly, this is more of "swap one form of duplication for another", but I found the existing code non-obvious in part because people have added new code under existing comments (which don't apply), in two places. At this point, duplicating the _condition_ (which is much less likely to change at all, let alone grow more complex) clarifies the code and makes the comments match the code they're adjacent to again. Test: treehugger Change-Id: Ic8f01dc5b4fd14e942bf8dd7c72cab7df06d99d5
2024-05-24<sys/thread_properties.h>: remove stray semicolon in the implementation.Elliott Hughes
Test: treehugger Change-Id: I5f9d9d5b45ca74de2eb837c881582a6a87fdf0f6
2024-05-24Revert^2 "Convert Android.mk under bionic to Android.bp"Kelly Hung
This reverts commit 1e32ad88ffb67cd6a4bea5b5a5edfc7f45ec68e6. Reason for revert: 1. aosp/3099017 merged. 2. verified in ABDT run https://android-build.corp.google.com/abtd/run/L00400030004085752/ Change-Id: I6f07c67ab84ea93406dee88a5d20d0674a2d2a79
2024-05-23Merge "Snap for 11878282 from d475ee45aabaa95f4ca50320cff4d0cf07dbb7fd to ↵Android Build Coastguard Worker
sdk-release" into sdk-release
2024-05-23Snap for 11878282 from d475ee45aabaa95f4ca50320cff4d0cf07dbb7fd to sdk-releaseAndroid Build Coastguard Worker
Change-Id: I461b054bb44bee4d0f3ca635abdd0e3437522384
2024-05-22Merge "Revert^2 "Switch the loader to a noexcept version of libc++"" into mainRyan Prichard
2024-05-22Merge "bionic: max_android_page_size to 16384" into mainSteven Moreland
2024-05-22Merge "Read maps data before using and do safe reads." into mainChristopher Ferris
2024-05-22bionic: max_android_page_size to 16384Steven Moreland
The maximum page size Android supports now is 16384, and Art only supports 16kB, so we can save a bit of space. Bug: 332556665 Test: N/A Change-Id: I23df607bcc5cf9e96d7b6a66169413cd1a883f7e
2024-05-22Merge "Don't leak a thread when timer_create() fails." into mainElliott Hughes
2024-05-22Merge "Revert "Convert Android.mk under bionic to Android.bp"" into mainSatish Yalla