am d7486c9c: Merge "Guard against negative buffer size"

* commit 'd7486c9cd3ff87cfda193cf5fca1d2f5ec0f7316':
  Guard against negative buffer size

am b86347c7: Merge "Guard against negative width and height"

* commit 'b86347c7f68c9739fcddf1338cc4fcd6ca9b3e2a':
  Guard against negative width and height

am 7f075373: Merge "Handle empty buffer in QemuPipeStream::writeFully"

* commit '7f07537302645d942da3f08c65f8f0a2bb842748':
  Handle empty buffer in QemuPipeStream::writeFully

Merge "Guard against negative buffer size"

Merge "Guard against negative width and height"

Guard against negative buffer size

When negative buffer sizes are passed, emulator crashes. This commit
checks the buffer size and handles properly.

Change-Id: I223d5f5bcde2455e4b39354f9527dceb193b8e47

Guard against negative width and height

When negative width or height passes into pixelDataSize, some bogus
values are returned. This commit checks both width and height to make
sure they are valid.

Change-Id: Ia49a9f464e6bb4894207ca4e07bbfdf34fbe741d

Handle empty buffer in QemuPipeStream::writeFully

Print error message and return 0 when this happens (and it does happen),
similar to what QemuPipeStream::readFully does.

Change-Id: I14f2f2877e6b22e676d5bfe34a54435220e46127

am 9c6d3726: Merge "Fix emulator crashes on glDrawElements command"

* commit '9c6d37269b85c41203a2f061958922315d55f6ce':
  Fix emulator crashes on glDrawElements command

Fix emulator crashes on glDrawElements command

The current implementation does not properly handle the case when
glDrawElements is called without a buffer binded. This commit
fixes this problem and fixes the crash.

Change-Id: Ic7a5b2fb90c860dad878ed53a369349d479ba4e5

am 7fa2b92d: Merge "Use more appropriate type in cb_handle_t::validate."

* commit '7fa2b92de2058a3556e5f38a135de415bb3d5e45':
  Use more appropriate type in cb_handle_t::validate.

Use more appropriate type in cb_handle_t::validate.

The interface specified by the HWC hal includes const native_handle
(through a typedef) which makes it annoying to const_cast away in order
to call validate.

Validate does not change the handle so this is safe.

Change-Id: I78b2997eb24074c186ab2d497b2faf332c7f4a85

am 6dc9b1b6: Merge "Make the size of "cb_handle_t" same for 32/64-bit" into lmp-dev

* commit '6dc9b1b64a3a01ddcce9e664dc0153021a595130':
  Make the size of "cb_handle_t" same for 32/64-bit

am 6c6a12d1: Remove display initialization from eglGetProcAddress API

* commit '6c6a12d1652c8872ae25162040224133888bf97a':
  Remove display initialization from eglGetProcAddress API

Merge "Make the size of "cb_handle_t" same for 32/64-bit" into lmp-dev

Remove display initialization from eglGetProcAddress API

We don't need to do the initialization in the eglGetProcAddress API. The work
can be finished in the eglInitialize API. When zygote pre-loading resources,
eglGetProcAddress will be called. If we let eglGetProcAddress do the display
initialization, zygote will get the "qemu-pipe" handle of the "opengles"
service. This is bad, because all the java processes which forked from zygote
will get the same handle from zygote. Then the host service cannot distinguish
from the device threads, and this will lead the emulator to crash.

Signed-off-by: Tina Zhang <tina.zhang@intel.com>
Signed-off-by: WeixingX Tian <weixingx.tian@intel.com>
(cherry picked from commit 7a2929cd6548693b581c17ba69308a49e03c56be)

Change-Id: I156970e004172e0bbe23a1a0c6c6b86850fd2ba6

Make the size of "cb_handle_t" same for 32/64-bit

cb_handle_t is usually created by Surfaceflinger which is a 64-bit process on
64-bit platform, and used by client processes which could be 32-bit processes
or 64-bit processes. Since the "cb_handle_t" is like a protocol, we should make
its size same for both 32-bit and 64-bit processes.

Signed-off-by: Tina Zhang <tina.zhang@intel.com>
(cherry picked from commit 3f62b50d45441ae077c958423e9e301acd5dcc11)

Change-Id: I12aace9446e4566d9b12b0f50970ec00e8537dd9

am 638ee524: Merge "Remove display initialization from eglGetProcAddress API"

* commit '638ee524f83053613c47ddea22c4bf98a0175c2f':
  Remove display initialization from eglGetProcAddress API

Merge "Remove display initialization from eglGetProcAddress API"

Remove display initialization from eglGetProcAddress API

We don't need to do the initialization in the eglGetProcAddress API. The work
can be finished in the eglInitialize API. When zygote pre-loading resources,
eglGetProcAddress will be called. If we let eglGetProcAddress do the display
initialization, zygote will get the "qemu-pipe" handle of the "opengles"
service. This is bad, because all the java processes which forked from zygote
will get the same handle from zygote. Then the host service cannot distinguish
from the device threads, and this will lead the emulator to crash.

Change-Id: I74ddc0db5c91c84afe0916f118937ef8f891b162
Signed-off-by: Tina Zhang <tina.zhang@intel.com>
Signed-off-by: WeixingX Tian <weixingx.tian@intel.com>

am 9b1e6c5a: Merge "Make the size of "cb_handle_t" same for 32/64-bit"

* commit '9b1e6c5a3ccd943b7158dce83387fbfcc1162d69':
  Make the size of "cb_handle_t" same for 32/64-bit

Make the size of "cb_handle_t" same for 32/64-bit

cb_handle_t is usually created by Surfaceflinger which is a 64-bit process on
64-bit platform, and used by client processes which could be 32-bit processes
or 64-bit processes. Since the "cb_handle_t" is like a protocol, we should make
its size same for both 32-bit and 64-bit processes.

Change-Id: I11011576a389db2662888abfaf62dfb8e5611b7c
Signed-off-by: Tina Zhang <tina.zhang@intel.com>

am cdd78a38: am aec15d91: am 646f0b25: Merge "opengl: rcOpenColorBuffer must be synchronous"

* commit 'cdd78a386dff8438a66d40abb4fa9c639c57b89c':
  opengl: rcOpenColorBuffer must be synchronous

am dc684041: am c04a27cb: opengl: rcOpenColorBuffer must be synchronous

* commit 'dc684041d9fbfd3b7db6fb0294286764d92564d0':
  opengl: rcOpenColorBuffer must be synchronous

am aec15d91: am 646f0b25: Merge "opengl: rcOpenColorBuffer must be synchronous"

* commit 'aec15d91ee48797f2b3c3fe61cca977900ebcd47':
  opengl: rcOpenColorBuffer must be synchronous

am 646f0b25: Merge "opengl: rcOpenColorBuffer must be synchronous"

* commit '646f0b25e63164950d1d99b9b8021be74c2321c6':
  opengl: rcOpenColorBuffer must be synchronous

am c04a27cb: opengl: rcOpenColorBuffer must be synchronous

* commit 'c04a27cb9a25ba26893212914fd725a85f8d6694':
  opengl: rcOpenColorBuffer must be synchronous

opengl: rcOpenColorBuffer must be synchronous

The gralloc register_buffer() function, which calls rcOpenColorBuffer,
must actually increment the reference count before returning.
Otherwise the buffer allocator may release its reference before the
client has obtained one, and the buffer will be freed prematurely.
Since rcOpenColorBuffer was just sending a message to the host without
waiting for it to be received/processed, this guarantee was not met.
Adding a return value makes the call synchronous.

Bug: 12988668
Change-Id: I8b2399cfb0f600f99b3387f630343291b59bc9a6

opengl: rcOpenColorBuffer must be synchronous

The gralloc register_buffer() function, which calls rcOpenColorBuffer,
must actually increment the reference count before returning.
Otherwise the buffer allocator may release its reference before the
client has obtained one, and the buffer will be freed prematurely.
Since rcOpenColorBuffer was just sending a message to the host without
waiting for it to be received/processed, this guarantee was not met.
Adding a return value makes the call synchronous.

Bug: 12988668
Change-Id: I8b2399cfb0f600f99b3387f630343291b59bc9a6

opengl: Fix const-ness of GLES prototypes to match new headers

Bug: 1502849
Change-Id: I902517c23354e14f9c63b1584db286d682315bb5

am dbda557d: am 5cb5779a: Merge "gralloc: don\'t initialize reserved_proc"

* commit 'dbda557d4dcf022850ea4ab4f89e1c154e2aea04':
  gralloc: don't initialize reserved_proc

am 5cb5779a: Merge "gralloc: don\'t initialize reserved_proc"

* commit '5cb5779ad0ad75a59bfb8ce42317a4ee18300281':
  gralloc: don't initialize reserved_proc

gralloc: don't initialize reserved_proc

gcc supports designated initializers in C++ mode as an extension, but
*only* when all of the specified fields are at the beginning of the
struct.  Hence initializing reserved_proc breaks compilation when new
procs are added.

Initializing reserved_proc makes a (spurious) warning go away, so
instead directly suppress that warning.

Change-Id: I279b7070b1aa7068c23fb59422589ff31980432d
Signed-off-by: Greg Hackmann <ghackmann@google.com>

am 67334d0b: Merge commit \'d9dc763708823aa865983bb8dc37b5ecc5669620\' into HEAD

* commit '67334d0b9d0da7595daf07a3e9bea6a55436e296':

Merge commit 'd9dc763708823aa865983bb8dc37b5ecc5669620' into HEAD

am 7076e464: Merge "Add 64-bit software GLES/EGL support in gralloc.goldfish module"

* commit '7076e4647af737393baf020c13e81a090f4a9f26':
  Add 64-bit software GLES/EGL support in gralloc.goldfish module

Add 64-bit software GLES/EGL support in gralloc.goldfish module

If the emulator chooses running without using the host GPU to
do the GLES/EGL acceleration, the gralloc.goldfish module
must let the emulator use gralloc.default module which
can support the software GLES/EGL. Add this support for
64-bit emulator.

Change-Id: I241e70ce26a8f0298a32e3a55037fc3028a2e2a7
Signed-off-by: Tina Zhang <tina.zhang@intel.com>

am 61a8f0bb: Merge "Fix the Chrome app crash in emulator issue"

* commit '61a8f0bbc146a4050a885e1e7c314dfc4ab72779':
  Fix the Chrome app crash in emulator issue

Merge "Fix the Chrome app crash in emulator issue"

am bd93fee7: Merge "Support GLES acceleration in 64bit emulator on device side"

* commit 'bd93fee73b02d0b223b441c34a4f8e04eb57291d':
  Support GLES acceleration in 64bit emulator on device side

Merge "Support GLES acceleration in 64bit emulator on device side"

Fix the Chrome app crash in emulator issue

The TLS desctuctor callback function eglReleaseThread needs to
access EGLThreadInfo but it has been released by tlsDestruct.
That causes the crash issue when Chrome app is opened.
Change the EGLThreadInfo pointer value to NULL just after
EGLThreadInfo object is released, can fix the problem.

Change-Id: I7cdc7bdcd602af00369e570b0d2be168b76143a6
Signed-off-by: Tina Zhang <tina.zhang@intel.com>

am 4f94a5f8: Merge "Remove the useless folder and files in emulator GLES/EGL drivers"

* commit '4f94a5f8b13a5fdd80988e77dd5d468b958b0117':
  Remove the useless folder and files in emulator GLES/EGL drivers

Remove the useless folder and files in emulator GLES/EGL drivers

Files in OpenglOsUtils folder are useless for emulator GLES/EGL drivers.
Remove them to simplify the content.

Change-Id: I0a3cb572328cfa22b8567e8cd938f2a228f6167a
Signed-off-by: Tina Zhang <tina.zhang@intel.com>

Support GLES acceleration in 64bit emulator on device side

Support both 32bit and 64bit emulator GLES acceleration.

Change-Id: Ibd8b970d23937ae40732324706074ff00f433a14
Signed-off-by: Tina Zhang <tina.zhang@intel.com>

am 48a76778: Merge "Remove #include <asm/page.h>."

* commit '48a76778937abadf5cbd260696a29d737134d0d9':
  Remove #include <asm/page.h>.

am 48a76778: Merge "Remove #include <asm/page.h>."

* commit '48a76778937abadf5cbd260696a29d737134d0d9':
  Remove #include <asm/page.h>.

Remove #include <asm/page.h>.

Change-Id: I1476efe9a706fe322ec23571c6375b847a0ccf2f

am 48107fa1: Merge "device/generic/goldfish: convert LOCAL_MODULE_PATH to LOCAL_MODULE_RELATIVE_PATH"

* commit '48107fa1dd354f8667b74f335c3f82f472b7d112':
  device/generic/goldfish: convert LOCAL_MODULE_PATH to LOCAL_MODULE_RELATIVE_PATH

device/generic/goldfish: convert LOCAL_MODULE_PATH to LOCAL_MODULE_RELATIVE_PATH

LOCAL_MODULE_PATH doesn't work for multiarch builds, replace it
with LOCAL_MODULE_RELATIVE_PATH.

Change-Id: I4e4ceec61d026bbe74ba604554c06104bde42e5e

am 59922432: Merge commit \'f09bc5256e48a44eec9ca47ba838c12d0e89a198\' into HEAD

* commit '59922432be859a46eeb76650b24308505625da36':

Merge commit 'f09bc5256e48a44eec9ca47ba838c12d0e89a198' into HEAD

am b9a62053: Merge commit \'828bd482a1aeacf96d5f3a3e0701029d9f890c5f\' into HEAD

* commit 'b9a62053cf74025a43c8baa118d52729e38cc176':

Merge commit '828bd482a1aeacf96d5f3a3e0701029d9f890c5f' into HEAD

am 2befc3ed: Merge "Modify the build targets for goldfish from to include aosp*."

* commit '2befc3edc583afb43a4f6740f0af0534eac8e9d5':
  Modify the build targets for goldfish from to include aosp*.

Modify the build targets for goldfish from to include aosp*.

This allows egl.cfg to be copied to $(TARGET_OUT)/lib/egl for aosp*
build targets. This patch fixes the emulator to boot with "-gpu on" flag.

Change-Id: I6c291a4a0fd82242d7f2ac41229be202aa7c71e0

am b9a7de1a: Merge "Delete buffer data when removing it from the SharedGroup namespace"

* commit 'b9a7de1a690bc638496e0550e6a9ebe859fc0315':
  Delete buffer data when removing it from the SharedGroup namespace

Delete buffer data when removing it from the SharedGroup namespace

Updating or deleting data associated with a buffer object was clearing
the name->data association, but not actually deallocating the data.

Thanks to manjian2006 for finding the bug and proposing the fix.

Bug: 60468
Change-Id: I477f1e67d33a486419127ac790f32ddbbf566089

am 3bd0e079: Merge "Return EGL_FALSE if eglChooseConfig failed to find any matching configs"

* commit '3bd0e079a9d5321f952c2871fb453775810879f7':
  Return EGL_FALSE if eglChooseConfig failed to find any matching configs

Return EGL_FALSE if eglChooseConfig failed to find any matching configs

Technically, we shouldn't have touched *num_config at all in this
case, but the remote interface to the host doesn't return a separate
boolean return value, only the number of matching configs.

This makes sure that callers that only check the return value of
eglChooseConfig notice a failure to find configs.

Change-Id: I7f660a4a5b8306184608a86c34329d3163ad05ba

get rid of HAL pixelformats 5551 and 4444

Change-Id: If69c5cdb02dd8308963165a09b5d7fe3b00077ae

Reconcile with jb-mr2-release jb-mr2-zeroday-release - do not merge

Change-Id: Ie82b1c1ca10b0d4e25b06ec917e23a3703177d5b

Flush QemuPipeStream when needed

Under some circumstances, close color buffer messages in the command stream
do not reach the host as the size of the undelivered commands is small.
This results in refcounts of ColorBuffer never dropping to zero and color
buffers not being freed.

The patch calls flush at the end of rcCloseColorBuffer_enc().
In addition, the stream is flushed before it is deleted in the destructor of
QemuPipeStream, since it may also have important messages.

bug: 9278661

Change-Id: I46f20af8e0c272ec8d831d62f57cb2f22c7b28a7

Flush QemuPipeStream when needed

Under some circumstances, close color buffer messages in the command stream
do not reach the host as the size of the undelivered commands is small.
This results in refcounts of ColorBuffer never dropping to zero and color
buffers not being freed.

The patch calls flush at the end of rcCloseColorBuffer_enc().
In addition, the stream is flushed before it is deleted in the destructor of
QemuPipeStream, since it may also have important messages.

bug: 9278661

Change-Id: I46f20af8e0c272ec8d831d62f57cb2f22c7b28a7

Gralloc/Camera3: Support new HAL_PIXEL_FORMAT_YCbCr_420_888 format

Enable flexible YUV format buffers from the camera.

 - Add gralloc alloc support for YCbCr_420_888, mapped to NV21
 - Add gralloc lock_ycbcr method
 - Add new format to list supported by camera HAL
 - Fix minor compilation warnings

Bug: 8734880
Change-Id: I68a8cc126985c7d5ae100a87b31c60ee59074cd3

goldfish: fix gralloc module

Explicitly setting reserved (in C++) breaks future extensions of the
module format.  Removed these explicit settings for future
compatibility.

Change-Id: Iaacd5c2f55121bcb8f1761f646ace374eceb2b4f

Add liblog

Bug: 8580410
Change-Id: I2c07b8ce85aff341575ee8c3f509f64e252b8ed4

migrate opengl and system from  development/tools

- components under system are moved one directory up like all other HALs

Change-Id: I03b870b870d83b247ac398cadfb155f03c9adfa0

Initial empty repository