Mesa (17.1): 26 new commits

Andres Gomez tanty at kemper.freedesktop.org
Wed Sep 6 21:10:24 UTC 2017


URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=5bc4e6553271087f12617eae6cedeb44641a50fe
Author: Andres Gomez <agomez at igalia.com>
Date:   Wed Sep 6 21:32:58 2017 +0300

    cherry-ignore: add "radeonsi/gfx9: proper workaround for LS/HS VGPR initialization bug"
    
    fixes: References 166823bf but it is not really a bug fix, just an
    enhancement.
    
    Signed-off-by: Andres Gomez <agomez at igalia.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=7cbc780d9a8994f3d25f7abdca80a175785af6db
Author: Andres Gomez <agomez at igalia.com>
Date:   Wed Sep 6 21:29:52 2017 +0300

    cherry-ignore: add "radv: Fix vkCopyImage with both depth and stencil aspects."
    
    fixes: Depends on earlier commit 78bef01da2b that did not land in
    branch.
    
    Signed-off-by: Andres Gomez <agomez at igalia.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=69f86c43cf0d4e50d4849e23b111a3eb23c81e27
Author: Andres Gomez <agomez at igalia.com>
Date:   Wed Sep 6 21:23:10 2017 +0300

    cherry-ignore: add "radeonsi/gfx9: always flush DB metadata on framebuffer changes"
    
    stable: 17.2 nomination only. Depends on earlier commit 5b62eb237c52
    which did not land in branch.
    
    Signed-off-by: Andres Gomez <agomez at igalia.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=a5b82d2691a15d24ccda50ce6a974c2b555c5ad4
Author: Andres Gomez <agomez at igalia.com>
Date:   Mon Sep 4 14:01:17 2017 +0300

    cherry-ignore: add "i965: Fix crash in fallback GTT mapping."
    
    stable: 17.2 nomination only. Depends on earlier commit f37ede40bad
    which did not land in branch.
    
    Signed-off-by: Andres Gomez <agomez at igalia.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=066969aa3575314e260ee7281a7e1382d0b57de7
Author: Andres Gomez <agomez at igalia.com>
Date:   Tue Aug 29 10:29:16 2017 +0300

    cherry-ignore: add "nir: Fix system_value_from_intrinsic for subgroups"
    
    stable: 17.2 nomination only. Depends on earlier commit 43ef75b394f
    which did not land in branch.
    
    Signed-off-by: Andres Gomez <agomez at igalia.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=ba3f439505efcf881e978797a136a984a04b40b1
Author: Andres Gomez <agomez at igalia.com>
Date:   Wed Aug 30 17:13:13 2017 +0300

    cherry-ignore: added 17.2 nominations.
    
    stable: 17.2 nominations only.
    
    Signed-off-by: Andres Gomez <agomez at igalia.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=80dcb7d4fdf86c0e90b53bfe60afb0106bfd703d
Author: Eric Engestrom <eric.engestrom at imgtec.com>
Date:   Thu Aug 31 16:55:56 2017 +0000

    util: improve compiler guard
    
    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 <lordheavym at gmail.com>
    Cc: Emil Velikov <emil.l.velikov at gmail.com>
    Cc: Rob Herring <robh at kernel.org>
    Signed-off-by: Eric Engestrom <eric.engestrom at imgtec.com>
    Reviewed-by: Laurent Carlier <lordheavym at gmail.com>
    Reviewed-by: Jose Fonseca <jfonseca at vmware.com>
    (cherry picked from commit 49b428470e28ae6ab22083e43fa41abf622f3b0d)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=eb5eb5b26d3dffdbfac9c12a9113a16859a7f0c0
Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Mon Aug 21 22:11:49 2017 -0700

    spirv: Add support for the HelperInvocation builtin
    
    I have no idea how this got missed but it's been missing since forever.
    
    Cc: mesa-stable at lists.freedesktop.org
    Reviewed-by: Samuel Iglesias Gonsálvez <siglesias at igalia.com>
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
    (cherry picked from commit e439908af9665b50443f1196cb55388c69d0c7d7)
    [Andres Gomez: resolve trivial conflicts]
    Signed-off-by: Andres Gomez <agomez at igalia.com>
    
    Conflicts:
    	src/compiler/spirv/vtn_variables.c

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=864a8113e5a0092480a0a0066d8cc0b88695ac7d
Author: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>
Date:   Sun Sep 3 23:35:37 2017 +0200

    radv: Actually set the cmd_buffer usage_flags.
    
    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 <airlied at redhat.com>
    (cherry picked from commit dec7b38fe62a1db46c5150a7368d3bb3c5e45305)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=ded9b0a168764352d48f2eca6a0b00f5ee68dd0d
Author: Roland Scheidegger <sroland at vmware.com>
Date:   Fri Sep 1 01:48:42 2017 +0200

    st/mesa: fix view template initialization in try_pbo_readpixels
    
    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 <brianp at vmware.com>
    Reviewed-by: Marek Olšák <marek.olsak at amd.com>
    Cc: mesa-stable at lists.freedesktop.org
    (cherry picked from commit 2b2c61f0df5c18355b65772d21be36339ba5e1d9)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=d3300c42b2bcbddb9f44d02e076915fc2fb1d828
Author: Karol Herbst <karolherbst at gmail.com>
Date:   Sun Aug 27 18:00:52 2017 +0200

    nvc0: write 0 to pipeline_statistics.cs_invocations
    
    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 <karolherbst at gmail.com>
    Reviewed-by: Ilia Mirkin <imirkin at alum.mit.edu>
    Cc: mesa-stable at lists.freedesktop.org
    (cherry picked from commit b672c3833b7ea51c09a6bcbed86b8fb6789448f1)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=e529fd4e116da284aa3340dc5427fa602a588163
Author: Ben Crocker <bcrocker at redhat.com>
Date:   Wed Aug 23 16:32:43 2017 -0400

    llvmpipe: lp_build_gather_elem_vec BE fix for 3x16 load
    
    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" <mesa-stable at lists.freedesktop.org>
    
    Signed-off-by: Ben Crocker <bcrocker at redhat.com>
    Reviewed-by: Roland Scheidegger <sroland at vmware.com>
    (cherry picked from commit 57c8ead0cd08e6aaf88a389f7ce528c4f0face65)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=a1bdc43dd203396647a03a50e64d67707d8322b5
Author: Ray Strode <rstrode at redhat.com>
Date:   Wed Aug 23 16:32:42 2017 -0400

    gallivm: correct channel shift logic on big endian
    
    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" <mesa-stable at lists.freedesktop.org>
    Signed-off-by: Ray Strode <rstrode at redhat.com>
    Reviewed-by: Roland Scheidegger <sroland at vmware.com>
    (cherry picked from commit 75cb6e36178b9474bbb59b76cbbcce2a67bf88d2)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=f4266d73b54d821a63c97a94d7494e7689e5dc5a
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date:   Thu Jul 27 15:04:25 2017 -0700

    anv/formats: Nicely handle unknown VkFormat enums
    
    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 <lionel.g.landwerlin at intel.com>
    Bugzilla: https://bugs.freedesktop.org/101956
    Cc: mesa-stable at lists.freedesktop.org
    (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 <jason.ekstrand at intel.com>
    Signed-off-by: Eric Engestrom <eric at engestrom.ch>
    Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>
    Reviewed-by: Samuel Iglesias Gonsálvez <siglesias at igalia.com>
    (cherry picked from commit 0c7272a66c633b0b11c0b81c0f3552201d083b3a)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=018e602dc629dfbda52b5c7711b5867d72ce33c8
Author: Charmaine Lee <charmainel at vmware.com>
Date:   Wed Aug 30 14:26:41 2017 -0700

    vbo: fix offset in minmax cache key
    
    Instead of saving primitive offset in the minmax cache key,
    save the actual buffer offset which is used in the cache lookup.
    
    Fixes rendering artifact seen with GoogleEarth when run with
    VMware driver.
    
    v2: Per Brian's comment, initialize offset to avoid compiler warning.
    
    Cc: mesa-stable at lists.freedesktop.org
    Reviewed-by: Brian Paul <brianp at vmware.com>
    (cherry picked from commit 2d93b462b4d978b0da417b35a7470e336bc4e783)
    [Andres Gomez: resolve trivial conflicts]
    Signed-off-by: Andres Gomez <agomez at igalia.com>
    
    Conflicts:
    	src/mesa/vbo/vbo_minmax_index.c
    
    Squashed with:
    
    vbo: fix build errors on android
    
    incompatible pointer to integer conversion assigning to 'GLintptr' (aka 'int')
    from 'const char *' [-Werror,-Wint-conversion]
    
          offset = indices;
                 ^ ~~~~~~~
    
    Fixes: 2d93b462b4d ("vbo: fix offset in minmax cache key")
    Signed-off-by: Tapani Pälli <tapani.palli at intel.com>
    Reviewed-by: Charmaine Lee <charmainel at vmware.com>
    Reviewed-by: Emil Velikov <emil.velikov at collabora.com>
    (cherry picked from commit 0986f686328216fa201769c630372fd4b6f8877a)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=82061efd48c7302c892cd9236b6f4abae54239ce
Author: Grazvydas Ignotas <notasas at gmail.com>
Date:   Mon Aug 28 00:29:36 2017 +0300

    radv: don't assert on empty hash table
    
    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 <notasas at gmail.com>
    Reviewed-by: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>
    (cherry picked from commit b8dd69e1b49a5c4c5c82e34f804a97f7448ff6c3)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=5b4678c3bcc7512f6ac020bb21441f986c55cbe4
Author: Emil Velikov <emil.velikov at collabora.com>
Date:   Sun Aug 27 11:20:28 2017 +0100

    egl/wayland: polish object teardown in dri2_wl_destroy_surface
    
    The wl_drm wrapper is created before the wl display/surface ones.
    Thus make sure we destroy it after them. In reality it should not make
    any difference either way.
    
    Fixes: 03dd9a88b0b ("egl/wayland: Use per-surface event queues")
    Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
    Reviewed-by: Daniel Stone <daniels at collabora.com>
    (cherry picked from commit 1a8015e753c6f52b24a473ea60e3ce0b38de7d42)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=618c883b8ec99162c02f1eba6796b281f987afc2
Author: Emil Velikov <emil.velikov at collabora.com>
Date:   Sun Aug 27 11:20:27 2017 +0100

    egl/wayland: plug leaks in dri2_wl_create_window_surface() error path
    
    We forgot to teardown the wl display/surface wrappers.
    
    Fixes: 03dd9a88b0b ("egl/wayland: Use per-surface event queues")
    Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
    Reviewed-by: Daniel Stone <daniels at collabora.com>
    (cherry picked from commit 83442112d7136b6d0c2f6d84c409c04664aeb154)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=52e70819b4302d06dbfc3fd3dd655e79aefba7c8
Author: Michael Olbrich <m.olbrich at pengutronix.de>
Date:   Tue Aug 29 11:51:20 2017 +0200

    egl/dri2: only destroy created objects
    
    dri2_display_destroy may be called by dri2_initialize_wayland_drm() if
    initialization fails. In this case, these objects may not be initialized.
    
    Cc: mesa-stable at lists.freedesktop.org
    Signed-off-by: Michael Olbrich <m.olbrich at pengutronix.de>
    Reviewed-by: Tapani Pälli <tapani.palli at intel.com>
    Reviewed-by: Emil Velikov <emil.velikov at collabora.com>
    (cherry picked from commit 81d5c31631840db704337489cf677cc596da79f5)
    [Andres Gomez: resolve trivial conflicts]
    Signed-off-by: Andres Gomez <agomez at igalia.com>
    
    Conflicts:
    	src/egl/drivers/dri2/egl_dri2.c

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=edbb6d405cb69849501c1df87dff69504604a2e6
Author: Grazvydas Ignotas <notasas at gmail.com>
Date:   Mon Aug 28 02:01:21 2017 +0300

    radv: clear dynamic_shader_stages on create
    
    Valgrind reports it's being used uninitialized.
    
    Fixes: f4e499ec791 "radv: add initial non-conformant radv vulkan driver"
    Signed-off-by: Grazvydas Ignotas <notasas at gmail.com>
    Reviewed-by: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>
    (cherry picked from commit 77803748336713854a4ade192253c0ba01c49047)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=47b22fdf7f3236b5699b19fb7514668416806c0c
Author: Emil Velikov <emil.velikov at collabora.com>
Date:   Mon Aug 7 17:23:19 2017 +0100

    egl: don't NULL deref the .get_capabilities function pointer
    
    One could easily introduce version 3 of the DRI2fenceExtension,
    extending the struct, while not implementing the above function.
    
    Thus we'll end up with NULL pointer, and dereferencing it won't fare
    too well.
    
    Fixes: 0201f01dc4e ("egl: add EGL_ANDROID_native_fence_sync")
    Cc: Rob Clark <robclark at freedesktop.org>
    Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
    Reviewed-by: Tapani Pälli <tapani.palli at intel.com>
    (cherry picked from commit f0d053cb6dda908d554d84629bd98e05d383a8f6)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=72a90be2c8c8274d14f5343b5c093a1f0132d469
Author: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>
Date:   Fri Aug 25 22:18:24 2017 +0200

    radv: Fix sparse BO mapping merging.
    
    If we merge a mapping with the mapping before it, we also need
    to not only change the offset, but also the bo offset.
    
    Fixes: 715df30a4e2 "radv/amdgpu: Add winsys implementation of virtual buffers."
    Reviewed-by: Dave Airlie <airlied at redhat.com>
    (cherry picked from commit 9b7e663da1d88f398a349e158c27b38a66b73fe3)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=ba87ab0543b460f0a4af37df53f1db0c5c8d0bf0
Author: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>
Date:   Fri Aug 25 14:14:12 2017 +0200

    radv: Fix off by one in MAX_VBS assert.
    
    e.g. 0 + 32 <= 32 should be valid.
    
    Fixes: f4e499ec791 "radv: add initial non-conformant radv vulkan driver"
    Tested-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
    Reviewed-by: Dave Airlie <airlied at redhat.com>
    (cherry picked from commit fba0e078695847bf1e4fdc9e6a44099bf901f4cf)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=371d75334ca411c49f2b2389efa61afa0603c862
Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Sun Aug 27 00:41:05 2017 -0400

    st/mesa: fix handling of vertex array double inputs
    
    The is_double_vertex_input needs to be set for arrays of doubles as
    well.
    
    Fixes KHR-GL45.enhanced_layouts.varying_array_locations
    
    Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
    Reviewed-by: Timothy Arceri <tarceri at itsqueeze.com>
    Reviewed-by: Nicolai Hähnle <nicolai.haehnle at amd.com>
    Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
    Cc: mesa-stable at lists.freedesktop.org
    (cherry picked from commit ae53bff8b13b433ca79904dfbda7264eb7188fa7)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=70f7f60f0cfe37fffb458aae09d3854eba0c5a27
Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Sun Aug 27 00:09:31 2017 -0400

    glsl: fix counting of vertex shader output slots used by explicit vars
    
    The argument to count_attribute_slots should only be set to true for
    vertex inputs, not for all vertex shader varyings.
    
    Fixes KHR-GL45.enhanced_layouts.varying_locations
    
    Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
    Reviewed-by: Timothy Arceri <tarceri at itsqueeze.com>
    Reviewed-by: Nicolai Hähnle <nicolai.haehnle at amd.com>
    Cc: mesa-stable at lists.freedesktop.org
    (cherry picked from commit eefeff09a784eb139cbf682b98926c0eaa00eb21)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=c62dc8e61e721ca7fcc3c01b74229cf2efa580f9
Author: Christian Gmeiner <christian.gmeiner at gmail.com>
Date:   Fri Aug 25 13:39:05 2017 +0200

    etnaviv: use correct param for etna_compatible_rs_format(..)
    
    Found by code inspection.
    
    Fixes: c9e8b49b885 ("etnaviv: gallium driver for Vivante GPUs")
    Cc: mesa-stable at lists.freedesktop.org
    Signed-off-by: Christian Gmeiner <christian.gmeiner at gmail.com>
    Reviewed-by: Eric Engestrom <eric.engestrom at imgtec.com>
    (cherry picked from commit 67fc3e37a7a38e279082848c064d5faacad44f54)




More information about the mesa-commit mailing list