Mesa (10.6): 26 new commits
Emil Velikov
evelikov at kemper.freedesktop.org
Mon Jun 15 07:59:30 UTC 2015
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=5d327b373531861f86a726db669b3d656f1b5f8d
Author: Emil Velikov <emil.l.velikov at gmail.com>
Date: Sun Jun 14 16:40:00 2015 +0100
docs: Add sha256sums for the 10.6.0 release
Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=3b9cde5c8138fb5cc45c652f2a5c15c5fa222bd7
Author: Emil Velikov <emil.l.velikov at gmail.com>
Date: Sun Jun 14 16:26:40 2015 +0100
docs: Update 10.6.0 release notes
Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=9719f26cc603f9eface0802a917f7b2167810b04
Author: Emil Velikov <emil.l.velikov at gmail.com>
Date: Sun Jun 14 16:18:43 2015 +0100
Bump version to 10.6.0(final)
Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=81027ea1e2dba8541877255e969fb6c1a8d93904
Author: Erik Faye-Lund <kusmabite at gmail.com>
Date: Wed Jun 10 23:35:04 2015 +0100
mesa: build xmlconfig to a separate static library
As we use the file from both the dri modules and loader, we end up with
multiple definition of the symbols provided in our gallium dri modules.
Additionally we compile the file twice.
Resolve both issues, effectively enabling the build on toolchains which
don't support -Wl,--allow-multiple-definition.
v2: [Emil Velikov]
- Fix the Scons/Android build.
- Resolve libgbm build issues (bring back the missing -lm)
Cc: Julien Isorce <j.isorce at samsung.com>
Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=90310
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=90905
Acked-by: Matt Turner <mattst88 at gmail.com>
Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
(cherry picked from commit 634f2002563b4fca68490c0a39518ea838f28fb1)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=a439cafc7cdc93e2f9f45ed8a809cd34a04207a9
Author: Emil Velikov <emil.l.velikov at gmail.com>
Date: Wed Apr 15 14:34:00 2015 +0100
targets/nine: link against libnir/libglsl_util
Based on commit 101142c4010(xa: support for drivers which use NIR)
Cc: "10.6" <mesa-stable at lists.freedesktop.org>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=90466
Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
(cherry picked from commit 83b5648a1e0b7c21536af18c0d29da2f2a31215e)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=1c2a462125acf644a17c07067c487dc3f07a32ca
Author: Emil Velikov <emil.l.velikov at gmail.com>
Date: Wed Apr 15 12:46:30 2015 +0100
pipe-loader: add libnir and libglsl_util to the link
Based on commit 101142c4010(xa: support for drivers which use NIR)
Cc: Rob Clark <robclark at freedesktop.org>
Cc: "10.6" <mesa-stable at lists.freedesktop.org>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=90466
Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
(cherry picked from commit ba512cc7fa5db0aeeb2fc0708920914cd3a5bf95)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=279b1d85cc93cb6eda8eff92d1f3a1ce4415884b
Author: Emil Velikov <emil.l.velikov at gmail.com>
Date: Wed Apr 15 13:40:55 2015 +0100
mesa; add a dummy _mesa_error_no_memory() symbol to libglsl_util
Rather than forcing everyone to provide their own definition of the symbol
provide a common (dummy) one.
This helps us resolve the build of the standalone pipe-drivers (amongst
others), which are missing the symbol.
Cc: Rob Clark <robclark at freedesktop.org>
Cc: "10.6" <mesa-stable at lists.freedesktop.org>
Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
(cherry picked from commit 1df5a6c71ee4a3c08b5da3f8bae24880af16b74c)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=28b3e4f9257d1deaa8b860de4ae03dd7f6456d75
Author: Emil Velikov <emil.l.velikov at gmail.com>
Date: Wed Apr 15 11:28:38 2015 +0100
freedreno: use CXX linker rather than explicit link against libstdc++
Cc: Rob Clark <robclark at freedesktop.org>
Cc: "10.6" <mesa-stable at lists.freedesktop.org>
Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
(cherry picked from commit 3f5dc9b94fc47f25821cec0a052df3d8f4cb5a1f)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=ea3d26eeb4db1a8510e5a99c702d5b9db828973d
Author: Samuel Iglesias Gonsalvez <siglesias at igalia.com>
Date: Wed May 13 12:18:31 2015 +0200
mesa/main: validate name syntax for array variables only
From ARB_program_interface_query:
"Note that if an interface enumerates a single active resource list
entry for an array variable (e.g., "a[0]"), a <name> identifying
any array element other than the first (e.g., "a[1]") is not
considered to match."
It doesn't apply to arrays of interface blocks but just to array
variables.
Signed-off-by: Samuel Iglesias Gonsalvez <siglesias at igalia.com>
Reviewed-by: Tapani Pälli <tapani.palli at intel.com>
(cherry picked from commit 4ee69a97bb0af0cc216539c48b246ea2abf8f208)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=39aa6b8bea1383c7fe1ff04a3637248a3bbe4fc5
Author: Dave Airlie <airlied at gmail.com>
Date: Wed Jun 10 13:51:59 2015 +1000
st/dri: check pscreen is valid before querying param
we don't check the validity of pscreen until dri_init_screen_helper
hit this trying to init glamor on a device with no driver (udl).
Acked-by: Michel Dänzer <michel.daenzer at amd.com>
Signed-off-by: Dave Airlie <airlied at redhat.com>
(cherry picked from commit 563706c14641fde2ab604d590b5425680354f280)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=1a47d37c994c51479d9c24a59b9d4944dd2db26c
Author: Matt Turner <mattst88 at gmail.com>
Date: Tue Jun 2 17:46:38 2015 -0700
i965: Use UW-typed immediate in multiply inst.
Some hardware reads only the low 16-bits even if the type is UD, but
other hardware like Cherryview can't handle this.
Fixes spec at arb_gpu_shader5@execution at sampler_array_indexing@fs-simple on
Cherryview.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=90830
Reviewed-by: Neil Roberts <neil at linux.intel.com>
Reviewed-by: Chris Forbes <chrisf at ijw.co.nz>
(cherry picked from commit d46d04529b9c1e55b4c3b65a7078bbbd7ab1a810)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=a2f216b329b97c5e033615e269a11228007d5e32
Author: Neil Roberts <neil at linux.intel.com>
Date: Fri May 29 13:41:48 2015 +0100
i965: Don't add base_binding_table_index if it's zero
When calculating the binding table index for non-constant sampler
array indexing it needs to add the base binding table index which is a
constant within the generated code. Often this base is zero so we can
avoid a redundant instruction in that case.
It looks like nothing in shader-db is doing non-constant sampler array
indexing so this patch doesn't make any difference but it might be
worth having anyway.
Reviewed-by: Matt Turner <mattst88 at gmail.com>
Reviewed-by: Chris Forbes <chrisf at ijw.co.nz>
Acked-by: Ben Widawsky <ben at bwidawsk.net>
(cherry picked from commit 7f62fdae1629d75dd581d1c57b28c2f099c5ef6b)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=9b8d49278183abb05861490f99a91fe6e433932b
Author: Neil Roberts <neil at linux.intel.com>
Date: Thu May 28 15:27:31 2015 +0100
i965: Don't use a temporary when generating an indirect sample
Previously when generating the send instruction for a sample
instruction with an indirect sampler it would use the destination
register as a temporary store. This breaks when used in combination
with the opt_sampler_eot optimisation because that forces the
destination to be null. This patch fixes that by avoiding the temp
register altogether.
The reason the temporary register was needed was because it was trying
to ensure the binding table index doesn't overflow a byte by and'ing
it with 0xff. The result is then or'd with samper_index<<8. This patch
instead just and's the whole thing by 0xfff. This will ensure that a
bogus sampler index won't overflow into the rest of the message
descriptor but unlike the previous code it won't ensure that the
binding table index doesn't overflow into the sampler index. It
doesn't seem like that should matter very much though because if the
shader is generating a bogus sampler index then it's going to just get
garbage out either way.
Instead of doing sampler_index<<8|(sampler_index+base_table_index) the
new code avoids one operation by doing
sampler_index*0x101+base_table_index which should be equivalent.
However if we wanted to avoid the multiply for some reason we could do
this by adding an extra or instruction still without needing the
temporary register.
This fixes a number of Piglit tests on Skylake that were using
indirect samplers such as:
spec at arb_gpu_shader5@execution at sampler_array_indexing@fs-simple
Reviewed-by: Matt Turner <mattst88 at gmail.com>
Reviewed-by: Chris Forbes <chrisf at ijw.co.nz>
Acked-by: Ben Widawsky <ben at bwidawsk.net>
Tested-by: Anuj Phogat <anuj.phogat at gmail.com>
(cherry picked from commit 6c846dc57b1d6f3e015a604dba1976f96c4be9e9)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=ef9020570448b520ecdf7d088003130e36211589
Author: Ben Widawsky <benjamin.widawsky at intel.com>
Date: Wed May 27 17:55:02 2015 -0700
i965: Disable compaction for EOT send messages
AFAICT, there is no real way to make sure a send message with EOT is properly
ignored from compact, nor can I see a way to actually encode EOT while
compacting. Before the single send optimization we'd always bail because we hit
the is_immediate && !is_compactable_immediate case. However, with single send,
is_immediate is not true, and so we end up trying to compact the un-compactible.
Without this, any compacting single send instruction will hang because the EOT
isn't there. I am not sure how I didn't hit this when I originally enabled the
optimization. I didn't check if some surrounding code changed.
I know Neil and Matt were both looking into this. I did a quick search and
didn't see any patches out there to handle this. Please ignore if this has
already been sent by someone. (Direct me to it and I will review it).
Reported-by: Neil Roberts <neil at linux.intel.com>
Reported-by: Mark Janes <mark.a.janes at intel.com>
Tested-by: Mark Janes <mark.a.janes at intel.com>
Signed-off-by: Ben Widawsky <ben at bwidawsk.net>
Reviewed-by: Matt Turner <mattst88 at gmail.com>
(cherry picked from commit b307921c3ff3b36607752f881a180272366a79cf)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=c6184b2b5c61b306e29b1a746b2b577520c9b691
Author: Alexander Monakov <amonakov at gmail.com>
Date: Tue Jun 9 20:58:22 2015 +0300
i965: do_blit_drawpixels: decode array formats
Correct a regression introduced by commit 922c0c9fd526 by converting "array
format", if received from _mesa_format_from_format_and_type, to mesa_format.
References: https://bugs.freedesktop.org/show_bug.cgi?id=90839
Signed-off-by: Alexander Monakov <amonakov at gmail.com>
Tested-by: AnAkkk <anakin.cs at gmail.com>
Reviewed-by: Jason Ekstrand <jason.ekstrand at intel.com>
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
Cc: mesa-stable at lists.freedesktop.org
(cherry picked from commit bd38f91f8d80897ca91979962d80d4bc0acef586)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=bf538839ed48b29b81e00181f7dee6859c87cdee
Author: Iago Toral Quiroga <itoral at igalia.com>
Date: Wed Jun 10 09:07:32 2015 +0200
i965: do not round line width when multisampling or antialiaing are enabled
In commit fe74fee8fa721a we rounded the line width to the nearest integer to
match the GLES3 spec requirements stated in section 13.4.2.1, but that seems
to break a dEQP test that renders wide lines in some multisampling scenarios.
Ian noted that the Open 4.4 spec has the following similar text:
"The actual width of non-antialiased lines is determined by rounding the
supplied width to the nearest integer, then clamping it to the
implementation-dependent maximum non-antialiased line width."
and suggested that when ES removed antialiased lines, they removed
"non-antialised" from that paragraph but probably should not have.
Going by that note, this patch restricts the quantization implemented in
fe74fee8fa721a only to regular aliased lines. This seems to keep the
tests fixed with that commit passing while fixing the broken test.
v2:
- Drop one of the clamps (Ken, Marius)
- Add a rule to prevent advertising line widths that when rounded go beyond
the limits allowed by the hardware (Ken)
- Update comments in the code accordingly (Ian)
- Put the code in a utility function (Ian)
Fixes:
dEQP-GLES3.functional.rasterization.fbo.rbo_multisample_max.primitives.lines_wide
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=90749
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
Cc: "10.6" <mesa-stable at lists.freedesktop.org>
(cherry picked from commit f9a18acb56c69b24c1e47cd326dc98e14fadcf94)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=7abb5e3f1307f21738f0233a27410ce49a4274e1
Author: Dave Airlie <airlied at gmail.com>
Date: Wed Jun 10 13:26:56 2015 +1000
nouveau: set imported buffers to what the kernel gives us
When we import a dma-buf fd from another driver the kernel
gives us the right info, and this trashes it.
Convert the kernel bo flags into the domain flags.
This helps getting reverse prime and glamor working.
Cc: mesa-stable at lists.freedesktop.org
Acked-by: Ben Skeggs <bskeggs at redhat.com>
Signed-off-by: Dave Airlie <airlied at redhat.com>
(cherry picked from commit c6877c9e5983287a0741b26a358b7d744aebe232)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=2353b2197c409ed028d89bfa90ba489c576f32c3
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date: Sat Jun 6 12:15:30 2015 -0700
i965/fs: Don't let the EOT send message interfere with the MRF hack
Previously, we just put the message for the EOT send as high in the file as
it would go. This is because the register pre-filling hardware will stop
all over the early registers in the file in preparation for the next thread
while you're still sending the last message. However, if something happens
to spill, then the MRF hack interferes with the EOT send message and, if
things aren't scheduled nicely, will stomp on it.
Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=90520
Reviewed-by: Neil Roberts <neil at linux.intel.com>
(cherry picked from commit 86e5afbfee5492235cab1a7be4ea49ac02be1644)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=10b7dba331dfeb9c8476e5e28709ca965a821be1
Author: Chris Wilson <chris at chris-wilson.co.uk>
Date: Fri Jun 5 14:45:18 2015 +0100
i965: Export format comparison for blitting between miptrees
Since the introduction of
commit 536003c11e4cb1172c540932ce3cce06f03bf44e
Author: Boyan Ding <boyan.j.ding at gmail.com>
Date: Wed Mar 25 19:36:54 2015 +0800
i965: Add XRGB8888 format to intel_screen_make_configs
winsys buffers no longer have an alpha channel. This causes
_mesa_format_matches_format_and_type() to reject previously working BGRA
uploads from using the BLT fast path. Instead of using the generic
routine for matching formats exactly, export the slightly more relaxed
check from intel_miptree_blit() which importantly allows the blitter
routine to apply a small number of format conversions.
References: https://bugs.freedesktop.org/show_bug.cgi?id=90839
Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
Cc: Jason Ekstrand <jason at jlekstrand.net>
Cc: Alexander Monakov <amonakov at gmail.com>
Cc: Kristian Høgsberg <krh at bitplanet.net>
Cc: Kenneth Graunke <kenneth at whitecape.org>
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
Cc: mesa-stable at lists.freedesktop.org
(cherry picked from commit 922c0c9fd526ce19b87bc74a3159dec7705c1de1)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=55104870a14d03c803b100b66cf5886478d52de1
Author: Chris Wilson <chris at chris-wilson.co.uk>
Date: Fri Jun 5 14:33:36 2015 +0100
i915: Blit RGBX<->RGBA drawpixels
The blitter already has code to accommodate filling in the alpha channel
for BGRX destination formats, so expand this to also allow filling the
alpha channgel in RGBX formats.
More importantly for the next patch is moving the test into its own
function for the purpose of exporting the check to the callers.
v2: Fix alpha expansion as spotted by Alexander with the fix suggested by
Kenneth
Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
Cc: Jason Ekstrand <jason at jlekstrand.net>
Cc: Alexander Monakov <amonakov at gmail.com>
Cc: Kristian Høgsberg <krh at bitplanet.net>
Cc: Kenneth Graunke <kenneth at whitecape.org>
Reviewed-by Kenneth Graunke <kenneth at whitecape.org>
Cc: mesa-stable at lists.freedesktop.org
(cherry picked from commit c2d0606827412b710dcaed80268fc665de8c9c5d)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=fbc04dcddb9e24dec1e878050d4161c1f4da4093
Author: Chris Wilson <chris at chris-wilson.co.uk>
Date: Fri Jun 5 13:49:08 2015 +0100
i965: Fix HW blitter pitch limits
The BLT pitch is specified in bytes for linear surfaces and in dwords
for tiled surfaces. In both cases the programmable limit is 32,767, so
adjust the check to compensate for the effect of tiling.
v2: Tweak whitespace for functions (Kenneth)
Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
Cc: Kristian Høgsberg <krh at bitplanet.net>
Cc: Kenneth Graunke <kenneth at whitecape.org>
Reviewed-by Kenneth Graunke <kenneth at whitecape.org>
Cc: mesa-stable at lists.freedesktop.org
(cherry picked from commit 8da79b8378ae87474d8c47ad955e4833edf98359)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=8631c00acbf17b078d5572b7f8f6ec4a91fac7e9
Author: Kenneth Graunke <kenneth at whitecape.org>
Date: Thu Jun 4 17:00:17 2015 -0700
prog_to_nir: Fix fragment depth writes.
In the ARB_fragment_program specification, the result.depth output
variable is treated as a vec4, where the fragment depth is stored in the
.z component, and the other three components are undefined.
This is different than GLSL, which uses a scalar value (gl_FragDepth).
To make this consistent for driver backends, this patch makes
prog_to_nir use a scalar output variable for FRAG_RESULT_DEPTH,
moving result.depth.z into the first component.
Fixes Glean's fragProg1 "Z-write test" subtest.
Cc: mesa-stable at lists.freedesktop.org
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=90000
Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
Reviewed-by: Matt Turner <mattst88 at gmail.com>
(cherry picked from commit 7b8f20ec5505a25958bcd98aabe73a7ca2b6cbba)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=add82b672d05b9a6c47b7929cdcdddf25e227b01
Author: Marek Olšák <marek.olsak at amd.com>
Date: Tue May 12 22:53:00 2015 +0200
clover: clarify and fix the EGL interop error case
Cc: 10.6 <mesa-stable at lists.freedesktop.org>
(cherry picked from commit 6acb61fc9c2c5f81569d17d90a480abc48ec6055)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=3514680b918c2605e082990a411d972b65d41602
Author: Marek Olšák <marek.olsak at amd.com>
Date: Tue May 12 00:44:20 2015 +0200
egl: fix setting context flags
Cc: 10.6 10.5 10.4 <mesa-stable at lists.freedesktop.org>
Reviewed-by: Emil Velikov <emil.l.velikov at gmail.com>
Reviewed-by: Chad Versace <chad.versace at intel.com>
(cherry picked from commit f9f894447e4e7442d5dfa489bb43f2823e2fc71d)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=4f68af254e55967d7b14f9cf8ced5f2509f0706b
Author: Roland Scheidegger <sroland at vmware.com>
Date: Thu Jun 4 14:35:59 2015 +0200
draw: (trivial) fix NULL pointer dereference
This probably got broken when the samplers were converted to be indexed
by shader type.
Seen when looking at bug 89819 though I'm not sure if that really was what
the bug was about...
Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
Reviewed-by: Brian Paul <brianp at vmware.com>
(cherry picked from commit 6e5970ffee0129fb94d8b7f0ebd4fac3992e7dce)
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=70ac6a2655334b3e4f662ca228c2f3681305e52d
Author: Kenneth Graunke <kenneth at whitecape.org>
Date: Thu Jun 4 01:24:07 2015 -0700
prog_to_nir: Make RSQ properly take the absolute value of its argument.
I just botched this when writing the original code.
From the ARB_vertex_program specification:
"The RSQ instruction approximates the reciprocal of the square root of
the absolute value of the scalar operand and replicates it to all four
components of the result vector."
Fixes a Glean vertProg1 subtest:
RSQ test 2 (reciprocal square root of negative value)
Cc: mesa-stable at lists.freedesktop.org
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=90547
Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
Reviewed-by: Francisco Jerez <currojerez at riseup.net>
(cherry picked from commit 15a12795c6f3edef0e1cbab39b6da3d5b8f64fc3)
More information about the mesa-commit
mailing list