goldfish-opengl: update to 4153dd9ab6a14

commit 4153dd9ab6a14a263e97c40faeab237b66c9f67b
Author: bohu <bohu@google.com>
Date:   Tue Aug 1 17:05:26 2017 -0700

get emulator iamge on oc-mr1-dev work with treble

Need to remove libui dependency
otherwise, libEGL wont load because it is looking for it;
but it does not seem to need it

BUG: 64401460

Change-Id: I81a77c85307524c2b075d430a84c73556f64524e
Merged-In: I6b435afce04164308767daef3586358b04021cbd

Merge cherrypicks of [2454758, 2454859, 2454837, 2454965, 2454984, 2455001, 2454760, 2454860, 2454838, 2454899, 2454970, 2455005, 2455027, 2454973, 2455006, 2455061, 2455007, 2454914, 2454987, 2454974] into oc-release

Change-Id: I3cd8a20b7940e3541c22af7cbacb71ce6eb3d0d0

Fix API 26 for public emulators, take 2

Bug: 37302997
Bug: 62333501

codename changed from O to REL

So add a case for API level too

Test: Build and run sdk_google_aw_x86-userdebug emulator

Change-Id: I91f3804082bc6d61922d31ce72081a2ec88bc6b0
(cherry picked from commit 256462a4a0fb181aca670fba4241e2fe0cd295f8)

gralloc ashmem hack to delay cb close

Tired of public emulators not working with O

See this CL for context:

https://android-review.googlesource.com/#/c/372376/

This CL moves that to the guest, and detects if
each gralloc cb has been opened at least once
before allowing gralloc_free to call rcCloseColorBuffer.

Images using Treble / HIDL gralloc that have this CL
should work with older emulator versions.

Bug: 37302997

Test: Build and run sdk_google_aw_x86-userdebug emulator

Change-Id: Ice72269e4295497825050385b15dbdec2258100c
(cherry picked from commit ef6676015bfec4538a552204b51e32f02d721fb0)

Emulator: update opengl to bb8d138b

Bug: 37302997

Test: Build and run sdk_google_aw_x86-userdebug emulator

To commit
bb8d138b3e83567d68810d059d69654fa5e8a303
Merge: 7c53152 bb1f88a
Author: Josh Gao <jmgao@google.com>
Date:   Fri Apr 14 23:08:26 2017 +0000
Rename socket_loopback_server to _socket_loopback_server.

Change-Id: Id602f0c1df3be423e37af5f6c91d037d713c3b7a
(cherry picked from commit 588c0a94e384e07c8d38185710a1a968823594ba)
(cherry picked from commit 1b5d5a70fe49f2826a225878ddcd8d6ad561c886)

merge in oc-release history after reset to master

merge in oc-release history after reset to master

[hwc2] Fix build

Change-Id: Ibd82639c1eb5619322e98a559f495bed36133998

merge in oc-release history after reset to 18d4bde820d1b8b073c20f306f8cc1103174eaf6

[hwc2] Add surfaceInterface/goldfishHwc2

bug: 36439031
bug: 36375335

When HWC2 is used (even the HWC2on1 adapter), one of the most fundamental
changes is that acquireBuffer/releaseBuffer by BufferQueue consumers
is coarsened, delaying releaseBuffer until the app has finished
with its eglSwapBuffers operation and handling releaseBuffer as part of
a post process (mPendingRelease/releasePendingBuffer/et al).

This would be OK, except for the fact that an acquired, unreleased buffer
takes up a free buffer slot.

Emulator uses GLES composition currently, so each eglSwapBuffers by an app
directly causes another eglSwapBuffers, each of which attempts to
dequeueBuffer, taking up 2 slots right there. This was usually OK, since
releaseBuffer would be interleaved, but now, with the delayed releaseBuffer,
there are no free buffer slots and we have a deadlock situation.

Fortunately, we can set the swapped-to Surface to "async mode", which takes
exactly this situation into account. Async mode, which is for Surfaces but
really affects the BufferQueueCore queue size, makes it so that there
is an extra buffer slot so that dequeueBuffer doesn't have to block.

This CL adds a small static library for setting the swapped-to Surface
to async mode. Note that one does not simply add libgui to the shared
libraries of system/egl/Android.mk, since libgui itself includes EGL/GLES2
as dependencies, putting us into DLL hell and causing all sorts of trouble!

Change-Id: I6ee4f0e6d0b668d60573887751ec6b02839df5c3

Merge "Don't validate glTexSubImage2D bounds for native buffer target"

Don't validate glTexSubImage2D bounds for native buffer target

bug: 36372213

We validate glTexSubImage2D having enough width/height/depth to work
correctly, by tracking glTexImage2D / glTexStorage2D calls,
but this breaks down once we put EGL images in the mix.

Previously we skipped validation for EGL_GL_TEXTURE_2D targets, but
it looks like for O and up, we also need to skip this validation for
EGL_NATIVE_BUFFER_ANDROID targets.

Change-Id: I68e6049a00e7f0927fc4097a36564b88aea1ff00

Merge "Change path to gralloc.default"

DO NOT MERGE ANYWHERE goldfish-opengl: adjust to the refactored qemu_pipe am: 28494a7ff7  -s ours am: 2e4f19c1bf  -s ours
am: 0ef61727d1  -s ours

Change-Id: I37c4c0dc7055526dfaf02deef8aca3483fec431c

DO NOT MERGE ANYWHERE goldfish-opengl: adjust to the refactored qemu_pipe am: 28494a7ff7  -s ours
am: 2e4f19c1bf  -s ours

Change-Id: Ida2024994bd063cd5d5e7f53fd8294dab4cd6f4d

DO NOT MERGE ANYWHERE goldfish-opengl: adjust to the refactored qemu_pipe
am: 28494a7ff7  -s ours

Change-Id: I289486fcde44223c2fac33213defa5ff5ee58303

Change path to gralloc.default

Libraries under /hardware/libhardware/modules should go to /vendor/.
This CL changes hardcoded path to gralloc.default.so

Bug: 35907904
Test: Compiled and checked install path. Also code searched to see
if there are hardcoded paths.

Change-Id: I6c304b048047053d782a0ed91abfbc3140bdd261

Merge "Emulator: make qemu_pipe.h self contained in goldfish-opengl project"

DO NOT MERGE ANYWHERE goldfish-opengl: adjust to the refactored qemu_pipe

Qemu_pipe is refactored into a library, adjust goldfish-opengl
accordingly.

Change-Id: I6fe6a98a962ac1bcfeb0395a575115054464d72e

Merge "extern "C" unconditionally"

Have single macro for taking care of goldfish_dma_context alignment.

Previously,

!defined(__aarch64__) || !defined(__x86_64__) is always true,

instead of the intended "add this field if not 64 bit."

OOPS

bug: 35711775
Change-Id: I46043c4402325845524d56d8d4f910f16add25cb

extern "C" unconditionally

This keeps older API levels from mangling GLESv3 entry points.

Change-Id: I5e751ca9f53f510d057693d368bc15c20f66e463

Emulator: make qemu_pipe.h self contained in goldfish-opengl project

Remove the dependency on the system/qemu_pipe.h or hardware/qemu_pipe.h;
and use the legacy qemu_pipe_open definition since goldfish-opengl is
shared accross all apis.

Change-Id: I3da10066d458a57385c408cee4d2ded0eec6e456

Merge "Also take into account arm64 and mips64 for goldfish_dma_context aligment."

Also take into account arm64 and mips64 for goldfish_dma_context aligment.

bug: 35618215

Change-Id: I147aeca391f97b0bccaed0b572b2fe13e627ad00

Don't restrict GLES 3.x based on API level nor arch.

bug: 35618215

Simpler that way, and we should be able to support all archs.

Change-Id: I77f5295047b05dc9188b15501be08b95b8f41302

Align goldfish_dma_context to 8 byte boundaries

bug: 35465006

It looks like when Chrome is started, we are in some strange world
where sizeof(goldfish_dma_context) = 16 instead of 24,
and we get in trouble with gralloc_register buffer,
and Chrome fails to boot.

TODO: Note that youtube videos inside chrome still don't seem to work,
for some reason (this doesn't depend on whether or not goldfish_dma is
used).

Change-Id: Ibc66885434b885993025b70d957ea8e1adf524de

[deqp] Fix state_query.string

bug: 34245902

Version string needs to match the glGetIntegerv(GL_MINOR_VERSION, ...).

Change-Id: I8b51104b0347c12c93c350dda7fb03c145f8c3e2

[deqp] Fix wrong firstIndex for non-instanced draws with nonzero divisor

bug: 34245909

If the vertex attribute divisor is nonzero,
that actually affects first index for non-instanced draws
at least.

TODO: Figure out if this affects the instanced draws too.

dEQP-GLES3.functional.draw.random.210 : Fail -> Pass

Change-Id: Ida45d4689e0418ad8679091556a9fcaa64bbe25e

Fix regression in fbo.completeness after GL_EXT_color_buffer_float

Change-Id: Ie17da0442c80b972e5a66353b3998160fe04f032

[validation] Message on vertex attrib index out of bounds

Change-Id: Ie7f79b63d31fbae6cb97b3ae96fa5bdb8fefca27

[interface][fbo] Pass extensions, GL_EXT_color_buffer_float

- Pass extensions from the underlying OpenGL to encoder for
validation.
- Add GL_EXT_color_buffer_float.

Change-Id: I67f8e68012bba9c5ffef24d9588a9615cf6f0e6b

Don't crash if user specified client array of nullptr

3dmark can run into timing/races and do this sort of thing.

Change-Id: Ic75f78d23cb26ebe18142dc78158f5ffb05967fd

[GLESv3] Add KHR_create_context semantics

If the ES 3.x feature is enabled:

1. If not on GLES1, select the highest available 3.x context if
   major version was not specified.
2. If on GLES2, upgrade to highest available 3.x context.

- In this case, need to allow GL_HALF_FLOAT_OES through to
keep from regressing CTS dEQP-GLES2.functional.fbo.completeness
if the feature is enabled.

Change-Id: I92194387f09f3a531c6d39b5e2659ff62c4c77cd

Remove extraneous ALOGD's

Change-Id: Iefe7f8db7cc38f744dcc3c706963f3a9095786ca

Merge "[fps] Flush every 200 draw calls instead of 2"

[WIP] ES 3.1 guestside changes

Change-Id: Ie4619ee6e161274383b9f6b0d2922344da20f62f

[fps] Flush every 200 draw calls instead of 2

Linux z840, quadro k2200 3dmark ice storm unlimited:

80k->100k

Change-Id: Ia4d68f3d096ad042e3279d5a1d848930ce4571a6

[GLESv3] Invalidate index range cache on buffer unmap/flush

Fixes incorrect out of bounds validation messages
while playing Vector 2 (ES 3.1 game)

Change-Id: I7290b6a1ba0417d897f119d5d7695c2c16d64e39

[validation] Check null buf before printing validation info

Change-Id: I2ba7c4c694ad4a75a6e68722e3fd7cde05a47a0f

[validation] Correctly compute wanted draw call size when draw count == 0.

bug: 34378638

Change-Id: I9912436048c28337de40d13536246a6b9e11fc7b

[GLESv3] Fix protocol error + build on older imgs

Change-Id: I6a28dfeed1242a89f1bf8a872f9f6ebd39585d8c

[GLESv3] Fix build

Change-Id: I1255b2505eec7ce032992b99c20df4a1dca4394f

[GLESv3] ES 3.x API v1

- ES 3.0 almost conformant (~80 dEQP-GLES3 failures)
- ES 3.1 has entry points

- This CL adds all the new entry points and updates all
  validation.

Change-Id: I8f7256d9443ecda9d7ef87580b467e711c6e8e92

[GLESv3] Transform feedback state tracking

Needed to pass some negative_api tests involving transform feedback.

Change-Id: I7be230bb6432c32641de6bb9249fd00ba3dfe926

[GLESv3] Shader state updates

- Track program pipelines
- Track separate shader programs
- Track original shader source in guest

Change-Id: I8f54089ddf12d03677275cde58ea7f267e410482

[GLESv3] Updated validation

Change-Id: I0881a6bd58717964c235075e94b021e7e24be420

[GLESv3] glUtils update for ES 3.x

Change-Id: Iaf9b5832af0822d9f663a5365e7ae9a7cee426cc

[GLESv3] FBO state

- Draw/read framebuffers
- Multisampled render buffers
- Multiple color attachments
- GL_DEPTH_STENCIL attachment

Change-Id: Ic0f51633a42a14d333b6dc3fb52c6fa9de0bfe17

[GLESv3] ES 3.x buffer state

- VAOs
- indexed buffers

Change-Id: I743c3e22bdbb67ca0b6cecac6b231cd674a5061f

[GLESv3] Texture state tracking upgrades

- TextureUtils to get the low down on all offset/size related calculations
(ripped from SwiftShader)
    - This involves track of all PBO fields
- Move TextureRec and friends to GLSharedGroup
- Implement new encoder utils to use in encoder

Change-Id: Ic6585f49d2928fe965181ec706d5fefa8713e0eb

[GLESv3] versions, es > 2, encoder

- Encoder with ES 3.x entry points
- Allow ES versions > 2 depending on host capability

Change-Id: I71e374f1685610bb0e4f9e04f4bba87b0cbaa02b

Merge "Revert "Fix boot for car emulator on git-master""

Revert "Fix boot for car emulator on git-master"

This reverts commit 3783edd49fd2ad6f200cbf9d04978132f2e41bd2.

Change-Id: I25dab3f8c151506a84601ab0c341d5dfc7c3b2ae

Revert "Fix git-master and sync device"

This reverts commit 057e300788707fca96aeb82c3cc4d1d9b475b032.

Change-Id: Ic2d2c808cf1587e839abfa7a8928792544f231ac

Fix git-master and sync device

Disable sync device for now until we fix rest of the issues

Change-Id: Ie4878b1231b481e5085d47e7525f0ae4b3c7e9db

Fix boot for car emulator on git-master

bug: 33623000

Change-Id: Ie17136f2adafdb567bf9a490cf1282f1703d4f54

Fix black screen in apps

- multi-line for loop body :)
- distinguish locked vs unlocked cb width

Change-Id: I4d978011a1b7a7aa5efb244a56353857285e3a03

Merge "Fix out-of-bounds vertex arrays"

Fix out-of-bounds vertex arrays

Index range cache got a bogus argument for "offset",
so it was not being invalidated properly.

When it wasn't being invalidated properly, our validation
could issue out-of-bounds errors even when the index buffer
was actually not out of bounds.

Change-Id: I9c59412bb20bd6ea16e25bf83f1e64d5889910e9

Merge "Handle repeated swapBuffer errors"

Handle repeated swapBuffer errors

Changed eglSwapBuffers to actually signal an error when swapBuffers
fails. This was hiding an error log at the SurfaceFlinger callsite.

When eglSwapBuffers is called after a prior error, we should not try to
enqueue a NULL buffer which was causing a segfault that takes down
SurfaceFlinger.

Bug: 33432596
Change-Id: I72f3301e0adbc4ea75266c9c1c9e01d9169036e7

Merge remote-tracking branch 'goog/stage-aosp-master' into HEAD am: 0bff4a1296 am: df28bb8ddf am: 4033cc718c
am: d8dad693b2

Change-Id: Ie80f047c5a8f81fbc81e8e60ba4c158022590002

Merge remote-tracking branch 'goog/stage-aosp-master' into HEAD am: 0bff4a1296 am: df28bb8ddf
am: 4033cc718c

Change-Id: I3ea15643bbabf94be2fdccb2982459aa8b284f45

Merge remote-tracking branch 'goog/stage-aosp-master' into HEAD am: 0bff4a1296
am: df28bb8ddf

Change-Id: I8651d7ebd3694ac715dcaac6f46d7094c5e327eb

Merge remote-tracking branch 'goog/stage-aosp-master' into HEAD
am: 0bff4a1296

Change-Id: Icdff0ea2225fe1dd8e1b587f829b005810413655

Merge remote-tracking branch 'goog/stage-aosp-master' into HEAD

Merge "Print more info if a draw call is out of bounds"

Merge "[GLESv3][VAO] enable/pointer for vertex attribs when possible"

Initial empty repository

Print more info if a draw call is out of bounds

Change-Id: I8f35d27a76d8b023f63543f2afdd4dc9c2a18a0b

[GLESv3][VAO] enable/pointer for vertex attribs when possible

If we don't forward enables/pointers to the host until
right before the draw, when a nonzero VAO is bound,
it defeats the purpose of VAO usage for performance.

This CL prepares the GLES2 encoder for VAO usage by
forwarding those calls whenever possible.

It gives a flag to sendVertexAttributes to not do any
changes in GL state if all attributes are backed by VBOs.

The reason we don't forward them 100% is because of
supporting either legacy or tricky behavior:

Legacy: specifying vertex arrays from host memory directly
with a host ptr passed to glVertexAttribPointer
Tricky: Using glVertexAttrib*f*(...) instead of
glVertexAttribPointer

Change-Id: I190aab86d7dcd7e01cfac0a1f60b44fac131abd0

Merge "Revert "GL2Encoder: Do not forward GLES API version queries""

Revert "GL2Encoder: Do not forward GLES API version queries"

This reverts commit dadc6b152a14c98bb49ce9848078a6c8bce9bf9f.

dEQP-GLES2.functional.fbo.completeness.renderable.renderbuffer.color0.rgb10_a2 Pass -> Fail

I will provide a proper fix for this; since we are supporting GLESv3, adding the client version to the tracked state and selectively rejecting such formats with GL_FRAMEBUFFER_UNSUPPORTED.

Change-Id: Id7fc205b6ddd46fa72c62cd9a8c67cadee78d46d

Merge "More accurate validation on gl vertex attributes"

More accurate validation on gl vertex attributes

This CL has more accurate validation for gl vertex attributes, and is
more tolerable if the user forgot to disable some vertex attribute
arrays.

Change-Id: I6bb205add25bf51f0ba70a5cb267c3319a17a842

Merge "GL2Encoder: Do not forward GLES API version queries"

GL2Encoder: Do not forward GLES API version queries

Original author: Chenglangjie Yang <chenglangjie.yang@intel.com>

GLESv3 adds two additional glGetIntegerv() parameters for returning
GLES API version number: GL_MAJOR_VERSION and GL_MINOR_VERSION. They
are not supported by GLESv2, though. Compare:

(v3) https://www.khronos.org/opengles/sdk/docs/man3/html/glGet.xhtml
(v2) https://www.khronos.org/opengles/sdk/docs/man/xhtml/glGet.xml

dEQP actually uses this API to determine if the device being tested
supports GLESv3. If GL2Encoder just forwards such queries to the
host, the result will be the desktop GL API version implemented by
the host GPU driver, which is almost always >= 3.0. This can mislead
dEQP into treating the emulated device as GLESv3-compatible, and
cause various GLESv2 tests to fail.

When applied to AOSP master, this patch fixes various dEQP-GLES2
failures in the following packages:

 dEQP-GLES2.functional.fbo.completeness.renderable.*
 dEQP-GLES2.functional.negative_api.*
 dEQP-GLES2.functional.draw.random
 dEQP-GLES2.functional.rasterization.primitives

When applied to Partner m-emu-dev, it causes a few regressions in
dEQP-GLES2 (actually false negatives), which will be addressed by
two other patches.

Change-Id: I801144aeee922d90af4c7dda6af68dcc0a40fb6d
Signed-off-by: Chenglangjie Yang <chenglangjie.yang@intel.com>
[Revised code and commit message]
Signed-off-by: Yu Ning <yu.ning@intel.com>

Revert "Fix the position of qemu_pipe header" am: 94ad9fbf53 am: b0e5b30f13
am: f10543b3b5

Change-Id: I239c1c42ac6e92290a7319c790fdbf91e354d94b

Revert "Fix the position of qemu_pipe header" am: 94ad9fbf53
am: b0e5b30f13

Change-Id: Id43618ad8b4c83b41b0dfa709152640366071fd4

Revert "Fix the position of qemu_pipe header"
am: 94ad9fbf53

Change-Id: I042af868b1f418b0ee5902dc928b405717808cd1

Revert "Fix the position of qemu_pipe header"

This change should not have made it into master.

This reverts commit 3855dfbf428269d3c79dd7ba4b257ec5e9b917ff.

Test: Built full-eng, mmma -j30 device/generic/goldfish-opengl.

Change-Id: I4bffeae147464a7a158e0daa95155d87d0a9677b
(cherry picked from commit 2e11850e11ad1b8a548176a5dc8c223f252227d9)

Revert "Revert "DO NOT MERGE ANYWHERE: Goldfish DMA-based gralloc""

This reverts commit 9b166b966d6f03f13f65d3134f9c5e1133ec10e2.

Change-Id: I9b10b45d1ebd3b51ede2f75cc8ab403914b0d777

Merge "Revert "DO NOT MERGE ANYWHERE: Goldfish DMA-based gralloc""

Revert "DO NOT MERGE ANYWHERE: Goldfish DMA-based gralloc"

This reverts commit df3a043b96b90d949643ddd0219fc0bf4ff68040.

Change-Id: Ic87fa2f922bd61123e64fc32fc8ebb13dec4c219

Merge "Fix build - add LOGV->ALOGV"

Fix build - add LOGV->ALOGV

Change-Id: I669211faa3c58488c5b9b23d900a4e6da8e5c512

Merge "DO NOT MERGE ANYWHERE: Goldfish DMA-based gralloc"

DO NOT MERGE ANYWHERE: Goldfish DMA-based gralloc

Also includes YV12/YUV420888 conversion on the host.

Change-Id: Ie0c23f6e18ae96b2bdeadf6dbd4828ca1e7de8a0

Merge "DO NOT MERGE ANYWHERE Accept EGL_CONTEXT_PRIORITY_LEVEL_IMG" into nyc-mr1-dev am: ff9b220285  -s ours am: ab60271bab  -s ours am: f09bf909ad  -s ours
am: d7e8dc892b  -s ours

Change-Id: I3aa8dddf4571f482028eb873e29b3be2b2340a24

DO NOT MERGE ANYWHERE Accept EGL_CONTEXT_PRIORITY_LEVEL_IMG am: 3a6d7c961e  -s ours am: db334bed50  -s ours am: 2737782b4d  -s ours
am: 1e5fd1e94e  -s ours

Change-Id: I1503e08d50a799e17c4f6bcb2a6d6a9ed5e6eb0b

Merge "DO NOT MERGE ANYWHERE Accept EGL_CONTEXT_PRIORITY_LEVEL_IMG" into nyc-mr1-dev am: ff9b220285  -s ours am: ab60271bab  -s ours
am: f09bf909ad  -s ours

Change-Id: I8247229fbca72d4a6b759935909f2e99c21e03b4

DO NOT MERGE ANYWHERE Accept EGL_CONTEXT_PRIORITY_LEVEL_IMG am: 3a6d7c961e  -s ours am: db334bed50  -s ours
am: 2737782b4d  -s ours

Change-Id: I4d55d909a22f5b9cf3298ef48d2aa00f6a0a8916

Merge "DO NOT MERGE ANYWHERE Accept EGL_CONTEXT_PRIORITY_LEVEL_IMG" into nyc-mr1-dev am: ff9b220285  -s ours
am: ab60271bab  -s ours

Change-Id: I14b8e7ee8e7e7a263ed170c3e595ac51b39bd6c5

DO NOT MERGE ANYWHERE Accept EGL_CONTEXT_PRIORITY_LEVEL_IMG am: 3a6d7c961e  -s ours
am: db334bed50  -s ours

Change-Id: Id2246625716dad271de3dc9ce9f2ba0aff2a3df9

Fix glLinkProgram with invalid program name

glLinkProgram might try setting up program attributes even when an
invalid program name is passed to it. This patch fixes it.

BUG: 33069416
Change-Id: I93ed140095b67df47b158ba837c4aac8eb9e8e50

Fix a typo in HostConnection.cpp

Change-Id: I76217f256e2a80ef12be6cdbef5d2846da4323b0

Merge "DO NOT MERGE ANYWHERE Accept EGL_CONTEXT_PRIORITY_LEVEL_IMG" into nyc-mr1-dev am: ff9b220285  -s ours am: ab60271bab  -s ours am: 78454be75a  -s ours
am: 0913dec00d  -s ours

Change-Id: I37ac6d6e71a1b0e5de8407e99ed483ffd45f2913

DO NOT MERGE ANYWHERE Accept EGL_CONTEXT_PRIORITY_LEVEL_IMG am: 3a6d7c961e  -s ours am: db334bed50  -s ours am: 096235db73  -s ours
am: 984acc9061  -s ours

Change-Id: I94e1c2dd22c4bac7cc2f478b3bf975c1efa6fc82

Merge "DO NOT MERGE ANYWHERE Accept EGL_CONTEXT_PRIORITY_LEVEL_IMG" into nyc-mr1-dev am: ff9b220285  -s ours am: ab60271bab  -s ours
am: 78454be75a  -s ours

Change-Id: I3e7a223d68cd9c09d3a89e09cfe6ee778136e72c