Hey Alexander,
On 12/21/2011 07:16 PM, Alexander von Gluck wrote:
* Doesn't reintroduce legacy drivers
* Adds Haiku mklib code
* Removes some broken PIPE_OS_HAIKU defines
* Removes an NDEBUG ifdef in link_uniforms.cpp,
there is an item that uses the union without
checking NDEBUG below.
For the case that index data is stored in element array buffer object,
and user called glMultiDrawElements, count the min/max_index before
calling vbo-draw_prims. vbo_get_minmax_index() isn't friendly to this
case. So do it while building the prim info.
Signed-off-by: Yuanhan Liu
Hi Maarten,
On 21.12.2011 21:52, Maarten Lankhorst wrote:
It would be nice if you inlined patches for easier reviewing. :)
Well I can try, but I can't promise that Thunderbird isn't badly fucking
up all whitespaces, newest version of the patch is in-lined below.
I'm spotting an overflow
If glUniform1i and friends are going to dump data directly in
driver-allocated, the pointers have to be updated when the storage
moves. This should fix the regressions seen with commit 7199096.
I'm not sure if this is the only place that needs this treatment. I'm
a little uncertain about the
From: Brian Paul bri...@vmware.com
The code for storing 1D, 2D and 3D tex images (whole or sub-images) was
all pretty similar. This consolidates those six paths.
v2: rework switch statement to catch unexpected targets
---
src/mesa/main/texstore.c | 484
Looks good to me.
Jose
- Original Message -
From: Brian Paul bri...@vmware.com
The code for storing 1D, 2D and 3D tex images (whole or sub-images)
was
all pretty similar. This consolidates those six paths.
v2: rework switch statement to catch unexpected targets
---
On Wed, Dec 21, 2011 at 12:58 PM, Eric Anholt e...@anholt.net wrote:
- temp = malloc(width * MAX_PIXEL_BYTES);
- if (!temp) {
- _mesa_error(ctx, GL_OUT_OF_MEMORY, glCopyPixels);
- return GL_FALSE;
+ /* different src/dst buffers */
+ ctx-Driver.MapRenderbuffer(ctx,
On Wed, Dec 21, 2011 at 1:01 PM, Eric Anholt e...@anholt.net wrote:
On Sun, 18 Dec 2011 20:08:13 -0700, Brian Paul bri...@vmware.com wrote:
Use format pack/unpack functions instead of deprecated renderbuffer
GetRow/PutRow functions.
---
src/mesa/swrast/s_stencil.c | 31
Hi list,
Multiple driver support in EGL is hard to get right, if not impossible.
On Linux desktop, we almost always want to use egl_dri2. It allows EGL
to loads DRI2 drivers, thus allowing it to share DRI2 drivers with
libGL.
In one case where the app wants to use OpenVG, libEGL needs to load
On Wed, Dec 21, 2011 at 1:30 PM, Eric Anholt e...@anholt.net wrote:
On Sun, 18 Dec 2011 20:08:21 -0700, Brian Paul bri...@vmware.com wrote:
Stop using deprecated renderbuffer PutRow() function. Note that we
aren't using Map/UnmapRenderbuffer() yet because this call is inside
a
On Wed, Dec 21, 2011 at 2:39 PM, Paul Berry stereotype...@gmail.com wrote:
During meta-operations (such as _mesa_meta_GenerateMipmap()), we need
to be able to draw even if GL_RASTERIZER_DISCARD is enabled. This
patch causes _mesa_meta_begin() to save the state of
GL_RASTERIZER_DISCARD and
On Wed, Dec 21, 2011 at 2:39 PM, Paul Berry stereotype...@gmail.com wrote:
If a client calls BeginTransformFeedback(), then
PauseTransformFeedback(), then EndTransformFeedback(), we need to make
sure that the transform feedback object is not left in a paused
state, otherwise the next call to
On Wed, Dec 21, 2011 at 10:36 AM, Marek Olšák mar...@gmail.com wrote:
Hi Brian,
Is there a reason to set _NEW_ARRAY when transitioning between
DrawArrays and DrawElements?
Probably not, actually. I was just being overly paranoid. I can fix that.
-Brian
On Thu, 22 Dec 2011 09:31:59 -0700, Brian Paul brian.e.p...@gmail.com wrote:
On Wed, Dec 21, 2011 at 1:01 PM, Eric Anholt e...@anholt.net wrote:
On Sun, 18 Dec 2011 20:08:13 -0700, Brian Paul bri...@vmware.com wrote:
if ((stencilMask stencilMax) != stencilMax) {
/* need to apply
On Thu, 22 Dec 2011 09:49:33 -0700, Brian Paul brian.e.p...@gmail.com wrote:
On Wed, Dec 21, 2011 at 1:20 PM, Eric Anholt e...@anholt.net wrote:
On Sun, 18 Dec 2011 20:08:19 -0700, Brian Paul bri...@vmware.com wrote:
The functions that read depth/stencil values understand all (packed)
On Wed, Dec 21, 2011 at 2:16 PM, Eric Anholt e...@anholt.net wrote:
On Sun, 18 Dec 2011 20:08:26 -0700, Brian Paul bri...@vmware.com wrote:
The only consequence is we can only use the occlusion_zless_16_triangle()
function with MESA_FORMAT_Z16.
I'm not following that conclusion, probably due
On Wed, Dec 21, 2011 at 7:34 PM, Chad Versace
chad.vers...@linux.intel.com wrote:
If the meta flag MESA_META_TEXTURE is present, then disable the texture
target GL_TEXTURE_EXTERNAL_OES.
Signed-off-by: Chad Versace chad.vers...@linux.intel.com
---
src/mesa/drivers/common/meta.c | 2 ++
1
From: Brian Paul bri...@vmware.com
The functions that read depth/stencil values understand all (packed)
depth/stencil buffer formats now so there's no reason to use the
wrappers.
Also, improve the format checks in fast_copy_pixels() to catch mismatched
depth/stencil cases.
v2: fix the test for
From: Brian Paul bri...@vmware.com
Stop using deprecated renderbuffer PutRow() function. Note that we
aren't using Map/UnmapRenderbuffer() yet because this call is inside
a swrast_render_start/finish() pair.
v2: use _mesa_pack_uint_24_8_depth_stencil_row(), per Eric.
---
On Thu, 22 Dec 2011 10:18:05 +0800, Yuanhan Liu yuanhan@linux.intel.com
wrote:
On Wed, Dec 21, 2011 at 05:57:35AM -0800, Eric Anholt wrote:
On Wed, 21 Dec 2011 17:33:41 +0800, Yuanhan Liu
yuanhan@linux.intel.com wrote:
If dynamic instruction store size is enabled, while after the
On Wed, 21 Dec 2011 16:36:45 -0800, Kenneth Graunke kenn...@whitecape.org
wrote:
X8 depth formats weren't supported until Ironlake (Gen 5).
Fixes GPU hangs introduced in d84a180417d1eabd680554970f1eaaa93abcd41e.
One example test case was fbo-missing-attachment-blit from.
Signed-off-by:
On Wed, 21 Dec 2011 13:39:05 -0800, Paul Berry stereotype...@gmail.com wrote:
This patch series ensures that meta-ops (such as glClear or
glGenerateMipmapEXT) function properly when transform feedback or
rasterizer discard is enabled.
Most of the code changes necessary to make this work are
On 12/22/2011 09:01 AM, Chia-I Wu wrote:
Hi list,
Multiple driver support in EGL is hard to get right, if not impossible.
On Linux desktop, we almost always want to use egl_dri2. It allows EGL
to loads DRI2 drivers, thus allowing it to share DRI2 drivers with
libGL.
In one case where
On 12/21/2011 01:39 PM, Paul Berry wrote:
This patch series ensures that meta-ops (such as glClear or
glGenerateMipmapEXT) function properly when transform feedback or
rasterizer discard is enabled.
Most of the code changes necessary to make this work are in core mesa:
patches 1/5 and 5/5
On 12/21/2011 01:39 PM, Paul Berry wrote:
Although i965 gen6 does not yet support ARB_transform_feedback2 or
NV_transform_feedback2, it needs to support pause/resume functionality
so that meta-ops will work correctly.
---
src/mesa/drivers/dri/i965/brw_draw.c |3 ++-
On 12/21/2011 01:33 AM, Yuanhan Liu wrote:
[snip]
+ int emit_endif = 1;
Please use bool and true/false rather than int.
/* In single program flow mode, we can express IF and ELSE instructions
* equivalently as ADD instructions that operate on IP. On platforms
prior
@@ -1219,14
Hi all
This fixes a memory leak of 32 bytes on exit.
From 924f8fdccb41b011f372bc57252005bcdb096105 Mon Sep 17 00:00:00 2001
From: Lauri Kasanen cur...@operamail.com
Date: Thu, 22 Dec 2011 21:28:33 +0200
Subject: [PATCH] gallivm: Close a memory leak
As reported by valgrind --leak-check=full
The error was erroneously removed in this commit:
719909698c67c287a393d2380278e7b7495ae018
mesa: Rewrite the way uniforms are tracked and handled
You also aren't even supposed to truncate the output to 'bufSize',
so just return like before.
Also fixup an old comment and add an assert.
---
(This
The surface states tracked by BRW_NEW_WM_SURFACES are no longer used
for just WM. They are also used for vertex texturing and transform
feedback. To avoid confusion, this patch renames BRW_NEW_WM_SURFACES
to BRW_NEW_SURFACES.
---
src/mesa/drivers/dri/i965/brw_context.h |4 ++--
After creating new binding table entries for transform feedback, we
need to set the dirty flag BRW_NEW_SURFACES, so that a new binding
table pointer will be sent to the hardware. Otherwise the new binding
table entries will not take effect.
---
src/mesa/drivers/dri/i965/gen6_sol.c |2 ++
1
Although there is not much documentation of this fact, there are in
fact two separate VF caches:
- an index-based cache (described in the Sandy Bridge PRM, vol 2
part 1, section 2.1.2 Vertex Cache). This cache stores URB
handles of vertex shader outputs; its purpose is to avoid redundant
On 12/21/2011 01:33 AM, Yuanhan Liu wrote:
Here is the final patch to enable dynamic eu instruction store size:
increase the brw eu instruction store size dynamically instead of just
allocating it statically with a constant limit. This would fix something
that 'GL_MAX_PROGRAM_INSTRUCTIONS_ARB
On 12/21/2011 01:33 AM, Yuanhan Liu wrote:
Hi, this is a new series of patches for dynamic eu instruction store
size. The first 4 is from Eric. I just grabed it to make it rebase to
current repo. The last 4 patch is from mine which some are based on
those patches from Eric.
Please help to
On 12/22/2011 02:06 PM, Paul Berry wrote:
Although there is not much documentation of this fact, there are in
fact two separate VF caches:
- an index-based cache (described in the Sandy Bridge PRM, vol 2
part 1, section 2.1.2 Vertex Cache). This cache stores URB
handles of vertex
From: Alexander von Gluck IV kallis...@unixzen.com
* Add Haiku as a platform to mklib
* Fix GLU to allow building static libGLU
* Remove a few existing Haiku defines that break the build
---
Makefile |1 +
acinclude.m4 |2 +-
bin/mklib
Commited. Thanks.
Jose
- Original Message -
Hi all
This fixes a memory leak of 32 bytes on exit.
From 924f8fdccb41b011f372bc57252005bcdb096105 Mon Sep 17 00:00:00
2001
From: Lauri Kasanen cur...@operamail.com
Date: Thu, 22 Dec 2011 21:28:33 +0200
Subject: [PATCH] gallivm:
Here's today's patch series for gen7 transform feedback. It runs on
top of a kernel patch at people.freedesktop.org:~anholt/linux on the
gen7-reset-sol branch. I expected it to be easy, but not this easy.
Remaining test failures:
tessellation polygon flat_lastwarn
We'll be growing more code in here as we actually enable the unit.
Reviewed-by: Kenneth Graunke kenn...@whitecape.org
---
src/mesa/drivers/dri/i965/Makefile.sources |1 +
src/mesa/drivers/dri/i965/brw_state_upload.c |1 +
src/mesa/drivers/dri/i965/gen7_disable.c |7 ---
The code was relying on gs.prog_data's copy of the
number-of-verts-per-prim, which segfaulted on gen7 since it doesn't
make a GS program. We can easily calculate that value right here.
---
src/mesa/drivers/dri/i965/brw_draw.c | 33 +++--
1 files changed, 27
Reviewed-by: Kenneth Graunke kenn...@whitecape.org
---
src/mesa/drivers/dri/i965/brw_defines.h | 76 ++-
src/mesa/drivers/dri/intel/intel_reg.h | 15 ++
2 files changed, 89 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_defines.h
Reviewed-by: Kenneth Graunke kenn...@whitecape.org
---
src/mesa/drivers/dri/intel/intel_extensions.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/mesa/drivers/dri/intel/intel_extensions.c
b/src/mesa/drivers/dri/intel/intel_extensions.c
index 7ab5d90..09ee9ba
Fixes the piglit discard-* tests.
Reviewed-by: Kenneth Graunke kenn...@whitecape.org
---
src/mesa/drivers/dri/i965/gen7_sol_state.c |8 +++-
1 files changed, 7 insertions(+), 1 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/gen7_sol_state.c
Fixes almost all of the transform feedback piglit tests. Remaining
are a few tests related to tesselation for
quads/trifans/tristrips/polygons with flat shading.
---
src/mesa/drivers/dri/i965/gen7_sol_state.c | 199 ++-
1 files changed, 191 insertions(+), 8 deletions(-)
Fixes piglit tesselation triangle_strip flat_last.
---
src/mesa/drivers/dri/i965/gen7_sol_state.c |5 +
1 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/gen7_sol_state.c
b/src/mesa/drivers/dri/i965/gen7_sol_state.c
index a5e28b6..93ca868 100644
---
On Thu, Dec 22, 2011 at 02:37:58PM -0800, Kenneth Graunke wrote:
On 12/21/2011 01:33 AM, Yuanhan Liu wrote:
Hi, this is a new series of patches for dynamic eu instruction store
size. The first 4 is from Eric. I just grabed it to make it rebase to
current repo. The last 4 patch is from mine
On 22 December 2011 16:54, Eric Anholt e...@anholt.net wrote:
The code was relying on gs.prog_data's copy of the
number-of-verts-per-prim, which segfaulted on gen7 since it doesn't
make a GS program. We can easily calculate that value right here.
---
src/mesa/drivers/dri/i965/brw_draw.c |
On 12/22/2011 04:54 PM, Eric Anholt wrote:
The code was relying on gs.prog_data's copy of the
number-of-verts-per-prim, which segfaulted on gen7 since it doesn't
make a GS program. We can easily calculate that value right here.
---
src/mesa/drivers/dri/i965/brw_draw.c | 33
From: Ian Romanick ian.d.roman...@intel.com
There are a couple scenarios where the source could be zero and the
operand could be either SRC_ALPHA or ONE_MINUS_SRC_ALPHA. For
example, if the source was ZERO. This would result in something like
(0).w, and a later call to ir_validate would get
On 22 December 2011 16:54, Eric Anholt e...@anholt.net wrote:
Fixes the piglit discard-* tests.
Reviewed-by: Kenneth Graunke kenn...@whitecape.org
---
src/mesa/drivers/dri/i965/gen7_sol_state.c |8 +++-
1 files changed, 7 insertions(+), 1 deletions(-)
diff --git
On Thu, Dec 22, 2011 at 02:33:03PM -0800, Kenneth Graunke wrote:
On 12/21/2011 01:33 AM, Yuanhan Liu wrote:
Here is the final patch to enable dynamic eu instruction store size:
increase the brw eu instruction store size dynamically instead of just
allocating it statically with a constant
On Thu, Dec 22, 2011 at 11:09:12AM -0800, Kenneth Graunke wrote:
On 12/21/2011 01:33 AM, Yuanhan Liu wrote:
[snip]
+ int emit_endif = 1;
Please use bool and true/false rather than int.
Yes, right. Will fix it.
/* In single program flow mode, we can express IF and ELSE
On 22 December 2011 16:54, Eric Anholt e...@anholt.net wrote:
Reviewed-by: Kenneth Graunke kenn...@whitecape.org
---
src/mesa/drivers/dri/i965/brw_defines.h | 76
++-
src/mesa/drivers/dri/intel/intel_reg.h | 15 ++
2 files changed, 89 insertions(+), 2
On 22 December 2011 16:54, Eric Anholt e...@anholt.net wrote:
Fixes almost all of the transform feedback piglit tests. Remaining
are a few tests related to tesselation for
quads/trifans/tristrips/polygons with flat shading.
---
src/mesa/drivers/dri/i965/gen7_sol_state.c | 199
On 22 December 2011 16:54, Eric Anholt e...@anholt.net wrote:
Here's today's patch series for gen7 transform feedback. It runs on
top of a kernel patch at people.freedesktop.org:~anholt/linux on the
gen7-reset-sol branch. I expected it to be easy, but not this easy.
This is fantastic,
On 12/22/2011 06:22 PM, Paul Berry wrote:
On 22 December 2011 16:54, Eric Anholt e...@anholt.net
mailto:e...@anholt.net wrote:
Fixes the piglit discard-* tests.
Reviewed-by: Kenneth Graunke kenn...@whitecape.org
mailto:kenn...@whitecape.org
---
On Fri, Dec 23, 2011 at 4:22 AM, Paul Berry stereotype...@gmail.com wrote:
FYI, this assertion should hold true until we implement
ARB_transfrom_feedback3 (which allows holes in the transform feedback
structure). I think Marek has some plans to implement that for Gallium (not
sure of his
On 12/22/2011 04:54 PM, Eric Anholt wrote:
Fixes almost all of the transform feedback piglit tests. Remaining
are a few tests related to tesselation for
quads/trifans/tristrips/polygons with flat shading.
---
src/mesa/drivers/dri/i965/gen7_sol_state.c | 199 ++-
1
On 12/22/2011 07:04 PM, Yuanhan Liu wrote:
On Thu, Dec 22, 2011 at 02:33:03PM -0800, Kenneth Graunke wrote:
On 12/21/2011 01:33 AM, Yuanhan Liu wrote:
[snip]
-#define BRW_EU_MAX_INSN_STACK 5
-#define BRW_EU_MAX_INSN 1
+#define BRW_EU_MAX_INSN_STACK 5
+#define BRW_EU_MAX_INSN
On Thu, Dec 22, 2011 at 07:51:46PM -0800, Kenneth Graunke wrote:
On 12/22/2011 07:04 PM, Yuanhan Liu wrote:
On Thu, Dec 22, 2011 at 02:33:03PM -0800, Kenneth Graunke wrote:
On 12/21/2011 01:33 AM, Yuanhan Liu wrote:
[snip]
-#define BRW_EU_MAX_INSN_STACK 5
-#define BRW_EU_MAX_INSN 1
Hi,
On Thursday, December 22, 2011 18:30:44 Brian Paul wrote:
I'm not sure if playback_vertex_list is more like DRAW_BEGIN_END or
DRAW_ARRAYS.
Maybe add a DRAW_DISPLAY_LIST enum value?
It's more like begin/end I think.
The begin/end code just sets the array state below the state tracking of
60 matches
Mail list logo