DO NOT MERGE. Revert "Dumpstate should hold a wakelock to save bug report time."
am: bb6b604623

Change-Id: I08b640b8c1ffae80205be5dcf179403c335e43ba

DO NOT MERGE. Revert "Dumpstate should hold a wakelock to save bug report time."

This reverts commit f87959e00732d7d737527f1248a71adea99ae29d.

BUG: 32402587
Fixes: 32365477

Change-Id: Ic4daec37efbaef1906450bf6609d5588d5c9a835

DO NOT MERGE. Added a is_zipping() function.
am: 03974984bf

Change-Id: Ib14224bf42315b3b1633662fac1f88610a824296

DO NOT MERGE. Added a is_zipping() function.

BUG: 32219165
Fixes: 32335112
Change-Id: I2bc630f9c840ccd3a2e0474ed16a766e8a405ad8

services: surfaceflinger: ASAN fix
am: 1d3df546d5

Change-Id: If0447f27ab0a1c3033b0a87fc8426f53c2a82ac5

services: surfaceflinger: ASAN fix

Move layer removal to the main thread, while the display is on.

Bug: 30281222
Change-Id: Id9f956c1e626819734868340e7fa12abf257b702

Merge changes I5d907c33,Ibce48cd2 into cw-f-dev

* changes:
  In SurfaceFlinger, test fence before merging
  libui: Change hasSignaled to return optional<bool>

Sensors: Add WRIST_TILT_GESTURE as wake sensor

Include WRIST_TILT_GESTURE sensor type when determining if
a sensor is a wake sensor or not in getDefaultSensor().

Change-Id: Ib9ae3990e063848163edd34be2ec021ed518e346

In SurfaceFlinger, test fence before merging

If the old fence for a given layer has already expired, we can replace
it with the new fence rather than merging the two.

This saves 300-400us per frame on Android Wear

Change-Id: I5d907c336c7383ae4e2e0f71e42f23749494a359

libui: Change hasSignaled to return optional<bool>

Since the android::Fence::wait() call can possibly return an error,
we need some way of indicating to the caller of hasSignaled that a
fence that has not yet signaled (because it is in an error state) will
never signal. To do this, we return an optional<bool>, where true and
false indicate both that the fence is valid and that it has or hasn't
signaled. If an error is returned from wait(), we return a default
optional value (that is neither true nor false).

Test: m

Change-Id: Ibce48cd2e71ddb8ccf6cabe3284afe0efca8c132

Merge "Add a feature to signal fitness quality heart rate monitors" into cw-f-dev

Add a feature to signal fitness quality heart rate monitors

Adding a feature to signal the presence of a fitness quality heart
rate monitor on the device.

Change-Id: Iaf5f6a2dc1c76e7f4ce40612588c368bcfc0d29d

libui: Add hasSignaled() method to Fence

Adds a hasSignaled() method to android::Fence. This will make it easier
to write client code that only cares whether a fence has signaled, and
allows such code to gain the performance benefits of wait(0) (which is
significantly faster than getSignalTime()) while still being obvious
about what it is trying to do.

Test: m

Change-Id: Ia28ce6cff3860a171254ea20a24f3f1bfbc67eb8

Move atrace init into on fs trigger
am: f920891b71

Change-Id: If6deb43479999924cfce63ca6b3994f9f7276c39

Move atrace init into on fs trigger

sysfs should be ready on ealier stage than boot

Bug: 32025203
Test: take systrace
Change-Id: Id73b6959f3075dc793d93551963193a211060da8

Merge "Use snprintf for SurfaceFlinger fence name" into cw-f-dev

Use snprintf for SurfaceFlinger fence name

snprintf is significantly faster than String8::format
This change saves ~50us per frame on Android Wear

Change-Id: I6ac47c2434662ca561208daae232f02a9f136aaf

Merge "Eliminate unnecessary calls to onBufferReleased" into cw-f-dev

Dumpstate should hold a wakelock to save bug report time. DO NOT MERGE.
am: 2cc37eaaa7

Change-Id: Ic58338aaf3dd67b9256e42fe4d61416b35c367de

Eliminate unnecessary calls to onBufferReleased

This change adds a callback to ProducerListener to indicate whether the
listener needs the 'onBufferReleased' notification. This allows us to
avoid making that binder call unnecessarily, saving ~170us per frame on
Android Wear.

By default the new callback returns true, so behavior for existing
clients should be unchanged. Only the DummyProducerListener returns
false.

Also note that it would be simpler to just pass NULL for the
ProducerListener if not for that fact that we still need it for death
notification.

Bug: b/31122630

Change-Id: I730834218a055d89e89f876dd77da8127eb78000

Dumpstate should hold a wakelock to save bug report time. DO NOT MERGE.

BUG: 31828706
BUG: 30832947

Change-Id: I0a4b1fcce91caa96ccbc4e890d9968e3033487de
(cherry picked from commit f87959e00732d7d737527f1248a71adea99ae29d)

Merge "Add "ip xfrm policy" to dumpstate output" into nyc-mr1-dev
am: a4bb5c7173

Change-Id: Iad51735819ce9e2d57072e657aa99d471b5baedc

Add "ip xfrm policy" to dumpstate output
am: 53ca94ab34

Change-Id: I0b7b5f39acd68f0e176b4c684c108c7d906e5515

Merge "Add "ip xfrm policy" to dumpstate output" into nyc-mr1-dev

EGL: check that display is still valid
am: d6e9946cdd

Change-Id: I6d5f5e6f4790f16c1bd52f96743c7e138973a1b4

EGL: check that display is still valid

Bug: 31522731
Change-Id: I84d82e55aba5b58dfdbcac9e208c36767fbedfd1

Add "ip xfrm policy" to dumpstate output

Bug: 30869013
Change-Id: I1f0e5d820f0153484c38ecb0f9c764fca02d786c

Merge "[sensorservice] Fix inconsistent states in SensorDevice" into cw-f-dev

Add socket dumps via ss to bugreports.
am: d3b809baff

Change-Id: I85daa9d7faf6cb0067955d235e4f12fe807adf5c

[sensorservice] Fix inconsistent states in SensorDevice

This CL avoids inconsistent disabled client list when sensorservice
is in restricted mode and a restricted sensor event connection is
destroyed.

The inconsistency can cause registerLister error in sensor CTS.

Also removed the additional logging message since they are no longer
necessary.

Bug: 28533594

Test: Running "cts-tradefed run singleCommand cts-dev --module
CtsHardwareTestCases --test android.hardware.cts.SensorBatchingTests"
continuously while keep registering and unregistering sensor listener
in a separate app (e.g. enable BreakIt mode in SensorLogger).

Change-Id: I601aea87c88511f5543a80c4921ee4822cb89dff

Add socket dumps via ss to bugreports.

Bug: 23113288
Change-Id: I6304425f968fcb22c75c3f6e64bf7992e34e0889

Merge "Surface: Add force disconnection method." into nyc-mr1-dev
am: 6cc1933af9

Change-Id: Ice30ea86acd7abfa71c0053dc1459361b4bbfb6c

Surface: Add force disconnection method.
am: 97b9c86338

Change-Id: Ia9f9dbdcbd9527d54dacb2cdc90eda443230e8d9

Recursively restorecon when SELinux label changes.
am: 993de79607

Change-Id: Ibf9b6beaa58e7e6c2f006e8c6f064330624eb29f

Merge "Surface: Add force disconnection method." into nyc-mr1-dev

Merge "Dumpstate should hold a wakelock to save bug report time." into cw-f-dev

Recursively restorecon when SELinux label changes.

PackageManager has been pretty aggressive about asking installd to
restorecon over app data when it thinks something might have
changed.  However, in the vast majority of cases these are no-op
requests, and we waste a bunch of time recursively walking all
private data, easily costing 60+ seconds on dogfooder devices.

This change updates the initial "create_app_data" command to kick off
a recursive restorecon if it detects that the top-level SELinux label
on the app private data directory changes.  The "create_app_data"
command is designed to ensure that an app's storage is ready, so
PackageManager always calls it at least once per boot before apps
can run.  (This change means that PackageManager no longer needs to
make separate "restorecon_app_data" calls.)

Test: booted, verified that a label change triggered restorecon
Bug: 30768146
Change-Id: I0c8d4018cf8ff888d0ae07a82adc3d61a6002aad

Merge "Show masked value for sensor with permission" into cw-f-dev

HWC2On1: Fall back to GLES for SolidColor
am: 5df47cb648

Change-Id: I485ae0aa189b5338f6f98dd3c716dd87dac60ab1

HWC2On1: Fall back to GLES for SolidColor

Even though SolidColor layers map cleanly to HWC_BACKGROUND composition
in HWC1, SurfaceFlinger never used HWC_BACKGROUND, so we can't trust
that HWC1 devices implemented it correctly. To preserve backwards
compatibility, this changes the behavior to fall back to client
composition to minimize incompatibilities with existing devices.

Bug: 30479781
Change-Id: I638339062e03f2c057b3e1624e7157587ddee7ec

Dumpstate should hold a wakelock to save bug report time.

b/30832947

Change-Id: I0a4b1fcce91caa96ccbc4e890d9968e3033487de

Surface: Add force disconnection method.

Add a new method forceScopedDisconnect to Surface. This will
be used by the framework to force disconnection at times where
the underlying GraphicBufferProducer may be about to be reused.
This is scoped by PID to avoid conflicting with remote producers.

Bug: 30236166
Change-Id: I857216483c0b550f240b3baea41977cbc58a67ed

Merge "Increase YV12BufferPow2 test threshold" into nyc-mr1-dev
am: 186c4c93a7

Change-Id: Ie0a96823e0f40392fa113a54ea9df49a3d5c34e5

Increase YV12BufferPow2 test threshold
am: 6a2593ac53

Change-Id: I632d9d46486edd1d603ec2289537c8f76553aea9

Merge "Increase YV12BufferPow2 test threshold" into nyc-mr1-dev

Merge "Installd: Add a delete_odex command" into nyc-mr1-dev
am: a721faee57

Change-Id: I37aee2b83f5809f1a65aa2955a56d22390337473

Installd: Add a delete_odex command
am: 3964da0636

Change-Id: If7f8fa9f0bb0684e386df9ad2c61975e855ea8de

Merge "Installd: Add a delete_odex command" into nyc-mr1-dev

Increase YV12BufferPow2 test threshold

The NPOT version already has 3 as the threshold and at least one
platform seems to have diff of 3 in one of the internal pixels for POW2
variant.

Bug: 21306103
Bug: 30920650
Change-Id: I7882a6ff43ffc862d95fea32c8ee8e7f19fb759d
Cherry-pick from master (e3747fd25918c943caef4d9c7158a668c786c55d)

Installd: Add a delete_odex command

Add a command to delete odex files.

Bug: 31347757
Change-Id: I29bca8751bcee8d6981c682fbbc816c73b78ac68

DO NOT MERGE ANYWHERE: BufferQueue consumers: Add discardFreeBuffer method am: 8211047138  -s ours am: 7b265d8ab5  -s ours
am: e373a97bd7  -s ours

Change-Id: I6f1db057f3a65265635cd3883118f65832fc7b85

DO NOT MERGE ANYWHERE: BufferQueue consumers: Add discardFreeBuffer method am: 8211047138  -s ours
am: 7b265d8ab5  -s ours

Change-Id: I555ef520067d4300450ef3b0e91f127d06e55b66

DO NOT MERGE ANYWHERE: BufferQueue consumers: Add discardFreeBuffer method
am: 8211047138  -s ours

Change-Id: If827f77c9c8cb36ad3a8f2eaeb6157bc59258a7a

Merge "Update default DispSync offsets" into nyc-mr1-dev
am: 0c083d43f2

Change-Id: I93de11437b64aa930f8edef39e0d86ccdbb532e4

Update default DispSync offsets
am: 28ffe198bf

Change-Id: Id3d014c7a289890c97ca91980407e060e993bd46

Merge "Update default DispSync offsets" into nyc-mr1-dev

Merge "Fix dangerous unsafe_get() in SensorRecord" into nyc-mr1-dev
am: 93ec87d2da

Change-Id: I5370524796b847cd64867e9c0b229463da36ae5b

Fix dangerous unsafe_get() in SensorRecord
am: eb05947ae2

Change-Id: I0d2de87d4efcb1b7cedf9ccf067813e639659577

Merge "Fix dangerous unsafe_get() in SensorRecord" into nyc-mr1-dev

Unlink app image if copy fails for A/B OTA
am: a029c0726d

Change-Id: I06074c8c2f9ae317e607326b37c216a1ee2fd867

Unlink app image if copy fails for A/B OTA

Usually this happens when the app was recompiled with interpret-only.
In this case, move_ab_path will fail for the app image. If this
occurs, delete the existing app image to not have a stale one wasting
storage space.

Test: Make fake AB ota, delete the a/b generated app image, take the
OTA, ensure there is no stale app image after reboot. Also tested
that other apps with valid app images still had an image.

Bug: 31323617
Change-Id: I699b5eb3dc8f82759c284a1d299865b2c3a890d0

DO NOT MERGE ANYWHERE: BufferQueue consumers: Add discardFreeBuffer method

This method releases all free buffers owned by the buffer queue,
in order to save memory (at the cost of potential future
reallocation of buffers).

Bug: 28695173
Change-Id: I458d10373e639e3144faf673af2ba01aca36e65a

Update default DispSync offsets

Updates the default DispSync offsets to 1ms/1ms to avoid SurfaceFlinger
racing against hardware vsync on untuned video-mode panels, which can
result in a significant number of dropped frames. Also moves and
updates the documentation of the offsets, which used to live in
build/target/board/generic/BoardConfig.mk.

Bug: 30801895
Change-Id: I4cb09d840e2f16fb3d05df4d7abf58d9ba36f83b

make sure anrd trace is being collected.

If bugreport is being taken when system is very busy, there's a chance a small
delay in creating the trace file will cause dumpstate to skip including the
file in the report. This fix makes sure the trace has been created before
proceeding.

Change-Id: I0243b8da8ad285921206e8bba88ea80125eb96a7

Show masked value for sensor with permission

Show masked values in sensor event log in dump even if sensor
requires permission. Information about number of event and timestamp
will aid debugging.

Bug: 31065418

Change-Id: If038d2884a7b13b853183df037a9d2dcacfb5284
(cherry picked from commit 2c238fc281da578d382f64443a7c9d7df89b2f39)

Merge "Boost priority of short-running SF FIFO threads slightly." into nyc-mr1-dev
am: 4df766b00a

Change-Id: Ie290d90935ffa779bf14b22821f8ed848fe975e4

Boost priority of short-running SF FIFO threads slightly.
am: 35520634e2

Change-Id: Ic4a53cc1399dff489414ac1c891a041c2500b8e2

Merge "Boost priority of short-running SF FIFO threads slightly." into nyc-mr1-dev

Merge "SF/HWC2: Add a layer minidump" into nyc-mr1-dev
am: a020f00bc1

Change-Id: I8abb4cefe2d13090446ab13d0af345b46496074e

SF/HWC2: Add a layer minidump
am: e22aec741f

Change-Id: Ib97254ebbf2e914c965b529e7d1e59a01db48214

Merge "SF/HWC2: Add a layer minidump" into nyc-mr1-dev

Added a NETWORK trace tag.
am: 0f97c1d775

Change-Id: Id0cb7f189d32e823cc93486edd3a4fcabd990890

Added a NETWORK trace tag.

BUG: 31281543

Change-Id: Idc5759eec12f2704c9b9cc48db181f2e669ccc32

Boost priority of short-running SF FIFO threads slightly.

The short-running FIFO threads should preempt the SurfaceFlinger main
thread in order to prevent deadlock.

bug 31223682

Change-Id: I29e7b4b8439bb876998a8aeeb2b8d6ee5b6b7d36

Merge "libgui: Add null pointer check in setDisplaySurface" into nyc-mr1-dev
am: 669867835b

Change-Id: Iba59fc196d462f51160f77f85f55c20d45a8d75a

libgui: Add null pointer check in setDisplaySurface
am: eddbef88cf

Change-Id: I3a3ddac84b3f38f6c911ff65642441fd187bbe54

Merge "libgui: Add null pointer check in setDisplaySurface" into nyc-mr1-dev

libgui: Add null pointer check in setDisplaySurface

Bug 31236602

Change-Id: I78d4e28e40db93f7e7e304355b370af0fa790b9d

Merge "Surface: Add parcel/unparceling for missing field." into nyc-mr1-dev
am: 4a8039121f

Change-Id: I9a1b1288848ba6fa59af82bd44cf2c12f6f70f20

Surface: Add parcel/unparceling for missing field.
am: 529a103fde

Change-Id: I48340f1436279282ba55de0886213021db6608f5

Merge "Surface: Add parcel/unparceling for missing field." into nyc-mr1-dev

Surface: Add parcel/unparceling for missing field.

Read/write new isSingleBuffered field in native code as well.
Currently just write 'no' unconditionally and discard on read.

Bug: 31162160
Change-Id: Icfb7a37fb37a41f6437fe08bcfad271474ba6983

Add requestor name to GraphicBuffer alloc metadata am: a1d24b45cc  -s ours
am: 8e0b50b01a

Change-Id: Ia02634e20bbb6ab5be7a27857099c6e1792ef601

Add requestor name to GraphicBuffer alloc metadata
am: a1d24b45cc  -s ours

Change-Id: I2108a173cf5b6df98c1e404ce634b2181ac4a536

Merge "Fail setDisplaySurface if can't enable async" into nyc-mr1-dev
am: 4db50cec13

Change-Id: I24fc902468e57e81d0e2536eeebf478db302cc63

Fail setDisplaySurface if can't enable async
am: 1aad24c0c4

Change-Id: Ie81fb73d61e9c3a6cbb53d7877e8ebbcc9e5aea5

Merge "Fail setDisplaySurface if can't enable async" into nyc-mr1-dev

Add requestor name to GraphicBuffer alloc metadata

Adds a requestor name (usually the BufferQueue consumer's name) to the
metadata that GraphicBufferAllocator stores on allocation so that
`dumpsys SurfaceFlinger` can attempt to attribute buffer usage to the
correct client.

Bug: 30776557
Change-Id: I6e0f346584c871bb3b9d5481f82b697b0475a916

Merge "Add requestor name to GraphicBuffer alloc metadata" into nyc-mr1-dev
am: af671072ca

Change-Id: I6c09c1e7f8e8a4ad2185f808350ab010aca68aa2

Add requestor name to GraphicBuffer alloc metadata
am: d4079aff75

Change-Id: I1848ee0b2fc0686a661072e14f042606fd0c209a

Merge "Add requestor name to GraphicBuffer alloc metadata" into nyc-mr1-dev

Add mangle table in dump_iptables
am: 329b90922b

Change-Id: I91942e4662bb11317d0d66cfa9233f0ae5116308

Merge "Revert "Hold a wakelock during dumpstate."" into cw-f-dev

Revert "Hold a wakelock during dumpstate."

This reverts commit d7803db0e98809e5a17e78885d452479349f339f.

Change-Id: I20066ca30a1b53507099dc62f27c3275781c61ee

Add requestor name to GraphicBuffer alloc metadata

Adds a requestor name (usually the BufferQueue consumer's name) to the
metadata that GraphicBufferAllocator stores on allocation so that
`dumpsys SurfaceFlinger` can attempt to attribute buffer usage to the
correct client.

Bug: 30776557
Change-Id: I6e0f346584c871bb3b9d5481f82b697b0475a916

Add mangle table in dump_iptables

Bug: 30839895
Change-Id: I3f939c74246db96f65e7aa06b60c3b9cf6a02606

Create graphic buffer using binder interfaces
am: 686715335a

Change-Id: I9233639d9d2474fdbb7bcd369b82ae0ed3ee83f3

Create graphic buffer using binder interfaces

Using binder interfaces rather than directly allocating the buffer prevents
SELinux warnings.

Bug: 29402015

Change-Id: I26616dcabf46b5419f0841873faaa6a3ee87ff68

Fix dangerous unsafe_get() in SensorRecord

Change raw pointers to wp<> and constify relevant lines in
SensorRecord.

Bug: 30261110

Change-Id: I7f28c593366af44192991061fa5f71cf925b22a8
(cherry picked from commit ded526e8c3539b5986b46dffc1ff8ab15d7da570)

Merge "Hold a wakelock during dumpstate." into cw-f-dev