From: Dave Airlie
Signed-off-by: Dave Airlie
---
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
index 3b8e2fe..460bc04 100644
--- a/s
From: Dave Airlie
This adds integer version of SSG that GLSL 1.30 can produce.
Signed-off-by: Dave Airlie
---
src/gallium/auxiliary/tgsi/tgsi_exec.c | 14 ++
src/gallium/auxiliary/tgsi/tgsi_info.c |1 +
src/gallium/include/pipe/p_shader_tokens.h |3 ++-
3 files ch
On Fre, 2012-01-06 at 10:42 -0500, j.gli...@gmail.com wrote:
> From: Jerome Glisse
>
> 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.
>
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 var
On 01/09/2012 04:54 AM, Dave Airlie wrote:
From: Dave Airlie
This adds integer version of SSG that GLSL 1.30 can produce.
Signed-off-by: Dave Airlie
---
src/gallium/auxiliary/tgsi/tgsi_exec.c | 14 ++
src/gallium/auxiliary/tgsi/tgsi_info.c |1 +
src/gallium/include
On 01/09/2012 04:54 AM, Dave Airlie wrote:
From: Dave Airlie
Signed-off-by: Dave Airlie
---
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_t
On 01/07/2012 11:37 PM, Eric Anholt wrote:
On Sat, 7 Jan 2012 15:08:19 -0700, Brian Paul wrote:
This fixes accum buffer operations. The accumulation buffer is the
only malloc-based renderbuffer for the intel drivers.
---
src/mesa/drivers/dri/intel/intel_fbo.c | 13 +
1 files
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 a/src
From: Dave Airlie
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.
Signed-off-by: Dav
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 default
On 01/09/2012 08:58 AM, Dave Airlie wrote:
From: Dave Airlie
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 interp
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 to
On Sun, 8 Jan 2012 14:43:51 -0800, Kenneth Graunke
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 for "we might hav
On Sun, 8 Jan 2012 14:43:57 -0800, Kenneth Graunke
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 split, I partition
On Sun, 8 Jan 2012 14:43:58 -0800, Kenneth Graunke
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.
pgp2grlRmzbyr.pgp
Descri
On Mon, Jan 9, 2012 at 5:06 PM, Brian Paul 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 informat
On Sun, 8 Jan 2012 14:43:49 -0800, Kenneth Graunke
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 seeing.
>
> While we're
On Sun, 8 Jan 2012 16:30:29 -0800, Kenneth Graunke
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 to Eric for catchi
On Sun, 8 Jan 2012 22:44:17 -0500, Matt Turner 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 || work), not that you actually
compute the expression that you
Sorry, I mean flatshade, not flatshade_first.
Marek
On Mon, Jan 9, 2012 at 5:30 PM, Marek Olšák wrote:
> On Mon, Jan 9, 2012 at 5:06 PM, Brian Paul 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 de
- Original Message -
> On Fri, Dec 23, 2011 at 7:51 PM, Chia-I Wu wrote:
> > On Sat, Dec 24, 2011 at 12:37 AM, Ian Romanick
> > wrote:
> >> On 12/23/2011 07:20 AM, Jose Fonseca wrote:
> >>>
> >>> - Original Message -
>
> Hi list,
>
> Multiple driver support i
-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 c
Signed-off-by: Jakob Bornecrantz
---
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
b/src/gallium/auxili
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
---
src/gallium/auxiliary/draw/draw_context.c | 16 +++-
1 files changed, 11 insertions(+), 5 deletions(-)
diff
Signed-off-by: Jakob Bornecrantz
---
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
--- a/src/gallium/drivers/s
- 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 sho
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
> b
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 10064
- 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, s
- 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 7:31 PM, Jose Fonseca 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 specifies
- Original Message -
> On Mon, Jan 9, 2012 at 7:31 PM, Jose Fonseca
> 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
> >>
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.
J
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 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 fail
>> if
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 2.1.
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 wouldn't
On 01/09/2012 08:24 AM, Eric Anholt wrote:
On Sun, 8 Jan 2012 14:43:51 -0800, Kenneth Graunke
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_
On 01/09/2012 12:51 PM, Jakob Bornecrantz wrote:
On Mon, Jan 9, 2012 at 8:35 PM, Brian Paul 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
But add a option to force it on for testing.
Signed-off-by: Jakob Bornecrantz
---
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
b/src/gallium/drivers/softpipe/sp_context.c
index a7
On Mon, Jan 9, 2012 at 9:27 PM, Brian Paul wrote:
> On 01/09/2012 12:51 PM, Jakob Bornecrantz wrote:
>>
>> On Mon, Jan 9, 2012 at 8:35 PM, Brian Paul wrote:
>>>
>>> On 01/09/2012 11:28 AM, Jakob Bornecrantz wrote:
So it turns out that softpipe uses LLVM in the draw module
sinc
From: Jerome Glisse
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 doesn't remove
the 64K i
On 01/09/2012 01:32 PM, Jakob Bornecrantz wrote:
On Mon, Jan 9, 2012 at 9:27 PM, Brian Paul wrote:
On 01/09/2012 12:51 PM, Jakob Bornecrantz wrote:
On Mon, Jan 9, 2012 at 8:35 PM, Brian Paulwrote:
On 01/09/2012 11:28 AM, Jakob Bornecrantz wrote:
So it turns out that softpipe uses LLV
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
--- a/src/mesa/main/fbob
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 the transform
feedback info in the gl_transform_feedback_info::Outputs arra
On 01/09/2012 08:28 AM, Eric Anholt wrote:
On Sun, 8 Jan 2012 14:43:57 -0800, Kenneth Graunke
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 per
On Thu, 5 Jan 2012 21:17:52 -0600, Andy Clayton 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 does not caus
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 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 the transform
>
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 = clamp(R0.x
---
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 @@ _mesa_init_sqrt_t
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 ddd70be..
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 ch
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
---
src/mesa/drivers/dri/i965/brw_defines.h |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_
It is unwise to use a stencil region's size to determine its
renderbuffer's size, because at region creation we fudge the width and
height to accomodate interleaved rows. (See the comment for MESA_FORMAT_S8
in intel_miptree_create()). Most users of stencil_region->{width,height}
should be converted
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: git://people.freedesktop.or
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 +++---
src/mesa/drivers/dri/i965/gen7_
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. A
Signed-off-by: Jakob Bornecrantz
---
.../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
b/src/gallium/auxiliary/target-helpers/inline_wrapper_sw_helper.h
inde
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
---
.../draw/draw_pt_fetch_shade_pipeline_llvm.c | 12
1 files changed, 0 insertions(+), 12 deleti
Signed-off-by: Jakob Bornecrantz
---
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
--- a/src/gallium/drivers/rbug/rbug_core.
Signed-off-by: Jakob Bornecrantz
---
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
+++ b/src/mesa/progra
Signed-off-by: Jakob Bornecrantz
---
src/mesa/program/nvvertparse.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/src/mesa/program/nvvertparse.c b/src/mesa/program/nvvertparse.c
index 91fe2c4..7b46bef 100644
--- a/src/mesa/program/nvvertparse.c
+++ b/src/mesa/program/nv
Signed-off-by: Jakob Bornecrantz
---
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
--- a/src/mesa/.gitignore
+++ b/src/mesa/.gitignore
@@ -3,3 +3,4 @@
depend.es*
depend.es*
ob
Signed-off-by: Jakob Bornecrantz
---
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
b/src/gallium/targets/xa-vmwgfx/.gitignore
Signed-off-by: Jakob Bornecrantz
---
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
--- a/src/gallium/winsys/svg
Signed-off-by: Jakob Bornecrantz
---
src/gallium/drivers/svga/svga_pipe_draw.c |3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/src/gallium/drivers/svga/svga_pipe_draw.c
b/src/gallium/drivers/svga/svga_pipe_draw.c
index c61d5d0..541a234 100644
--- a/src/gallium/drivers/
Signed-off-by: Jakob Bornecrantz
---
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 15cd402..0c5cb90 100644
--- a/s
Signed-off-by: Jakob Bornecrantz
---
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 100644
--- a/src
Signed-off-by: Jakob Bornecrantz
---
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 --
src/gallium/winsys/s
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 Bornecrantz
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
mesa-dev@lists.freed
On 01/09/2012 03:53 PM, Jakob Bornecrantz wrote:
Signed-off-by: Jakob Bornecrantz
---
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
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 doi
On 01/09/2012 02:53 PM, Jakob Bornecrantz wrote:
Signed-off-by: Jakob Bornecrantz
Reviewed-by: Ian Romanick
---
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
On 01/09/2012 02:53 PM, Jakob Bornecrantz wrote:
Signed-off-by: Jakob Bornecrantz
Reviewed-by: Ian Romanick
---
src/mesa/program/nvvertparse.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/src/mesa/program/nvvertparse.c b/src/mesa/program/nvvertparse.c
index 91f
On 9 January 2012 14:28, Ian Romanick 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
>> event that US
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 +-
src/mesa/drivers/dri/i965/brw_misc_stat
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 = R0
On 01/09/2012 08:36 AM, Eric Anholt wrote:
On Sun, 8 Jan 2012 22:44:17 -0500, Matt Turner 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 || work), not that
On Fri, 6 Jan 2012 16:49:59 -0800, "Ian Romanick" wrote:
> From: Ian Romanick
>
> The only error that the vertex shader precompile could generate is
> when register allocation fails. Since i965 can spill registers, this
> should never happen. It's also not something that can cause a link
> er
On Tue, Jan 10, 2012 at 12:26 AM, Brian Paul wrote:
> On 01/09/2012 03:53 PM, Jakob Bornecrantz wrote:
>>
>> Signed-off-by: Jakob Bornecrantz
>
>
> Maybe you could elaborate in the comment why the code is being dropped. Was
> it just unused or something else?
It was unused, I'll add a comment ab
On Fri, 6 Jan 2012 16:50:00 -0800, "Ian Romanick" wrote:
> From: Ian Romanick
>
> 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:
>
> * Trying to emit instructions that are
> 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
> Signed-off-by: Anuj Phogat
> ---
> src/mesa/drivers/dri/
On Fri, 6 Jan 2012 16:50:03 -0800, "Ian Romanick" wrote:
> From: Ian Romanick
>
> Signed-off-by: Ian Romanick
Instead of making the compile process whack the assembly prog struct as
a side-effect, I think better would be to put uses_kill in the prog_data
during the GLSL and the ARB compile, a
On Tue, Jan 10, 2012 at 12:01 AM, Alan Coopersmith
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 call instead of co
On Tue, Jan 10, 2012 at 12:29 AM, Brian Paul wrote:
> 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 doin
On Fri, 6 Jan 2012 16:50:04 -0800, "Ian Romanick" wrote:
> From: Ian Romanick
>
> 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
> ---
> src/mesa/drivers/dri/i965/brw_shader.cpp |
On Fri, 6 Jan 2012 16:49:51 -0800, "Ian Romanick" 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 Anholt
pgp9oNvz77r
1 - 100 of 110 matches
Mail list logo
