Merge "Fix clang static analyzer warnings."

Merge "atrace: support HAL services as well."

atrace: support HAL services as well.

Also poke services that are only registered with
hwbinder, so that they update their props (trace
categories) immediately.

Bug: 31262344
Test: builds, atrace works
Change-Id: I355ddd5f950acc27d598cd972e63f9711426b943

Merge "atrace.rc: move to post-fs trigger"

atrace.rc: move to post-fs trigger

Test: manually
Bug: 32025203
Change-Id: I787984fc8ab7151ecce9e62a25fa03b0ecf263cd

Merge "Fix warnings in framworks/native/opengl/include"

Merge "Revert "Workaround alloc-dealloc-mismatch error on ASan device boot.""

Merge "Revert "Revert "Pass --output-vdex-fd and input-vdex-fd to dex2oat."""

Revert "Workaround alloc-dealloc-mismatch error on ASan device boot."

This reverts commit 22819276b147e34a80e8ca696be5b3fda68764b9.

The underlying kernel bug has been fixed.

Test: SANITIZE_TARGET=address build should continue working

Change-Id: I6f3938047d9f60357e7bd9bccd3ef36d7686c73b

Revert "Revert "Pass --output-vdex-fd and input-vdex-fd to dex2oat.""

Test: device boots, adb shell cmd package compile works.

This reverts commit 5373d3028d53df00c244802f89c8a31a8c1a55a6.

bug:30937355

Change-Id: Iddab9cc8de866a281672fc3c509d43070c3f56de

Merge "Revert "Pass --output-vdex-fd and input-vdex-fd to dex2oat.""

Revert "Pass --output-vdex-fd and input-vdex-fd to dex2oat."

Mishandled create_vdex_filename.

bug:30937355

This reverts commit 196539e9ea6b096826c3489700588c286d8ca9c2.

Change-Id: Idd1a9f9567240899a103547a36dd036219f303cd

Merge "Dumpstate's netstat call should use -W."

Merge "Pass --output-vdex-fd and input-vdex-fd to dex2oat."

Merge "Keep protected content in EGLImage attrs as needed."

Pass --output-vdex-fd and input-vdex-fd to dex2oat.

Adjust to new dex2oat flags.

bug:30937355
Test: device boots

Change-Id: Ieab2b2102a7b9c17b77618538608c47d8ad52e2d

Merge "[NAN-AWARE] Rename NAN to Aware"

[NAN-AWARE] Rename NAN to Aware

Rename only (and any reformatting needed to pass lint) - no
functional changes!

(cherry-pick of commit 49f8c84e1872c47716ebc2c04acdc8e3f6192734)

Bug: 32263750
Test: All unit tests and integration (sl4a) tests pass.
Merged-In: I00700206c90c9e6a1a0a2a79248dd571fe43ec26
Change-Id: I6b4b146a470ccbe081fb2f18d75e261a843978e2

Merge "binder: print the bad cmd error to android log"

Fix warnings in framworks/native/opengl/include

Fix old-style cast warnings by using a macro to do C or C++ style
casts.  Remove a visibility attribute being applied to a typedef.
The warnings were being hidden by the use of -isystem to include
frameworks/native/opengl/include.

Bug: 31752268
Test: m -j native

Change-Id: I7db3f7181e3542fc7194e1f18010dd2df5d4c57e

Keep protected content in EGLImage attrs as needed.

When image_crop isn't supported, we also throw away the
protected_content_ext attrs in GLConsumer. Instead, keep them
around if they're used.

Bug: 32222880
Test: manual (cp from internal)
Change-Id: Ic182810cbaacdfaf912c57667f5176d3e47941a9

Fix clang static analyzer warnings.

frameworks/native/cmds/flatland/GLHelper.cpp:366:9: warning: Potential
leak of memory pointed to by 'src'

Test: Warning no longer appears
Change-Id: I2721d292cf1f387f34fb2e69f8370f1abff3f01f

binder: print the bad cmd error to android log

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

Signed-off-by: liangweikang <liangweikang900911@gmail.com>

Dumpstate's netstat call should use -W.

IPv6 addresses (which are quite normal on a real device connected to a
mobile network) are truncated by netstat by default. (80 columns!)

Supplying -W says "I care about seeing the IP addresses more than I
care about fitting on a VT100".

Bug: noticed in bugreport in http://b/32463933
Test: manual dumpstate run
Change-Id: I6f8b9ee258ceaa2beeca588fc95adb17fe07d0c4

Merge "TextOutput log improvement"

Merge "Enable multiple concurrent binderLibTest instances"

Enable multiple concurrent binderLibTest instances

Each instance tries to register a service with the
same name, so suffix the service name with the
instance's PID to avoid collisions.

Test: Tests run and pass in 2 processes at once
Change-Id: I685b01fbfbad4e9bd3eb6ea06f46ffc157169909
Signed-off-by: Connor O'Brien <connoro@google.com>

Merge "[binder]: Correct the index for getReturnString"

TextOutput log improvement

Add unit tests for log functions
Add support of std::endl
Add back support of char
Add back support of bool
Add back support of String16
Fix a build warning on union initialization

Test: run /data/nativetest(64)/binderTextOutputTest
Bug: 32181382
Change-Id: I4030b142beacc5dcd88a10904710fd6e19f7240c

Merge "dumpstate: Use private interfaces for buffer size properties"

[binder]: Correct the index for getReturnString

The cmd code can not as the index drectly.

Change-Id: Icddad46faef7a77acbd7edf118c081ac10501f47
Signed-off-by: songjinshi <songjinshi@xiaomi.com>

dumpstate: Use private interfaces for buffer size properties

Used to estimate the timeout to be applied to the logcat command.
This transition represents some technical debt and optimization.

Test: collect bugreport and confirm logcat output
Bug: 31750617
Bug: 21555259
Change-Id: I512b969a4a796fce468907494814315e30c5299d

Merge "Add licenses to ndk_headers."

Add licenses to ndk_headers.

Test: make ndk
Bug: None
Change-Id: Ia113ea42b1d62d87dfbb2fc456428c7db50a98f1

Merge "Fix log function for potential overflow"

Fix log function for potential overflow

On LP64 system, unsigned long can be 64bit
Also clean out unused static variables.

Test: flash on ARM64 device
Bug: 32181382
Change-Id: I44b7ea8a6588c475a3979d7bddeb08da7f54c27a

Merge "Convert to Android.bp"

Convert to Android.bp

See build/soong/README.md for more information.

Test: mmma -j frameworks/native/{libs/gui,opengl/libs}
Change-Id: I90b4e299ebd2a38afb6642bc242d4ad5424fb901
Merged-In: I90b4e299ebd2a38afb6642bc242d4ad5424fb901

Merge "Fix surfaceflinger file descriptor leak"

Fix surfaceflinger file descriptor leak

If a producer queues a buffer after a screen resize, the buffer
will not be removed from the buffer queue, resulting in a leak of
file descriptor.

Test: Fix was tested by the bug reporter (ARM).
Bug: 32012717

Change-Id: I3b7b0745d5fbdee97fc8df2db399c1a2fcdeb375

Merge "Remove ASAN specialization"

Remove ASAN specialization

The linker will rewrite library paths when running with asan, there's no
reason to do this manually:

10-11 23:26:33.971  8789  8789 E linker  : linker_asan dlopen translating "/system/lib64/libEGL.so" -> "/data/lib64/libEGL.so"
10-11 23:26:34.061  8790  8790 E linker  : linker_asan dlopen translating "/system/lib/libEGL.so" -> "/data/lib/libEGL.so"

Test: Boot SANITIZE_TARGET=address aosp_flounder-eng
Change-Id: I3eb9935603b267c2aefe09d4f24410cbd673feb6

Merge "Hardcode EGL Cache size to something more reasonable"

Hardcode EGL Cache size to something more reasonable

Most of our devices are setting much larger values here than the
default, so instead of hoping every device sets the larger values, just
hardcode the cache size to the larger values.

This also reduces build-time configuration, so we can share the built
library between more devices.

Test: /data/nativetest*/EGL_test/EGL_test on aosp_flounder-eng
Change-Id: Id9b80cecff837cc35043ad3c3cf1d1c60595c237

Merge "Remove EGL Hibernation"

Remove EGL Hibernation

Test: Booted aosp_flounder-eng
Merged-In: I0068bbd0a274bd0db237c30d958ea75b6d8aaf11
Change-Id: I0068bbd0a274bd0db237c30d958ea75b6d8aaf11

Merge "Move atrace init into on fs trigger"

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 "Rely on the platform -std default."

Rely on the platform -std default.

Bug: http://b/32019064
Test: builds
Change-Id: I88ea6b959b9d855a536e1475ed437105370203d0

Merge "Rely on the platform -std default."

Rely on the platform -std default.

Bug: http://b/32019064
Test: builds
Change-Id: Ie2df259641889b3ce95648bd2be48d061108162c

Merge "Migrate libvulkan to ndk_library."

Migrate libvulkan to ndk_library.

Test: make native
Bug: http://b/30465923
Change-Id: I97199a600b71447fd14181e06be3f9221c2529f3

Merge "Migrate khrplatform.h to ndk_headers."

Migrate khrplatform.h to ndk_headers.

Test: make native
Bug: http://b/30465923
Change-Id: Idaf0eb973b7569f92b1f1575869d13b8c5e706fd

Merge changes I6a2f4df7,I62e108bd,Icd83a5ef

* changes:
  Migrate libGLESv3 to ndk_library.
  Migrate libGLESv2 to ndk_library.
  Migrate libGLESv1_CM to ndk_library.

Merge "Fix anonymous struct and union warnings"

Merge "Migrate libEGL to ndk_library."

Fix anonymous struct and union warnings

gui/BufferItem.h and android/sensor.h uses anymous structs and nested
anonymous unions, which are GNU extensions.  sensor.h uses them as part
of its API, so disable the warnings in libgui, the only module that
tries to use it with -Weverything.  BufferItem.h only uses the unioned
fields inside libgui, remove the union and do the 64-bit to 32-bit
slicing manually so libvulkan doesn't need the warnings disabled.

Bug: 31752268
Test: m -j
Change-Id: I92d59b1202f4d6e5419edaa6d27b6e1c50ac0042

Migrate libGLESv3 to ndk_library.

Test: make checkbuild
Bug: http://b/30465923
Change-Id: I6a2f4df7f409596f487879eb7d43a7774dced894

Migrate libGLESv2 to ndk_library.

Test: make checkbuild
Bug: http://b/30465923
Change-Id: I62e108bdd27b6dc40fdf7ede7d579a286cbf3b9b

Migrate libGLESv1_CM to ndk_library.

Test: make checkbuild
Bug: http://b/30465923
Change-Id: Icd83a5ef5d6afcde9441f49de14a8e645d15da2f

Migrate libEGL to ndk_library.

Test: make checkbuild
Bug: http://b/30465923
Change-Id: I154a95c6fde9c34d770b6fe65bfd71ac51e05574

Merge "Rename dump methods to dumpState"

Rename dump methods to dumpState

The dump methods shadow the IBinder dump method.  The warnings were being
hidden by the use of -isystem to include frameworks/native/include.

Bug: 31752268
Test: m -j
Merged-in: Iafc64da43032d5d9d84b64640e70d93fd7051bcf
Change-Id: Iafc64da43032d5d9d84b64640e70d93fd7051bcf

Merge changes I2ce11db5,Idba8278b

* changes:
  Fix logging implicit sign conversions
  Fix libgui warnings

Merge changes I1c6a271f,I1e41e5f5,Ib2f48895,Ia8af95d2,Ie966e37d, ...

* changes:
  SF: Fix a couple of Layer ref count issues
  Build SF executable with USE_HWC2 when needed
  HWC2: Clear transform for SolidColor layers
  HWC2: Use SolidColor composition for DimLayer
  HWC2: Kill logspam for non-HWC virtual displays
  HWC2: Fix virtual displays
  HWC2: Don't set null handle for client layers
  HWC2: Hook up to BoardConfig variable
  HWC2: Skip validate/present if display is off

Fix logging implicit sign conversions

The warnings were being hidden by the use of -isystem to include
frameworks/native/include.

Bug: 31752268
Test: m -j
Merged-in: I2ce11db524ee9d8f846fdf3ed6fb882d5ef57956
Change-Id: I2ce11db524ee9d8f846fdf3ed6fb882d5ef57956

Fix libgui warnings

The warnings were being hidden by the use of -isystem to include
frameworks/native/include.

Bug: 31752268
Test: m -j
Merged-in: Idba8278bc0a73886ecb7978617854b433c0b5875
Change-Id: Idba8278bc0a73886ecb7978617854b433c0b5875

Merge "Fix struct vs. class mismatch in forward definitions"

Fix struct vs. class mismatch in forward definitions

The warnings were being hidden by the use of -isystem to include
frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I46208cec43599f775f7e2cb22b39d19b0ea762e1

Merge "Fix weak vtable warnings"

Fix weak vtable warnings

Move virtual destructors into cpp file so that the compiler knows which
translation unit to put the vtable into.  Hide the warning for
DeathRecipient, which has no virtual methods to move.  The warnings were
being hidden by the use of -isystem to include
frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I25329d66bfc1f6a5064d05ce7d12ad9b090601f8

Merge "Rewrite multicharacter constant using constexpr"

Rewrite multicharacter constant using constexpr

Multi character constants are implementation defined behavior, use a
constexpr instead.  The warnings were being hidden by the use of
-isystem to include frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I22435d1f66a073c303202da7a1940cb14b0b9fd8

Merge "Fix rule of three warnings"

Fix rule of three warnings

Sensor and BufferItem have destructors, the copy constructor and
assignment operators should not be implicit. The warnings were being
hidden by the use of -isystem to include frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: If375d85cd3d6ded4ee6b27b841bf10f563254682

SF: Fix a couple of Layer ref count issues

This is an attempt at fixing two reference counting issues for Layers.

The first issue is that since we were holding an sp<IBinder> (really a
reference to a LayerCleaner) inside the layer state for deferred
transactions, there was a possibility that it could end up being the
last strong reference to the LayerCleaner such that when it was
destroyed while applying a non-deferred transaction, it would attempt
to grab the SurfaceFlinger main lock to destroy its Layer. Since this
occurred in the main SurfaceFlinger loop, which was already holding
the lock to process transactions, this would cause a deadlock.

To fix this, the sp<IBinder> inside the layer state was changed to a
wp<IBinder>, only being promoted when it actually needs to be accessed
(i.e., when the deferred transaction is created).

The second issue is that we were promoting and holding a strong
reference to a Layer before calling into SurfaceFlinger to destroy it
on the onLayerDestroyed path (triggered when a LayerCleaner is
destroyed). After returning from the attempt to grab the SurfaceFlinger
main lock, it was possible that this strong reference was the last one
keeping the Layer alive, and destroying it at this point could cause
the HWC2 version of the layer to be destroyed at effectively any point,
even between validate/present.

To fix this, the promotion of the weak Layer reference was moved inside
the critical section where the SurfaceFlinger main lock is held.

Test: Cherry-pick from internal branch
Bug: 30503916
Bug: 30281222
Change-Id: I1c6a271f9a7b5d6eea9a9db61d971f262d0cfe84

Build SF executable with USE_HWC2 when needed

The SurfaceFlinger executable wasn't picking up the USE_HWC2 flag when
enabled by the TARGET_USES_HWC2 BoardConfig, causing a mismatch in the
size of the SurfaceFlinger object. This fixes that by also passing
USE_HWC2 when building the executable.

Test: Cherry-pick from internal branch
Bug: 30302693
Change-Id: I1e41e5f548d4b8329590a082d9b69155ba365be9

HWC2: Clear transform for SolidColor layers

Clears the transform for SolidColor layers, since it doesn't make sense
to transform layers without a buffer.

Test: Cherry-pick from internal branch
Bug: 30429522
Change-Id: Ib2f4889531556c7ab00e3b270319c321e9f32c84

HWC2: Use SolidColor composition for DimLayer

Switches SurfaceFlinger to try to use SolidColor HWC device
composition for DimLayers instead of always falling back to client
composition.

Test: Cherry-pick from internal branch
Bug: 30216498
Change-Id: Ia8af95d214ba72b5657f32f7827a1e6278629f17

HWC2: Kill logspam for non-HWC virtual displays

Removes some logspam and fixes some non-errors which were reported as
errors when running a virtual display that is not backed by hardware
composer.

Test: Cherry-pick from internal branch
Bug: 30022738
Change-Id: Ie966e37d95f53bcc050eef064d24bbe26748b7fd

HWC2: Fix virtual displays

Close the output buffer release fence per the contract in
hwcomposer2.h.

Also changes the HWC2 device to hold a weak pointer to the display,
which allows it to properly remove it when it gets torn down.

Test: Cherry-pick from internal branch
Bug: 29945684
Change-Id: Ib1b5d81e42a63d0e65b189f7c6c62e661e0ab6ae

HWC2: Don't set null handle for client layers

Removes the code that explicitly set a null buffer handle for client
layers.

Test: Cherry-pick from internal branch
Bug: 29463310
Change-Id: I570e6041ba72d64971bb6c364e8e7b6f84eeff08

HWC2: Hook up to BoardConfig variable

Uses TARGET_USES_HWC2 to enable the HWC2 code path in SurfaceFlinger

Test: Cherry-pick from internal branch
Bug: 27211042
Change-Id: I7b7e3e3f7a918e74df0be27418196c81b4332564

HWC2: Skip validate/present if display is off

Adds checks to ensure that the display is still on before attempting
to call into prepareFrame/commit (which in turn call into
validate/present).

Test: Cherry-pick from internal branch
Bug: 28596387
Change-Id: Iad87f02c3e44e86db43a15c258ee66b0cde14faa

Merge "Fix implicit sign casts in Parcel.h"

Fix implicit sign casts in Parcel.h

The warnings were being hidden by the use of -isystem to include
frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I20d9a5712c77894f9048ef78264d7dc9f59208ea

Merge "Installd: Add a delete_odex command"

Merge "Fix libinput warnings"

Fix libinput warnings

The warnings were being hidden by the use of -isystem to include
frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I13f03892c75ea3c3139ca676666b0d403871d241

Merge "Fix google-explicit-constructor warnings in  HardwareAPI."

Merge "Fix google-explicit-constructor warnings in gui and ui."

Merge "Fix google-explicit-constructor warnings in  opengl."

Installd: Add a delete_odex command

Add a command to delete odex files.

Bug: 31347757
Change-Id: I29bca8751bcee8d6981c682fbbc816c73b78ac68
(cherry picked from commit 3964da0636624ff9af42d1f316bed0752e37c490)

Merge "Fix order of operations warning in libui"

Fix order of operations warning in libui

The warnings were being hidden by the use of -isystem to include
frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I21f34567927125d3eeeecd8f00271a175f93af21

Merge "Fix struct vs. class mismatch in forward definitions"

Merge "Add missing liblog dependency"