summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Use demangle for function names.Christopher Ferris2017-06-022-2/+4
| | | | | | | Bug: 62141808 Test: Ran unit tests, ran debuggerd -b on android processes. Change-Id: I40f341bcb2b35dd0a020d245d6b6f2ed2df65481
* Merge "Fix handling of _ZLXX."Christopher Ferris2017-06-022-0/+10
|\
| * Fix handling of _ZLXX.Christopher Ferris2017-06-012-0/+10
| | | | | | | | | | | | | | | | Mangled names of the form _ZLXX, where XX is a number is now properly demangled. Test: Ran new unit tests. Change-Id: I89fbb36d2294c46da428c75bd1a2f39f69ac4b43
* | Merge "Change RegsTmpl to RegsImpl."Christopher Ferris2017-06-026-39/+35
|\ \ | |/ |/|
| * Change RegsTmpl to RegsImpl.Christopher Ferris2017-06-016-39/+35
|/ | | | | | | | | Also clang-format modified Regs.cpp slightly. Bug: 23762183 Test: Built and ran unit tests. Change-Id: I1c7c1b01974ee3f35059c42b8e2aef24d46c81a7
* Merge "adb: reunregress waiting for device on start-server."Treehugger Robot2017-06-023-20/+66
|\
| * adb: reunregress waiting for device on start-server.Josh Gao2017-05-313-20/+66
| | | | | | | | | | | | | | | | | | | | | | Move the invocation of adb_notify_device_scan_complete to the end of device_connected, where we decrement connecting_devices. Also, create a dedicated thread for handling hotplug events, instead of reusing the main thread for this, since the main thread blocks until device scan is complete. Test: `adb kill-server; adb devices` Change-Id: Ia73b1a57538174282a48ef73ab0a3e58152d6f83
* | Merge "Add DwarfSection classes."Christopher Ferris2017-06-0212-22/+1727
|\ \
| * | Add DwarfSection classes.Christopher Ferris2017-06-0112-22/+1727
| | | | | | | | | | | | | | | | | | | | | Bug: 23762183 Test: Ran new unit tests. Change-Id: Icca2a73c50d467718ba4ac41e1c8f541488620dd
* | | Merge changes Iddb0cb1e,Ic15e0b08,If1c9adb6Treehugger Robot2017-06-013-5/+33
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | * changes: debuggerd_client: increase pipe buffer size to max. crash_dump: don't notify ActivityManager if it crashed. crash_dump: clear the signal mask.
| * | | debuggerd_client: increase pipe buffer size to max.Josh Gao2017-06-012-0/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If a process tries to dump itself (e.g. system_server during ANRs), crash_dump will block trying to write to its pipe if it's not sufficiently large. Increase the pipe size to the max, and add a test to make sure that it's always at least 1MB (the default value). Bug: http://b/38427757 Test: debuggerd_test Change-Id: Iddb0cb1e5ce9e687efa9e94c2748a1edfe09f119
| * | | crash_dump: don't notify ActivityManager if it crashed.Josh Gao2017-06-011-3/+8
| | | | | | | | | | | | | | | | | | | | | | | | Bug: http://b/38427757 Test: killall -ABRT system_server, plus added logging Change-Id: Ic15e0b0870b1ec08a2f165ad0e5356afed02eece
| * | | crash_dump: clear the signal mask.Josh Gao2017-06-011-2/+7
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | crash_dump inherits its signal mask from the thread that forked it, which always has all of its signals blocked, now that sigchain respects sa_mask. Manually clear the signal mask, and reduce the timeout to a still-generous 2 seconds. Bug: http://b/38427757 Test: manually inserted sleep in crash_dump Change-Id: If1c9adb68777b71fb19d9b0f47d6998733ed8f52
* | | Merge "Fix bug found by fuzzer."Christopher Ferris2017-06-014-4/+79
|\ \ \
| * | | Fix bug found by fuzzer.Christopher Ferris2017-05-314-4/+79
| |/ / | | | | | | | | | | | | | | | | | | Also, add the demangle fuzzer code. Test: Ran fuzzer, ran new unit tests. Change-Id: If3e15e10af88b81602a8a0f0bfe071a015f6000b
* | | Merge "fs_mgr: Fix ZRAM error when ZRAM disk size beyond (2^31-1) bytes"Treehugger Robot2017-06-011-1/+1
|\ \ \
| * | | fs_mgr: Fix ZRAM error when ZRAM disk size beyond (2^31-1) bytesJinguang Dong2017-06-011-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Because the zram_size type is unsigned int.so if ZRAM size great than 2^31 -1, zram_fp will receive a negtive integer, while the ZRAM driver only accept natural number.We need to use printf formatting %u instand of %d. Test: 1. Config the zramdisk size 2348810240 and build a ramdisk 2. Reflash device and check below command: $adb shell dumpsys meminfo $ adb shell cat /sys/block/zram0/disksize ZRAM info display will be abnormal 3. Config the zramdisk size 2348810240 and apply with this patch. 4. Retest to step 2 and the ZRAM info will be ok. Change-Id: I473de33fbd0b66cf13eac3172684e9fef11b6ef0
* | | | Merge "cgroups used by init must be mounted before any services start"Treehugger Robot2017-06-011-10/+10
|\ \ \ \
| * | | | cgroups used by init must be mounted before any services startRobert Benea2017-05-311-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | init uses /acct and optionally /dev/memcg for tracking services and therefore these must be started before any services start. Test: check that cgroups are mounted appropriately. Change-Id: Ice095287963181fe687dbe6b7d291076e674d1cc
* | | | | Merge "fastboot: use filename instead of fd to generate filesystem image"Treehugger Robot2017-06-013-15/+32
|\ \ \ \ \
| * | | | | fastboot: use filename instead of fd to generate filesystem imageJin Qian2017-05-313-15/+32
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | mke2fs tool takes a filename and has its own open function to handle output file. Change fastboot in preparation to switch from make_ext4 to mke2fs. Test: fastboot format:ext4 userdata Bug: 35219933 Change-Id: I7a31cb215f443a4a7cb0bfc23ec28c121a6101e6
* | | | | Merge "logcat: logcatd: make logcatd killable"Treehugger Robot2017-05-311-0/+1
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | logcat: logcatd: make logcatd killableMark Salyzyn2017-05-311-0/+1
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Near last resort kill logcatd, set /proc/self/oom_score_adj to -600. Can kill to keep system_server, telephony and bluetooth. Test: manual, start logpersist, runs Bug: 62147352 Change-Id: I3723761bf1d2b79ce6a9557d2e78f91cc5497e96
* | | | Merge "logging: make LOG(FATAL) always run with the static analyzer."Treehugger Robot2017-05-311-1/+5
|\ \ \ \
| * | | | logging: make LOG(FATAL) always run with the static analyzer.George Burgess IV2017-05-311-1/+5
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ::android::base::GetMinimumLogSeverity() is defined externally, so the static analyzer was allowed to assume that we continue executing after a LOG(FATAL). I manually audited all of the code I have access to, and the only "change the minimum log severity" statements I can see keep FATAL enabled (...and continuing after a FATAL is highly sketchy to me anyway). (I'm sure I tested this at some point in making the previous patch. I probably broke it in a refactor before sending it out for review; my bad. :) ) Bug: None Test: m without the static-analyzer builds; m with it yields fewer false positives. Change-Id: I216cd2034e1daa8d6f6c5e776f64b4cce88bb938
* | | | Merge "tombstoned: allow intercepts for java traces."Narayan Kamath2017-05-3116-90/+237
|\ \ \ \
| * | | | tombstoned: allow intercepts for java traces.Narayan Kamath2017-05-3116-90/+237
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | All intercept requests and crash dump requests must now specify a dump_type, which can be one of kDebuggerdNativeBacktrace, kDebuggerdTombstone or kDebuggerdJavaBacktrace. Each process can have only one outstanding intercept registered at a time. There's only one non-trivial change in this changeset; and that is to crash_dump. We now pass the type of dump via a command line argument instead of inferring it from the (resent) signal, this allows us to connect to tombstoned before we wait for the signal as the protocol requires. Test: debuggerd_test Change-Id: I189b215acfecd08ac52ab29117e3465da00e3a37
* | | | Merge "adb: fix deadlock."Treehugger Robot2017-05-311-5/+9
|\ \ \ \
| * | | | adb: fix deadlock.Josh Gao2017-05-301-5/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | libusb hotplug callbacks are called with the libusb lock taken, and we call into libusb with our local mutex, so we need to enforce an ordering between the two. Instead of calling device_connected or device_disconnected directly, enqueue them onto the main thread. Bug: http://b/62200735 Test: manually hotplugged a device Change-Id: Ic5d55db83b47f4bb60f124ce94ddfe06f5f1a0c6
* | | | | Merge "bootstat: Log factory_reset_boot_complete separately from ota_*."Treehugger Robot2017-05-301-3/+5
|\ \ \ \ \
| * | | | | bootstat: Log factory_reset_boot_complete separately from ota_*.James Hawkins2017-05-301-3/+5
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | Bug: 37164359 Test: none Change-Id: I7ed68e299846050fdffa58db8d2bd35fe4599dbb
* | | | | Merge "init: allow ':' in property names"Tom Cherry2017-05-301-2/+2
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | init: allow ':' in property namesTom Cherry2017-05-301-2/+2
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | Bug: 62114389 Test: boot bullhead, get/set properties with ':' via command line Test: trigger an init trigger with a property containing a ':' Change-Id: Ib51853a1ef9d4f79d510c8175c0d9684e2025e23
* | | | Merge "add a performance benchmark for libziparchive"Treehugger Robot2017-05-303-0/+114
|\ \ \ \
| * | | | add a performance benchmark for libziparchiveSebastian Pop2017-05-303-0/+114
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds two benchmarks that measure the performance of some operations of libziparchive. Both benchmarks are creating a temporary zip file containing file names of uniformly distributed lengths. The creation of the zip file is not timed in the benchmarks. - FindEntry_no_match tries to find an inexisting entry in the files of the zip archive, in order to force the code to examine all the files in the archive. - Iterate_all_files uses the iterate function to list all the files in the archive. Bug: N/A Test: adb shell /data/ziparchive-benchmarks Change-Id: Ibdb524ba1c5ae55caddf0416ebbc09f8b6df0021
* | | | | Merge "Respect that status property when parsing fstab from device tree"Treehugger Robot2017-05-301-0/+9
|\ \ \ \ \
| * | | | | Respect that status property when parsing fstab from device treeSandeep Patil2017-05-301-0/+9
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For an fstab entry in device tree, fs_mgr now honors the status property as done by linux. i.e. the node (in this case, the fstab entry) is enabled if status is not set, "ok" or "okay". For every other value, the node is considered as disabled. Merged-In: I5ff8f710de2c54afc76b4af28108ca9075357ad1 Bug: 62127741 Test: Test sailfish w/ no status property, result: boots Test: Test with status = "disabled", result: skips mounting /vendor Test: Test with status = "ok", result: boots Change-Id: I5ff8f710de2c54afc76b4af28108ca9075357ad1 Signed-off-by: Sandeep Patil <sspatil@google.com> (cherry picked from commit be4302bf582ef9db2387b098bf80b4e5b6d1f0e9)
* | | | | Merge "init.rc: import /vendor/etc/init/hw/init.${ro.hardware}.rc"Treehugger Robot2017-05-301-0/+1
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | init.rc: import /vendor/etc/init/hw/init.${ro.hardware}.rcYueyao Zhu2017-05-251-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | /init.${ro.hardware}.rc would sometime rely on being parsed before other .rc files. In this case all vendors are still able to have a .rc file included before all the ones in /vendor/etc/init. Merged-In: I3fb6df13a39204a516874ea94f5e5ad84bca42c6 Bug: 38301110 Change-Id: I3fb6df13a39204a516874ea94f5e5ad84bca42c6 (cherry picked from commit f7c34ad4bf1f4afdb7f901bcf908128ad7c2ca3a)
* | | | | Merge "logging: always enable DCHECK for the static analyzer"Treehugger Robot2017-05-261-2/+2
|\ \ \ \ \ | |_|_|_|/ |/| | | |
| * | | | logging: always enable DCHECK for the static analyzerGeorge Burgess IV2017-05-261-2/+2
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This helps us have less false-positives. We do this instead of `#undef NDEBUG`, since undefing NDEBUG actually gave us more false-positives (...and build breakages) than simply leaving it defined. Bug: None Test: Ran the static analyzer across internal master. 213 fewer warnings (15 Medium, 2 Low, the remainder are 'Analyzer'). All of the dropped warnings I audited were false-positives. It adds ~3 Tidy warnings. Change-Id: Ibedab60ca7e9d2b0772896222b83d2e7ab064afe
* | | | Merge "ueventd: Break devices.cpp into discrete classes"Tom Cherry2017-05-2515-878/+1110
|\ \ \ \
| * | | | ueventd: Break devices.cpp into discrete classesTom Cherry2017-05-2515-878/+1110
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | devices.cpp handles too many things for creating one class. This change breaks it up into various files and classes. * Parsing is moved to ueventd_parser.cpp * Reading from the uevent socket and Cold booting is moved to a UeventListener class, in uevent_listener.cpp * Firmware handling is moved to firmware_handler.cpp * The remaining contents form a DeviceHandler class within devices.cpp Bug: 33785894 Test: boot bullhead x40, observe no major differences in /dev and /sys Test: boot sailfish x40, observe no major differences in /dev and /sys Test: init unit tests Change-Id: I846a2e5995fbb344c7a8e349065c18a934fa6aba
* | | | Merge "Fix warning on the build servers"Treehugger Robot2017-05-251-1/+1
|\ \ \ \ | |_|/ / |/| | |
| * | | Fix warning on the build serversDan Willemsen2017-05-251-1/+1
|/ / / | | | | | | | | | | | | | | | | | | | | | ANDROID_BUILD_TOP is only defined if envsetup.sh is sourced, the current directory in make is the top of the source tree. Test: <fresh terminal> make -j nothing Change-Id: I24834a6eeb6ac11de0cc19d84650daa086995d99
* | | Merge "Load vendor public libraries to sphal namespace"Treehugger Robot2017-05-254-16/+71
|\ \ \
| * | | Load vendor public libraries to sphal namespaceDimitry Ivanov2017-05-254-16/+71
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Load vendor public libraries to sphal namespace if it exists - preserve old behavior of loading these libraries to default namespace if sphal namespace is not present on the device. Bug: http://b/37410104 Test: cts-tradefed run singleCommand cts --skip-preconditions -m CtsJniTestCases on marlin (with enabled sphal configuration) and on angler where ld.config.txt is not present. Change-Id: Iaa3fa437ba2900acc2e5b9c78039fe1553e4c9dd (cherry picked from commit af0264bbe9f5e1228eb8fb486fa3d0d8e6e8605e)
* | | | Merge "Revert "Load {default,build}.prop from /{system,vendor,odm}/etc/""Elliott Hughes2017-05-252-32/+17
|\ \ \ \
| * | | | Revert "Load {default,build}.prop from /{system,vendor,odm}/etc/"Hung-ying Tyan2017-05-252-32/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 723f1a2b318a455983d21ccf3598885f766bb8c9. Bug: 62075509
* | | | | Merge "liblog: test: CTS: enoent does not qualify"Mark Salyzyn2017-05-251-1/+3
|\ \ \ \ \ | |_|/ / / |/| | | |
generated by cgit v1.2.3 (git 2.25.1) at 2024-12-26 15:53:01 +0000