Merge "SF: correct default early phase for high fps"

Merge "[SurfaceFlinger] Refine expensive rendering trigger."

[SurfaceFlinger] Refine expensive rendering trigger.

Previously we turned on expensive rendering when color mode is DISPLAY_P3
because wide color gamut implies V0_SCRGB dataspace with FP16 pixel format,
which can't be handled by hardware composer and it's also very expensive in GPU
composition. Now that DISPLAY_P3 color mode doesn't have this implication, we
should move the code to only trigger it if we hit GPU composition in DISPLAY_P3
color mode.

BUG: 114226412
Test: Build, flash and boot.
Test: Verify by modifying WCG to scRGB and checking GPU frequency.
Change-Id: Ia3d3c14574e9ac628497d2f9733e6dd84a2ed01d

Merge "Game Driver Metrics: plumb gpu global stats into statsd"

Game Driver Metrics: plumb gpu global stats into statsd

This change adds all the necessary binder calls to IGpuService.

Bug: 123529932
Test: adb shell cmd stats pull-source 10054
Change-Id: Ibd5b6d366bece45291e80c63f4270c93b56d963f

Merge "Ensure we select the right execution binary for all dexopt commands" am: 3e167da916 am: fefbbd7e13
am: c3608009a9

Change-Id: I30833bd583640650c937cc9d9643fb5b09ad3886

Merge "Ensure we select the right execution binary for all dexopt commands" am: 3e167da916
am: fefbbd7e13

Change-Id: Iee8b97e5bb8d52d7761d3f7748d9a4473d1fa0f3

Merge "Ensure we select the right execution binary for all dexopt commands"
am: 3e167da916

Change-Id: I8b32fe8ad5a98e60ca353029cfa6933e59cee315

Merge "Ensure we select the right execution binary for all dexopt commands"

Merge "Mark Layer as modified when changing zOrderRelativeOf."

Ensure we select the right execution binary for all dexopt commands

Some devices may not have the debug version of the binaries, so even if we
are asked to use them we can't...

Unify the logic which decides what execution binary to use and apply it to
all commands (dex2oat, profman, dexoptanalyzer).

(follow up fixes to 67a14f6584d6108c29b923ce6a77c0ed5cf347dc)

Test: installd tests
Bug: 124009579
Change-Id: Iec439c0ed63e709cdf35b7f718533eafd429b9c2

Merge "Update fence event path for kernel 4.10+" am: f3e024a065 am: 480610413d
am: f33fe4c782

Change-Id: Ie9816acdeabf413378ed2a2bc2a299dfeeff2492

Merge "Update fence event path for kernel 4.10+" am: f3e024a065
am: 480610413d

Change-Id: If72059250746f9c642d9b0174d9cb15b7dcdaf94

Merge "Update fence event path for kernel 4.10+"
am: f3e024a065

Change-Id: I885b096a6b30e9281293d088595f0b6e66005ae6

Merge "Update fence event path for kernel 4.10+"

Mark Layer as modified when changing zOrderRelativeOf.

If the layer is not marked as modified, the changes will not get copied
from current to drawing. This will break drawing state since it will be
missing info about what a layer is currently relativeOf.

Fixes: 125468401
Test: Can no longer repro issue in bug.
Change-Id: Id75a2bc87c179c4bd11141032acc612e08f0db7b

Remove dependency with configstore 1.2

The configstore 1.2 is deleted. So cut the dependency with configstore
1.2.

Bug: 124531214
Test: build & boot
Test: adb shell lshal | grep configstore & check configstore 1.2
Change-Id: Ic6b1875e4cf62d45176b83f69a48382c4d9c05c8

Merge "Revert "SurfaceFlinger: add stack guards around SurfaceFlinger::postComposition""

Merge "SF: Fix for a leaked mock reported in processesVirtualDisplayAdded"

Merge "SurfaceFlinger: fix dispatching DISPLAY_EVENT_CONFIG_CHANGED"

Merge "Fix ctad-maybe-unsupported warning"

Merge "Clean up Renderengine caching semantics"

Revert "SurfaceFlinger: add stack guards around SurfaceFlinger::postComposition"

This reverts commit 8164d48b4cb5f27ebb74e7f03875eb2b3b4b89b0.

Test: boot and open some apps
Bug: 122839222
Change-Id: I5ed78c6380a7a96af3c4435f2de1ca40452a71d3

Merge "Game Driver Metrics: make gpu stats info parcelable"

Update fence event path for kernel 4.10+

The sysfs path for controlling dma fence events changed yet again in
Linux 4.10, see kernel commit f54d1867005c3.

Test: adb shell atrace --list_categories | grep sync
Test: external/chromium-trace/systrace.py sync, check for dma_fence
      events in trace.html's event list
Change-Id: I45905cf199d5de3d5679151f19f8aa693adea5e1

SF: Fix for a leaked mock reported in processesVirtualDisplayAdded

I'm not sure why it only started being reported  recently, but removing
the injected virtual display from the SurfaceFlinger current/drawing
state fixes the leak.

Bug: None
Test: adb shell libsurfaceflinger_unittest
Change-Id: Ibaf1a8878e296707b9fbbd0a647908cc1205cdc2

Clean up Renderengine caching semantics

* Add unbindExternalTextureBuffer, so that callers such as
BufferLayerConsumer can help manage the cache directly, which improves
system utilization.
* Remove the CacheHint, as it's no longer needed.
* Remove the backing image cache in BufferLayerConsumer, as it has not
been used. We still need a shadow array to properly callback into
RenderEngine, but we no longer need to store EGLImages here.

Bug: 123107664
Test: systrace
Test: open and close apps with GL composition forced, check for buffer
leakage through dumpsys
Change-Id: Ie48f99868dd46a4e18b5d659eea520e97a9eb300

Fix ctad-maybe-unsupported warning

The upcoming compiler warns about the following code pattern:

frameworks/native/libs/gui/SurfaceComposerClient.cpp:192:5: error: 'lock_guard' may not intend to support class template argument deduction [-Werror,-Wctad-maybe-unsupported]                                                                                                                                                  std::lock_guard lock(mMutex);

Add explicit deduction guide to fix the warning.

Test: m checkbuild
Change-Id: I4b8b22cdd9b9714d0dfc56c1590a19fcdfbbeb07

Game Driver Metrics: make gpu stats info parcelable

This change also rename the GpuStats*Atom to GpuStats*Info because we'd
like to make GpuStats generic to all the authorized recipients, but atom
is too specific to statsd.

Bug: 126560138
Test: adb shell dumpsys gpu --gpustats
Change-Id: Ie071277ae849a32607979d07aca7682febaf4368

Merge "Move PublicFormat from libandroid_runtime to libui"

SurfaceFlinger: fix dispatching DISPLAY_EVENT_CONFIG_CHANGED

When setting a new desired config, we might be in the middle of a
previous config changed. We need to maintain the information about
whether we need to generate a config changed event and dispatch it when
we are done with the config switch.

Test: manual test to change config from SF
Bug: 122905403
Change-Id: I7efad63b666e83f6459a65a40e6cc880119ca7f6

Merge "SF: Add defensive check on luma sampling"

Merge "Revert "Caching between SF and HWC for BufferStateLayers""

Revert "Caching between SF and HWC for BufferStateLayers"

This reverts commit b28f007e51fb84ff78ce8b053d80efb0efdbce20.

Reason for revert: Causing pre-submit failure b/126704033

Change-Id: I17c99cd410b3fa3825d01fedd21a52451368b5c0

Merge "Expose allowAudioPlaybackCapture to PackageManagerNative"

Merge "SurfaceFlinger: add getAllowedDisplayConfigs"

Expose allowAudioPlaybackCapture to PackageManagerNative

Audio playback has both c++ and java API.
The flag is needed in the audio policy service in c++ to know if the app
can be captured by other apps.

Bug: 119500128
Test: dumpsys media.audio_policy
Change-Id: I949b5e31039b024a34a9646a18586cff5f8807ab
Signed-off-by: Kevin Rocard <krocard@google.com>

Merge "Game Driver: dump driver package info in dumpsys gpu"

Move PublicFormat from libandroid_runtime to libui

PublicFormat is defined in android_view_Surface.
However this and related functions are also used in libmediandk
which is both NDK and LL-NDK. As a LL-NDK it should run without
libandroid_runtime.

By moving PublicFormat definition and its utility functions
into libui, LL-NDK can use it without cost of libandroid_runtime.
And also PublicFormat will be deprecated(b/126594675).

This is a part cutting dependency from libmediandk to
libandroid_runtime.

Note: applied clang-format

Bug: 124268753
Test: m -j
Change-Id: I7215dde607b425f7a0d47a6541b83fbe3a0deaf4

Merge "Caching between SF and HWC for BufferStateLayers"

Merge "Remove id from RegionProto object"

SurfaceFlinger: add getAllowedDisplayConfigs

Add an API to ISurfaceComposer to get allowed display configurations.

Test: manual test that calls getAllowedDisplayConfigs()
Bug: 122905403
Change-Id: Ieeb04f0d002ecfb2dbb0acd22ef6f659806c3950

Game Driver: dump driver package info in dumpsys gpu

Bug: 126560138
Test: adb shell dumpsys gpu
Change-Id: I7215e68f32a83bc8efa631e296033ea7c7983a34

Merge "SF: tidy comments for luma sampling."

Caching between SF and HWC for BufferStateLayers

Bug: 120434937
Test: build, boot,libcompositionengine_test
Change-Id: I4a99faeb7aa88683aac19c7000db87bd75f9ebe9

SF: Add defensive check on luma sampling

The Add/Remove RegionSamplingListener should return BAD_VALUE
if the input was invalid.

Bug: 119639245
Test: ./libgui_test --gtest_filter="RegionSamplingTest.*"
Change-Id: I5c5f7a30131c52dead970b23230058a62a874902

Merge "SurfaceFlinger: fix flakiness in IdleTimerTest"

Merge "[SurfaceFlinger] Add GPU protected content support."

Merge "Game Driver: add GpuStats class to process and dump stats"

Merge changes I7c51f161,I9231f4b4

* changes:
  SF: Remove LayerBE
  SF: Refactor onLayerDisplayed

Merge changes from topic "display_policy"

* changes:
  SurfaceFlinger: add DISPLAY_EVENT_CONFIG_CHANGED
  SurfaceFlinger: add setAllowedDisplayConfigs

Remove id from RegionProto object

The region proto object contains an Id, which is assigned to the memory
address of the Region object.

This information is not necessary on winscope traces, remove.

Test: Flash a device. Collect a SF trace. Open trace in winscope. Check
if Id for region objects is removed from the property list

Change-Id: I4f91745d79d9f1812915836e5e7ac3b416c2052a

Merge "Don't store empty objects on SF traces"

Game Driver: add GpuStats class to process and dump stats

Bug: 123529932
Test: adb shell dumpsys gpu
Change-Id: I2d524b1eecb00be71d344c07e8e18244a44bbcb8

Merge changes from topic "frameworks/opengl/sync-gen"

* changes:
  glgen: sync generator with manual changes
  Sync generator script with manual Java changes

Merge "Use DEX2OATBOOTCLASSPATH in installd." am: 95ea7b751a am: 0fde25e90d
am: 8525a5557e

Change-Id: Ibeec24813af48a5142959fd77ec206486afed204

Merge "Use DEX2OATBOOTCLASSPATH in installd." am: 95ea7b751a
am: 0fde25e90d

Change-Id: I6d71f82f41f4341f9866591f978de333bf8092a5

Merge "gui: add test for region sampling"

Merge "Use DEX2OATBOOTCLASSPATH in installd."
am: 95ea7b751a

Change-Id: Iddddc8bbef7feb2188b991d46ef4c3d28d50011c

[SurfaceFlinger] Add GPU protected content support.

Switch to protected context when a layer with protected contents presents.
Previously we black out layers with protected contents, now that we add support
of protected contents in GPU composition, we can turn it on. In screenshot
cases for Recents, we still black out layers with protected contents.

BUG: 35315015
Test: Build, flash and boot
Test: Watch Youtube movie, verifed by force GPU composition.
Change-Id: I556bdd6f82c06c3d54ce62fab06a8c6f47599dcf

Merge "Use DEX2OATBOOTCLASSPATH in installd."

Merge "Add output fb cache to renderengine"

glgen: sync generator with manual changes

Syncs generator to accomodate manual changes to checked-in sources.

Adds note to EGL15.cpp that file is automatically generated.

Bug: 126352534
Bug: 124064285

Test: gen and diff based on generated command lines.
Change-Id: I31909de4c077679dec0323761fb06f388d128a14

Sync generator script with manual Java changes

Incorporate manual changes to generated Java files back into the
generator.

Bug: 126352534
Bug: 124338141
Test: ./gen and check output has no commands for java files to be run.

Change-Id: Ie5f48d704b1485316718ae19b20016a08e688916

Merge "Query the enable_apex_image flag from native experiment." am: 2cb4a0538f am: 15326fb133
am: 6186bfd2b6

Change-Id: I1a45757cec416206b34b8c63fb68a9d77242c0c7

Merge "Query the enable_apex_image flag from native experiment." am: 2cb4a0538f
am: 15326fb133

Change-Id: I305d96d9d5764a843cb8a070c70c987723b308be

Merge "Query the enable_apex_image flag from native experiment."
am: 2cb4a0538f

Change-Id: I35770107e30df01ef38c64926eec072b2a76a951

Merge "Query the enable_apex_image flag from native experiment."

Use DEX2OATBOOTCLASSPATH in installd.

Bug: 119800099
Bug: 122937705
Test: dexopt_test
Change-Id: Ie31762910a51ddb1211eed6a24456f0a70379d9c

Merge "add display_primary* to sysprop"

Merge "Whitelist the staging directory in installd" am: 99ba118644 am: 3c86bd1dba
am: 67e702ae57

Change-Id: I5fc45c2b1864b52a8b23dea383c0ba9708e0b716

gui: add test for region sampling

Adds a test that checks the correct luma is produced under a few
scenarios with different layerings.

Bug: 119639245
Test: ./libgui_test --gtest_filter="RegionSamplingTest.*"
Change-Id: Ic85a97f475a3414a79d3719bbd0b2b648bbccfb0

SF: correct default early phase for high fps

Test: run GL composition, app animation and check systrace
Change-Id: I00725c09cc5749e9e77d9f2a15140499d01570f3

Merge "Whitelist the staging directory in installd" am: 99ba118644
am: 3c86bd1dba

Change-Id: I3541435ea245841079e34ea5bb7dc704dd3330df

Merge "Whitelist the staging directory in installd"
am: 99ba118644

Change-Id: I5be2f0d4a82e7e352453371a3ca5f8d6e46ab2d3

Merge "Whitelist the staging directory in installd"

[automerger skipped] Merge "DO NOT MERGE - Merge pi-dev@5234907 into stage-aosp-master" into stage-aosp-master am: d65a2e6283 -s ours
am: e93f01cd75 -s ours
am skip reason: subject contains skip directive

Change-Id: I0509d61ba16b2919e7145e3763f7054c41464b87

[automerger skipped] Merge "DO NOT MERGE - Merge pi-dev@5234907 into stage-aosp-master" into stage-aosp-master
am: d65a2e6283 -s ours
am skip reason: subject contains skip directive

Change-Id: I4c6db4219a673da35e0666c1b51051d93ab38da9

Merge "Add enum type for supported HDR metadata"

SurfaceFlinger: fix flakiness in IdleTimerTest

Increase the timeout of waiting for reset callback to avoid flakinessas
a result of poor scheduling.

Test: adb shell /data/nativetest64/libsurfaceflinger_unittest/libsurfaceflinger_unittest --gtest_filter=IdleTimerTest* --gtest_repeat=1000
Change-Id: Ib87aebab4474baaa6583b9481137a268fafed368

[automerger skipped] DO NOT MERGE - Merge pi-dev@5234907 into stage-aosp-master am: 5f4277821a -s ours
am: 3f29cb565f -s ours
am skip reason: subject contains skip directive

Change-Id: Ic02ee51e4fe9dfe34ad4440da057bd258b6397d1

Add enum type for supported HDR metadata

Test: Builds

Bug: 112508112
Bug: 120990898
Change-Id: If6e01bad9275b057d9bdbe963a25b7a26b7198a2

[automerger skipped] DO NOT MERGE - Merge pi-dev@5234907 into stage-aosp-master
am: 5f4277821a -s ours
am skip reason: subject contains skip directive

Change-Id: I3d4582885b96c96308856ff969fce86a2b51edc3

Don't store empty objects on SF traces

Proto3 doesn't store default values for object, however, it stored the
object start-end tags if an empty object was created.

Do not create objects when they are empty

Test: Flash a device. Record SF trace. Open them in the new winscope
version with "display defaults" unchecked. Check if there are no empty
objects

Change-Id: I3f48047e43e9c62c572d5b83e63a0f96527ef146

Merge "Exclude secure layers from most screenshots taken by the system server."

Merge "Report DispSync model updates in the correct place."

SF: tidy comments for luma sampling.

Tidying of const, comment and one internal fn name for
Region Luma sampling.

Test: libgui_test --gtest_filter="RegionSamp*"

Change-Id: I46895c8ed5b2e2045d7ce7444085585b62b5ab6e

Merge "Rename resampling property"

SurfaceFlinger: add DISPLAY_EVENT_CONFIG_CHANGED

Add a new event to DisplayEventReceiver for display
configuration change. This event is sent by SF when display config
is changed.

Test: adb shell /data/nativetest64/libsurfaceflinger_unittest/libsurfaceflinger_unittest
Bug: 122905403
Change-Id: Ibb7e0ce7b83b91259ccb0e9c982f5e378b0ebfaf

SurfaceFlinger: add setAllowedDisplayConfigs

Add an API to ISurfaceComposer to set allowed display configurations.
This API is expected to be called by DisplayManager depends on the
current policy in place. Once setAllowedDisplayConfigs is called,
SF can only set a new display config if it is part of the allowed
configurations list.

Test: call setAllowedDisplayConfigs() from backdoor and
      observe config change.
Bug: 122905403
Change-Id: I1d0a3649bbe7a08efeb72dc270f0b2df330b021c

Merge "Rename IDumpstateListener errorcode" am: d7d075fbf6 am: c943242ff9
am: c6c5cdde65

Change-Id: I898f3d7363946d9fdda92afac5babf43b5ef2dc1

Merge "Log duration reporter to logcat" am: 86d900ff2d am: 46fe8a946d
am: 66cc69fe9c

Change-Id: I3eba5e7434484ac1add403109b69ce754a3bb26a

[automerger skipped] Merge "installd: parameterize user data snapshots with snapshot_id" am: 8a3c8b1a94 am: bfa5131193
am: a20fc9399a -s ours
am skip reason: change_id I4177a44a60cdfdc5429a28c613ce661b24482b8b with SHA1 b68f0d1493 is in history

Change-Id: I7627b17fffb6749937c925498a06af3721cd92b1

Merge "dumpstate: Fix IO Errors during zip writing." am: 01f59ce0f6 am: 48d93b7298
am: 8cc3318b1c

Change-Id: I9965667c7ce7762d567f1a59a0995324138542da

Add output fb cache to renderengine

On some devices this can shave off multiple milliseconds.

Bug: 123107664
Test: manual tests, systrace
Change-Id: If29b1753f899fec03852fb1ddaaa1a245f68424b

Report DispSync model updates in the correct place.

Otherwise, metrics sometimes aren't updated in the right spot.

Also, add the reference time to detailed trace info, as it's important
for event calculation.

Bug: 124383894
Test: systrace
Change-Id: I4ad834fc2076600116831e0f6c38c6ee8f178c11

Merge "Rename IDumpstateListener errorcode" am: d7d075fbf6
am: c943242ff9

Change-Id: I2a8b3bf992243d3d5508248127e690e050f925e3

Merge "Log duration reporter to logcat" am: 86d900ff2d
am: 46fe8a946d

Change-Id: I787245655aecb9dadc8c868fe63d68a5d147d971

Merge "installd: parameterize user data snapshots with snapshot_id" am: 8a3c8b1a94
am: bfa5131193

Change-Id: Ic32854f8ae690a7ca745a0513515b7c499f59c05

Merge "dumpstate: Fix IO Errors during zip writing." am: 01f59ce0f6
am: 48d93b7298

Change-Id: I503c787c5e24ac2082f387ee2fac319f722cfd94