Merge "installd: log failed unlinks." into nyc-dev
am: 45d15b8

* commit '45d15b835738497f812bec55affc9a1cba4ec343':
  installd: log failed unlinks.

Change-Id: Ic03a04b4451be24086614e08877853c9157b6d0a

Merge "installd: log failed unlinks." into nyc-dev

Merge "batteryservice: Expose Charge Counters." into nyc-dev
am: f99c6f4

* commit 'f99c6f4b0cd051558def409f9cd4ade628272588':
  batteryservice: Expose Charge Counters.

Change-Id: If327ab95cdfdd3d5819c4dc6242db94255840c43

Merge "batteryservice: Expose Charge Counters." into nyc-dev

installd: log failed unlinks.

bug: 28054110
Change-Id: Id46aacba293c81bb3bb6a0b9721b9b4fe11ed652

Merge "Caps Lock toggle with Meta + Alt (1/2)" into nyc-dev
am: 3e3a11a

* commit '3e3a11a7805efb61f482914abc2ff8f01d5ec1b0':
  Caps Lock toggle with Meta + Alt (1/2)

Change-Id: I23229acb384dbcbd471a6bc1f8230dc4b2ec43ce

Merge "Caps Lock toggle with Meta + Alt (1/2)" into nyc-dev

batteryservice: Expose Charge Counters.

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

Merge "SF/HWC2: Reorder makeCurrent to avoid deadlock" into nyc-dev
am: 79632ee

* commit '79632ee48a8cddf31a1ec15c0c5217b3d3608622':
  SF/HWC2: Reorder makeCurrent to avoid deadlock

Change-Id: Icbb2f2178e21c213ff879e0a04c4bfda25583d28

Merge "SF/HWC2: Reorder makeCurrent to avoid deadlock" into nyc-dev

Merge changes from topic \'vkjson\' into nyc-dev
am: 16ce3ce

* commit '16ce3cec53f3daf18f851c776e50e146f2dfacef':
  GpuService: Add 'help' and 'vkjson' commands
  Add service "gpu" in the SurfaceFlinger process

Change-Id: I46b4c2975b7e458c91e08920762906b368057084

Merge changes from topic 'vkjson' into nyc-dev

* changes:
  GpuService: Add 'help' and 'vkjson' commands
  Add service "gpu" in the SurfaceFlinger process

SF/HWC2: Reorder makeCurrent to avoid deadlock

Now that we are deferring the release of FramebufferSurface buffers
(so that we can return a non-speculative fence), we end up holding
two acquired buffers during the call into HWComposer::commit (whereas
we previously would have only been holding one, with the other having
been released). This means that if the EGL implementation dequeues a
buffer as part of eglMakeCurrent, neither of the FramebufferSurface
buffers will be free, and the call to dequeueBuffer will block.

This change fixes that issue by reordering eglMakeCurrent to occur
after the deferred release.

Bug: 25684127
Change-Id: I6ec55b3f7b7d0e0f5be6029751cb086feeb52fbe

Merge "SensorService: Remove unused function" into nyc-dev
am: 10a91a5

* commit '10a91a5a5b0c9e992a3ccf368ead9e27be75b15e':
  SensorService: Remove unused function

Change-Id: Ieacd16f693c3d9923015b38d2bffe5a47a5077d5

Merge "SensorService: Remove unused function" into nyc-dev

SensorService: Remove unused function

We remove the unused getSensorRecord function.  This function
was accessing the mActiveSensors field without obtaining a lock.
We're looking to simplify this code, so we just remove this instead
of trying to fix it.

Change-Id: I8c4ed3cc7ed17d90aa9a5d7f33f9ab9238f9403f

Merge "Updating wearable_core_hardware to support device administration." into nyc-dev
am: cf463a9

* commit 'cf463a9b97dee1ad91f7325b519ed84de3a0b61d':
  Updating wearable_core_hardware to support device administration.

Change-Id: Ief3e78ecb1a76ee94cd323bb233e3e4e2aa038a6

Merge "Updating wearable_core_hardware to support device administration." into nyc-dev

Close profile before leaving android::installd::clear_profile.
am: 64b59cc

* commit '64b59cc1c89225813295d1b5d6bf4183615da204':
  Close profile before leaving android::installd::clear_profile.

Change-Id: I52cf3109ad82c5f45f5880050b687e0fcf8607ca

Close profile before leaving android::installd::clear_profile.

The profile file descriptor used to be kept open and to
produce SElinux denials when installd ran dex2oat, as that
file descriptor was passed accross security domains.

Bug: 27943553
Change-Id: Ie025082f97c2736156bd5069d7a22aacde86f039

Merge "[sensorservice] temporarily remove Autolock mSensorsLock" into nyc-dev
am: 90b745d

* commit '90b745df266f3394b487910c18514f5f51b94569':
  [sensorservice] temporarily remove Autolock mSensorsLock

Change-Id: If8a42a50a69955a595ab3c6f5cce5dbad3e67118

Merge "[sensorservice] temporarily remove Autolock mSensorsLock" into nyc-dev

[sensorservice] temporarily remove Autolock mSensorsLock

Bug: b/27992752

Change-Id: I3d139c988d1a3522b75374423dcda74c5bd510a3

Merge "vulkan: rewrite top of loader" into nyc-dev
am: 1285a68

* commit '1285a682bf1bfc4f283d2c8f88d7fcfcf09f3363':
  vulkan: rewrite top of loader

Change-Id: Id3a9085d15d77f204185b2b8363839df28f6fc24

Merge "vulkan: rewrite top of loader" into nyc-dev

Merge "dumpstate: acquire /data/misc/logd/ log contents" into nyc-dev
am: 90e840f

* commit '90e840f00713bf9c1016c4f2584ef70e0a87ca14':
  dumpstate: acquire /data/misc/logd/ log contents

Change-Id: I76a387d6ebfdc3627aba268a3600d3d9df93c972

Merge "dumpstate: acquire /data/misc/logd/ log contents" into nyc-dev

Merge "Split increments to silence a compiler warning." am: 0f73649
am: 25b103a

* commit '25b103a4373708cf62bb346e0514d118fa96cbd7':
  Split increments to silence a compiler warning.

Change-Id: I62e0baefcebb9288e6dde9d4f954629bca6f967f

Merge "Added option to redirect stdout on run_command_always." into nyc-dev
am: 233fbed

* commit '233fbedd9dff8f55cb305f8aa97bf06dfa2d7622':
  Added option to redirect stdout on run_command_always.

Change-Id: I02f5b0302fd3b6ce8da8de10ee604d33dc782dc4

Merge "Split increments to silence a compiler warning."
am: 0f73649

* commit '0f736495ff243d6282693b113fe5c76e8a1a9457':
  Split increments to silence a compiler warning.

Change-Id: I4485ce044a9b8455bf3d32e62690e02cf0a64d53

Merge "Added option to redirect stdout on run_command_always." into nyc-dev

Merge "Split increments to silence a compiler warning."

Added option to redirect stdout on run_command_always.

There are cases where a command should not write to stdout. For example,
when sending the initial broadcast before the header is printed.

This change adds this option to run_command_always, and uses enums to
make it easier to read what's been passed to it.

BUG: 27953643

Change-Id: I9aed0cb0df4112b31d2079241114e206b9d0bb0c

media: update Dolby Vision Profile.
am: 9917903

* commit '991790301f69246c1ebc787cc17aedf5eb146678':
  media: update Dolby Vision Profile.

Change-Id: Iadeb05dba326c48ca5f01fdb5251fedffffc2d19

dumpstate: acquire /data/misc/logd/ log contents

Bug: 27965066
Change-Id: I78c7ef4b45179729f505c3fe5d84c33a22ce41bf

media: update Dolby Vision Profile.

Bug: 25684127
Change-Id: I2f712078fa6c40087041b078b2a546181b152a64

GpuService: Add 'help' and 'vkjson' commands

Bug: 26620936 and 27352427
Change-Id: Id0ad5b7184ddc624eddfb292a0d86546c26fb9ea

Split increments to silence a compiler warning.

Upstream clang complains that this line may be a possible misuse of the
comma operator, since the result from "p++" is being discarded. Rather
than overcomplicate things with a static_cast<void>(), it is easiest to
just move this to a separate statement.

Change-Id: I5d005eb82a02430f691f99a2f1e8b7237320f84a

Add service "gpu" in the SurfaceFlinger process

This service will handle shell commands for querying information about
the graphics hardware and driver. It currently does nothing.

Most shell command services are provided by ActivityManagerService. I
didn't want to put this there, because I'd rather not load graphics
drivers in the system_service process (robustness, security, etc.).
SurfaceFlinger is going to have them anyway, and is the only other
global always-running process that makes sense for this.

Change-Id: I27e451ea62b71aed01c6ececefd76d6707912eed

Merge "Fix a few known issues in SensorService" into nyc-dev
am: d5194bd

* commit 'd5194bd30cd8284ed65eb63e713fee65c31c9d5c':
  Fix a few known issues in SensorService

Change-Id: I3d4f08c13aa3df52afd0aa4031ad4c6a13ecad03

Merge "Fix a few known issues in SensorService" into nyc-dev

Fix a few known issues in SensorService

This CL fixes a few known issues in sensorservice which is related
to newly added dynamic sensor discovery API.

  * check and ensure handle uniqueness for dynamically discovered
    sensor.
  * add mutex for synchronizing r/w of various sensor lists.
  * ensure dynamic sensor list is reported sorted by handle.
  * code format fix

Bug:
b/27911774

Change-Id: Iec6df90ae150321ea9e4309b2ac1200b8dc37f05

Updating wearable_core_hardware to support device administration.

Bug: 27409742
Change-Id: Iaf6d6bbf67834079d4f37d7a84310c75588b3a25
(cherry picked from commit 7feabdebd0cfd92dd2f5bc0c258ebd45269a3d9b)

Merge "Add build id as part of bugreport name." into nyc-dev
am: fbe08bf

* commit 'fbe08bfdfdadfcf05bb83e255fa12fc44202aa73':
  Add build id as part of bugreport name.

Change-Id: I2bc0916fd801da3fa2ee878565acdafb4a8e82bb

Merge "Add build id as part of bugreport name." into nyc-dev

Add build id as part of bugreport name.

Rather than bugreport-DATE, the base bugreport name will be
bugreport-BUILD_ID-DATE, which will make it easier to identify
build associated with the report.

Since this change modifies the bugreport format, its version has been
increased to 'v1-dev4' and it is not enabled by default.

BUG: 27618668
Change-Id: I4492f8234520ee27c2eb0e4b8313fc937c498fea

Dumps systrace to a file.
am: 14e034a

* commit '14e034a02ec34b2c76afb06975bdfc943a9b5607':
  Dumps systrace to a file.

Change-Id: Idb27c190029e7d94315f7fedb64310775741fe75

Dumps systrace to a file.

The atrace command can take a while, so rather than use a pipe it's
better to write its output to a temporary file and add that file to the
zip, timing out if necessary.

BUG: 27821761
Change-Id: I09f438077d001167d92da3009016bcab815ce530

SF: Disable FenceTracker from makefile
am: 69a1a38

* commit '69a1a389321447301a8eaba235190dc1229aefbd':
  SF: Disable FenceTracker from makefile

Change-Id: Iff8b9ce5b551cb5e10d424e256c9462e5de50d76

SF: Disable FenceTracker from makefile

- Add tracing to the FenceTracker.
- Disable it by default with #ifdefs, it can be enabled in the
  makefile.

Bug 27882222

Change-Id: I640e028c4dc97b46e800a7f21cdb2cb6a4b4ff18

Merge "Add an option to write the trace to a file" into nyc-dev
am: 2af4358

* commit '2af4358f78f5376e829a07259a459da91c7b9ba4':
  Add an option to write the trace to a file

Change-Id: Ie5ee06d834025feaa159eb955f4caf7ea86054c3

Merge "Add an option to write the trace to a file" into nyc-dev

Revert "Avoid unlinking app image file"
am: a812bcf

* commit 'a812bcfd06df38a360545ae6655a90a9564c3aa9':
  Revert "Avoid unlinking app image file"

Change-Id: Ied465638979e1f1f89aa395b65fd70a02dd10d83

Revert "Avoid unlinking app image file"

Need unlink or the image file can change while being mmaped.

Bug: 27493510

This reverts commit 310c5e70b2e36df36811b6de7ad5c2ebd82338e5.

Change-Id: Ied09848e0b646f946f20f2ed114cac6b438bcda1

Add an option to write the trace to a file

Change-Id: I865a3f2c9f6a1fefa664b8215d1e5c94d7d0d0d3
Fixes: 27821761

Merge "Destroy the reference profile during uninstalls" into nyc-dev
am: 057facf

* commit '057facfdc6b512b4fe43ee19db50b5156cecff84':
  Destroy the reference profile during uninstalls

Change-Id: I1754a78379f46c499e7d54ef011545bb72df5e62

Merge "Do not unlink the profiles if not needed" into nyc-dev
am: 4820e72

* commit '4820e72dbfccad5974dc3d88f2cb462af35c6659':
  Do not unlink the profiles if not needed

Change-Id: I51ba64a994108143805e4d348ccfedfba5251f9e

Merge "Destroy the reference profile during uninstalls" into nyc-dev

Merge "Do not unlink the profiles if not needed" into nyc-dev

vulkan: rewrite top of loader

The top is responsible for layer discovery and chaining, and the bottom is
like a regular layer that is always enabled and is inserted just before
the driver.  Make the separation clear by rewriting the top and stripping
the layer managment code from loader.cpp.

Change-Id: I64e525e27bd4c297bccd94a1eb9b88e28088e85d

Call _exit() instead of exit() on child failure.
am: baa85bd

* commit 'baa85bd893665a48aee73d7ca7b73cd39b48f998':
  Call _exit() instead of exit() on child failure.

Change-Id: I7f78e1f2550cf58467c1a191b8a0c9668277ad1d

Call _exit() instead of exit() on child failure.

As explained on man _exit:

"The  function  _exit()  terminates  the calling process "immediately".
Any open file descriptors belonging to the process are closed;"

If exit() is called instead, the resulting zip file will be corrupted.

BUG: 27900023
BUG: 27804637
Change-Id: I6591749603753d6016f350c629b7082cf6780ae2

Destroy the reference profile during uninstalls

Bug: 27896463
Change-Id: I6b182e2c3276f6228a2ed70ee2c6f0c49c638760

Do not unlink the profiles if not needed

It's enough to just clear the profiles. The app might still run and we
will see misleading warning or we might miss recording profile
information.

Bug: 27895342
Change-Id: I04961f09e0965a3e4af87acb95a7c74773f7a44a

Merge changes I653b84e7,Ieafdc14a into nyc-dev
am: abddf72

* commit 'abddf727041d2ea0e678442b2d7905afc8b33bd4':
  Apply position updates immediately.
  Move crop outside of geometry state.

Change-Id: I7d4b9d49f6807748592c32432e4ab80da59bad37

Merge changes I653b84e7,Ieafdc14a into nyc-dev

* changes:
  Apply position updates immediately.
  Move crop outside of geometry state.

Merge "Define typical sensor sample rate in native code" into nyc-dev
am: a84b43c

* commit 'a84b43c914cdd0e665d7555f5800d5c06b895716':
  Define typical sensor sample rate in native code

Merge "Rename single buffer mode to shared buffer mode" into nyc-dev
am: 59d12e9

* commit '59d12e90ae09171fc8b50fd8fd02e3247c04840c':
  Rename single buffer mode to shared buffer mode

Merge "Dumping a trace isn\'t an error" into nyc-dev
am: 9767bd8

* commit '9767bd89a6f5aa510c649d03e485c086d4ebd614':
  Dumping a trace isn't an error

Increased estimated max progress of bugreports.
am: faf67e3

* commit 'faf67e3fe2451e75d0369906a579d3e1386a5064':
  Increased estimated max progress of bugreports.

resolve merge conflicts of 1ccaed4 to nyc-dev-plus-aosp

Change-Id: Ib30c59ff496ed76ec46f4c8f7d954d571bcbf4ae

Merge "Define typical sensor sample rate in native code" into nyc-dev

Merge "Switch GLES wrappers over to using Clang (and fix inline assembly)."
am: e9aeab8

* commit 'e9aeab858e950fa33062ebabe43ddeb00c08d2f4':
  Switch GLES wrappers over to using Clang (and fix inline assembly).

Merge "Address const issues in preparation for libcxx rebase."

Apply position updates immediately.

Restores the behavior prior to moving transform
in to geometry state. Now geometry state and scaling
mode only control parameters which would cause
buffer scaling.

Bug: 27729195
Bug: 27687126

Change-Id: I653b84e74407f2533c92f7647e2609fc043ed0a4

Rename single buffer mode to shared buffer mode

Change-Id: Id43d0737d9367981644e498942ebc0077d61038c
(cherry picked from commit 3559fbf93801e2c0d9d8fb246fb9b867a361b464)

Move crop outside of geometry state.

Prior to this CL, if Layer crop is changed while a
surface resize is pending, the crop will not be
applied until a buffer latches at the new size. This
CL makes the crop apply immediately. We can see this
is the desired behavior by looking at the two cases
where a resize is pending.

1. A resize is pending to make the surface smaller.
  In this case we need to be able to immediately update the crop so that
  we can shrink the visible region of the surface at an interactive rate
  with input. The window manager currently uses big surfaces and scaling
  modes to accomplish this.
2. A resize is pending to make the surface larger.
  In this case it doesn't matter. If we expand the crop immediately to
  the new surface size, then we have simply uncropped an area which is
  by definition transparent pixels (as we haven't latched a buffer
  at the new size yet).

This change has conceptual soundness as well. We can see that the
scaling mode will not affect properties that affect scaling (transform,
width/height) but not properties which do not (crop).

Bug: 27729195
Bug: 27687126
Change-Id: Ieafdc14aeecb23085793e3056a746d6f344781df

Merge "Switch GLES wrappers over to using Clang (and fix inline assembly)."

Define typical sensor sample rate in native code

Porting typical sensor sample rate (e.g. SENSOR_DELAY_UI)
defined in Java API to native code.

Change-Id: I8b59216415bc952b4e953ca6e8058beb1dd3c645

Address const issues in preparation for libcxx rebase.

Change-Id: I25a1e2946217049e9205589d6493e06362d2173b

Merge "Rename single buffer mode to shared buffer mode" into nyc-dev

Merge "Dumping a trace isn't an error" into nyc-dev

Dumping a trace isn't an error

Change-Id: I6180c4f72b45b3d64c44a197e7bda3809a69c8fc

Increased estimated max progress of bugreports.

The initial estimate was based on earlier work done before new sections
were added, and it was too slow.

Ideally we should have a per-device estimate, but for N we'll just
increase the common max.

BUG: 26373682
Change-Id: I1a8ce7f09fc6588ac1513a20fa1149aab8c38351

Switch GLES wrappers over to using Clang (and fix inline assembly).

Bug: http://b/18003438

The GLES wrapper implementations make direct calls to their underlying
function without specifying that they are retaining the original
argument register(s). This change makes the retention of these argument
registers more explicit, and then removes the LOCAL_CLANG override,
since Clang no longer optimizes the code into a bad state.

We also switch to "naked" functions, so that we control all stack layout
information. This prevents us from accidentally still corrupting
arguments on x86 (and for other functions with many parameters).

Change-Id: I37f2ef4a697373388950b41f0a292393c9fcbac7

Don\'t report errors when trying to delete a non-existing profile dir
am: b06f98a

* commit 'b06f98aabc5381fd6366526d9b31b5d0345481b6':
  Don't report errors when trying to delete a non-existing profile dir

Don't report errors when trying to delete a non-existing profile dir

Also, put destroy_app_current_profiles behind FLAG_STORAGE_DE flag.

Bug: 27081617
Change-Id: Ib8b9ff292784c659259ceb4339b803e3580da7ed

dumpstate: Add /data/misc/recovery
am: d6ab011

* commit 'd6ab01105bbd80dfa2fc2debc8e31d6422c378ee':
  dumpstate: Add /data/misc/recovery

dumpstate: Add /data/misc/recovery

(cherry picked from commit 23e86e91ff971a00f62954833a17df2dad1a13af)

Bug: 27176738
Change-Id: Ibc2342c6a29e5aebcac4875d7c2eed48fa8af7de

Merge "Remove spurious log" into nyc-dev
am: 7ff357a

* commit '7ff357ab080d0f3330a5642124a352f745d816bb':
  Remove spurious log

Merge "Remove spurious log" into nyc-dev

Remove spurious log

Bug 26696080

Change-Id: I60ce2c1fac653d33a033c1e22f0df87466287fba

Merge "Guard against failed array allocation" into nyc-dev
am: 834fde8

* commit '834fde83f9a924f23e920306d9f7590fb6fd0e0b':
  Guard against failed array allocation

Merge "Guard against failed array allocation" into nyc-dev

Guard against failed array allocation

Also sanity check the fd count of incoming Parcels.  Anything
exceeding our actual rlimit is inherently not legit.

Bug 26696080

Change-Id: Ib983dce8e8977d7e33195a4ee471115cdaf1c56b

Merge "Do not clean profiles unconditonally during app data clean up" into nyc-dev
am: d1a6a24

* commit 'd1a6a2495ff82893728571136f4f080164c8dae5':
  Do not clean profiles unconditonally during app data clean up

Merge "Do not clean profiles unconditonally during app data clean up" into nyc-dev

Merge "SF: Add FenceTracker" into nyc-dev
am: 830f500

* commit '830f5004da53a83e23b4102ba9b72e4cb366cb34':
  SF: Add FenceTracker

Merge "SF: Add FenceTracker" into nyc-dev

Merge "Improved (or warned about lack of) error handling." into nyc-dev
am: a207fca

* commit 'a207fca4eda8c17389ea13ec3762ecb2a49d81be':
  Improved (or warned about lack of) error handling.