Sanity check IMemory access versus underlying mmap am: 94b0d4e3ab am: ef6908e2b3 am: 97f49e50de am: 84f488f354 am: ebdad60d6b am: fc598c54d9 am: f9e5e80bc1 am: 15364d3ec0
am: 049c5df3e5

* commit '049c5df3e59a2d56c47deddc6ec20bf1eba4f50e':
  Sanity check IMemory access versus underlying mmap

Sanity check IMemory access versus underlying mmap am: 94b0d4e3ab am: ef6908e2b3 am: 97f49e50de am: 84f488f354 am: ebdad60d6b am: fc598c54d9 am: f9e5e80bc1
am: 15364d3ec0

* commit '15364d3ec0fda521cd796fc28a2a8d6a68e2cf95':
  Sanity check IMemory access versus underlying mmap

Sanity check IMemory access versus underlying mmap am: 94b0d4e3ab am: ef6908e2b3 am: 97f49e50de am: 84f488f354 am: ebdad60d6b am: fc598c54d9
am: f9e5e80bc1

* commit 'f9e5e80bc1fe9255fa65b7185514d86beee5952a':
  Sanity check IMemory access versus underlying mmap

Sanity check IMemory access versus underlying mmap am: 94b0d4e3ab am: ef6908e2b3 am: 97f49e50de am: 84f488f354 am: ebdad60d6b
am: fc598c54d9

* commit 'fc598c54d945c48ade547a52a01e2e704df1618b':
  Sanity check IMemory access versus underlying mmap

Sanity check IMemory access versus underlying mmap am: 94b0d4e3ab am: ef6908e2b3 am: 97f49e50de am: 84f488f354
am: ebdad60d6b

* commit 'ebdad60d6b0560493e50092c30888d3a7b5cc8c9':
  Sanity check IMemory access versus underlying mmap

Sanity check IMemory access versus underlying mmap am: 94b0d4e3ab am: ef6908e2b3 am: 97f49e50de
am: 84f488f354

* commit '84f488f354c33a7ece30cb569c85cd91f0259066':
  Sanity check IMemory access versus underlying mmap

Sanity check IMemory access versus underlying mmap am: 94b0d4e3ab am: ef6908e2b3
am: 97f49e50de

* commit '97f49e50ded651f1ddcf8a9981a7d97fe2816ad7':
  Sanity check IMemory access versus underlying mmap

Sanity check IMemory access versus underlying mmap am: 94b0d4e3ab
am: ef6908e2b3

* commit 'ef6908e2b3e6ee6514620acc338b458ade7c3640':
  Sanity check IMemory access versus underlying mmap

Sanity check IMemory access versus underlying mmap
am: 94b0d4e3ab

* commit '94b0d4e3ab023cfa03a7a4e85f3e09d3743da715':
  Sanity check IMemory access versus underlying mmap

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.

Now exists in user_de, not user.

bug: 27216576
Change-Id: I9d50d9584de7b349fc9dfc829b690c20dd02eccd

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

Sanity check IMemory access versus underlying mmap

Bug 26877992

Change-Id: Ibbf4b1061e4675e4e96bc944a865b53eaf6984fe

Merge "Fix Surface slot caching" into nyc-dev

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

* 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

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.

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

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

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.

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

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

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