Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/drivers/dri/i965/intel_extensions.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
b/src/mesa/drivers/dri/i965/intel_extensions.c
index c28c171..3088a1a
Looks like everything prior to this patch has landed;
Ken's two patches for the printf-like debug plumbing, and the
remaining patches from this series are:
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Sun, Apr 19, 2015 at 9:02 AM, Jason Ekstrand ja...@jlekstrand.net wrote:
On Sat, Apr 18
/* Only ARB_texture_gather but not GLSL 4.0 or ARB_gpu_shader5.
* used for relaxation of const offset requirements.
*/
static bool
-texture_gather_only(const _mesa_glsl_parse_state *state)
+texture_gather_only_or_es31(const _mesa_glsl_parse_state *state)
{
return
For the series:
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Fri, May 1, 2015 at 12:36 AM, Marek Olšák mar...@gmail.com wrote:
From: Marek Olšák marek.ol...@amd.com
---
tests/shaders/shader_runner.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/tests/shaders
Nevermind, pre-coffee. On re-reading the GLSL ES 3.1 spec, the offset
is required to be constant wherever texture offsets are used.
On Fri, May 1, 2015 at 10:03 AM, Ilia Mirkin imir...@alum.mit.edu wrote:
On Thu, Apr 30, 2015 at 5:56 PM, Chris Forbes chr...@ijw.co.nz wrote:
/* Only
It might be better to just prefetch no samplers in this case? -- a
shader that has this many active samplers probably doesn't actually
use them all in a single invocation.
On Thu, Apr 30, 2015 at 5:23 AM, Kenneth Graunke kenn...@whitecape.org wrote:
On Wednesday, April 29, 2015 07:47:26 PM
Have an:
Acked-by: Chris Forbes chr...@ijw.co.nz
On Fri, Apr 24, 2015 at 3:41 AM, Marius Predut marius.pre...@intel.com wrote:
On SNB and IVB hw, for 1 pixel line thickness or less,
the general anti-aliasing algorithm give up - garbage line is generated.
Setting a Line Width of 0.0 specifies
On the this is silly, I should really fix it list forever...
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Fri, Apr 24, 2015 at 6:02 PM, Kenneth Graunke kenn...@whitecape.org wrote:
Previously, sampler messages were decoded as
sampler (1, 0, 2, 2) mlen 6 rlen 8 { align1 1H };
I
Looks sensible to me.
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Sun, Apr 5, 2015 at 5:55 PM, Dave Airlie airl...@gmail.com wrote:
This is based on ARB_stencil_texturing draw.c except
it only tests stencil texturing.
It's also forward ported to core profile.
v2: fix test comment (Chris
For the series:
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Sat, Apr 4, 2015 at 11:46 PM, Kenneth Graunke kenn...@whitecape.org wrote:
Jordan added this in commit 741782b5948bb3d01d699f062a37513c2e73b076 for
Gen7 platforms. Embarassingly, this was missed for well over a year.
Fixes
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Wed, Apr 1, 2015 at 1:57 PM, Anuj Phogat anuj.pho...@gmail.com wrote:
Signed-off-by: Anuj Phogat anuj.pho...@gmail.com
---
tests/spec/arb_direct_state_access/getcompressedtextureimage.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
I'd adjust the write to surf[0] to use surf_type too.
Other than that, this patch is:
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Tue, Mar 31, 2015 at 10:04 AM, Anuj Phogat anuj.pho...@gmail.com wrote:
Makes no functional changes in the code.
Signed-off-by: Anuj Phogat anuj.pho
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Mon, Mar 30, 2015 at 12:01 AM, Timothy Arceri t_arc...@yahoo.com.au wrote:
Fix array sizes so that the arrays are not larger than
the minimum implementation requirement of GL_MAX_VERTEX_ATTRIBS.
Also assign something to the output so that its
For the series:
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Sat, Mar 28, 2015 at 5:22 PM, Matt Turner matts...@gmail.com wrote:
The typical case of mat4*mat4*vec4 is 80 scalar multiplications, but
mat4*(mat4*vec4) is only 32.
On HSW (with vec4 vertex shaders):
instructions in affected
Looks good to me. I should have considered this cross-stage case when
I fixed the first part of this bug...
Do you have a piglit test which hits this?
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Mon, Mar 23, 2015 at 8:12 PM, Tapani Pälli tapani.pa...@intel.com wrote:
Patch changes lowering
-LIBDRM_INTEL_REQUIRED=2.4.60
+LIBDRM_INTEL_REQUIRED=2.4.59
Hang on, what's this hunk doing here?
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Looks like the right thing.
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Mon, Mar 23, 2015 at 8:50 PM, Tapani Pälli tapani.pa...@intel.com wrote:
Signed-off-by: Tapani Pälli tapani.pa...@intel.com
---
.../linker/intrastage-const-arrays.shader_test | 22
++
1 file
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Sun, Mar 22, 2015 at 8:22 PM, Vinson Lee v...@freedesktop.org wrote:
Fix stray semicolon defect reported by Coverity.
Signed-off-by: Vinson Lee v...@freedesktop.org
---
tests/util/piglit_ktx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Sun, Mar 22, 2015 at 8:29 PM, Vinson Lee v...@freedesktop.org wrote:
Fix out-of-bounds read defect reported by Coverity.
Signed-off-by: Vinson Lee v...@freedesktop.org
---
.../geometry/geometry-end-primitive-optional-with-points-out.c
jordan.l.jus...@intel.com
Reviewed-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/drivers/dri/i965/brw_defines.h| 5 +
src/mesa/drivers/dri/i965/brw_fs.h | 3 +++
src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 11 +++
src/mesa/drivers/dri/i965
I think that having both the existing `struct brw_vs_compile` and a
function with the same name is going to cause confusion. (same with
the other non-fs stages)
On Sat, Mar 21, 2015 at 2:04 PM, Ian Romanick i...@freedesktop.org wrote:
On 03/20/2015 06:02 PM, Ian Romanick wrote:
On 03/20/2015
LGTM.
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Sat, Mar 7, 2015 at 1:15 PM, Anuj Phogat anuj.pho...@gmail.com wrote:
in different fragment shaders. This also applies to a case when gl_FragCoord
is redeclared with no layout qualifiers in one fragment shader and not
declared but used
With the fix Jason mentioned:
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Wed, Mar 18, 2015 at 10:19 AM, Matt Turner matts...@gmail.com wrote:
On Tue, Mar 17, 2015 at 2:15 PM, Jason Ekstrand ja...@jlekstrand.net wrote:
On Tue, Mar 17, 2015 at 2:09 PM, Matt Turner matts...@gmail.com wrote
For the series:
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Wed, Mar 18, 2015 at 10:51 AM, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
These patterns
gl_FrontFacing ? -1.0 : 1.0
gl_FrontFacing ? 1.0 : -1.0
gl_FrontFacing ? 0.0
Module: Mesa
Branch: master
Commit: 7c3095d6b71c410fd625ead797c78a0f5376904d
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=7c3095d6b71c410fd625ead797c78a0f5376904d
Author: Chris Forbes chr...@ijw.co.nz
Date: Sat Mar 14 07:10:10 2015 +1300
i965/disasm: Mark format() as being printf
Module: Mesa
Branch: master
Commit: 21ff9bfe1cea8c0a51e9f607cc580df62baa3445
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=21ff9bfe1cea8c0a51e9f607cc580df62baa3445
Author: Chris Forbes chr...@ijw.co.nz
Date: Sat Mar 14 07:10:11 2015 +1300
i965/disasm: Fix format strings
Most
Most of the brw_inst_* api returns 64bit values. This fixes disassembly
of sampler messages, etc.
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/drivers/dri/i965/brw_disasm.c | 48 +-
1 file changed, 24 insertions(+), 24 deletions(-)
diff --git a/src
This allows us to get warnings from GCC when we mess up the format
strings.
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/drivers/dri/i965/brw_disasm.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_disasm.c
b/src/mesa/drivers/dri/i965
Module: Mesa
Branch: master
Commit: f68a973dfb8926ac872b0b0e3b4b5c2163389d06
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=f68a973dfb8926ac872b0b0e3b4b5c2163389d06
Author: Chris Forbes chr...@ijw.co.nz
Date: Mon Dec 8 20:37:00 2014 +1300
i965/gen4-5: Cope with immutable-format
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Wed, Mar 4, 2015 at 2:25 PM, Jordan Justen jordan.l.jus...@intel.com wrote:
Same idea as this patch, only for gen6_gs_visitor:
commit 49a938a265f5959c9b558995cc658f80acb6eb18
Author: Jordan Justen jordan.l.jus...@intel.com
Date: Fri Feb 20 12
Thanks Matt -- yes, for 10.5 as well.
I'll also note that this fixes:
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=5
On Sat, Feb 28, 2015 at 8:48 PM, Matt Turner matts...@gmail.com wrote:
On Fri, Feb 27, 2015 at 11:03 PM, Chris Forbes chr...@ijw.co.nz wrote:
Previously, we
Module: Mesa
Branch: master
Commit: b51ff50a767cc78d678ed3d2c25995f5c4194fea
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=b51ff50a767cc78d678ed3d2c25995f5c4194fea
Author: Chris Forbes chr...@ijw.co.nz
Date: Sat Feb 28 19:57:20 2015 +1300
i965/gs: Check newly-generated GS-out VUE
if it happened
to match the VS-out layout.
Signed-off-by: Chris Forbes chr...@ijw.co.nz
Cc: 10.4 mesa-sta...@lists.freedesktop.org
---
src/mesa/drivers/dri/i965/brw_gs.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_gs.c
b/src/mesa/drivers/dri/i965
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Fri, Feb 27, 2015 at 5:52 PM, Jason Ekstrand ja...@jlekstrand.net wrote:
I'll admit that I don't know that portion of the code all that well, but it
looks sensible to me.
Reviewed-by: Jason Ekstrand jason.ekstr...@intel.com
On Thu, Feb 26, 2015
Looks reasonable, if it's going to be a while before an i965 backend is ready..
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Fri, Feb 20, 2015 at 12:38 PM, Dave Airlie airl...@gmail.com wrote:
This just fills in some blanks to avoid warnings in the i965 driver.
Signed-off-by: Dave Airlie
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Sat, Feb 7, 2015 at 1:32 AM, Kenneth Graunke kenn...@whitecape.org wrote:
The hardware's integer luminance formats are completely unusable;
currently we fall back to RGBA. This means we need to override
the texture swizzle to obtain the XXX1
with
GL_FRAMEBUFFER_INCOMPLETE_LAYER_TARGETS, even if no layered attachments
were present.
Reviewed-by: Chris Forbes chr...@ijw.co.nz
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89026
---
src/mesa/main/fbobject.c |1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/main/fbobject.c b/src/mesa
Renderbuffers are never layered. Previously we'd just leave whatever
value was present from the previous attachment.
Signed-off-by: Chris Forbes chr...@ijw.co.nz
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89026
---
src/mesa/main/fbobject.c | 1 +
1 file changed, 1 insertion(+)
diff
Apologies -- I'd not noticed that there was an equivalent patch
already attached to the bug report.
In that case, James Legg's patch is:
Reviewed-by: Chris Forbes chr...@ijw.co.nz
(and I'll push it shortly)
On Sun, Feb 8, 2015 at 1:47 PM, Chris Forbes chr...@ijw.co.nz wrote:
Renderbuffers
Looks good to me.
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Sat, Feb 7, 2015 at 1:32 AM, Kenneth Graunke kenn...@whitecape.org wrote:
+82 Piglits - 100% of border color tests now pass on Haswell.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
Cc: Chris Forbes chr...@ijw.co.nz
Cc
I think 'component_size' would be better over the various dmul, size_mul, etc.
On Fri, Feb 6, 2015 at 5:56 PM, Ilia Mirkin imir...@alum.mit.edu wrote:
From: Dave Airlie airl...@redhat.com
This adds support for the new uniform interfaces
from ARB_gpu_shader_fp64.
v2:
support
Matt, the default if not specified is zero.
2.5 ulps agrees with the spec, so:
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Wed, Feb 4, 2015 at 10:58 AM, Matt Turner matts...@gmail.com wrote:
On Mon, Feb 2, 2015 at 6:04 AM, Ilia Mirkin imir...@alum.mit.edu wrote:
This makes the relevant
of swizzles as static const
This saves about 0.5k in the text section for a gallium driver
on amd64.
Reviewed-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/state_tracker
dlist.c seems to be missing from this patch.
On Mon, Feb 2, 2015 at 4:12 AM, Ilia Mirkin imir...@alum.mit.edu wrote:
Signed-off-by: Ilia Mirkin imir...@alum.mit.edu
---
As requested, added a case for positive clamp as well as negative.
Also dropped version requirement to GL 2.1 and added a
You don't need the VAO, this is 2.1.
With that and Glenn's concern addressed,
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Mon, Feb 2, 2015 at 11:19 AM, Ilia Mirkin imir...@alum.mit.edu wrote:
Signed-off-by: Ilia Mirkin imir...@alum.mit.edu
---
v2 - v3:
*actually* include dlist.c
Glenn,
Fixed in the version of this patch which landed (d36fa601913 on master).
- Chris
On Mon, Feb 2, 2015 at 2:08 PM, Glenn Kennard glenn.kenn...@gmail.com wrote:
On Wed, 31 Dec 2014 21:43:51 +0100, Micah Fedke
micah.fe...@collabora.co.uk wrote:
---
src/glsl/glcpp/glcpp-parse.y| 3
With the new commit message,
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Sat, Jan 31, 2015 at 2:08 PM, Laura Ekstrand la...@jlekstrand.net wrote:
It's updated to:
GL: Update glext.h to Khronos Revision 29537.
Khronos Revision 29537 fixes ARB_direct_state_access function prototypes
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Tue, Jan 27, 2015 at 5:24 AM, Brian Paul bri...@vmware.com wrote:
AFAICT, the GL and GLSL specs don't say anything about the significance
of the order in which shaders are attached to a program object. For
example, if there's two vertex shader
, the calculation is such that we can
just do it in place and we don't need the scratch space at all.
Cc: Chris Forbes chr...@ijw.co.nz
---
src/mesa/drivers/dri/i965/brw_eu.h | 3 +--
src/mesa/drivers/dri/i965/brw_eu_emit.c | 5 ++---
src/mesa/drivers/dri/i965/brw_fs_generator.cpp
Ugh, I'd thinko'd this on first reading and assumed that the
adjustment was header.3 += temp -- but it's not; it's header.3 = r0.3
+ temp so this is fine.
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Fri, Jan 23, 2015 at 10:50 AM, Jason Ekstrand ja...@jlekstrand.net wrote:
Prior to this commit
Ian,
That's what the -2 variation added by that commit does.
- Chris
On Wed, Jan 21, 2015 at 7:29 AM, Ian Romanick i...@freedesktop.org wrote:
On 01/19/2015 10:55 PM, Tapani Pälli wrote:
Fixes currently failing Piglit case
interface-blocks-name-reused-globally.vert
Signed-off-by: Tapani
infrastructure
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
Reviewed-by: Chris Forbes chr...@ijw.co.nz
---
src/glsl/glcpp/glcpp-parse.y|3 +++
src/glsl/glsl_parser_extras.cpp |1 +
src/glsl/glsl_parser_extras.h |2 ++
src/mapi/glapi/gen/gl_API.xml |6 +-
src/mesa
Using the GLSL spec's definition (or something with equivalent
precision) is also required for ARB_shader_precision, which we'll have
really soon, so +1 from me too.
On Fri, Jan 16, 2015 at 8:01 PM, Iago Toral ito...@igalia.com wrote:
Thanks Roland and everyone else for the input. Based on the
The vec4 side is similarly tangled, except there are 3 constructors
like this spread across the src_reg and dst_reg classes.
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Sat, Jan 17, 2015 at 11:19 AM, Kenneth Graunke kenn...@whitecape.org wrote:
I dislike how fs_reg has a constructor
Looks OK to me.
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Thu, Jan 15, 2015 at 5:50 PM, Jason Ekstrand ja...@jlekstrand.net wrote:
v2: Add a comment as to why we are using variables instead of lowering to
an index like the rest of the input intrinsics.
Cc: Chris Forbes chr
Oops.
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Wed, Dec 24, 2014 at 7:12 PM, Timothy Arceri t_arc...@yahoo.com.au wrote:
From: Timothy Arceri t.arc...@bom.gov.au
dont assign values to shader inputs
---
I didn't realise my mistake as Mesa currently allows
inputs to change. I'm
()
+{
+ float f = textureProjGradOffset(s[i], coord, coord.xy, coord.zw,
+ ivec2(43, 7));
The offset components should really be restricted to -8/+7.
+ gl_FragColor = vec4(0, f, 0, 1);
+}
--
2.2.0
Otherwise,
Reviewed-by: Chris Forbes chr...@ijw.co.nz
Oops... sorry about that.
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Fri, Jan 9, 2015 at 11:53 AM, Vinson Lee v...@freedesktop.org wrote:
Fix dead assignment bugs reported by Clang Static Analyzer.
Signed-off-by: Vinson Lee v...@freedesktop.org
---
tests/spec/arb_shader_atomic_counters
Subtle..
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Jan 8, 2015 1:07 AM, Neil Roberts n...@linux.intel.com wrote:
Adds a test which tries setting a uint attrib value to 0x7f817f81.
If this value were interpreted as a float then it would look like a
signalling NaN. If this is passed through
Worth pointing out that when tessellation gets ported to NIR, output
reads are a real thing, and can't be lowered away like we do for other
stages (they allow cross-invocation communication, so have to actually
read back from the URB on i965)
On Wed, Jan 7, 2015 at 10:43 AM, Jason Ekstrand
Looks sensible to me.
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Tue, Dec 16, 2014 at 7:13 PM, Jason Ekstrand ja...@jlekstrand.net wrote:
---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 24 ++--
1 file changed, 22 insertions(+), 2 deletions(-)
diff --git a/src/mesa
Looks reasonable to me.
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Tue, Dec 16, 2014 at 7:13 PM, Jason Ekstrand ja...@jlekstrand.net wrote:
---
src/glsl/nir/nir_lower_samplers.cpp | 149
++--
1 file changed, 74 insertions(+), 75 deletions(-)
diff --git
path. Because from the result, both of them work
well.
Thanks,
Shuo
-Original Message-
From: Chris Forbes [mailto:chr...@ijw.co.nz]
Sent: Thursday, January 1, 2015 2:53 AM
To: Wang, Shuo
Cc: piglit@lists.freedesktop.org; Romanick, Ian D
Subject: Re: [Piglit] [PATCH] Add a new test
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Jan 3, 2015 8:53 AM, Brian Paul bri...@vmware.com wrote:
Currently, this test causes the GLSL compiler to abort in ir_validate.cpp
with the message:
ir_dereference_variable @ 0x182ee80 specifies undeclared variable
`unused_array' @ 0x182ed80
These three all pass on i965 with master, so I guess you have some
breakage at the gallium layer or below.
All three are:
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Fri, Jan 2, 2015 at 3:17 AM, Marek Olšák mar...@gmail.com wrote:
From: Marek Olšák marek.ol...@amd.com
---
tests/all.py
Both of these are also
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Jan 1, 2015 7:48 PM, Kenneth Graunke kenn...@whitecape.org wrote:
The logic in brw_blorp_surface_info::set uses brw_format_for_mesa_format
for source surfaces, and brw-render_target_format[] for destination
surfaces. We
I think it would be worth mentioning that you're using GL_TRIANGLE_FAN
specifically to hit the SW primitive restart path.
On Thu, Jan 1, 2015 at 3:24 AM, Wang Shuo shuo.w...@intel.com wrote:
This case is added for *Bug 87611 - Exercise gl_VertexID corner cases in i965
driver*, and used to
Seems reasonable to me.
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Fri, Dec 19, 2014 at 1:45 AM, Kenneth Graunke kenn...@whitecape.org wrote:
This is a partial revert of c89306983c07e5a88c0d636267e5ccf263cb4213.
It split the {start,base}_vertex_location handling into several steps:
1. Set
Nice! Both are
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Jan 1, 2015 5:04 PM, Kenneth Graunke kenn...@whitecape.org wrote:
Gen7.5+ platforms that support the Shader Channel Select feature leave
key-tex.swizzles[i] as SWIZZLE_NOOP except when GL_DEPTH_TEXTURE_MODE
is GL_ALPHA (which
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Mon, Dec 29, 2014 at 7:07 PM, Tapani Pälli tapani.pa...@intel.com wrote:
Currently compute shaders that fail to compile trigger assert
during piglit_compile_shader_text.
Signed-off-by: Tapani Pälli tapani.pa...@intel.com
---
tests/util/piglit
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Mon, Dec 22, 2014 at 11:36 PM, Timothy Arceri t_arc...@yahoo.com.au wrote:
---
...ge-unsized-interface-array-mismatch.shader_test | 55 +
...e-unsized-interface-array-mismatch2.shader_test | 56
OK, cool.
This patch is
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Mon, Dec 22, 2014 at 11:44 PM, Timothy Arceri t_arc...@yahoo.com.au wrote:
- Original Message -
From: Chris Forbes chr...@ijw.co.nz
Does this work correctly for arrays of interface blocks too?
Hi Chris
Are there some performance numbers to go with this?
On Tue, Dec 23, 2014 at 12:08 PM, Neil Roberts n...@linux.intel.com wrote:
Here are some patches to make the i965 driver use the blit pipeline
for sub-texture updates and also for 3D textures and array textures
(including cube map arrays)
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Tue, Dec 23, 2014 at 3:58 PM, Kenneth Graunke kenn...@whitecape.org wrote:
This was probably missed when moving from a fixed binding table layout
to a dynamic one that changes based on the shader.
Fixes newly proposed Piglit test fbo-mrt-new-bind
On Sat, Dec 20, 2014 at 8:11 AM, Rob Clark robdcl...@gmail.com wrote:
@@ -1452,6 +1593,20 @@ tgsi_transform_lowering(const struct
tgsi_lowering_config *config,
}
}
+ if ((info-processor == TGSI_PROCESSOR_FRAGMENT)
+ config-lower_alpha_test
+ (config-alpha_func
Did you mean to leave the non-DSA nvidia workaround in here? I
understand this might have been useful for getting it working
initially, but... if their driver is broken, it's broken.
On Tue, Dec 16, 2014 at 2:24 PM, Laura Ekstrand la...@jlekstrand.net wrote:
Dylan Baker: all.py: Add dsa tests.
Iago,
This doesn't matter for GL conformance -- but the impression I get is
that dEQP is aiming at something more.
In any case, the usual problem with this is inaccurate range
reduction, which is fixable in software at some performance cost. The
C library does this, for example.
- Chris
On
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Tue, Dec 9, 2014 at 8:08 PM, Matt Turner matts...@gmail.com wrote:
Back to the original commit (8313f444) adding the workaround, we were
enabling it on gens = 7, even though gens = 5 can't do multisampling.
I cannot find documentation that says
Possibly mark this for 10.4, since the assertion failures we hit when
emitting a pointless centroid workaround make other issues hard to
debug?
On Tue, Dec 9, 2014 at 8:44 PM, Chris Forbes chr...@ijw.co.nz wrote:
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Tue, Dec 9, 2014 at 8:08 PM, Matt
This is unfortunately sometimes necessary due to rebasing levels when
rendering into them.
16 piglits crash - pass, when building mesa with debug enabled.
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/drivers/dri/i965/intel_tex_validate.c | 13 ++---
1 file changed, 10
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
tests/all.py | 1 +
tests/spec/arb_gpu_shader5/CMakeLists.gl.txt | 1 +
tests/spec/arb_gpu_shader5/interpolateAtCentroid.c | 165 +
3 files changed, 167 insertions(+)
create mode
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
tests/all.py | 1 +
tests/spec/arb_gpu_shader5/CMakeLists.gl.txt | 1 +
.../arb_gpu_shader5/interpolateAtSample-nonconst.c | 186 +
3 files changed, 188 insertions(+)
create mode
This is interesting for mesa's varying packer, and i965's backend.
The API does not allow a swizzle to be supplied here, but the backend
must cope with one being introduced by packing.
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
tests/all.py | 1
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
tests/all.py | 1 +
tests/spec/arb_gpu_shader5/CMakeLists.gl.txt | 1 +
tests/spec/arb_gpu_shader5/interpolateAtSample.c | 182 +++
3 files changed, 184 insertions(+)
create mode
Module: Mesa
Branch: master
Commit: 092c73a7c32b240a26ffeab2ee475f6d590540b2
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=092c73a7c32b240a26ffeab2ee475f6d590540b2
Author: Chris Forbes chr...@ijw.co.nz
Date: Sun Dec 7 10:12:36 2014 +1300
i965: Fix regs read
Module: Mesa
Branch: master
Commit: 680f72d6f2259a1dd382a1876a594b407f933278
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=680f72d6f2259a1dd382a1876a594b407f933278
Author: Chris Forbes chr...@ijw.co.nz
Date: Sun Dec 7 10:07:16 2014 +1300
i965: Add opcode names for FS interpolation
Module: Mesa
Branch: master
Commit: ed56c16820aea0e115768d77861fe266380f2595
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=ed56c16820aea0e115768d77861fe266380f2595
Author: Chris Forbes chr...@ijw.co.nz
Date: Fri Dec 5 19:43:13 2014 +1300
i965/Gen6-7: Fix point sprites
Module: Mesa
Branch: master
Commit: 6b0196934509ac76293581f7ca69a3399ffb2e0a
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=6b0196934509ac76293581f7ca69a3399ffb2e0a
Author: Chris Forbes chr...@ijw.co.nz
Date: Sun Dec 7 11:49:28 2014 +1300
glcpp: Fix `can not` to `cannot` in error
Module: Mesa
Branch: master
Commit: b49a069bd3c96c8a144bb671863d0757fb421a07
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=b49a069bd3c96c8a144bb671863d0757fb421a07
Author: Chris Forbes chr...@ijw.co.nz
Date: Sun Nov 30 09:54:59 2014 +1300
glcpp: Disallow undefining GL_* builtin
Dead code elimination was eating the Y offset.
Fixes the piglit test:
spec/ARB_gpu_shader5/arb_gpu_shader5-interpolateAtOffset-nonconst
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/mesa/drivers
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/drivers/dri/i965/brw_shader.cpp | 9 +
1 file changed, 9 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_shader.cpp
b/src/mesa/drivers/dri/i965/brw_shader.cpp
index 183a4f1..8f6a3de 100644
--- a/src/mesa/drivers/dri/i965
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Sun, Dec 7, 2014 at 10:34 AM, Matt Turner matts...@gmail.com wrote:
---
src/mesa/drivers/dri/i965/brw_shader.cpp | 4
1 file changed, 4 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_shader.cpp
b/src/mesa/drivers/dri/i965
Both are:
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Sun, Dec 7, 2014 at 11:21 AM, Matt Turner matts...@gmail.com wrote:
---
src/mesa/drivers/dri/i965/brw_shader.cpp | 9 +
1 file changed, 9 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_shader.cpp
b/src/mesa/drivers
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Fri, Dec 5, 2014 at 10:18 AM, Timothy Arceri t_arc...@yahoo.com.au wrote:
V2: Add Catalyst test results
Test results:
AMD Radeon HD 6670 - Catalyst 13.251 OpenGL 4.3
intrastage-unsized-array-mismatch.shader_test
AMD: pass
intrastage-unsized
Both patches are:
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Fri, Dec 5, 2014 at 10:35 AM, Timothy Arceri t_arc...@yahoo.com.au wrote:
V2: add alternate version of unsized-mismatch test and fix
comment describing negitive test as a positive test
Test results:
AMD Radeon HD 6670
Ugh, yes, I'll fix that and land it at the same time.
Thanks :)
On Fri, Dec 5, 2014 at 11:12 AM, Ian Romanick i...@freedesktop.org wrote:
On 11/29/2014 12:54 PM, Chris Forbes wrote:
Fixes the piglit test: spec/glsl-es-3.00/compiler/undef-GL_ES.vert
Signed-off-by: Chris Forbes chr
of pointsprite
enables for front and back faces. Haswell ignores pointsprite
state correctly in hardware for non-point rasterization, so can
do this correctly, but it doesn't seem worth it.
Signed-off-by: Chris Forbes chr...@ijw.co.nz
Cc: 10.4 mesa-sta...@lists.freedesktop.org
Bugzilla: https
What's the perf impact on a platform which actually needs this?
I think there would be some further substantial gains to be had by
giving this its own dirty bit -- piles of other atoms listen to
BRW_NEW_VERTEX_PROGRAM, but don't care about the workaround bits.
On Thu, Dec 4, 2014 at 2:42 PM,
Half-ignore the first part of that... I thinko'd and assumed Baytrail
worked the same as Haswell here, but obviously doesn't. I am
interested in how it affects earlier gens, though.
On Thu, Dec 4, 2014 at 3:13 PM, Chris Forbes chr...@ijw.co.nz wrote:
What's the perf impact on a platform which
Either way,
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Thu, Dec 4, 2014 at 7:36 PM, Kenneth Graunke kenn...@whitecape.org wrote:
On Thursday, December 04, 2014 03:13:59 PM Chris Forbes wrote:
What's the perf impact on a platform which actually needs this?
I think there would be some
101 - 200 of 2070 matches
Mail list logo
