system-core

Fork

Cache only a small portion of page map data.

The scudo memory allocator will allocate a large virtual memory address
that causes procrank to allocate a huge amount of data. Modify so it
caches the data a bit at a time.

Remove some allocations that are not necessary since the data is only used
in the first loop.

Fix a few pread64 checks against the actual bytes read, rather than the
read is < 0.

Fix sort by rss in procrank.

Bug: 135694447

Test: Ran unit tests (only one unit test actually runs the modified code).
Test: Ran the old version of procrank using a static libmeminfo and verified
Test: that the same data is found as the new version.

Change-Id: I8adc169b5607ec994ff13a9e161a479350d84c4d
Merged-In: I8adc169b5607ec994ff13a9e161a479350d84c4d
(cherry picked from commit 7f8915c76e232dd228930e749ec289d10c547f85)

Fix a memory leak in gatekeeper.

In violation to the documentation of GateKeeper::GetAuthTokenKey and
GateKeeper::GetPasswordKey, the implementations in SoftGateKeeper
allocate and return buffers and relinquish ownership causing a memory
leak, because the caller expects the implementation to retain ownership.

Bug: 129768470
Bug: 134557251
Test: gatekeeper-unit-tests
Change-Id: I0af9539d3dcd47dfd1e7d80cdee700ea0c2d6d0f
Merged-In: I0af9539d3dcd47dfd1e7d80cdee700ea0c2d6d0f
(cherry picked from commit 6a9c4e7968e73393110b169b33fb636531fe7fc2)

Fix a memory leak in gatekeeper.

In violation to the documentation of GateKeeper::GetAuthTokenKey and
GateKeeper::GetPasswordKey, the implementations in SoftGateKeeper
allocate and return buffers and relinquish ownership causing a memory
leak, because the caller expects the implementation to retain ownership.

Bug: 129768470
Bug: 134557251
Test: gatekeeper-unit-tests
Change-Id: I0af9539d3dcd47dfd1e7d80cdee700ea0c2d6d0f
Merged-In: I0af9539d3dcd47dfd1e7d80cdee700ea0c2d6d0f
(cherry picked from commit 6a9c4e7968e73393110b169b33fb636531fe7fc2)

Snap for 5719381 from bb0838b5114419b4f8ecff67c0b6fb728399c7b7 to qt-qpr1-release

Change-Id: I2d8f8e10162f876d23a573d1bf43f871e599224b

rootdir: init.rc: use gc_urgent_sleep_time for f2fs

This accelerates f2fs GC when IdleMaint.

Bug: 128594671
Bug: 137190790
Change-Id: I1cea7c99ffd5963711c66460f1c8368b43aeaee2
Signed-off-by: Jaegeuk Kim <jaegeuk@google.com>

Snap for 5713042 from 8ba0b82605dc311ef2af59f9116f80212429d965 to qt-qpr1-release

Change-Id: I8f99b217af6785a97974599d56b8ae1f5aeaa07c

Avoid get same timestamps in log testcase
am: ac9e2af1b6

Change-Id: I1ec3face1c149dccedc59ef45fad2e7b73987cb7

Avoid get same timestamps in log testcase

for arm64, userspace get time by vDSO, no syscall overhead.
multiple_test_2 sometimes get 2 same timestamps now, then testcase
fails.

Bug: b/135555666
Test: ./CtsLogdTestCases64 --gtest_filter=*multiple_test*
Merged-In: Ic8a0ab6f571df1487fe5fb4e5f07dc612af69001
Change-Id: I83f8beae168f397a347972c5e60de86d16c8f7c6

Snap for 5706813 from 41d77405d6e8bde886fab97e788759e37c30b49e to qt-qpr1-release

Change-Id: I2b2301fedcf0ef0a6ccf6b02c34df0a799481876

Revert "Remove FUSE logic; it's only a sdcardfs wrapper."

This reverts commit 68e1093028185cde31dcacf34a59f6c08848d3d4.

Revert "Add support for esdfs mounting"

This reverts commit c336f86284ee817a2e846452ad80e5d014d11414.

dmabuf_dump: Change the default view to per-process stats

Add '-a' commandline option to show the old complete table that shows
dmabuf x process grid.

Fixes: 135668399
Test: dmabuf_dump
Test: dmabuf_dump -h
Test: dmabuf_dump -a
Test: dmabuf_dump -a 123
Test: dmabuf_dump 123 456
Test: dmabuf_dump <pid>

Change-Id: If228c97baa375c515d94baf3b5be674cfcc09919
Merged-In: If228c97baa375c515d94baf3b5be674cfcc09919
Signed-off-by: Sandeep Patil <sspatil@google.com>

dmabuf_dump: Fix aosp_x86-eng builds

Test: lunch aosp_x86-eng; mma -j
Test: lunch aosp_blueline-eng; mma -j
Bug: 136136199

Change-Id: Ie6f90c967b356a55db09acee7b3596661e001166
Merged-In: Ie6f90c967b356a55db09acee7b3596661e001166
Signed-off-by: Sandeep Patil <sspatil@google.com>

dmabuf_dump: Make output more readable and debug friendly

- Get rid of the PSS number that didn't make sense.
- Fix double counting in per-process size when buffers had file
descriptors and mmap'ed
- Make the output much more readable
- Print totals where they make sense.
- Dont print per-process fd and map refs separately, total ref count per
process is sufficient.

Bug: 135668399
Test: dmabuf_dump (on AOSP blueline)

Change-Id: I10dc0a332d49110dd68f0eaa94a326679361deea
Merged-In: I10dc0a332d49110dd68f0eaa94a326679361deea
Signed-off-by: Sandeep Patil <sspatil@google.com>

Snap for 5698755 from cac9a2b10bf269437490c40a5717fb2fb8cdb08c to qt-qpr1-release

Change-Id: Ibf513d2d4a45f33afa8216a587bff3848c687944

Merge "Make ext4 userdata checkpoints work with metadata encryption" into qt-dev
am: be4e6131d8

Change-Id: Ic522f20a8f1998ed63c90e0e2e6d469ef530b5f2

Merge "Make ext4 userdata checkpoints work with metadata encryption" into qt-dev

Make ext4 userdata checkpoints work with metadata encryption

When both ext4 user data checkpoints and metadata encryption are
enabled, we are creating two stacked dm devices. This had not been
properly thought through or debugged.

Test: Enable metadata encryption on taimen (add
keydirectory=/metadata/vold/metadata_encryption to flags for userdata in
fstab.hardware)
Unfortunately metadata is not wiped by fastboot -w, so it is
necessary to rm metadata/vold -rf whenever you wipe data.
fastboot flashall -w works
fastboot reboot -w works
A normal boot works
Disable checkpoint commits with
setprop persist.vold.dont_commit_checkpoint 1
vdc checkpoint startCheckpoint 10
adb reboot
wait for device to fully boot then
adb reboot
Wait for device to fully boot then
adb logcat -d | grep Checkpoint shows the rollback in the logs

This tests encryption on top of checkpoints with commit, encryption
without checkpoints, and rollback, which seems to be the key cases.

Also ran same tests on unmodified Taimen and Blueline

Bug: 135905679
Merged-In: I24387a2943dce28b918c34894f24911b20429be7
Change-Id: I24387a2943dce28b918c34894f24911b20429be7

Snap for 5691478 from 8e3b94c2e296f1d23b49e1a61d20f3ebf8f417b5 to qt-qpr1-release

Change-Id: I091613807dad197199709273960130fce78ae8fa

Fix IsUsable to check for each controller separately
am: cb06c2b376

Change-Id: Id04ea4a5a208a307a7a2a3a6c801d38232921f36

Fix IsUsable to check for each controller separately

CgroupController::IsUsable is using a global static variable to store the
existence of the controller. That means the first controller existence
check would affect all other controllers. Fix this by making this variable
to be a member of CgroupController class so that each controller can check
for its existence independently of other controllers.

Fixes: aa1d54f0ccbb ("Remove ACgroupController_getFlags to fix API breakage")
Bug: 136020193
Test: adb shell cat /proc/$pid/task/*/cgroup" prints "cpuset:/top-app"
Test: for new launched activity process
Change-Id: I4741a9126ea494122d5b2b1a0c4d7252bff6025c
Signed-off-by: Suren Baghdasaryan <surenb@google.com>

Snap for 5686022 from 5dd50fdb9c9386371d3ebafacb04c0270a80d455 to qt-qpr1-release

Change-Id: I3af64a5983084df6a1c4bd3a7088b8bbcf746644

Revert "adb: daemon: Assign valid fd to usb_handle ep0 file descriptor"
am: ba4684c2b2

Change-Id: Idd32e4c28031b07cda09477b819d4bce0434b4fb

Revert "adb: turn CHECKs into an error + transport restart."
am: 2547f740ea

Change-Id: Ic4ab7cf01bc53ece9f5459e60a7ba602ec086ef0

Revert "Reland "adbd: don't close ep0 until we receive FUNCTIONFS_UNBIND.""
am: 782036b7af

Change-Id: I73978ff63b91e14f79a719d0aec1ee7812749b2d

Revert "adb: daemon: Assign valid fd to usb_handle ep0 file descriptor"

This reverts commit b2d79c5e48684d3859481bcca0e967fc4021db12.

Bug: http://b/135753060
Test: treehugger
Change-Id: I2b1b2b7dfb7bab783e4ed40e295329525ceaa0dd

Revert "adb: turn CHECKs into an error + transport restart."

This reverts commit b41d7ae1a9e5cfd2a7df6d0edee50fdf97c636f0.

Bug: http://b/135753060
Test: treehugger
Change-Id: Ib7cd7b7eaffa55ec2884a096a1c11739b12af6d1

Revert "Reland "adbd: don't close ep0 until we receive FUNCTIONFS_UNBIND.""

This reverts commit c33aee8ac1c0958df32dcd2db27ca3d0d1badbd1.

Bug: http://b/135753060
Test: treehugger
Change-Id: Ic83a44e7c8f860a6c044ca20ec81f365a8a6ba4b

Snap for 5681502 from c745f09b33804242c43f823242f0112645ed3a98 to qt-qpr1-release

Change-Id: I273775fa3eb8585d140cfcbfd517aad621a92d68

Merge changes Ibb3f46b8,I875b87a0,I2d005e17 into qt-dev
am: 64e64eb990

Change-Id: I9541e903885ed2fd5fc1b431f881d4aa33f03411