Merge "Fix race in AppOpsManager client library" am: 712fe62ab2
am: f2936d7bec

* commit 'f2936d7bec9ce8e56f1bd4df052376dea192c809':
  Fix race in AppOpsManager client library

Merge "Fix race in AppOpsManager client library"
am: 712fe62ab2

* commit '712fe62ab282b3897fc1ee98e35b12d95663be52':
  Fix race in AppOpsManager client library

Merge "Fix race in AppOpsManager client library"

Changed permission of temporary bugreport and dumpstate log files to Shell.
am: 6fe9db67f6

* commit '6fe9db67f6c92d5fbf87d371da5cca412f672630':
  Changed permission of temporary bugreport and dumpstate log files to Shell.

Changed permission of temporary bugreport and dumpstate log files to Shell.

BUG: 27164100

Change-Id: I569cbe4854fad44c95c9bf3e2efa9c3c93317003

Merge "Split bugreport identifier into id and pid." into nyc-dev
am: 421c1c07e7

* commit '421c1c07e73ef91a8fb887ad0045f37335722b2e':
  Split bugreport identifier into id and pid.

Merge "Split bugreport identifier into id and pid." into nyc-dev

Merge "libbinder: Allow readByteVector to take uint8_t" am: dda39f2a22
am: f226203232

* commit 'f2262032320c31a87659df81b0b1db76ea3310dd':
  libbinder: Allow readByteVector to take uint8_t

Merge "libbinder: Allow readByteVector to take uint8_t"
am: dda39f2a22

* commit 'dda39f2a2229ea74cc4738daa1c6a1c758284d7d':
  libbinder: Allow readByteVector to take uint8_t

Merge "libbinder: Allow readByteVector to take uint8_t"

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

* commit 'da8ca93b430cdb9f1b559e065e194d6701d30291':
  SF: Prevent non-sync transactions from syncing

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
am: c30e3e7c09

* commit 'c30e3e7c095603c3c48b3c2691db80fec570c731':
  Send broadcasts as Shell.

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
am: 5d9d711a16

* commit '5d9d711a16d7560f4be62cad2413dbcb5feddfde':
  BQ: Fix iterator use

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

Split dumpsys into critical and non-critical sections.
am: fdf52d3697

* commit 'fdf52d3697aa0396bd9d8883892937b99a6772a0':
  Split dumpsys into critical and non-critical sections.

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

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

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

loader: Implement new layer initialization
am: a90ce61c1a

* commit 'a90ce61c1a4c0eae79b117f2137e9fe7ed41aa6d':
  loader: Implement new layer initialization

loader: Add GetName and GetDescription for logs
am: eff631189b

* commit 'eff631189b135a9579a7f8510f9b1ccadd6fb4f9':
  loader: Add GetName and GetDescription for logs

loader: Layer init structures
am: 1dd712aefd

* commit '1dd712aefdc771297823b24f26d768cd8bd14d49':
  loader: Layer init structures

loader: Only use driver\'s DebugReport if it has one
am: 6fecdd563f

* commit '6fecdd563f3bf94dadedc78512a1b28c08c67e07':
  loader: Only use driver's DebugReport if it has one

tools: Need to specify pQueuePriorities
am: ca472ab8e4

* commit 'ca472ab8e43efe92db30660db9997117e4b00ab9':
  tools: Need to specify pQueuePriorities

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
am: 3d92c53d99

* commit '3d92c53d99866ddd28c9ff20f610074c82f70d43':
  libvulkan: Support R5G6B5_UNORM_PACK16 as a swapchain format

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

Merge "dumpstate: Continued build breakage" am: 16072e7e17
am: c01e8dc4a6

* commit 'c01e8dc4a687ae3445f68b8d005e2b29a84f6e7f':
  dumpstate: Continued build breakage

Merge "dumpstate: Continued build breakage"
am: 16072e7e17

* commit '16072e7e1717bcf1ea8be99eb7af90974dbbeeb7':
  dumpstate: Continued build breakage

Merge "dumpstate: Continued build breakage"

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

* commit '148a1b45ff2c205f6aa9c46c7f039d29d877a35c':
  libvulkan: Un-revert and fix VK_PRESENT_MODE_MAILBOX_KHR

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

* commit '45ba96e44c0d7d5dac6e86f0bfbfc44b4f2618ac':
  vknulldrv: Report physical device limits

dumpstate: Continued build breakage

- drop assignment length in scanf
- drop -std=gnu99 as invalid for C++

Change-Id: I924d2561069b3c289cb9deb475f732c46bb5a67e

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

* commit 'c9456b532d83edafc997d60b2c31d67bb77ac3f9':
  libvulkan: Initialize VkSurfacePropertiesKHR::supportedCompositeAlpha

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
am: e59624db83

* commit 'e59624db8323445569b2a9d04aea694298f9a2eb':
  vkinfo: Fix bytes -> megabytes conversion

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.
am: 1b0225a479

* commit '1b0225a4792d92ff67ac8a0aa79352fad40bd805':
  Fixed format.

Fixed format.

Change-Id: If0322e3dcf56005541aa08e0ab91dae10d2d385d

Fixed C++ syntax.
am: 60869c9e1e

* commit '60869c9e1e91170d487715fc30b795d786630c54':
  Fixed C++ syntax.

Fixed C++ syntax.

Change-Id: I9702fdede589dd37cb498bbf63c0773816a7e3b8

dumpstate: report process times
am: a297c3258f

* commit 'a297c3258f6f7ea52cc2dcc42d62f85fda12a163':
  dumpstate: report process times

libbinder: Allow readByteVector to take uint8_t

Change-Id: I2f0d1f3b4c04f04e7a866e9a4cf76c899b4b4980
Test: AIDL integration tests pass
Bug: 27078230

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

Split bugreport identifier into id and pid.

Although 'pid' is more useful when diagnosing problems with the
bugreport workflow, it could be confusing to the end user. Hence, a
sequential id (started at 1 after a reboot) would be more useful.

BUG: 27076108
Change-Id: I6fd7ea8c703e8040034af183ef5cd35adfbcb4ef

resolve merge conflicts of c1c11d20de to nyc-dev-plus-aosp.

Change-Id: Ia9520e7ac0d13b440ab5a83991585c03eccf00a0

Merge changes I6b499f53,If821ed21
am: c658aee7d6

* commit 'c658aee7d6dc56b1b3b1722412029830e966ba71':
  dumpstate: report process times
  Add option to #define _DUMPSTATE_DRY_RUN_ to skip the actual dumps.

Merge changes I6b499f53,If821ed21

* changes:
  dumpstate: report process times
  Add option to #define _DUMPSTATE_DRY_RUN_ to skip the actual dumps.

dumpstate: report process times

- 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

Add option to #define _DUMPSTATE_DRY_RUN_ to skip the actual dumps.

(cherry picked from commit 93d705b6d742e8f00167ff2384e35c2205a6f9da)

Such option is useful when debugging dumpstate itself, since it will
finish much sooner and its output will be much smaller.

Change-Id: If821ed21715461bf82eea0b2be4b926239ad69da

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"

Fix race in AppOpsManager client library

If the service stays down for more than 10 seconds, client threads
will time out attempting to obtain a reference, and never release
the mutex.  This causes deadlock on the next attempt to speak
with the AppOpsManager.

Test: System without AppOpsManager service no longer hangs in camera
      code.

Change-Id: I5a8ac2f0f35345f72c3d2a45b9d1fc5a78d28a51

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