Merge "Revert "Add a simple SurfaceFlinger stress test.""

Merge "Rename typedef to dvr"

Revert "Add a simple SurfaceFlinger stress test."

This reverts commit 2d906b41c0720ba90fab20fe4ddae2400f9bbbd7.

Reason for revert: Semantic conflict introduced build break...

Change-Id: I3dbd11a8f73ab2636a732693639edcf869be8033

Merge "Revert "Extend hack to new driver version""

Revert "Extend hack to new driver version"

Rolling back along with reverted driver drop

This reverts commit fc00a1d4275c9f6502d56e3e54f399e27e9a314b.

Change-Id: I58e23d7782254824d023ce4973a75ba5d0c1f367

Merge "Add a simple SurfaceFlinger stress test."

Rename typedef to dvr

Bug: b/36040550
Test: None
Change-Id: I489bf73289daa895c531ee6fd47a75537fb34eca

Merge "get rid of the dependency on utils/CallStack.h"

Merge "Revert "get rid of IGraphicBufferAlloc""

Revert "get rid of IGraphicBufferAlloc"

This reverts commit 1da94dfed674e94edc37cdc8ef68530520edf169.

Bug: 36176799
Bug: 36175706
Change-Id: I6dd5fe8df564f0efa4392a800fb316e27a05076e

get rid of the dependency on utils/CallStack.h

this makes it more explicit that we depend on
libbacktrace (and therefore libbase, libunwind).

this also completely get rid of our direct
dependency on libutils

Test: compiled
Bug: treble
Change-Id: Ib7b9d6be4f08cef401afe0d3d347e0250924a94f

Merge "Update vsync client c api"

Merge changes from topic 'test-hwc2' am: 29943c1971 am: 92360b14a0
am: d47b3c13da

Change-Id: I616fe5646001c45933d466ee77824dc435819c5b

Merge changes from topic 'test-hwc2' am: 29943c1971
am: 92360b14a0

Change-Id: Ia517fe617338161966a6ee389d18526e8e2c7bc5

Merge changes from topic 'test-hwc2'
am: 29943c1971

Change-Id: I69fdf5bf2ff8f2c454e2e1a455e4d8d73b330aa3

Merge changes from topic 'test-hwc2'

* changes:
  test-hwc2: dump
  test-hwc2: virtual displays
  test-hwc2: color and hdr requests
  test-hwc2: present display
  test-hwc2: add client target support
  test-hwc2: get and accept display changes
  test-hwc2: validate display
  test-hwc2: set layer visible region
  test-hwc2: set layer buffer
  test-hwc2: set layer surface damage
  test-hwc2: set cursor position
  test-hwc2: set layer color
  test-hwc2: set layer source crop
  test-hwc2: set layer display frame
  test-hwc2: set layer z order
  test-hwc2: set layer transform
  test-hwc2: set layer plane alpha
  test-hwc2: set layer dataspace
  test-hwc2: set layer blend mode
  test-hwc2: set layer composition type
  test-hwc2: get display name
  test-hwc2: enable vsync
  test-hwc2: power mode support
  test-hwc2: get and set active config
  test-hwc2: get display configs and attributes
  test-hwc2: create and destroy layers
  test-hwc2: display type support
  test-hwc2: register callback functions
  test-hwc2: open and close hwc2 device

Update vsync client c api

* Added a required function
* Removed unused functions

Bug: b/36031165
Test: None
Change-Id: I45e141dc627e7cfa541d637d697a553aebb7f480

Fix external manual calculation bug.

Ignore file extensions anywhere under "Android" since those files
belong to specific apps.  Don't depend on GID being set for those
directories.

Bug: 35684969, 35441006, 34945234
Test: cts-tradefed run commandAndExit cts-dev -m CtsAppSecurityHostTestCases -t android.appsecurity.cts.StorageHostTest
Change-Id: Ifeb8d314cc2b803ac0d3595974daf91f51862061

Merge "Finer-grained locking for size operations."

Finer-grained locking for size operations.

Disk space measurements are read-only and don't perform mutations,
so other installd operations shouldn't block them.

If there's an ongoing parallel operation (such as a dexopt) that
could race and skew the results, that's no different than an actively
running app changing it's disk usage during the measurement.

This change also allows measurements to happen in parallel, so we can
no longer rely on getcwd() being stable, which means all fts(3) users
now need to use FTS_NOCHDIR.

Bug: 36032444, 35706513
Test: runtest -x frameworks/base/services/tests/servicestests/src/com/android/server/pm/InstallerTest.java
Change-Id: I67d303d3ecce148052d41444cef67381b1d34ab0

Merge changes Ib6b72ca5,I8ea5dfb9,Iaf954877,I803bf9f4 am: 42f482e139 am: 3049198484
am: 68d8c8bd43  -s ours

Change-Id: I2aec7aa0354f553e826c57747c6f47792aefa41a

Merge changes Ib6b72ca5,I8ea5dfb9,Iaf954877,I803bf9f4 am: 42f482e139
am: 3049198484

Change-Id: I562fc4e296a7e383ad48c4125397dab513d77b5e

Merge changes Ib6b72ca5,I8ea5dfb9,Iaf954877,I803bf9f4
am: 42f482e139

Change-Id: I777ac0dd59147dd8aa28d2093e6baab4d199be42

Merge changes Ib6b72ca5,I8ea5dfb9,Iaf954877,I803bf9f4

* changes:
  vulkan: update Vulkan headers to 1.0.43
  vulkan: update Vulkan headers to 1.0.42
  vulkan: update Vulkan headers to 1.0.41
  vulkan: update Vulkan headers to 1.0.39

Merge changes Ib4fb6c27,I9c41b483,I40372758,Ic787ce17

* changes:
  vulkan: update Vulkan headers to 1.0.43
  vulkan: update Vulkan headers to 1.0.42
  vulkan: update Vulkan headers to 1.0.41
  vulkan: update Vulkan headers to 1.0.39

vulkan: update Vulkan headers to 1.0.43

Test: mmm frameworks/native/vulkan
Merged-In: Ib4fb6c279349c821384f73991105218e14dd6a79
Change-Id: Ib6b72ca50bff647552f1cad1e726d5398104ffe1

vulkan: update Vulkan headers to 1.0.42

Test: mmm frameworks/native/vulkan
Merged-In: I9c41b483fbf48a7404b54bfe405c40b58422ea74
Change-Id: I8ea5dfb9ad79fee64032e9d90b21ee087e4e5baa

vulkan: update Vulkan headers to 1.0.41

Includes some header reordering in generated code due to a new version
of clang-format.

Test: mmm frameworks/native/vulkan
Merged-In: I40372758fb7214a627ab686d8a3de6bf9beae205
Change-Id: Iaf9548776d3462fcd9bcc236be6966ecce265565

vulkan: update Vulkan headers to 1.0.39

Test: make ; mmm frameworks/native/vulkan
Merged-In: Ic787ce171633dce0d17b3ba838d0c3441ac728c3
Change-Id: I803bf9f4cd0ca4db8579a5c28495e8a66088bae2

vulkan: update Vulkan headers to 1.0.43

Test: de.saschawillems.vulkanGears
Change-Id: Ib4fb6c279349c821384f73991105218e14dd6a79

Merge "dumpsys: remove usage of String16::std_string" am: 561b4e5d89 am: a8f36c30d0
am: 250d06bea5

Change-Id: I21f7533d90919a4384f52a3bd5a4976a2cef1d00

Merge "dumpsys: remove usage of String16::std_string" am: 561b4e5d89
am: a8f36c30d0

Change-Id: I5562e6daa4bd3989ec813c0941e483cab9f3ca82

Merge "dumpsys: remove usage of String16::std_string"
am: 561b4e5d89

Change-Id: Ie64c773351594a8925d32b5513e912418bcf35d7

Merge "dumpsys: remove usage of String16::std_string"

vulkan: update Vulkan headers to 1.0.42

Test: mmm frameworks/native/vulkan
Change-Id: I9c41b483fbf48a7404b54bfe405c40b58422ea74

vulkan: update Vulkan headers to 1.0.41

Test: mmm frameworks/native/vulkan
Change-Id: I40372758fb7214a627ab686d8a3de6bf9beae205

vulkan: update Vulkan headers to 1.0.39

Test: make ; mmm frameworks/native/vulkan
Change-Id: Ic787ce171633dce0d17b3ba838d0c3441ac728c3

Merge "Allow replacing of sensord"

Merge "Delete SurfaceFlinger CFlags (phase offset)"

Merge changes from topic 'blobcache'

* changes:
  convert BlobCache to STL
  move BlobCache over (unmodified) to libEGL
  Remove dependency to most of libutils

Merge changes from topic 'libutils-remove-string'

* changes:
  RegionHelper.h: include <limits> directly.
  opengl egl_object.h: add <atomic> include.

Allow replacing of sensord

If a makefile variable is set.

Bug: None
Test: None
Change-Id: I416428efc0b6127048e3ef3860ed5ba7f94efe88

Merge "Use std::string for profile operations instead of const char" am: 7c7d888d94 am: ccaa1f3f78
am: 0045d606d1

Change-Id: I6ad63a777811dbc5e5d74870441d8c76dc3273af

Merge "Use std::string for profile operations instead of const char" am: 7c7d888d94
am: ccaa1f3f78

Change-Id: If09e67000ca5818c27bbfe521554c22209c0d04f

Merge "Use std::string for profile operations instead of const char"
am: 7c7d888d94

Change-Id: I44676d7bc0b9297ccb920066284f371fa2e40c3c

Merge "Use std::string for profile operations instead of const char"

convert BlobCache to STL

This finally gets rid of the utils/BlobCache.h
dependency

Test: compiled & run
Bug: vndk-stable
Change-Id: Idfc6ace472479fae99694f56edb973800b2bef20

Add a simple SurfaceFlinger stress test.

Obviously such tests are always a little arbitrary but this one
seems pretty useful. It runs in ~2 seconds, I got 10 passes in a row with
the fix for 36117490 applied, and 3 system restarts in a row (before giving up)
without the fix applied. Clean up the other tests in preparation for presubmit enabling.

Test: Quis custodiet ipsos custodes?
Bug: 36117490
Change-Id: Ieb9a113f5b21ffb4b8ccd933069ab44e0a2b7a9d

Merge "surfaceflinger: simplify HWC buffer cache clean up"

Use std::string for profile operations instead of const char

Will make things cleaner when adding secondary dex profile support.

Also, add tests to verify profile path creation.

Bug: 26719109
Test: /data/nativetest64/installd_utils_test/installd_utils_test
Change-Id: I8cb1a11cefee21f5001d2b729110696d52fc8323

Merge changes I20d546fd,I15363803 am: 632dbe2c77 am: dc9fcf4218
am: 0c90e1c416

Change-Id: I7f976a58fea69e98bbc3126c8b5288c082422363

Merge changes I20d546fd,I15363803 am: 632dbe2c77
am: dc9fcf4218

Change-Id: I6f091a21f326682deaec7527b40a235fecc4c267

Merge changes I20d546fd,I15363803
am: 632dbe2c77

Change-Id: I5fe86348657aa5e7f733be63c2134af4fe4a5365

Merge changes I20d546fd,I15363803

* changes:
  Allow "android" package name.
  Update is_valid_package_name() to match framework.

Delete SurfaceFlinger CFlags (phase offset)

These config values are now coming from configStore.

Test: Manual
Change-Id: I7128e926dbeaf5763749131b328bcf51576f9bee

Allow "android" package name.

Test: /data/nativetest64/installd_utils_test/installd_utils_test
Bug: 36062649

(cherry picked from commit ab7ac8d5a04bd3f38b85ce20ae5bb382f2a26585)

Change-Id: I20d546fdd20813e96bcb5f061ca5b352dff44450

Update is_valid_package_name() to match framework.

The framework has a new format for /data/app directory names in OC
to protect user privacy.  This change updates the validation logic
in installd to support the new format.

For consistency the logic now mirrors the package name validation
logic used by PackageParser.

Test: /data/nativetest64/installd_utils_test/installd_utils_test
Bug: 33705143

(cherry picked from commit 367ace2f77105f7a09b34a50bc875fd4fa591177)

Merged-In: I41d270942479c52e063c29a2cb9552d7b238b86e
Change-Id: I15363803fb45310697e29be810e800dc1b1902d0

Merge "Re-add mistakenly removed lock in SurfaceFlinger::removeLayer."

move BlobCache over (unmodified) to libEGL

Test: compiled & run
Bug: vndk-stable
Change-Id: I7e816d80456bd1fa890b10b8d5f255deea49b841

Remove dependency to most of libutils

Most of libutils is replaced by using the STL,
this is fine in this case because none of it
leaks out of EGL’s internals.

Test: compiled & run
Bug: vndk-stable

Change-Id: I42ded4043ddc98ed7eaa975fbbb2e754cd3219af

Merge "[SurfaceFlinger] Use sf phase offset from CS"

Re-add mistakenly removed lock in SurfaceFlinger::removeLayer.

Lock was mistakenly removed with "Add detachChildren transaction"
when refactoring the weak-pointer to a strong-pointer.

Test: I will try and follow up with a stress-test in Transaction_test.
Bug: 36117490
Change-Id: I94171c86334bb52b974392186e1c3ebbf9f33a05

surfaceflinger: simplify HWC buffer cache clean up

When a Layer is no longer connected, we destroy the associated HWC
layers on next call to SurfaceFlinger::rebuildLayerStacks or when
the Layer is destroyed.  There is no need to listen to
onBuffersReleased.  Besides, we need to perform the cleanup from the
main thread as we only talk to HWC process from the main thread.

While at it, move HWComposerBufferCache to its own files.

Bug: 35320590
Test: manual
Change-Id: Ifa32f24076b094c8fa9cda8572b03d5bfb8e0b93

RegionHelper.h: include <limits> directly.

Includes are transitively imported by <string> in String8.h + String16.h
but that include is being removed.

Test: pass
Change-Id: I378d533ee5445bc0286c7535581a491f55ca36c9

opengl egl_object.h: add <atomic> include.

Includes are transitively imported by <string> in String8.h + String16.h
but that include is being removed.

Test: pass
Change-Id: I6176389d41aa5eb6fdf8b8afe3378eeb877bc525

Merge "Revise virtual touchpad interface."

Remove debug flags

I accidentally left the debug flags on my last commit.

Bug: None
Test: Still compiles and works.
Change-Id: I9fc652e643aa53ebb531332c3b156b8f001837d8

dumpsys: remove usage of String16::std_string

Test: pass

Change-Id: Ib74a126b91d25376a7235f73bbc12ada62ae7e69

Merge "Fix DummyNativeWindow to respond to new query"

Merge "Reland: Remove NativeWindow and unused DVR functions"

Fix DummyNativeWindow to respond to new query

The `NATIVE_WINDOW_IS_VALID` query must now return true in order for
`eglCreateWindowSurface` to succeed. Technically this isn't true for our
dummy window, but we only need it to create the context, it gets dropped
immediately afterwards.

Bug: 36102224
Test: Created a simple test application. Also ran a Vr application with
      these changes applied.

Change-Id: I2d559962d28db4b1cb6ad188269e4e54cf47245d

Reland: Remove NativeWindow and unused DVR functions

1/ Remove the NativeWindow-backed DVR DisplaySurface implementation.
2/ Unused API such as: dvrCreateDisplaySurface and friends are removed.

Bug: 36033302
Test: Built system, ran basicvr, ran particles on marlin and lucid. Also
updated graphics_app_tests to stop testing deprecated API.

Change-Id: I586a6933f18a5c116bcffdf390ada122fadb9545

Merge changes from topic 'igraphicbufferallocator'

* changes:
  get rid of IGraphicBufferAlloc
  Remove a dependency of EGL on ui/GraphicBuffer.h
  remove EGL dependency on libbinder

Merge "Disable input-vdex when doing speed-profile." am: 028ad63e80 am: a5582c290c
am: a15409a340

Change-Id: I3261131e0f71f6305cdb177ef227ab114be23c60

Merge "Disable input-vdex when doing speed-profile." am: 028ad63e80
am: a5582c290c

Change-Id: I297b481892c75babde5b239bf6b1fe4e3701244e

Merge "Disable input-vdex when doing speed-profile."
am: 028ad63e80

Change-Id: I3ec636cd526bcdb749fc3af87f2ac0d301fbcafd

Merge "Disable input-vdex when doing speed-profile."

Disable input-vdex when doing speed-profile.

input-vdex and dexlayout are incompatible with each other.

bug: 36084905
bug: 36076104
bug: 36059948
bug: 35872504

Test: device boots, vdex works on OTA, input-vdex is not passed
      for speed-profile.

Change-Id: I194311041899399db422867670f5e788d6b9ceeb

get rid of IGraphicBufferAlloc

buffers can now be allocated in-process.

Test: compile & run
Bug: cleanup
Change-Id: I3d4317a9bed20a6d8be2b7ac8fbb85738efb3657

Remove a dependency of EGL on ui/GraphicBuffer.h

the conversion to a ANativeWindowBuffer is now
handled by libnativewindow.

Test: compiled & run
Bug: cleanup
Change-Id: I52dd64574afbc1eff5bdbefacec234a79f8acd97

remove EGL dependency on libbinder

This gets rid of the
eglCreateNativeClientBufferANDROID extension,
which is replaced by NDK’s AHardwareBuffer.

Bug: 35101209

Test: Not tested yet
Change-Id: I7905eb875139781a23e9fa0b3447ea1ad25c71cf

Merge "Create libhwc2on1adapter"

Merge changes from topic 'vintf_arch' am: 45370128a5 am: c2b36f5696
am: 628dafc85e

Change-Id: Ib17fb1e919fdc5a1d3ce652debf1f0d530fde57f

Merge changes from topic 'vintf_arch' am: 45370128a5
am: c2b36f5696

Change-Id: Id6a10176042b63fcefe8298dae2df4c5ac4c5297

Merge changes from topic 'vintf_arch'
am: 45370128a5

Change-Id: Ia8e62d1a64e5a893b651c46e5872854e2f52034f

Merge changes from topic 'vintf_arch'

* changes:
  Update to use bitness info in VINTF.
  Fix forEachTable will copy the table.

Merge "check that the NativeWindowType is valid in eglCreateWindowSurface"

Create libhwc2on1adapter

Extract the adapter out of SurfaceFlinger library to allow code
behind HAL to use it.

Test: Manual
Change-Id: I415aa2674564eccb2c38f76086807f834177f80f

Update to use bitness info in VINTF.

Bug: 35966597
Test: lshal -ir has bitness info for passthrough ref section
Test: lshal --init-vintf has bitness info in the manifest.
Change-Id: Ie9f52bb1ca30ce69856dbf44fab8287563014550

Fix forEachTable will copy the table.

std::initializer_list<T> has T deduced to Table, which will
copy the table. Use pointers instead.

Test: pass
Change-Id: I7da40eb0f5d0171244a9b9caacf37b41fe0af304

Merge "Delete typo in SurfaceFlinger documentation"

Merge "Revert "Remove NativeWindow and unused DVR functions""

Merge "Allow "android" package name."

check that the NativeWindowType is valid in eglCreateWindowSurface

This will avoid certain crash (null dereference),
and return an error instead. This is useful for
app developers as java language apps are not
expected to crash when passed bad arguments

This doesn’t fix bug 36065206, but was prompted
by it.

Test: compile / run
Bug: 36065206
Change-Id: I90d25730b3dd292d756b4c38b51320a466fa6d48

test-hwc2: dump

Test: Add "#define HAVE_NO_SURFACE_FLINGER" to
          frameworks/native/libs/gui/BufferQueueCore.cpp.
      Recompile and flash.
      Run "mm" in frameworks/native/services/surfaceflinger/tests/hwc2.
      Push test-hwc2 to device.
      Run "adb root && adb shell stop".
      Run test case. Ex: "./test-hwc2"

Change-Id: I0215fea464fd126b5f9041435bef7e00b46c6ac8

test-hwc2: virtual displays

Test: Add "#define HAVE_NO_SURFACE_FLINGER" to
      frameworks/native/libs/gui/BufferQueueCore.cpp.
      Recompile and flash.
      Run "mm" in frameworks/native/services/surfaceflinger/tests/hwc2.
      Push test-hwc2 to device.
      Run "adb root && adb shell stop".
      Run test case. Ex: "./test-hwc2"

Change-Id: Ia10ec761712ee03f56782e3df2d6eabf7fb77760

test-hwc2: color and hdr requests

Test: Add "#define HAVE_NO_SURFACE_FLINGER" to
          frameworks/native/libs/gui/BufferQueueCore.cpp.
      Recompile and flash.
      Run "mm" in frameworks/native/services/surfaceflinger/tests/hwc2.
      Push test-hwc2 to device.
      Run "adb root && adb shell stop".
      Run test case. Ex: "./test-hwc2"

Change-Id: Id515de0fac7e842724ae8f27e865fad4933256e6

test-hwc2: present display

Test: Add "#define HAVE_NO_SURFACE_FLINGER" to
      frameworks/native/libs/gui/BufferQueueCore.cpp.
      Recompile and flash.
      Run "mm" in frameworks/native/services/surfaceflinger/tests/hwc2.
      Push test-hwc2 to device.
      Run "adb root && adb shell stop".
      Run test case. Ex: "./test-hwc2"

Change-Id: I845320afdf635d1716310d1183bcb10515615517

Delete typo in SurfaceFlinger documentation

Test: None (this is a comment)
Change-Id: I0876a40697fe5bb5245513520bc1f9a4bf1139f7

Merge "Fix the usage of EIGEN_STATIC_ASSERT_SAME_MATRIX_SIZE"