Merge "libgui: export EGL headers." into oc-dr1-dev
am: be2e363c9e

Change-Id: Id87eb01463f03ba0bcbe4948a93f3add60fc0b7e

Merge "libgui: export EGL headers." into oc-dr1-dev

Merge "Revert "libui: Load a.h.graphics.mapper passthrough service on library load"" into oc-dr1-dev
am: 274f72eb9b

Change-Id: Ie94b3b6d08c4ca0a13902fd4ec71e4bc7808831f

Merge "Revert "libui: Load a.h.graphics.mapper passthrough service on library load"" into oc-dr1-dev

Revert "libui: Load a.h.graphics.mapper passthrough service on library load"

This reverts commit 75ca86f072f82d38b21e0a13234ecbc2c24cb6ab.

avc: denied { read } name="hw" scontext=u:r:cameraserver:s0
tcontext=u:object_r:system_file:s0 tclass=dir
avc: denied { read } name="hw" scontext=u:r:folio_daemon:s0
tcontext=u:object_r:system_file:s0 tclass=dir
avc: denied { read } name="hw" scontext=u:r:hal_imsrtp:s0
tcontext=u:object_r:system_file:s0 tclass=dir
avc: denied { read } name="hw" scontext=u:r:idmap:s0
tcontext=u:object_r:system_file:s0 tclass=dir
avc: denied { read } name="hw" scontext=u:r:mediaextractor:s0
tcontext=u:object_r:system_file:s0 tclass=dir
avc: denied { read } name="hw" scontext=u:r:mediametrics:s0
tcontext=u:object_r:system_file:s0 tclass=dir
avc: denied { read } name="hw" scontext=u:r:webview_zygote:s0
tcontext=u:object_r:system_file:s0 tclass=dir

Bug: 62353585
Bug: 62912747
Bug: 35197529
Test: build and boot device. Launch camera, take pictures.
Change-Id: Ic6884f7235edb9191034a285414eb93aa44ac23c

Merge "Make VR HWC DPI a configurable property" into oc-dr1-dev
am: 281a0c26c0

Change-Id: I3ec8cc3bea80ab946c7576f9de2c470ace2fb72f

Merge "Make VR HWC DPI a configurable property" into oc-dr1-dev

Merge "IWYU RegionHelper += <limits>" am: bbf3096e22 am: f08008db9f
am: 836513bfbf

Change-Id: I164150132bd6518e52894823b10d108e86eed81b

Merge "IWYU RegionHelper += <limits>" am: bbf3096e22
am: f08008db9f

Change-Id: Ida5d67d96a8c3eb87f25202872b2c30ed5b6c25e

Merge "IWYU RegionHelper += <limits>"
am: bbf3096e22

Change-Id: I9dd08e98eee2924554b623f755869fac35a80605

Merge "IWYU RegionHelper += <limits>"

Disable setting saved_cmdline_size am: ba54d5baf4
am: acc79d7cfe

Change-Id: I8be566b029a6ec93334e7a4dd37eec12846b1f60

Disable setting saved_cmdline_size am: ba54d5baf4
am: 64dac11cc0

Change-Id: I293be9ed5486024c479a0ef133086f825e259048

Disable setting saved_cmdline_size
am: ba54d5baf4

Change-Id: Icd17c5257c03fd9e33114c5760ece6c21c74d025

Disable setting saved_cmdline_size
am: ba54d5baf4

Change-Id: I447ad42ad4c245013e488701a61d8ce2348a368d

IWYU RegionHelper += <limits>

Test: compiles when transitive dependency is removed
Bug: 35363681
Change-Id: Ia405e41102e6ded07720e424167df7e4d080581e

Disable setting saved_cmdline_size

Minimal revert of the addition of increasing
the saved_cmdline_size. This change
exposed kernel bugs that prevented
atrace from working.

Bug: 62375794
Test: manual
Change-Id: Ie946b9246ce9ba22af59dbf0b9fe8a59a5fa1f3f

EGL: add color space validation
am: 15f79786e1

Change-Id: I05600bda5553b6e5f49a055f30cfef60db4fa9e8

libgui: export EGL headers.

Many libgui headers use these headers so not exporting
these breaks BOARD_VNDK_VERSION=current builds.

Test: headers resolved with BOARD_VNDK_VERSION=current
Change-Id: I085faeda4f8ad8a07d91e81222b8031a0dd1858a

EGL: add color space validation

We want Android to support multiple color spaces as
that can be implemented entirely in the eglapi.cpp wrapper.
These additional dataspaces do not require special behavior
by the driver and may not recognize EGL_GL_COLORSPACE_DISPLAY_P3_EXT,
EGL_GL_COLORSPACE_DISPLAY_P3_LINEAR_EXT, and related attributes.
Therefor, need to filter out those attributes before calling
the driver.
In the future there may be new extensions that do require
driver support. When that happens we'll need to update
stripColorSpaceAttribute to treat that correctly.

Test: Android CTS
      adb -d shell am start \
      -n com.drawelements.deqp/android.app.NativeActivity \
      -e cmdLine '"deqp --deqp-case=dEQP-EGL.functional.wide_color.* \
      --deqp-log-filename=/sdcard/dEQP-Log.qpa"'
Bug: 62424735
Change-Id: I0f0867feb73055100636d326213183ef1f052b7c
(cherry picked from commit eeaa52bc65b5cfaca1f0de44635b57919179df29)

Merge "surfaceflinger: discard stale commands" into oc-dev am: f8ca9a0875
am: 3addef29f7

Change-Id: Ica0906d38bbf2df75f10a6108600c3c1646ab80d

Merge "LibGUI remove DEBUG" into oc-dev am: be597daebd
am: 45bf98c36e

Change-Id: Ic223a6dfb10324fbdd2b2609072a0088eff4fbd9

Disable skip Validate on client composition am: 269c236109
am: 0ff681ca77

Change-Id: I7331077ef129e55aed38f10f19732a22eb842bd6

Merge "surfaceflinger: discard stale commands" into oc-dev am: f8ca9a0875
am: 49cdc635a2

Change-Id: I84da842894b25f11b665bd3b4adf597b3aee94ef

Merge "LibGUI remove DEBUG" into oc-dev am: be597daebd
am: d9517fe958

Change-Id: Id677c4ba2c28922c4a3f64fdc2b114d3e3031dbd

Disable skip Validate on client composition am: 269c236109
am: 89283e56d6

Change-Id: Id8e05d219ef73616cdef7152008b487050f25373

Merge "surfaceflinger: discard stale commands" into oc-dev
am: f8ca9a0875

Change-Id: I6b7ce1798d2fe753c577ff3baa954283c372b80c

Merge "LibGUI remove DEBUG" into oc-dev
am: be597daebd

Change-Id: Iece9f1428655c8674cf62a92ecb0a088ad74b629

Disable skip Validate on client composition
am: 269c236109

Change-Id: I4c0a4a90ff0ca2e39075ad643d1892597301d3fd

Merge "surfaceflinger: discard stale commands" into oc-dev
am: f8ca9a0875

Change-Id: Ief69f6fcf0c738612b14b94d65b3ae1555973d6c

Merge "LibGUI remove DEBUG" into oc-dev
am: be597daebd

Change-Id: I65756ce89b4a10d2df403f9f9a4ce2cacdaafda0

Disable skip Validate on client composition
am: 269c236109

Change-Id: Ib0d2f8a9e59cf4618ed029fe35391f566d11bcfb

Merge "surfaceflinger: discard stale commands" into oc-dev

Merge "LibGUI remove DEBUG" into oc-dev

Disable skip Validate on client composition

SurfaceFlinger should not skip validate when GLComposition is
needed. This can happen when SF decides to force GLComposition
(like when a screen rotation occurs).

Bug: 62906801
Test: Manual
Change-Id: I248146eade395a124cb149773287abd88ae36511

surfaceflinger: discard stale commands

When we skip validate, we may have stale commands on the command
buffer.  They should be discarded.

Bug: 62906801
Test: manual
Change-Id: Ie7921ba75ab2252b834cda6924e1fcfdf236f759

LibGUI remove DEBUG

DEBUG_ONLY_CODE flag enables expensive checks such as
BufferQueueCore::validateConsistencyLocked() which when running
represents 15.1% of SurfaceFlinger total runtime. Enabling these
check for "eng" build only make "userdebug" build closer to "user".

Test: Manual
Bug: 62905241
Change-Id: I77065ecf4e62261c3f02b1cad9966d5c4845c639

Merge "Skip Validate am: 249c0ae80a am: 9837ce8210" into oc-dr1-dev-plus-aosp

Skip Validate am: 249c0ae80a
am: 9837ce8210

Change-Id: If81e98cd140426c5cc33595205fd9d4c26bef044

Skip Validate am: 249c0ae80a
am: 8976c6ae37

Change-Id: I400dfd8d484779e5523353b002164f4f5ea41e03

Skip Validate
am: 249c0ae80a

Change-Id: I336fce473d21e5fdfb80ad8eb48f1df5a24497d8

Skip Validate
am: 249c0ae80a

Change-Id: I55a564d600a422915f43111f4e02a10857512fcf

Skip Validate

b/37474580
b/62806392

Test: marlin, ryu

Change-Id: I09e0c52cb7c914dcd883dc771d97a365c89f7037

Merge "Update dumpstate binary file name" into oc-dr1-dev
am: 8d897e6003

Change-Id: Idfd13cc794eb170c16d203f7f8b78077ac37db80

Merge "Update dumpstate binary file name" into oc-dr1-dev

Merge changes from topic 'vendor_available_libgui' into oc-dr1-dev
am: 8c72947af1

Change-Id: Ie7315f7aea25663576402d04a58bbeaa9662e3c4

Merge changes from topic 'vendor_available_libgui' into oc-dr1-dev

* changes:
  Make libgui available to vendors
  Mark EGL/GLES libs as vendor_available
  Change export_include_dirs into export_header_lib_headers

Merge "Check key before edit value in countFlushCompleteEventsLocked" into oc-dr1-dev
am: 8669acc97d

Change-Id: I5fa61c3ffa5f41a5093dba86ad255687d29466b4

Merge "Check key before edit value in countFlushCompleteEventsLocked" into oc-dr1-dev

Update dumpstate binary file name

Keep modem_log_all.tar for a while for vendors

Bug: 62865279
Test: Verify file name in bugreport
Change-Id: Ia64f6c8e978877040eecfbc6066f0d741d91ddf3

Merge "Add DVR method to get native display metrics" into oc-dr1-dev
am: f412b2b9fc

Change-Id: I4cfecf4ccc8bd5a8c3b9c3c11d885ac0233b71d3

Merge "Add DVR method to get native display metrics" into oc-dr1-dev

Merge "Avoid SELinux violation at vndservicemanager" into oc-dr1-dev
am: 5e6abb3f0a

Change-Id: Id757814f2d04cdbbe66f5dcc30fcf34f14338aa2

Merge "Avoid SELinux violation at vndservicemanager" into oc-dr1-dev

Make libgui available to vendors

libgui is now available to vendors, especially for BufferQueue in
between the OMX and Camera HALs for encoding the captured video stream.

In doing so, its headers were moved from frameworks/native/include to
its local directory frameworks/native/libs/gui/include. This is required
because global include path is not supported when building for vendor.
However, in order not to affect platform modules that are still relying
on the global include path, symlinks are provided.

frameworks/native/include/gui -> frameworks/native/libs/include/gui
frameworks/native/include/private/gui ->
frameworks/native/libs/include/private/gui

Bug: 37731063
Test: BOARD_VNDK_VERSION=current m -j libgui.vendor
Change-Id: I0bab8d2e31959085ad58b1ce4c610334ee3da033

Mark EGL/GLES libs as vendor_available

EGL/GLES libs are marked as vendor_available, otherwise vendors are not
able to use OpenGL. The libs were not simply declared as LL-NDK because
doing so causes the dual loading of libui.so in vendor processes;
platform version of libui.so is loaded since it is depended by libEGL.so
whereas there also is vendor version of libui.so in /system/lib/vndk.

When libEGL.so is built for vendors, 1) libgraphicsenv is removed from
its dependency list as the lib must not be available to vendors and 2)
eglGetNativeClientBufferANDROID is unusable since the function relies on
AHardwareBuffer_to_ANativeWindowBuffer which isn't available to vendor.

Bug: 37731063
Test: BOARD_VNDK_VERSION=current m -j libEGL.vendor
Test: BOARD_VNDK_VERSION=current m -j libGLESv2.vendor

Change-Id: I83a128b14a5d852bb0f2cd821a8e3f82c66a313f

Change export_include_dirs into export_header_lib_headers

Remove the duplicated information that headers for libbinder is in the
'include' dir.

Test: m -j
Change-Id: I5527c2709ff9d3a996c619f973345e61407cd3f0

dumpstate: Add the 2nd fd for binary file
am: 9fbfad0b6d

Change-Id: I6c8111188e7fdb8b4b6282aa3511a4d7187cd69e

Merge "Move include/binder to libs/binder/include" am: d46d9f1243 am: b212140bbf  -s ours
am: 7f683a8b8f  -s ours

Change-Id: I0a02efc1cac647857ba32235fac2c160d7dcb8db

Merge "Move include/binder to libs/binder/include" am: d46d9f1243
am: b212140bbf  -s ours

Change-Id: If344d9c14b3cf767956d55e0dd5d13b5a700c7a4

Merge "Move include/binder to libs/binder/include"
am: d46d9f1243

Change-Id: I41744e18ca3731b3027cb459b97945270a4d94ef

Merge "Move include/binder to libs/binder/include"

dumpstate: Add the 2nd fd for binary file

Bug: 33820081
Test: Verify binary file is added in bugreport
Change-Id: Ibe1c219ca4bf288adcb76996c6178426a4d04333

Avoid SELinux violation at vndservicemanager

Avoid following SELinux violation.

avc: denied { read } for comm="vndservicemanag"
name="nonplat_service_contexts" dev="rootfs" ino=17045
scontext=u:r:vndservicemanager:s0
tcontext=u:object_r:service_contexts_file:s0 tclass=file permissive=0
ppid=1 pcomm="init" pgid=1 pgcomm="init"

This violation caused by vndservicemanager reading service_contexts at
svcmgr_handler(). In main() loading as well, processing is divided by
the VENDORSERVICEMANAGER flag. Therefore, even in svcmgr_handler(),
processing is divided by flags like main().

Bug: 62562415
Test: mma
Change-Id: I06b0308a80fc6ea1ca57cd10d9555dd269b8e12d

Make VR HWC DPI a configurable property

Bug: 62833924
Test: temporary logging
Change-Id: I4e38ab9dd9ef6eb42813b3809dc38d1f103d2bf3

Merge "Merge "surfaceflinger: fix a potential child layer leak" into oc-dev am: 1b02b6329e am: 989ed5c4c1" into oc-dr1-dev-plus-aosp

Merge "surfaceflinger: fix a potential child layer leak" into oc-dev am: 1b02b6329e
am: 989ed5c4c1

Change-Id: I1255ff4ef6234b61996e6f7c8dfb86c890e4268e

Merge "surfaceflinger: fix a potential child layer leak" into oc-dev am: 1b02b6329e
am: 4d5cc90559

Change-Id: Ib956b914044fee1941ef966bbf96c151bbbcf6a4

Merge "surfaceflinger: fix a potential child layer leak" into oc-dev
am: 1b02b6329e

Change-Id: I3548aaa0a071f5b1009c5b1070405a09d7841095

Merge "surfaceflinger: fix a potential child layer leak" into oc-dev
am: 1b02b6329e

Change-Id: Ie2bd5a6d1d37bb5b7c7b679b8c472389ff619188

Merge "surfaceflinger: fix a potential child layer leak" into oc-dev

Check key before edit value in countFlushCompleteEventsLocked

Fix an unchecked editValueFor that potentially causes log fatal
when sensor is removed from connection shortly before arrival of
flush complete message.

Bug: 62806749
Test: compiles, flush() still working.
Change-Id: I3684251b7bb746b01d0d8849024217c80ad52f9d

Add DVR method to get native display metrics

This seems like the kind of infomration we should be able to communicate
to vrcore.

Bug: 62494565
Test: Builds and loads, still need to test with vrcore
Change-Id: I2342fabcd2bcb8a34c0b783391b60defc381f3c6
(cherry picked from commit 6c5a92f142b5fa1907ff91a7dce4d09ddcb554c6)

Merge "Fix minor version bump reflected in lshal --init-vintf" am: ac6575097d am: 99065302ed
am: cf64ff00ea  -s ours

Change-Id: I9811de0500a5d5172744c962549524416c3183e3

Merge "Fix minor version bump reflected in lshal --init-vintf" am: ac6575097d
am: 99065302ed

Change-Id: Ic2a1f895312dee90e9e0eaf691edb48df4897c4e

Merge "Fix minor version bump reflected in lshal --init-vintf"
am: ac6575097d

Change-Id: I05dafdba2c7f302dc6200bd2d274e30a33f77de7

Merge "Fix minor version bump reflected in lshal --init-vintf"

Fix minor version bump reflected in lshal --init-vintf
am: e86f70b474

Change-Id: Id573ba5c3b436249a5f407e575a01a61e7985e62

Merge "Fix incorrect bitness for lshal --init-vintf" am: e162a97703 am: 126eee4d14
am: c77bc022f7  -s ours

Change-Id: I69a55e1df55604c5ec36a93fbd070fe333e01cc1

Merge "Fix incorrect bitness for lshal --init-vintf" am: e162a97703
am: 126eee4d14

Change-Id: I435466a45aa5217f99c73007d5f07c1754216995

Merge "Fix incorrect bitness for lshal --init-vintf"
am: e162a97703

Change-Id: Ib59a256b4491b2bf93e8eca1fa85d9a193719446

Merge "Fix incorrect bitness for lshal --init-vintf"

Fix minor version bump reflected in lshal --init-vintf

Test: lshal --init-vintf has no warnings, and power@1.1 / vibrator@1.1
      is shown in output.

Bug: 62675393
Change-Id: I1ffd8b6dc7fcadcf7228cf2b8838e918bb3afe6b
Merged-In: I1ffd8b6dc7fcadcf7228cf2b8838e918bb3afe6b

Fix minor version bump reflected in lshal --init-vintf

Test: lshal --init-vintf has no warnings, and power@1.1 / vibrator@1.1
      is shown in output.

Bug: 62675393
Change-Id: I1ffd8b6dc7fcadcf7228cf2b8838e918bb3afe6b
Merged-In: I1ffd8b6dc7fcadcf7228cf2b8838e918bb3afe6b

Merge "Fix incorrect bitness for lshal --init-vintf" into oc-dr1-dev
am: 5cf813eef0

Change-Id: I1267c96addf4c501fd695faddf5e4d59bd51f8a0

Merge "Fix incorrect bitness for lshal --init-vintf" into oc-dr1-dev

Fix incorrect bitness for lshal --init-vintf

Test: lshal --init-vintf, graphics.mapper has 32+64.

Change-Id: Ic586c8333064cc5636679ce8ad4e1231307191f4
Merged-In: Ic586c8333064cc5636679ce8ad4e1231307191f4
Fixes: 62675393

Fix incorrect bitness for lshal --init-vintf

Test: lshal --init-vintf, graphics.mapper has 32+64.

Change-Id: Ic586c8333064cc5636679ce8ad4e1231307191f4
Merged-In: Ic586c8333064cc5636679ce8ad4e1231307191f4
Fixes: 62675393

Merge "Merge changes from topic 'layer-wp-race' into oc-dev am: a17b14eb92 am: c35a5e29f3" into oc-dr1-dev-plus-aosp

Merge changes from topic 'layer-wp-race' into oc-dev am: a17b14eb92
am: c35a5e29f3

Change-Id: I590c8667049be99f407972953c41ff88e555bb2a

Merge changes from topic 'layer-wp-race' into oc-dev am: a17b14eb92
am: f39c08c906

Change-Id: I579504e93cc6c21dd13e5b8468d8b87037133a0c

Merge changes from topic 'layer-wp-race' into oc-dev
am: a17b14eb92

Change-Id: I1af51f57255024d36e9cae5c221361f36cb28da4

Merge changes from topic 'layer-wp-race' into oc-dev
am: a17b14eb92

Change-Id: I2fb898abae8b4602aa7257b45293f582f0641f24

Merge changes from topic 'layer-wp-race' into oc-dev

* changes:
  surfaceflinger: Layer::getParent requires state lock held
  surfaceflinger: distinguish mCurrentParent/mDrawingParent
  surfaceflinger: protect Client::mParentLayer with a lock

Merge "Add method to explicitly create Surface. am: 4e623e259c am: 84051d9c97" into oc-dr1-dev-plus-aosp

Add method to explicitly create Surface. am: 4e623e259c
am: 84051d9c97

Change-Id: I4368e31fdb9b7ec0912bddbe75399c7d778383b8

Add method to explicitly create Surface. am: 4e623e259c
am: 05e6335833

Change-Id: Id470d62303b9055522d1cb6ad456bcbe4d320cfd

surfaceflinger: fix a potential child layer leak

We should not remove a child layer from its already removed parent.
Call p->removeChild only after we've checked that the ancestor is
alive.

Apply e6b63e1ae12692327f7e46d5f10d6ade5a7bf192 and this fix to
SurfaceFlinger_hwc1.cpp as well.

Bug: 37121786
Test: manual stress test
Change-Id: I7b811450a998acc4ad9690bd4eda058ce6588e14

surfaceflinger: Layer::getParent requires state lock held

We rely on mStateLock to synchronize accesses to
Layer::mCurrentParent.

Bug: 38505866
Test: manual stress test
Change-Id: I5f8ec358ed7e35df28f8c6aec31ae6ee51cb5b93

Add method to explicitly create Surface.
am: 4e623e259c

Change-Id: I22ff0cd39e80478a74e4589084d5ccabfbe54443