am afaf14b9: Use descriptive aliases instead of HWC_NUM_DISPLAY_TYPES

* commit 'afaf14b9fbfe8943d845e2f01e8a401ad7a4d854':
  Use descriptive aliases instead of HWC_NUM_DISPLAY_TYPES

am 13f01cbd: Pass sp<Fence>s around instead of file descriptors

* commit '13f01cbdbd34779a234bc674df79e23672fd5c0b':
  Pass sp<Fence>s around instead of file descriptors

Use descriptive aliases instead of HWC_NUM_DISPLAY_TYPES

Bug: 8384764
Change-Id: Ia2563fab19dbc6e4a95bed03445e609334841cca

Pass sp<Fence>s around instead of file descriptors

Change-Id: Iac70584a2207940730e8f803a543e4e9a4000c47

am 8d8c7597: Merge "Release virtual display buffer immediately after HWC set" into jb-mr2-dev

* commit '8d8c75976171d194ca8f94727b22e3bf7676b9dd':
  Release virtual display buffer immediately after HWC set

Merge "Release virtual display buffer immediately after HWC set" into jb-mr2-dev

am ed985574: make sure screenshot are in a format supported by Bitmap.java

* commit 'ed985574148a938bc3af24442eead313cc62521c':
  make sure screenshot are in a format supported by Bitmap.java

make sure screenshot are in a format supported by Bitmap.java

Change-Id: I0fb9cc4088f9c1fd27e6c017b0a7c5617adb4660

am 7133f863: Merge "improved CallStack a bit" into jb-mr2-dev

* commit '7133f8638e105548f6cca66a0ba9a6951993b7de':
  improved CallStack a bit

am 50210b9a: fix a typo that broke all screenshots

* commit '50210b9a8d19cb90fc283d8d99e46cd34ac17d2e':
  fix a typo that broke all screenshots

Merge "improved CallStack a bit" into jb-mr2-dev

fix a typo that broke all screenshots

Bug: 8450197

Change-Id: I5b986cc0ff9c5e689d06a51ba68ab537d03d1f3d

am 0188eeff: Merge "only use glReadPixels() when needed when taking screenshots" into jb-mr2-dev

* commit '0188eeff1fecc3813ddb236be3e2b7a9ad156482':
  only use glReadPixels() when needed when taking screenshots

Merge "only use glReadPixels() when needed when taking screenshots" into jb-mr2-dev

improved CallStack a bit

- added a ctor that updates and dumps the stack immediately
- added a "logtag" parameter to dump()

Change-Id: Ie51c256071d282591752243bdb4f68cf9ff8829d

am 504e1f26: Merge "initialize planeAlpha (to 0xFF) for the FB_TARGET" into jb-mr2-dev

* commit '504e1f2638f6a5d7ad7ce66a89eaaaba15b452db':
  initialize planeAlpha (to 0xFF) for the FB_TARGET

Merge "initialize planeAlpha (to 0xFF) for the FB_TARGET" into jb-mr2-dev

initialize planeAlpha (to 0xFF) for the FB_TARGET

Bug: 8422625
Change-Id: I7b72456c2e01ed92e675cc1bfa0cfd0d2ce5064e

am 43122e7e: atrace: use creat instead of truncate

* commit '43122e7e672eb170334a4467dd41cf4bd545bae5':
  atrace: use creat instead of truncate

atrace: use creat instead of truncate

Change-Id: Ie25c704f33c419c0c542249ae2841393862df0ab

only use glReadPixels() when needed when taking screenshots

some drivers don't support this yet, so we use a system
property to enable the glReadPixels "workaround" for them:

  ro.bq.gpu_to_cpu_unsupported=1

Change-Id: I74d6a3a8f0cee8d5a507b72c760cf247e39195e0

am aaebffd5: Merge "CpuConsumer: Properly track acquired buffers" into jb-mr2-dev

* commit 'aaebffd52f1bfa3d239bba0d60b699c01c6d791c':
  CpuConsumer: Properly track acquired buffers

Merge "CpuConsumer: Properly track acquired buffers" into jb-mr2-dev

am ac006439: Merge "Revert "Revert "Change SurfaceControl setPosition to take floats""" into jb-mr2-dev

* commit 'ac006439bb635cd99d8dcbefe95fa680e21519b5':
  Revert "Revert "Change SurfaceControl setPosition to take floats""

Merge "Revert "Revert "Change SurfaceControl setPosition to take floats""" into jb-mr2-dev

Revert "Revert "Change SurfaceControl setPosition to take floats""

This reverts commit bbb57f3331c7182399ed82e9c4f93a965677dde3

Change-Id: Ib8f039f530d6b8175b5205c6c0008b135d4001a2

am 7c1a487b: fix build. update test to new API

* commit '7c1a487ba8c0a3b591a77e2ddcb33ef9bdfaff64':
  fix build. update test to new API

fix build. update test to new API

Change-Id: I31ca84c85edff3e2ff90c554769e058110f27d49

am ff7c2342: Merge "rework how we take screenshots for a CPU consumer" into jb-mr2-dev

* commit 'ff7c2342e246696defa121a984d9731d000adeaf':
  rework how we take screenshots for a CPU consumer

Merge "rework how we take screenshots for a CPU consumer" into jb-mr2-dev

am a6c1ab52: Merge "Move screenshots near beginning of dumpstate." into jb-mr2-dev

* commit 'a6c1ab52e30fc63c8822e1f0cc6c5aea6328a69b':
  Move screenshots near beginning of dumpstate.

Merge "Move screenshots near beginning of dumpstate." into jb-mr2-dev

Release virtual display buffer immediately after HWC set

Previously we only queued a virtual display buffer to the sink when
the next frame was about to be displayed. This may delay the "last"
frame of an animation indefinitely. Now we queue the buffer as soon as
HWC set() returns and gives us the release fence.

Bug: 8384764
Change-Id: I3844a188e0f6ef6ff28f3e11477cfa063a924b1a

am 30041bd3: Merge "Updated MediaDrm API based on review input." into jb-mr2-dev

* commit '30041bd3a487c3bfe45a5b802a0d3443cfcc0c51':
  Updated MediaDrm API based on review input.

Merge "Updated MediaDrm API based on review input." into jb-mr2-dev

Reconcile with jb-mr2-release - do not merge

Change-Id: I9b1d766e6519a720cee8327d1ec71d9f69dd7387

merge in jb-mr2-release history after reset to jb-mr2-dev

rework how we take screenshots for a CPU consumer

We're not using IMemoryHeap as a transport anymore,
instead we're providing a CpuConsumer and use the
IGraphicBufferProducer version of the screenshot API.

However, some GPU drivers don't support properly
a GPU to CPU path, to work around this, we use a
temporary BufferQueue on the server side for the
GL rendering, and we use glReadPixels into the
CpuConsumer (we're now using a CPU to CPU path
which is always supported).

Currently this "wrapping" is always performed,
but it can be bypassed on devices that support
the GPU to CPU path.

This also addresses a DoS attack vector on
SurfaceFlinger, where an application could
consume all of SF's filedescriptors by creating
a lot of screenshots in a row.

Bug: 8390553

Change-Id: I9e81514c2a7711b9bb393f74305be7d2abe08f1c

am 9cd766a5: disable RefBase consistency checks (NDEBUG)

* commit '9cd766a517caeb58669722fad7fdf623a8c1b83c':
  disable RefBase consistency checks (NDEBUG)

disable RefBase consistency checks (NDEBUG)

Bug: 8328715
Change-Id: Ib57646ff909fd8744610f37f3b50d90d884dff31

am 4e37ddff: Fix a crasher with RefBase debugging and vectors of wp<>

* commit '4e37ddff43784a5a784beb4b62ea3f3136e1634b':
  Fix a crasher with RefBase debugging and vectors of wp<>

Fix a crasher with RefBase debugging and vectors of wp<>

background:
we have some code to fix-up the IDs of references when
using RefBase's DEBUG_REFS when those refs are managed by
arrays wp<> or sp<> (this is because wp<> / sp<> don't have
a trivial ctor when DEBUG_REFS is enabled, and Vector
treats them as trivial for obvious performance reasons)

this is complicated by the fact that we don't want to have
to recompile everything when enabling DEBUG_REFs (i.e.: the
Vector code cannot know wheter it's enabled or not for its
template stuff).

problem:
there was a bug in the fix-up code for wp<> which was trying
to access the weakref_impl from the RefBase* however, this was
moronic since RefBase could have been destroyed if there wasn't
any more strong refs -- and this happned. Instead we need to get
the weakref_impl directly from the wp<>

Change-Id: Ie16e334204205fdbff142acb9faff8479a78450b

am ca987c87: Merge "A few tweaks to RefBase debugging" into jb-mr2-dev

* commit 'ca987c877549b0d5472e91ded98f3ba6fea8a96d':
  A few tweaks to RefBase debugging

Merge "A few tweaks to RefBase debugging" into jb-mr2-dev

Move screenshots near beginning of dumpstate.

Bug: 8420215
Change-Id: I6adffd1182ed8f99b8d11d43785a4eb4a388ceb5

am ef64b75a: Don\'t log an error on HWC1.1+ devices with no FB HAL

* commit 'ef64b75a936ffd194673dbce6766995f86b42ba9':
  Don't log an error on HWC1.1+ devices with no FB HAL

Don't log an error on HWC1.1+ devices with no FB HAL

Change-Id: I015e5a1a1f926181e51f82525f69beff71bd70cd

am fae23b87: Merge changes I61ae54f3,I57cb668e,I7a3f1e1a,Id28a2f9b into jb-mr2-dev

* commit 'fae23b8757a6e1b70997db28a2eaf34f9ddc9b84':
  Add BufferQueueInterposer and use it for virtual displays
  Add DisplaySurface abstraction
  Fix argument types in IGraphicBufferProducer methods
  Minor cleanups/fixes before virtual display refactoring

Merge changes I61ae54f3,I57cb668e,I7a3f1e1a,Id28a2f9b into jb-mr2-dev

* changes:
  Add BufferQueueInterposer and use it for virtual displays
  Add DisplaySurface abstraction
  Fix argument types in IGraphicBufferProducer methods
  Minor cleanups/fixes before virtual display refactoring

merge in jb-mr2-release history after reset to jb-mr2-dev

A few tweaks to RefBase debugging

- stacks are now saved in /data/debug which must be
  created and writable by the user.

- removed "always fatal" DEBUG_REFS option, it wasn't
  really needed.

- DEBUG_REFS_ENABLED_BY_DEFAULT is not the default anymore
  (usually people want to target which refs they're tracking)

Change-Id: I37fae72e9dacde6ce1fa8f7dbe2bc01b1a1b95e5

am 79fb99d0: Merge "atrace: add support for tracing kernel functions" into jb-mr2-dev

* commit '79fb99d0dea4d301d1bf25273c6af4907ca124bc':
  atrace: add support for tracing kernel functions

Merge "atrace: add support for tracing kernel functions" into jb-mr2-dev

Add BufferQueueInterposer and use it for virtual displays

BufferQueueInterposer allows a client to tap into a
IGraphicBufferProducer-based buffer queue, and modify buffers as they
pass from producer to consumer. VirtualDisplaySurface uses this to
layer HWC composition on top of GLES composition before passing the
buffer to the virtual display consumer.

Bug: 8384764
Change-Id: I61ae54f3d90de6a35f4f02bb5e64e7cc88e1cb83

Add DisplaySurface abstraction

DisplayDevice now has a DisplaySurface instead of using
FramebufferSurface directly. FramebufferSurface implements
DisplaySurface, and so does the new VirtualDisplaySurface class.
DisplayDevice now always has a surface, not just for virtual displays.

In this change VirtualDisplaySurface is just a stub; buffers still go
directly from GLES to the final consumer.

Bug: 8384764
Change-Id: I57cb668edbc6c37bfebda90b9222d435bf589f37

Fix argument types in IGraphicBufferProducer methods

Bug: 8384764
Change-Id: I7a3f1e1a0584a70af04f9eafef900505389d2202

Minor cleanups/fixes before virtual display refactoring

None of these should change behavior, except for removing some
incorrect log messages when using a virtual display.

- HWComposer::getAndResetReleaseFenceFd() checks the HWC version, so
  no need to do that in the DisplayDevice::onSwapBuffersCompleted().
  However, it should check that mFramebufferSurface is not NULL like
  it is for virtual displays.
- Comment that FramebufferSurface::dump() overrides the non-virtual
  ConsumerBase::dump(), and fix it so the right thing happens
  regardless of the static type of the pointer/reference the callee
  has. FramebufferSurface::dump() could be removed right now, but I'd
  need to bring it back in a later change.
- Use the right enum for validating display type ids.
- Don't try to send hotplug events for virtual displays.
- Mark virtual displays as connected so HWComposer::prepare() doesn't
  think something is wrong when it gets a non-NULL layer list.
- Remove unused FramebufferSurface methods.

Bug: 8384764
Change-Id: Id28a2f9be86b45f4bb7915fdf7752157035f4294

atrace: add support for tracing kernel functions

This change adds support for tracing specific kernel function calls using the
function_graph tracer.  It adds a '-k' option to atrace that accepts a comma
separated list of kernel function names for which tracing will be enabled.

Change-Id: I872b2f1d474b8ebb904053853fc8cf8c0a98089c

merge in jb-mr2-release history after reset to jb-mr2-dev

am ce7e2781: Merge "Add Vector::resize()" into jb-mr2-dev

* commit 'ce7e278151d268f3cf55bcecd05f6be8c7e82e0f':
  Add Vector::resize()

am 041a0752: don\'t allow screenshots without the READ_FRAMEBUFFER permission

* commit '041a075262ef5fc886e46fd4eec3dd79ee2e60c0':
  don't allow screenshots without the READ_FRAMEBUFFER permission

Merge "Add Vector::resize()" into jb-mr2-dev

don't allow screenshots without the READ_FRAMEBUFFER permission

the recent screenshot rework allowed the older screenshot
interface to work without that permission

Change-Id: I6c4743f4591c81106e3b823d55a055f7b4907de1

am 7670d3cb: Merge "get rid of purgatory and fix QueuesToWindowComposer query" into jb-mr2-dev

* commit '7670d3cb2b5d38c60135f34a2446e1ae809d1b32':
  get rid of purgatory and fix QueuesToWindowComposer query

Merge "get rid of purgatory and fix QueuesToWindowComposer query" into jb-mr2-dev

am a493be58: don\'t capture hidden layers in screenshots

* commit 'a493be5825d15f6a94d1afb5910db075a2a7abc1':
  don't capture hidden layers in screenshots

don't capture hidden layers in screenshots

Bug: 8389956
Change-Id: I9ed836395258732c743c6fd44092bd01020dde13

Add Vector::resize()

Bug: 8384764
Change-Id: Icee83d389f3e555eba7d419b64c8d52a9aa21b8b

am 1791eefd: fix a couple race-conditions in RefBase::promote()

* commit '1791eefd69e07a7d8a311ee8a298bbd2de77f046':
  fix a couple race-conditions in RefBase::promote()

get rid of purgatory and fix QueuesToWindowComposer query

the purgatory list wasn't needed anymore; in fact it had no effect as
buffer life-time management is now handled by the BufferQueue.

For QueuesToWindowComposer we keep a list of wp<> on the IBinder
for IGraphicBufferProducers we hand over to clients so we can
easily check if an IGraphicBufferProducer is ours. We clean-up the
list when our IGraphicBufferProducer are destroyed.

Bug: 8349142
Change-Id: I1aa06652ade8c72d0004a3f5e6c3d6e8a82fc2ae

fix a couple race-conditions in RefBase::promote()

Bug: 8390295
Change-Id: I7a48e3bf5b213cc1da2b8e844c6bb37ee24cb047

am 7212ff29: am 20e154f1: Merge "Second try at adding a compatibility symbol for the MemoryBase constructor."

* commit '7212ff29c6f4e4cd192fee6f072e80b36d8a728b':
  Second try at adding a compatibility symbol for the MemoryBase constructor.

am 5b00af24: am 7212ff29: am 20e154f1: Merge "Second try at adding a compatibility symbol for the MemoryBase constructor."

* commit '5b00af2435d67ccf806c918f6482949870fd993b':
  Second try at adding a compatibility symbol for the MemoryBase constructor.

am 7212ff29: am 20e154f1: Merge "Second try at adding a compatibility symbol for the MemoryBase constructor."

* commit '7212ff29c6f4e4cd192fee6f072e80b36d8a728b':
  Second try at adding a compatibility symbol for the MemoryBase constructor.

am a11713ce: Merge "Increase screenshot timeout." into jb-mr2-dev

* commit 'a11713ce3e81d7838092168fad110bea3212a9f3':
  Increase screenshot timeout.

Merge "Increase screenshot timeout." into jb-mr2-dev

Increase screenshot timeout.

Devices with large displays take longer than 5 seconds to write.

Change-Id: If74bc4a3cec366ae28eca549bce28ef212e47d8f

am 801e632a: am 6250f6cb: am d8151306: Merge "Revert "Add a compatibility symbol for the MemoryBase constructor.""

* commit '801e632a1913819a7bf8542615caa80018c0f266':
  Revert "Add a compatibility symbol for the MemoryBase constructor."

am 6250f6cb: am d8151306: Merge "Revert "Add a compatibility symbol for the MemoryBase constructor.""

* commit '6250f6cb2be039be150de6c836f8595384838342':
  Revert "Add a compatibility symbol for the MemoryBase constructor."

am d77d4e0e: Merge "Support broadcast when dumpstate finishes." into jb-mr2-dev

* commit 'd77d4e0edcb0d0f99dce2acea5278875be0b02a0':
  Support broadcast when dumpstate finishes.

Merge "Support broadcast when dumpstate finishes." into jb-mr2-dev

merge in jb-mr2-release history after reset to jb-mr2-dev

am c47f0988: Revert "Add a compatibility symbol for the MemoryBase constructor."

* commit 'c47f0988a36c3032a3c8850c4df3a704dd59e468':
  Revert "Add a compatibility symbol for the MemoryBase constructor."

Revert "Add a compatibility symbol for the MemoryBase constructor."

Cherry pick from AOSP to fix the build.
See https://android-review.googlesource.com/#/c/53690/

This reverts commit 7f974ff2f317ffbf9256579109f000ec876c3037

Change-Id: I1b123f446d0ff41e11ff81d4ef5f1472b59cac93

am 5ca39ea5: am 9b57ad73: am 8facd1b2: Merge "Add a compatibility symbol for the MemoryBase constructor."

* commit '5ca39ea57a4bfa04e90dd2aafc31c8f20bd8bf1c':
  Add a compatibility symbol for the MemoryBase constructor.

am 9b57ad73: am 8facd1b2: Merge "Add a compatibility symbol for the MemoryBase constructor."

* commit '9b57ad737bceba9f8d4f5be303c70fa72be00eaf':
  Add a compatibility symbol for the MemoryBase constructor.

am 20e154f1: Merge "Second try at adding a compatibility symbol for the MemoryBase constructor."

* commit '20e154f16f315d7ae0b3204db0004d19a8b0bc48':
  Second try at adding a compatibility symbol for the MemoryBase constructor.

Merge "Second try at adding a compatibility symbol for the MemoryBase constructor."

Second try at adding a compatibility symbol for the MemoryBase constructor.

Third-party libraries are currently trying to use the
MemoryBase constructor but failing because we fixed the
definition of ssize_t. This is a stop-gap for users of
this private API until we can get them fixed.

Bug: 8253769
Change-Id: Ie7c86f45fa39cb53539cab0ffe8585a585656714

am d8151306: Merge "Revert "Add a compatibility symbol for the MemoryBase constructor.""

* commit 'd8151306f539f8c2f3b66c1fe06be9566ba8aca3':
  Revert "Add a compatibility symbol for the MemoryBase constructor."

Merge "Revert "Add a compatibility symbol for the MemoryBase constructor.""

Revert "Add a compatibility symbol for the MemoryBase constructor."

This reverts commit 7f974ff2f317ffbf9256579109f000ec876c3037

Change-Id: I28ebfc5ac88bd025471b613aa7958d23749e6991

am 8facd1b2: Merge "Add a compatibility symbol for the MemoryBase constructor."

* commit '8facd1b2ea02796b575f123b420590d2b8354910':
  Add a compatibility symbol for the MemoryBase constructor.

Merge "Add a compatibility symbol for the MemoryBase constructor."

Add a compatibility symbol for the MemoryBase constructor.

Third-party libraries are currently trying to use the
MemoryBase constructor but failing because we fixed the
definition of ssize_t. This is a stop-gap for users of
this private API until we can get them fixed.

Bug: 8253769
Change-Id: I8a19770f3252d88ee87023fde625cc6289924b0d

Support broadcast when dumpstate finishes.

Add flag that sends BUGREPORT_FINISHED broadcast when dumpstate
is finished, including the resulting filesystem paths.  Also reduces
directory creation to 0770.

Bug: 7005318
Change-Id: Id4c6b699a56f8acd859b7ab73368500e1a8f3c67

size IMemoryHeap properly for screenshots

since we're using glReadPixels(), we only need to use
the width (as opposed to the stride) of the source
screenshot.

Bug: 8374664
Change-Id: I145c80f4fff5444df7c77c4f52e70a7203caddbd

am 6a531717: size IMemoryHeap properly for screenshots

* commit '6a531717cddf1d30be0946884d21c27e5b6b390c':
  size IMemoryHeap properly for screenshots

size IMemoryHeap properly for screenshots

since we're using glReadPixels(), we only need to use
the width (as opposed to the stride) of the source
screenshot.

Bug: 8374664
Change-Id: I145c80f4fff5444df7c77c4f52e70a7203caddbd

am 35ffa6a8: Surface can now be created only from an IGraphicBufferProducer

* commit '35ffa6a868f1aa650c90956a4129bb70f780fc99':
  Surface can now be created only from an IGraphicBufferProducer