From: Dave Airlie airl...@redhat.com
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp |5 -
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
From: Dave Airlie airl...@redhat.com
This adds integer version of SSG that GLSL 1.30 can produce.
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/gallium/auxiliary/tgsi/tgsi_exec.c | 14 ++
src/gallium/auxiliary/tgsi/tgsi_info.c |1 +
On Fre, 2012-01-06 at 10:42 -0500, j.gli...@gmail.com wrote:
From: Jerome Glisse jgli...@redhat.com
Virtual address space put the userspace in charge of their GPU
address space. It's up to userspace to bind bo into the virtual
address space. Command stream can them be executed using the
Okay so we have a gallium/GL/GLSL problem with interpolation, lets see
if I can describe the issue:
In the beginning there was GL interpolation rules, glShadeModel was
the only information you had.
Then came GLSL which allowed overriding the GL defaults using specifiers.
So the gl_FrontColor
On 01/09/2012 04:54 AM, Dave Airlie wrote:
From: Dave Airlieairl...@redhat.com
This adds integer version of SSG that GLSL 1.30 can produce.
Signed-off-by: Dave Airlieairl...@redhat.com
---
src/gallium/auxiliary/tgsi/tgsi_exec.c | 14 ++
On 01/09/2012 04:54 AM, Dave Airlie wrote:
From: Dave Airlieairl...@redhat.com
Signed-off-by: Dave Airlieairl...@redhat.com
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp |5 -
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
This fixes accum buffer operations. The accumulation buffer is the
only malloc-based renderbuffer for the intel drivers.
v2: apply x/y offset to returned pointer
---
src/mesa/drivers/dri/intel/intel_fbo.c | 15 +++
1 files changed, 15 insertions(+), 0 deletions(-)
diff --git
From: Dave Airlie airl...@redhat.com
This introduces an unspecified interpolation paramter that is only allowed for
color semantics, so a specified GLSL interpolation will override the ShadeModel
specified interpolation, but not vice-versa.
This fixes a lot of the interpolation tests in piglit.
On 01/09/2012 06:35 AM, Dave Airlie wrote:
Okay so we have a gallium/GL/GLSL problem with interpolation, lets see
if I can describe the issue:
In the beginning there was GL interpolation rules, glShadeModel was
the only information you had.
Then came GLSL which allowed overriding the GL
On 01/09/2012 08:58 AM, Dave Airlie wrote:
From: Dave Airlieairl...@redhat.com
This introduces an unspecified interpolation paramter that is only allowed for
color semantics, so a specified GLSL interpolation will override the ShadeModel
specified interpolation, but not vice-versa.
This fixes
I don't really care for these patches. They obfuscate the code and
hide the real issue which Keith touched on. We need to have some sort
of additional dirty tracking logic to determine when there's no real
changes to the pipe_resource for a texture.
One of the things on my to-do list is
On Sun, 8 Jan 2012 14:43:51 -0800, Kenneth Graunke kenn...@whitecape.org
wrote:
Without MI_SET_CONTEXT, there's no guarantee that another program hasn't
reprogram the GPU's clip state since our last batch. We need to submit
it every time.
Found by inspection.
BRW_NEW_CONTEXT is the flag
On Sun, 8 Jan 2012 14:43:57 -0800, Kenneth Graunke kenn...@whitecape.org
wrote:
According to a comment in gen6_sf_state.c, calls to get_attr_override
need both _NEW_PROGRAM and _NEW_LIGHT. Since Gen7 reuses the same
function, the same dirty bits should apply.
When I performed the SF/SBE
On Sun, 8 Jan 2012 14:43:58 -0800, Kenneth Graunke kenn...@whitecape.org
wrote:
From my reading of main/enable.c:735, it appears that
ctx-VertexProgram.PointSizeEnabled is covered by _NEW_PROGRAM.
Oh, and you go fix it here. I'd rather not see the flag dropped in the
previous commit.
On Mon, Jan 9, 2012 at 5:06 PM, Brian Paul bri...@vmware.com wrote:
On 01/09/2012 06:35 AM, Dave Airlie wrote:
Okay so we have a gallium/GL/GLSL problem with interpolation, lets see
if I can describe the issue:
In the beginning there was GL interpolation rules, glShadeModel was
the only
On Sun, 8 Jan 2012 14:43:49 -0800, Kenneth Graunke kenn...@whitecape.org
wrote:
While investigating an etracer issue on Ivybridge, I did a lot of careful
reading of the tracked state atoms' dirty bits. Found a bunch of issues.
Unfortunately, none of these patches fixed the issues I was
On Sun, 8 Jan 2012 16:30:29 -0800, Kenneth Graunke kenn...@whitecape.org
wrote:
This brings the code in sync with gen6_sf_state.c; presumably the
mistake was a botched rebase on initial Ivybridge bring-up patches.
Found by diffing batch buffer dumps and noticing the random values.
Thanks
- Original Message -
On Fri, Dec 23, 2011 at 7:51 PM, Chia-I Wu olva...@gmail.com wrote:
On Sat, Dec 24, 2011 at 12:37 AM, Ian Romanick
i...@freedesktop.org wrote:
On 12/23/2011 07:20 AM, Jose Fonseca wrote:
- Original Message -
Hi list,
Multiple driver support
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 01/06/2012 05:54 PM, Kenneth Graunke wrote:
--- Makefile|2 +- configs/default |
4 +- docs/relnotes-7.12.html | 83
---
docs/relnotes-8.0.html | 83
Looks good Tom.
Jose
- Original Message -
Target specific intrinsics are also prefixed with llvm, so this
assert
was preventing us from using them.
---
src/gallium/auxiliary/gallivm/lp_bld_intr.c |7 ---
1 files changed, 0 insertions(+), 7 deletions(-)
diff --git
So it turns out that softpipe uses LLVM in the draw module
since it has no way of telling draw not to use it. This patch
series fixes that. Also makes the draw initialization fail
if draw can't initialize LLVM in the normal case.
Cheers, Jakob.
Jakob Bornecrantz (3):
draw: Make it possible to
Signed-off-by: Jakob Bornecrantz wallbra...@gmail.com
---
src/gallium/auxiliary/draw/draw_context.c | 48 -
src/gallium/auxiliary/draw/draw_context.h |2 +
2 files changed, 35 insertions(+), 15 deletions(-)
diff --git a/src/gallium/auxiliary/draw/draw_context.c
The r300 driver requires LLVM when building and other drivers that
depend on it for all TNL, like i915g will be a lot slower without it.
Signed-off-by: Jakob Bornecrantz wallbra...@gmail.com
---
src/gallium/auxiliary/draw/draw_context.c | 16 +++-
1 files changed, 11 insertions(+),
Signed-off-by: Jakob Bornecrantz wallbra...@gmail.com
---
src/gallium/drivers/softpipe/sp_context.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/gallium/drivers/softpipe/sp_context.c
b/src/gallium/drivers/softpipe/sp_context.c
index a720600..dd81442 100644
---
- Original Message -
There are 3 changes:
1) stride is specified for each buffer, not just one, so that drivers
don't
have to derive it from the outputs
2) new per-output property dst_offset, which specifies the offset
into the buffer in dwords where the output should be
Looks good to me AFAICT.
Jose
- Original Message -
---
src/mesa/main/format_unpack.c | 499
+
src/mesa/main/format_unpack.h |3 +
2 files changed, 502 insertions(+), 0 deletions(-)
diff --git a/src/mesa/main/format_unpack.c
Looks good to me.
Jose
- Original Message -
---
src/mesa/swrast/s_blit.c | 183
+++--
1 files changed, 158 insertions(+), 25 deletions(-)
diff --git a/src/mesa/swrast/s_blit.c b/src/mesa/swrast/s_blit.c
index 1063024..34317cc 100644
---
- Original Message -
Hi guys,
Just a quick note, I've just spent a week or so trying to see where
gallium and softpipe were w.r.t GL3.0 support.
I've pushed a branch to my repo called softpipe-gl3. It contains
patches in various state of usefulness but it brings the piglit
Looks good to me.
Jose
- Original Message -
---
src/mesa/main/fbobject.c | 44
1 files changed, 44 insertions(+), 0 deletions(-)
diff --git a/src/mesa/main/fbobject.c b/src/mesa/main/fbobject.c
index ddd70be..6ff09f7 100644
---
Tom,
I think this is perfectly sensible thing to do if it helps, but
tgsi_output_type definition should be added to tgsi_info.h and not
src/gallium/include/pipe/p_shader_tokens.h as it is not really used anywhere in
the tokenized representation.
Jose
- Original Message -
I've been
We were naively emitting each component at a time, even if we were
emitting the same value to multiple channels. Improves on a codegen
regression from the old VS to the new VS on some unigine shaders
(because we emit constant vecs/matrices as immediates instead of
loading them as push constants,
- Original Message -
We were comparing 32-bit Z buffer values against 16-bit fragment
values.
Need to do scaling like for the 24-bit case.
Triangle Z testing was OK since it didn't hit this code path.
---
src/mesa/swrast/s_depth.c | 36
+---
1
On Mon, Jan 9, 2012 at 7:31 PM, Jose Fonseca jfons...@vmware.com wrote:
- Original Message -
There are 3 changes:
1) stride is specified for each buffer, not just one, so that drivers
don't
have to derive it from the outputs
2) new per-output property dst_offset, which
- Original Message -
On Mon, Jan 9, 2012 at 7:31 PM, Jose Fonseca jfons...@vmware.com
wrote:
- Original Message -
There are 3 changes:
1) stride is specified for each buffer, not just one, so that
drivers
don't
have to derive it from the outputs
2) new
On 01/09/2012 11:28 AM, Jakob Bornecrantz wrote:
So it turns out that softpipe uses LLVM in the draw module
since it has no way of telling draw not to use it. This patch
series fixes that. Also makes the draw initialization fail
if draw can't initialize LLVM in the normal case.
Cheers, Jakob.
On 01/09/2012 12:09 PM, Jose Fonseca wrote:
- Original Message -
We were comparing 32-bit Z buffer values against 16-bit fragment
values.
Need to do scaling like for the 24-bit case.
Triangle Z testing was OK since it didn't hit this code path.
---
src/mesa/swrast/s_depth.c | 36
On Mon, Jan 9, 2012 at 8:35 PM, Brian Paul bri...@vmware.com wrote:
On 01/09/2012 11:28 AM, Jakob Bornecrantz wrote:
So it turns out that softpipe uses LLVM in the draw module
since it has no way of telling draw not to use it. This patch
series fixes that. Also makes the draw initialization
On 01/09/2012 09:16 AM, Jason Wood wrote:
On 01/06/2012 05:54 PM, Kenneth Graunke wrote:
+Mesa 8.0 implements the OpenGL 2.1 API, but the
version reported by +glGetString(GL_VERSION) depends on the
particular driver being used. +Some drivers don't support all the
features required in OpenGL
Previous to this patch, we didn't do the limit check for
MAX_TRANSFORM_FEEDBACK_INTERLEAVED_COMPONENTS until the end of the
store_tfeedback_info() function, *after* storing all of the transform
feedback info in the gl_transform_feedback_info::Outputs array. This
meant that the limit check
On 01/09/2012 12:51 PM, Jakob Bornecrantz wrote:
On Mon, Jan 9, 2012 at 8:35 PM, Brian Paulbri...@vmware.com wrote:
On 01/09/2012 11:28 AM, Jakob Bornecrantz wrote:
So it turns out that softpipe uses LLVM in the draw module
since it has no way of telling draw not to use it. This patch
series
But add a option to force it on for testing.
Signed-off-by: Jakob Bornecrantz wallbra...@gmail.com
---
src/gallium/drivers/softpipe/sp_context.c |5 -
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/src/gallium/drivers/softpipe/sp_context.c
On Mon, Jan 9, 2012 at 9:27 PM, Brian Paul bri...@vmware.com wrote:
On 01/09/2012 12:51 PM, Jakob Bornecrantz wrote:
On Mon, Jan 9, 2012 at 8:35 PM, Brian Paulbri...@vmware.com wrote:
On 01/09/2012 11:28 AM, Jakob Bornecrantz wrote:
So it turns out that softpipe uses LLVM in the draw
From: Jerome Glisse jgli...@redhat.com
Virtual address space put the userspace in charge of their GPU
address space. It's up to userspace to bind bo into the virtual
address space. Command stream can them be executed using the
IB_VM chunck.
This patch add support for this configuration. It
On 01/09/2012 01:32 PM, Jakob Bornecrantz wrote:
On Mon, Jan 9, 2012 at 9:27 PM, Brian Paulbri...@vmware.com wrote:
On 01/09/2012 12:51 PM, Jakob Bornecrantz wrote:
On Mon, Jan 9, 2012 at 8:35 PM, Brian Paulbri...@vmware.comwrote:
On 01/09/2012 11:28 AM, Jakob Bornecrantz wrote:
So
On 01/06/2012 04:13 PM, Brian Paul wrote:
---
src/mesa/main/fbobject.c | 44
1 files changed, 44 insertions(+), 0 deletions(-)
diff --git a/src/mesa/main/fbobject.c b/src/mesa/main/fbobject.c
index ddd70be..6ff09f7 100644
---
On 01/09/2012 08:28 AM, Eric Anholt wrote:
On Sun, 8 Jan 2012 14:43:57 -0800, Kenneth Graunkekenn...@whitecape.org
wrote:
According to a comment in gen6_sf_state.c, calls to get_attr_override
need both _NEW_PROGRAM and _NEW_LIGHT. Since Gen7 reuses the same
function, the same dirty bits
On Thu, 5 Jan 2012 21:17:52 -0600, Andy Clayton q3a...@gmail.com wrote:
Hi all,
So this is my attempt at a fix for
https://bugs.freedesktop.org/show_bug.cgi?id=34006. Any feedback on if
this looks right to anyone is much appreciated. Sven Arvidsson did
confirm on bugzilla that the patch
https://bugs.freedesktop.org/show_bug.cgi?id=41152
Bug 41152 depends on bug 34006, which changed state.
Bug 34006 Summary: [glsl regression] multiple games under wine trigger infinite
loop in glsl optimization
https://bugs.freedesktop.org/show_bug.cgi?id=34006
What|Old Value
On 9 January 2012 13:00, Ian Romanick i...@freedesktop.org wrote:
On 01/09/2012 12:02 PM, Paul Berry wrote:
Previous to this patch, we didn't do the limit check for
MAX_TRANSFORM_FEEDBACK_**INTERLEAVED_COMPONENTS until the end of the
store_tfeedback_info() function, *after* storing all of
No functional change. In the function
__indirect_glAreTexturesResident(), the variable cmdlen is only used
if USE_XCB is not defined. This patch avoids a compile warning in the
event that USE_XCB is defined.
---
src/glx/single2.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff
We were doing the kill of the updated channels, then adding our copy
to the list of available stuff to copy. But if the copy was updating
its own source channels, we didn't notice, breaking this code:
R0.xyzw = arg0 + arg1;
R0.xyzw = R0.wwwx;
gl_FragColor.xyzw =
---
src/mesa/main/imports.h | 16
1 files changed, 12 insertions(+), 4 deletions(-)
diff --git a/src/mesa/main/imports.h b/src/mesa/main/imports.h
index b7e8743..522ffeb 100644
--- a/src/mesa/main/imports.h
+++ b/src/mesa/main/imports.h
@@ -568,13 +568,19 @@
On 01/09/2012 01:54 PM, Ian Romanick wrote:
On 01/06/2012 04:13 PM, Brian Paul wrote:
---
src/mesa/main/fbobject.c | 44
1 files changed, 44 insertions(+), 0 deletions(-)
diff --git a/src/mesa/main/fbobject.c b/src/mesa/main/fbobject.c
index
On 01/09/2012 01:27 PM, Paul Berry wrote:
No functional change. In the function
__indirect_glAreTexturesResident(), the variable cmdlen is only used
if USE_XCB is not defined. This patch avoids a compile warning in the
event that USE_XCB is defined.
---
src/glx/single2.c |1 +
1 files
It was named GEN6_WM_DEPTH_RESOLVE. Luckily, this caused no conflict,
because the value is identical for gen6 and gen7.
Signed-off-by: Chad Versace chad.vers...@linux.intel.com
---
src/mesa/drivers/dri/i965/brw_defines.h |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git
This patch modifies all batches needed for HiZ. The batch length for
3DSTATE_HIER_DEPTH_BUFFER is also corrected from 4 to 3.
Performance +6.7% on Citybench.
num-frames: 400
resolution: 1918x1031
avg-hiz-off: 127.90 fps
avg-hiz-on: 136.50 fps
kernel:
Chad Versace (3):
i965: Fix misnamed GEN7_WM_DEPTH_RESOLVE
i965: Replace references to stencil region size with buffer size
i965/gen7: Enable HiZ
src/mesa/drivers/dri/i965/brw_defines.h |2 +-
src/mesa/drivers/dri/i965/brw_misc_state.c |8 +++---
The first patch is a fix for the scons release build of the vmwgfx-dri driver
I managed to break without noticing it. The others are mostly warning
silencers that I found when doing a scons release build, I'm not sure if this
is the good way of doing them, but it got rid of the warnings at least.
Signed-off-by: Jakob Bornecrantz ja...@vmware.com
---
.../target-helpers/inline_wrapper_sw_helper.h |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/src/gallium/auxiliary/target-helpers/inline_wrapper_sw_helper.h
This peice of code has been here since the inital commit (c5c5cd71) and the
code that used instance_id_index was removed in (caede752) by José.
Signed-off-by: Jakob Bornecrantz ja...@vmware.com
---
.../draw/draw_pt_fetch_shade_pipeline_llvm.c | 12
1 files changed, 0
Signed-off-by: Jakob Bornecrantz ja...@vmware.com
---
src/gallium/drivers/rbug/rbug_core.c |3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/src/gallium/drivers/rbug/rbug_core.c
b/src/gallium/drivers/rbug/rbug_core.c
index b80bcd4..253d21b 100644
---
Signed-off-by: Jakob Bornecrantz ja...@vmware.com
---
src/mesa/program/symbol_table.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/src/mesa/program/symbol_table.c b/src/mesa/program/symbol_table.c
index 004f1f8..4f6f31f 100644
--- a/src/mesa/program/symbol_table.c
+++
Signed-off-by: Jakob Bornecrantz ja...@vmware.com
---
src/gallium/targets/xa-vmwgfx/.gitignore |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
create mode 100644 src/gallium/targets/xa-vmwgfx/.gitignore
diff --git a/src/gallium/targets/xa-vmwgfx/.gitignore
Signed-off-by: Jakob Bornecrantz ja...@vmware.com
---
src/gallium/winsys/svga/drm/vmw_context.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/src/gallium/winsys/svga/drm/vmw_context.c
b/src/gallium/winsys/svga/drm/vmw_context.c
index b5ca841..2edb5ea 100644
---
Signed-off-by: Jakob Bornecrantz ja...@vmware.com
---
src/gallium/drivers/svga/svga_tgsi_decl_sm30.c |7 ---
1 files changed, 0 insertions(+), 7 deletions(-)
diff --git a/src/gallium/drivers/svga/svga_tgsi_decl_sm30.c
b/src/gallium/drivers/svga/svga_tgsi_decl_sm30.c
index
Signed-off-by: Jakob Bornecrantz ja...@vmware.com
---
src/gallium/drivers/svga/svga_tgsi_decl_sm30.c |5 ++---
1 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/src/gallium/drivers/svga/svga_tgsi_decl_sm30.c
b/src/gallium/drivers/svga/svga_tgsi_decl_sm30.c
index 0c5cb90..a4e80b5
Signed-off-by: Jakob Bornecrantz ja...@vmware.com
---
src/gallium/targets/dri-vmwgfx/target.c |1 -
src/gallium/winsys/svga/drm/vmw_context.c | 21 +
src/gallium/winsys/svga/drm/vmw_context.h |4
src/gallium/winsys/svga/drm/vmw_screen.c | 10 --
On 01/ 9/12 02:53 PM, Jakob Bornecrantz wrote:
- struct pipe_shader_state pss = { 0 };
+ struct pipe_shader_state pss;
+ memset(pss, 0, sizeof(pss));
Those do the same thing, just via a function call instead of compiler builtin.
What warning is being silenced by that change?
--
On 01/09/2012 03:53 PM, Jakob Bornecrantz wrote:
Signed-off-by: Jakob Bornecrantzja...@vmware.com
Maybe you could elaborate in the comment why the code is being
dropped. Was it just unused or something else?
-Brian
___
mesa-dev mailing list
On 01/09/2012 03:53 PM, Jakob Bornecrantz wrote:
Signed-off-by: Jakob Bornecrantzja...@vmware.com
---
src/gallium/drivers/svga/svga_tgsi_decl_sm30.c |5 ++---
1 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/src/gallium/drivers/svga/svga_tgsi_decl_sm30.c
On 01/09/2012 03:53 PM, Jakob Bornecrantz wrote:
The first patch is a fix for the scons release build of the vmwgfx-dri driver
I managed to break without noticing it. The others are mostly warning
silencers that I found when doing a scons release build, I'm not sure if this
is the good way of
On 01/09/2012 02:53 PM, Jakob Bornecrantz wrote:
Signed-off-by: Jakob Bornecrantzja...@vmware.com
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
---
src/mesa/program/nvvertparse.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/src/mesa/program/nvvertparse.c
On 9 January 2012 14:28, Ian Romanick i...@freedesktop.org wrote:
On 01/09/2012 01:27 PM, Paul Berry wrote:
No functional change. In the function
__indirect_**glAreTexturesResident(), the variable cmdlen is only used
if USE_XCB is not defined. This patch avoids a compile warning in the
No functional change. In the function
__indirect_glAreTexturesResident(), the variable cmdlen is only used
if USE_XCB is not defined. This patch avoids a compile warning in the
event that USE_XCB is defined.
v2: just move cmdlen declaration inside the #else part.
---
src/glx/single2.c |2
On 01/09/2012 03:56 PM, Paul Berry wrote:
No functional change. In the function
__indirect_glAreTexturesResident(), the variable cmdlen is only used
if USE_XCB is not defined. This patch avoids a compile warning in the
event that USE_XCB is defined.
v2: just move cmdlen declaration inside the
Enabled only for nvc0 at the moment.
---
src/gallium/docs/source/screen.rst |2 ++
src/gallium/drivers/i915/i915_screen.c |1 +
src/gallium/drivers/llvmpipe/lp_screen.c |2 ++
src/gallium/drivers/nv50/nv50_screen.c |2 ++
src/gallium/drivers/nvc0/nvc0_screen.c |1 +
---
src/mesa/state_tracker/st_extensions.c |8
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 14 +++---
2 files changed, 19 insertions(+), 3 deletions(-)
diff --git a/src/mesa/state_tracker/st_extensions.c
b/src/mesa/state_tracker/st_extensions.c
index a9d4054..c8171ef
On 01/09/2012 02:30 PM, Chad Versace wrote:
Chad Versace (3):
i965: Fix misnamed GEN7_WM_DEPTH_RESOLVE
i965: Replace references to stencil region size with buffer size
i965/gen7: Enable HiZ
src/mesa/drivers/dri/i965/brw_defines.h |2 +-
On 01/09/2012 03:56 PM, Paul Berry wrote:
No functional change. In the function
__indirect_glAreTexturesResident(), the variable cmdlen is only used
if USE_XCB is not defined. This patch avoids a compile warning in the
event that USE_XCB is defined.
v2: just move cmdlen declaration inside the
On 01/09/2012 01:36 PM, Eric Anholt wrote:
We were doing the kill of the updated channels, then adding our copy
to the list of available stuff to copy. But if the copy was updating
its own source channels, we didn't notice, breaking this code:
R0.xyzw = arg0 + arg1;
R0.xyzw =
On 01/09/2012 08:36 AM, Eric Anholt wrote:
On Sun, 8 Jan 2012 22:44:17 -0500, Matt Turnermatts...@gmail.com wrote:
GLSL spec boneheadedly says (a ^^ a) must actually be evaluated.
What it's trying to say there is that any side-effects of both LHS and
RHS will occur (as opposed to how or ||
On Tue, Jan 10, 2012 at 12:26 AM, Brian Paul bri...@vmware.com wrote:
On 01/09/2012 03:53 PM, Jakob Bornecrantz wrote:
Signed-off-by: Jakob Bornecrantzja...@vmware.com
Maybe you could elaborate in the comment why the code is being dropped. Was
it just unused or something else?
It was
On Fri, 6 Jan 2012 16:50:00 -0800, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
There are several things that could cause the fragment shader
precompile to fail. Looking for calls to 'fail' in brw_fs_visitor.cpp
finds three classes of things:
*
glDrawBuffer(GL_FRONT_AND_BACK) results in to segmentation fault if
intel-is_front_buffer_rendering is not enabled with GL_FRONT_AND_BACK.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=44153
Reported-by: Yi Sun yi@intel.com
Signed-off-by: Anuj Phogat anuj.pho...@gmail.com
On Fri, 6 Jan 2012 16:50:03 -0800, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
Instead of making the compile process whack the assembly prog struct as
a side-effect, I think better would be to put
On Tue, Jan 10, 2012 at 12:01 AM, Alan Coopersmith
alan.coopersm...@oracle.com wrote:
On 01/ 9/12 02:53 PM, Jakob Bornecrantz wrote:
- struct pipe_shader_state pss = { 0 };
+ struct pipe_shader_state pss;
+ memset(pss, 0, sizeof(pss));
Those do the same thing, just via a function
On Fri, 6 Jan 2012 16:50:04 -0800, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
Instead, do the uniform setting and input / output mapping directly in
brw_link_shader. Hurray for not generating Mesa IR!
Signed-off-by: Ian Romanick
On Fri, 6 Jan 2012 16:49:51 -0800, Ian Romanick i...@freedesktop.org wrote:
Patches 01/14 through 03/14 was previously sent to the list. It
tickled a pre-existing bug (fixed by patch recently posted to the list
by Eric) on some Intel chipsets.
Patches 1-7 and 11 are:
Reviewed-by: Eric
Fixes piglit
glsl-1.30/compiler/interpolation-qualifiers/local-smooth-01.frag.
---
src/glsl/ast_to_hir.cpp | 23 +++
1 files changed, 23 insertions(+), 0 deletions(-)
diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp
index 1aebca4..cde7052 100644
---
We're not quite ready to actually support it in the implementation,
but at least this allows GL 3.0 API-reliant applications to hopefully
run successfully, though they won't get multisampling.
---
src/mesa/drivers/dri/i965/brw_context.c |8
1 files changed, 8 insertions(+), 0
The EXT_texture_array required only 64, but GL 3.0 required 256.
Since we're already exposing values that can get us way beyond our
ability to map the single object directly, go ahead and expose all the
way to hardware limits.
Tested with new piglit EXT_texture_array/maxlayers on gen7.
---
Hi Dave,
This regresses interpolation-none-gl_FrontColor-flat-vertex.shader_test
piglit test on i915g.
Stéphane
On Sat, Jan 7, 2012 at 00:39, Dave Airlie
airl...@kemper.freedesktop.org wrote:
Module: Mesa
Branch: master
Commit: a103c61d278b7f56208818c4b949c408c00286fa
URL:
On 01/09/2012 06:04 PM, Eric Anholt wrote:
The EXT_texture_array required only 64, but GL 3.0 required 256.
Since we're already exposing values that can get us way beyond our
ability to map the single object directly, go ahead and expose all the
way to hardware limits.
Tested with new piglit
On Mon, 2012-01-09 at 23:53 +0100, Jakob Bornecrantz wrote:
Signed-off-by: Jakob Bornecrantz ja...@vmware.com
---
src/mesa/.gitignore |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/src/mesa/.gitignore b/src/mesa/.gitignore
index ce83eaf..0cf001e 100644
---
On 01/06/2012 04:49 PM, Ian Romanick wrote:
From: Ian Romanickian.d.roman...@intel.com
Track the calculated data in gl_shader_program instead of the
individual assembly shaders.
Signed-off-by: Ian Romanickian.d.roman...@intel.com
---
src/glsl/link_uniforms.cpp | 15
On Mon, 2012-01-09 at 15:38 -0500, j.gli...@gmail.com wrote:
From: Jerome Glisse jgli...@redhat.com
Virtual address space put the userspace in charge of their GPU
address space. It's up to userspace to bind bo into the virtual
address space. Command stream can them be executed using the
On 01/09/2012 05:36 PM, Eric Anholt wrote:
On Fri, 6 Jan 2012 16:50:03 -0800, Ian Romanicki...@freedesktop.org wrote:
From: Ian Romanickian.d.roman...@intel.com
Signed-off-by: Ian Romanickian.d.roman...@intel.com
Instead of making the compile process whack the assembly prog struct as
a
On 01/09/2012 05:21 PM, Eric Anholt wrote:
On Fri, 6 Jan 2012 16:49:59 -0800, Ian Romanicki...@freedesktop.org wrote:
From: Ian Romanickian.d.roman...@intel.com
The only error that the vertex shader precompile could generate is
when register allocation fails. Since i965 can spill registers,
98 matches
Mail list logo
