Merge "BufferQueueProducer: Signal consumer when a buffer is detached" into nyc-dev

vulkan: Updated IHV documentation

Also deleted DevelopersGuide.pdf, which was very out of date and has
been superceded by public documentation on developer.android.com.

Change-Id: I7e47177191a773eea6db3ee2af0a287877cb8532

Merge "Revert "Attach profiles to bug reports"" into nyc-dev

Revert "Attach profiles to bug reports"

This reverts commit 0faf2b870f934c1cdd883d29642fd84da3b352e2.

Bug: 28610953
Bug: 29395357
Change-Id: I609fe344dfc61311e37b91b48480d9392764a598

Merge "Revert "GLConsumer: add build-time disable of gpu protected content"" into nyc-dev

Revert "GLConsumer: add build-time disable of gpu protected content"

This reverts commit dea97b7aa2b5cfbb7d42f30bcbf86a01778b609b.

Bug: 28545166

Change-Id: I4b069e14c2b99e190ebdea6df1ed83827ffe17b6

Merge "Fixed file descriptor leak in IMemory" into nyc-dev

Add FrameStats default constructor am: 9a8c42a72b am: 8cd7d257de am: 67105cc24d am: b990d75315 am: a1952f0a6d am: 81d678cf18 am: ed968dc61c
am: d1d1d4e922

Change-Id: I581c4df5e3866e8a2326830e6b9e6ca2ffb3bf1c

Add FrameStats default constructor am: 9a8c42a72b am: 8cd7d257de am: 67105cc24d am: b990d75315 am: a1952f0a6d am: 81d678cf18
am: ed968dc61c

Change-Id: Id30d35b89c46fda89990bed6acf932240bdb6890

Add FrameStats default constructor am: 9a8c42a72b am: 8cd7d257de am: 67105cc24d am: b990d75315 am: a1952f0a6d
am: 81d678cf18

Change-Id: I24f1e2594fdbd2022a534332009cad9564becd54

Add FrameStats default constructor am: 9a8c42a72b am: 8cd7d257de am: 67105cc24d am: b990d75315
am: a1952f0a6d

Change-Id: Icc23cc2a483b13c6ac2134008d553574fffe06f4

Add FrameStats default constructor am: 9a8c42a72b am: 8cd7d257de am: 67105cc24d
am: b990d75315

Change-Id: Ibc0caec8dc613a6701791f1a633f3793522615af

Add FrameStats default constructor am: 9a8c42a72b am: 8cd7d257de
am: 67105cc24d

Change-Id: Icf85c944e561763b0a9c353f84d614a992168c20

Add FrameStats default constructor am: 9a8c42a72b
am: 8cd7d257de

Change-Id: Ic0df6b15e28ac2acda566fa7f30c9460d36f2a00

Add FrameStats default constructor
am: 9a8c42a72b

Change-Id: Ib78d68c71438391711ea3b91102408d5ad0b966b

Merge "Fix assignment of 565 format when creating native client buffers." into nyc-dev

Fix assignment of 565 format when creating native client buffers.

Change-Id: Ia379f469a6e18e1f31748dc97aa596d03e507f84
Fixes: 29279664

Fixed -t order (should be *after* dumpsys :-)

BUG: 29042642
Change-Id: I1b88634b0152f849a57f8e82adc82bdc5e609db7

BufferQueueProducer: Signal consumer when a buffer is detached

When a buffer is detached, it should be cleared from the consumer-side
cache as well.

Not doing so can leak buffers no longer owned by the queue for extended
lengths of time, depending on the exact usage patterns of the producer
and consumer.

Bug: 28695173
Change-Id: I2c618be51f2c827d92d0e19b92a2d5befda87c38

Merge "Set version to 1.0" into nyc-dev

Merge "Expose more useful functions that can be used by dumpstate_board()." into nyc-dev

Fixed file descriptor leak in IMemory

Include dup in critical section to avoid multiple dup calls from
different threads.
The leak happens more frequently since patch in frameworks/av -
6773d4777f4ccbbe6377e4ae1b42c117066ae6ba when recording video since it
will access IMemory from CameraUtils::isNativeHandleMetadata

Bug: 28924660
Author: Henrik Antonsson <henrik.antonsson@sonymobile.com>
Change-Id: I769a914c3adc505b522659a0c43719fb2922e6fd

Merge "Improved error handling." into nyc-dev

Improved error handling.

adb bugreport now properly handles stderr.

BUG: 29161586
Change-Id: I7a59d1feadda3d160eae6fe447a7a3c7e0843cbb

Merge "Explicitly sets dumpsys timeout." into nyc-dev

Explicitly sets dumpsys timeout.

The run_command() function takes a timeout as argument, and dumpsys can
also optionally takes a timeout with the -t argument, hence it makes
sense to pass the same value for both (otherwise dumpsys will timeout in
10s).

BUG: 29042642
Change-Id: Ib6e987d94e966686bab71223d7f910af9e549814

Merge "Attach profiles to bug reports" into nyc-dev

Attach profiles to bug reports

Includes the entire '/data/misc/profiles' folder in the bug report
ZIP file if not user build.

Bug: 28610953
Change-Id: I99d46f024e5557cf94e6045a51c46a605a00c52b

Merge "SurfaceFlinger: Add mode to apply position with resize." into nyc-dev

Merge "Fix Parcelable::readParcelableVector<T>" into nyc-dev

SurfaceFlinger: Add mode to apply position with resize.

For some cases, like scaled windows with shadows,
we need to be able to apply the position concurrent with
window resize. This is because the scaling of the shadows causes
the top left coordinate of the non shadow surface content
to change before and after the resize.

Bug: 28899837
Change-Id: I522eacfbbcd79707dc1e5ab71901a263b3004ba9

Expose more useful functions that can be used by dumpstate_board().

BUG: 28640932
Fixes: 28867848
Change-Id: I38123e5b348f4560212559176eb60dbad4f22364

Merge "Enable profman pretty printing" into nyc-dev

Merge "SF: Also edit currentState in doTransaction" into nyc-dev

Enable profman pretty printing

Bug: 28748264
Change-Id: Ia625a7976d9b95ec3b7e1d03af39af9f39015f3b

Merge "libgui: Check result in hook_dequeueBuffer_DEPRECATED" into nyc-dev

Merge "Allow setting alpha to 0 when creating native client buffers." into nyc-dev

libgui: Check result in hook_dequeueBuffer_DEPRECATED

Surface::hook_dequeueBuffer_DEPRECATED calls c->dequeueBuffer without
checking the returned result value.  It should check for failing
result and return on error without using the values that
c->dequeueBuffer did not set.

Bug: 28692406
Change-Id: I10f9a23ad1052c5149040124366e0eb91b616267

SF: Also edit currentState in doTransaction

Bug 28899837

Change-Id: I46c8301549c627c5d1c7efa661055aba787a33d6

Allow setting alpha to 0 when creating native client buffers.

Note that RGBA sizes are checked later in the function anyway, so there was no
reason to disallow 0, but it did preclude setting alpha to 0 for 565 buffers.

Fixes: 29126799

Change-Id: Iba11649976221ab385227fc022ae74b14c342203

Fix Parcelable::readParcelableVector<T>

Parcelable::readParcelableVector<T> was apparently never
used. It now is and needed some adjustments to actually
work.

Bug: 22914603
Change-Id: I58ead3b3956299d549cff245588f56f3220563d3

libvulkan: reduce log level for swapchain usage errors

Bug: 26927424
Change-Id: Id9346356436663db83a2d6fa8fefc4ea24d7854b

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

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

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 "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

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

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

Set version to 1.0

BUG: 27366368
Change-Id: Iac8ed413d41d0c26a39343025920f433d5d7ddba

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

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

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

Add FrameStats default constructor

Bug 28592402

Change-Id: I857e46c9ab3ffae0d96923d665d13a4128a6cafa

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.

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

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

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

stagefright: define mpeg2 HighP level constant

Bug: 28671284
Change-Id: Ic97fd9c3cb4ce04f9cfcd18a4daf68db3349694b

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

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

Bug: 28671284
Change-Id: I54e98d727f974544d51b2a14b565ad24912ab242

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

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

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

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

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

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

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

dumpstate: convert sprintfs to snprintfs

Bug: 28731007
Change-Id: Icfa4d6dfaf69e989ec785146a6bde1afb6a6f345

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

Bug: 28213888
Change-Id: I79901693c30f0e69730643b424e504a8d3628624

libvulkan: Discover layer libraries in entire search path

Previously we only searched the unpacked native library path for layer
libraries. Now we search all directories in the dynamic linker's
search path, including unpacked libraries in the APK itself.

Bug: 28213888
Change-Id: I37772fd90f705592ae1725c7c8603b0174e6b133

Merge changes from topic 'vk-debug-report' into nyc-dev

* changes:
  vulkan: improve vulkan::driver logcat messages
  vulkan: use Logger in vulkan::api
  vulkan: use Logger in the generated code
  vulkan: add DebugReportLogger
  vulkan: make debug report callbacks available in DeviceData
  vulkan: avoid duplicated app messages
  vulkan: constify DebugReportCallbackList::Message
  vulkan: refactor DebugReportCallbackList

Merge "SF: Apply translations to transparent region" into nyc-dev