Merge "Handled failure of execvp and increases am timeout." into nyc-dev
am: 9b02ecf

* commit '9b02ecfe308302eae98d50fa165b1faae695f3df':
  Handled failure of execvp and increases am timeout.

Merge "Handled failure of execvp and increases am timeout." into nyc-dev

Handled failure of execvp and increases am timeout.

The run_command_always() call fork and runs the command in the child
code, and user to have the above logic:

    if (pid == 0) {
        // boiling code
        execvp(command, (char**) args);
        // execvp's result will be handled after waitpid_with_timeout() below...
        _exit(-1); // ...but it doesn't hurt to force exit, just in case
    }

Recently, the _exit(-1) line was removed and there's been reports of
dumpstate hanging after, because it tries to wait for pid 0 to
finish (and in some cases it even tries to kill it).

Also increased the timeout when running am to send broadcasts.

BUG: 27804637
BUG: 27809534
Change-Id: Ia129253e367e5886d305947e56a643fd660a33bd

Merge "Add final crop implementation" into nyc-dev
am: 41371bf

* commit '41371bfa0dad4a4332a8cc30b066d95546e4c4fc':
  Add final crop implementation

Merge "Add final crop implementation" into nyc-dev

Merge "Delete the entire profile directory when the app is uninstalled" into nyc-dev
am: 5b81754

* commit '5b817549fcea6f58bbeaaffbf654ddd8751ebbdc':
  Delete the entire profile directory when the app is uninstalled

Merge "Delete the entire profile directory when the app is uninstalled" into nyc-dev

Merge "sensorservice: Strengthen compiler warnings." into nyc-dev
am: 2376b4e

* commit '2376b4e1cb9a24d0d7dcd1d57458261610818ef5':
  sensorservice: Strengthen compiler warnings.

Merge "sensorservice: Strengthen compiler warnings." into nyc-dev

Merge "media: add HDRStaticInfo and DescribeHDRStaticInfoParams" into nyc-dev
am: b706c05

* commit 'b706c054f9aa3366c6c41f2ddaa985fb5ba6a00a':
  media: add HDRStaticInfo and DescribeHDRStaticInfoParams

Merge "media: add HDRStaticInfo and DescribeHDRStaticInfoParams" into nyc-dev

Merge "dumpsys: make timeout value an argument" into nyc-dev
am: ca9b292

* commit 'ca9b29213820ac399f9e2b9aac827377fa7e6622':
  dumpsys: make timeout value an argument

Merge "dumpsys: make timeout value an argument" into nyc-dev

dumpsys: make timeout value an argument

Also rework the args parsing using getopt_long.

Bug: 27788406
Change-Id: I60fd8e614616334d47b8de574f3c81a121a6ac0a
Signed-off-by: Thierry Strudel <tstrudel@google.com>

Merge "MediaResourceMonitor: Change argument type from string to int" into nyc-dev
am: a8b00c1

* commit 'a8b00c1c666f0d6d280ffa5bfed4aa09a4dab6ad':
  MediaResourceMonitor: Change argument type from string to int

Merge "MediaResourceMonitor: Change argument type from string to int" into nyc-dev

MediaResourceMonitor: Change argument type from string to int

Bug: 27529962
Change-Id: Ic4228aa74c9730134c61a2668932c6ab3b785afc

Merge "Install non-working WSI functions if not enabled" into nyc-dev
am: 2d4cd92

* commit '2d4cd92a72209c2bda90d0ff7e44e117999f5d1d':
  Install non-working WSI functions if not enabled

Merge "Install non-working WSI functions if not enabled" into nyc-dev

sensorservice: Strengthen compiler warnings.

We turn on Wall and Werror for our sensorservice module, and
turn on Wextra for sensorservice and libsensorservice.

This doesn't trigger any new warnings, so this is just a
future-proofing move.

Change-Id: I6574632cd83866e2925697664c33ab9537a4993b

Merge "Add debug dump if about to segfault" into nyc-dev
am: dd264bd

* commit 'dd264bdb06d6caa1e94fcd1d2a760d34d1c0cb89':
  Add debug dump if about to segfault

Merge "Add debug dump if about to segfault" into nyc-dev

Install non-working WSI functions if not enabled

b/27537020
These changes make it so that WSI calls dead end if the
application hasn't explicitly enabled them.

Change-Id: I44f95244ec7fd17855daf1c46e9cb6ab4932840f

Delete the entire profile directory when the app is uninstalled

We were deleting just the files themselves. We need to delete the
directory as well.

Bug: 27696217
Change-Id: I202a52c711a2132ba10b926f4d0e9ec953f25b43

Add debug dump if about to segfault

Bug 27674961

Change-Id: I82242d2c1b7cc9646d233bab896f718652e9ce2c

Add final crop implementation

Bug 26559810

Change-Id: Idaccd13cd625c92d18665ddecebdbb266ea365f3

Fixed typo.
am: 26ae723

* commit '26ae72392e76e4acc82842f4a53ad41bcbc6811e':
  Fixed typo.

Fixed typo.

BUG: 27419521

Change-Id: Ie9f0b24876353e532f4ebe2ff4935938c3c7a781

Merge "Don\'t use su to when calling am or dumpsys." into nyc-dev
am: 7363baf

* commit '7363bafd0a94aa98a267edbbcb27abc9ef72f1dd':
  Don't use su to when calling am or dumpsys.

Merge "Don't use su to when calling am or dumpsys." into nyc-dev

Merge "Capture systrace buffers." into nyc-dev
am: 7f72f4e

* commit '7f72f4e577c8f9f88096353a60c3a2b43006f7d9':
  Capture systrace buffers.

Merge "Capture systrace buffers." into nyc-dev

Capture systrace buffers.

When tracing is enabled (/sys/kernel/debug/tracing/tracing_on = 1),
dumpstate will run 'atrace --async_dump' and redirect its output to a
systrace.txt entry in the zip file.

BUG: 27419521
Change-Id: Ia6de46a691b25febac31331fe0aa8701c9a84ebb

Merge "Fix issue #27252896: Security Vulnerability -- weak binder" into nyc-dev
am: b49576d

* commit 'b49576d5241859a3cf03c9f34b8030ad25d3806b':
  Fix issue #27252896: Security Vulnerability -- weak binder

Merge "Fix issue #27252896: Security Vulnerability -- weak binder" into nyc-dev

Fix issue #27252896: Security Vulnerability -- weak binder

Sending transaction to freed BBinder through weak handle
can cause use of a (mostly) freed object.  We need to try to
safely promote to a strong reference first.

Change-Id: Ic9c6940fa824980472e94ed2dfeca52a6b0fd342

Merge "Removed HardwarePropertiesManagerService helper." into nyc-dev
am: 79315fb

* commit '79315fb51fedbef2c7f27df6fe1bb270ea6a9ff1':
  Removed HardwarePropertiesManagerService helper.

Merge "Removed HardwarePropertiesManagerService helper." into nyc-dev

Merge "media: add and clarify OMX float numerical type" into nyc-dev
am: 891b1ce

* commit '891b1ce1decbeb80081f22513925c14c9c2015ec':
  media: add and clarify OMX float numerical type

Merge "DO NOT MERGE BQ: fix some uninitialized variables" into klp-dev am: b4eac74 am: dc5d0f4  -s ours am: 12a01ba  -s ours am: 6fdc372 am: fdd9851  -s ours am: e98a9a9 am: c240884 am: 1899ffd am: d53600c  -s ours
am: fac5847

* commit 'fac5847d76aa2762ae07cd8741c155de55b2a206':
  DO NOT MERGE BQ: fix some uninitialized variables

BQ: fix some uninitialized variables am: 93c617f am: f16b857 am: 1ed2469 am: 46da938 am: b2428c7 am: ce3faa0 am: 6c42071
am: 413dc41

* commit '413dc41e1d310834ace8a24f835b3af2ee86180f':
  BQ: fix some uninitialized variables

Merge "media: add and clarify OMX float numerical type" into nyc-dev

Merge "DO NOT MERGE BQ: fix some uninitialized variables" into klp-dev am: b4eac74 am: dc5d0f4  -s ours am: 12a01ba  -s ours am: 6fdc372 am: fdd9851  -s ours am: e98a9a9 am: c240884 am: 1899ffd
am: d53600c  -s ours

* commit 'd53600cc8441a9b552727e8f278f9c4b38224806':
  DO NOT MERGE BQ: fix some uninitialized variables

BQ: fix some uninitialized variables am: 93c617f am: f16b857 am: 1ed2469 am: 46da938 am: b2428c7 am: ce3faa0
am: 6c42071

* commit '6c42071f84c450a884c72d38a44ab3f45dd16343':
  BQ: fix some uninitialized variables

Merge "DO NOT MERGE BQ: fix some uninitialized variables" into klp-dev am: b4eac74 am: dc5d0f4  -s ours am: 12a01ba  -s ours am: 6fdc372 am: fdd9851  -s ours am: e98a9a9 am: c240884
am: 1899ffd

* commit '1899ffd6767574f1457b3ab5779e525ecc360d2a':
  DO NOT MERGE BQ: fix some uninitialized variables

BQ: fix some uninitialized variables am: 93c617f am: f16b857 am: 1ed2469 am: 46da938 am: b2428c7
am: ce3faa0

* commit 'ce3faa072a58dc01cecffa61d7df0a1d8bca212b':
  BQ: fix some uninitialized variables

Merge "DO NOT MERGE BQ: fix some uninitialized variables" into klp-dev am: b4eac74 am: dc5d0f4  -s ours am: 12a01ba  -s ours am: 6fdc372 am: fdd9851  -s ours am: e98a9a9
am: c240884

* commit 'c240884dccdf9f40725113ab88ebf615325dab19':
  DO NOT MERGE BQ: fix some uninitialized variables

BQ: fix some uninitialized variables am: 93c617f am: f16b857 am: 1ed2469 am: 46da938
am: b2428c7

* commit 'b2428c71c20a2e67a257eba540e6de2ca0fbae22':
  BQ: fix some uninitialized variables

Merge "DO NOT MERGE BQ: fix some uninitialized variables" into klp-dev am: b4eac74 am: dc5d0f4  -s ours am: 12a01ba  -s ours am: 6fdc372 am: fdd9851  -s ours
am: e98a9a9

* commit 'e98a9a910692cf7dee3bac2b58cf5f08c59db38f':
  DO NOT MERGE BQ: fix some uninitialized variables

BQ: fix some uninitialized variables am: 93c617f am: f16b857 am: 1ed2469
am: 46da938

* commit '46da9385ad96fb1c2baab64fdea01eb3b0b41dd0':
  BQ: fix some uninitialized variables

Merge "DO NOT MERGE BQ: fix some uninitialized variables" into klp-dev am: b4eac74 am: dc5d0f4  -s ours am: 12a01ba  -s ours am: 6fdc372
am: fdd9851  -s ours

* commit 'fdd9851ba326132c11d177bf5152b0c66cd87764':
  DO NOT MERGE BQ: fix some uninitialized variables

BQ: fix some uninitialized variables am: 93c617f am: f16b857
am: 1ed2469

* commit '1ed24698aa431aa4c3198fa3b8e830f1c29812a6':
  BQ: fix some uninitialized variables

Merge "DO NOT MERGE BQ: fix some uninitialized variables" into klp-dev am: b4eac74 am: dc5d0f4  -s ours am: 12a01ba  -s ours
am: 6fdc372

* commit '6fdc37205c32dcbd23718d68e45806c76f1536f8':
  DO NOT MERGE BQ: fix some uninitialized variables

BQ: fix some uninitialized variables am: 93c617f
am: f16b857

* commit 'f16b85752d7344814663c658175cce1a933e976b':
  BQ: fix some uninitialized variables

Merge "DO NOT MERGE BQ: fix some uninitialized variables" into klp-dev am: b4eac74 am: dc5d0f4  -s ours
am: 12a01ba  -s ours

* commit '12a01ba5f4f3f8ba122dfbf689e976019f353b3d':
  DO NOT MERGE BQ: fix some uninitialized variables

BQ: fix some uninitialized variables
am: 93c617f

* commit '93c617fd2a5e7910e0ba5c0ed6da152d30920679':
  BQ: fix some uninitialized variables

Merge "DO NOT MERGE BQ: fix some uninitialized variables" into klp-dev am: b4eac74
am: dc5d0f4  -s ours

* commit 'dc5d0f46de1f8a800b3af340ca57278989df151a':
  DO NOT MERGE BQ: fix some uninitialized variables

Merge "DO NOT MERGE BQ: fix some uninitialized variables" into klp-dev
am: b4eac74

* commit 'b4eac742c9e3f0238d5d03b237b2038df885ed2c':
  DO NOT MERGE BQ: fix some uninitialized variables

Merge "DO NOT MERGE BQ: fix some uninitialized variables" into klp-dev

media: add and clarify OMX float numerical type

Bug: 27172163
Change-Id: Idb399c05d14ee51f944251430d3561479feee6e3

Merge "MemoryDealer: expose internal allocation alignment" into nyc-dev
am: d6c017d

* commit 'd6c017de4238c0aa448596dae23cbdb9d10ae7fd':
  MemoryDealer: expose internal allocation alignment

Merge "MemoryDealer: expose internal allocation alignment" into nyc-dev

Merge "Added -q flag to showmap." into nyc-dev
am: 792a10b

* commit '792a10b940deeb1d0649f29a2f5777ac668ef780':
  Added -q flag to showmap.

Merge "Added -q flag to showmap." into nyc-dev

Merge "Add user activity event type for accessibility." into nyc-dev
am: cb4aed9

* commit 'cb4aed99849583a0395a0e06b66ddf2d35526f9d':
  Add user activity event type for accessibility.

Merge "Add user activity event type for accessibility." into nyc-dev

Added -q flag to showmap.

dumpstate calls showmap for each pid, and since most of them are empty,
it ends up polluting logcat with entries like:

03-17 14:49:05.974 12160 12160 E dumpstate: command '/system/xbin/su
root showmap -q 9867' failed: No such file or directory

BUG: 26906985
Change-Id: I7e064b2ec8901b6ccee49b228be96ff2847c703c

BQ: Allow shared buffer to be the first queued buffer
am: 295a9fc

* commit '295a9fc8aa87daa2cded5c1a279b8cd24e9a9a9f':
  BQ: Allow shared buffer to be the first queued buffer

MemoryDealer: expose internal allocation alignment

Bug: 27722308
Change-Id: Ie4ee1f54edcb711d39f8419a160af1495af0152e

BQ: Allow shared buffer to be the first queued buffer

- When single-buffer mode is enabled the shared buffer will now be
  either the first buffer dequeued or the first buffer queued.
  Whichever comes first.

Bug 24940410

Change-Id: I9f7e2630640e37fe1fb967aed41a3025567247a8

Merge "Native: Sensor: For ndk sensor enable request, enable sensor with SENSOR_DELAY_NORMAL delay instead of 0 delay, by default " into nyc-dev
am: ec3c05b

* commit 'ec3c05b2d47efaca03ec42572fc2631af5267438':
  Native: Sensor: For ndk sensor enable request, enable sensor with SENSOR_DELAY_NORMAL delay instead of 0 delay, by default

Merge "Sensor: Add batch support for native applications" into nyc-dev
am: 2b6df49

* commit '2b6df49ecd89aa59046eafb8b7b5bf027fa56ab7':
  Sensor: Add batch support for native applications

Merge "Fix eglSurfaceAttrib return value" into nyc-dev
am: f2b1a15

* commit 'f2b1a15d360e1a980372ceb232cf49ce7f56ccdc':
  Fix eglSurfaceAttrib return value

Merge "loader: cache top-of-chain PhysicalDevice" into nyc-dev
am: 8daa5e7

* commit '8daa5e772d2d6f03fc19cc26bcd96724581f0223':
  loader: cache top-of-chain PhysicalDevice

Merge "Improve logging of child commands." into nyc-dev
am: d3d45c3

* commit 'd3d45c3dddf48d557f2b0d622df8ebc1317f9bb5':
  Improve logging of child commands.

Merge "Otapreopt: Load properties from files" into nyc-dev
am: b25c2d3

* commit 'b25c2d375c400556a553f64744d4a080add6a68c':
  Otapreopt: Load properties from files

Merge "Otapreopt: Remove b_mount_path" into nyc-dev
am: ae3cd06

* commit 'ae3cd0687610ec585119b54ac18141f6aea4e593':
  Otapreopt: Remove b_mount_path

Merge "atrace:  Adding dumping message to logcat." into nyc-dev
am: 325232c

* commit '325232ccb52981758ab7286e046c0c8a039b5cac':
  atrace:  Adding dumping message to logcat.

Installd: Make extract-only = interpret-only
am: bb7a770

* commit 'bb7a77082fde8f73971adadf50faecdf5f660b48':
  Installd: Make extract-only = interpret-only

Merge "Native: Sensor: For ndk sensor enable request, enable sensor with SENSOR_DELAY_NORMAL delay instead of 0 delay, by default " into nyc-dev

Native: Sensor: For ndk sensor enable request, enable sensor with
SENSOR_DELAY_NORMAL delay instead of 0 delay, by default

In case of one or more sensor applications running in background
(say Auto Rotation, Auto Brightness, some sensor ndk daemon like
panning, etc), which are using the common sensor (say accelerometer)
running at UI delay(66 ms), and then some 3rd party sensor ndk daemon
starts(say Direct Call feature, GPS ndk daemon for correcting latitude
and longitude using sensor data while using navigation, etc), which
enables the same sensor (accelerometer) but does not set the delay,
then that sensor chip will start to generate data at minimum delay
possible because of passing 0 delay value during enable.

Ideally, this is wrong behaviour because none of the sensor-ndk daemon
or android application has requested for minimum delay yet and so sensor
should generate data still at 66ms only. This in turn will make
significant increment in current consumption because of fastest sampling
rate.

Also, as you know, ASensorEventQueue_enableSensor api and
ASensorEventQueue_setEventRate api are not deprecated and is still used
irrespective of HAL 1_3 or latest 1_4. In fact, these are the only two
api currently available for enabling and setting delay. Direct Call,
panning, Smart alert, gps daemon, etc are such examples which uses these
ndk api

Small Code Snippet:
AutoRotation starts:
mSensorManager.registerListener(
        listener1, mAccelerometer, SensorManager.SENSOR_DELAY_UI);

AutoBrightness start:
mSensorManager.registerListener(
        listener2, mAccelerometer, SensorManager.SENSOR_DELAY_NORMAL);
mSensorManager.registerListener(
        listener2, mLight, SensorManager.SENSOR_DELAY_NORMAL);

Panning:
ASensorEventQueue_enableSensor(mSensorEventQueue1, mAccelerometer);
ASensorEventQueue_setEventRate(
        mSensorEventQueue1, mAccelerometer, SENSOR_DELAY_UI);
(Here, first 0 delay will be set, but UI delay will override it with
function call setEventRate)
... more possible android-apps/sensor-ndk daemon

So currently, accelerometer is running at 66 ms delay
because DELAY_UI (66ms) is less than DELAY_NORMAL (200ms)

Now, sensor gps ndk daemon starts, enables the sensor but does not set
delay:
ASensorEventQueue_enableSensor(mSensorEventQueue2, mAccelerometer);
--> This call will request for 0ms delay so minimum delay will be set
(say 10ms)

Therefore now, accelerometer will run at 10 ms delay which is wrong
behavior because delay should still be 66 ms ideally. Also, running
sensor at lowest delay unnecessarily increases current consumption.
(Unless requested for minimum delay like in games)

Change-Id: Ib18d0d77f80b2621a204d491c3f61eed9a8b7251
Signed-off-by: Aniroop Mathur <a.mathur@samsung.com>
Signed-off-by: Aniroop Mathur <aniroop.mathur@gmail.com>

Merge "Sensor: Add batch support for native applications" into nyc-dev

Merge "Fix eglSurfaceAttrib return value" into nyc-dev

Merge "loader: cache top-of-chain PhysicalDevice" into nyc-dev

Don't use su to when calling am or dumpsys.

su is not available on user builds anymore, hence the bugreport
notifications would never be sent on those builds. Instead, it should
explicitly drop root using system calls.

BUG: 27583193
Change-Id: Ia6256b241fdd6ab4c059fb764b10b4445ad6551d

Add user activity event type for accessibility.

Bug: 19531976
Change-Id: I95096dbf252556e677a8a9f3bffe8abaab353ab4

Merge "Improve logging of child commands." into nyc-dev

Improve logging of child commands.

When execvpe() is called it doesn't return, so the child error handling
should be done at the parent, otherwise the failure is not printed in the bugreport.

BUG: 27583193
Change-Id: I6161e4c72b7b0da7b82bff7fd72d7a5aaf4d6e89

Merge "Otapreopt: Load properties from files" into nyc-dev

loader: cache top-of-chain PhysicalDevice

Bug: 27615575
If a layer wraps the VkPhysicalDevice object the
loader would be unable to figure out what VkInstance
it was associated with since it no longer matches
the VkPhysicalDevice queried from the driver.
Now cache the VkPhysicalDevice's from the top of
the chain as well to resolve this.

Change-Id: I7d04884dc3aa8623ce2f67e4f6575f74b82e2834

Merge "Otapreopt: Remove b_mount_path" into nyc-dev

Otapreopt: Load properties from files

Refactor file parsing into a helper.

Load system properties, completely, from the right sources. Add code
to parse init.environ.rc for environment variables.

Bug: 25612095
Change-Id: I3e5b2ff620ccb374c775ed1922baa0d9b8ed5974

Sensor: Add batch support for native applications

Currently, it is not possible for native applications to use the
sensor in batch mode. As it is a 'C' header file so function
overloading is not possible. Therefore, lets add a new function to
enable the sensor, set the delay and batch timeout, all together.

Change-Id: I2671ba37f48bae63945bf24e9f4ea009d33793f7
Signed-off-by: Aniroop Mathur <a.mathur@samsung.com>

Fix eglSurfaceAttrib return value

Bug 24940410

Change-Id: Ibd6be5861fe48c5d2230d48a7b65ae0b21a01b9b

Merge "atrace:  Adding dumping message to logcat." into nyc-dev

Installd: Make extract-only = interpret-only

Test the impact of verifying and quickening at install time and
post-OTA boot time.

Bug: 26833007
Bug: 27688727
Bug: 27689078
Change-Id: Ib2ebe71747f2b8dc7a2864f3d02b5c2217d29d6c

Otapreopt: Remove b_mount_path

The final chosen design runs the tool in a chroot, no B mount path
required.

Bug: 25612095
Change-Id: Iba920455799da1aff97f759afc784ce43c4bf0b1

DO NOT MERGE BQ: fix some uninitialized variables

Bug 27555981
Bug 27556038

Change-Id: I436b6fec589677d7e36c0e980f6e59808415dc0e

BQ: fix some uninitialized variables

Bug 27555981
Bug 27556038

Change-Id: I436b6fec589677d7e36c0e980f6e59808415dc0e

Merge "Revert "Add a callback thread to ConsumerBase"" into nyc-dev
am: 49380be2cf

* commit '49380be2cfbc43adb79ef559b7bd8578040948cc':
  Revert "Add a callback thread to ConsumerBase"