Merge "SF: Prevent non-sync transactions from syncing" into nyc-dev

SF: Prevent non-sync transactions from syncing

Clears the sync handle and frame number after registering a deferred
transaction to prevent other non-synchronized transactions from
erroneously getting marked as synchronized.

Also changes the application of pending states such that all valid
pending transactions are applied instead of breaking after the first
one.

Bug: 27061473
Change-Id: I98f0f26e9d9bb0d266563f209e94dfd776d8f8ea

Merge "Send broadcasts as Shell." into nyc-dev

Send broadcasts as Shell.

The BUGREPORT_STARTED was being sent as root, which would cause it to
bypass Shell restrictions when DISALLOW_DEBUGGING_FEATURES is set.

Also added the missing --receiver-background and unhardcoded shell's uid
from previous changes.

BUG: 26814173

Change-Id: I867361efafb96c7e709f83806a9301afc637a27e

Merge "BQ: Fix iterator use" into nyc-dev

Split dumpsys into critical and non-critical sections.

Order in which we report stats in bugreport can disrupt cpu and memory
stats. This CL sorts the bugreport process
from least impactful to most impactful to try and keep the
system stats as close to its initial state as possible throughout the BR (eg.
things like dumpsys meminfo should run before dumping thread stacks)

BUG: 26639621
Change-Id: I29dccfd423d83a6a69b7e025d45cb9cebfc41cc4

BQ: Fix iterator use

Don't use an iterator after it's been removed from the container.

Change-Id: I374b454bba95d10dcec6a600bf8f16e5636e426b

loader: Can't dlclose until all done with the layers

The loader was freeing the layers (dlclose) at the
bottom of the DestroyInstance chain. Would get a
segfault when returning from DestroyInstance_Bottom
as layers would try to run that are no longer in memory.

Change-Id: I25cc611fed9d8b3a90fd2652e53cd450daefa08c
(cherry picked from commit 3160835e44fcb0c5367d2f50b132e41c789998d5)

loader: Implement new layer initialization

We now call down the instance and device chains
as part of the creation process. The loader puts
together "LinkInfo" structures that each layer
uses to reference the next layer down in the stack.
These info structures are attached to the CreateInfo
pNext structure. Also have an info structure that
the loader terminator functions (CreateInstance_Bottom
and CreateDevice_Bottom) that includes loader state
they need.
As part of this the CreateDevice function now lives
on the device chain and it has been split into
Top and Bottom functions. Top builds the LinkInfo
structures and Bottom makes the actual call to the ICD
to create the device object.

Change-Id: I5f5a6d579c34942511cce288ba8fa51f9f89f67f
(cherry picked from commit 9091095245975597a0b2031c3b4847f780c1721e)

loader: Add GetName and GetDescription for logs

Handy to have more layer detail for logging.

Change-Id: I6933a8fcb90b9cfb70fc85a358244863aac9d702
(cherry picked from commit cf275f37b4a903b5dcc95b5100db667c906aa33f)

loader: Layer init structures

Update to match LoaderAndTools vk_layer.h
which includes the new layer initialization structures.

Change-Id: Ie6bf2a1705373bad3c77dd0b04be1263f1c4fc46
(cherry picked from commit f7491b28bfcc47ce09b32168a5fd25b58c6aceab)

loader: Only use driver's DebugReport if it has one

Change-Id: Ic020b9d5a95c9ddd20dd4c94fd6e7de050b83f2b
(cherry picked from commit b776ba1cff149bb724a1958cc3f470ba5c113b63)

tools: Need to specify pQueuePriorities

Change-Id: I689c3e8da9e8cf00705f2f90d472c1268a978f1b
(cherry picked from commit 363ac5d35ca076f419c14f14bce603d5d82553fa)

Merge "libvulkan: Support R5G6B5_UNORM_PACK16 as a swapchain format" into nyc-dev

Merge "libvulkan: Un-revert and fix VK_PRESENT_MODE_MAILBOX_KHR" into nyc-dev

Merge "vknulldrv: Report physical device limits" into nyc-dev

Merge "libvulkan: Initialize VkSurfacePropertiesKHR::supportedCompositeAlpha" into nyc-dev

libvulkan: Support R5G6B5_UNORM_PACK16 as a swapchain format

Change-Id: I489cf12965372d5281d200102a4632e6e8d75ede

Merge "vkinfo: Fix bytes -> megabytes conversion" into nyc-dev

libvulkan: Initialize VkSurfacePropertiesKHR::supportedCompositeAlpha

Also rearrange the code to set the fields of this struct in the same
order they appear in the struct, to make it easier to tell when
something is missing.

Bug: 26991609
Change-Id: I681ed42c96ac75bb1589d30ab75f760f660eacdd

vknulldrv: Report physical device limits

Limits are set to the minimum values allowed by the spec.

Change-Id: Ib3ca99ad6d1537e2ed3fa29e15368419f2331ba7

libvulkan: Un-revert and fix VK_PRESENT_MODE_MAILBOX_KHR

This reverts commit 9ec497c, which was a revert of f333922, and fixes
the problem that caused the initial revert.

MODE_MAILBOX corresponds to BufferQueue "async" mode; in that mode we
need one extra un-dequeued buffer at all times. We weren't accounting
for that, and weren't allocating enough bufers.

Bug: 26918467
Change-Id: Ice67e6d16828e50413ae453ecf3c67b8fb288f1a

Fixed format.

Change-Id: If0322e3dcf56005541aa08e0ab91dae10d2d385d

Fixed C++ syntax.

Change-Id: I9702fdede589dd37cb498bbf63c0773816a7e3b8

dumpstate: report process times

(cherry picked from commit 0751efa64cddfab69749236193e776058c9db595)

- Fix per-pid helper to pick up comm if cmdline is empty
  and assume kernel thread.
- Add per pid report command line user time if not kernel thread,
  system time and I/O time, plus percentage if non zero.

Bug: 26966375
Change-Id: I6b499f535cc3687eb468b32ae39e7d151dffb11f

vkinfo: Fix bytes -> megabytes conversion

Also only report 256 MiB (chosen arbitrarily) as the null driver's
memory heap size. The previous value wasn't reasonable on 64-bit
systems (2^63 + 1).

Change-Id: Id57a4895752af45d046072ae8fee386ca219e82d

vkinfo: Print VkPhysicalDeviceFeatures

Change-Id: Ifa045de906ce2612382f470168438f80c7188b0a

Merge "vkinfo: improve indentation, add verbosity parameters" into nyc-dev

vknulldrv: Report valid VkPhysicalDeviceFeatures

Change-Id: I616f8e40d42abb812cc150d31b3a07d41d86b43c

vkinfo: improve indentation, add verbosity parameters

Change-Id: I23b4a08a24d6fb1463db55dbbb342eca866fc228

vulkan: Document codegen instructions and code style

Change-Id: I7565506a866b72f4ce31ac6bdd3d97e357497ab0
(cherry picked from commit e0d27bb1f3671401c2f6d84a60111f5c695c97d3)

Improved dumpstate logging.

dumpstate generates 3 types of output:

- stdout: used for the bugreport itself and is redirected to a file
  if requested.
- stderr: used for errors, is only useful when dumpstate is called from
  adb (otherwise is ignored).
- ALOG: logged into logcat and available in the bugreport.

This current approach has 2 problems:

- stderr output is ignored when dumpstatet is called from bugreport
  services.
- bugs on dumpstate are often hard to diagnose
  (AKA 'Who watches the watchmen?').

This change mitigates these problems by redirecting stderr into a log
file (which will also be included in the bugreport notification) and by
keeping the .tmp file around (for the extreme cases where the .zip file
could not be generated or got corrupted).

BUG: 26906985
Change-Id: I73e7b1dd10ad9f83b7aa1043131ff3b74c426fdb

Merge "Migrate default app data on non-FBE devices."

Migrate default app data on non-FBE devices.

When a system app requests "forceDeviceEncrypted" they expect their
default app storage to point at a consistent location regardless of
device FBE support.  So when booting upgraded non-FBE devices, we
may need to migrate any data from CE to DE.  Note that on non-FBE
devices these are just semantic locations with identical protection.

This migration *only* works for non-FBE devices; changing
forceDeviceEncrypted flags on an FBE device always requires a full
data wipe.

Bug: 26668510
Change-Id: I8bd5b8ba882e6bd067c0381041b27c35d6e47788

Merge "openmax: Use unsigned 32bit integer for intra refresh period."

Merge "Enable wildcard in the list of kernel functions" am: fda23631b9 am: ec3ae45d30
am: 2b71580ff6

* commit '2b71580ff6b444f53b9c24f86ad436696720b896':
  Enable wildcard in the list of kernel functions

Merge "Enable wildcard in the list of kernel functions" am: fda23631b9
am: ec3ae45d30

* commit 'ec3ae45d3094c5564f2689ede3eded5926953297':
  Enable wildcard in the list of kernel functions

Merge "Enable wildcard in the list of kernel functions"
am: fda23631b9

* commit 'fda23631b978bec793b67037dc2841a6a15531fb':
  Enable wildcard in the list of kernel functions

Merge "Enable wildcard in the list of kernel functions"

Merge "dumpstate: refine calculated eMMC performance values" am: 088e7586ea am: 917b324e19
am: a960bc95b2

* commit 'a960bc95b2430ba01fc4b04b4527d32e00eeb4b5':
  dumpstate: refine calculated eMMC performance values

Merge "dumpstate: refine calculated eMMC performance values" am: 088e7586ea
am: 917b324e19

* commit '917b324e19121766a9f8112d0fbc4424f172e615':
  dumpstate: refine calculated eMMC performance values

Merge "dumpstate: refine calculated eMMC performance values"
am: 088e7586ea

* commit '088e7586ea7d0103d1b5395172a4594eddbf5b4a':
  dumpstate: refine calculated eMMC performance values

Merge "dumpstate: refine calculated eMMC performance values"

dumpstate: refine calculated eMMC performance values

Correct the raw performance with the average queue depth. Add a
report for I/O per second, and the average queue depth. Round
values up.

Bug: 26966375
Change-Id: I2a8d233872d5e5ff79d433abba3c34d5945701f4

openmax: Use unsigned 32bit integer for intra refresh period.

Bug:26587630
Change-Id: Ib658bcf93093aa4e1b4863f245efbd7b7e6a0389

Merge "Follow constant refactoring in framework."

Merge "system_server BINDER_TYPE_FD driver ashmem accessors" am: b23a388c64 am: 3196c839af
am: 6ff4920cb1

* commit '6ff4920cb12fa4776d27b54e2e4daa27bb78fc10':
  system_server BINDER_TYPE_FD driver ashmem accessors

Merge "system_server BINDER_TYPE_FD driver ashmem accessors" am: b23a388c64
am: 3196c839af

* commit '3196c839af9ea74c59f5860c0d1f6e7e20b7a65e':
  system_server BINDER_TYPE_FD driver ashmem accessors

Merge "system_server BINDER_TYPE_FD driver ashmem accessors"
am: b23a388c64

* commit 'b23a388c64d2fef51482feaaf89c345488f3cabc':
  system_server BINDER_TYPE_FD driver ashmem accessors

Merge "system_server BINDER_TYPE_FD driver ashmem accessors"

Merge "installd: Run dex2oat with relative path to the APK"

installd: Run dex2oat with relative path to the APK

Previous CL (I6aaddffe79c55b9a6560c2d6542cf87abbd4fb11) started
passing the absolute path of an APK to dex2oat. This renders APKs
dexopted while in a temporary folder during install out-of-date when
moved to the final location.

Bug: 26409579
Change-Id: Icccf2d1bb34e2d5f4d98efae3c8f5609784abafd

Enable wildcard in the list of kernel functions

The handling of the -k option is too restrictive
as it stricly checks the names of the kernel functions
after it wrote them in /d/tracing/set_ftrace_filter.

However, a common usage of that function filtering with ftrace
is to use a wildcard character (*) that the ftrace tracer
automatically expands to all the matching kernel functions.

This enables the support of the wildcard (*) character
in the -k option to trace kernel functions.

Change-Id: Ifffae975c20e1c253157a3a6b44a14b4f342b9d0
Signed-off-by: Thomas Buhot <thomas.buhot@intel.com>
Signed-off-by: Zhiquan Liu <zhiquan.liu@intel.com>

Merge "openmax: Add OMX_IndexConfigAndroidIntraRefresh for intra refresh."

Merge "Binder::Status: Add operator<< to help with gtest logging" am: 259eb56171 am: e4fe50ca42
am: 4b41905563

* commit '4b41905563dc35190d60e92c2ea51d210e600caa':
  Binder::Status: Add operator<< to help with gtest logging

Merge "Binder::Status: Add operator<< to help with gtest logging" am: 259eb56171
am: e4fe50ca42

* commit 'e4fe50ca421f0d8508a86f3b03f452d59136f5ae':
  Binder::Status: Add operator<< to help with gtest logging

Merge "Binder::Status: Add operator<< to help with gtest logging"
am: 259eb56171

* commit '259eb56171420f62be325eadc38673d967527ab6':
  Binder::Status: Add operator<< to help with gtest logging

Merge "Binder::Status: Add operator<< to help with gtest logging"

openmax: Add OMX_IndexConfigAndroidIntraRefresh for intra refresh.

Bug:26587630

Change-Id: I153d6d617d2aaf9e216dcdf3dec809b778decebf

Follow constant refactoring in framework.

Bug: 25796509
Change-Id: Ib19aa570fb5fbf26752c5487f1221c9959b1143a

Binder::Status: Add operator<< to help with gtest logging

Change-Id: I22fcf7c6314451701ba947af2d69d4e45b22345a

Merge changes from topic 'rotary-master'

* changes:
  inputflinger: Add support for scaling and true value reporting
  inputflinger: Initial support for rotary encoders.

Merge "Added --skip option to skip services."

Merge "Added --help option to dumpsys."

Merge "Create app images only if dalvik.vm.appimageformat property exists"

Create app images only if dalvik.vm.appimageformat property exists

If the dalvik.vm.appimageformat property does not exist, do not
generate an application image.

Other improvements:
unlink app image if the property is not set, or we are extract only.

Bug: 22858531
Change-Id: I50d8b66787a4e8b6135ad2b00744c7f1ae7c7f53

loader: Test for valid ICD before init dispatch

Test that the Instance object returned by the ICD
is valid before initializing the loader's instance
dispatch table. Not a huge deal since that test
should never fail in production, but flows better.

Change-Id: I7063fb797038e05ea2d0cf9b0794f668b3510cde

Revert "libvulkan: Implement VK_PRESENT_MODE_MAILBOX_KHR"

This reverts commit f333922399393609f5bf022a3467b7541bd1ad0f.
Hologram demo dies with this change.

Added --skip option to skip services.

This option will be used by dumpstate so critical services (like meminfo
and cpuinfo) can be dumped earlier on and then skipped later (when the
rest of the services are dumped).

BUG: 26639621
Change-Id: Ied61431c4d03d7d2eb010edd77872fb7e6bae3bd

Pass app image format property to dex2oat

Enables using lz4 for app images.

Bug: 22858531

Change-Id: Ife2bca8883c2868d30b45e0eec210f96d589068e

Merge "BQ: Flexible resizing"

system_server BINDER_TYPE_FD driver ashmem accessors

check if device matches the ashmem rdev, before calling
ashmem_get_size_region. This eliminates making this call
when associated with other driver file descriptors.

Bug: 26374183
Bug: 26918423
Bug: 26871259
Change-Id: I1f88c2c93ea35a73c8e14125f3d1a6c67fa4f15b

Avoid generating app images for extract only

The extract only app images have no resolved classes and only
contain dex cache arrays.

Bug: 22858531
Change-Id: I4351ef374f66d2882cc23fca14495e63a0e30e89

Added --help option to dumpsys.

When invoked with --help, dumpsys will print:

usage: dumpsys
         To dump all services.
or:
       dumpsys [--help | -l | SERVICE [ARGS]]
         --help: shows this help
         -l: only list services, do not dump them
         SERVICE: dumps only service SERVICE, optionally passing ARGS to it

BUG: 26639621
Change-Id: Ic7f7a5ec80fb3da1dc5a412b07ef2859d31b09d2

Merge "Avoid unlinking app image file"

Merge "Improved how the Shell directories are created."

Improved how the Shell directories are created.

When dumpstate is run for the first time, the
/data/data/com.android.shell/files/bugreports does not exist, which was
crashing dumpstate because the code that added the version.txt entry was
not checking if the zip_writer was NULL.

The crash itself was fixed by adding a sanity check in the functions
that add entries to the zip file, but that only hid the real problem:
it is necessary to create the parent directories before creating the zip
file, otherwise the first run will always generate a .txt file (since
dumpstate falls back to .txt when it cannot create the .zip).

This change also improves how the parent directories are created by
checking if they exist first, rather than always calling mkdir().

BUG: 26949960

Change-Id: I1434be5c36a3fad0b3a2a26c7eaaab03a1228c30

Merge "BQ: Conditionally compile validateConsistencyLocked"

Merge "omx: add extensions for querying audio profiles"

Merge "Fix typo from last commit." am: ae8442856b am: 9d6acce68a
am: 805e3f03dc

* commit '805e3f03dc3848e47b99a45b49ed9bbf65906e00':
  Fix typo from last commit.

Merge "Fix typo from last commit." am: ae8442856b
am: 9d6acce68a

* commit '9d6acce68a257e73463593627bb4b31b140679e7':
  Fix typo from last commit.

Merge "Fix typo from last commit."
am: ae8442856b

* commit 'ae8442856bfdcfd4daf287a880ce7ce4eadaa3e4':
  Fix typo from last commit.

Merge "Fix typo from last commit."

Fix typo from last commit.

Bad commit: https://android-review.googlesource.com/#/c/200449/1

Change-Id: I412e2740951f15cf843bd36a50476070c0cb1ebd

Merge "Added metadata entries to zipped bugreport:"

Merge "Clean up warnings in inputflinger." am: f3606ef25f am: ca5ad095f4
am: 82d47db481

* commit '82d47db4815ab8b49f4c399ba16d2af12926cfe1':
  Clean up warnings in inputflinger.

Merge "Clean up warnings in EGL." am: c312141adc am: d380a770fd
am: f4acff7c49

* commit 'f4acff7c490c6bf8a88661a15ca4bf2536ca4854':
  Clean up warnings in EGL.

Merge "Clean up warnings in inputflinger." am: f3606ef25f
am: ca5ad095f4

* commit 'ca5ad095f428f7c93628db5813e427c47598d102':
  Clean up warnings in inputflinger.

Merge "Clean up warnings in EGL." am: c312141adc
am: d380a770fd

* commit 'd380a770fd22526fee5becd32ce754dcf4a181d4':
  Clean up warnings in EGL.

Merge "Clean up warnings in inputflinger."
am: f3606ef25f

* commit 'f3606ef25fb588283020c4df178076852eecc4e3':
  Clean up warnings in inputflinger.

Merge "Clean up warnings in EGL."
am: c312141adc

* commit 'c312141adc9a5f00b5d13d0980d132d2a8c6f3c9':
  Clean up warnings in EGL.

Merge "Clean up warnings in inputflinger."

Merge "Clean up warnings in EGL."

Clean up warnings in inputflinger.

Bug: http://b/26936282
Change-Id: Ie1fb281425c3b8c4b3e6e88dc5722d51b77ee093

Clean up warnings in EGL.

Bug: http://b/26936282
Change-Id: I059628332d638f080a5236eed62cf0afa1d4d6ba

Added metadata entries to zipped bugreport:

- version.txt: bugreport format version
- main-entry.txt: name of the entry containing the main bugreport (flat file)

Also documented the zip format versions on bugreport-format.txt.

BUG: 26910355
BUG: 26639621

Change-Id: I76b9f6d330c36ad554fae8e691c9ea3ab3c97edd

BQ: Flexible resizing

- Allow the producer to call setMaxDequeuedBufferCount and the
  consumer to call setMaxAcquiredBufferCount when buffers are
  currently dequeued/acquired as long as the new value is not less
  than the number of dequeued/acquired buffers.

Bug 22768206

Change-Id: I599a4027a6ae9cb0a1c0d5ec60cb5e65b86a345b

BQ: Conditionally compile validateConsistencyLocked

- Only enable it on userdebug and eng builds so that it won't slow
  down user builds.

Change-Id: I70933a23d54657fdf5cbc23873da4fbee84b9a26

BQ: Improved buffer/slot tracking

- Explicitly track active buffers and unused slots on top of the
  already existing tracking for free slots and free buffers.

Change-Id: Ife2678678e96f0eb0b3fb21571058378134bd868

Merge "Installd: Introduce otapreopt"