Replace all instances of intptr_t with uintptr_t.

When compiling 32 bit binaries against a 64 bit binder interface,
implicit promotions of intptr_t types to uint64_t for fields in the
binder ioctl structures can result in invalid pointers because of sign
extension.

Fix signed/unsigned comparison warnings in driver interface test.

Merge "surfaceflinger: use Mutex timedLock instead of tryLock loop"

Merge "Revert "Update protobuf library from 2.3 to 2.6.""

Revert "Update protobuf library from 2.3 to 2.6."

This reverts commit a42cdf169060e1bb65c77bc8a394d266403d8583.

Change-Id: I8f75e6fcc921041ebdfd35c4724134fcb2c739c6

Merge "Update protobuf library from 2.3 to 2.6."

Merge "Fix message when a command times out."

surfaceflinger: use Mutex timedLock instead of tryLock loop

Rather than trying to acquire the state lock without waiting three
times at 1 second intervals in SurfaceFlinger::dump(), just try to
acquire the lock once with a 1 second timeout. Avoids spurious mutex
acquire failures that lead to flaky
com.android.cts.jank.opengl.CtsHostJankOpenGl results.

Bug: 18842510
Change-Id: I00ce6109647de2aef8831dd2f8fa98652ba7f4e0

Fix message when a command times out.

The previous version printed out the nanoseconds elapsed, not seconds.

Change-Id: I10f6d7bf7b7a4647658313ed83781fda6fd744a5

Merge "We have POSIX clocks, and we don't need to ask for the openssl headers."

We have POSIX clocks, and we don't need to ask for the openssl headers.

This code doesn't need to run on the Mac, and the BoringSSL transition
will be easier without references to external/openssl.

Change-Id: I14ea11fa2911af0f4e95e69636e6a2c26126cbe0

Update protobuf library from 2.3 to 2.6.

Remove generated cpp/h files for gltrace.proto as we can use the build
system to generate these dynamically.

Change-Id: I949d32fb5126f1c05e2a6ed48f6636a4a9b15a48

Merge "Installd: Disallow relocation for decryption"

Installd: Disallow relocation for decryption

When decrypting a device, a tmpfs is temporarily mounted as /data,
the size of which is usually small. When the zygote, system server
and necessary apps are brought up, they will be compiled into the
tmpfs.

If the system image contains prebuilts, they will be relocated instead
of compiled. This is unnecessary. In this special situation it is
acceptable to run out of the prebuilt oat files without relocation,
which can save space in the tmpfs.

This patch ensures that apps are not being relocated.

Change-Id: I42bfb7e3039574b7e4f2772e0d395f093d59ed1b
Signed-off-by: Hyangseok Chae <neo.chae@lge.com>

Merge "InputDispatcher: Optimize count()"

Merge "Installd: Swap file for dex2oat"

Installd: Swap file for dex2oat

Add swap file support for dex2oat to installd. Only use in low-memory
mode.

Bug: 18596910

(cherry picked from commit f68ce4de7fcfcc090af7985972b304eed7dbaf63)

Change-Id: I131448f3907115054a592af73db86d2b9257ea33

Merge "Add binder tests"

Add binder tests

Change-Id: Ib0b38708dd6387fc4d5a4857ca74a217f3edad1a

Merge "Add binder driver interface test"

Add binder driver interface test

Test ioctls and some binder commands without using libbinder.

Change-Id: I86dde56ff1380bca17d086021ac5324dc5210354

Merge "Installd: Remove dalvik paths"

Installd: Remove dalvik paths

Dalvik is dead. Dexopt is dead.

Change-Id: I9ff6b638a3e2c52ae7aec33d7b08a7bee012854f

Merge "Fix clang -Wc++11-narrowing warnings."

Fix clang -Wc++11-narrowing warnings.

Clang compiler gives warning to implicit conversions of
literal structure fields, although the compiler generated
implicit conversions are what we want in these cases.

Change-Id: I8f04e7f2fa1db0ed1d1937135d9091532eec3d5b

Merge "GUI: Remove unused variables"

GUI: Remove unused variables

For build-system CFLAGS clean-up, remove unused variables.

Bug: 18632512
Change-Id: I79fdcf830683d7278f4bf114cfb76a26da1b88ff

Merge "libgui: Enable -Weverything and -Werror"

Merge "Libui: Remove unused variables"

Libui: Remove unused variables

For build-system CFLAGS clean-up, remove unused variables.

Bug: 18632512
Change-Id: Ib78912ae442ddc6410960ab1efb4fb2faa1a9208

libgui: Enable -Weverything and -Werror

Enables -Weverything and -Werror, with just a few exceptions for
warnings we can't (or shouldn't need to) work around.

This is a squashed commit based on an initial change with a couple of
fixes to avoid breaking certain targets. The source commits are:
  d723bd7669b4fc88dc282d8bf8ba5ecb2849d22f
  00d504c06ea034befe143e6b8cb34d004670ed02
  429ba89cd293633d2f882165066b422c96597df2

Change-Id: I034abec27bf4020d84af60d7acc1939c59986dd6

Merge "Fix crash when user provides large values in the Parcel."

Fix crash when user provides large values in the Parcel.

Bug: 18102648
Change-Id: Ie6a24718e586a34424238363de80f9545951514f
(cherry-picked from commit 8afa1c4ab86d724feb7716e153b7835385534590)

Merge "Fix -Werror on 64-bit builds"

Merge "Disable struct padding warning/error"

Merge "libui: Enable -Weverything and -Werror"

Merge "Enable clang for libui/libgui/surfaceflinger"

Merge "Fix for corruption when numFds or numInts is too large."

Fix -Werror on 64-bit builds

Add another static_cast to avoid an error on 64-bit builds

Change-Id: I62fb2c67766c4aab70d9d1834f255d06ff82dd8d
(cherry picked from commit 116c7a113cb2db4d3f7708627ac4713c7e92cd43)

Disable struct padding warning/error

Adds -Wno-padded, which was triggering on 64-bit devices

Change-Id: If27bac27881f2081a222b32213901f5087554b07
(cherry picked from commit 4e84bd310c876e1450077959333c87e6e83dbf63)

libui: Enable -Weverything and -Werror

Enables -Weverything and -Werror, with just a few exceptions for
warnings we can't (or shouldn't need to) work around.

Change-Id: I470a267e9b1c373f2d5c960f005c3b0e02b2db63
(cherry picked from commit 303b9a51239d36d237a7d40c67b5085cdb3c1059)

Enable clang for libui/libgui/surfaceflinger

Enables clang and C++11 for libui/libgui/surfaceflinger, and
eliminates all compile-time warnings.

Change-Id: Ie237fdb5ae44f2bfcddaa884f9c65ec3f08ae50f
(cherry picked from commit f10c46ef855b6410b20ebd8b1351d4d78d8eca8e)

Fix for corruption when numFds or numInts is too large.

Bug: 18076253
Change-Id: I4c5935440013fc755e1d123049290383f4659fb6
(cherry picked from commit dfd06b89a4b77fc75eb85a3c1c700da3621c0118)

Merge "OpenGL tests: Fix unused variables"

Merge "Avoid overwriting EGL14.EGL_NO_SURFACE"

Merge "binder: Add {read,write}Uint32 methods to Parcel"

binder: Add {read,write}Uint32 methods to Parcel

Adds readUint32 and writeUint32 methods to the Parcel class. This
saves a lot of static_casting in anything implementing a Binder
interface on the native side.

Change-Id: Iafc73b0633654a3a4c49767f41806b56906c924f

Merge "binder: Return pid_t/uid_t instead of int"

binder: Return pid_t/uid_t instead of int

Makes IPCThreadState::getCalling{Pid,Uid} return a more
appropriate type.

Change-Id: I97f8a83c1c0722bc1afbf8a6df1a288f8f1f0d2c

InputDispatcher: Optimize count()

Count items as they are added and removed rather than iterating over the
entire list to count them.

Increases performance slightly, particularly when tracing is turned on,
which causes count to be queried more often, and when the number of
items in the queues grow large.  This can happen due to applications not
responding, for example.

Change-Id: I0f11f7edd46089612af910cdfabfeb3ee685d7d9

Merge "Don't error out because SensorEventConnection::dump overloads BBinder::dump"

Don't error out because SensorEventConnection::dump overloads BBinder::dump

clang warns about SensorEventConnection::dump overloading BBinder::dump
in some conditions.

Since the cause isn't fixable without changing the API, tell clang not
to error out on it.

Change-Id: I15659de7c5499d937019599cfefb01305039e6b5
Signed-off-by: Bernhard Rosenkränzer <Bernhard.Rosenkranzer@linaro.org>

Merge "Fix build with clang in C++11 mode"

Fix build with clang in C++11 mode

Now that we're in C++11 mode by default, clang complains about switch
statements with case values not matching the type -- since some
binder_driver_return_protocol values are > 0x7fffffff, we need to make
the switch statements operate on uint32_t rather than int32_t.

BUG: 18466763
Change-Id: Iedbfd5c7a3d3d9f087d2eab4ff21343ad7a2a448
Signed-off-by: Bernhard Rosenkränzer <Bernhard.Rosenkranzer@linaro.org>

OpenGL tests: Fix unused variables

For build-system CFLAGS clean-up, fix unused variables.

Change-Id: I1863418f7467412d7698dd0ce0c8e40d1bd0348e

Avoid overwriting EGL14.EGL_NO_SURFACE

Both EGL14 and EGLExt have the same initialization codes for
EGL14.EGL_NO_SURFACE, EGL14.EGL_NO_CONTEXT and EGL14.EGL_NO_DISPLAY.
Since EGLExt is initialized later, they are overwritten by EGLExt's
initialization codes.
Therefore, EGL_NO_SURFACE returned by methods in EGL14 is not
actually EGL14.EGL_NO_SURFACE object and it makes several problems in
handling error cases.
For instance, "Let's fish" game application cannot be run on L.

To solve the problem, this patch makes EGL14.EGL_NO_SURFACE,
EGL14.EGL_NO_CONTEXT and EGL14.EGL_NO_DISPLAY initialized just once.

Change-Id: Icce878164ff0b715ad2b5a2cd038a9616c7cf1e9

Merge "Fix potential format string vulnerability"

Fix potential format string vulnerability

A sensor name containing a format string could have interesting side
effects...

Change-Id: If7f1378aa68572d9716c339728eab18faa6b9f2a
Signed-off-by: Bernhard Rosenkränzer <Bernhard.Rosenkranzer@linaro.org>

Merge "Binder: Remove unused functions & variables"

Binder: Remove unused functions & variables

For build-system CFLAGS clean-up, remove unused functions and
variables.

Change-Id: I88abc2bc99e6b91c7aa56470ff9671bf8bf29f8d

Merge "Mark 2 unused paramters as such"

Mark 2 unused paramters as such

Mark 2 unused paramters to prevent clang from erroring out

Change-Id: Ifc9b4a5b820bd6318e9259e737e708d25fdb174b
Signed-off-by: Bernhard Rosenkränzer <Bernhard.Rosenkranzer@linaro.org>

Merge "We HAVE_LITTLE_ENDIAN."

We HAVE_LITTLE_ENDIAN.

Change-Id: I68551ad663f2649a06b1958e9bef152e6d026e72

Merge "Fix installd command line parser"

Merge "C++11 compatibility."

C++11 compatibility.

 * Explicit conversion for atomic_uintptr_t initialization.
 * Fix string literal concatenation to not be a UD literal.
 * Use __typeof__ instead of typeof (should become decltype once this
   actually moves to C++11).

Bug: 18466763
Change-Id: I4eedddfb945a2a703ed27317cb6e2b3041b1ebfc

Merge "Kill INCLUDE_SYS_MOUNT_FOR_STATFS."

Kill INCLUDE_SYS_MOUNT_FOR_STATFS.

Change-Id: I692179cae9d476dd5a0b5fa92813b61934dd7c87

Merge "Add an installd command to mark the end of boot."

Add an installd command to mark the end of boot.

bug: 18280671

(cherry picked from commit b1a6c2f95a455aa2829340fcc8d233493855f4d4)

Change-Id: I97542875ffff1b68ab324e9b04fb9266c82633cc

Merge "Remove comparisons that can't be true"

Merge "Fix class vs. struct mismatch"

Fix class vs. struct mismatch

egl_connection_t is a struct - shouldn't be forward-declared as a class

Change-Id: I39f5d9bff9fc64f4694fb2bef7e4afa131b449cc
Signed-off-by: Bernhard Rosenkränzer <Bernhard.Rosenkranzer@linaro.org>

Remove comparisons that can't be true

keyBitmask and ledBitmask are arrays, "!their address" always evaluates to
false. clang complains about this.

Change-Id: I6aed45a203b0eeaad2093dbdebea03a2c2fc1a3d
Signed-off-by: Bernhard Rosenkränzer <Bernhard.Rosenkranzer@linaro.org>

Merge "Replace IInterface::asBinder() with a static"

Replace IInterface::asBinder() with a static

so we can do NULL checks again, and update calls to IInterface::asBinder()
to use the new static version.

Change-Id: Ia7b10eb38ca55b72278bfd33d3bf647f338b4e6a

Merge "Move frameworks/native to libc++."

Move frameworks/native to libc++.

I'm pretty sure I had done this before... Must have been new
dependencies that slipped in when L merged in.

Bug: 15193147
Change-Id: Ib9b1b8a65665d25c33c6141b9476a8e13c4084e0

Merge "Add support for dalvik.vm.isa.<instruction set>.variant"

Merge "kill HAVE_FORKEXEC"

Merge "Frameworks/native: Wall Werror in batteryservice & powermanager"

Merge "Frameworks/native: Wall Werror in surfaceflinger"

Frameworks/native: Wall Werror in batteryservice & powermanager

Turn on -Wall -Werror in services/batteryservice and services/powermanager.
Fix warnings.

Change-Id: I1da667083013379cd8240377c48d63fa5c53033d

Frameworks/native: Wall Werror in surfaceflinger

Turn on -Wall -Werror in services/surfaceflinger. Fix warnings.

Change-Id: Ifef830300d9d4bc657d8f1257c02bfe8c2b4d9c5

kill HAVE_FORKEXEC

Bug: 18317407
Change-Id: I0c7b5886e5de12bdd95cfcc046bc87e4a6aeef5b

Merge "Frameworks/native: Add pragmas to glgen headers"

Frameworks/native: Add pragmas to glgen headers

The code currently being generated by glgen is not the cleanest, but
we would like to get core/jni onto -Werror. Thus add pragmas turning
off warnings locally to the C headers.

Also fix signed-comparison in two functions.

Longer term TODO: Emit clean code.

Change-Id: Iee8582f8c0c1de076d64851d3b6ca467afd5bc43

Merge "EGL: make x86/64 wrappers resistant to -fno-omit-frame-pointer flag"

EGL: make x86/64 wrappers resistant to -fno-omit-frame-pointer flag

Wrapper functions were broken on builds that forced -fno-omit-frame-pointer flag.

Change-Id: I5a80f9587fb3db821b4156af56acda59a0b4579b
Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Signed-off-by: Yong Yao <yong.yao@intel.com>

Add support for dalvik.vm.isa.<instruction set>.variant

Bug: 18056890
Change-Id: Ia266fa9f2a79158ced82016106aa1cd2529259c9

Merge "Add missing include to fix libc++ build."

Add missing include to fix libc++ build.

This was previously committed, but I guess it got lost in the merge
queue.

Bug: 15193147
Change-Id: Ic0e5d90ab403fb833d6a3bb7c55d279c56534869

Fix installd command line parser

Parser does not take into account that the last argument can be empty.
In this case it will skip end of line marker and continue parsing.

Change-Id: I2d029a69ed22cef497893e592ff3275ced97f85f
Signed-off-by: Serguei Katkov <serguei.i.katkov@intel.com>

am d12586b8: Merge "Move hwc tests to libc++."

* commit 'd12586b8593cf0cb894d02104a9794ed9dd85d16':
  Move hwc tests to libc++.

Merge "Move hwc tests to libc++."

Move hwc tests to libc++.

Since these tests were pulling in a static library linked against
stlport, the tests needed to be explicit about which STL they're
using. Might as well be libc++.

Bug: 15193147
Change-Id: I4e0fc4ea10f7982dd04e7baaf57139dd5d55543e

am dbab409c: Merge "Avoid two warnings from clang."

* commit 'dbab409c49ca5dd6686912d0b3eeb19ef1c37a3e':
  Avoid two warnings from clang.

Merge "Avoid two warnings from clang."

Avoid two warnings from clang.

Clang compiler warns about undefined static functions like getReturnString
before all references are removed inside IF_LOG_COMMANDS.
To remove such warnings, we can leave those static functions defined
and the compiler will remove them at the end when they are not used.

Add a cast to long before converting from int to void*,
to avoid type cast warnings on 64 bit targets.

BUG: 17043248
Change-Id: Ica31101522279d2c8eacc90e2939767002fb827b