Merge "Allocate buffers locally in systems without SurfaceFlinger" am: afcf267cbd
am: 225a2901af

* commit '225a2901af2fb08e1cc47103565bc35eedb45752':
  Allocate buffers locally in systems without SurfaceFlinger

Merge "Allocate buffers locally in systems without SurfaceFlinger"
am: afcf267cbd

* commit 'afcf267cbd86316cd98c96bc82400479c965d8d6':
  Allocate buffers locally in systems without SurfaceFlinger

Merge "Allocate buffers locally in systems without SurfaceFlinger"

Merge "Otapreopt: Add path for patching boot image" into nyc-dev
am: 5f15b52447

* commit '5f15b52447c547e4cf6172671c3e7139952adc6b':
  Otapreopt: Add path for patching boot image

Merge "Otapreopt: Add path for patching boot image" into nyc-dev

Otapreopt: Add path for patching boot image

Add a path to use a preopted boot image during A/B OTA. This uses
a hard-coded check for the default preopted boot.art location.

Bug: 25612095
Change-Id: Ia6b954d5aca6b3455ac9feacdf9739fd964417e8

installd: fix path to code_cache.
am: 2f6d9e2fec

* commit '2f6d9e2fec8ab76ce35d5d6dee0a7838fc10b66c':
  installd: fix path to code_cache.

installd: fix path to code_cache.

Now exists in user_de, not user.

bug: 27216576
Change-Id: I9d50d9584de7b349fc9dfc829b690c20dd02eccd

Merge "omx: Add AllocateNativeHandleParams structure" into nyc-dev
am: 7514282174

* commit '7514282174a08133f79363ade70c7bc8dc94623f':
  omx: Add AllocateNativeHandleParams structure

Merge "omx: Add AllocateNativeHandleParams structure" into nyc-dev

Allocate buffers locally in systems without SurfaceFlinger

In Brillo systems, we have no applications and configure
SELinux policy statically.  In this model, we do not need
a special service to allocate buffers, since any process
that needs buffers can be appropriately marked at build time.

Bug: 26936651
Test: A test program on brilloemulator can allocate buffers for
      the camera with this patch.

Change-Id: Ifcdb7a4b878f9a26f4899c7b75a12447dcdb0ac9

Merge "libbinder: add getters for PersistableBundle keys" am: 4f321d1629
am: 0c81a4a5fb

* commit '0c81a4a5fbed3c6b8351d567c1737491c11582f3':
  libbinder: add getters for PersistableBundle keys

Merge "libbinder: add getters for PersistableBundle keys"
am: 4f321d1629

* commit '4f321d1629e2b848b4e9a5304ab4a9262bc81569':
  libbinder: add getters for PersistableBundle keys

Merge "libbinder: add getters for PersistableBundle keys"

Merge "Fix -Wgnu-designator warnigns in cmds/flatland/Main.cpp" am: f4475023e7
am: 8b8d7741e6

* commit '8b8d7741e68ea10648369e1b80f8697413c22bfc':
  Fix -Wgnu-designator warnigns in cmds/flatland/Main.cpp

Merge "Fix -Wgnu-designator warnigns in cmds/flatland/Main.cpp"
am: f4475023e7

* commit 'f4475023e728db1456a72b0e9a1292e11fb38245':
  Fix -Wgnu-designator warnigns in cmds/flatland/Main.cpp

Merge "Fix -Wgnu-designator warnigns in cmds/flatland/Main.cpp"

Merge "Fix Surface slot caching" into nyc-dev
am: 3130422a1d

* commit '3130422a1d7a7d2a5d528cd161e860fb5f66d353':
  Fix Surface slot caching

Merge "Fix Surface slot caching" into nyc-dev

Fix -Wgnu-designator warnigns in cmds/flatland/Main.cpp

Bug: http://b/27068438
Change-Id: I3064950f761411fe5271c6259867ad437c71e66d

Merge "Replace AppOpsManager locking with std::lock_guard" am: d3a9228d1d
am: 3f40743c4f

* commit '3f40743c4f2cc86c505238cf5f597efca41277de':
  Replace AppOpsManager locking with std::lock_guard

Merge "Replace AppOpsManager locking with std::lock_guard"
am: d3a9228d1d

* commit 'd3a9228d1d2388f21b4e8013cf2fad7b4c91ace7':
  Replace AppOpsManager locking with std::lock_guard

Merge "Replace AppOpsManager locking with std::lock_guard"

dumpstate: remove length modifier on assignment suppressed scanf item
am: 60292e5c83

* commit '60292e5c8324475c5afbd0a6703b6b3ae2c94b1a':
  dumpstate: remove length modifier on assignment suppressed scanf item

dumpstate: remove -std=gnu99
am: 861dd225fc

* commit '861dd225fcd53f3e8d10469600648226c6e887cf':
  dumpstate: remove -std=gnu99

dumpstate: remove length modifier on assignment suppressed scanf item

gcc throws warning for using the two together as there is no point specifying
storage size if the item is not to be stored.  With -Werror enabled in this
module, such warning breaks the build under gcc compiling.

Change-Id: I6478c7cdb39f4709a63089191cc4ff541564cea4

dumpstate: remove -std=gnu99

-std=gnu99 no longer has effect now that dumpstate is C++.

The flag also break build if target platform is configured to default
gcc compiler since gcc throws warning for having -std=gnu99 when compiling
C++ and we have -Werror for the module.

Reproduce:
mm LOCAL_CLANG=false
...
error: command line option '-std=gnu99' is valid for C/ObjC but not for C++ [-Werror]

Change-Id: If1c272868ec4626873e7d91342b1fea51a15c52e

Merge changes Id64bb14c,I47fb1475,Iefd17b62,Ib0e963e8,I236183d5 into nyc-dev
am: deee0d4d19

* commit 'deee0d4d1983f495bd57e2506d0f96993c5255f6':
  loader: Add extension before CreateInstance chain
  vkinfo: Add debug command line option
  vkinfo: Add validation layers
  loader: Fix enumerate device extension properties
  loader: Fix enumerate device layer properties

Merge changes Id64bb14c,I47fb1475,Iefd17b62,Ib0e963e8,I236183d5 into nyc-dev

* changes:
  loader: Add extension before CreateInstance chain
  vkinfo: Add debug command line option
  vkinfo: Add validation layers
  loader: Fix enumerate device extension properties
  loader: Fix enumerate device layer properties

Merge "Add missing liblog dependency" am: 39f1abac3c
am: 3b3e298583

* commit '3b3e2985832ffb5d3fb3b6af4dc8be66f8fd01b0':
  Add missing liblog dependency

Merge "Add missing liblog dependency"
am: 39f1abac3c

* commit '39f1abac3c21e2f5a4a450b21d1c52ba5f099ddb':
  Add missing liblog dependency

Merge "Add missing liblog dependency"

Merge "Don\'t rely on AppOpsManager in systems without applications" am: 46c32a1b1c
am: c61c5c05ef

* commit 'c61c5c05ef2c64d6a0dca6be8eab7842143529b6':
  Don't rely on AppOpsManager in systems without applications

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

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

Add missing liblog dependency

Bug: http://b/27171986
Change-Id: Ifbe01c6b325e7198a95ea865f557b0c9acb65809

Merge "Don\'t rely on AppOpsManager in systems without applications"
am: 46c32a1b1c

* commit '46c32a1b1c895ec7630686aa8228785479f8fe92':
  Don't rely on AppOpsManager in systems without applications

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

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

Replace AppOpsManager locking with std::lock_guard

Change-Id: Ice4c101659837cea594194cfb335f9f3f6837f38

Merge "Don't rely on AppOpsManager in systems without applications"

Merge "Fix race in AppOpsManager client library"

libbinder: add getters for PersistableBundle keys

Add getters for the keys in each map stored in
PersistableBundle. This will allow users of
PersistableBundle to iterate over entries of
a specific type in PersistableBundle.

Also, add "using" statements to PersistableBundle
for std containers to reduce verbosity.

BUG: 27171235
Change-Id: Idd4b3bf713071ca8d8d023f1f83bdaf3ccbdd46f
TEST: libbinder compiles successfully.

loader: Add extension before CreateInstance chain

A developer can cause the loader to enable VK_EXT_debug_report
extension by setting a specific property. However, that check
was happening after the instance chain was called and thus
no layers saw the extension enabled.
Also fix use of AddExtensionToCreateInfo to always allocate
and insert VK_EXT_debug_report. Otherwise cannot distinguish
between the failure case and the extension is already present
case.

Change-Id: Id64bb14cb2086e94190625b4364329d634afda70

vkinfo: Add debug command line option

Really handy for loader debug to switch between
waiting and not waiting for a debugger to attach.

Change-Id: I47fb1475d8aa62b8f4b2b7e6a156793f108a83ed

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

omx: Add AllocateNativeHandleParams structure

This reuses the EnableAndroidNativeBuffersParams structure

Bug: 26782004
Change-Id: I6d71c6686bbb3001dfe0e9d8ccb92c17bf43c67f

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

vkinfo: Add validation layers

Change-Id: Iefd17b622f0d91219b81bd1df09ffeb3397f9553

loader: Fix enumerate device extension properties

Same as device layer property issue.
Loader needs to provide aggregate data rather
than send to instance chain.

Change-Id: Ib0e963e8d1780e4870110c556051da921f94353b

loader: Fix enumerate device layer properties

Loader was calling down the chain when is needs to
respond with aggregate data. This patch fixes
https://buganizer.corp.google.com/u/0/issues/26982782

Change-Id: I236183d5d781a6efc3d84db4a51376b8cc29664f

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

Fix Surface slot caching

- Now that it's possible to resize a BufferQueue while buffers are
  dequeued/acquired, it's no longer correct for Surface to clear its
  cache when the BufferQueue is resized since it must keep at least
  the currently dequeued buffers.
- Add an onSlotsFreed callback to IProducerListener so that producers
  that wish to be notified about buffers being freed can do so. Note
  that this isn't currently used in Surface.
- Review and fixup all the places where the producer/consumer
  listeners for freed buffers should be called.

Change-Id: I4ab0c92bc69b75a3c072ddf5d74d78f941dba4c8

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