summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* docs: add release notes for 17.2.1mesa-17.2.1Emil Velikov2017-09-171-0/+199
| | | | Signed-off-by: Emil Velikov <[email protected]>
* Update version to 17.2.1Emil Velikov2017-09-171-1/+1
| | | | Signed-off-by: Emil Velikov <[email protected]>
* radv: Don't allocate CMASK for linear images.Bas Nieuwenhuizen2017-09-131-1/+3
| | | | | | | | We can't use it anyway in fast clears, and on GFX9 it seems to actually hange the card if we specify it. Fixes: f4e499ec791 "radv: add initial non-conformant radv vulkan driver" (cherry picked from commit 1a172fb113554fa03053052b161d5e22fc2fcb1f)
* radv: Disable multilayer & multilevel DCC.Bas Nieuwenhuizen2017-09-131-0/+1
| | | | | | | | | | | | | The current DCC init routine doesn't account for initializing a single layer or level. Multilayer seems hard for small textures on pre-GFX9 as tre metadata for the layers can be interleaved. For GFX9 multilevel textures are a problem for similar reasons. So just disable this for now, until we handle the texture modes correctly. Fixes: f4e499ec791 "radv: add initial non-conformant radv vulkan driver" (cherry picked from commit bee83b26611c0a4a554aa37c59187351242e82fd)
* docs/egl: remove reference to EGL_DRIVERS_PATHEric Engestrom2017-09-131-21/+0
| | | | | | | | | Support for external egl drivers was dropped a few years ago. Fixes: 209360bbb91bb10346eb "egl/main: drop support for external egl drivers" Signed-off-by: Eric Engestrom <[email protected]> Reviewed-by: Emil Velikov <[email protected]> (cherry picked from commit 85b66d20969685c26d3f7d22b6fc7b988872fa88)
* radv/winsys: fix flags vs va_flags thinko.Dave Airlie2017-09-131-1/+1
| | | | | | Fixes: e8d57802f (radv/gfx9: allocate events from uncached VA space) Signed-off-by: Dave Airlie <[email protected]> (cherry picked from commit a5add6fb30e24c4e0177a98758888682d7e6ea36)
* egl/x11/dri3: adding missing __DRI_BACKGROUND_CALLABLE extensionEmil Velikov2017-09-131-0/+1
| | | | | | | | Fixes: 3b7b6adf3ac ("egl: Implement __DRI_BACKGROUND_CALLABLE") Cc: Timothy Arceri <[email protected]> Signed-off-by: Emil Velikov <[email protected]> Reviewed-by: Marek Olšák <[email protected]> (cherry picked from commit f24bc1816242895d0bc0711a50bd2dfd3cb8b21a)
* radv: Fix vkCopyImage with both depth and stencil aspects.Bas Nieuwenhuizen2017-09-131-99/+107
| | | | | | Fixes: f4e499ec791 "radv: add initial non-conformant radv vulkan driver" Reviewed-by: Dave Airlie <[email protected]> (cherry picked from commit ff23e03d60d264a64fce340c132404c6070050a0)
* radv: Actually set the cmd_buffer usage_flags.Bas Nieuwenhuizen2017-09-131-0/+1
| | | | | | | | | Otherwise, the simultaneous uage bit doesn't get set from the begin info, which we need for batchchaining. Fixes: f4e499ec791 "radv: add initial non-conformant radv vulkan driver" Reviewed-by: Dave Airlie <[email protected]> (cherry picked from commit dec7b38fe62a1db46c5150a7368d3bb3c5e45305)
* radv: don't assert on empty hash tableGrazvydas Ignotas2017-09-131-0/+3
| | | | | | | | | | | | | | Currently if table_size is 0, it's falling through to: unreachable("hash table should never be full"); But table_size can be 0 when RADV_DEBUG=nocache is set, or when the table allocation fails (which is not considered an error). Fixes: f4e499ec791 "radv: add initial non-conformant radv vulkan driver" Signed-off-by: Grazvydas Ignotas <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> (cherry picked from commit b8dd69e1b49a5c4c5c82e34f804a97f7448ff6c3)
* mesa/st: remove unwanted backup fileEric Engestrom2017-09-131-479/+0
| | | | | | | | Fixes: 0ac78dc92582a59d4319 "util: move string_to_uint_map to glsl" Cc: Emil Velikov <[email protected]> Signed-off-by: Eric Engestrom <[email protected]> Reviewed-by: Emil Velikov <[email protected]> (cherry picked from commit ac0d8dc3fa3b294104d0c8ba54262e7b07389cb9)
* radeonsi: update dirty_level_mask before dispatchingSamuel Pitoiset2017-09-132-0/+6
| | | | | | | | | | This fixes a rendering issue with Hitman when bindless textures are enabled. Fixes: 2263610827 ("radeonsi: flush DB caches only when transitioning from DB to texturing") Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Marek Olšák <[email protected]> (cherry picked from commit 59101e771d63d3d623828a173454fdbbcb719ec3)
* glsl: fix glsl_struct_field size calculations for shader cacheNicolai Hähnle2017-09-131-7/+4
| | | | | | | | | | | | | | | | | | | Found by address sanitizer: ==22621==ERROR: AddressSanitizer: heap-buffer-overflow on address 0x61400000cbd8 at pc 0x7f561610a4ff bp 0x7ffca85f9d50 sp 0x7ffca85f94f8 READ of size 344 at 0x61400000cbd8 thread T0 #0 0x7f561610a4fe (/usr/lib/x86_64-linux-gnu/libasan.so.3+0x5f4fe) #1 0x7f560bb305a5 in memcpy /usr/include/x86_64-linux-gnu/bits/string3.h:53 #2 0x7f560bb305a5 in blob_write_bytes ../../../mesa-src/src/compiler/glsl/blob.c:136 #3 0x7f560be7d7ff in encode_type_to_blob ../../../mesa-src/src/compiler/glsl/shader_cache.cpp:153 #4 0x7f560be81222 in write_program_resource_data ../../../mesa-src/src/compiler/glsl/shader_cache.cpp:950 #5 0x7f560be81222 in write_program_resource_list ../../../mesa-src/src/compiler/glsl/shader_cache.cpp:1118 #6 0x7f560be81222 in shader_cache_write_program_metadata(gl_context*, gl_shader_program*) ../../../mesa-src/src/compiler/glsl/shader_cache.cpp:1407 #7 0x7f560b825fdb in link_program ../../../mesa-src/src/mesa/main/shaderapi.c:1163 Fixes: 073a84ff60db ("glsl: stop adding pointers from glsl_struct_field to the cache") Reviewed-by: Timothy Arceri <[email protected]> (cherry picked from commit 4da6cf6c98aeca8c143ecb5e0ada5e1eee81a0f5)
* cherry-ignore: add EGL+gbm swast patchesEmil Velikov2017-09-131-0/+4
| | | | | | They depend on an earlier commit which did not land in branch. Signed-off-by: Emil Velikov <[email protected]>
* ac/surface: match Z and stencil tile configNicolai Hähnle2017-09-131-6/+47
| | | | | | | | | | | | | Fixes various piglit tests on Stoney, see the comment. Cc: [email protected] Reviewed-by: Marek Olšák <[email protected]> (cherry picked from commit cffc0ae0d9e572a82ce88c09e6ee8e2ed2a9be91) [Emil Velikov: attribute for the different gfx6_surface_settings()] Signed-off-by: Emil Velikov <[email protected]> Conflicts: src/amd/common/ac_surface.c
* radeonsi: apply a mask to gl_SampleMaskIn in the PS prologNicolai Hähnle2017-09-133-5/+76
| | | | | | | | | | | | | | | | | | | gl_SampleMaskIn is supposed to contain set bits only for the samples that are covered by the current fragment shader invocation, but the VGPR initialization hardware loads the set of all bits that are covered at the current pixel. Fixes various tests in dEQP-GLES31.functional.shaders.sample_variables.sample_mask_in.* Cc: [email protected] Reviewed-by: Marek Olšák <[email protected]> (cherry picked from commit 92c4277990799641d4986ce66a62366228713945) [Emil Velikov: attribute for the lack of add_arg*checked() API] Signed-off-by: Emil Velikov <[email protected]> Conflicts: src/gallium/drivers/radeonsi/si_shader.c
* glsl: stop adding pointers from bindless structs to the cacheTimothy Arceri2017-09-131-4/+8
| | | | | | | | This is so we always create reproducible cache entries. Consistency is required for verification of any third party distributed shaders. Reviewed-by: Samuel Pitoiset <[email protected]> (cherry picked from commit 4009370232f87227655f8f41949e9a56650fb1f5)
* glsl: stop adding pointers from shader_info to the cacheTimothy Arceri2017-09-131-6/+25
| | | | | | | | This is so we always create reproducible cache entries. Consistency is required for verification of any third party distributed shaders. Reviewed-by: Samuel Pitoiset <[email protected]> (cherry picked from commit a6618afd2740f997cc23bd75de712af6452589a7)
* compiler: move pointers to the start of shader_infoTimothy Arceri2017-09-131-3/+3
| | | | | | | | This will allow us to easily skip them when writting the struct to disk cache. Reviewed-by: Samuel Pitoiset <[email protected]> (cherry picked from commit 3ea3f757237d977adfb69293654522647d1812b3)
* glsl: always write a name/label string to the cacheTimothy Arceri2017-09-131-4/+7
| | | | | | | | | | In the following patch we will stop writing the pointer to cache. Unfortunately adding empty strings to that cache seems to be the only thing we can do here once we no longer have the pointers. Reviewed-by: Samuel Pitoiset <[email protected]> (cherry picked from commit 44918a197905e4e9d09e32b7d07945948ca2da07)
* glsl: don't write uniform storage offset if there isn't oneTimothy Arceri2017-09-131-3/+10
| | | | | | | | This is so we always create reproducible cache entries. Consistency is required for verification of any third party distributed shaders. Reviewed-by: Samuel Pitoiset <[email protected]> (cherry picked from commit 22154823d21f17a5fdff7b1fe3c92b61f5389d88)
* glsl: add has_uniform_storage() helper to shader cacheTimothy Arceri2017-09-131-6/+13
| | | | | Reviewed-by: Samuel Pitoiset <[email protected]> (cherry picked from commit 2662269ad791141eb61c14d7a57df2800fa6d91c)
* glsl: stop adding pointers from glsl_struct_field to the cacheTimothy Arceri2017-09-131-7/+38
| | | | | | | | This is so we always create reproducible cache entries. Consistency is required for verification of any third party distributed shaders. Reviewed-by: Samuel Pitoiset <[email protected]> (cherry picked from commit 073a84ff60dbaef6331e2bcd8f5ca2de0b2ff7d2)
* glsl: stop adding pointers from gl_shader_variable to the cacheTimothy Arceri2017-09-131-12/+28
| | | | | | | | This is so we always create reproducible cache entries. Consistency is required for verification of any third party distributed shaders. Reviewed-by: Samuel Pitoiset <[email protected]> (cherry picked from commit 37d453b55a7b4c1b10d28dba4acfe12de4634bc8)
* glsl: allow NULL to be passed to encode_type_to_blob()Timothy Arceri2017-09-131-0/+10
| | | | | | | This will be used by the following commit. Reviewed-by: Samuel Pitoiset <[email protected]> (cherry picked from commit 37eb67714e0092a5d80ec40d10ca1fe72fa64657)
* util: improve compiler guardEric Engestrom2017-09-133-6/+43
| | | | | | | | | | | | | | | | Glibc 2.26 has dropped xlocale.h, but the functions needed (strtod_l() and strdof_l()) can be found in stdlib.h. Improve the detection method to allow newer builds to still make use of the locale-setting. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102454 Cc: Laurent Carlier <[email protected]> Cc: Emil Velikov <[email protected]> Cc: Rob Herring <[email protected]> Signed-off-by: Eric Engestrom <[email protected]> Reviewed-by: Laurent Carlier <[email protected]> Reviewed-by: Jose Fonseca <[email protected]> (cherry picked from commit 49b428470e28ae6ab22083e43fa41abf622f3b0d)
* i965: Set "Subslice Hashing Mode" to 16x16 on Apollolake.Kenneth Graunke2017-09-132-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As of 4.11, the kernel isn't bothering to set the subslice hashing mode on Apollolake, leaving it at the default of 8x8. (It initializes it to 16x4 on most platforms.) Performance data for GPUTest Triangle on Apollolake at 1024x640: X-tiled RT: ----------- 8x8 -> 16x4: 2.4325% +/- 0.383683% (n=107) 8x8 -> 8x4: -3.75105% +/- 0.592491% (n=40) 8x8 -> 16x16: 6.17238% +/- 0.67157% (n=30) Y-tiled RT: ----------- 8x8 -> 16x4: 1.30307% +/- 0.297292% (n=205) 8x8 -> 8x4: -0.769282% +/- 0.729557% (n=35) 8x8 -> 16x16: 3.00254% +/- 0.715503% (n=40) 8x MSAA RT (INTEL_FORCE_MSAA=8): -------------------------------- 8x8 -> 16x4: 1.38889% +/- 0.93729% (n=7) 8x8 -> 8x4: -2.10643% +/- 1.15153% (n=3) 8x8 -> 16x16: 3.87183% +/- 1.08851% (n=5) Based on this, we choose 16x16 for Apollolake. Skylake GT2 with X-tiled buffers appears to be a toss-up between 16x4 and 16x16, and with Y-tiled buffers it doesn't seem to really matter. So we'll leave Skylake alone for now. The hashing mode doesn't seem to make a measurable impact on more complex benchmarks. Acked-by: Matt Turner <[email protected]> (cherry picked from commit ebd2fd6ef3b50dc85190b407d0921c91b6c4f1ca)
* radv/gfx9: fix image resource handling.Dave Airlie2017-09-131-8/+19
| | | | | | | | | | | GFX9 changes how images are layed out, so this needs updating. Fixes: dEQP-VK.query_pool.statistics_query.* Cc: "17.2" <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> Signed-off-by: Dave Airlie <[email protected]> (cherry picked from commit 3633bae36b56a8667d31096b3c3472ab388c4fbf)
* radv/ac: bump params array for image atomic comp swapDave Airlie2017-09-131-1/+1
| | | | | | | | | | | | | For the comp_swap case this was overflowing and crashing sometimes. Fixes: dEQP-VK.image.atomic_operations.compare_exchange.* Cc: "17.2" <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> Signed-off-by: Dave Airlie <[email protected]> (cherry picked from commit aba441be44a2e4bb914e805c4cd064c72e4ea038)
* radv/gfx9: set mip0-depth correctly for 2d arrays/3d imagesDave Airlie2017-09-131-2/+2
| | | | | | | | | | | | | This field covers the whole resource. Fixes: dEQP-VK.pipeline.image.suballocation.sampling_type.combined.view_type.3d.format.* dEQP-VK.texture.filtering.3d.combinations.* Cc: "17.2" <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> Signed-off-by: Dave Airlie <[email protected]> (cherry picked from commit ebd2a5354df64ef504f1e213afd2d6ab9d69a776)
* radv: handle GFX9 1D texturesDave Airlie2017-09-132-14/+76
| | | | | | | | | | | | | | | | As GFX9 can't handle 1D depth textures, radeonsi and apparantly pro just update all 1D textures to 2D, and work around it. This ports the workarounds from radeonsi. Reviewed-by: Bas Nieuwenhuizen <[email protected]> Cc: "17.2" <[email protected]> Signed-off-by: Dave Airlie <[email protected]> (cherry picked from commit 1bcb953e16600e0d9329b678a6eacee98d326547) Conflicts: src/amd/common/ac_nir_to_llvm.c
* radv: don't use iview for meta image width/height.Dave Airlie2017-09-132-13/+21
| | | | | | | | | | | | | Work out the width/height from the level manually, as on GFX9 we won't minify the iview width/height. This fixes: dEQP-VK.api.image_clearing.core.clear_color_image* on gfx9 Reviewed-by: Bas Nieuwenhuizen <[email protected]> Cc: "17.2" <[email protected]> Signed-off-by: Dave Airlie <[email protected]> (cherry picked from commit 2f5b4490b59062f224e73f7a2d5cc0d8bd8eaa7b)
* cherry-ignore: add execution_type() fix to the listEmil Velikov2017-09-131-0/+3
| | | | | | | As mentioned by Matt: "Without commit 4fab67a4415, this patch isn't needed at all." Signed-off-by: Emil Velikov <[email protected]>
* st/glsl_to_tgsi: only the first (inner-most) array reference can be a 2D indexNicolai Hähnle2017-09-131-1/+1
| | | | | | | | | | Don't get distracted by record dereferences between array references. Fixes dEQP-GLES31.functional.tessellation.user_defined_io.per_vertex_block.* Cc: [email protected] Reviewed-by: Marek Olšák <[email protected]> (cherry picked from commit 03203b74486357c2bc77c53302f0f667f1df3ffa)
* radv: use simpler indirect packet 3 if possible.Dave Airlie2017-09-131-14/+24
| | | | | | | | | | | | | | This fixes some observed hangs on CIK GPUs. Reviewed-by: Bas Nieuwenhuizen <[email protected]> Cc: "17.2" <[email protected]> Signed-off-by: Dave Airlie <[email protected]> (cherry picked from commit 219d29e4d83d8c901f127c8a004c45c23c15751e) [Emil Velikov: add the hunk in radv_emit_indirect_draw] Signed-off-by: Emil Velikov <[email protected]> Conflicts: src/amd/vulkan/radv_cmd_buffer.c
* radv/gfx9: allocate events from uncached VA spaceDave Airlie2017-09-133-3/+8
| | | | | | | | | | | | | This copies what amdgpu-pro does, and allocates the memory for an event with an uncached mtype. This fixes hangs with: dEQP-VK.api.command_buffers.record_simul_use_primary Reviewed-by: Bas Nieuwenhuizen <[email protected]> Cc: "17.2" <[email protected]> Signed-off-by: Dave Airlie <[email protected]> (cherry picked from commit e8d57802fea60d4b353761ba1bdb08545733c007)
* radv/winsys: use amdgpu_bo_va_op_raw.Dave Airlie2017-09-131-7/+24
| | | | | | | | | | | This is a precursor to the gfx9 fix to use uncached for the event memory. Move to the interface which allows setting the flags, but wrap it to avoid having to copy it around the place. Reviewed-by: Bas Nieuwenhuizen <[email protected]> Cc: "17.2" <[email protected]> Signed-off-by: Dave Airlie <[email protected]> (cherry picked from commit 76ac8fafadca86941f2723f7f7176b22112f5c0c)
* st/mesa: skip draw calls with pipe_draw_info::count == 0Marek Olšák2017-09-131-1/+6
| | | | | | | | | | Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102502 Cc: 17.2 <[email protected]> Tested-by: Alexandre Demers <[email protected]> Tested-by: Dieter Nützel <[email protected]> Acked-by: Timothy Arceri <[email protected]> (cherry picked from commit e4018fdd85d8b58250c2cff8cdf4edd6b2b3c70b)
* radeonsi/gfx9: always flush DB metadata on framebuffer changesNicolai Hähnle2017-09-133-2/+12
| | | | | | | | | | | | | | This fixes GL45-CTS.shader_image_load_store.basic-glsl-earlyFragTests. Cc: [email protected] Reviewed-by: Marek Olšák <[email protected]> (cherry picked from commit 34124e412f00432ba8b3b8d16e3f2168aa596622) [Emil Velikov: resolve trivial conflicts] Signed-off-by: Emil Velikov <[email protected]> Conflicts: src/gallium/drivers/radeonsi/si_pipe.h src/gallium/drivers/radeonsi/si_state.c
* Revert "radv: disable support for VEGA for now."Dave Airlie2017-09-131-5/+0
| | | | | | | | | | | | | This reverts commit 611076a41aac3095a82dff2432943d7f8d429822. With the two previous commits, vega shouldn't be unstable, doesn't pass CTS, but can do a complete run, and games shouldn't hang anymore, so bring it back online. Reviewed-by: Bas Nieuwenhuizen <[email protected]> Cc: "17.2" <[email protected]> Signed-off-by: Dave Airlie <[email protected]> (cherry picked from commit e38685cc62e2a452521c49a3ad09d82495c6b1d0)
* radv/gfx9: set descriptor up for base_mip to level range.Dave Airlie2017-09-131-1/+4
| | | | | | | | | | | This is required on GFX9, fixes a bug in Talos where all the mipmaps overlay each other. Just pushing this as well as it fixes Talos. Cc: "17.2" <[email protected]> Signed-off-by: Dave Airlie <[email protected]> (cherry picked from commit 6d929d3f85604768971d04fa2548f02ffca1696e)
* radv: disable 1d/2d linear optimisation on gfx9.Dave Airlie2017-09-131-3/+4
| | | | | | | | | | | | | | This causes hangs in some of the CTS tests with a 2d 1536x2 texture. This fixes hangs with: dEQP-VK.pipeline.image.suballocation.sampling_type.combined.iew_type.1d_aray.format.r4g4b4a4_unorm_pack16.count_1.size.512x1_array_of_3 if we reenable it, make sure these don't regress. Reviewed-by: Bas Nieuwenhuizen <[email protected]> Cc: "17.2" <[email protected]> Signed-off-by: Dave Airlie <[email protected]> (cherry picked from commit d118ff876559e468d2d243470947eee53171fe57)
* spirv: Add support for the HelperInvocation builtinJason Ekstrand2017-09-131-1/+4
| | | | | | | | | I have no idea how this got missed but it's been missing since forever. Cc: [email protected] Reviewed-by: Samuel Iglesias Gonsálvez <[email protected]> Reviewed-by: Ian Romanick <[email protected]> (cherry picked from commit e439908af9665b50443f1196cb55388c69d0c7d7)
* st/mesa: fix view template initialization in try_pbo_readpixelsRoland Scheidegger2017-09-131-1/+1
| | | | | | | | | | | | | I think this is what the code was meant to do, albeit as far as I can tell the redundant initialization some analyzers complain about should work as well just fine (only the first layer will be used, if the view contains one or more layers doesn't really matter). Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102467 Reviewed-by: Brian Paul <[email protected]> Reviewed-by: Marek Olšák <[email protected]> Cc: [email protected] (cherry picked from commit 2b2c61f0df5c18355b65772d21be36339ba5e1d9)
* i965: Fix crash in fallback GTT mapping.Kenneth Graunke2017-09-131-2/+4
| | | | | | | | We can't perf_debug without a context. Cc: [email protected] Reviewed-by: Anuj Phogat <[email protected]> (cherry picked from commit 52b65dfda84e4d433a332dd84596eef054eb5cbb)
* freedreno: skip batch-cache for compute shadersRob Clark2017-09-131-7/+1
| | | | | | | | | It is kind of pointless for compute, and avoids issues with apps kicking off more than 32 compute shaders at once. Signed-off-by: Rob Clark <[email protected]> Cc: "17.2" <[email protected]> (cherry picked from commit dc9e08b0c3b04ba77ed59b8700e9f43edccb3168)
* nvc0: write 0 to pipeline_statistics.cs_invocationsKarol Herbst2017-09-131-0/+1
| | | | | | | | | | | | | | cs_invocations are currently unsupported, but leaving the field uninitialized is even worse. fixes on nvc0: * KHR-GL45.pipeline_statistics_query_tests_ARB.functional_default_qo_values * KHR-GL45.pipeline_statistics_query_tests_ARB.functional_non_rendering_commands_do_not_affect_queries Signed-off-by: Karol Herbst <[email protected]> Reviewed-by: Ilia Mirkin <[email protected]> Cc: [email protected] (cherry picked from commit b672c3833b7ea51c09a6bcbed86b8fb6789448f1)
* llvmpipe: lp_build_gather_elem_vec BE fix for 3x16 loadBen Crocker2017-09-131-2/+28
| | | | | | | | | | | | | | | | | | | | | | | Fix loading of a 3x16 vector as a single 48-bit load on big-endian systems (PPC64, S390). Roland Scheidegger's commit e827d9175675aaa6cfc0b981e2a80685fb7b3a74 plus Ray Strode's patch reduce pre-Roland Piglit failures from ~4000 to ~2000. This patch fixes three of the four regressions observed by Ray: - draw-vertices - draw-vertices-half-float - draw-vertices-half-float_gles2 One regression remains: - draw-vertices-2101010 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=100613 Cc: "17.2" "17.1" <[email protected]> Signed-off-by: Ben Crocker <[email protected]> Reviewed-by: Roland Scheidegger <[email protected]> (cherry picked from commit 57c8ead0cd08e6aaf88a389f7ce528c4f0face65)
* gallivm: correct channel shift logic on big endianRay Strode2017-09-131-1/+7
| | | | | | | | | | | | | | | | | | | | lp_build_fetch_rgba_soa fetches a texel from a texture. Part of that process involves first gathering the element together from memory into a packed format, and then breaking out the individual color channels into separate, parallel arrays. The code fails to account for endianess when reading the packed values. This commit attempts to correct the problem by reversing the order the packed values are read on big endian systems. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=100613 Cc: "17.2" "17.1" <[email protected]> Signed-off-by: Ray Strode <[email protected]> Reviewed-by: Roland Scheidegger <[email protected]> (cherry picked from commit 75cb6e36178b9474bbb59b76cbbcce2a67bf88d2)
* anv/formats: Nicely handle unknown VkFormat enumsJason Ekstrand2017-09-131-5/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes some crashes in the dEQP-VK.memory.requirements.core.* tests. I'm not sure whether or not passing out-of-bound formats into the query is supposed to be allowed but there's no harm in protecting ourselves from it. Reviewed-by: Lionel Landwerlin <[email protected]> Bugzilla: https://bugs.freedesktop.org/101956 Cc: [email protected] (cherry picked from commit 242211933a06826961709c2689a1d30f735ab7b9) Squashed with: anv: fix off by one in array check `anv_formats[ARRAY_SIZE(anv_formats)]` is already one too far. Spotted by Coverity. CovID: 1417259 Fixes: 242211933a0682696170 "anv/formats: Nicely handle unknown VkFormat enums" Cc: Jason Ekstrand <[email protected]> Signed-off-by: Eric Engestrom <[email protected]> Reviewed-by: Jason Ekstrand <[email protected]> Reviewed-by: Samuel Iglesias Gonsálvez <[email protected]> (cherry picked from commit 0c7272a66c633b0b11c0b81c0f3552201d083b3a)