https://bugs.freedesktop.org/show_bug.cgi?id=65514
Nicholas Miell changed:
What|Removed |Added
Assignee|dri-devel@lists.freedesktop |mesa-dev@lists.freedesktop.
On Fri, Jun 07, 2013 at 05:48:05PM -0700, Tom Stellard wrote:
> On Fri, Jun 07, 2013 at 05:24:42PM +0200, Michel Dänzer wrote:
> >
> > The most important difference to the previous version of these is that
> > whole quad mode is now enabled and M0 initialized appropriately for the
> > LDS instruct
On 06/08/2013 01:30 PM, Chris Forbes wrote:
Does that flush still need to happen, if you're no longer emitting
CMD_PIPELINE_SELECT?
Unfortunately, yes, for a rather stupid reason: if our current batch was
getting full, BLORP might run out of space and doesn't have any
mechanism for handling t
Fixes "Uninitialized scalar field" defect reported by Coverity.
Signed-off-by: Vinson Lee
---
src/glsl/link_uniforms.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/glsl/link_uniforms.cpp b/src/glsl/link_uniforms.cpp
index 010296b..d42b6cd 100644
--- a/src/glsl/link_u
Does that flush still need to happen, if you're no longer emitting
CMD_PIPELINE_SELECT?
On Sun, Jun 9, 2013 at 7:01 AM, Kenneth Graunke wrote:
> Now that we emit invariant state at startup (and never select the media
> pipeline), the 3D pipeline will always already be selected, even if BLORP
> is
From: Ian Romanick
This has the (intended!) side effect that vertex shader inputs and
fragment shader outputs will appear in the IR in the same order that
they appeared in the shader code. This results in the locations being
assigned in the declared order. Many (arguably buggy) applications
dep
Now that Gen6+ relies on hardware contexts, we don't need to record an
occlusion query value at the end of each batch. That means we no longer
need to reserve space for the absurd number of PIPE_CONTROLs required to
do that on Sandybridge.
See commit 4e087de51ad0e7ba4a7199d3664e1d096f8dc510, whic
We always allocate the maximum amount of space and never change it, so
it makes sense to do it once. Programming it on startup also lets us
skip re-programming it from BLORP.
This removes a tiny amount of overhead from our drawing loop.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i
This removes a tiny bit of code from our drawing loop.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_state.h| 4 +++-
src/mesa/drivers/dri/i965/brw_state_upload.c | 5 -
src/mesa/drivers/dri/i965/gen7_disable.c | 13 ++---
3 files changed, 9 insertion
Now that we emit invariant state at startup (and never select the media
pipeline), the 3D pipeline will always already be selected, even if BLORP
is the first operation. So this is unnecessary.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/gen6_blorp.cpp | 18 --
Now that we have hardware contexts, we can safely initialize our GPU
state once at startup, rather than needing a state atom with the
BRW_NEW_CONTEXT flag set.
This removes a tiny bit of code from our drawing loop.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_misc_state.c
The existing code already returned a boolean; this just clarifies that.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_state_upload.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_state_upload.c
b/src/mesa/drivers/dri/i96
These atoms don't actually exist.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_state.h | 9 -
1 file changed, 9 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_state.h
b/src/mesa/drivers/dri/i965/brw_state.h
index 8d886be..9afc6bb 100644
--- a/src/mesa/drive
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_state_upload.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
This series is also available on the 'state5' branch of ~kwg/mesa.
diff --git a/src/mesa/drivers/dri/i965/brw_state_upload.c
b/src/mesa/drivers/dri/i965
From: Roland Scheidegger
Use new util_fill_box helper for util_clear_render_target.
(Also fix off-by-one map error.)
---
src/gallium/auxiliary/util/u_surface.c | 39 +--
src/gallium/auxiliary/util/u_surface.h |7 +
src/gallium/drivers/llvmpipe/lp_rast.c | 54 +
On Sat, Jun 8, 2013 at 12:11 AM, Kenneth Graunke wrote:
> We already implemented this for ES3, so we just need to turn it on.
>
> Fixes 6 Piglit tests:
> spec/glsl-1.50/compiler/built-in-functions/determinant-mat[234].{vert,frag}
Reviewed-by: Matt Turner
_
Changed code checking `if(visual->haveStencilBuffer)` to
`if(visual->stencilBits > 0)`
---
src/gallium/state_trackers/dri/common/dri_screen.c | 2 +-
src/mesa/drivers/dri/nouveau/nouveau_context.c | 4 ++--
src/mesa/main/clear.c | 2 +-
src/mesa/main/context.c
Changed code checking `if(visual->haveAccumBuffer)` to `if(visual->accumRedBits
+ visual->accumGreenBits + visual->accumBlueBits + visual->accumAlphaBits) > 0)`
---
src/mesa/main/accum.c | 2 +-
src/mesa/main/clear.c | 2 +-
src/mesa/main/context.c | 2 +-
3 files changed, 3 insertions(+), 3
Changed code checking `if(visual->haveDepthBuffer)` to `if(visual->DepthBits >
0)`
---
src/gallium/state_trackers/dri/common/dri_screen.c | 2 +-
src/mesa/drivers/dri/nouveau/nouveau_context.c | 4 ++--
src/mesa/main/clear.c | 2 +-
src/mesa/main/context.c
Don't know why it was not already activated! Code seem to be common between
dri2 and gallium.
Piglit tested on r600 gallium.
---
src/gallium/state_trackers/egl/common/egl_g3d.c |1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/state_trackers/egl/common/egl_g3d.c
b/src/gallium/st
We already implemented this for ES3, so we just need to turn it on.
Fixes 6 Piglit tests:
spec/glsl-1.50/compiler/built-in-functions/determinant-mat[234].{vert,frag}
Signed-off-by: Kenneth Graunke
---
src/glsl/builtins/profiles/150.glsl | 4
1 file changed, 4 insertions(+)
diff --git a/sr
21 matches
Mail list logo
