brw_set_compression_control took a GLboolean as an argument, then
promptly used a switch statement to compare it with various enumeration
values. Clearly it's not actually a boolean.
Introduce a new enumeration type, enum brw_compression, and use that.
Found by converting GLboolean to bool; clan
On Sat, Oct 8, 2011 at 1:44 AM, Jose Fonseca wrote:
>
>
> - Original Message -
>> NOTE: This is a candidate for the stable branches.
>> ---
>> src/gallium/auxiliary/pipebuffer/pb_bufmgr_cache.c | 7 +++
>> 1 files changed, 7 insertions(+), 0 deletions(-)
>>
>> diff --git a/src/gall
On 10/07/2011 07:06 PM, Ian Romanick wrote:
> From: Ian Romanick
>
> Signed-off-by: Ian Romanick
> Cc: Bryan Cain
> ---
> src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 119
> +---
> 1 files changed, 2 insertions(+), 117 deletions(-)
>
Reviewed-by: Bryan Cain
_
From: Ian Romanick
Generate the program parameters list by walking the IR instead of by
walking the list of linked uniforms. This simplifies the code quite a
bit, and is probably a bit more correct. The list of linked uniforms
should really only be used by the GL API to interact with the
applic
From: Ian Romanick
Signed-off-by: Ian Romanick
Cc: Bryan Cain
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 119 +---
1 files changed, 2 insertions(+), 117 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_tracker/st_glsl_to_tgs
From: Ian Romanick
Fold _mesa_get_active_uniform into its only caller in the process.
More changes are coming soon.
Signed-off-by: Ian Romanick
---
src/mesa/main/uniform_query.cpp | 78 +++
src/mesa/main/uniforms.c| 61 +--
From: Ian Romanick
This has the same value has gl_program_parameter::DataType field.
Signed-off-by: Ian Romanick
---
src/mesa/main/uniform_query.cpp |6 --
1 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/mesa/main/uniform_query.cpp b/src/mesa/main/uniform_query.cpp
i
From: Ian Romanick
Having a few of these includes or forward declarations inside the
'extern "C"' block can cause problems later. Specifically, it
prevents C++ linkage functions from being added to ir_to_mesa.h and
makes G++ angry if 'struct foo' is seen both inside and outside an
'extern "C"'.
From: Ian Romanick
This simplificiation was enabled by the earlier refactors that
eliminated the references to the assembly shaders stored in the
gl_shader_program structure.
Signed-off-by: Ian Romanick
---
src/mesa/main/context.c | 22 +++---
src/mesa/program/prog_prin
This should be the last batch of internal clean-ups before the "real"
work. I should be able to send some cleaned up uniform rework patches
(as discuessed at XDC last month) next week.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://list
https://bugs.freedesktop.org/show_bug.cgi?id=41023
Alexandre Demers changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|
On 10/07/2011 04:45 PM, Chad Versace wrote:
> On 10/07/2011 04:23 PM, Brian Paul wrote:
>> On 10/07/2011 04:55 PM, Chad Versace wrote:
>>> If this flag is set, then _mesa_meta_begin will save/restore the state of
>>> GL_SELECT and GL_FEEDBACK render modes.
>>>
>>> Intel's futue resolve meta-ops wil
On 10/07/2011 04:23 PM, Brian Paul wrote:
> On 10/07/2011 04:55 PM, Chad Versace wrote:
>> If this flag is set, then _mesa_meta_begin will save/restore the state of
>> GL_SELECT and GL_FEEDBACK render modes.
>>
>> Intel's futue resolve meta-ops will require this, since buffer resolves
>> may occur
- Original Message -
> NOTE: This is a candidate for the stable branches.
> ---
> src/gallium/auxiliary/pipebuffer/pb_bufmgr_cache.c |7 +++
> 1 files changed, 7 insertions(+), 0 deletions(-)
>
> diff --git a/src/gallium/auxiliary/pipebuffer/pb_bufmgr_cache.c
> b/src/gallium/aux
From: Ian Romanick
The hash table needs a copy of the key that it can keep for
comparisons during searches.
Signed-off-by: Ian Romanick
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=41499
Cc: Stéphane Marchesin
Cc: Luzipher
Cc: Michał Lipski
---
src/mesa/program/hash_table.h | 18
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/glsl/linker.cpp |5 -
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/src/glsl/linker.cpp b/src/glsl/linker.cpp
index 9463f53..42075cb 100644
--- a/src/glsl/linker.cpp
+++ b/src/glsl/linker.cpp
@@ -1342,7 +1342,10 @@
On 10/07/2011 04:55 PM, Chad Versace wrote:
If this flag is set, then _mesa_meta_begin will save/restore the state of
GL_SELECT and GL_FEEDBACK render modes.
Intel's futue resolve meta-ops will require this, since buffer resolves
may occur when the GL_RENDER_MODE is GL_SELECT.
Signed-off-by: Ch
Sine i965 will soon use meta-ops to perform HiZ resolves, the meta-op stack
will exceed depth 2. I bumped it to 8 because... 8 is bigger than 2, but
not too big.
Signed-off-by: Chad Versace
---
src/mesa/drivers/common/meta.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git
If this flag is set, then _mesa_meta_begin will save/restore the state of
GL_SELECT and GL_FEEDBACK render modes.
Intel's futue resolve meta-ops will require this, since buffer resolves
may occur when the GL_RENDER_MODE is GL_SELECT.
Signed-off-by: Chad Versace
---
src/mesa/drivers/common/meta.
This is required in order for meta-ops to save/restore the GL_RENDER_MODE
state, which is implemented in the next commit.
Signed-off-by: Chad Versace
---
src/mesa/main/feedback.c |2 +-
src/mesa/main/feedback.h |3 +++
2 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/src/me
In dd_function_table, close the Doxygen group beginning with
\name Support for multiple T&L engines
---
src/mesa/main/dd.h |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/src/mesa/main/dd.h b/src/mesa/main/dd.h
index 7875564..4e017ae 100644
--- a/src/mesa/main/dd.h
+++
NOTE: This is a candidate for the stable branches.
---
src/gallium/auxiliary/pipebuffer/pb_bufmgr_cache.c |7 +++
1 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/src/gallium/auxiliary/pipebuffer/pb_bufmgr_cache.c
b/src/gallium/auxiliary/pipebuffer/pb_bufmgr_cache.c
index 58
On Fri, Oct 7, 2011 at 10:48 PM, Jose Fonseca wrote:
> Looks good overall. Comments inline.
>
> - Original Message -
>> From: Dave Airlie
>>
>> This add support for unsigned/signed integer types via adding a
>> 'pure' bit
>> in the format description table. It adds 4 new u_format get/put
Looks good overall. Comments inline.
- Original Message -
> From: Dave Airlie
>
> This add support for unsigned/signed integer types via adding a
> 'pure' bit
> in the format description table. It adds 4 new u_format get/put
> hooks,
> for get/put uint and get/put sint so that accessors
https://bugs.freedesktop.org/show_bug.cgi?id=41571
Summary: libglapi.so.0: undefined symbol: is_selinux_enabled
Product: Mesa
Version: git
Platform: All
OS/Version: All
Status: NEW
Severity: critical
Priority: mediu
From: Dave Airlie
---
src/mesa/state_tracker/st_format.c | 453 ++--
1 files changed, 386 insertions(+), 67 deletions(-)
diff --git a/src/mesa/state_tracker/st_format.c
b/src/mesa/state_tracker/st_format.c
index 6eb8a50..6e8ab94 100644
--- a/src/mesa/state_trac
From: Dave Airlie
This adds support for readpixels integer paths, it deals with the
signed/unsigned crossovers.
Signed-off-by: Dave Airlie
---
src/mesa/state_tracker/st_cb_readpixels.c | 40 +++-
1 files changed, 38 insertions(+), 2 deletions(-)
diff --git a/src/mes
From: Dave Airlie
This add support for unsigned/signed integer types via adding a 'pure' bit
in the format description table. It adds 4 new u_format get/put hooks,
for get/put uint and get/put sint so that accessors can get native access
to the integer bits. This is used to avoid precision loss v
From: Dave Airlie
This fixes up the integer format choosing to pick the closest mesa format
then the most likely fallback.
(the formatting in this file needs cleaning in another patch).
Signed-off-by: Dave Airlie
---
src/mesa/main/texformat.c | 204 +--
From: Dave Airlie
This just adds a simple packing for GL_UNSIGNED_INT/GL_INT destination formats.
This is enough for at least the gallium drivers to pack both unsigned and
signed types for read pixels.
Signed-off-by: Dave Airlie
---
src/mesa/main/pack.c | 63
These are 5 more patches in the integer support set, the first just
adds a pack int routine, the second fixes up the texture format picking.
The 3rd adds all the gallium formats I've needed so far, we may require
R8G8B8X8 formats as well, I need to write more tests. The last two
patches add state t
When updating a register reference to reflect the fact that we were
taking its absolute value, the fragment shader back-end failed to
clear the negate flag, resulting in abs(-x) getting computed as
-abs(x).
I also found (and fixed) a similar problem in brw_eu.h, but I'm not
aware of an actual mani
On 10/07/2011 11:20 AM, Stéphane Marchesin wrote:
2011/10/7 Ian Romanick:
On 10/06/2011 06:54 PM, Stéphane Marchesin wrote:
Hi Ian,
This regresses Chrome GPU acceleration for all GPUs (I tested i915g,
llvmpipe, i965).
See also bugzilla #41499 and #41508. I tried to debug this a little
yest
2011/10/7 Ian Romanick :
> On 10/06/2011 06:54 PM, Stéphane Marchesin wrote:
>>
>> Hi Ian,
>>
>> This regresses Chrome GPU acceleration for all GPUs (I tested i915g,
>> llvmpipe, i965).
>
> See also bugzilla #41499 and #41508. I tried to debug this a little
> yesterday, but I couldn't see what was
On 10/06/2011 06:54 PM, Stéphane Marchesin wrote:
Hi Ian,
This regresses Chrome GPU acceleration for all GPUs (I tested i915g,
llvmpipe, i965).
See also bugzilla #41499 and #41508. I tried to debug this a little
yesterday, but I couldn't see what was going wrong. None of our piglit
or GLES
I think ARL is allowed to have no destination register, right? In that
case, there should be a special case not to eliminate ARLs.
Marek
On Fri, Oct 7, 2011 at 5:40 PM, Brian Paul wrote:
> From: Brian Paul
>
> This fixes a bug where we'd wind up emitting an invalid instruction like
> MOVE R[0].
---
src/egl/main/egldisplay.c |4
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/src/egl/main/egldisplay.c b/src/egl/main/egldisplay.c
index 39ba883..4960800 100644
--- a/src/egl/main/egldisplay.c
+++ b/src/egl/main/egldisplay.c
@@ -146,7 +146,11 @@ _eglNativePlatformDetec
For example, GL_TRIANLGES is converted to _3DPRIM_TRILIST.
The conversion is necessary because HiZ and MSAA resolve operations emit
a 3DPRIM_RECTLIST, which cannot be conveyed by GLenum.
As a consequence, brw_gs_prog_key.primitive is also converted.
v2
- [anholt] Split brw_set_prim into brw
The "slight optimization to avoid the GS program" in brw_set_prim() is not
used by Gen 6, since Gen 6 doesn't use a GS program. Also, Gen 6 doesn't use
reduced primitives.
Also, document that intel_context.reduced_primitive is only used for Gen < 6
Signed-off-by: Chad Versace
---
src/mesa/drive
No problem, Chad. I'll let you take care of it completely.
I usually run debug builds but some (useful) warnings only pop up in
optimized builds so I was just trying to clean those up. Maybe you
can make an optimized build when you're done to check for warnings.
Thanks.
-Brian
On 10/07/20
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Brian,
I'm sorry to trouble you about this patch again. These assertions are for
non-obvious assumptions in the Intel drivers that come into play
only when an experimental hardware feature (HiZ) is enabled. I'd like to
comb the i915 driver later today
https://bugs.freedesktop.org/show_bug.cgi?id=41441
Brian Paul changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|
On 10/07/2011 12:10 PM, Kevin H. Hobbs wrote:
> On 10/07/2011 10:53 AM, Paul Gotzel wrote:
>> Brain,
>>
>> I'm using osmesa with VTK and in this particular test I'm rendering
>> just lines. The ratios stay the same for just triangles. Just for
>> these simple primitives 7.11 seems to be about 4 t
From: Brian Paul
This previously generated unused variable warnings in non-debug builds.
---
src/mesa/drivers/dri/i915/i830_vtbl.c |3 +--
src/mesa/drivers/dri/i915/i915_vtbl.c |3 +--
2 files changed, 2 insertions(+), 4 deletions(-)
diff --git a/src/mesa/drivers/dri/i915/i830_vtbl.c
b
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
For patches 1, 2, 4, 5:
Reviewed-by: Chad Versace
Comments on patch 3.
- --
Chad Versace
c...@chad-versace.us
On 10/07/2011 07:30 AM, Brian Paul wrote:
> From: Brian Paul
>
> ---
> src/mesa/drivers/dri/intel/intel_decode.c |2 +-
> 1 files
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 10/07/2011 07:30 AM, Brian Paul wrote:
> From: Brian Paul
>
> ---
> src/mesa/drivers/dri/i915/i830_vtbl.c |2 ++
> src/mesa/drivers/dri/i915/i915_vtbl.c |2 ++
> 2 files changed, 4 insertions(+), 0 deletions(-)
>
> diff --git a/src/mesa/
On 10/07/2011 10:53 AM, Paul Gotzel wrote:
> Brain,
>
> I'm using osmesa with VTK and in this particular test I'm rendering
> just lines. The ratios stay the same for just triangles. Just for
> these simple primitives 7.11 seems to be about 4 times slower that
> 7.6.1.
These builds on the VTK d
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 10/07/2011 07:21 AM, Brian Paul wrote:
> From: Brian Paul
>
> Fixes https://bugs.freedesktop.org/show_bug.cgi?id=32458
> ---
> src/mesa/main/formats.c |8
> src/mesa/main/mipmap.c | 38 +-
> 2 f
From: Brian Paul
This fixes a bug where we'd wind up emitting an invalid instruction like
MOVE R[0]., R[1]; - note the empty/zero writemask. If we don't write to
any dest register channels, cull the instruction.
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp |8 +++-
1 files changed, 7
Brain,
I'm using osmesa with VTK and in this particular test I'm rendering
just lines. The ratios stay the same for just triangles. Just for
these simple primitives 7.11 seems to be about 4 times slower that
7.6.1.
Can you give me some pointers on where to get started with a new
gallium based o
From: Brian Paul
Silences a warning about comparing to an unsigned variable. It looks like
the result of swizzle_for_size() is always assigned to unsigned vars.
---
src/mesa/drivers/dri/i965/brw_vec4.h |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/
From: Brian Paul
---
src/mesa/drivers/dri/i965/brw_vec4.h |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_vec4.h
b/src/mesa/drivers/dri/i965/brw_vec4.h
index 5f44268..1caf1a6 100644
--- a/src/mesa/drivers/dri/i965/brw_vec4.h
+++ b/src/mesa
From: Brian Paul
---
src/mesa/drivers/dri/i915/i830_vtbl.c |2 ++
src/mesa/drivers/dri/i915/i915_vtbl.c |2 ++
2 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/src/mesa/drivers/dri/i915/i830_vtbl.c
b/src/mesa/drivers/dri/i915/i830_vtbl.c
index 7810f56..d8f9634 100644
--- a
From: Brian Paul
---
src/mesa/drivers/dri/i965/brw_fs.cpp |1 +
src/mesa/drivers/dri/i965/brw_wm.c |1 +
2 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
b/src/mesa/drivers/dri/i965/brw_fs.cpp
index 2000180..e073eaa 100644
--- a/src/m
From: Brian Paul
---
src/mesa/drivers/dri/intel/intel_decode.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/mesa/drivers/dri/intel/intel_decode.c
b/src/mesa/drivers/dri/intel/intel_decode.c
index ac8d690..b4de42d 100644
--- a/src/mesa/drivers/dri/intel/intel_dec
From: Brian Paul
Fixes https://bugs.freedesktop.org/show_bug.cgi?id=32458
---
src/mesa/main/formats.c |8
src/mesa/main/mipmap.c | 38 +-
2 files changed, 41 insertions(+), 5 deletions(-)
diff --git a/src/mesa/main/formats.c b/src/mesa/main/fo
Hi,
Recently I've been working on the shader optimization for r600g, and now
I have the initial working implementation of simple alu scheduler and
register allocator. It has no piglit regressions, though it's still a
work in progress and there are known issues with some applications.
I've pushed
On Fri, Oct 7, 2011 at 10:38 AM, Christoph Bumiller
wrote:
>
> Could we please keep GPU_FINISHED, it coincides with the D3D EVENT query
> (it doesn't require a call to begin_query, only to end).
>
> And no it's not more natural to how it's done by hardware, nv50+ fences
> are also just queries.
>
On 07.10.2011 04:04, Marek Olšák wrote:
> On Fri, Oct 7, 2011 at 3:21 AM, Zack Rusin wrote:
>> On Thursday, October 06, 2011 04:58:45 PM Marek Olšák wrote:
>>> I am cc'ing Zack, because he was the one to design the first interface.
>> Hi Marek.
>>
>> I'm swamped right now and won't have time to re
59 matches
Mail list logo
