Mesa (10.5): 26 new commits
Emil Velikov
evelikov at kemper.freedesktop.org
Sat Mar 7 19:19:11 UTC 2015
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=31fcb21ef523434a254c0bbff515345c2c6d8152
Author: Matt Turner <mattst88 at gmail.com>
Date: Fri Feb 27 10:59:17 2015 -0800
i965: Avoid applying negate to wrong MAD source.
For some given GLSL IR like (+ (neg x) (* 1.2 x)), the try_emit_mad
function would see that one of the +'s sources was a negate expression
and set mul_negate = true without confirming that it was actually a
multiply.
Cc: 10.5 <mesa-stable at lists.freedesktop.org>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89315
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89095
Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
(cherry picked from commit d528907fd2950c7bb968fff66dd79863cd128890)
[Emil Velikov: drop the changes in brw_vec4_visitor.cpp]
Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
Conflicts:
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=0cd8e357e323f39603bfb178108d94f83d6a3c37
Author: Laura Ekstrand <laura at jlekstrand.net>
Date: Wed Feb 25 15:45:47 2015 -0800
main: Fix target checking for CopyTexSubImage*D.
This fixes a dEQP test failure. In the test,
glCopyTexSubImage2D was called with target = 0 and failed to throw
INVALID ENUM. This failure was caused by _mesa_get_current_tex_object(ctx,
target) being called before the target checking. To remedy this, target
checking was separated from the main error-checking function and
called prior to _mesa_get_current_tex_object.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89312
Reviewed-by: Anuj Phogat <anuj.phogat at gmail.com>
(cherry picked from commit ca65764d6042d2ea220a1e3952490f79c226f3e0)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=8b4db9c6877fab9ed37a7377fe6592ff76828555
Author: Laura Ekstrand <laura at jlekstrand.net>
Date: Wed Feb 25 10:34:03 2015 -0800
main: Fix target checking for CompressedTexSubImage*D.
This fixes a dEQP test failure. In the test,
glCompressedTexSubImage2D was called with target = 0 and failed to throw
INVALID ENUM. This failure was caused by _mesa_get_current_tex_object(ctx,
target) being called before the target checking. To remedy this, target
checking was made into its own function and called prior to
_mesa_get_current_tex_object.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89311
Reviewed-by: Anuj Phogat <anuj.phogat at gmail.com>
(cherry picked from commit 549078cb5a95e0ee381d036b8c36bc41506f21bc)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=b0400a58dbbfba8dce7427cf489361a8db2c9312
Author: Frank Henigman <fjhenigman at google.com>
Date: Thu Feb 19 15:09:20 2015 -0800
intel: fix EGLImage renderbuffer _BaseFormat
Correctly set _BaseFormat field when creating a gl_renderbuffer
with EGLImage storage.
Change-Id: I8c9f7302d18b617f54fa68304d8ffee087ed8a77
Signed-off-by: Frank Henigman <fjhenigman at google.com>
Reviewed-by: Stéphane Marchesin <marcheu at chromium.org>
Reviewed-by: Chad Versace <chad.versace at intel.com>
(cherry picked from commit e43729943e67972e547a19123fb3afca6b77202b)
Nominated-by: Chad Versace <chad.versace at intel.com>
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=ef1c87ba09d34e4985fa9cbdc4e32b2945314dc6
Author: Matt Turner <mattst88 at gmail.com>
Date: Sat Feb 28 12:12:22 2015 -0800
Revert SHA1 additions.
The shader-cache isn't finished, so the configure checks are a bit
premature and will only stand to confuse users of Mesa 10.5.0.
This is a squash of the follow four reverts:
Revert "Rename sha1.c and sha1.h to mesa-sha1.c and mesa-sha1.h"
Revert "configure: Add machinery for --enable-shader-cache (and --disable-shader-cache)"
Revert "sha1: Fix gcry_md_hd_t typo."
Revert "mesa: Add mesa SHA-1 functions"
Reviewed-by: Carl Worth <cworth at cworth.org>
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=a71223eb594cee382cde633b318fc43a6e85c1dd
Author: Andrey Sudnik <andrey.sudnik at intel.com>
Date: Thu Mar 5 11:16:49 2015 -0800
i965/vec4: Don't lose the saturate modifier in copy propagation.
Cc: 10.4, 10.5 <mesa-stable at lists.freedesktop.org>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89224
Reviewed-by: Matt Turner <mattst88 at gmail.com>
Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
(cherry picked from commit 0dfec59a2785cf7a87ee5128889ecebe810b611b)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=47a3ae1f20a5e936c2f4bed864dd2d9427ed651d
Author: Kenneth Graunke <kenneth at whitecape.org>
Date: Wed Mar 4 18:14:31 2015 -0800
i965: Split Gen4-5 BlitFramebuffer code; prefer BLT over Meta.
A while back I switched intel_blit_framebuffer to prefer Meta over the
BLT. This meant that Gen8 platforms would start using the 3D engine
for blits, just like we do on Gen6-7.5.
However, I hadn't considered Gen4-5 when making that change. The BLT
engine appears to be substantially faster on 965GM than using Meta to
drive the 3D engine. This isn't too surprising: original Gen4 doesn't
support tile offsets (that came on G45), and the level/layer fields
don't work for cubemap rendering, so for inconvenient miplevel
alignments, we end up blitting or copying data to/from temporaries
in order to render to it. We may as well just use the blitter.
I chose to use the BLT on Gen4-5 because they use the same ring for
both 3D and BLT; Gen6+ splits it out.
Fixes regressions on 965GM due to botched tile offset code (we should
fix those properly as well, but they're longstanding bugs - for now,
put things back to the status quo).
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89430
Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
Reviewed-by: Topi Pohjolainen <topi.pohjolainen at intel.com>
Reviewed-by: Jordan Justen <jordan.l.justen at intel.com>
Cc: "10.5" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit aa0705c06c03d2b882ac7b185ed123bc8a10d716)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=dbf974636af53d15cd9b0924262a1f29d21dcfb1
Author: Matt Turner <mattst88 at gmail.com>
Date: Wed Mar 4 15:21:53 2015 -0800
i965: Tell intel_get_memcpy() which direction the memcpy() is going.
The SSSE3 swizzling code was written for fast uploads to the GPU and
assumed the destination was always 16-byte aligned. When we began using
this code for fast downloads as well we didn't do anything to account
for the fact that the destination pointer given by glReadPixels() or
glGetTexImage() is not guaranteed to be suitably aligned.
With SSSE3 enabled (at compile-time), some applications would crash when
an SSE aligned-store instruction tried to store to an unaligned
destination (or an assertion that the destination is aligned would
trigger).
To remedy this, tell intel_get_memcpy() whether we're uploading or
downloading so that it can select whether to assume the destination or
source is aligned, respectively.
Cc: 10.5 <mesa-stable at lists.freedesktop.org>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89416
Tested-by: Uriy Zhuravlev <stalkerg at gmail.com>
Reviewed-by: Jason Ekstrand <jason.ekstrand at intel.com>
(cherry picked from commit 2e4c95dfe2cb205c327ceaa12b44a9273bdb20dc)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=35909f0e12fcb726e5f25fff7ff9a11293e054fd
Author: Emil Velikov <emil.l.velikov at gmail.com>
Date: Sat Mar 7 17:52:02 2015 +0000
mapi: fix commit 90411b56f6bc817e229d8801ac0adad6d4e3fb7a
Handle static glapi as well.
Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=b83333ac5b9eeae24739d47902e3fd75e9cfeaeb
Author: Emil Velikov <emil.l.velikov at gmail.com>
Date: Sat Mar 7 17:50:29 2015 +0000
cherry-ignore: ignore a few more commits picked without -x
Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=30079d6846124e9580335393f75464f8b65149fe
Author: Emil Velikov <emil.l.velikov at gmail.com>
Date: Sat Feb 28 17:20:01 2015 +0000
egl/main: no longer export internal function
With the split of the gallium egl module we had previously it required
access to some of the internal functions. As the only build (automake)
that did this no longer builds it we can now appropriately hide those
functions.
Cc: 10.5 <mesa-stable at lists.freedesktop.org>
Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
Reviewed-by: Matt Turner <mattst88 at gmail.com>
Reviewed-by: Brian Paul <brianp at vmware.com>
(cherry picked from commit dd438ae34bdbaa6651cdd226d5fec15a892923bf)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=e5eab59b576fd7c1c2bcb78dcca899331fb3f5e9
Author: Matt Turner <mattst88 at gmail.com>
Date: Fri Feb 27 10:22:21 2015 -0800
i965/fs: Don't propagate cmod to inst with different type.
Cc: 10.5 <mesa-stable at lists.freedesktop.org>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89317
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
(cherry picked from commit 1e128e9b69c6336762a2b6ee5d356c763b9ae3b0)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=57f54b067b5c6d4fbf3ceb2bc1138aed8c913c8a
Author: Matt Turner <mattst88 at gmail.com>
Date: Tue Mar 3 16:09:58 2015 -0800
r300g: Check return value of snprintf().
Would have at least prevented the crash the previous patch fixed.
Cc: 10.4, 10.5 <mesa-stable at lists.freedesktop.org>
Bugzilla: https://bugs.gentoo.org/show_bug.cgi?id=540970
Reviewed-by: Tom Stellard <thomas.stellard at amd.com>
(cherry picked from commit ade0b580e75bdea227eec5345f6681b678d0811b)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=e0670715bc511bb814ab42b10d9608a93d6ba439
Author: Matt Turner <mattst88 at gmail.com>
Date: Tue Mar 3 16:02:40 2015 -0800
r300g: Use PATH_MAX instead of limiting ourselves to 100 chars.
When built with Gentoo's package manager, the Mesa source directory
exists seven directories deep. The path to the .test file is too long
and is silently truncated, leading to a crash. Just use PATH_MAX.
Cc: 10.4, 10.5 <mesa-stable at lists.freedesktop.org>
Bugzilla: https://bugs.gentoo.org/show_bug.cgi?id=540970
Reviewed-by: Tom Stellard <thomas.stellard at amd.com>
(cherry picked from commit f5e2aa1324dd6a9666bb21834097d2fbc3cb99b6)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=8357abf4c3a7ea7f2c4e2dcc4381333a979a0758
Author: Daniel Stone <daniels at collabora.com>
Date: Mon Mar 2 13:52:59 2015 +0000
egl: Take alpha bits into account when selecting GBM formats
This fixes piglit when using PIGLIT_PLATFORM=gbm
Tom Stellard:
- Fix ARGB2101010 format
Cc: "10.4 10.5" <mesa-stable at lists.freedesktop.org>
Reviewed-by: Alex Deucher <alexander.deucher at amd.com>
Reviewed-by: Chad Versace <chad.versace at intel.com>
(cherry picked from commit 65c8965d033cf9ade5e6f3c88bda6d247d46af9d)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=b0f60c891aed6340a4c4e4d1dbf594b97c404d4f
Author: Marc-Andre Lureau <marcandre.lureau at gmail.com>
Date: Fri Feb 27 19:40:19 2015 +0100
gallium/auxiliary/indices: fix start param
Since commit 28f3f8d, indices generator take a start parameter. However, some
index values have been left to start at 0.
This fixes the glean/fbo test with the virgl driver, and copytexsubimage
with freedreno.
Reviewed-by: Ilia Mirkin <imirkin at alum.mit.edu>
Cc: "10.4 10.5" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit 073a5d2e84ac9d95f0d037aeb04889822e76aa4e)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=73efd2891f9e044c89e9d24ac4b93931c02ea2fc
Author: Matt Turner <mattst88 at gmail.com>
Date: Tue Jan 13 15:35:57 2015 -0800
i965/fs: Don't use backend_visitor::instructions after creating the CFG.
This is a fix for a regression introduced in commit a9f8296d ("i965/fs:
Preserve the CFG in a few more places.").
The errata this code works around is described in a comment before the function:
"[DevBW, DevCL] Errata: A destination register from a send can not be
used as a destination register until after it has been sourced by an
instruction with a different destination register.
The framebuffer write's sources must be in message registers, which SEND
instructions cannot have as a destination. There's no way for this
errata to affect anything at the end of the program. Just remove the
code.
Cc: 10.4, 10.5 <mesa-stable at lists.freedesktop.org>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=84613
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
(cherry picked from commit e214000f258ae564e64d839cccee9418526f226b)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=73f8e342a7d33e296eb7a92b3dffae55a66f8f9f
Author: Jason Ekstrand <jason at jlekstrand.net>
Date: Fri Feb 27 18:53:11 2015 -0500
main/base_tex_format: Properly handle STENCIL_INDEX1/4/16
This takes "fbo-stencil blit GL_STENCIL_INDEX1/4/16" from crash to pass on
BDW.
Cc: 10.5 <mesa-stable at lists.freedesktop.org>
Reviewed-by: Matt Turner <mattst88 at gmail.com>
(cherry picked from commit c4925d7f3b66d63fbdd7b7607cd809db1e58bee9)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=53d66c0c7c39fbe2a587a849e0714d6f9a01f476
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date: Fri Feb 27 12:29:03 2015 -0800
meta/TexSubImage: Stash everything other than PIXEL_TRANSFER/store in meta_begin
Previously, there were bugs where if the app set a scissor it could affect
the area of the texture that was downloaded. There was also potential that
the framebuffer SRGB state could affect downloads. This ensures that those
will get saved/restored and can't affect the texture download.
Cc: 10.5 <mesa-stable at lists.freedesktop.org>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89292
Reviewed-by: Neil Roberts <neil at linux.intel.com>
(cherry picked from commit b1ab02d9c0cc11ba8ef4efaba9452d644b6a0811)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=560fb4ee450659ed603338301f8d0af827d56f16
Author: Matt Turner <mattst88 at gmail.com>
Date: Sat Feb 28 13:36:21 2015 -0800
i965: Consider scratch writes to have side effects.
We could do better by tracking scratch reads and writes.
Cc: 10.5 <mesa-stable at lists.freedesktop.org>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=88793
Reviewed-by: Jason Ekstrand <jason.ekstrand at intel.com>
(cherry picked from commit da20bf068ef0f816968d9bc4dfea81facf0fd680)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=446aa309e18e0c5a6335d7dd9688a4e521d29043
Author: Matt Turner <mattst88 at gmail.com>
Date: Sat Feb 28 11:14:02 2015 -0800
mesa: Correct backwards NULL check.
Cc: "10.4, 10.5" <mesa-stable at lists.freedesktop.org>
Reviewed-by: Emil Velikov <emil.l.velikov at gmail.com>
Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
(cherry picked from commit 491d42135ad0e5670756216154f2ba9fc79d4ba7)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=4c45e239385473da58d79100b0af25bd0b9edf9c
Author: Matt Turner <mattst88 at gmail.com>
Date: Sat Feb 28 11:08:17 2015 -0800
mesa: Free memory allocated for luminance in readpixels.
Cc: "10.4, 10.5" <mesa-stable at lists.freedesktop.org>
Reviewed-by: Brian Paul <brianp at vmware.com>
Reviewed-by: Iago Toral Quiroga <itoral at igalia.com>
(cherry picked from commit 87109acbed9c9b52f33d58ca06d9048d0ac7a215)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=9267820be63fedade5ab4087c00e2239c0129cbe
Author: Matt Turner <mattst88 at gmail.com>
Date: Sat Feb 28 11:00:51 2015 -0800
mesa: Indent break statements and add a missing one.
Always indenting break statements makes spotting missing ones easier.
Cc: "10.4, 10.5" <mesa-stable at lists.freedesktop.org>
Reviewed-by: Brian Paul <brianp at vmware.com>
Reviewed-by: Iago Toral Quiroga <itoral at igalia.com>
(cherry picked from commit 2b2fa1865248c6e3b7baec81c4f92774759b201f)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=b730f6e18eb290962474aade3bfb2eea3105063b
Author: Chris Forbes <chrisf at ijw.co.nz>
Date: Sat Feb 28 19:57:20 2015 +1300
i965/gs: Check newly-generated GS-out VUE map against correct stage
Previously, we compared our new GS-out VUE map to the existing *VS*-out
VUE map, which is bogus.
This would mostly manifest as redundant dirty flagging where the GS is
in use but the VS and GS output layouts differ; but there is a scary
case where we would fail to flag a GS-out layout change if it happened
to match the VS-out layout.
Signed-off-by: Chris Forbes <chrisf at ijw.co.nz>
Cc: "10.5, 10.4" <mesa-stable at lists.freedesktop.org>
Reviewed-by: Matt Turner <mattst88 at gmail.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=88885
(cherry picked from commit b51ff50a767cc78d678ed3d2c25995f5c4194fea)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=f5fbed4831767b086a904281cdb7fe9fb9e9ae62
Author: Matt Turner <mattst88 at gmail.com>
Date: Thu Feb 26 22:49:47 2015 -0800
i965/vec4: Fix implementation of i2b.
I broke this in commit 2881b123d. I must have misread i2b as b2i.
Cc: 10.5 <mesa-stable at lists.freedesktop.org>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=88246
Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
(cherry picked from commit 43ef2657a08f850c5756f28520f2cbe506807f24)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=f7cc5fb2cbe42924655f8d41f9464e943a94b9ac
Author: Ian Romanick <ian.d.romanick at intel.com>
Date: Fri Feb 27 14:17:50 2015 -0800
i965/fs/nir: Use emit_math for nir_op_fpow
It appears that all the other instructions that need it already use it.
This one just got missed.
Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
Reviewed-by: Jason Ekstrand <jason.ekstrand at intel.com>
Cc: "10.5" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit b8a1637119249c1d5e76c27d0053360bbb7f4e77)
More information about the mesa-commit
mailing list