Move multi-user OBB under user-specific path.

Support apps that may manually build OBB paths using
getExternalStorageDirectory() by making multi-user OBB paths live
under user-specific path.

This new structure means we can simplify the emulated_legacy paths
to be a single recursive bind mount.

Bug: 7212801
Change-Id: I85a0caebca52a6ff3bfe61c961518a91f0b8f8ff

am 9bd5a9db: Reconcile jb-dev and AOSP

* commit '9bd5a9db106863069dd6c12de881acca6103f8d7':
  Fix an ugly bug where try/catch offsets weren't being mapped properly. do not merge.
  Fix a bug where the max blowup of annotations was incorrect. do not merge.
  Fix a bug where debugInfos' size wasn't being computed. do not merge.
  Make dex merging stateless. do not merge.
  Support debug info in dexmerge. do not merge.

Reconcile jb-dev and AOSP

Change-Id: If3d2ce0f23b929f92d49649e015386670ed16863

am 74d4652e: am e64cfc1c: am b79d90eb: Merge "Log when the native trace is unavailable"

* commit '74d4652ebd432295a5442b024f51bce531abea15':
  Log when the native trace is unavailable

am e64cfc1c: am b79d90eb: Merge "Log when the native trace is unavailable"

* commit 'e64cfc1cdacd8a820acce596f6986e17e2f5e330':
  Log when the native trace is unavailable

am b79d90eb: Merge "Log when the native trace is unavailable"

* commit 'b79d90ebd197b9040c06d73f5dc8bbc8fdad1d54':
  Log when the native trace is unavailable

Merge "Log when the native trace is unavailable"

Log when the native trace is unavailable

This should allow us to differentiate between "couldn't get the
stack" and "didn't try to get the stack".  Also show the thread's
state (e.g. 'R' for running, 'D' for uninterruptible syscall).

Bug 7053953

(cherry-pick of b3667a19f5c573b7785876979af4781292d27327.)

Change-Id: I0a40cb3d3cdd9aef8589a39586cccd9c229aa8cb

Log when the native trace is unavailable

This should allow us to differentiate between "couldn't get the
stack" and "didn't try to get the stack".  Also show the thread's
state (e.g. 'R' for running, 'D' for uninterruptible syscall).

Bug 7053953

Change-Id: Ibc9e6a6f0fc855a4e5e8e133122232dc939bcb16

am 90f12e0b: am 4535e29e: am a177aa50: Merge "Use 0xde as the poison value after JIT code cache reset."

* commit '90f12e0b4b08b9776b02580b223ec8968a5725b0':
  Use 0xde as the poison value after JIT code cache reset.

am 4535e29e: am a177aa50: Merge "Use 0xde as the poison value after JIT code cache reset."

* commit '4535e29e384a400a62fe259674565c0e21c4c182':
  Use 0xde as the poison value after JIT code cache reset.

am a177aa50: Merge "Use 0xde as the poison value after JIT code cache reset."

* commit 'a177aa50c665e97f92ee7519ed9d32b0e89e84d3':
  Use 0xde as the poison value after JIT code cache reset.

Merge "Use 0xde as the poison value after JIT code cache reset."

Use 0xde as the poison value after JIT code cache reset.

(cherry-pick of 60497bbcee68d98b2bdcb3fb64682b00d61b942d.)

Change-Id: I4f8c1bac2ac3bdc1d931e91f3e2e8038fe4c2b5a

Merge "Use 0xde as the poison value after JIT code cache reset." into jb-mr1-dev

Use 0xde as the poison value after JIT code cache reset.

Change-Id: Ic25ede5bcd988f240d8c24df6bc5da81e2b03249

am 8e80f76a: am d207be35: am f31a0600: Merge "Optimize those StrictMath routines that are identical to Math routines."

* commit '8e80f76abdeedb75ca0837545f6c33c2c452a928':
  Optimize those StrictMath routines that are identical to Math routines.

am d207be35: am f31a0600: Merge "Optimize those StrictMath routines that are identical to Math routines."

* commit 'd207be351aa5f8707a7829fc5cfd602b69b5bcc5':
  Optimize those StrictMath routines that are identical to Math routines.

am f31a0600: Merge "Optimize those StrictMath routines that are identical to Math routines."

* commit 'f31a0600bb84e04280440c84a5966a624a89e117':
  Optimize those StrictMath routines that are identical to Math routines.

Merge "Optimize those StrictMath routines that are identical to Math routines."

Optimize those StrictMath routines that are identical to Math routines.

We can just use the existing Math intinsics.

Bug: 7146208
Change-Id: I9e78d33cf65a5dcc5a7c0133e67bd9c3c1e43f23

Merge "Missing EMULATED_STORAGE_TARGET isn't an error." into jb-mr1-dev

am 91a17e5e: (-s ours) Reconcile with jb-release - do not merge

* commit '91a17e5e78ddc49b761133151831770379dc777f':
  Test if new build is kicked off.

Reconcile with jb-release - do not merge

Change-Id: I20cffa47e360af1eac86d8a04aa53441f9260bc6

Missing EMULATED_STORAGE_TARGET isn't an error.

Devices without emulated storage should just skip the tmpfs.

Bug: 7131382
Change-Id: I9c697e4faf7864ec3a2695cd3770fbf0c9a5cce3

Test if new build is kicked off.

Change-Id: Ia6a25f00cb28e80111b78d87180575d57c55b062

Include user identifier in external storage paths.

When building external storage paths, always include user in path
to enable cross-user paths and aid debugging.

Each Zygote process continues to only have access to the appropriate
user-specific emulated storage through bind mounts. A second set of
mounts continue supporting legacy /sdcard-style paths. For example,
a process running as owner has these mount points:

/storage/emulated_legacy
/storage/emulated_legacy/Android/obb
/storage/emulated/0
/storage/emulated/obb

This change also creates two nested namespaces around zygote. The
first namespace is created when the zygote starts, and it holds a
single writable tmpfs, since rootfs is ro. The second namespace is
created quickly after forking, and is used to hold the private
user-specific bind mounts. (This design saves us from creating a
tmpfs-per-process.)

Bug: 7131382
Change-Id: Id80403ec956f0bff6997411cb1f82f273c76b85f

Fix an ugly bug where try/catch offsets weren't being mapped properly. do not merge.

In dex files, there are several places where one object refers
to another object by its position in the file. The dex merger
is generally very careful to adjust these mappings when combining
dex files.

Unfortunately one of these cases was broken. Each try_item refers
to a corresponding encoded_catch_handler by its byte offset in a
list. Most of the time this byte offset is the same in the input
dex file and the output dex file. But encoded_catch_handlers are
variable-length because they use a variable-length uleb128 encoding
to address the type_idx being caught. When dex files are merged,
some exception types may go from having a small index to having
a large index, increasing the number of bytes required to encode
that index. This breaks our ability to directly copy over offsets
as we were doing previously.

Bug: http://code.google.com/p/android/issues/detail?id=36490

(cherry picked from commit d43341a24abe339d474b0b0d92669917ae2eb9bf)

Change-Id: I4e6ab791925a918513122b46e039c02dfef62ace

Fix a bug where the max blowup of annotations was incorrect. do not merge.

I'm not sure where the 1.34 number comes from but it's incorrect.
From the spec, the encoded_annotation is made up of a single byte
plus an unlimited number of uleb128 values. Each of these values
can double in width in the worst case. I received (personal) email
from one user who'd run into a case worse than the incorrect 1.34
limit.

(cherry picked from commit bf7dfeea94f21dd0e097cf5f786f9995722fd70d)

Change-Id: I033a0d76ea0324d434d732e7757df0c3fd925b68

Fix a bug where debugInfos' size wasn't being computed. do not merge.

Previously debugInfos' size of 0 caused a malformed dex file
because debug data was present but not mentioned in the table
of contents.

(cherry picked from commit 55ac44b64744cd5692d6495bca99dfc8181e0cfc)

Change-Id: I2084c06521ea200772c523cdaa28c8019fa76025

Remove /sdcard hack for shell users.

Using a /storage/sdcard0 symlink and a tmpfs staging area was used
to enable "/sdcard" paths for both zygote processes and the shell
user, but it breaks on devices with both emulated and physical
external storage.

Specifically, vold can't mount a physical sdcard onto the primary
storage symlink, and zygote can't re-bind-mount when secondary
storage is remounted after dropping privileges.

Bug: 7094858, 7064600
Change-Id: Id91e3b3e521a3b3e1bb5358920adc66a0a43917f

am e599c468: am cfeacbdd: Merge "Remove unnecessary compiler pragma."

* commit 'e599c4684ac47fe6417b9c482b7ff651513988ff':
  Remove unnecessary compiler pragma.

am cfeacbdd: Merge "Remove unnecessary compiler pragma."

* commit 'cfeacbdd0cdd71a7974ac623e6534d0221d93ed9':
  Remove unnecessary compiler pragma.

Merge "Remove unnecessary compiler pragma."

am 86e036a8: am e6516a53: Merge "Fix an ugly bug where try/catch offsets weren\'t being mapped properly."

* commit '86e036a83d596ebacb20c87c8f7bd8a8dfa98d9c':
  Fix an ugly bug where try/catch offsets weren't being mapped properly.

am e6516a53: Merge "Fix an ugly bug where try/catch offsets weren\'t being mapped properly."

* commit 'e6516a5395a20f2bf923753e4dd87ffd304074fb':
  Fix an ugly bug where try/catch offsets weren't being mapped properly.

Merge "Fix an ugly bug where try/catch offsets weren't being mapped properly."

am 93ee4913: am f3b60d2f: Merge "Fix a bug where the max blowup of annotations was incorrect."

* commit '93ee4913aa3d9a6b23f303fe0d03a0d07a8ae4d2':
  Fix a bug where the max blowup of annotations was incorrect.

am f3b60d2f: Merge "Fix a bug where the max blowup of annotations was incorrect."

* commit 'f3b60d2f07280dd6b0753223d4590e11e18a3985':
  Fix a bug where the max blowup of annotations was incorrect.

Merge "Fix a bug where the max blowup of annotations was incorrect."

Fix a bug where the max blowup of annotations was incorrect.

I'm not sure where the 1.34 number comes from but it's incorrect.
From the spec, the encoded_annotation is made up of a single byte
plus an unlimited number of uleb128 values. Each of these values
can double in width in the worst case. I received (personal) email
from one user who'd run into a case worse than the incorrect 1.34
limit.

Change-Id: I3b676e6d2b274aaa538ca61ce23945b3d49aff04

Fix an ugly bug where try/catch offsets weren't being mapped properly.

In dex files, there are several places where one object refers
to another object by its position in the file. The dex merger
is generally very careful to adjust these mappings when combining
dex files.

Unfortunately one of these cases was broken. Each try_item refers
to a corresponding encoded_catch_handler by its byte offset in a
list. Most of the time this byte offset is the same in the input
dex file and the output dex file. But encoded_catch_handlers are
variable-length because they use a variable-length uleb128 encoding
to address the type_idx being caught. When dex files are merged,
some exception types may go from having a small index to having
a large index, increasing the number of bytes required to encode
that index. This breaks our ability to directly copy over offsets
as we were doing previously.

Bug: http://code.google.com/p/android/issues/detail?id=36490
Change-Id: I3bdadf20899fdb5d4d074e69103b33c0404a31f8

Make dex merging stateless. do not merge.

This is required to run dx in Eclipse which doesn't unload the dx
library after each run.

(cherry-pick of 7aa5ce7e990dc3766eba97cd0932b62e4de21503.)

(cherry picked from commit 5e90165d5918f18aec0964bc475f4064e9143599)

Change-Id: Id5818f516539d13423b7cef4b2b1af30a9686b45

Support debug info in dexmerge. do not merge.

Bug: 4090053

Signed-off-by: Jesse Wilson <jesse@swank.ca>
(cherry picked from commit bc23c4f3ebaefebb3f1be7732767631f91e165ea)

Change-Id: Ic59d3cc1866c2e4328e406f22cb23fc08d5c6d91

Merge "Access to all users' external storage." into jb-mr1-dev

Merge "Shared OBB storage across users." into jb-mr1-dev

am 8e98f776: am 3e46814d: Merge "Don\'t crash in dvmQuasiAtomicsShutdown if we never ran dvmQuasiAtomicsStartup."

* commit '8e98f7761d88d0230305d1855e4f418fe3554398':
  Don't crash in dvmQuasiAtomicsShutdown if we never ran dvmQuasiAtomicsStartup.

am 3e46814d: Merge "Don\'t crash in dvmQuasiAtomicsShutdown if we never ran dvmQuasiAtomicsStartup."

* commit '3e46814d8984243612176c9b15d8d012dcca2143':
  Don't crash in dvmQuasiAtomicsShutdown if we never ran dvmQuasiAtomicsStartup.

Merge "Don't crash in dvmQuasiAtomicsShutdown if we never ran dvmQuasiAtomicsStartup."

Don't crash in dvmQuasiAtomicsShutdown if we never ran dvmQuasiAtomicsStartup.

dalvikvm --help would fail here:

  #0  __pthread_mutex_destroy (mutex=0x0) at pthread_mutex_destroy.c:28
  #1  0xf7b36e13 in dvmDestroyMutex (pMutex=0x0) at dalvik/vm/Thread.h:465
  #2  dvmQuasiAtomicsShutdown () at dalvik/vm/Atomic.cpp:188
  #3  0xf7b65781 in dvmShutdown () at dalvik/vm/Init.cpp:1889
  #4  0xf7b65f31 in ~ScopedShutdown (this=<synthetic pointer>, __in_chrg=<optimized out>) at dalvik/vm/Init.cpp:1317
  #5  dvmStartup (argc=11, argv=0x804b360, ignoreUnrecognized=false, pEnv=0x804b398) at dalvik/vm/Init.cpp:1548
  #6  0xf7b790e2 in JNI_CreateJavaVM (p_vm=0xffffc758, p_env=0xffffc75c, vm_args=0xffffc760) at dalvik/vm/Jni.cpp:3502
  #7  0x08048958 in main (argc=13, argv=0xffffc8b8) at dalvik/dalvikvm/Main.cpp:212

Change-Id: I8c833ca5ea454d9b2d1714a569fb728562088659

Remove unnecessary compiler pragma.

With dlmalloc 2.8.6 the compiler pragmas to suppress warnings are not
necessary.
Remove unnecessary dlmalloc dl prefix #define.

Depends upon: https://android-review.googlesource.com/42351

Change-Id: Idc45cee9342efec0aab63d5172b3308a0fca3c4f

Access to all users' external storage.

System services holding this permission have external storage bound
one level higher, giving them access to all users' files.

Bug: 7003520
Change-Id: Id95d6d5b2fa2ff75c0c94f18f81cb118f837f665

Revert "Revert "Trim pre-Zygote fork, fix under-estimate of heap limit.""

This reverts commit 4c7e27186d87ccd40646fd4f514dc982ff5b40b1.

Revert "Revert "Upgrade to dlmalloc 2.8.5.""

This reverts commit 729eebbb4e4ec5b826b7230b4c02267da341b70b.

Shared OBB storage across users.

To avoid downloading large OBB files separately for each user,
provide a shared view of /sdcard/Android/obb to all apps.  This new
shared view is read-only by default, but it's mounted writable if
an app requests the WRITE_EXTERNAL_OBB_STORAGE permission.

Bug: 7008879
Change-Id: Ibc6a3dbf360bf1b9d356bf52595201d155ff73b8

am 4014e152: am a5731795: Merge "Correctly handle switches on values in a high register"

* commit '4014e152c86a0a6929382f8ac767d572ada21042':
  Correctly handle switches on values in a high register

am a5731795: Merge "Correctly handle switches on values in a high register"

* commit 'a57317952de3fc37876f97e8655cc773b97df891':
  Correctly handle switches on values in a high register

Merge "Correctly handle switches on values in a high register"

am cc686898: am f6104618: Merge "Fix a bug where debugInfos\' size wasn\'t being computed."

* commit 'cc68689852b84b244ca393d748836f54cae5b5a7':
  Fix a bug where debugInfos' size wasn't being computed.

am f6104618: Merge "Fix a bug where debugInfos\' size wasn\'t being computed."

* commit 'f610461867348df0c5a624d75f39b34071044fb2':
  Fix a bug where debugInfos' size wasn't being computed.

Merge "Fix a bug where debugInfos' size wasn't being computed."

am f76ab40f: am ecda5d85: Merge "Fix for 6994917 GC is slower since JOO33B per FrameworkPerf test"

* commit 'f76ab40f7065b4e0202b83b8a7a0c4175d695801':
  Fix for 6994917 GC is slower since JOO33B per FrameworkPerf test

am ecda5d85: Merge "Fix for 6994917 GC is slower since JOO33B per FrameworkPerf test"

* commit 'ecda5d8585466d63f185ff09047643e9bd9cb261':
  Fix for 6994917 GC is slower since JOO33B per FrameworkPerf test

Merge "Fix for 6994917 GC is slower since JOO33B per FrameworkPerf test"

Revert "Trim pre-Zygote fork, fix under-estimate of heap limit."

This reverts commit 2f8d5e11f735e9ee8e235c31b3f113f6c90ad63c.

am 6464661f: am b48569ac: Merge "Fix build warning."

* commit '6464661ff5ca7dd83f60f68e7b40af0ed3ec8760':
  Fix build warning.

am bd561cbf: am a9dd9337: Merge "Trim pre-Zygote fork, fix under-estimate of heap limit."

* commit 'bd561cbffebbff1a8b17e031841ab2fcd2777cff':
  Trim pre-Zygote fork, fix under-estimate of heap limit.

am b48569ac: Merge "Fix build warning."

* commit 'b48569acbb7b3702f1071befa5ebfeae7a46303b':
  Fix build warning.

Merge "Fix build warning."

Fix build warning.

Change-Id: I1f9813c9dc1dfc388f82f37584fe60256b6a8b12

am a9dd9337: Merge "Trim pre-Zygote fork, fix under-estimate of heap limit."

* commit 'a9dd9337605a610841c86fe1bd90a8fa9b8d28c9':
  Trim pre-Zygote fork, fix under-estimate of heap limit.

Merge "Trim pre-Zygote fork, fix under-estimate of heap limit."

Fix for 6994917 GC is slower since JOO33B per FrameworkPerf test

Fixes the GC aspect of the regression in FrameworkTest for Gc.
Large bitmap test seems to be unaffected.

Changes are:
We now have minimum sleep of 1ms instead of 10ms in dvmSuspendAll.
Some heuristics to avoid doing extra GCs if an allocation waited for a concurrent Gc.

Change-Id: I3797c7dc813e26bd1f7bb0a9dbe55eb70dd1c9c9

Merge "Fix build warning." into jb-mr1-dev

Fix build warning.

Change-Id: I1f9813c9dc1dfc388f82f37584fe60256b6a8b12

One more errno for encrypted devices.

Bug: 7017585
Change-Id: I4c2cc95a5c3a44bab4a93f3a1f8907869f0ce8e6

Fix a bug where debugInfos' size wasn't being computed.

Previously debugInfos' size of 0 caused a malformed dex file
because debug data was present but not mentioned in the table
of contents.

Change-Id: I07171aaee12fef9f303fc505909f44ef1a714114

Trim pre-Zygote fork, fix under-estimate of heap limit.

Card scanning uses dvmHeapSourceGetValue to scan from the heap base to
the total foot print of the heap. Trimming the Zygote may lower the
footprint of the heap by creating a hole in the middle. This hole is
unaccounted for in the card scanning leading to missed dirty cards.

This was a regression introduced by:
https://android-review.googlesource.com/#/c/41728/
as it didn't #define MORECORE_CANNOT_TRIM 1 that was previously defined
here:
https://android-review.googlesource.com/#/c/41731/4/libcutils/mspace.c

Introduce a notion of dvmHeapSourceGetLimit so that card scanning
doesn't rely on footprint. Trim the heap before forking the Zygote. This
change continues to enable morecore trimming of mspaces.

Change-Id: I3f0b12371097f50a1f920cac25846ca08685e7ed

Correctly handle switches on values in a high register

When dx generates a switch on a value in a high register, it prepends a move
instruction, to move the value to a low register. However, this causes the
switch data pseudo-instruction to use address of the move instruction as the
base address of the switch targets, rather than the address of the switch
instruction

Change-Id: I0708d35ec3059c30221cd0a64f426244411d31ea

Merge "Iteration on multi-user external storage." into jb-mr1-dev

Iteration on multi-user external storage.

Zygote now mounts a tmpfs over any existing /storage contents, giving
it a clean slate for its bind mounts.  This supports a symlink that
enables legacy /sdcard paths for shell users.  Move to using paths
defined from environment variables, instead of header files.

Bug: 6925012
Change-Id: Ia4a2ac35ecfe98b617c27ff3bd73deb29e395e55

Revert "Upgrade to dlmalloc 2.8.5."

This reverts commit bba37bd191843ef29ef9c7a8839e98b73debfffa.

Revert "Ensure we fence the writing of clazz to an Object post allocation."

This reverts commit 821ab588af4566a800d4b5608092ccc1dd3764af.

Ensure we fence the writing of clazz to an Object post allocation.

We hold the heap lock during the calloc of an object in the GCed heap.
We do a non-volatile store following this for the clazz, this may lead to a
race where an object is allocated but appears to have no clazz.

Found investigating Bug: 7028104

Change-Id: I390ad4c7b9632127bd27393742be347b84d9269a

am 1129f6a7: am b931f906: Merge "Upgrade to dlmalloc 2.8.5."

* commit '1129f6a78708a8d8861f8053616c3961aefdc1ef':
  Upgrade to dlmalloc 2.8.5.

am b931f906: Merge "Upgrade to dlmalloc 2.8.5."

* commit 'b931f9065990520bf34b8c6128e069b4cca4a7ec':
  Upgrade to dlmalloc 2.8.5.

Merge "Upgrade to dlmalloc 2.8.5."

Upgrade to dlmalloc 2.8.5.

Switch to using dlmalloc 2.8.5. Define mspace functionality directly
using dlmalloc rather than taking from libcutils.
Remove growth limit check in tryMalloc that only checks initial growth
limit.
Implement trimming at the end of the mspace.

Depends upon: https://android-review.googlesource.com/41717

Change-Id: Ia2c6b50bdb0b0d5aae4b18deefbd1bf50dfa49d5

Merge "Multi-user external storage support." into jb-mr1-dev

Multi-user external storage support.

When a specialized zygote process needs external storage access,
create a private mount namespace and create a bind mount to give
them access.  Support both single-user and multi-user modes, which
dictate the bind mount source.

Bug: 6925012
Change-Id: Ib987ba09f59ce03a6aeb9ee8bcb522965a2dfb67

am 031072f0: am 57573862: Merge "hardening: eliminate all text relocations from lidbvm"

* commit '031072f007d3ffd0d2facd6277005f2dac93c19c':
  hardening: eliminate all text relocations from lidbvm

am 57573862: Merge "hardening: eliminate all text relocations from lidbvm"

* commit '57573862ca6b377c73958d5ba876a406445b4490':
  hardening: eliminate all text relocations from lidbvm

Merge "hardening: eliminate all text relocations from lidbvm"

hardening: eliminate all text relocations from lidbvm

This patch consists of:
- changes to mterp/ that turn all literals from absolute
  to PC relative, so the relocations can be resolved at
  (build) link time
- changes to compiler/template/ that result in the
  compiler templates to live in the non-executable
  .data.rel.ro section (this code is never executed
  directly, only from the jit heap, so there is no
  reason to put it in the .text section)

Change-Id: I2dc97bd4720b393a74b7277a188f0c7b681fc932
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@gmail.com>

am 219a3f5f: am 21212cbd: Merge "Fix ALOGV bit rot."

* commit '219a3f5fdb4a8b85646037e2c09b3c7e408b1ec8':
  Fix ALOGV bit rot.

am 21212cbd: Merge "Fix ALOGV bit rot."

* commit '21212cbd65aa7ec7d69f0c6f5b98794045f750a5':
  Fix ALOGV bit rot.

Merge "Fix ALOGV bit rot."

Fix ALOGV bit rot.

Change-Id: Ia182af8f5ca3967a21003b3d1513145f8e6e5db0

am aa1d0261: am 464388e8: Merge "Don\'t crash on invalid string/type ids."

* commit 'aa1d026167c9024bf08784527b5c4176ee8c6523':
  Don't crash on invalid string/type ids.

am 464388e8: Merge "Don\'t crash on invalid string/type ids."

* commit '464388e8756d362f91d110b5741ba785ea3d44bd':
  Don't crash on invalid string/type ids.