am da3b1e7f: (-s ours) am 23707911: (-s ours) am 701473cf: (-s ours) Merge "DO NOT MERGE SurfaceFlinger: Stop using IGBC::BufferItem"

* commit 'da3b1e7ff20559ba368a22421c15ffe2cc079386':
  DO NOT MERGE SurfaceFlinger: Stop using IGBC::BufferItem

am c6fdc017: (-s ours) am 355d8118: (-s ours) am 7d89f5b6: (-s ours) Merge "DO NOT MERGE libgui: Prepare for IGBC::BufferItem removal"

* commit 'c6fdc017fe9ea00dfd3817978c169f64b78e6e95':
  DO NOT MERGE libgui: Prepare for IGBC::BufferItem removal

am c795dc00: (-s ours) am 1ca558f7: (-s ours) am f44d13e7: (-s ours) Merge "DO NOT MERGE Add dataSpace to buffer queues; remove old format enums."

* commit 'c795dc007f779103a8fae707e135827d185d943b':
  DO NOT MERGE Add dataSpace to buffer queues; remove old format enums.

am 23707911: (-s ours) am 701473cf: (-s ours) Merge "DO NOT MERGE SurfaceFlinger: Stop using IGBC::BufferItem"

* commit '237079112d245f8415970bd219cb174dae21ac3b':
  DO NOT MERGE SurfaceFlinger: Stop using IGBC::BufferItem

am 355d8118: (-s ours) am 7d89f5b6: (-s ours) Merge "DO NOT MERGE libgui: Prepare for IGBC::BufferItem removal"

* commit '355d81184ea6b423e6fa514797e3529b9062c9cd':
  DO NOT MERGE libgui: Prepare for IGBC::BufferItem removal

am 701473cf: (-s ours) Merge "DO NOT MERGE SurfaceFlinger: Stop using IGBC::BufferItem"

* commit '701473cf9bb1d1da2a0f55d9732d3016e1f33d59':
  DO NOT MERGE SurfaceFlinger: Stop using IGBC::BufferItem

Merge "DO NOT MERGE SurfaceFlinger: Stop using IGBC::BufferItem"

am 7d89f5b6: (-s ours) Merge "DO NOT MERGE libgui: Prepare for IGBC::BufferItem removal"

* commit '7d89f5b6562da8bb316f04f91dd7cc9fce33d7fd':
  DO NOT MERGE libgui: Prepare for IGBC::BufferItem removal

am 1ca558f7: (-s ours) am f44d13e7: (-s ours) Merge "DO NOT MERGE Add dataSpace to buffer queues; remove old format enums."

* commit '1ca558f79861f3c652b5b0726532518aa7c92ef1':
  DO NOT MERGE Add dataSpace to buffer queues; remove old format enums.

Merge "DO NOT MERGE libgui: Prepare for IGBC::BufferItem removal"

am f44d13e7: (-s ours) Merge "DO NOT MERGE Add dataSpace to buffer queues; remove old format enums."

* commit 'f44d13e7e574f181750fa37f439ab50cf05ab3df':
  DO NOT MERGE Add dataSpace to buffer queues; remove old format enums.

Merge "DO NOT MERGE Add dataSpace to buffer queues; remove old format enums."

DO NOT MERGE SurfaceFlinger: Stop using IGBC::BufferItem

Switches all uses of IGraphicBufferConsumer::BufferItem (and
BufferQueue::BufferItem) to the BufferItem in libgui. Depends on
frameworks/native I699ed0a6837076867ca756b28d1ffb2238f7a0d9.

Cherry pick of I187b3a7d05196b6289596afac8fb9a9d4aebff76

Change-Id: I5bc79fb96b6cba6021af64b20890967aa3b7fcbf

DO NOT MERGE libgui: Prepare for IGBC::BufferItem removal

Currently, there are two instances of BufferItem: one inside of
IGraphicBufferConsumer, and a standalone one inside of libgui. They
only differ in the name of one of the fields, and we want to remove
the IGBC version. This changes things so that client code may be
incrementally switched over to the libgui version.

This is a squashed commit containing the following changes:
    I64f495105f56cbf5803cea4aa6b072ea29b70cf5
    I1394e693314429ada93427889f10b7b01c948053
    I9c3bc8037fa9438d4d9080b8afb694219ef2f71f
    I699ed0a6837076867ca756b28d1ffb2238f7a0d9
    Iac8425e1241774304a131da2fb9dec6e82922f13

Change-Id: Ic4d51f5df6dbc70b376d13fceba2335b9bae4f3d

DO NOT MERGE Add dataSpace to buffer queues; remove old format enums.

- Wire up new dataSpace parameter through buffer queue stack
- Update tests to include the parameter
- Switch eglApi to using dataSpace to indicate sRGB gamma/linear
  difference
- Remove RAW_SENSOR in favor of RAW16
- Remove use of sRGB format enums
- Add default dataspace to buffer queue core
- Add query for default dataspace

Cherry pick of I070bd2e7c56506055c419004c29e2e3feac725df

Change-Id: I461952389c18051176c6b75e664f20ad369f5760

am 4b9d0b9b: am f3201498: Merge "dumpstate: add O_CLOEXEC"

* commit '4b9d0b9b676afec086bf579015b2f74623b357fd':
  dumpstate: add O_CLOEXEC

am f3201498: Merge "dumpstate: add O_CLOEXEC"

* commit 'f320149886ba8ab8849c8b8f3df256e3789f3905':
  dumpstate: add O_CLOEXEC

Merge "dumpstate: add O_CLOEXEC"

dumpstate: add O_CLOEXEC

Add O_CLOEXEC to various file descriptor calls, to avoid
leaking file descriptors to dumpstate's child processes.

Bug: 18342188
Change-Id: I74c47a98dfddc29c618067ad53d879b98ed1d87a

libgui: Fix bad merge in Surface.h

Fixes a bad merge from AOSP.

Change-Id: Idb69b34787e8f4e76fb64130573168e5fde6a87c

am 663eaf42: am ad36432d: Merge "Surface: Permit an IProducerListener on connect"

* commit '663eaf422d325768808b416d66ecd4648bb14799':
  Surface: Permit an IProducerListener on connect

am ad36432d: Merge "Surface: Permit an IProducerListener on connect"

* commit 'ad36432de81c4c88d726680b7c52e8357df98455':
  Surface: Permit an IProducerListener on connect

am 42f0ce10: am c1930630: Merge "libgui: Only allow one return from useFenceSync"

* commit '42f0ce10757e9a1dbf9c997fd0db0eee93c15984':
  libgui: Only allow one return from useFenceSync

am c1930630: Merge "libgui: Only allow one return from useFenceSync"

* commit 'c193063050e6be193838527fb8bc7a4588bfdc22':
  libgui: Only allow one return from useFenceSync

Merge "Surface: Permit an IProducerListener on connect"

am 5bebe7e1: am 55466b48: Merge "GLConsumer: Fix unsigned subtraction during crop"

* commit '5bebe7e14d645b02602f0d8a1e19db2f8ebec35d':
  GLConsumer: Fix unsigned subtraction during crop

am 55466b48: Merge "GLConsumer: Fix unsigned subtraction during crop"

* commit '55466b48504f9a904c54f3cae4fc1a43dec73a6c':
  GLConsumer: Fix unsigned subtraction during crop

am e31591ba: am f87d1bcd: Merge "Update maxNumber to be smaller."

* commit 'e31591baf0a9d9e269e388fdc707057b11009c95':
  Update maxNumber to be smaller.

Merge "libgui: Only allow one return from useFenceSync"

libgui: Only allow one return from useFenceSync

An #ifdef was causing multiple returns in this function, which was
triggering an unreachable code error on some build targets. This
adds an #else to ensure that only one return is exposed in any given
build.

Cherry pick of I9ed7c3e769bc9094ad31d810944227d1d8302886

Change-Id: Ibddc5706159d168031d1805fcce3eedc068296cf

am 7ab0d37d: (-s ours) am 3d5efd55: Merge "libgui: Enable -Weverything and -Werror"

* commit '7ab0d37dbfd9a8777e7c67043c523c6419ed170b':
  libgui: Enable -Weverything and -Werror

am f87d1bcd: Merge "Update maxNumber to be smaller."

* commit 'f87d1bcd65f1889b78a8f5db6526e91216eaeb5d':
  Update maxNumber to be smaller.

Merge "GLConsumer: Fix unsigned subtraction during crop"

am 3d5efd55: Merge "libgui: Enable -Weverything and -Werror"

* commit '3d5efd553ea433e81464b0b817b3455716884bb0':
  libgui: Enable -Weverything and -Werror

Merge "Update maxNumber to be smaller."

Merge "libgui: Enable -Weverything and -Werror"

Merge "libgui: Plumb attach/detach through Surface"

Surface: Permit an IProducerListener on connect

This change allows clients of Surface to provide an IProducerListener
callback object to Surface::connect, which will be passed down to the
underlying IGraphicBufferProducer.

Cherry pick of I5ea5229bf3a329bf02c6bd20e7247039c75d136b

Change-Id: I6f8f52c72654e4cee649721383819bafe378f964

GLConsumer: Fix unsigned subtraction during crop

Since some variables had been switched from signed to unsigned, there
was a section of code that was guaranteed to be incorrect because it
effectively did 'if (a < b) { c = a - b; }'. This change fixes it.

Cherry pick of I9cdd6c9a0179801addebb5d6dc1fbaddf8f53c62

Bug: 19346631
Change-Id: Id13a46f74c9ae7278463ce22b586f4dc21b5e453

Update maxNumber to be smaller.

There shouldn't be more than 4096 fds (probably signficantly smaller) and
there shouldn't be more than 4096 ints.

Cherry pick of I3a3e50ee3078a4710e9737114e65afc923ed0573

Bug: 18076253
Change-Id: I82a883572b401f115d252dcd3d00aa7252b49b0e

libgui: Enable -Weverything and -Werror

Enables -Weverything and -Werror, with just a few exceptions for
warnings we can't (or shouldn't need to) work around.

Cherry pick of I034abec27bf4020d84af60d7acc1939c59986dd6 plus a
couple of minor changes to CpuConsumer.cpp to make it work with a
prior change:
    Uncomment CC_LOGV on line 46
    Change C-style cast to static_cast on line 71

Change-Id: Iaec610477ea0122317b0578fb74caf2383d4cf08

libgui: Plumb attach/detach through CpuConsumer

Adds CpuConsumer::{detachNextBuffer,attachAndReleaseBuffer}, which
can be used to more carefully manage the ownership of GraphicBuffers.

Bug: 19628705
Change-Id: Ia7a7e30da6d81eb2367241998f14988db0afc3bf

Revert "libgui: Plumb attach/detach through CpuConsumer"

This reverts commit 43078e2827211139f0614719c920f3c5bff6c3fa.

Change-Id: Ifef774d9922bb90450384e810e4767b56cc29de3

libgui: Plumb attach/detach through Surface

Exposes the attachBuffer and detachNextBuffer calls from
IGraphicBufferProducer to the public Surface interface. Also moves
the version of connect that takes a producer callback from protected
to public.

Bug: 19628705
Change-Id: I9ebc3013c4d9c84c4e8ef150c00e03f8af80319e

libgui: Plumb attach/detach through CpuConsumer

Adds CpuConsumer::{detachNextBuffer,attachAndReleaseBuffer}, which
can be used to more carefully manage the ownership of GraphicBuffers.

Bug: 19628705
Change-Id: Ia7aa1ac59c2f768f2d8a0f35ad23067936a7427c

libgui: Remove IGBC::BufferItem

Removes IGraphicBufferConsumer::BufferItem. Depends on the
following changes:
    I187b3a7d05196b6289596afac8fb9a9d4aebff76
    I0ddd38df37500cfd6b21d1e768ed14e39c5cd9fc

Change-Id: Id1fa56d092188f2cb712768d5d2fc6a9027fb73c

Merge "openmax: add OMX_IndexConfigPriority."

Merge remote-tracking branch 'goog/mirror-m-wireless-internal-release'

am 5e44f932: am b6b81d6a: Merge "[MIPS] Fix mips32r1 build"

* commit '5e44f932a28ebecaefed942224efa0b0d4bf36c5':
  [MIPS] Fix mips32r1 build

am b6b81d6a: Merge "[MIPS] Fix mips32r1 build"

* commit 'b6b81d6acd6949a731308666ac211864f824ab50':
  [MIPS] Fix mips32r1 build

Merge "[MIPS] Fix mips32r1 build"

Merge "libgui: Fix infinite recursion"

libgui: Fix infinite recursion

One of the overloads of BufferQueueConsumer::acquireBuffer was
calling itself infinitely instead of calling the other overload.
This fixes that issue.

Bug: 19733425
Change-Id: Iac8425e1241774304a131da2fb9dec6e82922f13

Merge "SurfaceFlinger: Stop using IGBC::BufferItem"

Merge "libgui: Prepare for IGBC::BufferItem removal"

Merge "libgui: Add a BufferItem overload to IGBC::acquire"

Merge "libgui: Add BufferItem(IGBC::BufferItem)"

Merge "libgui: Add a union to BufferItem for refactoring"

SurfaceFlinger: Stop using IGBC::BufferItem

Switches all uses of IGraphicBufferConsumer::BufferItem (and
BufferQueue::BufferItem) to the BufferItem in libgui. Depends on
frameworks/native I699ed0a6837076867ca756b28d1ffb2238f7a0d9.

Change-Id: I187b3a7d05196b6289596afac8fb9a9d4aebff76

libgui: Prepare for IGBC::BufferItem removal

Switches some dependencies from IGraphicBufferConsumer::BufferItem to
android::BufferItem and adds some methods to facilitate incrementally
changing client code to do the same.

Change-Id: I699ed0a6837076867ca756b28d1ffb2238f7a0d9

openmax: add OMX_IndexConfigPriority.

Bug: 19620911
Change-Id: I776c1537e5fd88052609cab66d60905f7e30400a

libgui: Add a BufferItem overload to IGBC::acquire

Adds an overload of IGraphicBufferConsumer::acquireBuffer which takes
an android::BufferItem instead of an IGBC::BufferItem.

Change-Id: I9c3bc8037fa9438d4d9080b8afb694219ef2f71f

libgui: Add BufferItem(IGBC::BufferItem)

Add a constructor for building a BufferItem from an
IGraphicBufferConsumer::BufferItem.

Change-Id: I1394e693314429ada93427889f10b7b01c948053

libgui: Add a union to BufferItem for refactoring

Currently, there are two instances of BufferItem: one inside of
IGraphicBufferConsumer, and a standalone one inside of libgui. They
only differ in the name of one of the fields, so this change modifies
the one inside of libgui to have a union of both names so that the
one inside of IGBC can eventually be refactored away.

Change-Id: I64f495105f56cbf5803cea4aa6b072ea29b70cf5

am 53cab935: am 1fc747d2: am 5a25a63d: Merge "Installd: Pass debuggable flag"

* commit '53cab935d54b6ca013c6a236e09d9160a6aa7f54':
  Installd: Pass debuggable flag

am 105aa28e: am b275b4dc: am 3949f664: Merge "SurfaceFlinger: Fix null pointer exception"

* commit '105aa28eeff9d31eebc7645aecb7630918a09b2a':
  SurfaceFlinger: Fix null pointer exception

am 1fc747d2: am 5a25a63d: Merge "Installd: Pass debuggable flag"

* commit '1fc747d20c1a390e1de3d482d2eafde6dc7e6236':
  Installd: Pass debuggable flag

am 5a25a63d: Merge "Installd: Pass debuggable flag"

* commit '5a25a63de2693177b15817fb19b684ca7bfec582':
  Installd: Pass debuggable flag

Merge "Installd: Pass debuggable flag"

Installd: Pass debuggable flag

Pass the debuggable flag from the package manager to dex2oat.

Change-Id: Id17ec72babe2ee88713a0d274eff86508de30666

am b275b4dc: am 3949f664: Merge "SurfaceFlinger: Fix null pointer exception"

* commit 'b275b4dc3e5dcbae1e171bd51165a93f16332ef4':
  SurfaceFlinger: Fix null pointer exception

am 3949f664: Merge "SurfaceFlinger: Fix null pointer exception"

* commit '3949f664c71ad1269ee0c921d176e3b4484015cc':
  SurfaceFlinger: Fix null pointer exception

Merge "SurfaceFlinger: Fix null pointer exception"

SurfaceFlinger: Fix null pointer exception

Fixes a null pointer exception in getActiveConfig

Bug: 19416124
Change-Id: I45a01f67affb63a348866ce42a6013c5eb71fe8f

Merge "Add dataSpace to buffer queues; remove old format enums."

Merge "Surface: Permit an IProducerListener on connect"

Merge "drm: add event for resource manager."

Merge "SurfaceFlinger: Add --static-screen stats"

am f11b58f2: am fcf1689c: am 5fec7915: Merge "service_manager: reorder permission checks for find"

* commit 'f11b58f296039a0bf90336e4c38c014f11542092':
  service_manager: reorder permission checks for find

SurfaceFlinger: Add --static-screen stats

Adds a --static-screen option to dumpsys SurfaceFlinger, which
displays screen-on time broken down by the time between the prior
frame and the current frame. An example dump looks like this:

$ adb shell dumpsys SurfaceFlinger --static-screen
Static screen stats:
  < 1 frames: 12.235 s (3.5%)
  < 2 frames: 29.898 s (8.7%)
  < 3 frames: 15.370 s (4.4%)
  < 4 frames: 13.103 s (3.8%)
  < 5 frames: 15.780 s (4.6%)
  < 6 frames: 2.022 s (0.6%)
  < 7 frames: 0.201 s (0.1%)
  7+ frames: 256.887 s (74.4%)

The buckets are exclusive, so '< 3 frames' covers the interval
[2, 3) frames

Bug: 19543586
Change-Id: I3253a54c23995d25e96016997acedd0775956b60

am fcf1689c: am 5fec7915: Merge "service_manager: reorder permission checks for find"

* commit 'fcf1689c1a1654cb3b480356a1ba3ebf198f2943':
  service_manager: reorder permission checks for find

am 5fec7915: Merge "service_manager: reorder permission checks for find"

* commit '5fec79156d749322dcc0376da1d6cf4d9ee9ccf4':
  service_manager: reorder permission checks for find

Merge "service_manager: reorder permission checks for find"

service_manager: reorder permission checks for find

Reorder the find permission checks. This avoids generating misleading
SELinux denials when a service doesn't exist, or when a service is
prohibited to isolated apps.

The original reason for structuring the code this way is explained
in https://android-review.googlesource.com/#/c/100530/4/cmds/servicemanager/service_manager.c@172
The concern at the time was to avoid leaking a situation where
a caller could probe for the existance of a service. This turns out
to be unnecessary. The same return value is used for both a
permission denied and a service not found. The only side effect
is the generation of an SELinux audit log, which likely won't be
accessible to the calling application.

Change-Id: I9760e1821ed16102fa5f9bec07f8c34944565be9

drm: add event for resource manager.

Bug: 19265536
Change-Id: I524962f84aefae28a8ffd411d1bb6521bedb02a7

GLConsumer: Fix unsigned subtraction during crop

Since some variables had been switched from signed to unsigned, there
was a section of code that was guaranteed to be incorrect because it
effectively did 'if (a < b) { c = a - b; }'. This change fixes it.

Bug: 19346631
Change-Id: I9cdd6c9a0179801addebb5d6dc1fbaddf8f53c62

Add dataSpace to buffer queues; remove old format enums.

- Wire up new dataSpace parameter through buffer queue stack
- Update tests to include the parameter
- Switch eglApi to using dataSpace to indicate sRGB gamma/linear
  difference
- Remove RAW_SENSOR in favor of RAW16
- Remove use of sRGB format enums
- Add default dataspace to buffer queue core
- Add query for default dataspace

Change-Id: I070bd2e7c56506055c419004c29e2e3feac725df

Surface: Permit an IProducerListener on connect

This change allows clients of Surface to provide an IProducerListener
callback object to Surface::connect, which will be passed down to the
underlying IGraphicBufferProducer.

Change-Id: I5ea5229bf3a329bf02c6bd20e7247039c75d136b

resolved conflicts for merge of 04e13acd to master

Change-Id: I9bbc9907a5cbde8b374398493bc0dc6b056ea734

am 69a141e0: am 9d68ed0e: Merge "Add timeout for dump_file."

* commit '69a141e07fab2672cf2cc23dcc124c46995fbf98':
  Add timeout for dump_file.

am 9d68ed0e: Merge "Add timeout for dump_file."

* commit '9d68ed0ecd3a101c13a163cfe730b13b1564b442':
  Add timeout for dump_file.

Merge "Add timeout for dump_file."

resolved conflicts for merge of d6c8ae9c to master

Change-Id: I3cbb6ed0958b4328aca1798c7fe6080c31555a3a

am 2f655dae: am bb67beca: Merge "dumpstate: add pstore read"

* commit '2f655daeea66bf5d0e065b45a7abd515cfeeaf97':
  dumpstate: add pstore read

am bb67beca: Merge "dumpstate: add pstore read"

* commit 'bb67beca35e7d570ede193a316a92022fbc2f810':
  dumpstate: add pstore read

Merge "dumpstate: add pstore read"

dumpstate: add pstore read

Call logcat -L to pull any LAST Android logs. If the kernel is not
configured for PSTORE and PSTORE_PMSG, the hopes are the empty
content will pressure vendors to slice up and configure support.

Change-Id: I5fddfa1e0f59f24fccc30b257ba68af8a8cf8640

Add timeout for dump_file.

It turns out dump_file is used on a number of /proc and system files.
In one case, the read of a file stalled and caused a bugreport to
hang forever. It's still possible if there is a kernel bug that this
could stall forever, but less likely.

Also, change the return type of nanotime to uint64_t.

Testing:
- Created a named fifo and verified that dump_file fails with a timeout.
- Created a large /data/anr/traces.txt to verify that large files still
  dump properly and that the additional NONBLOCK parameter doesn't cause
  a problem.
- Created a dummy /data/tombstones/tombstone_00 to verify that the
  dump of these files still works.
- Compared a dump using the old dumpstate to the new dumpstate to verify
  nothing obviously different.

Bug: 19117030
Change-Id: I0d3dd27583c853cdaccd2fd278748cb5f9ccd4fb

am beb89e11: am 6f57b269: am 56f8c6c4: Merge "Error compiling waitforvsync.cpp"

* commit 'beb89e11ca777c92b7e2cc06c25f47d5a75ee7ad':
  Error compiling waitforvsync.cpp

am 6f57b269: am 56f8c6c4: Merge "Error compiling waitforvsync.cpp"

* commit '6f57b269f9ae779a0e85919cdc7a54bfb383e402':
  Error compiling waitforvsync.cpp