Mesa (18.1): 21 new commits

Dylan Baker dbaker at kemper.freedesktop.org
Fri May 11 15:34:26 UTC 2018


URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=ce9f53f5d9685b94baf35be5faed96fcecd18abd
Author: Dylan Baker <dylan at pnwbakers.com>
Date:   Fri May 11 08:34:16 2018 -0700

    Bump version to rc4

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=e4c54e10315e312cd1a8c05175e67b99e4167e37
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date:   Wed May 9 15:06:13 2018 -0700

    i965,anv: Set the CS stall bit on the ISP disable PIPE_CONTROL
    
    From the bspec docs for "Indirect State Pointers Disable":
    
        "At the completion of the post-sync operation associated with this
        pipe control packet, the indirect state pointers in the hardware are
        considered invalid"
    
    So the ISP disable is a post-sync type of operation which means that it
    should be combined with a CS stall.  Without this, the simulator throws
    an error.
    
    Fixes: 766d801ca "anv: emit pixel scoreboard stall before ISP disable"
    Fixes: f536097f6 "i965: require pixel scoreboard stall prior to ISP disable"
    Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
    (cherry picked from commit a8a740f272a808a2694524b43fc33d2f0c0e3709)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=3b9b1175aaec05544fe0ac02d486d6a9286e3188
Author: Matt Turner <mattst88 at gmail.com>
Date:   Wed May 9 16:36:58 2018 -0700

    gallium/tests: Fix assignment of EXTRA_DIST
    
    Fixes: 6754c2e83d79 ("autotools: Include new meson files")
    (cherry picked from commit 0f959215c340150cb6075f5c2d3ccfc5d109558f)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=7da026cb98204f9a74a3f2521cd620c6e056921f
Author: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
Date:   Tue May 8 17:25:55 2018 +0100

    anv: emit pixel scoreboard stall before ISP disable
    
    We want to make sure that all indirect state data has been loaded into
    the EUs before disable the pointers.
    
    Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
    Reviewed-by: Rafael Antognolli <rafael.antognolli at intel.com>
    Fixes: 78c125af3904c ("anv/gen10: Ignore push constant packets during context restore.")
    (cherry picked from commit 766d801ca32118a722fb2e58a48ee9a96897d3b7)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=2c140183d411b17951a2a570674b9adab222d10a
Author: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
Date:   Tue May 1 12:32:45 2018 +0100

    i965: require pixel scoreboard stall prior to ISP disable
    
    Invalidating the indirect state pointers might affect a previously
    scheduled & still running 3DPRIMITIVE (causing page fault). So stall
    on pixel scoreboard before that.
    
    v2: Fix compile issue :(
    
    v3: Stall on pixel scoreboard
    
    v4: Drop the post sync operation (Lionel)
    
    Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
    Reviewed-by: Rafael Antognolli <rafael.antognolli at intel.com>
    Fixes: ca19ee33d7d39 ("i965/gen10: Ignore push constant packets during context restore.")
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106243
    (cherry picked from commit f536097f67521180dafd270b28ac9a852af9c141)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=e3106b460eaeb3744d289d0d949891a7912ff816
Author: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
Date:   Wed May 9 16:40:37 2018 +0100

    i965: silence unused variable
    
    Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
    Fixes: 2dc29e095f9da ("i965: Don't leak blorp on Gen4-5.")
    Reviewed-by: Caio Marcelo de Oliveira Filho <caio.oliveira at intel.com>
    (cherry picked from commit 3853f1c6f4b97edde22c767a80c137da6e39904a)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=5519bb3c659165c487603fa67bff90d62b606a5c
Author: Jan Vesely <jan.vesely at rutgers.edu>
Date:   Tue May 8 01:39:04 2018 -0400

    winsys/radeon: Destroy fd_hash table when the last winsys is removed.
    
    Fixes memory leak on module unload.
    v2: Use util_hash_table helper function
    
    CC: <mesa-stable at lists.freedesktop.org>
    Reviewed-by: Marek Olšák <marek.olsak at amd.com>
    Signed-off-by: Jan Vesely <jan.vesely at rutgers.edu>
    (cherry picked from commit 45dfa6f4e77fbb21f312eb6101db6c25acd4d483)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=5fc5b1dab831b8846fb09db7cad6f83f00c01111
Author: Jan Vesely <jan.vesely at rutgers.edu>
Date:   Wed May 9 15:06:33 2018 -0400

    gallium/auxiliary: Add helper function to count the number of entries in hash table
    
    CC: <mesa-stable at lists.freedesktop.org>
    Reviewed-by: Marek Olšák <marek.olsak at amd.com>
    Signed-off-by: Jan Vesely <jan.vesely at rutgers.edu>
    (cherry picked from commit d146768d139f887105464f0db5600dd046452a8f)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=e90f0d711c6429d34aa60fef4c878c0badce1ea3
Author: Dave Airlie <airlied at redhat.com>
Date:   Wed May 9 23:17:09 2018 +0100

    r600: fix constant buffer bounds.
    
    If you have an indirect access to a constant buffer on r600/eg
    use a vertex fetch in the shader. However apps have expected
    behaviour on those out of bounds accessess (even if illegal).
    
    If the constants were being uploaded as part of a larger
    upload buffer, we'd set the range of allowed access to a lot
    larger than required so apps would get values back from
    other parts of the upload buffer instead of the expected out
    of bounds access.
    
    This fixes rendering bugs in Trine and Witcher 1, thanks
    to iive for nagging me effectively until I figured it out :-)
    
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91808
    Cc: <mesa-stable at lists.freedesktop.org>
    
    Reviewed-by: Roland Scheidegger <sroland at vmware.com>
    (cherry picked from commit ce027ac5c798b39582288e5d7d9973b3cdda591e)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=0d2d5d2ffa090c610612c5e677e3946b238d46dc
Author: Ross Burton <ross.burton at intel.com>
Date:   Wed May 9 21:48:43 2018 +0100

    src/intel/Makefile.vulkan.am: add missing MKDIR_GEN
    
    Out of tree builds can try to write into a directory that doesn't exist yet:
    
    | Traceback (most recent call last):
    |   File "../../../mesa-18.0.2/src/intel/vulkan/anv_icd.py", line 46, in <module>
    |     with open(args.out, 'w') as f:
    | IOError: [Errno 2] No such file or directory: 'vulkan/intel_icd.x86_64.json'
    | Makefile:4882: recipe for target 'vulkan/intel_icd.x86_64.json' failed
    
    Add missing MKDIR_GEN calls to solve this.
    
    Cc: <mesa-stable at lists.freedesktop.org>
    Reviewed-by: Matt Turner <mattst88 at gmail.com>
    (cherry picked from commit 1755654d9f533240dc19954b0dbccaf74e645265)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=8c384f62a27c013ea06ff7093bf4e195572b809c
Author: Rhys Perry <pendingchaos02 at gmail.com>
Date:   Fri May 4 23:27:51 2018 +0100

    mesa: fix error handling in get_framebuffer_parameteriv
    
    CC: <mesa-stable at lists.freedesktop.org>
    Signed-off-by: Rhys Perry <pendingchaos02 at gmail.com>
    Reviewed-by: Tapani Pälli <tapani.palli at intel.com>
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
    (cherry picked from commit 5ac16ed0476d9914927b8e7a592d8d7ac1589586)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=312c2f047e5ce7fe277d34d7d99b9945c60f4e60
Author: Michel Dänzer <michel.daenzer at amd.com>
Date:   Fri Apr 27 17:41:48 2018 +0200

    dri3: Only update number of back buffers in loader_dri3_get_buffers
    
    And only free no longer needed back buffers there as well.
    
    We want to stick to the same back buffer throughout a frame, otherwise
    we can run into various issues.
    
    Bugzilla: https://bugs.freedesktop.org/105906
    Bugzilla: https://bugs.freedesktop.org/106399
    Fixes: 3160cb86aa92 "egl/x11: Re-allocate buffers if format is suboptimal"
    Reported-by: Sergii Romantsov <sergii.romantsov at globallogic.com>
    Tested-by: Eero Tamminen <eero.t.tamminen at intel.com>
    Acked-by: Daniel Stone <daniels at collabora.com>
    (cherry picked from commit 6f81e07ecb8c0793dc482307d5d96fd3df95b7d2)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=eea5ec86d594b396acc6bb18b8f85cf1965fbbfa
Author: Jan Vesely <jan.vesely at rutgers.edu>
Date:   Mon May 7 23:40:47 2018 -0400

    pipe-loader: Free driver_name in error path
    
    CC: <mesa-stable at lists.freedesktop.org>
    Signed-off-by: Jan Vesely <jan.vesely at rutgers.edu>
    Reviewed-by: Marek Olšák <marek.olsak at amd.com>
    (cherry picked from commit 0783399d796823fbefaab7af5551d0df90bd9dac)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=2136602167046dab855d0fadf45c09ce51ecc32b
Author: Brian Paul <brianp at vmware.com>
Date:   Tue May 8 08:24:21 2018 -0600

    glsl: change ast_type_qualifier bitset size to work around GCC 5.4 bug
    
    Change the size of the bitset from 128 bits to 96.  This works around an
    apparent GCC 5.4 bug in which bad SSE code is generated, leading to a
    crash in ast_type_qualifier::validate_in_qualifier() (ast_type.cpp:654).
    
    This can be repro'd with the Piglit test tests/spec/glsl-1.50/execution/
    varying-struct-basic-gs-fs.shader_test
    
    Bugzilla:https://bugs.freedesktop.org/show_bug.cgi?id=105497
    Cc: mesa-stable at lists.freedesktop.org
    Reviewed-by: Charmaine Lee <charmainel at vmware.com>
    Tested-by: Charmaine Lee <charmainel at vmware.com>
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
    (cherry picked from commit 901db25d5b7cd2ac2dd648b370c4bddf23dd5c44)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=3cdbd79dccdcf2aae7155f27add58d09fca59c52
Author: Jan Vesely <jan.vesely at rutgers.edu>
Date:   Mon May 7 22:54:09 2018 -0400

    eg/compute: Drop reference to kernel_param bo in destructor
    
    CC: <mesa-stable at lists.freedesktop.org>
    Signed-off-by: Jan Vesely <jan.vesely at rutgers.edu>
    Reviewed-by: Marek Olšák <marek.olsak at amd.com>
    (cherry picked from commit a9e4be9212868f619ac492aaf86b0aa68d4395c4)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=795fffc2dfcd9b74ed126efebd7246c8ebd123d9
Author: Jan Vesely <jan.vesely at rutgers.edu>
Date:   Thu May 3 23:55:00 2018 -0400

    eg/compute: Drop reference on code_bo in destructor.
    
    Signed-off-by: Jan Vesely <jan.vesely at rutgers.edu>
    Reviewed-by: Nicolai Hähnle <nicolai.haehnle at amd.com>
    (cherry picked from commit ea1fff4416036066cff51826f95b4703d7211008)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=817414b00b4aa95268e7a4f478b8cb708ea2b030
Author: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
Date:   Mon Apr 23 17:52:37 2018 +0100

    intel: devinfo: fix assertion on devices with odd number of EUs
    
    I forgot to change the assert in the second helper function in a
    previous change.
    
    This hit the assert() on a Broadwell platform with 1 slice, 3
    subslices but all EUs disabled in subslice 1 & 2.
    
    Fixes: c1900f5b0fb ("intel: devinfo: add helper functions to fill fusing masks values")
    Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
    Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
    (cherry picked from commit 3cdf1bf97db19b94577b49079cbbc521157d4e03)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=bf24436c6dca56e0afbe96ecdc2e3e77faf4dc5b
Author: Kenneth Graunke <kenneth at whitecape.org>
Date:   Mon Apr 23 10:40:04 2018 -0700

    i965: Don't leak blorp on Gen4-5.
    
    We used to only initialize BLORP on Gen6+.  When we added it on Gen4-5,
    we forgot to destroy it unconditionally.
    
    Fixes: 752d7af77a52898cebf5597def4fdd38b1d6303e (i965: Add blorp support for gen4-5)
    Reviewed-by: Matt Turner <mattst88 at gmail.com>
    (cherry picked from commit 2dc29e095f9daffd9da722919fa08ca912ab8f04)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=95474b31c43666dc40efc6ca5571a203206ce041
Author: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>
Date:   Sat May 5 15:34:44 2018 +0200

    vulkan/wsi: Only use LINEAR modifier for prime if supported.
    
    This was setting the LINEAR modifier if neither the
    X server nor the driver supported modifiers.
    
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106180
    Fixes: c80c08e226 "vulkan/wsi/x11: Add support for DRI3 v1.2"
    CC: 18.1 <mesa-stable at lists.freedesktop.org>
    Tested-by: Abel Garcia Dorta <mercuriete at gmail.com>
    Acked-by: Daniel Stone <daniels at collabora.com>
    Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>
    (cherry picked from commit b17cfb08a3fc9a599eff64fffe48daba398a672f)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=f366968e86b65955b6b0de0dafcbb72e9925ca5d
Author: Jan Vesely <jan.vesely at rutgers.edu>
Date:   Mon May 7 22:52:19 2018 -0400

    r600: Cleanup constant buffers on context destruction
    
    CC: <mesa-stable at lists.freedesktop.org>
    Signed-off-by: Jan Vesely <jan.vesely at rutgers.edu>
    Reviewed-by: Marek Olšák <marek.olsak at amd.com>
    (cherry picked from commit a1e8fcce3eafa59228bb9bb50179c04f150ca9ca)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=227fbe2799a01ce66e7d886d4007829d4cfffc43
Author: Ian Romanick <ian.d.romanick at intel.com>
Date:   Fri Jun 16 18:48:22 2017 -0700

    mesa: Add missing support for glFogiv(GL_FOG_DISTANCE_MODE_NV)
    
    Found by inspection, so I made a piglit test too.
    
    Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
    Cc: mesa-stable at lists.freedesktop.org
    Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
    (cherry picked from commit f2db3be6200410ffd269f8789f3c97292ce03b0f)




More information about the mesa-commit mailing list