Do not send 0 byte string content.
am: cafd355375

Change-Id: I8c2cc596f4f5d6e9ecc0de6130be8bdfcb19c1ec

Revert "Revert "Remove limit of system property name length""
am: 16b2a4de14

Change-Id: I585a55769cdca1d7f9edee479b5ec94d8821b673

Do not send 0 byte string content.

Trying to send even 0 bytes to closed socket leads to
broken pipe error. Sometimes property service is just
quick enough and closes the socket between send(valuelen)
and send(value) in the case where valuelen is 0.

Bug: http://b/34670529
Test: adb reboot 20 times and make sure phone service did not fail
Test: run bionic-unit-tests --gtest_filter=prop*
Change-Id: I96f90ca6fe1790614e7efd3015bffed1ef1e9040

Merge "[cfi] Handle large libraries correctly."
am: d8388cd303

Change-Id: I3032727f90e26837b69b558551ccd420ae7bc680

Revert "Revert "Remove limit of system property name length""

This reverts commit 489f58b5eaedd5a80635bb3a7b39e97037c585f6.
Bug: http://b/33926793
Bug: http://b/34670529
Test: Run bionic-unit-tests --gtest_filter=prop*

Change-Id: Id4e94652dc2310a21f5b7bd3af098bf79df3f380

Merge "[cfi] Handle large libraries correctly."

Merge "Revert "Remove limit of system property name length""
am: 40250bb683

Change-Id: Ic2565a4ad5d42146378364c06250260ca466cc98

Merge "Revert "Remove limit of system property name length""

Revert "Remove limit of system property name length"

This reverts commit 5c1ce278f31bc6fc338b9b951f1adc7c2b343d7c.
Bug: http://b/33926793
Bug: http://b/34670529

Change-Id: I0dc4a8ae55576c69b34b2958d8e664f7066b9c54

Merge "Removed include log\logger.h from log_fake.cpp"
am: f0b42ae973

Change-Id: Iba9012e9206234e104c355443803d278a861b336

Merge "Removed include log\logger.h from log_fake.cpp"

Removed include log\logger.h from log_fake.cpp

Test: Compile malloc_debug_unit_tests
Change-Id: I635687016570ba52c571435b87d8e89f7f04d609

Merge "Remove limit of system property name length"
am: 440d6c7644

Change-Id: I3ff9fb98065166cb53dfe153448e059df9ca6a71

Merge "Remove limit of system property name length"

Merge "Remove libc++ dependency from prebuilt test-libs"
am: 8bb01c0719

Change-Id: I2d28e3f2be50a9726e4712bf7f731c1ecd215e47

Merge "Remove libc++ dependency from prebuilt test-libs"

Merge "Mark ifunc tests failing on arm as known failures"
am: 3685e4515f

Change-Id: I1bf5509ecb820df66e28aff3b80b8c1cdb857b53

Merge "Mark ifunc tests failing on arm as known failures"

Mark ifunc tests failing on arm as known failures

Bug: http://b/27930475
Test: run bionic-unit-tests on arm and arm64
Change-Id: I5084b6a5ef4b56ca9985da63a5e6896f771810f0

Remove libc++ dependency from prebuilt test-libs

Bug: http://b/27549327
Test: run bionic-unit-tests --gtest_filter=dl*:Dl* on angler
Change-Id: Ia09ed2c00213bc07bf2a4c772e675419f7a30cb3

Remove limit of system property name length

This change introduces new __system_property_read_callback
method to use in place of deprecated __system_property_read
__system_property_set() and get() should just work but now
do not have limit on system property names.

Bug: http://b/33926793
Test: boot device, run adb shell propget
Test: boot device with old version of init (protocol v1)
Test: run bionic-unit-tests --gtest_filter=prop*
Change-Id: I619fb5a7e27a272aac30011579665f6160888bc7

[cfi] Handle large libraries correctly.

Fallback to unchecked if the shadow offset overflows int16_t.
This may happen when a library's data segment is larger than 256MB.

Also updated some comments.

Bug: 22033465
Test: bionic device tests

Change-Id: I8eef42f75099f24aed566499ff1731a0bbf01ff3

Merge "Add props files for each partition"
am: 7844b4c5db

Change-Id: I25c7789f689e3aadcca99e5fd8a6908db9bc57d2

Merge "Add props files for each partition"

Add props files for each partition

The following files will be added.
- /vendor/default.prop for default prop overrides from vendor
 partition.
- /odm/default.prop for default prop overrides from odm partition.
- /odm/build.prop for build prop overrides from odm partition.

Test: tested default/build prop files with enabling early mount, but
didn't test files of odm partition because odm partition doesn't
exist now.
Bug: 34116668
Change-Id: Ia99895a0afcdc522463d95b2f3e2841dcf2e3516

Merge "Add seccomp support library"
am: ad3b3995e7

Change-Id: I1115c8c39de622d594447a56d2cd0893991effac

Merge "Add seccomp support library"

Add seccomp support library

Policy library which exports an autogenerated policy from SYSCALLS.TXT
blocking any other calls.

Test: Generate policy, install onto Sailfish, check boots, Chrome runs,
calls are blocked.
Bug: 32313202

Change-Id: Ib590704e50122f077eeae26561eb9b0a70386551

Merge "Runtime support for CFI"
am: 082bec5b41

Change-Id: I47ea89f7161ff7c00a89a291ba7386c2b6f5c191

Merge "Runtime support for CFI"

Merge changes from topic 'kill_debuggerd'
am: 2d261e8e9e

Change-Id: Ibec7e7093bbe580cc126225c2f2d1f280de51a43

Rename libdebuggerd_client to libdebuggerd_handler.
am: 2a3b4fa6ec

Change-Id: Iac8466c0995571ce51a81f903903115442f5bc17

Reserve a realtime signal for debuggerd.
am: f8a6d4745d

Change-Id: If7eced969434e50beec04625c905ef374cdcdfb1

Merge changes from topic 'kill_debuggerd'

* changes:
  Rename libdebuggerd_client to libdebuggerd_handler.
  Reserve a realtime signal for debuggerd.

Runtime support for CFI

Control Flow Integrity support in bionic.

General design:
http://clang.llvm.org/docs/ControlFlowIntegrityDesign.html#shared-library-support

This CL implements subsections "CFI Shadow" and "CFI_SlowPath" in the above document.

Bug: 22033465
Test: bionic device tests
Change-Id: I14dfea630de468eb5620e7f55f92b1397ba06217

Merge "Declare dependency on android_filesystem_config.h"
am: 2f238b32cd

Change-Id: Ia69ebf6c7d1aeefc61ff301197d3d994a009b477

Merge "Declare dependency on android_filesystem_config.h"

Merge "Revert "property_contexts: split into platform and non-platform components""
am: dbfa1bd292

Change-Id: I7279216d6987fa853ee759997e2c571fc01ce5ed

Revert "property_contexts: split into platform and non-platform components"
am: e2f5ea14c5

Change-Id: I9a794b93b077791ce46016cb9f8965a52c98b88e

Merge "Revert "property_contexts: split into platform and non-platform components""

Revert "property_contexts: split into platform and non-platform components"

This reverts commit 8213615461f7d39620e44f73980acc6107c76ac7.

Change-Id: Id2982eafadc5d1c307b60c1f8abe78df8dc8b82e

Merge "property_contexts: split into platform and non-platform components"
am: 0b6d0a8b38

Change-Id: I10c9caa9f85f21e681b8ab02cd3ea5e8ef489eb0

property_contexts: split into platform and non-platform components
am: 8213615461

Change-Id: Ifa1bb1e295cef8ecf74a4a4702114a88f3e6859e

Merge "property_contexts: split into platform and non-platform components"

Declare dependency on android_filesystem_config.h

android_filesystem_config.h comes from a directory outside bionic/libc
so it can't be referenced directly, add it as a source file through
a filegroup module.

Bug: 34283327
Test: builds
Test: libc rebuilds after touch system/core/include/private/android_filesystem_config.h
Change-Id: I90f6b7b25b70842b8619d558074449f13e6e6b03

Rename libdebuggerd_client to libdebuggerd_handler.

Bug: http://b/30705528
Test: debuggerd_test
Change-Id: I625a9a2e7304d172873ba9344b74efa57ad7bdad

Reserve a realtime signal for debuggerd.

Allocate __SIGRTMIN + 3 for triggering native stack dumps (like SIGQUIT
for Java processes).

Bug: http://b/30705528
Test: debuggerd_test
Change-Id: I81d622fba32a651555268a8a2ba6721c61c93a58

Merge "linker: convert 'mov pc' instruction to 'bx'"
am: 8d0e0d4952

Change-Id: I8664911d4c068c9c719f85e44b7a5ac99670581f

Merge "linker: convert 'mov pc' instruction to 'bx'"

linker: convert 'mov pc' instruction to 'bx'

From Linux 3.17 6ebbf2ce437b33022d30badd49dc94d33ecfa498:

    ARMv6 and greater introduced a new instruction ("bx") which can be used
    to return from function calls.  Recent CPUs perform better when the
    "bx lr" instruction is used rather than the "mov pc, lr" instruction,
    and this sequence is strongly recommended to be used by the ARM
    architecture manual (section A.4.1.1).

Signed-off-by: Park Ju Hyung <qkrwngud825@gmail.com>
Test: No regressions detected
Test: Passes full CTS run

Change-Id: Ie268f9893e3df0f68fbfe82a13f3c7cc5c5909d8
Signed-off-by: Alex Naidis <alex.naidis@linux.com>

Merge "Switch bionic-unit-tests to libc++_static."
am: 37f1d11482

Change-Id: Ie427d277e5f1997a170e90cf98846cf1244d7a84

Merge "Switch bionic-unit-tests to libc++_static."

Merge "Describe our dynamic linker compatibility policy."
am: 98d200899b

Change-Id: I247f8d16edb201e251670e699a3a16244a4e510e

Merge "Describe our dynamic linker compatibility policy."

Switch bionic-unit-tests to libc++_static.

Bug: http://b/27549327
Test: run bionic-unit-tests, bionic-unit-tests-glibc.
Change-Id: I719899624dec83728a3d5b8169079e5c91c63c04

Describe our dynamic linker compatibility policy.

Bug: N/A
Test: N/A
Change-Id: I6f472d1ab22636de74fd103e9a480fb1ab51e314

Merge "Beef up process_vm_readv/writev tests"
am: a970ac50d1

Change-Id: Ic1628705232788fdd871c0e91afe933707a15169

Merge "Beef up process_vm_readv/writev tests"

Beef up process_vm_readv/writev tests

LLDB uses process_vm_readv quite extensively (it's an order of
magnitude faster than PTRACE_PEEKDATA). Add a bit more tests of
the function to make sure it stays working.

Change-Id: I64e17e6d56842f118a9485e3a18f42ca5e1b7577
Test: run the new test

Merge "localtime_r(3) should act as if it calls tzset(3)."
am: 1b2975d54b

Change-Id: I85c99ed325efbd119b5a7513b4cd997f9dc6b5aa

Merge "localtime_r(3) should act as if it calls tzset(3)."

Merge "Improve fmemopen tests."
am: 27fa754287

Change-Id: Ie62d2d2850fc4dee566317d5e0fcde54693694a1

Merge "Improve fmemopen tests."

localtime_r(3) should act as if it calls tzset(3).

See code comment.

Bug: http://b/31339449
Test: ran tests & benchmarks
Change-Id: I6b6a63750ef41664dc4698207e6a53e77cc28cdf

Merge "Add support for expected failures to gtest runner."
am: 94ea19895c

Change-Id: I91e4f3673d6715326a2f1a43937343e59e15048c

Merge "Add support for expected failures to gtest runner."

Improve fmemopen tests.

Tag fmemopen_NULL as a known failure, and make it reliably fail by
memsetting the buffer we read to.

Bug: http://b/33251022
Test: bionic-unit-tests/bionic-unit-tests64 on bullhead
Test: bionic-unit-tests-glibc --gtest_filter="*memopen*"
Change-Id: I381783282359851c9de47146dafbb5a291960c2a

Merge "Hide __system_property_add/update, deprecate find_nth"
am: 907fcc98b2

Change-Id: I68bbcf32432de9dba91116faae981701e122dbaa

Merge "Hide __system_property_add/update, deprecate find_nth"

Hide __system_property_add/update, deprecate find_nth

These functions are supposed to be used only by the
property service.

__system_property_find_nth is deprecated and no longer part
of NDK. Call to this function will result in abort for apps
targeting Android O.

Bug: http://b/34114501
Test: bionic-unit-tests --gtest_filter=prop*
Change-Id: I9846965bf248e2ddf45cd7b293618245bbd87145

Add support for expected failures to gtest runner.

Bug: http://b/33251022
Test: manually added tests that unexpectedly failed
Test: bionic-unit-tests/bionic-unit-tests64 on bullhead
Change-Id: Ie9aef80014b9fd6512b68f0e42abc298732a9b6e

Merge "Fix streaming(memcpy) performance on Cortex-A7"
am: 0625beb270

Change-Id: I24340a1b7d9e93505abf517e4ecc24175c58a164

Merge "Fix streaming(memcpy) performance on Cortex-A7"

Merge "Revert "Revert "move android_ids into bionic"""
am: 940d460f59

Change-Id: Id5e500e39cd544fab4398318ff48289764af1e59

Revert "Revert "move android_ids into bionic""
am: 3f6eee9793

Change-Id: I6a65710f926a76e48705fa4594f479f5d48df068

Merge "Revert "Revert "move android_ids into bionic"""

Merge "Dynamic linker errors should go to stderr."
am: 74e1fe1cc0

Change-Id: I045e0116e73a69989e80d8bd281d38ee6c5112b9

Merge "Dynamic linker errors should go to stderr."

property_contexts: split into platform and non-platform components

Bug: 33746484
Test: Successfully boot with original service and property contexts.
Test: Successfully boot with split serivce and property contexts.

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

Dynamic linker errors should go to stderr.

Also add the missing \n to the end of the error.

Addresses review comment.

Bug: N/A
Test: manual
Change-Id: I7c997a71e6e02381d21ac762595ba90370db9d05

Merge "Exit rather than abort if asked to run a non-PIE executable."
am: 71952ef625

Change-Id: Ibb2940eef382fabce0c2c5252fe962c145f5be95

Merge "Exit rather than abort if asked to run a non-PIE executable."

Merge "Move to .md files for even trivial documentation."
am: 39b6d59049

Change-Id: Ia1950ac9cf0eaa9ca19904ae54d2903471e77626

Merge "Move to .md files for even trivial documentation."

Move to .md files for even trivial documentation.

So it's automatically displayed for folks browsing the source.

Bug: N/A
Test: N/A
Change-Id: I80e823415f1fb12ad8ce5348e926519b6e29580f

Exit rather than abort if asked to run a non-PIE executable.

Each release we're asked to investigate tombstones from code that hasn't
been allowed to run on Android since L. This is just wasting our time,
and clearly the "obviousness" of aborting rather than exiting hasn't ensured
that all app developers rebuild their old binaries. In some cases it seems
like they run them "just in case" and don't care if they fail.

Bug: http://b/34112178
Test: ran libsupervisor.so from com.ss.android.article.news
Change-Id: I8a3f196c4755601a3888281566fbb7b817f01dca

Merge "Fix bionic-unit-tests-static crash"
am: 500b9be411

Change-Id: I98d7587228db28d920354f925b0d20e3734838df

Merge "Fix bionic-unit-tests-static crash"

Fix bionic-unit-tests-static crash

The crash happens because for static executables call to dlopen
results in crash. This change moves dlopen() == nullptr check
from static variable to a functions so that the dlopen crash only
affects tests calling dlopen but not the whole executable.

Also make static tests report crashes to debugerd

Test: run bionic-unit-tests-static (for arm 32 and 64)
Bug: http://b/34129417
Change-Id: I7c4d8caf2a43250234fe24496b1c95eab572769f

Merge "Make /dev/__properties__ 0711"
am: 11798bcc35

Change-Id: I4c418dd0567013b003541977f5b2f97a35ac29e3

Merge "Make /dev/__properties__ 0711"

Make /dev/__properties__ 0711

Don't allow processes to read the contents of the directory
/dev/__properties__. This is an implementation detail of the properties
system that processes shouldn't be concerned with.

Test: Device boots and no problems reading individual properties.
Test: ls -la /dev/__properties__ fails
Change-Id: I00130fe4529525935654bff91e3cc59253b86e26

Merge "Add declaration of tgkill to signal.h."
am: d26267b8b9

Change-Id: I398886a42d4d3d72b430159ac9c6dcaee5e26d09

Merge "Add declaration of tgkill to signal.h."

Add declaration of tgkill to signal.h.

Expose a useful function that we've had since Jelly Bean.

Bug: http://b/34111810
Test: TreeHugger
Change-Id: Iaf3097f224c09b533f36050cf21394ba148007ad

Merge "Downgrade dynamic section checks to warning"
am: fb07c36bc0

Change-Id: I84efe43a93ae0689492ddf229c52e6710e00bc4a

Merge "Downgrade dynamic section checks to warning"

Downgrade dynamic section checks to warning

For apps targeting pre-O releases print warn about
invalid dynamic section.

Bug: http://b/33842263
Test: Start an app in question and check logs for warning.
Test: Run bionic-unit-test --gtest_filter=dl*:Dl*
Change-Id: I80bd51f2133b956fc04e20a6538cccd50118c1b8

Fix streaming(memcpy) performance on Cortex-A7

Stream-mode detection for L1 in A7-core is failing for
non cache-line-size (non 64 byte) aligned addresses.
This leads to destination data getting cached unnecessarily.
This A7 issue is confirmed by ARM

This issue is solved by aligning destination address to 64 byte before
entering the loop in memcpy routine.
Though we get lower score for micro_bench memcpy when L1 cache is bypassed,
it is desirable since it avoids unnecessary eviction of other process data
from L1 which is good for overall system performance.

Higher micro_bench memcpy numbers for < 64byte alignment shows good numbers
but this is at the cost of L1 cache pollution. During memcpy/memset,
unnecessary data is filled in L1 cache, this causes eviction of other
process data from L1.
For example during msmset(0), L1 cache gets filled with 0s which should be
avoided.

Additionally, there is another issue with cortex A7 that impacts performance
for all alignments / all Android Wear versions:
Store Buffer on A7 is 32 byte which limits the 32-byte back to back stores.
In the current implementation back to back 32bytes writes is causing CPU stalls.
This issue can be solved by interleaved Loads and Stores.
This helps in avoiding CPU stalls during memcpy by utilizing efficiently the
A7 internal load and store buffers.

Change-Id: Ie5f12f2bb5d86f627686730416279057e4f5f6d0

Merge "versioner: start using C++17."
am: c816e9fa03

Change-Id: Ie148482169ab9cfae841a94b95b8266c520a45e6