Mesa (10.5): 23 new commits

Emil Velikov evelikov at kemper.freedesktop.org
Wed Jun 3 14:36:47 UTC 2015


URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=07aa6d78bbc9639dd7bbc86bbb375aae1103ddfd
Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Thu Mar 5 12:10:15 2015 -0500

    nv30: avoid doing extra work on clear and hitting unexpected states
    
    Clearing can happen at a time when various state objects are incoherent
    and not ready for a draw. Some of the validation functions don't handle
    this well, so only flush the framebuffer state. This has the advantage
    of also not doing extra work.
    
    This works around some crashes that can happen when clearing.
    
    Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
    Reviewed-by: Tobias Klausmann <tobias.johannes.klausmann at mni.thm.de>
    (cherry picked from commit aba3392541f38f82e3ebde251fdcca78e90adbf3)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=70816a66d5431db37cb6207efe6439b895746d16
Author: Marek Olšák <marek.olsak at amd.com>
Date:   Tue May 26 19:32:36 2015 +0200

    st/dri: fix postprocessing crash when there's no depth buffer
    
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89131
    
    Cc: 10.6 10.5 <mesa-stable at lists.freedesktop.org>
    Reviewed-by: Brian Paul <brianp at vmware.com>
    (cherry picked from commit 25e9ae2b79f32631e7255807a242e5fc4e39984c)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=f8bb4a814d9aac2825737e48fefc16785cb9060c
Author: Ben Widawsky <benjamin.widawsky at intel.com>
Date:   Wed May 20 19:20:14 2015 -0700

    i965: Emit 3DSTATE_MULTISAMPLE before WM_HZ_OP (gen8+)
    
    Starting with GEN8, there is documentation that the multisample state command
    must be emitted before the 3DSTATE_WM_HZ_OP command any time the multisample
    count changes. The 3DSTATE_WM_HZ_OP packet gets emitted as a result of a
    intel_hix_exec(), which is called upon a fast clear and/or a resolve. This can
    happen before the state atoms are checked, and so the multisample state must be
    put directly in the function.
    
    v1:
    - In v0, I was always emitting the command, but Ken came up with the condition to
    determine whether or not the sample count actually changed.
    - Ken's recommendation was to set brw->num_multisamples after emitting
    3DSTATE_MULTISAMPLE. This doesn't work. I put my best guess as to why in the XXX
    (it was causing 7 regressions on BDW).
    
    v2:
    Flag NEW_MULTISAMPLE state. As Ken found, in state upload we check for the
    multisample change to determine whether or not to emit certain packets. Since
    the hiz code doesn't actually care about the number of multisamples, set the
    flag and let the later code take care of it.
    
    Jenkins results:
    http://otc-mesa-ci.jf.intel.com/view/dev/job/bwidawsk/136/
    
    Fixes around 200 piglit tests on SKL. I'm somewhat surprised that it seems to
    have no impact on BDW as the restriction is needed there as well.
    
    Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
    Signed-off-by: Ben Widawsky <ben at bwidawsk.net>
    Reviewed-by: Neil Roberts <neil at linux.intel.com> (v0)
    Reviewed-by: Kenneth Graunke <kenneth at whitecape.org> (v2)
    (cherry picked from commit e2d84d99f5a66738e8f584bdfea66182f36fe46c)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=8c56fc92750a4a6c390354c0b1428cdc9edac5b5
Author: Emil Velikov <emil.l.velikov at gmail.com>
Date:   Wed Jun 3 12:03:35 2015 +0100

    cherry-ignore: add clover build fix not applicable for 10.5
    
    The 10.5 codebase uses a compat/wrapper implementation for the string
    class. As such it already explicitly includes the string header.
    
    Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=6338bd6f7000e9fc60161b27f4f66be857d22351
Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Mon May 25 17:46:45 2015 -0400

    nv30: falling back to draw path for edgeflag does no good
    
    The problem is that the EDGEFLAG has to be toggled at vertex submission
    time. This can be done from either the draw or the regular paths. Avoid
    falling back to draw just because there's an edgeflag.
    
    Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
    Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit 3ec18152858fd9aadb398d78d5ad2d2b938507c1)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=7087a507bfccf8e96889eb8f86a48268b7525d20
Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Sat May 23 22:11:38 2015 -0400

    nv30/draw: switch varying hookup logic to know about texcoords
    
    Commit 8acaf862dfe switched things over to use TEXCOORD instead of
    GENERIC, but did not update the nv30 swtnl draw paths. This teaches the
    draw logic about TEXCOORD.
    
    Among other things, this fixes a crash in demos/arbocclude when using
    swtnl. Curiously enough, the point-sprite piglit works without this.
    
    Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
    Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit 25be70462dbb7ee994e69ffccc3de94e4114e667)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=93650a80e9d9cee000c88a6e96be83c1e5788fc6
Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Mon May 25 21:14:13 2015 -0400

    nv30/draw: allocate vertex buffers in gart
    
    These are only used once per draw, so it makes sense to keep them in
    GART. Also take this opportunity to modernize the buffer mapping API
    usage.
    
    Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
    Reviewed-by: Ben Skeggs <bskeggs at redhat.com>
    Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit c3d36a2e1a87a4aded662db7a5d320ee7ac3a8b5)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=23c0c94ac36945c08ec21db8fa2b4e36d137320f
Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Mon May 25 21:12:46 2015 -0400

    nv30/draw: only use the DMA1 object (GART) if the bo is not in VRAM
    
    Instead of always having it in the data, let the bo placement decide it.
    This fixes glxgears with swtnl forced on.
    
    Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
    Reviewed-by: Ben Skeggs <bskeggs at redhat.com>
    Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit fdad7dfbdae07b9273fc8f57e63258dbe542c9b5)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=efaf906415f1cc9cf123cd89dd346c35d327d26b
Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Mon May 25 20:15:09 2015 -0400

    nv30/draw: fix indexed draws with swtnl path and a resource index buffer
    
    The map = assignment was missing.
    
    Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
    Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit 3600439897c79d37c3c654546867ddfa0c420743)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=adee8f1ca5ba636b6ab996b2a05b9516a74e04a5
Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Sun May 17 17:56:44 2015 -0400

    glsl: avoid leaking linked gl_shader when there's a late linker error
    
    This makes piglit mixing-clip-distance-and-clip-vertex-disallowed have 0
    definitely lost blocks with valgrind. (Same non-0 number of possibly
    lost blocks though.)
    
    Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
    Reviewed-by: Tobias Klausmann <tobias.johannes.klausmann at mni.thm.de>
    Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit 5646f0f18a620292524eebcd77353ff3d3687eb2)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=56d13627ebb07cac52401714f11a92be6b2d34e6
Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Sun May 17 17:32:24 2015 -0400

    st/mesa: don't leak glsl_to_tgsi object on link failure
    
    Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
    Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
    Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit bb973723a5e1f27817b6be2c2fa4fb3ea28e733c)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=a10c9c6725259c55281a17efc98566c659ba9146
Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Mon May 25 14:06:01 2015 -0400

    nv30/draw: draw expects constbuf size in bytes, not vec4 units
    
    This fixes glxgears with NV30_SWTNL=1 forced on. Probably fixes a bunch
    of other situations where we fall back to the swtnl path.
    
    Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
    Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit 147816375d22a653176ab28ed650fa811ceea83f)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=2d610edf5c998de8297ce1d0f3034efbbcf76ee5
Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Sun May 24 11:56:21 2015 -0400

    nv30/draw: avoid leaving stale pointers in draw state
    
    Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
    Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit 89585edf3c01c94b62d163adf0209568efa68568)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=0f7c9815a0897ce7da1b87a4695d2c9f867da527
Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Sat May 23 20:58:53 2015 -0400

    nv30: fix clip plane uploads and enable changes
    
    nv30_validate_clip depends on the rasterizer state. Also we should
    upload all the new clip planes on change since next time the plane data
    won't have changed, but the enables might.
    
    This fixes fixed-clip-enables and vs-clip-vertex-enables shader tests.
    
    Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
    Reviewed-by: Tobias Klausmann <tobias.johannes.klausmann at mni.thm.de>
    Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit 7518fc3c66e9b5703b987bccca7970a344deadfa)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=ece96b4954d4b44cea19926c46f9f91fad4ca35e
Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Sun May 24 02:23:16 2015 -0400

    nv30: avoid leaking render state and draw shaders
    
    Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
    Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit 9870ed05dd333a20662479b9b1e3a8db542924c4)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=e692b6428cdeac5a16c5a5d7c3326e7550116970
Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Sun May 24 01:31:11 2015 -0400

    nv30: don't leak fragprog consts
    
    Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
    Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit 605ce36d7f4a90c4062d6940bea82ab483bbe3b2)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=7d76e8905321f884a9bf58d818e4c6852c3c0832
Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Sat May 23 01:57:41 2015 -0400

    nv50/ir: avoid messing up arg1 of PFETCH
    
    There can be scenarios where the "indirect" arg of a PFETCH becomes
    known, and so the code will attempt to propagate it. Use this
    opportunity to just fold it into the first argument, and prevent the
    load propagation pass from touching PFETCH further.
    
    This fixes gs-input-array-vec4-index-rd.shader_test and
    vs-output-array-vec4-index-wr-before-gs.shader_test on nvc0 at least.
    
    Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
    Reviewed-by: Tobias Klausmann <tobias.johannes.klausmann at mni.thm.de>
    Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit fa7f9f123b70f313d3c073b52c9c16b4b8df28f8)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=60e81b47e539793eac5fbf8d69e7ff083b89a195
Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Sat May 23 17:35:42 2015 -0400

    nvc0: a geometry shader can have up to 1024 vertices output
    
    The 1024 is already reported everywhere, not sure where this 0x1ff came
    from.
    
    Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
    Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit 921917c8d8e707dd854e7be05fba7a3e55bc71bf)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=c04aaa5d9950ab907194388d5f78bde4c41f6638
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date:   Tue May 19 17:35:29 2015 -0700

    i965/fs: Fix implied_mrf_writes for scratch writes
    
    We build the entire message in the generator so all the MRF writes are
    implied.
    
    Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
    Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
    (cherry picked from commit 6ca67f62e885f0e42c0cef2db5c0ae837adfe646)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=a642ec055179f26e2fa663001c4ed98fd9298dfd
Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Fri May 22 19:03:58 2015 -0400

    nvc0/ir: LOAD's can't be used for shader inputs
    
    We forgot to convert to VFETCH in case of indirect access. Fix that.
    
    This avoids crashes on the new gs-input-array-vec4-index-rd and
    vs-output-array-vec4-index-wr-before-gs but they still fail.
    
    Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
    Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit 217301843aea0299ab245e260b20af7ad250e9d8)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=c13b16029d3313f0b7789d0e968b40d9d5e8f436
Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Fri May 22 19:02:41 2015 -0400

    nv50/ir: guess that the constant offset is the starting slot of array
    
    When we get something like IN[ADDR[0].x+5], we will now guess that we
    should look at IN[5] for the "base" information.
    
    Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
    Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit 0bab3962f5f313ea829c95920c02f32afb23715d)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=58b9e11084707f78c43d139a3289d39dabf15c56
Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Fri May 22 16:40:08 2015 -0400

    nvc0/ir: set ftz when sources are floats, not just destinations
    
    In the case of a compare, the destination might be a predicate, but we
    still want to flush denorms.
    
    Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
    Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
    (cherry picked from commit d1eea18a595a468dbc2267a8d14197a3b1a5a4b6)

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=d88de40d7aeb8e5ee65ff8b47a7a08b7addf7b4c
Author: Emil Velikov <emil.l.velikov at gmail.com>
Date:   Wed Jun 3 11:49:19 2015 +0100

    get-pick-list.sh: Require explicit "10.5" for nominating stable patches
    
    A nomination unadorned with a specific version is now interpreted as
    being aimed at the 10.6 branch, which was recently opened.
    
    Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>




More information about the mesa-commit mailing list