Merge "NDK docs: Fixed in-line comments for A8 and ARGB444 bitmap configurations. Test: Verified changes with romainguy@. Related bugs: b/26922523"
am: 270fff6a7c

Change-Id: Ibfc593ff8a9d913e083b8fda4341ca5110ba87a1

Merge "NDK docs: Fixed in-line comments for A8 and ARGB444 bitmap configurations. Test: Verified changes with romainguy@. Related bugs: b/26922523"

NDK docs: Fixed in-line comments for A8 and ARGB444 bitmap configurations.
Test: Verified changes with romainguy@.
Related bugs: b/26922523

Change-Id: Id51445b09896407e808b519f3ee2f4be90f5b7eb

Merge "SurfaceFlinger supports ClearClientTarget for Sideband"
am: 6560727f37

Change-Id: Idfeaf9f6e29eda948b44a1110ae18a5099a0977e

Merge "SurfaceFlinger supports ClearClientTarget for Sideband"

SurfaceFlinger supports ClearClientTarget for Sideband

When clearing the layer's intersection with the dirty region,
HWC2::Composition::Sideband layer needs to be cleared as well.

Bug: 33443134
Test: manual - enter PIP mode with Live Channel
Change-Id: I4fecc7b5db8a2bf220b681bd087264290d1a2443

Merge "Fix warning: macro redefined"
am: 909ef30a63

Change-Id: Ia1b083546fe41a81acf83e0cb0b8921700aa1bfa

Merge "Fix warning: macro redefined"

Merge "lshal: print pids of client processes."
am: 1ca609d756

Change-Id: Ide68ec1bc811d0ed3c23763bc1689d43282c6503

Merge "lshal: print pids of client processes."

lshal: print pids of client processes.

Bug: 34777099
Bug: 34861460

Test: lshal
Test: NFC / light / audio with and without persistent.hal.binderized;
      run lshal to reflect this.

Change-Id: I50aa5d99fb14957b1e40a405836e68c235a22006

Merge "Do not -1 for lshal."
am: 50752b732e

Change-Id: I8e0f285f5b17b11f32b504dda3a4c31da770653f

Merge "Do not -1 for lshal."

Merge "Disable integer sanitization on host."
am: b79d9f9d49

Change-Id: I517a019007bcf12b0884c22234abf25482d50cca

Merge "Disable integer sanitization on host."

Merge "Fix a use-of-uninitialized-value warning"
am: d047e10558

Change-Id: I4bd31c59331b6ef26e36bfff3fe57becf9b5ecc2

Merge "Fix a use-of-uninitialized-value warning"

Do not -1 for lshal.

Test: compiles
Change-Id: I99e6306a6b82ec85bc84f9572f3e4f2f8cdf2847

Merge "Fix -Wdouble-promotion warnings"
am: 7920557287

Change-Id: I2b7a8f6c24990220f90cb3d2f9221228ec330be5

Merge "Fix -Wdouble-promotion warnings"

Disable integer sanitization on host.

Test: NFC until https://android-review.googlesource.com/#/c/329510
Change-Id: I0183556d5b945c689934664fe7f15caa18a0997d

Fix a use-of-uninitialized-value warning

Caught by clang's static analyzer:
frameworks/base/core/jni/com_google_android_gles_jni_GLImpl.cpp:190:30:
warning: The right operand of '+' is a garbage value
            buf = (char*)buf + offset;

This is because getPointer doesn't always set `offset` to a value. We
could fix it locally by initializing `offset`, but it seems to be less
error-prone to just fix getPointer instead.

Bug: None
Test: Still builds; warning is now gone.
Change-Id: I90bea0c94cf8d7f92a0d9a4db4e64e7e563bfb09

Merge "Add permission bits to open calls with O_CREAT"
am: 9dc2e6bc43

Change-Id: Ifa1d1607751dc89cb341a97c1b737bf2ee7bc260

Merge "Add permission bits to open calls with O_CREAT"

Add permission bits to open calls with O_CREAT

Open needs a third argument if you pass it O_CREAT:
https://linux.die.net/man/3/open

This turns into a compile-time error with FORITFY, so we need this fixed
before our unbroken FORTIFY can go in.

(The TEMP_FAILURE_RETRY open isn't detected by clang FORTIFY, but I
noticed that open_reference_profile potentially passes in O_CREAT as an
open_flag.)

Bug: 32073964
Test: Now builds with clang FORTIFY; CtsCompilationTestCases passes;
manually verified that /data/misc/profiles/ref/*/primary.prof has rw
permissions after a successful `cmd package compile -m speed-profile`.
Change-Id: Ie707d5ad403d2f86c769277b3e0f147c45000a6b

Merge "Replace strlen with arraysize in array bounds."
am: 70a5f12045

Change-Id: I3357cadb32858a94d94c6b4bd82fe69f241027b4

Merge "Replace strlen with arraysize in array bounds."

Fix -Wdouble-promotion warnings

With the new clang FORTIFY, this code emits the following error:

error: implicit conversion increases floating-point precision: 'float'
to 'double' [-Werror,-Wdouble-promotion]

This is because strncpy is now a function, not a macro defined in a
system header. So, the warning is considered to appear in user code
(instead of system code), and is therefore no longer suppressed.

Bug: 32073964
Test: Now builds with clang FORTIFY
Change-Id: Id5fcfe91755535f90b542e11bc42b66976fe24fc
Merged-In: I5e8292d7800434268793bc4db4a96f20b7022e17

Replace strlen with arraysize in array bounds.

clang-FORTIFY doesn't let us evaluate strlen("foo") as a constant
expression that can be used in static array bounds. Instead, we'd need
to either do __builtin_strlen("foo") or arraysize("foo") - 1. Just use
arraysize here, since the extra 1 byte per array isn't likely to be a
terrible thing.

Happy to swap to __builtin_strlen if anyone cares deeply enough.

Bug: 32073964
Test: Now builds with clang FORTIFY.
Change-Id: I4c9b3eb74775901554b939246722bd3a22326d10

Merge changes from topic 'lshal'
am: 1f2d5a9cda

Change-Id: If2ed2950dfa6f7a4598e143e9e22b34426a8f196

Add debugDump() to IServiceManager.
am: 5ccdab9bc5

Change-Id: I4823ca88072ebe167fa6a3b894cad610cea88c94

Add an lshal command.
am: 85dd3ad89f

Change-Id: I43be1cd548f30d7d6e954411fc3db8b13e20f3c0

Merge changes from topic 'lshal'

* changes:
  Add debugDump() to IServiceManager.
  Add an lshal command.

Merge "Fix two bugs in external storage accounting."
am: d1361ceb0c

Change-Id: I6f661104fbe8527bfbb1f9599beb395e5bc9d218

Merge "Fix two bugs in external storage accounting."

Add debugDump() to IServiceManager.

Test: mma
Change-Id: I04289500c7ee42a84d3593b4c0e45a0b829370a7

Add an lshal command.

lshal lists services registered to the binderized
IServiceManager and all services available to the
passthrough IServiceManager as *.so libraries. It
also dumps the reference count for each service.

Bug: 34712252
Test: adb shell lshal
Change-Id: I9436a033cf736b8a8c307a2e87549c9db6379f3f

Fix two bugs in external storage accounting.

When doing a manual tree walk, we always want to inherit fts_number
from the parent fts node.

For create_data_media_package_path(), two of the arguments had been
swapped.

Test: cts-tradefed run commandAndExit cts-dev -m CtsAppSecurityHostTestCases -t android.appsecurity.cts.StorageHostTest
Bug: 27948817, 30927649
Change-Id: Ic3cbf387cf119aa82af6e380bde9d9b069f6bfb0

Merge "Add mode to open call with O_CREAT"
am: af78fe13a0

Change-Id: I488ffb3be9152532fd3e7eddd7bc78d311673420

Merge "Add mode to open call with O_CREAT"

Add mode to open call with O_CREAT

This change was already applied internally. For us to submit FORTIFY, we
need to push it out here. Otherwise, we get compile-time errors about
this.

Bug: 32073964
Test: Builds with FORTIFY.
Change-Id: Ib500aca933468c06f7ceb796a9bb6a16c118e366
Merged-In: I63eb3d5499ae7e9ff9a77393763e39b747fd4611

Merge "Properly align a packed structure."
am: 24257dda48

Change-Id: I66d07aab4dfa72afe44270d866514102f660aab6

Merge "Properly align a packed structure."

Merge "Update atrace to work with tracefs, not debugfs"
am: a92616ecfe

Change-Id: I139cd33d0a943bff3385cbd3ebc52eb47bc82be5

Merge "Update atrace to work with tracefs, not debugfs"

Merge "dumpsys: add option to list hardware services."
am: 4d741ebedc

Change-Id: I78070b7363e0562a0b236035989dcb9ae666d6d6

Merge "dumpsys: add option to list hardware services."

dumpsys: add option to list hardware services.

This gives a convenient way to dump information about hidl services.
When hidl services dump information themselves as well, that dump will
be made here as well.

Bug: 33382892
Test: dumpsys_test, dumpsys --hw works as expected
Change-Id: Ib08b5d052e63ed6260035ee8d3ad3dac296202b1

Update atrace to work with tracefs, not debugfs

When tracefs is mounted, tracing is under /sys/kernel/tracing not
/sys/kernel/debug/tracing. Modify atrace to detect this situation and
get the right files.

Bug: 31856701
Test: Run atrace, check output is same where ever tracefs is mounted
Change-Id: I78df0da91d9f56c56077d1b8b9dba7b0126b2d8d

Merge "Add Binder support for Parcelable exceptions."
am: 7ebfeb9d1c

Change-Id: Ia9fe213e51fef0fe2b000ed188b943079f9c9535

Merge "Add Binder support for Parcelable exceptions."

Merge "Add i2c tracing support to atrace"
am: 2630e78f53

Change-Id: Icc06121228858c35b90ef5ad0d4a0ecc470a3909

Merge "Add i2c tracing support to atrace"

Merge "Add feature flag for TelephonyManager#get/setAllowedCarriers"
am: c4fc77792c

Change-Id: I4368431b3537bde427d8bb68f33fc8b6cf876f15

Merge "Add feature flag for TelephonyManager#get/setAllowedCarriers"

Add i2c tracing support to atrace

Cherry-picking from internal master

Test: None
Change-Id: Iaf0daa0982bd8d2355d3590d43ad78711e764ee0

Merge "Measure external storage using new GIDs."
am: f5a21ba425

Change-Id: Id8f48276b4768a9e49b943e7142c33d7a365374a

Merge "Measure external storage using new GIDs."

Measure external storage using new GIDs.

We just defined a new range of GIDs that will be used to mark per-app
data files stored on external storage, so start measuring them.

Also measure all OBB files using another new GID that was defined.

Note that we're relying on the per-app cache GIDs to track cache
usage on *both* internal and external storage, which means that stats
and extStats won't always add up.  (The framework already combines
these values together, and we didn't want to waste precious GID
space on explicitly tracking cache files on external storage.)

Test: builds, boots
Bug: 27948817, 34263266
Change-Id: Ife087df299ff8ee1a75fce1e39b4b737cf9375d5

Merge "First pass at updated cache clearing logic."
am: 45270e18cc

Change-Id: Icade9c982866bb45c63cc484b77820fb9c1b77ea

Merge "First pass at updated cache clearing logic."

Merge "TYPE_DYNAMIC_SENSOR_META should be wake up sensor"
am: 29280addfd

Change-Id: I45ad2448abf8da0ab2d1cdc4e29c7d2d30d3affe

Merge "TYPE_DYNAMIC_SENSOR_META should be wake up sensor"

Merge "Clean up and simplify the logic in dexopt"
am: 57eb11ed2d

Change-Id: I9571ed4171d592d78e76278ffbeee9f978d4de43

Merge "Clean up and simplify the logic in dexopt"

Merge changes from topic 'kill_debuggerd'
am: 134a7193a0

Change-Id: Icb705df114dc073ef66507c59884e77ea1cf828b

dumpstate: use O_APPEND for anr file.
am: 989b812dcd

Change-Id: Ibaa04b99195886df0ff14b2f6569e37918373619

dumpstate: switch to using libdebuggerd_client.
am: d2db024a86

Change-Id: I219627b359a88b8a694b4f4cd46de9cea9767655

Merge "libGLES_android: Add generic 32bpp BGRA EGLConfig"
am: 0233bd47ed

Change-Id: Ie6f7c202ceb22e76e28ad5973cf668bddd3b8552

Merge changes from topic 'kill_debuggerd'

* changes:
  dumpstate: use O_APPEND for anr file.
  dumpstate: switch to using libdebuggerd_client.

Merge "libGLES_android: Add generic 32bpp BGRA EGLConfig"

Clean up and simplify the logic in dexopt

Make some order in dexopt logic by extracting independent parts in their
own helpers.

It will make further updates of dexopt simpler and easier to reason
about (e.g. for secondary dex files).

Test: device boots
      adb shell cmd package compile -m speed|speed-profile packageName
      install new packages

Bug: 32871170

(cherry picked from commit 4a391f80601d7f65d7e63353363e6d32868161d5)

Change-Id: If0c96174e904519a4332e54553690b691ff63149

First pass at updated cache clearing logic.

The old clearing algorithm is very naive and it sorts all cached files
globally by modified time.  This sadly lets apps gamify the system by
setting their modified times far in the future, and it's very
expensive because it requires a global filesystem traversal to free
up even the smallest amount of data.

Instead, this CL introduces a much more fair cache clearing algorithm
that deletes files from specific UIDs based on how much cache space
that UID is using proportional to the space allocated to them.  This
new design has several nice properties:

-- It uses the recently added quotactl() feature to rapidly target
the apps that are using the most space.
-- We only need to traverse the filesystem for UIDs that actively
enter the crosshairs of the clearing algorithm.
-- Disciplined apps who stay under their allocated quota will be
the last to have their cached data cleared.
-- This design can be easily adapted to support additional features
such as atomic purging and tombstones.

In summary, the new algorithm is extremely efficient when freeing up
the typical small-to-medium amounts of disk space, and is only
moderately less efficient than the old algorithm when forced to clear
all cached data.

Test: builds, boots, clearing strategy looks sane
Bug: 33965858
Change-Id: I66f95089cb33f1add3f31fcf1082ab2469870fda

Add feature flag for TelephonyManager#get/setAllowedCarriers

Add android.hardware.telephony.carrierlock feature flag declaration.

Bug: 33480084
Test: cts
Change-Id: Ib6ede97d7ee8d6e924c1483ee51f52bdfa09950b

Merge "Add systrace details to storage measurements."
am: a693382d94

Change-Id: I4bdc09bd3e62dfbc522b527537bcd526a391fbaf

Merge "Add systrace details to storage measurements."

Merge "servicemanager: selinux: set selinux callbacks early"
am: bfbe2d711c

Change-Id: I0e3e088eb3d8404d1707ac61aa18eda5f0cbad8d

Merge "servicemanager: selinux: set selinux callbacks early"

Add systrace details to storage measurements.

Can be used to find bottlenecks, and confirm that quota stuff is
being used when expected.

Test: builds, boots
Bug: 34341727
Change-Id: Ifefc155e5d3b0c74245e0d287e84df699ce09d16

dumpstate: use O_APPEND for anr file.

Bug: http://b/30705528
Test: ran dumpstate, grepped dmesg for new selinux denials
Change-Id: I28814edcf37a7edbde79279feff0e7def64316f2

dumpstate: switch to using libdebuggerd_client.

Bug: http://b/30705528
Test: manual inspection of dumpstate output
Change-Id: I8f9822f67f1a81fcfa2b3a9120e6631b19b29854

Add Binder support for Parcelable exceptions.

Follow the new framework changes that support sending EX_PARCELABLE
exception types.

Test: builds, boots
Bug: 33749182
Change-Id: I7a856fa89f23aab4f782c4e5ae2beb8166fa8da4

Merge "Handle devices without quota, speed up lookup."
am: 36ba03a248

Change-Id: I38d4e01057e3cd65f4362056336294e8136df459

Merge "Handle devices without quota, speed up lookup."

servicemanager: selinux: set selinux callbacks early

SELinux library error logs are never be shown in logcat if
there's a failure before the callbacks are set. We missed
all logs from service context initialization because of this.

Bug: 33746484
Test: Restart new service manager to make sure context init logs
show up in logcat.

Change-Id: I5479cf5fe10ceb1a210eda26946b6ea344792e24
Signed-off-by: Sandeep Patil <sspatil@google.com>

Merge changes Ic746ea89,Iebaa8063
am: 950e3f5696

Change-Id: I61d240c0345e7b0a5a5d499c003230396aa469ec

Wrap all installd operations in lock.
am: 7a9059e8ed

Change-Id: Ide1ef93a041ee5ff683babb865e382d1cb7a7356

Mask st_mode before comparing it.
am: 24ef15bd00

Change-Id: I2b9ec79a679428cb6018798110bf4c3aa2936ae0

Merge changes Ic746ea89,Iebaa8063

* changes:
  Wrap all installd operations in lock.
  Mask st_mode before comparing it.

Handle devices without quota, speed up lookup.

Start tracking which block devices have quota support, and gracefully
clear FLAG_USE_QUOTA when no support is present.

Also build a cached map of mounted volumes that support quota, which
halves the average quota calculation speed from 0.70ms to 0.35ms,
since we're no longer parsing procfs every time.

Test: builds, boots, common operations work
Bug: 34249218
Change-Id: Ie791df7801b67495331f3eea256c018860c9b4f6

Wrap all installd operations in lock.

Historically installd has a single lock up in the framework, but
we're starting to call in from multiple locations, so installd needs
to protect itself.

This specifically applies to any disk measurement requests, which
use the optimized fts(3) mode which uses chdir() internally.

Test: builds, boots, common operations work
Bug: 34330051
Change-Id: Ic746ea890ebcc7b2bf923404874b04dbc19d42ad

Mask st_mode before comparing it.

Otherwise it will never match the requested mode_t.

Test: builds, boots, skips when matching
Bug: 34201111
Change-Id: Iebaa8063a72eb6eaf2e773c1d35204617d0f16b8

Merge "liblog: use log/log.h when utilizing ALOG macros"
am: 9a30f044ca

Change-Id: Ibf35cc60eeb53bc421a2f886fa63b37ffa2083c1

Merge "liblog: use log/log.h when utilizing ALOG macros"

liblog: use log/log.h when utilizing ALOG macros

Audit dropout because ALOG is behind MYLOG ...

Test: compile
Bug: 34250038
Change-Id: I009478157d2267e10f8c09a7faa4b8f7c7c4a1e7

Merge "Don't print error if we can't talk to hwservicemanager."
am: 53d36e4476

Change-Id: I3a524a89d8d6b022b4a825197e29b6a69687c8d9

Merge "Don't print error if we can't talk to hwservicemanager."

Don't print error if we can't talk to hwservicemanager.

This currently fails as a non-root user due to SEPolicy,
and is causing CTS test failures. Until we figure out
the policy on b/34242478, don't print an error to allow
the test to pass again.

Test: cts-tradefed run cts --module CtsAtraceHostTestCases --test
android.atrace.cts.AtraceHostTest#testSimpleRun

Bug: 33814619
Change-Id: Ie5578639028e7890284d71bbf1b753f5803ca4af

Merge "Replace cutils/log.h and rationalize log/log.h and android/log.h"
am: d655e61a22

Change-Id: I86d040528ab10e503e03f033f12d47a8142c3c6b

Merge "Replace cutils/log.h and rationalize log/log.h and android/log.h"