Remove c++14'isms from system_properties

Change-Id: If78e7d2770e8f8321f0d1824c3c52f93820dd325

Don't use StringPrintf() in gtest runner.

Change-Id: I0cd0b3cbb952c65b1c449e88ce12964b93029538

Merge "Separate properties by selabel" am: 376b94f1e7 am: 84bc4f5cc7
am: 555da4236c

* commit '555da4236ccbabd871558e2e2febfe5b05425286':
  Separate properties by selabel

Merge "Separate properties by selabel" am: 376b94f1e7
am: 84bc4f5cc7

* commit '84bc4f5cc765f54c18c3674e775de5fe88c1e499':
  Separate properties by selabel

Merge "Separate properties by selabel"
am: 376b94f1e7

* commit '376b94f1e777375eb9ebb04a882b898aafa94625':
  Separate properties by selabel

Merge "Separate properties by selabel"

Merge "Improve unit test runner." am: f57ae1cd43 am: b561043d3a
am: 0ce7d168da

* commit '0ce7d168da911712b1e75fce663d14f14cabadd7':
  Improve unit test runner.

Merge "Re-add putw for LP32" am: 22dca83e1c am: 5bfa22ceab
am: f3e4983f6c

* commit 'f3e4983f6cf79a2fad7c3a98609f86b5f7dddcb9':
  Re-add putw for LP32

Merge "Improve unit test runner." am: f57ae1cd43
am: b561043d3a

* commit 'b561043d3af664d87bda3519e9e06dbfc8a24130':
  Improve unit test runner.

Merge "Improve unit test runner."
am: f57ae1cd43

* commit 'f57ae1cd433f2bdc02cec6b0d2666f7708f1fc37':
  Improve unit test runner.

Merge "Improve unit test runner."

Improve unit test runner.

1. Read unit test's output while the test is running. Previously
we only read output when the test finishes, which has trouble
when the test outputs too many stuff.
2. Report failed unit test's exit code. It is useful when the
test doesn't fail in ASSERT_xxx, but in somewhere else.

Bug: 25392375
Change-Id: Ie90823337f7c2ee25fa489a5534801d991258f95

Merge "Re-add putw for LP32" am: 22dca83e1c
am: 5bfa22ceab

* commit '5bfa22ceabfcb418337d262967172a90dc573e4b':
  Re-add putw for LP32

Merge "Re-add putw for LP32"
am: 22dca83e1c

* commit '22dca83e1c37d3b465b9fa432bbf7cb4bd0df868':
  Re-add putw for LP32

Merge "Re-add putw for LP32"

Re-add putw for LP32

This was missed when switching to LOCAL_SRC_FILES_EXCLUDE

Change-Id: I6ea23c9eb31abe11e0ec4abfc2ee2f2a43c76ce2

Separate properties by selabel

The purpose of this change is to add read access control to the property
space.

In the current design, a process either has access to the single
/dev/__properties__ file and therefore all properties that it contains
or it has access to no properties.  This change separates properties
into multiple property files based on their selabel, which allows
creation of sepolicies that allow read access of only specific sets of
properties to specific domains.

Bug 21852512

Change-Id: Ice265db79201ca811c6b6cf6d851703f53224f03

Merge "Explicitly disallow default c-tor" am: b24f7a0513 am: 9f525b66eb
am: 64ffc84d4d

* commit '64ffc84d4de667e1bcca2f38bd542e788c6d8063':
  Explicitly disallow default c-tor

Merge "Explicitly disallow default c-tor" am: b24f7a0513
am: 9f525b66eb

* commit '9f525b66eb0ef7fd03d9387ec1efc1ee833f5077':
  Explicitly disallow default c-tor

Merge "Explicitly disallow default c-tor"
am: b24f7a0513

* commit 'b24f7a0513f7468ccf7563d7d347c6e7bab391d9':
  Explicitly disallow default c-tor

Merge "Explicitly disallow default c-tor"

Explicitly disallow default c-tor

Change-Id: Ia52995a459443159e80383d5b396c3edd90a08ae

Merge "Add bionic-unit-tests-gcc{32,64}, compiled with gcc." am: 9cbabd8fe5 am: d74566026d
am: faee342e14

* commit 'faee342e143e308ebf4f68945d2c400541170215':
  Add bionic-unit-tests-gcc{32,64}, compiled with gcc.

Merge "Add bionic-unit-tests-gcc{32,64}, compiled with gcc." am: 9cbabd8fe5
am: d74566026d

* commit 'd74566026d439e915ce56c9cf739181187d05192':
  Add bionic-unit-tests-gcc{32,64}, compiled with gcc.

Merge "Add bionic-unit-tests-gcc{32,64}, compiled with gcc."
am: 9cbabd8fe5

* commit '9cbabd8fe5ef3834682996ff23b4a5325e0586cb':
  Add bionic-unit-tests-gcc{32,64}, compiled with gcc.

Merge "Add bionic-unit-tests-gcc{32,64}, compiled with gcc."

Merge "Init stdio in __libc_init_common." am: 6bef152af2 am: 1f4b0c44b3
am: 2b941035dd

* commit '2b941035dd13f2b411fee5549883a1a8b6b9a30e':
  Init stdio in __libc_init_common.

Merge "Enable using clang to build __cxa_thread_atexit_impl." am: af87c9ccfc am: 4262d3e9ac
am: b2ddaafc68

* commit 'b2ddaafc68aa87c29ea0afb8bbf48ab08c5cd63e':
  Enable using clang to build __cxa_thread_atexit_impl.

Merge "Init stdio in __libc_init_common." am: 6bef152af2
am: 1f4b0c44b3

* commit '1f4b0c44b37bbf840a8210c8948e1d6efb0cdc11':
  Init stdio in __libc_init_common.

Merge "Enable using clang to build __cxa_thread_atexit_impl." am: af87c9ccfc
am: 4262d3e9ac

* commit '4262d3e9ac2462502875f5a7f1925a44ce29b299':
  Enable using clang to build __cxa_thread_atexit_impl.

Merge "Init stdio in __libc_init_common."
am: 6bef152af2

* commit '6bef152af2c622ee0c57f9c374b76f3dd352e52b':
  Init stdio in __libc_init_common.

Merge "Enable using clang to build __cxa_thread_atexit_impl."
am: af87c9ccfc

* commit 'af87c9ccfc3afbc7db9f0bc131d122453b5c882d':
  Enable using clang to build __cxa_thread_atexit_impl.

Merge "Init stdio in __libc_init_common."

Merge "Enable using clang to build __cxa_thread_atexit_impl."

Enable using clang to build __cxa_thread_atexit_impl.

Remove previous workaround as we no longer use
__thread in __cxa_thread_atexit_impl.cpp.

Change-Id: Ic1062995db488859b341acdda0b5f6635e10d7e8

Merge "Don\'t use __thread in __cxa_thread_finalize()." am: 28d3f00cf4 am: 51ca18d2a1
am: 5f6ff42bb6

* commit '5f6ff42bb615ce71e7bed66b45ea27fc29d2c4f4':
  Don't use __thread in __cxa_thread_finalize().

Merge "Don\'t use __thread in __cxa_thread_finalize()." am: 28d3f00cf4
am: 51ca18d2a1

* commit '51ca18d2a11eedff94a11c1cb996f38c2011b0dd':
  Don't use __thread in __cxa_thread_finalize().

Merge "Don\'t use __thread in __cxa_thread_finalize()."
am: 28d3f00cf4

* commit '28d3f00cf4545812503c835ea906fa83309e48ed':
  Don't use __thread in __cxa_thread_finalize().

Merge "Don't use __thread in __cxa_thread_finalize()."

Init stdio in __libc_init_common.

Previously we call __sinit() lazily. But it is likely to cause data
races like in https://android-review.googlesource.com/#/c/183237/. So
we prefer to call __sinit() explicitly at libc initialization.

Bug: 25392375

Change-Id: I181ea7a4b2e4c7350b45f2e6c86886ea023e80b8

Merge "Fix pthread_test according to tsan report." am: aec13988da am: 6ac2fdd0f4
am: 9df638fcd9

* commit '9df638fcd940428fc51d2a5583bd473af2b07f29':
  Fix pthread_test according to tsan report.

Merge "Fix pthread_test according to tsan report." am: aec13988da
am: 6ac2fdd0f4

* commit '6ac2fdd0f4926a3d5c8e1424c4cdf5d82805c288':
  Fix pthread_test according to tsan report.

Merge "Fix pthread_test according to tsan report."
am: aec13988da

* commit 'aec13988dab7ff32cb005f42b952e3b9b55a4779':
  Fix pthread_test according to tsan report.

Merge "Fix pthread_test according to tsan report."

Merge "Consider  when building libc_thread_atexit_impl module" am: 2bb85c848a am: 4f94b7bfc0
am: 2b834401ef

* commit '2b834401ef62ce639070450d12623632e067c5df':
  Consider $(use_clang) when building libc_thread_atexit_impl module

Merge "Consider  when building libc_thread_atexit_impl module" am: 2bb85c848a
am: 4f94b7bfc0

* commit '4f94b7bfc082cd6f52d02a789512d1e6a6a01af6':
  Consider $(use_clang) when building libc_thread_atexit_impl module

Merge "Consider  when building libc_thread_atexit_impl module"
am: 2bb85c848a

* commit '2bb85c848abb89951ec01342d6c8278311346395':
  Consider $(use_clang) when building libc_thread_atexit_impl module

Merge "Consider $(use_clang) when building libc_thread_atexit_impl module"

Add bionic-unit-tests-gcc{32,64}, compiled with gcc.

* bionic-unit-tests{32,64} are compiled with clang/llvm.
* Skip one single test in __cxa_thread_atexit_test.cpp
  when compiled with aarch64 clang/llvm.
  Aarch64 clang/llvm generates relocation references to
  "thread_local" symbols not supported by Android linker.

BUG: 25642296
Change-Id: Ia0497b79c4b335228afeb48a26e0592217909953

Consider $(use_clang) when building libc_thread_atexit_impl module

Change-Id: Ia369d94979f418198a2ee891385942d9bc5604d7

Don't use __thread in __cxa_thread_finalize().

Currently we use __thread variable to store thread_local_dtors,
which makes tsan test fork_atexit.cc hang. The problem is as below:
The main thread creates a worker thread, the worker thread calls
pthread_exit() -> __cxa_thread_finalize() -> __emutls_get_address()
-> pthread_once(emutls_init) -> emutls_init().
Then the main thread calls fork(), the child process cals
exit() -> __cxa_thread_finalize() -> __emutls_get_address()
-> pthread_once(emutls_init).
So the child process is waiting for pthread_once(emutls_init)
to finish which will never occur.

It might be the test's fault because POSIX standard says if a
multi-threaded process calls fork(), the new process may only
execute async-signal-safe operations until exec functions are
called. And exit() is not async-signal-safe. But we can make
bionic more reliable by not using __thread in
__cxa_thread_finalize().

Bug: 25392375
Change-Id: Ife403dd7379dad8ddf1859c348c1c0adea07afb3

Merge "Remove cflags and ldflags from crt defaults" am: a01108d9bf am: a36d1e0d31
am: a31e637e0a

* commit 'a31e637e0a3f33c06ceff6efa38b29bc1b2e598c':
  Remove cflags and ldflags from crt defaults

Merge "Remove cflags and ldflags from crt defaults" am: a01108d9bf
am: a36d1e0d31

* commit 'a36d1e0d31727d645d0f6b162489eecb8e8e2f22':
  Remove cflags and ldflags from crt defaults

Merge "Remove cflags and ldflags from crt defaults"
am: a01108d9bf

* commit 'a01108d9bf415854b9d36334483133adff3d7405':
  Remove cflags and ldflags from crt defaults

Merge "Remove cflags and ldflags from crt defaults"

Merge "move arch variant structs down a level" am: 8bd27182c5 am: 912e27a4b2
am: aba3af1956

* commit 'aba3af19566b4cd7b4d28a0910ea6120f0764ba0':
  move arch variant structs down a level

Merge "move arch variant structs down a level" am: 8bd27182c5
am: 912e27a4b2

* commit '912e27a4b255e85546bf64fc1f3e68398525ff1c':
  move arch variant structs down a level

Merge "move arch variant structs down a level"
am: 8bd27182c5

* commit '8bd27182c505cdc8031ab23fea5d1dccb6b7405a':
  move arch variant structs down a level

Merge "move arch variant structs down a level"

Remove cflags and ldflags from crt defaults

After ToolchainCflags were added to soong crt no longer needs to try to
recreate the bare minimum cflags for compiling for each architecture.
Also always use GCC to match crt.mk

Change-Id: I01a833ab70d989033c84f072e3660d060189688c

Merge "Introduce anonymous namespace" am: db8caa740a am: 321314f2e9
am: 1efba58604

* commit '1efba586043a97812ea86347099bd7cbabd94594':
  Introduce anonymous namespace

Merge "Introduce anonymous namespace" am: db8caa740a
am: 321314f2e9

* commit '321314f2e90d33ef6a653080784754d483bb6d44':
  Introduce anonymous namespace

Merge "Introduce anonymous namespace"
am: db8caa740a

* commit 'db8caa740a409c2ce75f875df84857e6be69eb9f':
  Introduce anonymous namespace

Merge "Introduce anonymous namespace"

Introduce anonymous namespace

The anonymous namespace is introduced to
handle cases when linker can not find the
caller. This usually happens when caller
code was not loaded by dynamic linker;
for example mono-generated code.

Bug: http://b/25844435
Bug: http://b/22548808
Change-Id: I9e5b1d23c1c75bc78548d68e79216a6a943a33cf

move arch variant structs down a level

Use blueprint's new anonymous embedded struct feature to move the arch
variant properties down a level, replacing arch.cortex_a9.srcs with
arch.arm.cortex_a9.srcs, while still supporting top-level properties
like arch.arm.srcs.

Change-Id: Ib41c80e3549440d5efdfd293a15cffa3f51a0fe4

Merge "Change _stdio_handles_locking into _caller_handles_locking." am: 17554356cc am: edcee4e401
am: 2327635f6e

* commit '2327635f6ea2ef519c334283d1bf048e2116a5f0':
  Change _stdio_handles_locking into _caller_handles_locking.

Merge "Change _stdio_handles_locking into _caller_handles_locking." am: 17554356cc
am: edcee4e401

* commit 'edcee4e4011b1a6e327791493ee1abde3ded2142':
  Change _stdio_handles_locking into _caller_handles_locking.

Merge "Change _stdio_handles_locking into _caller_handles_locking."
am: 17554356cc

* commit '17554356cc865908e5cbd7814c8be7e47c9c4ec5':
  Change _stdio_handles_locking into _caller_handles_locking.

Merge "Change _stdio_handles_locking into _caller_handles_locking."

Merge "Handling invalid section headers" am: a7fc7f9909 am: a8f8cbb3fc
am: fd7abc5dc9

* commit 'fd7abc5dc9d384edb5b75faa97d96a1d56164e52':
  Handling invalid section headers

Merge "Handling invalid section headers" am: a7fc7f9909
am: a8f8cbb3fc

* commit 'a8f8cbb3fcfcb654474ff041013cc081a70f4812':
  Handling invalid section headers

Merge "Handling invalid section headers"
am: a7fc7f9909

* commit 'a7fc7f9909c221a0f64c5c5ecc5fadd5fba467c5':
  Handling invalid section headers

Merge "Handling invalid section headers"

Handling invalid section headers

The linker crashes if native library has invalid section
headers. This change adds boundary checks on target offsets
and generates dlerror instead of crash.

Bug: http://b/25800330
Change-Id: Ibe282029997302b9b557637c3aad064d7d0febc5

Merge "Use FUTEX_WAIT_BITSET to avoid converting timeouts." am: 74ed96d597 am: 4469973bfb
am: 386aa431b5

* commit '386aa431b539feb537b9fd3a5f64b1ab45a635e6':
  Use FUTEX_WAIT_BITSET to avoid converting timeouts.

Merge "Use FUTEX_WAIT_BITSET to avoid converting timeouts." am: 74ed96d597
am: 4469973bfb

* commit '4469973bfbfc672e006a24242a149c2e03d623f1':
  Use FUTEX_WAIT_BITSET to avoid converting timeouts.

Merge "Use FUTEX_WAIT_BITSET to avoid converting timeouts."
am: 74ed96d597

* commit '74ed96d59731aa3661494330804d866e825209d3':
  Use FUTEX_WAIT_BITSET to avoid converting timeouts.

Merge "Use FUTEX_WAIT_BITSET to avoid converting timeouts."

Change _stdio_handles_locking into _caller_handles_locking.

It is reported by tsan that funlockfile() can unlock an unlocked mutex.
It happens when printf() is called before fopen() or other stdio stuff.
As FLOCKFILE(fp) is called before __sinit(), _stdio_handles_locking is false,
and _FLOCK(fp) will not be locked. But then cantwrite(fp) in __vfprintf()
calls__sinit(), which makes _stdio_handles_locking become true, and
FUNLOCKFILE(fp) unlocks _FLOCK(fp).

Change _stdio_handles_locking into _caller_handles_locking,
so __sinit() won't change its value. Add test due to my previous fault.

Bug: 25392375
Change-Id: I483e3c3cdb28da65e62f1fd9615bf58c5403b4dd

Merge "Move some utility functions to linker_utils" am: fb3219fbd1 am: 8bd9e9ec69
am: 1299ef37f7

* commit '1299ef37f772500a325b2920e46d1a4b69942f2f':
  Move some utility functions to linker_utils

Merge "Move some utility functions to linker_utils" am: fb3219fbd1
am: 8bd9e9ec69

* commit '8bd9e9ec697c052f4de00903188f6d01ff688cef':
  Move some utility functions to linker_utils

Merge "Move some utility functions to linker_utils"
am: fb3219fbd1

* commit 'fb3219fbd1dbb0a369d52cbd0200330fd8852bc1':
  Move some utility functions to linker_utils

Merge "Move some utility functions to linker_utils"

Move some utility functions to linker_utils

Also adds unit-tests for page_start, page_offset, and safe_add

Change-Id: Ia1325b4682d367328a01599a19848e4ffcd2c0ea

Merge "Improve error message for files with no sections" am: a1ab0d8ed1 am: d34c33764e
am: d1487e9169

* commit 'd1487e9169362a2736c33e7da8c67a7a901ebcb4':
  Improve error message for files with no sections

Merge "Improve error message for files with no sections" am: a1ab0d8ed1
am: d34c33764e

* commit 'd34c33764e0af32f9065c506b8bfb801156e8f98':
  Improve error message for files with no sections

Merge "Improve error message for files with no sections"
am: a1ab0d8ed1

* commit 'a1ab0d8ed1c4025233b0d816164eefb1b84159b9':
  Improve error message for files with no sections

Merge "Improve error message for files with no sections"

Merge "Disable clang for mips/mips64 libc." am: 8d5fb0d787 am: bb06a84672
am: d8adbe4a9c

* commit 'd8adbe4a9ccf99135341704386ff167f126f4e21':
  Disable clang for mips/mips64 libc.

Merge "Disable clang for mips/mips64 libc." am: 8d5fb0d787
am: bb06a84672

* commit 'bb06a846727ecdda6227d95d4fa8570e0cdf3e11':
  Disable clang for mips/mips64 libc.

Merge "Disable clang for mips/mips64 libc."
am: 8d5fb0d787

* commit '8d5fb0d78716379d07eddbd75c88c9b7082436ef':
  Disable clang for mips/mips64 libc.

Improve error message for files with no sections

Bug: http://b/25801618
Change-Id: I4f4f368e727ff48c84781279e3d17d4ac2d1b6b0

Merge "Disable clang for mips/mips64 libc."

Disable clang for mips/mips64 libc.

* Many processes, including adbd, failed to start in mips/mips64
  emulator when libc.so was compiled by clang.

BUG: 25291096
Change-Id: If3434ebdca4a3a6bf6102b120ee838a7ab66cd74

Merge "mmap: fix calculation of is_private_anonymous variable" am: 43c9045017 am: cb3af215dc
am: 5cecfce9ff

* commit '5cecfce9ff049300c015fc12d3e966e0ed772e62':
  mmap: fix calculation of is_private_anonymous variable

Merge "mmap: fix calculation of is_private_anonymous variable" am: 43c9045017
am: cb3af215dc

* commit 'cb3af215dc67899728bc3ef9d33f82358bee1335':
  mmap: fix calculation of is_private_anonymous variable

Merge "mmap: fix calculation of is_private_anonymous variable"
am: 43c9045017

* commit '43c90450174ab8839c05702ac01c4092f5b6cd19':
  mmap: fix calculation of is_private_anonymous variable

Merge "mmap: fix calculation of is_private_anonymous variable"

mmap: fix calculation of is_private_anonymous variable

Currently is_private_anonymous is calculated as true if _either_
MAP_PRIVATE or MAP_ANONYMOUS is set, which is a mistake.
According to Documentation/vm/ksm.txt, "KSM only merges anonymous
(private) pages, never pagecache (file) pages". MAP_PRIVATE can
still be set on file cache pages so in order to not redundantly
set MADV_MERGEABLE on pages that are not fitted for it, both
MAP_PRIVATE and MAP_ANONYMOUS should be set.

Along with this fix, add an extra check that the mapped page is
not a stack page before setting MADV_MERGEABLE for it. Stack pages
change too quickly and always end up in KSM 'page_volatile' list.

Change-Id: If4954142852f17cc61f02985ea1cb625a7f3dec6