Mesa (20.0): 27 new commits

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Wed Feb 19 19:28:47 UTC 2020


URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=9abde3412d3572837646ffd681d9009e3b5d259d
Author: Dylan Baker <dylan at pnwbakers.com>
Date:   Wed Feb 19 11:27:25 2020 -0800

    VERSION: bump for 20.0.0 release

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=dfdfddc4bb593841a0d47f515f40a498916543cf
Author: Dylan Baker <dylan at pnwbakers.com>
Date:   Wed Feb 19 11:27:00 2020 -0800

    docs: Empty new_features.txt

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=604fa0dcad1be0b8db7eb9c8f95bcd0eb5f2fc4e
Author: Dylan Baker <dylan at pnwbakers.com>
Date:   Wed Feb 19 11:26:47 2020 -0800

    Docs: Add 20.0.0 release notes

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=555cb9e7d217c14680b61cee62605c005e086ebc
Author: Danylo Piliaiev <danylo.piliaiev at globallogic.com>
Date:   Wed Feb 12 12:45:55 2020 +0200

    st/nir: Unify inputs_read/outputs_written before serializing NIR
    
    Otherwise input/output interfaces won't be unified when reading
    NIR from a cache.
    
    Fixes piglit test on iris:
      clip-distance-vs-gs-out.shader_test
    
    Fixes: 19ed12af
    
    Signed-off-by: Danylo Piliaiev <danylo.piliaiev at globallogic.com>
    Reviewed-by: Timothy Arceri <tarceri at itsqueeze.com>
    Tested-by: Marge Bot <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3787>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3787>
    (cherry picked from commit b684ba6ce7fc6cabf42132559192e1065891e67a)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=a06fc57fda64a62a48fa3b07cb55e9aae2569788
Author: luc <814640709 at qq.com>
Date:   Sat Feb 15 12:55:16 2020 +0800

    zink: confused compilation macro usage for zink in target helpers.
    
    Fixes: 8d46e35d16e ("zink: introduce opengl over vulkan")
    Reviewed-by: Erik Faye-Lund <erik.faye-lund at collabora.com>
    Tested-by: Marge Bot <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3831>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3831>
    (cherry picked from commit 692093fbdc93343dbe500128fdd23167d73036d9)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=f27e5d9df5bc9c85d45c2cb1f2a4997b453365fe
Author: Mathias Fröhlich <mathias.froehlich at web.de>
Date:   Fri Dec 13 17:09:56 2019 +0100

    egl: Implement getImage/putImage on pbuffer swrast.
    
    This change adds getImage/putImage callbacks to the swrast pbuffer
    loader extension.
    This fixes a recent crash with Weston as well as a crashing
    test with classic swrast without an official gitlab issue.
    
    v2: Determine bytes per pixel differently and fix non X11 builds.
    v3: Plug memory leak and fix crash on out of bounds access.
        (Daniel Stone)
    v4: Follow the code structure of the wayland get/put image
        implementation - hopefully being more obvious.
        Handle 64 bits formats.
        Use BufferSize directly.
        (Emil Velikov)
    v5: Change pixel size computation.
        (Eric Engestrom)
    
    Reviewed-by: Daniel Stone <daniels at collabora.com>
    Reviewed-by: Emil Velikov <emil.velikov at collabora.com>
    Closes: https://gitlab.freedesktop.org/mesa/mesa/issues/2219
    Fixes: d6edccee8da "egl: add EGL_platform_device support"
    Signed-off-by: Mathias Fröhlich <Mathias.Froehlich at web.de>
    Tested-by: Marge Bot <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3711>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3711>
    (cherry picked from commit c7617d8908a970124321ce731b43d5996c3c5775)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=52714f58ff0252227d9b2a2e12e412df341747b4
Author: Alyssa Rosenzweig <alyssa.rosenzweig at collabora.com>
Date:   Thu Feb 13 17:14:05 2020 -0500

    pan/midgard: Use fprintf instead of printf for constants
    
    I was wondering where those constants disappeared to :-)
    
    Signed-off-by: Alyssa Rosenzweig <alyssa.rosenzweig at collabora.com>
    Fixes: 968f36d1fc0 ("pan/midgard: Support disassembling to a file")
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3835>
    (cherry picked from commit 8ab0bf1f939af480997fafd8bf562644a60df08a)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=01dab4b7feef6575cff55187cedee1d3f0609079
Author: Alyssa Rosenzweig <alyssa.rosenzweig at collabora.com>
Date:   Thu Feb 13 17:12:51 2020 -0500

    pan/midgard: Don't crash with constants on unknown ops
    
    Just use a dummy name instead.. we can't know a priori what type an
    unknown op will consume, but we don't want to dereference a null
    pointer.
    
    Signed-off-by: Alyssa Rosenzweig <alyssa.rosenzweig at collabora.com>
    Fixes: 24360966ab3 ("panfrost/midgard: Prettify embedded constant
    prints")
    
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3835>
    (cherry picked from commit 6af14d3685fac433193b92f9ad6c9f8a3eaf87ff)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=3b65640ea556f323a0b42187edc2fff0b6e6dd10
Author: Alyssa Rosenzweig <alyssa.rosenzweig at collabora.com>
Date:   Mon Feb 3 15:12:24 2020 -0500

    pan/midgard: Fix missing prefixes
    
    I was wondering where those were going... :)
    
    Signed-off-by: Alyssa Rosenzweig <alyssa.rosenzweig at collabora.com>
    Fixes: c1952779d68 ("pan/decode: Dump to a file")
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3835>
    (cherry picked from commit fbe1fd3de0aa7c618286ee79082f7bbcd7b8e171)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=e6d598b69f510527ca75d6947b2a2813a7524ba9
Author: Francisco Jerez <currojerez at riseup.net>
Date:   Fri Jan 24 15:09:52 2020 -0800

    intel/fs/gen12: Workaround data coherency issues due to broken NoMask control flow.
    
    Together with the fixup_nomask_control_flow() pass introduced in a
    previous patch, this implements a less invasive alternative to the
    workaround documented in the hardware spec for GEN:BUG:1407528679,
    which doesn't involve disabling structured control flow.
    
    Under some conditions Gen12 hardware can end up executing a BB with
    all channels disabled, which will lead to the execution of any NoMask
    instructions in it, even though any execution-masked instructions will
    be correctly shot down.  This could break assumptions of the SWSB pass
    if the data computed by a NoMask instruction is synchronized against
    by using an SWSB annotation baked into a regular execution-masked
    instruction, since the first (NoMask) instruction may be executed
    redundantly by the hardware, even though the second will correctly be
    shot down, potentially leading to a RaW or WaW hazard if a third
    instruction subsequently accesses the destination register of the
    first instruction.
    
    Reviewed-by: Caio Marcelo de Oliveira Filho <caio.oliveira at intel.com>
    Cc: 20.0 <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit e14529ff3262a527d630cecac655f69c8ae15c3f)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=5188cd51e2bf2ab98913472e9376735178dd7c76
Author: Francisco Jerez <currojerez at riseup.net>
Date:   Thu Jan 23 22:27:21 2020 -0800

    intel/fs/gen12: Fixup/simplify SWSB annotations of SIMD32 scratch writes.
    
    Found by inspection.  Existing code was trying to avoid assuming that
    an SBID had been assigned to the virtual instruction, but
    synchronizing the header setup with respect to the previous SIMD16
    SEND by using SYNC.ALLRD doesn't really seem possible unless the SEND
    instruction had been assigned an SBID.  Assert-fail instead if no SBID
    has been allocated.
    
    Fixes: 15e3a0d9d264becc "intel/eu/gen12: Set SWSB annotations in hand-crafted assembly."
    Reviewed-by: Caio Marcelo de Oliveira Filho <caio.oliveira at intel.com>
    Cc: 20.0 <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit 4e4e8d793f050eac84f2a850ab2e5c24c4c459ac)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=96052fdc24ecd95667e4b1cc86c1b4abfded676e
Author: Francisco Jerez <currojerez at riseup.net>
Date:   Thu Jan 23 22:55:33 2020 -0800

    intel/fs/gen12: Workaround unwanted SEND execution due to broken NoMask control flow.
    
    This is a less invasive alternative to the workaround documented in
    the hardware spec for GEN:BUG:1407528679, which doesn't involve
    disabling structured control flow (it's unlikely that switching to
    GOTO/JOIN would have actually fixed the problem anyway).
    
    Under some conditions Gen12 hardware can end up executing a BB with
    all channels disabled, which will lead to the execution of any NoMask
    instructions in it, even though any execution-masked instructions will
    be correctly shot down.  This may break assumptions of some NoMask
    SEND messages whose descriptor depends on data generated by live
    invocations of the shader.
    
    This avoids the problem by predicating certain instructions on an ANY
    horizontal predicate that makes sure that their execution is omitted
    when all channels of the program are disabled.  The shader-db impact
    of this patch seems to be minimal:
    
    total instructions in shared programs: 17169833 -> 17169913 (0.00%)
    instructions in affected programs: 30663 -> 30743 (0.26%)
    helped: 0
    HURT: 42
    
    total cycles in shared programs: 336966176 -> 336968568 (0.00%)
    cycles in affected programs: 2367290 -> 2369682 (0.10%)
    helped: 0
    HURT: 13
    
    Reviewed-by: Caio Marcelo de Oliveira Filho <caio.oliveira at intel.com>
    Cc: 20.0 <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit a8ac0bd759cbf9a5984df4bc9f553a3dca41a8ab)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=fa5b4bb2df3237c2247f13a845c37c2293708058
Author: Francisco Jerez <currojerez at riseup.net>
Date:   Thu Jan 23 23:01:32 2020 -0800

    intel/fs: Add virtual instruction to load mask of live channels into flag register.
    
    Reviewed-by: Caio Marcelo de Oliveira Filho <caio.oliveira at intel.com>
    Cc: 20.0 <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit 008f95a043dac909f6e647c3102f37bb978b148c)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=5df7a39c1b2579c1856a5ac2d7556c5afdfe050a
Author: Francisco Jerez <currojerez at riseup.net>
Date:   Thu Jan 23 23:00:54 2020 -0800

    intel/fs/gen7: Fix fs_inst::flags_written() for SHADER_OPCODE_FIND_LIVE_CHANNEL.
    
    We need to pass a width of 32 since the opcode bashes the whole f1.0
    register on IVB.  This is unlikely to have caused problems since f1.0
    is largely unused currently.  That's likely to change soon though,
    even on platforms other than Gen7.
    
    Reviewed-by: Caio Marcelo de Oliveira Filho <caio.oliveira at intel.com>
    Cc: 20.0 <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit b8b509fb921a7c2f687b9f7e5075c72847740ac4)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=d92cf573457d9614ed7ba5a9c1cc78038075c577
Author: Francisco Jerez <currojerez at riseup.net>
Date:   Thu Jan 23 22:01:00 2020 -0800

    intel/fs/cse: Make HALT instruction act as CSE barrier.
    
    Found by inspection.  This seems particularly likely to cause problems
    with instructions dependent on the current execution mask like
    SHADER_OPCODE_FIND_LIVE_CHANNEL or the FS_OPCODE_LOAD_LIVE_CHANNELS
    instruction I'm about to introduce, but one could imagine it leading
    to data corruption if CSE ever managed to combine two instructions
    before and after the FS_OPCODE_PLACEHOLDER_HALT, since the one before
    may not be executed for some channels.
    
    Reviewed-by: Caio Marcelo de Oliveira Filho <caio.oliveira at intel.com>
    Cc: 20.0 <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit c9e33e5cbf6b8b953932f283e1f0abcb6c77eb1f)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=5547fc53c04b7a6c7323066c4f6a711f4ec0fdb0
Author: Marek Olšák <marek.olsak at amd.com>
Date:   Wed Feb 12 14:55:27 2020 -0500

    radeonsi: don't wait for shader compilation to finish when destroying a context
    
    This was a hack for glsl_types deinitialization and it predates the proper
    fix, which was the addition of glsl_type_singleton_decref.
    
    This fixes a crash when the context is destroyed via the atexit handler.
    
    Cc: 19.3 20.0 <mesa-stable at lists.freedesktop.org>
    Reviewed-by: Timothy Arceri <tarceri at itsqueeze.com>
    Tested-by: Marge Bot <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3800>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3800>
    (cherry picked from commit 7e2b4bf256610cc016202893d7b4b4ef60b25b53)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=99d3ad1c71fa755baa3c81f27b05e792a29ea4ba
Author: Dylan Baker <dylan at pnwbakers.com>
Date:   Tue Feb 18 11:21:39 2020 -0800

    .pick_status.json: Update to bee5c9b0dc13dbae0ccf124124eaccebf7f2a435

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=0dce180b4386860e18539c77c38afbcf57b8c37a
Author: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer at amd.com>
Date:   Thu Jan 30 12:16:46 2020 +0100

    radeonsi/ngg: add VGT_FLUSH when enabling fast launch
    
    (cherry-picked from 3da91b3327fb93d0364c0ca9d0216f695160831d)
    
    Tested-by: Marge Bot <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3846>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3846>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=1a35b3d28691f5dd8dde36420af5090a05a350f3
Author: Krzysztof Raszkowski <krzysztof.raszkowski at intel.com>
Date:   Mon Feb 17 12:26:40 2020 +0100

    gallium/swr: simplify environmental variabled expansion code
    
    There were 2 versions of code doing the same thing.
    Since std::regexp are locale-sensitive better is to leave old
    good way to do this.
    This commit fix problem with bd934ff613c006344e757dfdac9f136ead1b183a.
    
    Reviewed-by: Jan Zielinski <jan.zielinski at intel.com>
    Tested-by: Marge Bot <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3843>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3843>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=ed9060683173df5e052ff15363c72464f27e3229
Author: Timothy Arceri <tarceri at itsqueeze.com>
Date:   Mon Feb 10 11:22:32 2020 +1100

    glsl: fix gl_nir_set_uniform_initializers() for image arrays
    
    The if was incorrectly checking for an image type on what could
    be an array of images. Here we change it to use the type stored
    in uniform storage which has already been stripped of arrays,
    this is what the above code for samplers does also.
    
    Fixes: 2bf91733fcb5 ("nir/linker: Set the uniform initial values")
    
    Reviewed-by: Alejandro Piñeiro <apinheiro at igalia.com>
    Tested-by: Marge Bot <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3757>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3757>
    (cherry picked from commit 676869e1d4bb1660430fcdb99443238a7de50eb8)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=ea43cc67026345680476d80fac4cb5b41d2b3f90
Author: Dylan Baker <dylan at pnwbakers.com>
Date:   Fri Feb 14 09:40:58 2020 -0800

    .pick_status.json: Update to 946eacbafb47c8b94d47e7c9d2a8b02fff5a22fa

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=edcc8ba73adc233852cf82909f76a421208930b0
Author: Thong Thai <thong.thai at amd.com>
Date:   Thu Feb 13 11:03:12 2020 -0500

    Revert "st/va: Convert interlaced NV12 to progressive"
    
    This reverts commit 2add63060b51ea2ae432d10e1bd52d6cc0a4dcbb.
    
    Closes: https://gitlab.freedesktop.org/mesa/mesa/issues/2454
    Fixes: 2add63060b51 "st/va: Convert interlaced NV12 to progressive"
    Signed-off-by: Thong Thai <thong.thai at amd.com>
    Acked-by: Leo Liu <leo.liu at amd.com>
    Tested-by: Marge Bot <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3815>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3815>
    (cherry picked from commit 08cff938b76b6fe146334e44dc97b6be8274a281)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=6aacd69a027a162c7c5374a4169c253135588e2c
Author: Erik Faye-Lund <erik.faye-lund at collabora.com>
Date:   Wed Feb 12 12:48:37 2020 +0100

    Revert "nir: Add a couple trivial abs optimizations"
    
    These were already added in 9fdaeb7776c ("nir: add min/max optimisation"),
    and there's no point in doing them twice.
    
    This reverts commit e4d346c86db0ae332fcdf55eac0e075cfb99a7eb.
    
    Fixes: e4d346c86db ("nir: Add a couple trivial abs optimizations")
    Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
    Tested-by: Marge Bot <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3786>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3786>
    (cherry picked from commit 0c1ba69a27090d3ef000943b47468705fe0454c4)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=1c0b76a3826185f9d0572eea1fec8464b0516b4e
Author: Tapani Pälli <tapani.palli at intel.com>
Date:   Wed Feb 12 08:45:47 2020 +0200

    iris: fix aux buf map failure in 32bits app on Android
    
    Cc: mesa-stable at lists.freedesktop.org
    Reported-by: Zhifang Long <zhifang.long at intel.com>
    Signed-off-by: Tapani Pälli <tapani.palli at intel.com>
    Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
    Tested-by: Marge Bot <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3784>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3784>
    (cherry picked from commit fdd20be324eabab7da1ba67cf7e379398d771186)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=a429930e0ac97694b69eb1f13c3050eca8753449
Author: Tapani Pälli <tapani.palli at intel.com>
Date:   Mon Jan 27 10:05:20 2020 +0200

    glsl: fix a memory leak with resource_set
    
       ==7265== 248 (120 direct, 128 indirect) bytes in 1 blocks are definitely lost in loss record 1,438 of 1,465
       ==7265==    at 0x483980B: malloc (vg_replace_malloc.c:309)
       ==7265==    by 0x598A2AB: ralloc_size (ralloc.c:119)
       ==7265==    by 0x598F861: _mesa_set_create (set.c:127)
       ==7265==    by 0x599079D: _mesa_pointer_set_create (set.c:570)
       ==7265==    by 0x58BD7D1: build_program_resource_list(gl_context*, gl_shader_program*, bool) (linker.cpp:4026)
       ==7265==    by 0x548231B: st_link_shader (st_glsl_to_ir.cpp:170)
       ==7265==    by 0x54DA269: _mesa_glsl_link_shader (ir_to_mesa.cpp:3119)
    
    Fixes: a6aedc66 ("st/glsl_to_nir: use nir based program resource list builder")
    Signed-off-by: Tapani Pälli <tapani.palli at intel.com>
    Reviewed-by: Timothy Arceri <tarceri at itsqueeze.com>
    Tested-by: Marge Bot <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3574>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3574>
    (cherry picked from commit f7d1bf075ab833eb39c35f9e213dc4fc115b46b1)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=5c0a93b6eb3ed3e32ed053bbce5bc2d22f713148
Author: Peng Huang <shawn.p.huang at gmail.com>
Date:   Sat Feb 1 22:31:00 2020 -0500

    radeonsi: make si_fence_server_signal flush pipe without work
    
    glSignalSemaphoreEXT sometime doesn't signal the semaphore, it is
    because radeonsi doesn't flush if gl context doesn't have pending
    work. Fix the porblem by always submit ib.
    
    Reviewed-by: Marek Olšák <marek.olsak at amd.com>
    Cc: 19.3 20.0 <mesa-stable at lists.freedesktop.org>
    Tested-by: Marge Bot <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3779>
    Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3779>
    (cherry picked from commit 0660cbf4262891e6380faf0d99217b2d27873051)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=0e0abec7b64997621efec70104494d812c46b183
Author: Dylan Baker <dylan at pnwbakers.com>
Date:   Thu Feb 13 10:39:27 2020 -0800

    .pick_status.json: Update to 2a98cf3b2ecea43cea148df7f77d2abadfd1c9db



More information about the mesa-commit mailing list