| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
|
| |
The ndk_platform backend will soon be deprecated because the ndk backend
can serve the same purpose. This is to eliminate the confusion about
having two variants (ndk and ndk_platform) for the same 'ndk' backend.
Bug: 161456198
Test: m
Change-Id: I4498856a47896024698c53b3fc84e53959d639b8
|
| |\
| |
| |
| |
| |
| | |
Original change: https://android-review.googlesource.com/c/platform/system/memory/libmemtrack/+/1656471
Change-Id: Ifbc1106bfcfe01130c24f1cdde6ee5ebfd5201d3
|
| |/
|
|
|
|
|
|
|
|
|
| |
libmemtrack no longer talks directly to memtrack HAL.
All requests for memtrack data goes through the memtrack
proxy service to allow access controls check on the calling
context's UID/PID.
Bug: 177664629
Test: adb shell dumpsys meminfo
Change-Id: I37950a1ffc0a7a0ab1fae3a0c0132bf29561543f
|
| |\
| |
| |
| |
| |
| | |
Bug: 180401296
Merged-In: Iaed0adc0e4479d50326f9e3093b2fa497cfc0ec0
Change-Id: I06bf2f47428370a32e53fe35470795e5830cc5e0
|
| | |\
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
4669770b47
Original change: https://android-review.googlesource.com/c/platform/system/memory/libmemtrack/+/1534121
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: Iaed0adc0e4479d50326f9e3093b2fa497cfc0ec0
|
| | |\ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
762a69d031 am: 1a4fb47b70
Original change: https://android-review.googlesource.com/c/platform/system/memory/libmemtrack/+/1530886
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: Ib1fc1ba51ff04ece344f77b3440e50dd20c17527
|
| | |\ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
611adabcaf
Original change: https://android-review.googlesource.com/c/platform/system/memory/libmemtrack/+/1518702
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: Ie17d4ec0ba274765c8650d9fda87d879944197d6
|
| | |\ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
42b8e536f1 am: 06b17daa1f
Original change: https://android-review.googlesource.com/c/platform/system/memory/libmemtrack/+/1518561
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: Ibda0e81b0d0b6f701a6f3dbd6f7902b5c1451555
|
| | |\ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Original change: https://android-review.googlesource.com/c/platform/system/memory/libmemtrack/+/1509322
Change-Id: I14fd1f8e9b2d87c7b57cae5198f971e42e8ef4ee
|
| | |\ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/memory/libmemtrack/+/13113106
Change-Id: Ia2f3290bc97c145c57cf83ca9a92600dbfd02360
|
| | |\ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
d6428f2514 am: 7bdedf3b87
Original change: https://android-review.googlesource.com/c/platform/system/memory/libmemtrack/+/1496020
Change-Id: Id111be326d955c2e15add02b6bd36661d136147d
|
| | | |\ \ \ \ \ \ \
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
Original change: https://android-review.googlesource.com/c/platform/system/memory/libmemtrack/+/1496020
Change-Id: Ib3488514362d1cae2bceb543031791d6d851cc25
|
| |\ \ \ \ \ \ \ \ \ \
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Original change: https://android-review.googlesource.com/c/platform/system/memory/libmemtrack/+/1589040
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: I62bb77539566c59839490cf994da637f8880fb1e
|
| | | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Added SPDX-license-identifier-Apache-2.0 to:
Android.bp
Bug: 68860345
Bug: 151177513
Bug: 151953481
Test: m all
Exempt-From-Owner-Approval: janitorial work
Change-Id: If1d9cd310873bc53374d2fbf4e6e27cf61e2468e
|
| |\| | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Original change: https://android-review.googlesource.com/c/platform/system/memory/libmemtrack/+/1560591
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: I6aed41db0c871f6da042f12aa3047c12e8867f76
|
| | | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Bug: 150578172
Test: m
Change-Id: I5e7c15d048d145037d50c1d473e36c0c9a4d52c7
|
| |\| | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Original change: https://android-review.googlesource.com/c/platform/system/memory/libmemtrack/+/1537301
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: I6a0184354e3403d1dae74fbac85daaf9dbd2dd14
|
| | |\ \ \ \ \ \ \ \ \ \ |
|
| | | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
Bug: 176477627
Test: dumpsys meminfo
Change-Id: Id02e06ac469ace70ed687c52cc8016db6c61cc9f
|
| |\| | | | | | | | | | |
| |_|_|_|_|_|_|_|_|_|/
|/| | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Original change: https://android-review.googlesource.com/c/platform/system/memory/libmemtrack/+/1556963
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: If24406aecf55d7fdf2a3c42bdd029133e6c112b2
|
| | |/ / / / / / / / /
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
Change-Id: I9194416ee1ff1689d9d9225e9bf7d13587182dea
|
| |\| | | | | | | | |
| |_|_|_|_|_|_|_|/
|/| | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Original change: https://android-review.googlesource.com/c/platform/system/memory/libmemtrack/+/1534121
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: I4cbec66b1cb9fb992ad70e17168904b7a0d0c1a0
|
| | | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
________________
Regarding the issues caused by the reverted version of these changes.
Root cause of the reported issues is that ServiceManager cannot get
the IMemtrack AIDL service, and getService() will busy loop for 5 seconds
trying to get the service.
```
12-22 12:22:22.501 1827 10885 W ServiceManager: Service android.hardware.memtrack.IMemtrack/default didn't start. Returning NULL
12-22 12:22:22.501 1827 10885 I ServiceManager: Waiting for service 'android.hardware.memtrack.IMemtrack/default' on '/dev/binder'...
```
Also because the results of this getService() was not cached, we will
attempt to get this service each time there is a new request from
libmemtrack, causing another 5 seconds delay each time.
Fix:
We check if the service is declared before attempting to retrieve it.
We use the more efficient waitForService(), instead of getService()
which can busy loop for 5 seconds.
Cache the results of these operations so that we don’t have to retrieve
the service each time there is a request from libmemtrack.
Boot Time Regression Test
Test: adb shell logcat -b all | grep TotalBootTime
Old version of this topic: 44209ms
LKGB: 19823ms
New version of this topic: 20058ms
Cts Graphics Test
Test: atest CtsGraphicsTestCases:android.graphics.cts.BitmapTest#testWrappedHardwareBufferBitmapNotLeaking
Old version of this topic: Fails
LKGB: Passes
New version of this topic: Passes
Memoinfo Test
Test: adb shell dumpsys meminfo
Old version of this topic: `*** SERVICE 'meminfo' DUMP TIMEOUT (10000ms) EXPIRED ***`
LKGB: <Expected output>
New version of this topic: <Expected output>
Battery Saver Test
Test: Unplug Device; Tap Battery Save Icon from the pull down menu
Old version of this topic: Battery Saver Icon does not become enabled (blue highlight)
LKGB: Battery Saver Icon Enabled
New version of this topic: Battery Saver Icon Enabled
Bug: 175021432
Test: memtrack_test
Change-Id: I6ce5c454e6a5c9b718277b4b3004f600dd0f6786
|
| |\| | | | | | | |
| |_|_|_|_|_|_|/
|/| | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Original change: https://android-review.googlesource.com/c/platform/system/memory/libmemtrack/+/1530886
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: Ie90647052bedb99d3f80791a5b7b7fcef9adfb32
|
| | |\ \ \ \ \ \ \ |
|
| | | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Revert "Add stable aidl memtrack HAL to product packages"
Revert "Add stable aidl memtrack hal to vndk list"
Revert "Memtrack HAL stable aidl sepolicy"
Revert "Add android.hardware.memtrack-unstable-ndk_platform"
Revert submission 1518702-memtrack-aidl
Reason for revert: Broken tests and boot time regressions
Reverted Changes:
Ic4dd70e2c:Add android.hardware.memtrack-unstable-ndk_platfor...
Iaf99d0ca4:Add stable aidl memtrack HAL to product packages
Iac54ae2ba:Add stable aidl memtrack hal to vndk list
If310210a3:libmemtrack: Add support for AIDL memtrack HAL
Ib6c634def:Memtrack HAL: Add stable AIDL implementation
I5e1d0e006:Memtrack HAL stable aidl sepolicy
Change-Id: I25c24458138e3c967e0d67a40e103487f10a1a07
|
| |\| | | | | | | |
| |_|_|_|_|_|_|/
|/| | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Original change: https://android-review.googlesource.com/c/platform/system/memory/libmemtrack/+/1518702
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: I70e15eb330dc92a0c74a6cddb53e224ebc63fb92
|
| | |\| | | | | | |
|
| | | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Bug: 175021432
Test: memtrack_test
Change-Id: If310210a37305331d96846c36ce5fcc2e5bd2047
|
| |\| | | | | | |
| |_|_|_|_|_|/
|/| | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
42b8e536f1
Original change: https://android-review.googlesource.com/c/platform/system/memory/libmemtrack/+/1518561
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: Iaf6e9b1d260fe0edcf5344e28602ee1d07bcb17e
|
| |/| | | | | | |
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
This reverts commit a37cf741cfcd997b52b77962dcd1b2c6585bf804.
Reason for revert: As libhardware will not be product_available,
libmemtrack may not be product_available.
Change-Id: Iaf3cdcccd84e5c9d08d4730d2bd4a45f4e40b820
|
| |\ \ \ \ \ \ \
| | |/ / / / /
| |/| | | | /
| |_|_|_|_|/
|/| | | | |
| | | | | | |
Original change: https://android-review.googlesource.com/c/platform/system/memory/libmemtrack/+/1509322
Change-Id: Ib085f05f3923fbf2065b526c99a04812746907d2
|
| | |/ / / /
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Bug: 174144799
Test: clang-format -i <file>
Change-Id: I1932da37b74e1d84a539176a667f2ab9177985f4
|
| |\ \ \ \ \
| | |/ / /
| |/| | /
| |_|_|/
|/| | | |
Bug: 172690556
Change-Id: Icfb36684b0705881ffa2e5c706386784b0a85efe
|
| | |\ \ \
| | | |/
| | |/|
| | | |
| | | |
| | | | |
stag-aosp-master am: 800d172aab -s ours am: 4e9ceb22eb am: e769551fa9
Change-Id: I3220f520ff0e507301b9d930c061e250e75e7b89
|
| | |\ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
qt-qpr1-dev-plus-aosp@6304901 into stag-aosp-master am: c374c6d4f4 -s ours am: 9bedc41732 am: e46785544e
Change-Id: I9c8862d11220df28b9ddee187cfb36265532da3c
|
| |\ \ \ \ \
| |_|_|/ /
|/| | | /
| | |_|/
| |/| |
| | | | |
Original change: https://android-review.googlesource.com/c/platform/system/memory/libmemtrack/+/1496020
Change-Id: I94eaf71cc1d5a2bc03278ac2f794ed81ed0e908d
|
| |/ / /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
vendor_available modules were available to product modules.
However, not all vendor_available modules are required to be
available to product modules. Some modules want to be available only
to product modules but not vendor modules.
To cover the requirement, we separate product_available from
vendor_available.
vendor_available will not provide product available module.
Bug: 150902910
Test: build
Change-Id: I7b306b4b4039ef697f5ebb2bd0c44595863be0b7
|
| |\ \ \
| |_|/
|/| |
| | |
| | |
| | | |
stag-aosp-master am: 800d172aab -s ours am: 4e9ceb22eb
Change-Id: I7c467651b74e98eff869f46eccf504aae8235d14
|
| | |\ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
stag-aosp-master am: 800d172aab -s ours
am skip reason: subject contains skip directive
Change-Id: I765eb160159f7d1922d8c049045f573942e4397d
|
| | | |\ \
| | | | |
| | | | |
| | | | |
| | | | | |
Bug: 151763422
Change-Id: I909519f577973fe8f9ff53fdb4b061b82e92d0c0
|
| | | | |\ \
| | | |/ /
| | |/| |
| | | | |
| | | | |
| | | | | |
stag-aosp-master am: c374c6d4f4
Change-Id: I5f923d99683232ff968faff8956f03e7da32c7eb
|
| |\| | | |
| |_|_|/
|/| | |
| | | |
| | | |
| | | | |
qt-qpr1-dev-plus-aosp@6304901 into stag-aosp-master am: c374c6d4f4 -s ours am: 9bedc41732
Change-Id: Idf2da75055903ba8199dc55a4e6a22a8ab923b7b
|
| |/| | |
| |/ /
| | |
| | |
| | |
| | |
| | |
| | | |
qt-qpr1-dev-plus-aosp@6304901 into stag-aosp-master am: c374c6d4f4 -s ours
am skip reason: subject contains skip directive
Change-Id: I6d3b4b9bed9492af19281d32f528b0ba5992b6fa
|
| | |\|
| | |
| | |
| | |
| | | |
Bug: 151763422
Change-Id: Ia1a535a65d18898cb2a7a5b733414b936cc14f11
|
| |\ \ \
| | |/
| |/|
| | |
| | |
| | | |
am: b0b959d437
Change-Id: Id3822f50f6d09c0085038da5d919d525d38f637e
|
| | |\|
| | |
| | |
| | | |
Change-Id: I972a9da561c72b4c80d8a269459fe92f3f650e8f
|
| | |/|
| | |
| | |
| | |
| | |
| | | |
am: 344ba42d98
Change-Id: I40a60642e3a11b893036f62a6ddceaaca19033d9
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
get_instance() only attempts to get the memtrack module once, since
it is a static variable. Therefore it makes sense to log an error if
the module is not available only once. The error shouldn't be
printed in logcat more than once during boot, since the API is only
used by one process (SystemServer).
This should allow devices that do not have anything to report in
memtrack (like HiKey and DragonBoard) to get rid of their dummy
memtrack implementation (which is optional anyway).
Test: boot aosp_x86_64 and check logcat (only one error)
Change-Id: I785f1ac288b7e497ff511ddb3d2b866fa5190d07
|