libvulkan: reduce log level for swapchain usage errors
am: 42a9eecb23

* commit '42a9eecb23c0b72bbee1eb2ef6b0d6586159d1c3':
  libvulkan: reduce log level for swapchain usage errors

Change-Id: I8b053f14678f9e03497ac2b621e117df581d9779

libvulkan: reduce log level for swapchain usage errors

Bug: 26927424
Change-Id: Id9346356436663db83a2d6fa8fefc4ea24d7854b

Merge "Otapreopt: Fix after shared-library work" into nyc-mr1-dev

Otapreopt: Fix after shared-library work

Commit b63d91fd2737680351876406277b6c759f4db33c broke the installd
to otapreopt connection. Fix this. Also refactor a bit to make an
inadvertant break harder.

Bug: 25612095
Change-Id: I8e7b944ecbd5457e4ac4d6ffffdb0468618cf65f

Merge "libvulkan: Slightly better handling of swapchain re-creation" into nyc-dev
am: b2dcc3a122

* commit 'b2dcc3a1220ea5ef6acb5a63659da3c34b1a295a':
  libvulkan: Slightly better handling of swapchain re-creation

Change-Id: I0f66b9e5521388eaf95ce29f5c1f44817780a7b9

Merge "libvulkan: Slightly better handling of swapchain re-creation" into nyc-dev

Merge "vulkan: remove EnumerateDeviceLayerProperties internally" into nyc-dev
am: 691fcb9603

* commit '691fcb96033db2f09030e8c71e86dad7e2fbfd43':
  vulkan: remove EnumerateDeviceLayerProperties internally

Change-Id: I4678a10c62147c7c69bf0f0e81e6e447ed819b2c

Merge "vulkan: remove EnumerateDeviceLayerProperties internally" into nyc-dev

Merge "Camera: Add media/camera NDK documents" into nyc-dev
am: 470241b168

* commit '470241b168a9db7514bc0df775109db591836d4b':
  Camera: Add media/camera NDK documents

Change-Id: Ib66856fba84301834a81f0c361f5244bd152eeb3

Merge "Camera: Add media/camera NDK documents" into nyc-dev

libvulkan: Slightly better handling of swapchain re-creation

Previously we would fail vkCreateSwapchainKHR if
VkSwapchainCreateInfo::oldSwapchain was used, because we were unable
to dequeue all buffers for the new swapchain from an already-used
native window.

Now we disconnect and re-connect to the native window in order to
fully reset state, and allow us to dequeue all buffers. Additionally,
we tag the old swapchain as out-of-date, so future image acquires and
presents will fail with VK_ERROR_OUT_OF_DATE_KHR on that swapchain.
This is less than ideal, but better than what we had, and the best we
have time to do for N.

Bug: 26927424
Change-Id: Ifaa5048376f72a63ecb1dca3d1ff85dbee2c24d0

Merge "SF: Re-enable fence tracker by default" into nyc-mr1-dev

Merge "BQ: Don\'t return fences from dequeue in shared buffer mode" into nyc-dev
am: eb6377eb13

* commit 'eb6377eb133f902e737fab370df7679430a1d8d1':
  BQ: Don't return fences from dequeue in shared buffer mode

Change-Id: Ieb075d9598edcaec542ee06663aa292111968474

Merge "Push empty sync transaction before taking screenshot in tests" into nyc-dev
am: 08588dd760

* commit '08588dd76073a8635aa4175d5b98fc5db86afec4':
  Push empty sync transaction before taking screenshot in tests

Change-Id: Id7648a55288f3f8962e6e314584c9690c3f27854

Merge "BQ: Don't return fences from dequeue in shared buffer mode" into nyc-dev

Merge "Push empty sync transaction before taking screenshot in tests" into nyc-dev

Merge "Create primary profile file together with app data folders" into nyc-dev
am: d25aab3ce8

* commit 'd25aab3ce8aab1b95ac60922981c87c6963a0a6b':
  Create primary profile file together with app data folders

Change-Id: Id186b701d0cc755be0093c704ccd2aa8e0899611

Merge "Create primary profile file together with app data folders" into nyc-dev

vulkan: remove EnumerateDeviceLayerProperties internally

We never use it.  Since it is also deprecated, we will never need it.

Bug: 27910383
Change-Id: If4e556bb69890153be2987b165482e43ab06a890

Push empty sync transaction before taking screenshot in tests

Because of ag/1043009 it's now necessary to push an empty sync
transaction before taking a screenshot to ensure that all transactions
have been applied.

Bug 29074285

Change-Id: I9f1b4a5b6b1f39e1ff26d536817aef6f40865177

SF: Re-enable fence tracker by default

- Don't check whether the fences have completed or not unless the data
  has been requested.

Bug 22431447

Change-Id: Id4cb8bfba38547884849ef23c9b6f42f0d4d0750

installd support for profman --dump-info-for
am: 6727c2d0bf

* commit '6727c2d0bfc876c62d6ee60d8a05d26bef1d56ef':
  installd support for profman --dump-info-for

Change-Id: I3df4eee3cb34f6c163e73351180b227b8d6864e8

BQ: Don't return fences from dequeue in shared buffer mode

Bug 29007505

Change-Id: I112ae6bd64ddc9a923ad9080e85a71adb96146af

installd support for profman --dump-info-for

Bug: 28748264
Change-Id: Idcd3ed86294c1d0c75c26ca938b88744a25e387a

Merge "svcmgr: don\'t send reply to one-way txn (part deux)" into nyc-dev
am: e4ecdb66d5

* commit 'e4ecdb66d5c2eed64cbd5b267e0621cab7b8a63b':
  svcmgr: don't send reply to one-way txn (part deux)

Change-Id: Id062b1b3998ab018654bc7830f275d1ac6d64f6d

Merge "svcmgr: don't send reply to one-way txn (part deux)" into nyc-dev

Use explicit path for /vendor/xbin/wlutil
am: 0816520c5c

* commit '0816520c5cd60519d8e221ed92497aa5464e3039':
  Use explicit path for /vendor/xbin/wlutil

Change-Id: I1fd9593c8d9c0b0e3dd87c9a70b6b2e81c9b100a

Use explicit path for /vendor/xbin/wlutil

At this time /vendor/xbin is not in root's path.

Bug: 28850734
Change-Id: Ib6c2b6f8b0285854670c843dcc437cfcc84484a8

Merge "vulkan: fix crash when only non-global layers are enabled" into nyc-dev
am: a101f6a446

* commit 'a101f6a446722697e0ed6afb128829937b90d60a':
  vulkan: fix crash when only non-global layers are enabled

Change-Id: I65baa95e0b00f3fbc136d435c4481256a7e551bd

Merge "vulkan: fix crash when only non-global layers are enabled" into nyc-dev

Camera: Add media/camera NDK documents

Include media/camera ndk headers to codegen source.
Add docs/images directory for image links in generated documents.

Bug: 27102995
Change-Id: I9c75e2ceba3458c9962c26cf786e1402fd963696

Add "ip maddr" to dumpstate output for debugging.
am: f7f2b0fd05

* commit 'f7f2b0fd059bc26e27c022f3fb446fc74eab6c88':
  Add "ip maddr" to dumpstate output for debugging.

Change-Id: Ic3898db87bc7b726079d922ff24eaa1f8201f288

vulkan: fix crash when only non-global layers are enabled

We fail to set up get_instance_proc_addr_ and get_device_proc_addr_ when
there are layers but only non-global ones.

Bug: 28990561
Change-Id: I82704f946deb5a76640f9dfb792e6fd118c78aa7

Add "ip maddr" to dumpstate output for debugging.

Bug: 28966611
Change-Id: I8fdab94fbd41448e52f2fed406e8f43edf7f18ad

Merge "Check the existence of raft dir." into nyc-mr1-dev

svcmgr: don't send reply to one-way txn (part deux)

Do not split transaction into two when sending synchronous responses.

Partially revert changes associated with binder_send_reply from
previous incantations in commit 51f592c7bad13d3d8e8f7cfec25edfef42901802
that resulted in a performance regression.

Signed-off-by: Mark Salyzyn <salyzyn@google.com>
Bug: 27111612
Bug: 28696050
Bug: 28946245
Change-Id: I12edc10a2d1408a71507a7983afa7f17b5a6a81b

Mark the multiwindow divider as a trusted overlay.
am: b9be0cf397

* commit 'b9be0cf397eb6053fb5b472d91cf2d6a22c6693f':
  Mark the multiwindow divider as a trusted overlay.

Change-Id: I1906ca39ce35a436ba65c3815e0bdda8a3c00e9e

Mark the multiwindow divider as a trusted overlay.

Having it be untrusted breaks the permissions granting dialog in
multiwindow mode.

Bug: 28828041
Change-Id: I9193dbce7315ebaa205673edd4f406d0b72665c8

Create primary profile file together with app data folders

Bug: 28785119
Bug: 28625993
Change-Id: I12c0de85bc3dc35199acf32c10fe69f93e76d7fc

Check the existence of raft dir.

b/28430346

Change-Id: I6ceb54f60c766d7b73ae93daa81f57b5a229e291

Merge "Write compressed and encoded raft logs to stdout if not zip_writer" into nyc-mr1-dev

Write compressed and encoded raft logs to stdout if not zip_writer

Change-Id: Ib9948bebf9d5e2ef823e08b48fb67c36e0079a45

Merge "BufferQueue/SF: Add OccupancyTracker" into nyc-mr1-dev

Merge "SF: Force empty sync transactions to apply anyway" into nyc-dev
am: 7443c1f253

* commit '7443c1f253008acd868af1e35449dd5eb3139c8d':
  SF: Force empty sync transactions to apply anyway

Change-Id: Iaa91350ba6e499d145e7adfcc1874601fff33775

Merge "SF: Force empty sync transactions to apply anyway" into nyc-dev

SF: Force empty sync transactions to apply anyway

Following the change to not flush transactions in the screen shot
path, this enables the use of empty synchronous transactions as a
flush mechanism for previous asynchronous transactions.

Bug: 27098060
Change-Id: I282fdfffb72db148edcbf1ded65b15816fc714a2

Merge "SF: Remove forced transaction on screenshot" into nyc-dev
am: 854071a38f

* commit '854071a38f250bd0638f1dfdb99b9ccc0010e7d0':
  SF: Remove forced transaction on screenshot

Change-Id: I79297918957661befdec24345bdfff2720433188

Merge "SF: Remove forced transaction on screenshot" into nyc-dev

Merge "HWC2: Fix breakage from header change" into nyc-dev
am: 80bc52aefa

* commit '80bc52aefa56e590f05bc30ff0182d860f00b128':
  HWC2: Fix breakage from header change

Change-Id: I02f8b45ded3d00dc2d34689f5b902d77ee74ca1e

Merge "HWC2: Fix breakage from header change" into nyc-dev

stagefright: define mpeg2 HighP level constant
am: 9e332babb9

* commit '9e332babb92d329c3c9b2e58fef77252cc11e52b':
  stagefright: define mpeg2 HighP level constant

Change-Id: I4db62224022bccc89ee4fdda2e32813a1f1c074d

stagefright: define mpeg2 HighP level constant

Bug: 28671284
Change-Id: Ic97fd9c3cb4ce04f9cfcd18a4daf68db3349694b

vulkan: advertise VK_EXT_debug_report
am: 319382517a

* commit '319382517afe913285df5bbf76a8a484e6adfbd0':
  vulkan: advertise VK_EXT_debug_report

Change-Id: I904abeac1234e146f73b14a817c42adc9b6bc7bb

vulkan: add a wrapper for hwvulkan_device_t
am: 31b2e4f01d

* commit '31b2e4f01d31aae107340cbc4221b8d49fb42767':
  vulkan: add a wrapper for hwvulkan_device_t

Change-Id: I4d64d13427218c437babb31d85982c438ed7f2af

vulkan: advertise VK_EXT_debug_report

We use VK_EXT_debug_report to report most messages and we should always
advertise the extension.  This is complicated however because the HAL may
also advertise the extension.

Do the minimal and cache whether the HAL supports VK_EXT_debug_report in
class Hal.  More refactorings will be required if we want to be more
flexible.

Bug: 28753291
Change-Id: I0298b2c3024e2ed059411128dbbe264c0b289a4b

vulkan: add a wrapper for hwvulkan_device_t

Add class Hal as a wrapper for hwvulkan_device_t.

Bug: 28753291
Change-Id: I9612ee5515c7f2266cfc1ef0c8583fa98c82dd86

Merge "vulkan: do not use exported functions internally" into nyc-dev
am: e6759c3d88

* commit 'e6759c3d886491f01dd2295f71d22609b3b9b259':
  vulkan: do not use exported functions internally

Change-Id: I138aa42e7098b69d00517d9bac3d811b9b56e8a7

Merge "vulkan: do not use exported functions internally" into nyc-dev

Resolve merge conflicts of 6108f5d to nyc-mr1-dev

BUG: 28867848
Change-Id: I99392f1706af5fb9268ad1b32aa7f4b10e397d84

Merge "Expose functions used to add entries to the zip file." into nyc-dev

Expose functions used to add entries to the zip file.

Currently only dumpstate.cpp is using the new functions and variables
used to add an entry to the zip file so they're defined locally, but
they should be exposed through the .h so they can be used by the
device-specific dumpstate_board() implementations.

This CL moves the following definitions to the .h:

- bugreport_dir
- ZIP_ROOT_DIR
- add_zip_entry()
- add_zip_entry_from_fd()

It also refactored functions that were taking a specific file location
to use bugreport_dir instead, to avoid pollution on main().

BUG: 28867848
Change-Id: Ic5faa203b0863bc950d640f22fb91977e52b6bfc

omx: add asString for new enumerations and fix some issues
am: 1b9c8069f4

* commit '1b9c8069f423f25c4dc166618d9fbe8d94d7f7c0':
  omx: add asString for new enumerations and fix some issues

Change-Id: I774af98ced6dd8bf7aac8873e3d93655fd5c6f2a

omx: add asString for new enumerations and fix some issues

Bug: 28671284
Change-Id: I54e98d727f974544d51b2a14b565ad24912ab242

Merge "omx: add missing MPEG4 levels" into nyc-dev
am: 6472c8c660

* commit '6472c8c660232300a0d2bf8721ec028a2db229a9':
  omx: add missing MPEG4 levels

Change-Id: If2a70773c6fcc4975de4a6fbc161e544f5d40715

Merge "omx: add missing MPEG4 levels" into nyc-dev

vulkan: do not use exported functions internally

Our vkGet*ProcAddr uses the exported functions.  They will break when any
of the exported functions are overridden (e.g., through LD_PRELOAD).

Unexport and move all exported functions to vulkan::api namespace.
Re-export them by having vkFoo as a wrapper to vulkan::api::Foo.

Another option is to re-export vulkan::api::Foo by having vkFoo as an
alias using __attribute__((alias)).  That results in smaller binaries.
But we will not be able to catch mismatches between vulkan.h and
vulkan.api.

To avoid future breakage, define VK_NO_PROTOTYPES for all files except
api_gen.cpp.

Bug: 28886971
Change-Id: I08fde7ebb247f8c7e040ccf812b40b02094d3c7f

Merge "Ensure memory ordering around libagl and EGL refcount operations" into nyc-dev
am: bac438bbb6

* commit 'bac438bbb682b09b618f47f2e36628a464afc8ea':
  Ensure memory ordering around libagl and EGL refcount operations

Change-Id: Ie2643fd243a9aa66080457d318a35550c502fd67

Merge "Ensure memory ordering around libagl and EGL refcount operations" into nyc-dev

Ensure memory ordering around libagl and EGL refcount operations

The android_atomic_inc/android_atomic_dec functions don't impose
sufficient memory ordering. Using them for object refcounting could
allow an object to be destroyed prior to writes by a different thread
being visible.

Bug: 28820690
Change-Id: Ie018091035174255a22ebc52852528cdaec2d648

vulkan: Update vulkan.h to 1.0.13
am: 2676338dd6

* commit '2676338dd692b7d1e1c276d82e6b0492db53ab2e':
  vulkan: Update vulkan.h to 1.0.13

Change-Id: I7dce6307edb8d04a5706da465b5de8457d7d236a

HWC2: Fix breakage from header change

Updates the shim and adapter to handle the header changes introduced
by "HWC2: Add format to VD create, surface damage" in libhardware.

Bug: 22767098
Change-Id: I8142301c121d9b5602a4455f3fbadefefd685cf0

vulkan: Update vulkan.h to 1.0.13

vulkan.h and vk_platform.h imported from
github.com:KhronosGroup/Vulkan-Docs @ 2656f459333b3a1dc63619a9ebd83490eea22e93

Updated vulkan.api to reflect the changes in vulkan.h. There were no
changes to any of the code we generate. I didn't update the
vulkan_h.tmpl, since we're not currently using it and it was already
out of date.

Bug: 28878216
Change-Id: I71676683148fae7ad8799bd39386a931319aa395

Merge "libvulkan: Don\'t try to load compressed/unaligned libraries from APK" into nyc-dev
am: b59de7fa32

* commit 'b59de7fa32b0bcaf52d00290d075d8e9c8f2dd2b':
  libvulkan: Don't try to load compressed/unaligned libraries from APK

Change-Id: I046e14ca7462635672f12f9388123c2ec6dea826

Merge "libvulkan: Don't try to load compressed/unaligned libraries from APK" into nyc-dev

Fix the DUMPSYS skip.
am: 34222565fc

* commit '34222565fc9cf167d8ebe6068a9695f4b0b0f1b0':
  Fix the DUMPSYS skip.

Change-Id: I0ea7926024cec428785687582a81326a750ca669

Fix the DUMPSYS skip.

b/28869921

Change-Id: Idc108b2d3cfd237f459467a3c399a0ba5e248d24

libvulkan: Don't try to load compressed/unaligned libraries from APK

Bug: 28825642
Change-Id: I46ea3a54010cccf9e23696a4aff52a42a31d92b1

omx: add missing MPEG4 levels

Bug: 28671284
Change-Id: I96b50580cc29c11732fd2b496ba6d832b7067256

Merge "vulkan: fix double dlopen for layer libraries" into nyc-dev
am: 79994146e1

* commit '79994146e18fefe37550111c5703c6a6859644d6':
  vulkan: fix double dlopen for layer libraries

Change-Id: If4e42ce9d3ead2a746ab54def62ed334e7e57010

Merge "vulkan: fix double dlopen for layer libraries" into nyc-dev

Merge "Make bugreport_dir a common variable." into nyc-mr1-dev

Merge "Preserve parameters to getProcAddress functions" into nyc-dev
am: accafc69f5

* commit 'accafc69f50aeeb57615e14ff0b787ea322a0627':
  Preserve parameters to getProcAddress functions

Change-Id: I1610c9053c3c710be0914d74b8a5e22c5218f525

Merge "SF: Fix arbitrary rotation with inverse display transform" into nyc-dev
am: 885ac76b90

* commit '885ac76b90e2f3e5cfaec7bbc8ff2e1de8d62435':
  SF: Fix arbitrary rotation with inverse display transform

Change-Id: Idceb5782ac8e71e7f4e4833946a32b85df59fc69

HWC2: Set composition type before setting buffer
am: 0f67b3f01f

* commit '0f67b3f01fc74bbd8cf4be24fce4a1e42f102282':
  HWC2: Set composition type before setting buffer

Change-Id: I41f3215b2950b55d3a86b3cb01c12056371b3802

Merge "Preserve parameters to getProcAddress functions" into nyc-dev

Merge "SF: Fix arbitrary rotation with inverse display transform" into nyc-dev

Make bugreport_dir a common variable.

Recently, new functions were added to dump specific files, and each of
them was requiring a file location, which was polluting main() with
variables specific to each function.

BUG: 28430346
BUG: 28640932
Change-Id: I90f044a2adb961e3038145dfd635cd3ed0042e34

Add fingerprint sensor navigation keys.

Change-Id: If27d1170ede8bf3124644d47d90384313137bb18

Allow fingerprint navigation keys to be remapped by device rotation.

Change-Id: Ibfd827504e8c9aa6895bdca79eeb13be2784d5fe

vulkan: fix double dlopen for layer libraries

Bug: 28826014
Change-Id: I10693426f987e63bb490ac8d1f4445ce2d195d8d

SF: Fix arbitrary rotation with inverse display transform

Bug 28043535

Change-Id: I8b939e17f91cf7e5b617bbcceb9df4efcc9d0161

Preserve parameters to getProcAddress functions

The contents of register r0 parameter were overwritten after commit
4690754. That used an "r" register to hold larger offsets, but doing
that overwrote r0 before it was passed on to the next call.
Applying the offset as two parts allows the total largest offset of
3588+1020 to be reached without using a register to hold that value.
Use an "addne" instruction to add the _api offset of up to 1020 within
the extensions array.
Use a "ldrne" offset to add the offset to the start of the extensions
array.

Bug: 26541117
Change-Id: Ie19e4aa584c9fa6fdec4de18e858777ac59d8dbd

HWC2: Set composition type before setting buffer

SurfaceFlinger was incorrectly setting the buffer handle before
setting the composition type. This led to cases where the layer was
initially marked as Invalid or Client, allowing HWC2 to ignore the
buffer handle, even though the layer was later changed to Device
before presentation.

Bug: 28814745
Change-Id: I41189495dcb254f04e6b2b329c044301c5cff97c

SF: Remove forced transaction on screenshot

Removes the forced transaction that screenshots used to perform
before capturing the screen. This was initially added to prevent
screenshot capture from sneaking in between when a transaction is
closed and when it is actually applied (at the next vsync), but in
the presence of deferred transactions today, forcing a transaction
to occur doesn't guarantee that the last transaction that
WindowManager pushed has actually been committed yet, since it may be
waiting for a buffer with which to synchronize.

If WindowManager wants to guarantee some arrangement of windows is
present for a screenshot, it needs to set the synchronous flag to
true when calling closeGlobalTransaction and wait for the call to
return.

Bug: 27098060
Change-Id: Iaa36d76df521076cc8ed16576f651543bae419a8

Merge "dumpstate: convert sprintfs to snprintfs" into nyc-dev
am: 853f8f9c0b

* commit '853f8f9c0b95120731459c1f7567af69ce457fe0':
  dumpstate: convert sprintfs to snprintfs

Change-Id: I62360c6f0daec9fa1149e91141e0bb5f5a09c2d2

Merge "dumpstate: convert sprintfs to snprintfs" into nyc-dev

dumpstate: convert sprintfs to snprintfs

Bug: 28731007
Change-Id: Icfa4d6dfaf69e989ec785146a6bde1afb6a6f345

vulkan: check flags for early debug report callbacks
am: ede02ac245

* commit 'ede02ac245b5f2aa1139d6cb0e2dc7c0f8f290d2':
  vulkan: check flags for early debug report callbacks

Change-Id: I5b1641339d64bafde62264a224b752caaa404d68

BufferQueue/SF: Add OccupancyTracker

Adds an OccupancyTracker to BufferQueue. This module keeps track of
how many buffers are in the queue over time, which, in combination
with various aggregation of these statistics, allows SurfaceFlinger
to report what fraction of the time a given layer was double- or
triple-buffered.

Change-Id: Ida6e967dc5483c00a633e9fe03998e420dd88502

vulkan: check flags for early debug report callbacks

We have a different path to invoke early (that is, vkCreateInstance) debug
report callbacks.  Check for "flags" in this path as well.

Bug: 28753666
Change-Id: I49b5f2920b38ffdf69036f352166ae3b0aca62c5

libvulkan: Load layer libraries into the app namespace
am: 40c07a1d00

* commit '40c07a1d002561fc8db987ec30d890b041fb6336':
  libvulkan: Load layer libraries into the app namespace

Change-Id: If91efcb98b45650931bd6a01879378a4e73fcc11