GLES2: sequence operator (,) eval order and side effects
am: 073a0ffc99

Change-Id: Idfa309085b6573838072790386573084b5a50a57

GLES2: sequence operator (,) eval order and side effects

Ported from:
https://github.com/KhronosGroup/WebGL/blob/master/sdk/tests/conformance/glsl/bugs/sequence-operator-evaluation-order.html
Which was added in:
https://github.com/KhronosGroup/WebGL/pull/981
https://github.com/KhronosGroup/WebGL/pull/1932

These tests were known to fail in on Pixel 2016 / Android 7.1.1.

New tests:
  dEQP-GLES2.functional.shaders.operator.sequence.side_effects.affect_ternary
  dEQP-GLES2.functional.shaders.operator.sequence.side_effects.affect_and
  dEQP-GLES2.functional.shaders.operator.sequence.side_effects.affect_or
Components: AOSP

Change-Id: I7bbcaecc08279df6f30b6855cc99b19b402892c7

Merge "GLES3 tests for equality of structs with array members"
am: fc76a6f2b1

Change-Id: I34c82aef8ce9e77a07441e898fb2db0d4a000f37

Merge "GLES3 tests for equality of structs with array members"

Merge "Fix using representable texture channel ranges."
am: b845c09634

Change-Id: I2124ea6f2a1f3c889f25d1f5f76191d183afe036

Merge "Fix using representable texture channel ranges."

Merge "Tolerate flush/finish test assumption failures."
am: b821430a47

Change-Id: I319da491080e82250693a32a72ab0d3c5c318aa6

Merge "Tolerate flush/finish test assumption failures."

GLES3 tests for equality of structs with array members

The new test case adds regression coverage for the Qualcomm driver
shader compiler crash seen in Android N:
https://crbug.com/695687
It is a port of this WebGL test case:
https://github.com/KhronosGroup/WebGL/blob/master/sdk/tests/conformance2/glsl3/compare-structs-containing-arrays.html

New tests: dEQP-GLES3.functional.shaders.struct.local.array_member_equality_vertex, dEQP-GLES3.functional.shaders.struct.local.array_member_equality_fragment
Components: AOSP
Change-Id: Ie130439f80fddb081fb36db02c24a9e18f5c4bca

Tolerate flush/finish test assumption failures.

These tests assume that draw calls are queued up until they're forced
to be executed by a ReadPixels, Flush, or Finish call. The spec does
not require this, and implementations are free to eagerly start
execution of draw calls.

Thus this change reports a CompatibilityWarning if the assumption isn't
met, instead of a Failure.

Google bug: b/71858368
Components: AOSP
Tests: dEQP-GLES2.functional.flush_finish,
dEQP-GLES3.functional.flush_finish

Merge "Test glRenderbufferStorage with unsized depth format"
am: eff5df467e

Change-Id: I51ed332c33fcfeb80300da859553ddce1218bac0

Merge "Test glRenderbufferStorage with unsized depth format"

Test that points are clamped to ALIASED_POINT_SIZE_RANGE
am: b2931a8957

Change-Id: I8a6f2916eb20800d000f08a632351f80de1527a8

Test that points are clamped to ALIASED_POINT_SIZE_RANGE

This test verifies that the point size written to gl_PointSize is
clipped to the range ALIASED_POINT_SIZE_RANGE before rasterization.

See also: http://crbug.com/740560
In particular this comment: http://crbug.com/740560#c27

New tests: dEQP-GLES2.functional.rasterization.limits.points
Components: AOSP
Change-Id: I98708ebece4be9c2bce3c7ba3b57454aec657cce

Test glRenderbufferStorage with unsized depth format

The OpenGL ES 2.0 specification only supports formats from Table 4.5,
which are all sized internal formats, while ES 3.0 explicitly states
that "<internalformat> must be a sized internal format".

A color format (GL_RGBA) was already being negatively tested for, but
not a depth format.

Merge "Port WebGL multisampled renderbuffer resize test"
am: a73c1d9300

Change-Id: Ic8e82d353fce2a664eab902649d72159974e3895

Merge "Port WebGL multisampled renderbuffer resize test"

Suppress new warnings from gcc-6 am: 9a58d041fe am: fd34041ed0
am: 7d5a16bb55

Change-Id: If9065cd6d94a08fcd6e46868fd0bd038291af83f

Suppress new warnings from gcc-6 am: 9a58d041fe
am: fd34041ed0

Change-Id: I0b9eca4091dc271141027d68a6a46d58937e56ac

Suppress new warnings from gcc-6
am: 9a58d041fe

Change-Id: I176dd24b363db336f4f42f8d8c2c5aebb2632dcb

Suppress new warnings from gcc-6

check_build_sanity currently fails with the system gcc on rodete. Let's
just suppress this warning for now. Intend to back out the suppression
and fix the warnings in next release.

Bug: b/72219443
Test: build
Change-Id: Ic842a3de3d247d53c49117562b51e8eece3752f9

Port WebGL multisampled renderbuffer resize test

Port a WebGL test for a driver bug affecting resizing/reallocation
of multisampled renderbuffers on Qualcomm drivers:
* conformance2/renderbuffers/multisampled-renderbuffer-initialization.html

New tests: dEQP-GLES3.functional.fbo.msaa.renderbuffer_resize.*
Components: AOSP
Change-Id: Ib4377eaf35c1d4b5f431cd433e06301003966d54

Merge "Update owners file" into oreo-mr1-cts-dev am: 2797a75d5d
am: d478affed3

Change-Id: I0ceba22258c7665856fd17175c84ed8ac34a6bbd

Modify compute shader grid setup. am: a7c3d75202
am: a792af73ad

Change-Id: Ie0d8991f931ffe2f9574d3ea73806b81c543926e

Merge "Update owners file" into oreo-mr1-cts-dev
am: 2797a75d5d

Change-Id: I1245c8ed48b7d9fd4f41a57bf52a66c58d4edf7c

Modify compute shader grid setup.
am: a7c3d75202

Change-Id: Id377bb796299a4a37df027ef522494b5e7590723

Merge "Update owners file" into oreo-mr1-cts-dev

Merge "Accept EGL_BAD_PARAMETER for eglCreatePixmapSurface negative test"
am: aff54309fc

Change-Id: I903b320d225df76c4d75442956003011280847a3

Merge "Accept EGL_BAD_PARAMETER for eglCreatePixmapSurface negative test"

Update owners file

Bug: 72113298
Test: build/make/tools/checkowners.py -c -v OWNERS
Change-Id: I117459d91f39055aaa5f0bc6abfbda32f31cdab2
(cherry picked from commit 412471299c0fac5cb5af37fcdeadbf39a4a36744)

Merge changes from topic "tests-dropped-from-o"
am: a4ce935b02

Change-Id: Ia85d5ba6176e6ddefdb8925e7cfc760695a8b0a2

Modify compute shader grid setup.

Modify how the grid is set up for compute shader draw indirect test,
in order to avoid verfication failures due to difference in
floating-point computations on the CPU vs the GPU.

The existing code assumes infinite FP precision.

This change will guarantee the next value of pos(X) to be equal to
the first value of pos(X+1), avoiding gaps/discontinuities.

Affects:
dEQP-GLES31.functional.draw_indirect.compute_interop.large.drawarrays_combined_grid_1000x1000_drawcount_1
dEQP-GLES31.functional.draw_indirect.compute_interop.large.drawarrays_combined_grid_1000x1000_drawcount_5000
dEQP-GLES31.functional.draw_indirect.compute_interop.large.drawarrays_combined_grid_1000x1000_drawcount_8
dEQP-GLES31.functional.draw_indirect.compute_interop.large.drawarrays_separate_grid_1000x1000_drawcount_1
dEQP-GLES31.functional.draw_indirect.compute_interop.large.drawarrays_separate_grid_1000x1000_drawcount_5000
dEQP-GLES31.functional.draw_indirect.compute_interop.large.drawarrays_separate_grid_1000x1000_drawcount_8

Bug: 71515808
Change-Id: I21fe8ec9ea48b15f6171265df802bc47be775f56

Merge changes from topic "tests-dropped-from-o"

* changes:
  Reinstate excluded UBO tests
  Reinstate dEQP-VK.image.mutable.*

Merge changes from topic "multiview"
am: cbc74ae174

Change-Id: I1a1e3f85f1d8373ec30ce0c1fbb1c5f726ba5223

Merge changes from topic "multiview"

* changes:
  Add preliminary dEQP multiview test.
  Add multiview entry points to GL wrappers.

Limit maximum allocation size in memory mapping tests am: adff582b50 am: 71ef361f3e am: fe861b61a5 am: 8d3d9ceddb
am: cf839197ca

Change-Id: I6dca2adb743d2b01e26368c751bdd9dc52db1dfb

Improve memory tracking in vulkan memory mapping tests am: 3a8ec35d52 am: b1f38f97cf  -s ours am: 346da766ee am: 40f1c9ad64
am: 6b9f336340

Change-Id: Ibd008af258542db886e60cb4b440394cc1696718

Optimize random memory mapping tests reference memory am: 7e69d67a2c am: b2dcf86f17  -s ours am: 1883d6ef48 am: 225bb3c908
am: 81b4c18ba8

Change-Id: I757ae460f4317cd5eebbce9abd93d40033925180

Limit maximum allocation size in memory mapping tests am: adff582b50 am: 71ef361f3e am: fe861b61a5
am: 8d3d9ceddb

Change-Id: Ifa3b4bdebfc75dd273119a117fc9bfe131a60c2f

Improve memory tracking in vulkan memory mapping tests am: 3a8ec35d52 am: b1f38f97cf  -s ours am: 346da766ee
am: 40f1c9ad64

Change-Id: I39c3d0f7df1cb3da09e1758d9716c599ffad8cdd

Optimize random memory mapping tests reference memory am: 7e69d67a2c am: b2dcf86f17  -s ours am: 1883d6ef48
am: 225bb3c908

Change-Id: I1ef89fe87aaa220a66cfa26008a06bda04295862

Limit maximum allocation size in memory mapping tests am: adff582b50 am: 71ef361f3e
am: fe861b61a5

Change-Id: Id7e2ade9ceaad34bee00b2b36bfde50ca2492dd5

Improve memory tracking in vulkan memory mapping tests am: 3a8ec35d52 am: b1f38f97cf  -s ours
am: 346da766ee

Change-Id: Iece2a8d3c40b646e982008d82f40b16dfdbf94ca

Optimize random memory mapping tests reference memory am: 7e69d67a2c am: b2dcf86f17  -s ours
am: 1883d6ef48

Change-Id: I9f6246177cc3651b84370fedd7f48fd9dddf1f94

Limit maximum allocation size in memory mapping tests am: adff582b50
am: 71ef361f3e

Change-Id: I138e6025b0cfb77ffd8197b796e32b06011cbc8a

Improve memory tracking in vulkan memory mapping tests am: 3a8ec35d52
am: b1f38f97cf  -s ours

Change-Id: I8fc045601456dd0b8d0026c4e70db43cf4998886

Optimize random memory mapping tests reference memory am: 7e69d67a2c
am: b2dcf86f17  -s ours

Change-Id: Iaa15b3997481353a88aef19cdde4348ae2cc3f75

Limit maximum allocation size in memory mapping tests
am: adff582b50

Change-Id: Iff465c2be4d7c27702036f8f060a458c55830732

Improve memory tracking in vulkan memory mapping tests
am: 3a8ec35d52

Change-Id: I54613d0c408d80a65cc175e86495d0ffd4c650b5

Optimize random memory mapping tests reference memory
am: 7e69d67a2c

Change-Id: I2c91c3720102d1c4c949bf19a2b6af51c59581f1

Add preliminary dEQP multiview test.

This is a pretty basic test of multiview, we'll be enhancing the test
in the near future to provide coverage for additional texture formats.

Test: Passes on Pixel 2 hardware, ran dEQP with --deqp-case=dEQP-GLES3.functional.multiview.*
Change-Id: I16e855641da04e3a09e54d82808f1034e87bc4fc
Component: AOSP

Multiple draw buffers in blend_equation_advanced am: 123de4f307 am: ddd1655bf6
am: 99cea239f4

Change-Id: I06b91641fc9014c2ac7dbe9f953953973a1656d8

Multiple draw buffers in blend_equation_advanced am: 123de4f307
am: ddd1655bf6

Change-Id: I73797808261dc6a3b0522ee103fa5e44eb97eb4c

Multiple draw buffers in blend_equation_advanced
am: 123de4f307

Change-Id: I1ce06acaa4f5f725097bcc248b7dbdf048bac61e

Reinstate excluded UBO tests

These were excluded in O CTS because of an inadvertent shader codegen
change. Reinstate the tests for the next version.

Bug: b/71902667
Test: check_build_sanity
Change-Id: I58dd09d7d7d43081f27287bfa51d7f63d6b1e51b
Affects: dEQP-VK.ubo.random.*

Multiple draw buffers in blend_equation_advanced

GL_NVX_blend_equation_advanced_multi_draw_buffers adds support for
multiple draw buffers with advanced blend equations. This change
modifies the below test to check for this extension and to expect
GL_NO_ERROR if it is present.

Components: AOSP

VK-GL-CTS issue: 770
Bug: b/71901344

Affects:
dEQP-GLES31.functional.debug.negative_coverage.get_error.advanced_blend.attachment_advanced_equation
(cherry picked from commit 2cb35a235c1a7bd3c63b296c37e186eec34e08e2)

Change-Id: I865fa30e527b5899197bade465602bafc92a54a1

Reinstate dEQP-VK.image.mutable.*

These were pulled from the O CTS because they were introduced in a minor
1.0.2.x release. They can now be reinstated.

Bug: b/65845002
Test: check_build_sanity
Change-Id: Icaf0e6073abda5d00f2b6e994bef3fffc31a1cda

Merge "Reduce GPU workload of pipeline_barrier tests" into oreo-cts-dev am: 4c26442e3d am: d9b0dd51d2
am: b77437eac1

Change-Id: Ia33c8df0576ce7eddd8d338ceee85cd9ff14a59a

Merge "Reduce GPU workload of pipeline_barrier tests" into oreo-cts-dev am: 4c26442e3d
am: d9b0dd51d2

Change-Id: Iddae106a643abc1cae3a66877bc6f32126ea5781

Merge "Reduce GPU workload of pipeline_barrier tests" into oreo-cts-dev
am: 4c26442e3d

Change-Id: I32a3a35837be16886acebd28e30ccdadffa73811

Merge "Reduce GPU workload of pipeline_barrier tests" into oreo-cts-dev

glshared: Add watchdog touches to avoid timeouts on slow hardware am: 0d9250e9eb am: 65715b7354
am: 01ef881911

Change-Id: I8aebeed9d98e4b29d4de1d761a6164c7389abb91

glshared: Add watchdog touches to avoid timeouts on slow hardware am: 0d9250e9eb
am: 65715b7354

Change-Id: I7530fe8d79d43fb5a269347cd302709c7498b764

glshared: Add watchdog touches to avoid timeouts on slow hardware
am: 0d9250e9eb

Change-Id: If675c974a2389787e53d2d618509ca81a48990d9

Add multiview entry points to GL wrappers.

Prepare for upcoming multiview conformance tests.

Change-Id: Ie3d8bbb4ae747e90c4f3e4a9632ebc4e5a085d00
Test: Built dEQP along with a pending change that tests multiview.
Components: Framework

glshared: Add watchdog touches to avoid timeouts on slow hardware

Based on diffs posted to deqp-external-requests

Bug: b/71753331
Bug: b/67854695

Change-Id: Id25e334814caf65f72b76e66d3b07fccba5ad355
Component: AOSP

Fix using representable texture channel ranges.

A value of 4294967295 (2^32 - 1) is not exactly representable in
IEEE-754 single-precision floating-point format, and so it gets rounded
to the next representable value, which is 4294967296. However, this
value can't be cast to an unsigned 32-bit integer without overflowing.

GLSL does not define what happens on overflow, and IEEE-754 defines it
as an exception but GLSL forbids exceptions. Hence some implementations
may produce unexpected results. dEQP assumed clamping to the largest
representable integer.

This change fixes that false assumption by reducing the range to values
representable in both float and integer formats.

Note that 32-bit integer formats can still hold values slightly larger
than these ranges. So while previously the floating-point ranges were
too large to represent integer values, they are now too small. This
can't be fixed without separating the integer format tests and only
using integer values to represent their ranges. This doesn't appear
necessary for the time being since the tests that use these floating-
point ranges have large 12/256 tolerances for the output color.

Google bug: 70910885
Component: Framework

Merge remote-tracking branch oreo-mr1-cts-dev into 
am: 5ea85b4a84

Change-Id: Icf5146122985c65678f7ae1181272a1942b33b6f

Merge remote-tracking branch oreo-mr1-cts-dev into 

Test: presubmit
Bug: 69859815
Change-Id: I47e94db994e07a7b159d840d1a2283bf072eb09f

Run Draw*BaseVertex if EXT_draw_elements_base_vertex is present

Draw*BaseVertex methods are enabled if
the EXT_draw_elements_base_vertex extension is present
Follow-up on 1b1ea05b

Components: AOSP
Bug: 70716357
Test: run on google devices

Affects: dEQP-GLES31.functional.draw_base_vertex.*

Change-Id: If8a4860a956c771ec5ea3341e2227f437b12a286
(cherry picked from commit 079ca561114713d31e05c4124b8daeef764cf9b8)
Merged-In: 079ca561114713d31e05c4124b8daeef764cf9b8

Report tests using Draw*BaseVertex as NotSupported

Draw*BaseVertex methods were added in GLES 3.2.
Report tests using these methods as NotSupported if
GLES 3.1 context is returned.

Components: AOSP

Affects: dEQP-GLES31.functional.draw_base_vertex.*

Bug: 70716326
Test: run on google devices

Change-Id: I87c918d5f66a0a5b547f8e625c662203a7f85048
(cherry picked from commit 1b1ea05b4e8d6e9dbb07187b0af6bc12c315fb4e)
Merged-In: 1b1ea05b4e8d6e9dbb07187b0af6bc12c315fb4e

Remove dEQP-GLES31.*vertex_array.*_invalid_map tests

Tests made assumptions about GL error returned from glDrawElements*
functions when passed a mapped buffer object, but, that error is not
supported by the specification.

Affects: dEQP-GLES31.*vertex_array.*
Components: AOSP
Bug: 37942569
Bug: 70716290
Test: run on google devices

Change-Id: Ic066dfbe717ff1e7741bec8ee870b09b99362dc7
(cherry picked from commit 428a2256159883c651a409c4e0cbe031953f4c4e)
Merged-In: 428a2256159883c651a409c4e0cbe031953f4c4e

Merge "Switch to libc++."
am: 8df1a93dca

Change-Id: Ib26677e35124485b8ded53fe6fec04de73889bc2

Merge "Switch to libc++."

Switch to libc++.

gnustl is being removed.

Bug: None
Test: make checkbuild
Change-Id: If41a886c8dcf2abc5aa4e30de4bce58c45cd6387

Fix mustpass generation
am: 8367ae87c7

Change-Id: I68e0b39304465e0d9dd0d21f01b89a9665d2f2ec

Fix mustpass generation

In 303f5900b4 we started emitting this element, but the generator was
slightly bogus.

Test: build mustpass

Change-Id: I2cf36637ba3264e1c254407d8f49f8c96b9d0cac

Add suite component to DEQP test module
am: 303f5900b4

Change-Id: I227e143828f1cb087b2ddae662ca8aebe72d5d3b

Add suite component to DEQP test module

Test: build
Bug: 65303193
Change-Id: I5102a2c0d7b4e49eafeaddea344208325ca7f300

Accept EGL_BAD_PARAMETER for eglCreatePixmapSurface negative test

Running CTS conformance test on wayland results in test failure for
dEQP-EGL.functional.negative_api.create_pixmap_surface, since the test
expects EGL_BAD_CONFIG, but actual returned error is EGL_BAD_PARAMETER.

According to EGL_EXT_platform_wayland spec,
(https://www.khronos.org/registry/EGL/extensions/EXT/EGL_EXT_platform_wayland.txt)
any call to eglCreatePlatformPixmapSurfaceEXT() on wayland will return
EGL_BAD_PARAMETER.

Ideally, both eglCreatePixmapSurface() and
eglCreatePlatformPixmapSurfaceEXT() should be considered the same, since
internally both calls do the same thing.

This is one of those situations where multiple errors could be reported
given that multiple invalid parameters are provided (invalid platform +
invalid config).

This change accepts both error values EGL_BAD_CONFIG and
EGL_BAD_PARAMETER for eglCreatePixmapSurface negative test.

VK-GL-CTS public issue: 68

Affects: dEQP-EGL.functional.negative_api.create_pixmap_surface
Test: Run the test on wayland platforms
Components: AOSP

Change-Id: Ic20e22a1916e2901acbf37faa86d1f7644465dab

Merge "Avoid watchdog timeout in Vulkan builtin precision tests" into oreo-cts-dev am: aa801278e8
am: 62b142f433

Change-Id: I5f3391624c6f5f935971d47c1a107e5c0e4733f4

Merge "Avoid watchdog timeout in Vulkan builtin precision tests" into oreo-cts-dev
am: aa801278e8

Change-Id: I5818c937676ad5c4c79814be1c39c8c264949977

Merge "Avoid watchdog timeout in Vulkan builtin precision tests" into oreo-cts-dev
am: aa801278e8

Change-Id: If513bb7ec4ab4b80695291633dc4f7990c69f28b

Merge "Avoid watchdog timeout in Vulkan builtin precision tests" into oreo-cts-dev

Permit different errors in context creation tests
am: 990fc78bd1

Change-Id: I1d23f694114ff6498354010267df2b8b1633186e

Permit different errors in context creation tests

dEQP-EGL.functional.negative_api.create_context tests error conditions
EGL supports OpenGL ES, but not a specifically requested version (e.g.,
creating an OpenGL ES 1 context when only OpenGL ES 2 is supported). The
error that should be raised varies based on the governing specification.
EGL 1.4 mandates EGL_BAD_CONFIG. EGL 1.5 mandates EGL_BAD_MATCH.
EGL_KHR_create_context has langage mandating *both* errors in separate
paragraphs.

Accept either error code as appropriate for EGL version and extension
support.

DO NOT MERGE: Merge Oreo MR1 into master
am: 5afbb268c4  -s ours

Change-Id: If4fbb47b1bf271462e3f39bccd92a4565fe01ab4

DO NOT MERGE: Merge Oreo MR1 into master

Exempt-From-Owner-Approval: Changes already landed internally
Change-Id: I71216cb23c2124938de9ae2a75b7cd7a0811d57f

Don't assume rendering complete before buffer latch
am: 9e0214cf61

Change-Id: Iee3ccf2afef375ac45e60133537a141d685d607f

Don't assume rendering complete before buffer latch am: 9e0214cf61
am: eb88a8ca3d

Change-Id: Ic8680791e52474a3d100314c6c90406ae8319a55

Don't assume rendering complete before buffer latch
am: 9e0214cf61

Change-Id: I72902995d20e746aa7066a46b097be4a2dab5453

Limit maximum allocation size in memory mapping tests

Fixes also ReferenceMemory::m_flushed memory usage counting when
estimating reference memory usage (size should be divided by atomSize).

Fixes #468
Bug: b/69677943

Change-Id: Iad754e211e8070b09254d2451e03418fc2be541b

Improve memory tracking in vulkan memory mapping tests

- Round up allocation size to multiple of page size for
  memory usage tracking.
- Update reference memory usage tracking to follow new reference memory
  implementation.

Bug: b/69677943

Change-Id: I219c2b3abe0ee0731f5a7e2bb44492e5b85b95c2

Optimize random memory mapping tests reference memory

Bug: b/69677943

Change-Id: I7af6fe67a25462dcb0acc66c238a8708157060df

Don't assume rendering complete before buffer latch

It's valid for the buffer to latch before rendering is completed, and
this is causing intermittent failures.

Bug: b/69981429

Test: --deqp-case=dEQP-EGL*get_frame_timestamps*

Change-Id: Ia9f1d0e4ffa7f3e16a46fc4566157a651d1b0601

Avoid watchdog timeout in Vulkan builtin precision tests

Calculating the reference intervals can take a substantial amount of
time. On some very slow targets, this can cause us to trip the watchdog
in the verification loop.

Same approach and limit as in modules/glshared/glsBuiltinPrecisionTests.cpp

Affected tests:
dEQP-VK.glsl.builtin.precision.*

Component: Vulkan

Bug: 67854695
Test: run on google devices
Change-Id: I492b656e7e2a9aa724bb3785bdf1416a4c22ec04

Reduce GPU workload of pipeline_barrier tests

Tests with large buffer sizes will now clamp the
GPU workload to avoid triggering execution timeout
on implementations.

b/68360750

Affects: dEQP-VK-memory.pipeline_barrier.*

Components: Vulkan

VK_GL_CTS issue: 813

Change-Id: If8772973fd49cac0b8cc5fa66c33af4de38f2f68

x11: Call XInitThreads()
am: a8bd356e58

Change-Id: I4b4980ba14f0e0482ccbe1d5d26b149b9e63ef05

Merge commit 'eb9533febd01cde88ea23e2b4fc7fa35efb086ab' into HEAD

Change-Id: If7b754ce356073d0bfedca11e361c781f48b2c45

x11: Call XInitThreads()

Because the X11 manual says we must.

From man:XInitThreads(3):

  The XInitThreads function initializes Xlib support for concurrent
  threads.  This function must be the first Xlib function
  a multi-threaded program calls, and it must complete before any other
  Xlib call is made.

(See also Vulkan spec section "Xlib Platform")

Fixes crash in multithreaded test:

  Test case 'dEQP-EGL.functional.multithread.window'..
  [xcb] Unknown request in queue while dequeuing
  [xcb] Most likely this is a multi-threaded client and XInitThreads has not been called
  [xcb] Aborting, sorry about that.
  deqp-egl: xcb_io.c:165: dequeue_pending_request: Assertion `!xcb_xlib_unknown_req_in_deq' failed.
  Aborted (core dumped)

Change-Id: I2124be592a8d2d9036423cc2f0d71066e0e22939