Installd: Fully decouple pre- and post-bootcomplete properties

Do not fall back to dalvik.vm.dex2oat-threads if
dalvik.vm.boot-dex2oat-threads is not set. Also do not ever use
the value of dalvik.vm.image-dex2oat-threads. Instead use the
default behavior of dex2oat in that situation.

Change-Id: I42e60996341414b95b8df2d243d5e07f7f5ff9b6

Merge "Pass dexopt flags as integer"

Pass dexopt flags as integer

Instead of using a series of booleans, create a single flags integer
that contains all of the dexopt options.

Change-Id: I35542aa73ca57b0e765d19b1339b1429849c1ae8

Merge "Installd: Take boot status as dexopt parameter"

Installd: Take boot status as dexopt parameter

Expect the boot status explicitly as a parameter so that we do not
have to rely on dev.bootcomplete, which isn't meaningfully set
when the device needs the decryption screen on boot.

Bug: 23898216
(cherry picked from commit 72ebebed876a62e719b098e43d9d516361bde029)
Change-Id: I9b34298caf70b1e5d40970cc0d04c469016a80a7

Merge "Enable integer overflow sanitization in libinput."

Enable integer overflow sanitization in libinput.

Bug: 24171356
Change-Id: I1412b78171c60b0252560956a672f2d92a7011e6

Merge "Eliminate multiple benign overflow conditions."

Merge "Remove usage of SharedBuffer"

Merge "Enable multiple benign overflow conditions."

Merge "Eliminate benign overflow condition triggered upon loop termination in Input.cpp"

Eliminate multiple benign overflow conditions.

In InputTransport.cpp, there are multiple loops in which loop
termination occurs when the value becomes zero. These termination
conditions are all written value-- > 0, which, since value is
unsigned, result in an unsigned integer overflow when value is 0.
These loops were refactored to eliminate these conditions.

Bug: 24171356
Change-Id: Ie135c4306d1f2cef2778e295242305ed5139221a

Enable multiple benign overflow conditions.

In VelocityTracker.cpp, there are multiple loops in which loop
termination occurs when the value becomes zero. These termination
conditions are all written value-- > 0, which, since value is
unsigned, result in an integer overflow condition when value is 0.
These loop were refactored to eliminate these conditions.

Bug: 24171356
Change-Id: Ie44940cfef8a122ee1aff59c021274ba39a368bf

Eliminate benign overflow condition triggered upon loop termination in Input.cpp

In readFromParcel, a while loop is terminated when sampleCount = 0. The
decrement operation that was here would decrease sampleCount, an unsigned value,
below 0, triggering an unsigned integer overflow. The while loop was refactored
to eliminate this condition.

Bug: 24171356
Change-Id: I7669f54a41d11548b33e322b025431c6f6038952

Merge "constify generated JNINativeMethod fn ptr tables"

constify generated JNINativeMethod fn ptr tables

Change-Id: Icbad34e298f6d8e60ef12e9a91200994249fe2f5

Remove usage of SharedBuffer

- Remove getSharedBuffer() from Region
- Don't use SharedBuffer for memory management in HWCLayerVersion1,
  instead keep shallow copies of the Regions.

Bug 23962051

(cherry picked from commit d814cf2a3e3a2fdb73efa80539fe8af0a93da1db)

Change-Id: I8fa17beed57936648c7b4defc9219dff1d5b337f

Merge "asan: update condition to work with multiple SANITIZE_TARGET values."

asan: update condition to work with multiple SANITIZE_TARGET values.

The goal is to enable SANITIZE_TARGET='address coverage', which
will be used by LLVMFuzzer.

Bug: 22850550
Change-Id: Iaa300aa20798f2e9aa613842b463edf8aeebe3a7

Merge "Installd: Allow different behavior before bootcomplete"

Installd: Allow different behavior before bootcomplete

Check dev.bootcomplete in dex2oat(). Use the information for
two changes.

Only switch to the background when we're post bootcomplete.
This will ensure better utilization after upgrades.

Add a second dex2oat-threads property that is used pre
bootcomplete. A separation of these phases allows using less
cores when the device is up, freeing up resources for other
purposes, e.g., avoid jank. The precedence during boot is the
boot property, or the image property if the former doesn't
exist, or the default property as a fallback.

Bug: 23898216
Bug: 24004256

(cherry picked from commit 21948005f0294a64a42932b21a33ae75295dc3d9)

Change-Id: I5063f3fc4b437cbe88c4e94584e01c1c78eccc4d

Merge "Update libpowermanager to expose enum and export headers."

Update libpowermanager to expose enum and export headers.

Move libpowermanager's transaction ID enum (needed when
implementing BnPowerManager::onTransact()) into
IPowerManager.h and update the library's Android.mk file to
export its headers.

Bug: 22122485
Change-Id: Ie97198c3d2ebe8e880125e19a001ef7162009858

Merge "Remove BnServiceManager."

Remove BnServiceManager.

As far as I can tell (e.g. https://goo.gl/nxpFBZ from 2009),
BnServiceManager is unused. It's only relevant for
libbinder-based C++ daemons implementing the
IServiceManager.aidl interface. service_manager.c rolls its
own binder code instead.

Change-Id: I837a897d8a201b04bf3ff65c87b0d35d10196199

Merge "Benign unsigned integer overflow in Parcel"

Merge "Enable fsanitize integer in ui"

Benign unsigned integer overflow in Parcel

The realloc case in continueWrite did not update the
gParcelGlobalAllocCount value when an allocation occurred.

In addition, there are conditions that could cause the
gParcelGlobalAllocCount value to be decremented below 0, resulting
in a benign unsigned integer overflow that can cause corrupted values
to be returned through system profiling mechanisms.

BUG: 23972600
Change-Id: Ibe07db91a811a04b486760eb78d81c926ba8503d

Merge "Enable fsanitize integer in binder"

Enable fsanitize integer in binder

Enables checks for undefined or suspicious integer behavior. If
suspicious behavior is detected, the process is aborted instead of
potentially resulting in a security bug, such as #20674694 and
#19573085.

Bug: 23836686
Change-Id: I14d6d565afcb8a6435fd5a8e8175e00bbe5d9b6c

Enable fsanitize integer in ui

Enables checks for undefined or suspicious integer behavior. If
suspicious behavior is detected, the process is aborted instead of
potentially resulting in a security bug.

Bug: 23834892
Change-Id: I9111bce2ebb36e045f4da1dbf78f90f04a0aa81c

Merge "move init.trace.rc to frameworks/native"

move init.trace.rc to frameworks/native

init.trace.rc will be renamed to atrace.rc and use the LOCAL_INIT_RC
mechanism to be included on /system appropriately.

Bug 23186545

Change-Id: Ic5833b4bb6e6364e4fef6ab4882ae75f8f3856e9

Merge "Add a tag for tracing system server."

Merge "Whitelist windows module"

Merge "include/android/sensor: add enum for LINEAR_ACCELERATION"

Merge "ignore SIGPIPE signal in native services"

ignore SIGPIPE signal in native services

When run dump method via dumpsys command with invalid pipe fd, native
services can crash with SIGPIPE.
So ignore SIGPIPE signal.

Bug: https://code.google.com/p/android/issues/detail?id=184099
Change-Id: Icb15e0dff34a245c08c1b4c4bed51992ae382da4
Signed-off-by: Young-Ho Cha <ganadist@gmail.com>

include/android/sensor: add enum for LINEAR_ACCELERATION

Change-Id: I6e387f905ba87bde810952daf939ba0f529d18b8

Whitelist windows module

Bug: 23566667
Change-Id: I4fa1243421885a0c1a8febbbcefb0d7e9cdcc99b

Merge "BatteryProperties: Adds batteryCycleCount, batteryCurrent, batteryFullCharge fields in the the battery properties."

BatteryProperties: Adds batteryCycleCount, batteryCurrent,
batteryFullCharge fields in the the battery properties.

Bug: 22981343
Change-Id: I426b4b7d1eb544d2b1d97d87ae1eba0fdf35c503
Signed-off-by: Ruchi Kandoi <kandoiruchi@google.com>

Merge "libagl: Fix buffer read overrun in eglCreatePbufferSurface"

Merge "bundle init.rc contents with its service"

libagl: Fix buffer read overrun in eglCreatePbufferSurface

The code was assuming that EGL_NONE==0, which isn't true.

Bug: 23403170
Change-Id: Ic1eccdef086b4d610bd78dbb6b2ae883c91dc322

Merge "binder: Change from C11 <stdatomic.h> to C++11 <atomic>."

bundle init.rc contents with its service

Bug: 23186545
Change-Id: I2a70a39122b146d5bbc9b55156e769782e011bff

binder: Change from C11 <stdatomic.h> to C++11 <atomic>.

Change-Id: Ib831a9ccfcc7089940d37883d860574359c270e2

Merge "binder: Add a throughput performance test for binder."

binder: Add a throughput performance test for binder.

Change-Id: I04801cd505a439307f2a963f6267db1831b3829f

Merge "Tracing for binder locks/transactions"

Merge "Lose HAVE_ANDROID_OS from frameworks/native."

Lose HAVE_ANDROID_OS from frameworks/native.

Change-Id: I8e6af2a46a9d875192fc7957ada9b5f66d84bad6

Merge "More O_DIRECTORY | O_NOFOLLOW changes"

More O_DIRECTORY | O_NOFOLLOW changes

Fix a few other spots where we're not using O_NOFOLLOW
with O_DIRECTORY.

Add O_CLOEXEC for good measure.

https://code.google.com/p/android/issues/detail?id=181945

Bug: 23044050
Change-Id: Ib535cbba29cdd587443ea9e1e75831f57fd7a93c

Merge "installd _delete_dir_contents: add O_NOFOLLOW"

installd _delete_dir_contents: add O_NOFOLLOW

Add O_NOFOLLOW to make sure we're not following symlinks.

https://code.google.com/p/android/issues/detail?id=181945

Bug: 23044050
Change-Id: I3408309fd401aefb6ebecb2a052bba198eed7633

Add a tag for tracing system server.

This change is needed for
https://android-review.googlesource.com/#/c/157562/

BUG: 21739901
Change-Id: Ic4616065749236fbde9263b8f1898c434913564a

Merge "Add a trace constant for PackageManager"

Add a trace constant for PackageManager

Change-Id: Icd9afc86c88296919e4750d71cb0aef314194bee

Merge "Native/opengl: Remove unused variables from test"

Native/opengl: Remove unused variables from test

For build-system CFLAGS clean-up.

Bug: 18632512

(cherry picked from commit a75f729650b18478e3ff822dbf686406ed7f7169)

Change-Id: Ibe2d59c7e0677f02e4bf5214d34655cd300d5f15

Merge "dumpstate: add logcat -b all -S"

dumpstate: add logcat -b all -S

- For those that are proud to call themselves logspam police
- Every time someone declares that something is too chatty, or that
  they lost their logs because they were declared too chatty and
  their associated logs were aggresively pruned we generally
  ask them to report the logger statistics to pinpoint what software
  product that is the elephant in the room.
- Every time we want to spawn a new 'stop being so spammy' bug spawned
  off a bugreport collected for another purpose, we *wish* we had the
  logger statistics to help add gravitas to the claim that some piece
  of software is the top, or near the top, polluter.

Change-Id: Ifae33cd21d0ae2917a3b4381502d723725b1701c

Merge "Enable atrace to read categories from file."

Merge "dumpstate: remove unnecessary mkdir"

dumpstate: remove unnecessary mkdir

/data/anr is now created in init.rc. This code is unnecessary.

Bug: 22385254
Change-Id: I4ea9c311d52eee7fb6b90235fbd12a4c9364d71f

Merge "Installd: Add a swap override flag"

Installd: Add a swap override flag

Add dalvik.vm.dex2oat-swap system property to override a default
decision.

Bug: 20658562

(cherry picked from commit c968c0175e967e39e72f557b5e014b9575ba4727)

Change-Id: I34368c6e435d1a9ceec20a0bf1c8c6213e527f5e

Merge "With SANITIZE_TARGET, load EGL libs from the instrumented path."

Merge "Binder on Windows? Unlikely."

Binder on Windows? Unlikely.

Change-Id: I36c1bee4ed33b9b0d24109a6dbaefa0548dd5aa8

With SANITIZE_TARGET, load EGL libs from the instrumented path.

Bug: 21785137
Change-Id: Iac56419a9ca776c5704bed44d3f0a1a1edd9d172

Merge "Installd: Fix delete_code_cache() clearing the wrong cache dir"

Installd: Fix delete_code_cache() clearing the wrong cache dir

Fix a typo in delete_code_cache() introduced by commit c03de09, which
causes the function to empty /data/data/<package_name>/cache/ rather
than /data/data/<package_name>/code_cache/.

The following CTS test case should now pass:

 com.android.cts.appsecurity.SplitTests#testClearCodeCache

Change-Id: I2d9452d70ed39b3ea29ad65fb35e788c2fef5100
Signed-off-by: Hao Li <hao.i.li@intel.com>
Signed-off-by: Yu Ning <yu.ning@intel.com>

Merge "sf: Initialize EventThread before creating HWC"

Merge "Fix async_stop"

Fix async_stop

Change-Id: I68a258cf71cfcd999bbd568ab3c96255c11bec41
(cherry picked from commit 2c237ee1d5961b9d11618e0c5a97b3bb7fb694b2)

Enable atrace to read categories from file.

Add -f option to atrace, which enables atrace to use the
categories written in the specified file.

BUG: 21739901
Change-Id: I2cf1f277280434b28c426c68e2864b62aebd20cc

Tracing for binder locks/transactions

Added logic to pull traces from the binder driver.

Change-Id: I9aa00b4650fc3f90d7bd55328dfef11e2514f7b4

sf: Initialize EventThread before creating HWC

Once HWC is created, it could use any of the provided hooks, which
could lead to a crash if the EventThread (handler) isn't initialized
prior to creating HWC.

Change-Id: I5ea35fe9bcb150fb74aae1295b798bd787ad6cee

Merge "dumpstate: modify logcat timeout to compensate for storage speed"

dumpstate: modify logcat timeout to compensate for storage speed

Discover the logcat buffer size, divide by the worst storage
speed in kb/s, and multiply by ten to estimate the timeout in
ms for each named buffer. If that value is less than 20 seconds,
then select 20 seconds. Apply this to each logcat request.

Bug: 21555259
Change-Id: Id36aebff83cb06e94350e4a84fa266b66a8444fc

Merge "sf: Fix incorrect state reporting in dumpsys"

sf: Fix incorrect state reporting in dumpsys

Dumpsys utility accesses layer compositionType variable
with out proper protection. These variables are modified
during hwc_prepare call. Existing HAL lock protection is not
sufficient to address this issue. Failure to do this will
result in incorrect state reporting in dumpsys.

A new displayLock mutex in HWComposer will be used in both
dumpsys and draw calls to ensure correct state is accessed.

Change-Id: I8a57de59525adc0e089b3bed95c067c01e42b666

Merge "sf: Add a NULL check in getDisplayConfigs"

sf: Add a NULL check in getDisplayConfigs

Validate the display binder by adding a NULL check in getDisplayConfigs.
This will prevent a false match if the caller queries the display
configs for an inactive display (whose binder is NULL by default).

Without this change we might end up attempting to index the display
config array, which is unpopulated for inactive displays, and this will
result in a crash. (See getDisplayInfo in SurfaceComposerClient.cpp for
an example of this scenario)

Change-Id: Ib32a7dc8378d3438df0dba1ecd608bbcfc837717

Merge "GLConsumer: Fix crop math"

GLConsumer: Fix crop math

When we have excess pixels that need to be removed from (for example)
the left and right sides, we currently do something like:

  left += excess / 2;
  right -= excess / 2;

If excess is odd, however, this removes 1 too few pixels since the odd
pixel gets rounded down twice. This changes the math to effectively:

  left += excess / 2;
  right -= (excess - excess / 2);

Which removes the correct total number of pixels.

Bug: 19611086
Change-Id: I8d1ad9fe7ba67c149794c148663d12acbccccef0
(cherry picked from commit ec4cb387502d5bd09e49b55605cf679fe3a0207a)

Merge "SF: Swap w/h when capturing rotated screenshots"

SF: Swap w/h when capturing rotated screenshots

Swaps width and height when capturing a screenshot that is rotated by
90 or 270 degrees.

Bug: 8433742
Change-Id: Ibf0b604f541e3cc271e56fe0ad04dc366beb5d79
(cherry picked from commit 3502416204d9dbd905012ee586d8bd145323809f)

Merge "SF: Don't crop surface damage against viewport"

SF: Don't crop surface damage against viewport

Apply the display transform to surface damage, but don't intersect it
with the viewport.

Bug: 21486007
Change-Id: I94e697cea52096a960385d5f3b89609fdcf30a02
(cherry picked from commit 17833a1b0d70cd5cea342cadea9fdbbf16748bc6)

Merge "libgui/SurfaceFlinger: Add getConsumerName"

libgui/SurfaceFlinger: Add getConsumerName

Adds a getConsumerName method to IGraphicBufferProducer and Surface.
Currently, the name is cached inside of IGBP and is update on connect
and dequeueBuffer, which should be good enough for most uses.

Bug: 6667401
Change-Id: I22c7881d778e495cf8276de7bbcd769e52429915
(cherry picked from commit c6f30bdee1f634eb90d68cb76efe935b6535a1e8)

Merge "libgui: Add generation numbers to BufferQueue"

libgui: Add generation numbers to BufferQueue

This change allows producers to set a generation number on a
BufferQueue. This number will be embedded in any new GraphicBuffers
created in that BufferQueue, and attempts to attach buffers which have
a different generation number will fail.

It also plumbs the setGenerationNumber method through Surface, with the
additional effect that any buffers attached to the Surface after
setting a new generation number will automatically be updated with the
new number (as opposed to failing, as would happen on through IGBP).

Bug: 20923096
Change-Id: I32bf726b035f99c3e5834beaf76afb9f01adcbc2
(cherry picked from commit 812ed0644f8f8f71ca403f4e5793f0dbc1fcf9b2)

Merge "Call selinux_status_open() prior to use of selinux_status_updated()."

Call selinux_status_open() prior to use of selinux_status_updated().

The selinux_status_*() interfaces require calling selinux_status_open()
first.  Since this was not being called by servicemanager, the
selinux_status_updated() call was always failing and servicemanager
was not reloading service_contexts upon a policy reload.

Change-Id: I6ac62e114bbca99c5f0999feb918fcde15009274
Signed-off-by: Stephen Smalley <sds@tycho.nsa.gov>

Merge "libgui: Propagate error by clearing sp<GB>"