Commit graph

64675 commits

Author SHA1 Message Date
Steven Moreland
c29cf50cf9 Merge "libcutils: API to disable memory mitigations" am: 32535585fc
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1449538

Change-Id: Ie5cc92e3bbd6b05a2351ba2b143d92f4ff6e4e79
2020-10-12 17:22:08 +00:00
Steven Moreland
32535585fc Merge "libcutils: API to disable memory mitigations" 2020-10-12 16:59:23 +00:00
Wenhao Wang
d969516a1c Merge "rpmb_dev: Switch to RPMB provisioning scheme" am: 323353413f
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1446626

Change-Id: Ic1bb9f2cb2b84f535fa8bd7a05f85199658d7f48
2020-10-10 05:26:59 +00:00
Wenhao Wang
323353413f Merge "rpmb_dev: Switch to RPMB provisioning scheme" 2020-10-10 05:06:56 +00:00
Steven Moreland
d22659b8ea libcutils: API to disable memory mitigations
Convenience API to disable memory mitigations, where we can specify
logging and also have a single place to update once a related bug is
fixed.

Bug: 166675194
Test: use API to disable memory mitigations
Change-Id: I4a3529ea1911925c49e31cc3809c2743e401c196
2020-10-10 00:16:24 +00:00
Treehugger Robot
68cd2d7c81 Merge changes from topic "bug_168791309" am: 8d96bfa9bd
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1454640

Change-Id: I40f47d42101e9e62ec27341a3fb0847d9b5f3d2b
2020-10-09 03:30:44 +00:00
Baligh Uddin
e55e30f5d9 Remove liblog, logcat, logd, logwrapper am: d2c21a10d3
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1454058

Change-Id: I2ee94c97a3518b9c4ae3617ab38418cde5f88e0e
2020-10-09 03:30:34 +00:00
Treehugger Robot
8d96bfa9bd Merge changes from topic "bug_168791309"
* changes:
  Add symlinks to satisfy include paths.
  Remove liblog, logcat, logd, logwrapper
2020-10-09 02:32:02 +00:00
Xin Li
3deb0ae5fe Skip ab/6749736 in stage.
Merged-In: Ib9e04266b47bdc6129c10c72ee35222c49beff1e
Change-Id: I06da53955499fd138300cd2c9fd25f3e9d7b3617
2020-10-08 17:23:57 -07:00
Baligh Uddin
62e9b3305d Add symlinks to satisfy include paths.
BUG: 168791309
Test: Local build + TH
Change-Id: Iafba3e4e8dc837c57af2332057a3c773c3922cb1
2020-10-08 23:19:43 +00:00
Baligh Uddin
071d88912e Merge "Add OWNERS block, so we can perform migration." am: 83e9bc346a
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1455042

Change-Id: I4e33c75a70c5a4283f8a65a8ee568fcce718c4fb
2020-10-08 23:18:53 +00:00
Baligh Uddin
d2c21a10d3 Remove liblog, logcat, logd, logwrapper
These subdirectories have moved to platform/system/logging.

BUG: 168791309
Test: Local build + TH
Change-Id: Iaee2ff59d4450f3e59dc9ea8b0e257b2de53e478
2020-10-08 23:17:52 +00:00
Baligh Uddin
83e9bc346a Merge "Add OWNERS block, so we can perform migration." 2020-10-08 23:05:36 +00:00
Baligh Uddin
23007f0e48 Add OWNERS block, so we can perform migration.
BUG: 168791309
TEST: YOLO
Change-Id: I6c9c12931203fe2460b2fbdca333f83541324022
2020-10-08 22:39:51 +00:00
Tom Cherry
07fbf02366 Merge "logd: single std::mutex for locking log buffers and tracking readers" am: 8582aa21df
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1451058

Change-Id: I193814f318fc30c8bb7c71565202c93d311480f9
2020-10-08 20:29:12 +00:00
Tom Cherry
8582aa21df Merge "logd: single std::mutex for locking log buffers and tracking readers" 2020-10-08 19:58:27 +00:00
Akilesh Kailash
4d5571310f Merge "libsnapshot:snapuserd: Handle signals" am: c81477c5e4
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1449996

Change-Id: I13a5e584bb8629e4ad3e918675cc454f9d7532b3
2020-10-08 03:15:53 +00:00
Akilesh Kailash
c81477c5e4 Merge "libsnapshot:snapuserd: Handle signals" 2020-10-08 02:49:04 +00:00
Tom Cherry
c581886eea logd: single std::mutex for locking log buffers and tracking readers
There are only three places where the log buffer lock is not already
held when the reader lock is taken:
1) In LogReader, when a new reader connects
2) In LogReader, when a misbehaving reader disconnects
3) LogReaderThread::ThreadFunction()

1) and 2) happen sufficiently rarely that there's no impact if they
additionally held a global lock.
3) is refactored in this CL.  Previously, it would do the below in a loop
  1) Lock the reader lock then wait on a condition variable
  2) Unlock the reader lock
  3) Lock the log buffer lock in LogBuffer::FlushTo()
  4) In each iteration in the LogBuffer::FlushTo() loop
    1) Lock then unlock the reader lock in FilterSecondPass()
    2) Unlock the log buffer lock to send the message, then re-lock it
  5) Unlock the log buffer lock when leaving LogBuffer::FlushTo()
If these locks are collapsed into a single lock, then this simplifies to:
  1) Lock the single lock then wait on a condition variable
  2) In each iteration in the LogBuffer::FlushTo() loop
    1) Unlock the single lock to send the message, then re-lock it

Collapsing both these locks into a single lock simplifes the code and
removes the overhead of acquiring the second lock, in the majority of
use cases where the first lock is already held.

Secondly, this lock will be a plain std::mutex instead of a RwLock.
RwLock's are appropriate when there is a substantial imbalance between
readers and writers and high contention, neither are true for logd.

Bug: 169736426
Test: logging unit tests
Change-Id: Ia511506f2d0935a5321c1b2f65569066f91ecb06
2020-10-07 15:00:49 -07:00
Akilesh Kailash
f594785a26 libsnapshot:snapuserd: Handle signals
1: poll events to accept client request.
2: Mask all signals except SIGINT and SIGTERM.

Bug: 168258493
Test: cow_snapuserd_test

Signed-off-by: Akilesh Kailash <akailash@google.com>
Change-Id: I330170ac15199d2a97cc3b752dbf3b62b7b51a40
2020-10-07 21:45:45 +00:00
Treehugger Robot
5d0170c836 Merge "fs_mgr: adb-remount-test.sh: filter out more administrivia mounts." am: 5f7c13fb1e
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1452555

Change-Id: Ia2226dd612d1bc48b11a5e70d6838dd218c7d86a
2020-10-07 20:27:46 +00:00
Treehugger Robot
5f7c13fb1e Merge "fs_mgr: adb-remount-test.sh: filter out more administrivia mounts." 2020-10-07 20:04:53 +00:00
Tom Cherry
c75729b3d9 Merge "Reland: "Remove logging symlinks from system/core/include"" am: 8401907adc
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1451056

Change-Id: Ib94409e7f154f38a699d9522134fee363d2b4123
2020-10-07 15:38:09 +00:00
Mark Salyzyn
08cd8b70c2 fs_mgr: adb-remount-test.sh: filter out more administrivia mounts.
binder /dev/binderfs binder rw,relatime,max=1048576,stats=global 0 0
/sys/kernel/debug /sys/kernel/debug debugfs rw,seclabel,relatime 0 0
/dev/block/dm-3 240548676 3692368 236856308   2% /data_mirror/cur_profiles

are administrative mounts uninteresting to the adb remount test.

Fix system and vendor devt tests, turn them into warnings instead.

Signed-off-by: Mark Salyzyn <salyzyn@google.com>
Bug: 170256128
Bug: 169988379
Test: adb-remount-test.sh
Change-Id: Ie19722a9ad813dce014bc27610e6db035e6c69e9
2020-10-07 08:36:20 -07:00
Tom Cherry
8401907adc Merge "Reland: "Remove logging symlinks from system/core/include"" 2020-10-07 15:11:10 +00:00
Treehugger Robot
c4b150d176 Merge "logd: fix bad test assumptions" am: 3f452134da
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1452015

Change-Id: I4988c897ab5e9c5e5dc9f2d6d3040e56d89abf7d
2020-10-07 07:53:32 +00:00
Treehugger Robot
3f452134da Merge "logd: fix bad test assumptions" 2020-10-07 03:12:49 +00:00
Treehugger Robot
73afed6c6f Merge "Clarify adb bugreport failures on pre-N devices." am: 4c7104ab2a
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1451737

Change-Id: Idb45ba95bf0970e6cdd37405414857e9052e084f
2020-10-07 01:20:28 +00:00
Treehugger Robot
95c3867dc1 Merge "Remove vts10 related configs" am: 52578dda5b
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1451616

Change-Id: I90fbfd7426ce63fd2a2c59f77d7ae95ee5e620d5
2020-10-07 01:20:11 +00:00
Treehugger Robot
4c7104ab2a Merge "Clarify adb bugreport failures on pre-N devices." 2020-10-07 00:59:39 +00:00
Treehugger Robot
52578dda5b Merge "Remove vts10 related configs" 2020-10-07 00:35:38 +00:00
Arve Hjønnevåg
397e40b158 Merge "trusty: keymaster: set_attestation_key: Change wrapped key" am: 3e7c3504a3
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1446633

Change-Id: I44fd518b31ede08ebf01628d2252dd11c0bc57d7
2020-10-06 23:51:54 +00:00
Arve Hjønnevåg
3e7c3504a3 Merge "trusty: keymaster: set_attestation_key: Change wrapped key" 2020-10-06 23:31:38 +00:00
Tom Cherry
340afd38cb logd: fix bad test assumptions
SerializedLogChunk.three_logs assumes that log buffers are
zero-initialized, but they are not.  This causes test failures on
host.

Test: this test passes
Change-Id: I0dfa282bdc36eaa4e8e39d85c5227f717b45ec2a
2020-10-06 15:54:26 -07:00
Elliott Hughes
f118cd95f1 Clarify adb bugreport failures on pre-N devices.
The error was meant to imply "without the filename" but (a) that wasn't
spelled out and (b) anyone who did just try the command would probably
be unpleasantly surprised by the massive amount of spam on their
terminal. So give them copy & paste instead.

(I did consider using their supplied filename, but since that's almost
certainly blah.zip, it seemed uncool to silently create a large text
file called "something.zip"!)

Bug: http://b/170225883
Test: untested for lack of a working pre-N device right now
Change-Id: I834939c963ca09927ccd4dc5ed8e88c65455838e
2020-10-06 13:53:38 -07:00
Dan Shi
67b7dd570f Remove vts10 related configs
Bug: 169582597
Test: build
Change-Id: I38e1e0f8e32b86ab14b905d040c1c62245a1a9e3
2020-10-06 13:52:44 -07:00
Xin Li
8c42a18cf8 Merge "DO NOT MERGE - Merge RP1A.201005.006" am: 9602e3d31a
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1449500

Change-Id: I8b43f619590b1076e7a6973c9a5a6974568f25f9
2020-10-06 20:24:10 +00:00
The Android Open Source Project
1a016c3af8 DO NOT MERGE - Merge RP1A.201005.006 am: 7fd5f8d2bc
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1449500

Change-Id: If6136de7e90083c318bf1ebb0e24172dbfeacb16
2020-10-06 20:23:54 +00:00
Xin Li
9602e3d31a Merge "DO NOT MERGE - Merge RP1A.201005.006" 2020-10-06 20:17:27 +00:00
Treehugger Robot
7c15f1090d Merge "liblog: add pthread.h to vndk log/log.h" am: 6d452d7e64
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1451017

Change-Id: I8c9a70e10407281e5ab8e5b412a1f2d36a98de7f
2020-10-06 20:02:36 +00:00
Treehugger Robot
6d452d7e64 Merge "liblog: add pthread.h to vndk log/log.h" 2020-10-06 19:30:34 +00:00
Tom Cherry
1783f12106 Reland: "Remove logging symlinks from system/core/include"
Soong will instead directly include those paths.

Bug: 165825252
Test: treehugger
Change-Id: If1ee31ebf83715c0dfc4797305784fbfa7bd97db
2020-10-06 10:45:11 -07:00
Tom Cherry
824ef9ade6 liblog: add pthread.h to vndk log/log.h
Too many vendors assume that this is included, and it's not worth the
effort to clean up.

Bug: 165825252
Test: build
Change-Id: Ib99f0de4aac64134c21c0ee09f7ea576ebd0fe9e
2020-10-06 10:33:17 -07:00
The Android Open Source Project
7fd5f8d2bc DO NOT MERGE - Merge RP1A.201005.006
Bug: 168212094
Change-Id: I79a7517bce1c012eac2a2a7c1c2cffc5a3516eaa
Merged-In: Ia85067d4258bde4b875c832d6223db5dd26b8838
2020-10-06 04:30:21 +00:00
Tom Cherry
d5ed093bc0 Merge "logd: add locks around ~SerializedFlushToState" am: 1bacb2c389
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1449535

Change-Id: I54186d82ba2603a8076ff6714a8f2f59a330643a
2020-10-05 22:52:15 +00:00
Tom Cherry
1bacb2c389 Merge "logd: add locks around ~SerializedFlushToState" 2020-10-05 22:22:52 +00:00
Tom Cherry
28b9eb5130 logd: add locks around ~SerializedFlushToState
This accesses shared resources in SerializedLogBuffer and therefore
requires a lock.

Bug: 169736426
Test: malloc_debug_system_tests
Change-Id: I807c65f4719481f933b4917a50f83f933b1929fb
2020-10-05 14:59:19 -07:00
Treehugger Robot
939b78f03a Merge "liblog: add unistd.h to vndk log/log.h" am: bf426993db
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1449379

Change-Id: Ib330c53d055a1a89e25ed24269bda6926ac0e704
2020-10-05 21:32:38 +00:00
Treehugger Robot
bf426993db Merge "liblog: add unistd.h to vndk log/log.h" 2020-10-05 21:14:13 +00:00
Tom Cherry
c1fe112b9d Merge "Revert "Remove logging symlinks from system/core/include"" am: c274f9ff69
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1449381

Change-Id: Ic04ff0477b8fe8cfbc14019eac7f6001cae7f62b
2020-10-05 19:54:26 +00:00