surfaceflinger: Use landscape resolution for default dpi

When HWC doesn't provide DPI values for a display, we pick a default
DPI based on resolution. The intent was that 1080p and higher displays
would get XHIGH density, and lower resolutions would get TV density.
In KK (and possibly forever) we had a bug that we'd always use TV
density. That was fixed in L, but that fix exposed a pre-existing bug
that we always used the display's height in its native orientation,
rather than in landscape orientation. So an 800x1280 tablet like N7v1
started getting XHIGH density instead of the intended TV density.

Bug: 17461633
Change-Id: Ia57fa49e61f36bdda63ce283ef62c9953297222c

Merge "Revert "Fix sockfd leakage in SensorService. Remove sockfd from Looper  when the connection is removed from mActiveConnections."" into lmp-dev

Revert "Fix sockfd leakage in SensorService. Remove sockfd from Looper  when the connection is removed from mActiveConnections."

This reverts commit 5105960d49bfa86efeeea8f87c87aa96b5771c37.

Change-Id: I4d39ded2e50ef8a8e0aac27e438876203385a1f0

Merge "Only mark repeated keys as PASS_TO_USER if the original key was." into lmp-dev

Only mark repeated keys as PASS_TO_USER if the original key was.

Bug: 17629695
Change-Id: Ic569aba4149c6cc321c2cf8aede7c5e8a1196234

Merge "surfaceflinger: Check sideband stream changed to trigger sideband Layer update." into lmp-dev

surfaceflinger: Check sideband stream changed to trigger sideband Layer update.

BUG: 17382907
Change-Id: I65fd037bd25238ae68a3976468ae7fd7ce49c562

Fix sockfd leakage in SensorService. Remove sockfd from Looper
when the connection is removed from mActiveConnections.

Bug: 17472228
Change-Id: Iaabe0968bd2104ef961ff5e8659bf91712e2e1db

Add rate-limited logging for unimplemented es functions.

Bug: 17322013

Change-Id: I7aa5ab2f76cf344cfb75802edbfd44e8e7526ff1

Merge "Include the VPN server IP address in the VPN state file." into lmp-dev

am 60d31a0f: am 3c648e37: Update comment to use correct GPS file for wearable.

* commit '60d31a0fc0724d4538d105a800a1565e5f843130':
  Update comment to use correct GPS file for wearable.

am 3c648e37: Update comment to use correct GPS file for wearable.

* commit '3c648e373a44237c1924fc886daf8418a6e9f4cf':
  Update comment to use correct GPS file for wearable.

Update comment to use correct GPS file for wearable.

Change-Id: I91c4363dbcc6cada8d6e9599d1b15e77febee0e5

Include the VPN server IP address in the VPN state file.

This will be used by LegacyVpnRunner to create a throw route
pointing at it.

Bug: 17462989
Change-Id: I3ebf0cec726dd12b2c57ba5d66775f8c02b25b70

surfaceflinger: add getDisplayStats() method

This is used by media service to schedule video frames at the
proper time, based on precise vsync timings.

Bug: 14659809
Change-Id: I1a90603f3dc09dca9aa4f90a3aa845fab56e0a5e

Merge "Fix typo for android.hardware.audio.output feature in handheld device config" into lmp-dev

Merge "SensorService fixes." into lmp-dev

SensorService fixes.

   i) Initialize sensor_event.flags to zero or wake_up flag before sending dropped flush_complete_events. This will avoid unnecessary acks.
Currently it is not set to anything which may result in garbage values for flags field.
   ii) Change the dump format of SensorEventConnection.
   iii) Add const to method signatures wherever applicable.

Bug id: https://b2.corp.google.com/issues/17498626

Change-Id: Ia736ffb01c366540aab73cdcf596ce852c052593

Add system feature for ConnectionService API support (2/3)

Bug: 17445268

Change-Id: Ib96e50e2558d2744f74419fd022f3f20d578a2de

Fix typo for android.hardware.audio.output feature in handheld device config

Bug: 15318046
Change-Id: I6437a2e87a909a8ef4ce6fcdc56ff06c5599a15a

Rename managed_profiles feature to managed_users

Bug: 17392243
Change-Id: Ib60d1563ecdda9a979196ef51e849be2f9e9f841

Merge "Make dumpstate vibrate immediately." into lmp-dev

Make dumpstate vibrate immediately.

Previously, the vibration was not performed until after stacks
were gathered which takes a long time.  Moved the vibration
to happen earlier so we provide better user feedback for the
three-button salute when collecting a bug report.

Deleted some dead code for playing begin/end sounds.

Improved the timing measurement code to help track down why
bug reports are so slow.  (They take over a minute now which
can cause us to lose valuable diagnostic information.)

Bug: 17474152
Change-Id: Iac73f7993d7dc85196aad96f459b22fd4a710f94

SensorService fixes

  i) Significant Motion multiple clients fix. Make a copy of
  mActiveConnections vector before cleaning up SensorEventConnections
  when one-shot sensors trigger.
  ii) Maintain a mapping between flush_complete_events and
  SensorEventConnections to accurately map flush() API calls and
  corresponding flush_complete_events
  iii) Remove all references to 1_1 and 1_2 HALs.
  iv) Dynamically allocate sensor_event buffers in SensorService main
  threadLoop.

Bug: 17412359
Change-Id: If3c3986197660cafef2d2e0b4dc7582e229cf1c4

Merge "Add feature flags for verified boot and securely remove users." into lmp-dev

Merge "Add support for max res decode" into lmp-dev

Merge "Use time() instead of clock() for timeouts." into lmp-dev

Use time() instead of clock() for timeouts.

The clock() function returns the processor time used by the process. This
is not a good timeout mechanism since the code is suspended most of the
time waiting for the forked process to finish. Replace with the time()
function.

Bug: 17154069
Change-Id: Ib383329f9db269445d71a100553b08e71fddda3f

Update ScreenshotClient to take advantage of rotation in surfaceflinger's screencap api

Change-Id: I836649d9d5cd5958ce34e47f26f4a36d2d05d24c

Merge "Add rotation to surfaceflingers screen cap." into lmp-dev

Merge "Add new system APK locations." into lmp-dev

Add new system APK locations.

Add /system/priv-app and /oem/app paths to system paths.  Also allow
rmdex on system apps, and quietly ignore when it was already removed.

Also relax logging when clearing code cache, since it's optional.

Bug: 17205122
Change-Id: I4fc4d0f63a3596937c4defbd64e1f8a1c563f02d

Add rotation to surfaceflingers screen cap.

+ This is needed so that activity manager does not
  have to do cpu side rotations when capturing recents
  thumbnails.

Change-Id: If998008e675ad01305db8399fd643cf4608b7025

Merge "Add FEATURE_GAMEPAD." into lmp-dev

Merge "Make IPowerManager native conform to .aidl for oneway" into lmp-dev

Add feature flags for verified boot and securely remove users.

Bug: 17405636
Change-Id: I06c0319f7903787910c52d65025b0d77d9e0575f

Add feature android.hardware.audio.output xml file

Bug: 15318046
Change-Id: I49f0881752493ac0bad238b43ce7e1e147119528

Add FEATURE_GAMEPAD.

This allows micro-consoles or other devices to signify that there's a
game controller in the box, even if the user hasn't connected it.

Change-Id: I93cfba8bd1376aa5d1ec772a5b643b974eaa36c5

Make IPowerManager native conform to .aidl for oneway

But provide a temporary escape hatch for AudioFlinger.
This oneway option will be removed as soon as possible.

Bug: 16408906
Change-Id: I20d6da1969ae05b96e72795463470eb4c1f8fbdc

Merge "SensorService flush fixes." into lmp-dev

Merge "IPowerManager: make all binder call one way" into lmp-dev

IPowerManager: make all binder call one way

make all binder call to power manager service one way so that
they don't block calling thread and are processed in the
incoming order.

Bug: 16408906.
Change-Id: I94034df7f0a36d967e085b53fdc37fff7b44757b

am 6de9f394: (-s ours) am 8c967b9b: DO NOT MERGE: Removing android prefix for ECG feature

* commit '6de9f39476c6bbb1d196dbc6af23a27b177c7bf9':
  DO NOT MERGE: Removing android prefix for ECG feature

am 8c967b9b: DO NOT MERGE: Removing android prefix for ECG feature

* commit '8c967b9bea7c80a8bd7d085fa9c9e390495dd1e8':
  DO NOT MERGE: Removing android prefix for ECG feature

DO NOT MERGE: Removing android prefix for ECG feature

Bug: 17272904

Change-Id: Ia476bd8853e098600b681bb769536774596b9345

Merge "Fix undefined behavior (not returning a value from a function with return)." into lmp-dev

Merge "Fix type for temp variable" into lmp-dev

Fix undefined behavior (not returning a value from a function with return).

Bug: 17379740

This function was missing a return statement, even though it declares a return
type. This is undefined behavior, which clang miscompiled into just falling
through to a later function in some cases. Adding the return statement
fixes the undefined behavior (and probably fixes a bug too).

Change-Id: I05c03b6473b831769dc4fa8b5ba43fb4249f7626

SensorService flush fixes.

 i) Fix a possible race condition when adding a pending flush connection
 to mActiveSensors data structure.
 ii) Invalidate flush_complete_events as soon as they are consumed by a
 connection so that they are not used by other connections which are
 registered for the same sensor.

Change-Id: I1491bea8c2081c4aab1feacc60add13b7aa26bb7

Merge "Fix CAS argument type." into lmp-dev

Merge "New keycode definitions for TV remotes." into lmp-dev

Fix CAS argument type.

Gcc apparently doesn't check; clang does.

Bug:17067219
Change-Id: I184084a5063131dbc857c63f97e70e78a05e1a76
(cherry picked from commit 1ddaadb59883df5ca821b4b08fdd41a1720a3b6f)

Revert "Revert "Revert "Revert "Remove incorrect android_atomic_...64 use.""""

After fixing b/16874785.

This reverts commit f010a05c7e9a23b6083294aff4a8183ab01f686e.

Original comment, which actually describes the effect of this:

Change the mExtras field in Binder.h to be a stdatomic.h atomic
value, and replace references to it with proper stdatomic.h calls.
This removes one of a small number of remaining 64 bit
android_atomic references.  It also replaces the erroneously
non-atomic read accesses to mExtras.

It would be better if this used the C++11 <atomic> facility,
but we don't quite have that yet.

Fixes

Bug:16513433

Change-Id: I1645ca5d6f60595bf5d388913665ce4b8780b26d
(cherry picked from commit 3effababf2980d029339522fdc914bdeb913d99b)

New keycode definitions for TV remotes.

Keys which are necessary for supporting world-wide broadcast
standard or used commonly for TV/STB remotes.

Change-Id: I2721ceb7d956c87f1d4106b99f9d21947ae13d47

Fix type for temp variable

Change-Id: I8e2d229da326e5182692f94660336169fb249da7

Add logging for switch state

Bug: 17004602
Change-Id: I15c9feb7b28274183742936c20e34e7fb07e0b70

Add support for max res decode

bug: 16034599
Change-Id: Ia1de1457ca29fe1dd7e55f4bb45da45ea3f6e5b1

Remove obsolete prunedexcache

Bug: 16875245

(cherry picked from commit 5f727f86942a188354c028356f1167794fdc8655)

Change-Id: I06e14c405aa4af295795982c1d236be3cb00e893

Add android.software.live_tv feature

Bug: 17284986
Change-Id: I777d045f61a50f7ea473cb26bb509123c440587a

Merge "GLConsumer: Fix eglTerminate/eglInit edge case." into lmp-dev

Merge "Make clear data stricter." into lmp-dev

Make clear data stricter.

It should nuke everything under the data directory.  PMS will come
around and generate the lib symlink after we're finished.

Bug: 16739202
Change-Id: Ib70af2e1450e3bed6781fb497b9cc67e1e594c49

Set compiler filter flag to true when vmSafeMode is present.

Bug: 12457423
Change-Id: I33345c5cce76681d8a6592ebae8c9f96a9f0f8bd

GLConsumer: Fix eglTerminate/eglInit edge case.

If a display is terminated and then initialized, we can't detect
this using the display itself (it has the same value), but all
EglImages still become invalid for the display. This patch detects
this during image binding and forces creation of a new EglImage.

Bug: 10430249
Change-Id: I75101c50962f21263dca3ec6e241a2e5a3c23dad

Merge "Add vm_safe_mode to dexopt." into lmp-dev

Merge "Pass isa features flag to dex2oat." into lmp-dev

Add vm_safe_mode to dexopt.

The flag enforces interpret-only flag for dex2oat.

Bug: 12457423
Change-Id: Ifdafcc1afa32996577fa44c5682eeb58c79772ac

Pass isa features flag to dex2oat.

Bug: 16716262
Change-Id: Ifbc3a1520bc177aa87855edb60c36346f24c77cd

surfaceflinger: Trace HW vsync enable

This makes it possible to tell whether two HW vsync signals are too
far apart because one was late, or because we turned HW vsync off
briefly between them.

Bug: 17259382
Change-Id: If4fba2a8a6013568349949ce87c5c36f0468d2a2

Merge "Add native flag to reflect new density type" into lmp-dev

Merge "Fix deadlock when killing adb bugreport" into lmp-dev

Add native flag to reflect new density type

Bug:17007265
Change-Id: If0129c4ad222fe28628d6f7cd8ee78fc6424e177

SensorService fixes

     1) Limit maxEventsToWrite depending on the size of the socket
     buffer allocated (Fix for write failure on manta).
     2) Do not call flush() if the sensor is not activated (OEM issue).

Change-Id: I81ea78dbdef3a3773af505ca1c9a5e39492c0b48

Fix deadlock when killing adb bugreport

Leave default signal handler (terminate) for parent process,
add SIG_IGN as signal handler for children and let them
go down when the parent gets SIGPIPE.

Bug: 17109154
Change-Id: Id33db3e97a32f289eb2a9a1a0ca8acbe3dcd285d

Merge "Add Voice Assist key." into lmp-dev

Merge "Update currentConfig variable when active display is changed." into lmp-dev

Fix validation of system paths in installd.

System apps are now installed under their own directory
(system_app_dir/app_dir/app.apk). The new path doesn't pass installd
validation because of obsolete checks which verify that the path does
not contain subdirectories past the system_app_dir.

The CL fixes the validation to accept at most on subdirectory.

Bug: 17109858
Change-Id: I13abb52c0016610ff436f6a26bb6b3b85dc4dfb0

Merge "Allow apk path to contain one subdirectory." into lmp-dev

Add Voice Assist key.

Bug: 16292420
Change-Id: I607fadc2edf673079bfd9f7406e6e45603474957

Update currentConfig variable when active display is changed.

Bug: 17182607
Change-Id: I8631c105a9e0fa402a7d9670717becc9857af935

SensorService fixes.

   i) Add more debug stats (acks needed, acks received)
  ii) Comment out write failure message.

Change-Id: Iac892a66530849a2dd023d6f5628cd070a79537d

am 514e30a9: am c3d1889e: Merge "surfaceflinger: skip composition for empty frames" into klp-modular-dev

* commit '514e30a96cefad109509ce01622fa7f82f3f318c':
  surfaceflinger: skip composition for empty frames

am c3d1889e: Merge "surfaceflinger: skip composition for empty frames" into klp-modular-dev

* commit 'c3d1889e508038efe240ed1974ed377a2e12835c':
  surfaceflinger: skip composition for empty frames

am 1f1ca890: Adding an ECG hardware feature

* commit '1f1ca890d4d82cd8d54d428c3120945a0a308975':
  Adding an ECG hardware feature

Allow apk path to contain one subdirectory.

In the current directory layout this prevented rm_dex and move_dex
commands to validate the apk path and thus cleaning up resources.

Bug: 16888084
Change-Id: Iba579d075a9c6d7de047e7ffef95441498257086

Merge "atrace: avoid unnecessary writes to trace_clock" into lmp-dev

sf: Clear display configs when hot pluggable display is disconnected

Display configs for external displays are stored in the framework
everytime we receive a hot plug to connect. However, since the
configs are not cleared on disconnect, framework will just assume
that the configs are valid. This does not work for use cases when
you connect/disconnect external displays with different resolutions.

e.g. 1080p to 4K and vice-versa

With this change we clear the display configs and repopulate when
we receive a hot plug to connect.

Change-Id: I2eeab186a8d8668a53390a2413b2ce5e044a1845
Acked-by: Tatenda Chipeperekwa <tatendac@codeaurora.org>

atrace: avoid unnecessary writes to trace_clock

Writing to trace_clock erases the trace buffer, even if the value
hasn't changed.  This prevents use of --async_start and --async_dump
to leave background tracing running and dump after an even that
needs debugging, because --async_dump writes to trace_clock and
resets the buffer before it can read it.

Read and parse the current value from trace_clock before writing,
and skip the write if the value isn't changing.

Change-Id: Ia2ec5bb654fb0bd179771b511ff261731ba47dca

Adding an ECG hardware feature

Change-Id: I368772d4cd6b2713fb5180bedc91bf2b5bc015b4

Merge "Destroy eglSurface before recreating." into lmp-dev

am 2eaec5a8: am 1f1ca890: Adding an ECG hardware feature

* commit '2eaec5a833f69b3c460386e38aa6affe5e4b1772':
  Adding an ECG hardware feature

Destroy eglSurface before recreating.

Bug: 16856208

Change-Id: Ifb5c8997afab4bc922356a3542d1f899a2546855

Fix inverse orientation when original is not applied first.

Change-Id: I6f1a11950c77c99800acbf08a672bc1b2310b0b5

Fix camera orientation by swapping horizontal and vertical flips when needed.

Bug: 16637957

Change-Id: I66de597546fdc19e0af9e6150ca20460ab36bf8b

Merge "surfaceflinger: skip composition for empty frames" into klp-modular-dev

surfaceflinger: skip composition for empty frames

By not committing the results of composition for empty frames, we
avoid spitting out series of black frames for virtual displays that
don't have visible layers. We still draw one black frame when going
from having layers to not having any. In particular, this avoids
having a series of empty frames due to re-compositing the primary
display in the period between creating the virtual display and adding
layers to it.

Bug: 16786752
Change-Id: I7e9b2ed2e407d8d49c7af736b447d4c6181b0ad8

Merge "Fix camera orientation by swapping horizontal and vertical flips when needed." into lmp-dev

Merge "Incorporate TransformToDisplayInverse into the crop calculation." into lmp-dev

Incorporate TransformToDisplayInverse into the crop calculation.

Change-Id: Ia9757b3a43d3b8f99df9fef2ed4d11c43b5abdd2

Merge "Within CpuConsumer, use gralloc lockAsync/unlockAsync" into lmp-dev