>From the EXT_transform_feedback spec:
The error INVALID_OPERATION is also generated by BeginTransformFeedbackEXT
if no binding points would be used, either because no program object is
active or because the active program object has specified no varying
variables to record.
.
>From the EXT_transform_feedback spec:
The error INVALID_OPERATION is generated by
BeginTransformFeedbackEXT if any transform feedback buffer object
binding point used in transform feedback mode does not have a
buffer object bound.
This required adding a new NumBuffers field to th
---
src/mesa/main/shaderapi.c |2 +-
src/mesa/main/transformfeedback.c |2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/main/shaderapi.c b/src/mesa/main/shaderapi.c
index b71b44b..52a9bd4 100644
--- a/src/mesa/main/shaderapi.c
+++ b/src/mesa/main/shade
On Mon, Jan 2, 2012 at 10:59 PM, Ian Romanick wrote:
> On 01/02/2012 08:27 AM, Alex Deucher wrote:
>>
>> On Thu, Dec 22, 2011 at 9:35 AM, Vadim Girlin
>> wrote:
>>>
>>> If glUniform1i and friends are going to dump data directly in
>>> driver-allocated, the pointers have to be updated when the sto
https://bugs.freedesktop.org/show_bug.cgi?id=44407
Jos van Wolput changed:
What|Removed |Added
Status|RESOLVED|REOPENED
Resolution|WORKSFORME
On 01/02/2012 11:17 AM, Vincent Lejeune wrote:
This is similar to Gallium's existing glsl_to_tgsi::remove_output_read
lowering pass, but done entirely inside the GLSL compiler.
Signed-off-by: Vincent Lejeune
Signed-off-by: Kenneth Graunke
v2 [Kayden]:
- Don't reallocate the array for every sh
On 01/02/2012 08:27 AM, Alex Deucher wrote:
On Thu, Dec 22, 2011 at 9:35 AM, Vadim Girlin wrote:
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'
On 01/02/2012 09:18 AM, Marek Olšák wrote:
On Mon, Jan 2, 2012 at 5:56 PM, Dave Airlie wrote:
On Mon, Jan 2, 2012 at 4:40 PM, Marek Olšák wrote:
On Mon, Jan 2, 2012 at 3:31 PM, Dave Airlie wrote:
On Mon, Jan 2, 2012 at 1:07 PM, Marek Olšák wrote:
Fetching int as float and vice versa is no
On 12/23/2011 10:35 PM, Eric Anholt wrote:
On Thu, 22 Dec 2011 17:43:57 -0800, "Ian Romanick" wrote:
From: Ian Romanick
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 woul
https://bugs.freedesktop.org/show_bug.cgi?id=44407
Ian Romanick changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|
On 01/02/2012 06:18 PM, Jeremy Huddleston wrote:
xdriinfo is failing to build in tinderbox (through no change of its own). The
relevant changes in mesa were bce506f..7705833. My guess is that this was
caused by one of:
glx: Add __glX_send_client_info super function
glx: Initial implementatio
https://bugs.freedesktop.org/show_bug.cgi?id=44407
Bug #: 44407
Summary: git mesa/src/glx fails to compile
Classification: Unclassified
Product: Mesa
Version: git
Platform: x86-64 (AMD64)
OS/Version: Linux (All)
Status
xdriinfo is failing to build in tinderbox (through no change of its own). The
relevant changes in mesa were bce506f..7705833. My guess is that this was
caused by one of:
glx: Add __glX_send_client_info super function
glx: Initial implementation of glXCreateContextAttribsARB
Here's the tinderb
From: Ian Romanick
A lot of tests in 'make check' will fail under these circumstances,
but at least the build should work.
Signed-off-by: Ian Romanick
---
src/glx/create_context.c | 14 ++
src/glx/glxextensions.c |9 +
2 files changed, 23 insertions(+), 0 deletions(-
From: Ian Romanick
Signed-off-by: Ian Romanick
---
configure.ac | 28 ++--
1 files changed, 6 insertions(+), 22 deletions(-)
diff --git a/configure.ac b/configure.ac
index c29b0bb..e526d17 100644
--- a/configure.ac
+++ b/configure.ac
@@ -940,16 +940,8 @@ if test "x$en
On Mon, Jan 2, 2012 at 17:37, Eric Anholt wrote:
> - if (intel->gen == 6 || (intel->gen == 7 && override_version >= 30))
> + if (intel->gen == 6 ||
> + (intel->gen == 7 && intel->intelScreen->kernel_has_gen7_sol_reset))
> ctx->Extensions.EXT_transform_feedback = true;
>
If the ke
https://bugs.freedesktop.org/show_bug.cgi?id=43317
Brian Paul changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|
Ian,
I'm getting some warnings/errors with the latest GLX changes:
clientinfo.c: In function ‘__glX_send_client_info’:
clientinfo.c:128:7: warning: implicit declaration of function
‘xcb_glx_set_client_info_2arb’
clientinfo.c:139:7: warning: implicit declaration of function
‘xcb_glx_set_client_
On 01/02/2012 11:37 AM, Eric Anholt wrote:
> Here's my patch series for finishing off gen7 transform feedback.
> It's not quite ready because it relies on new libdrm, which relies on
> the kernel patch landing, so I won't push without requiring a new
> libdrm release.
LGTM.
Reviewed-by: Kenneth G
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 48 +---
src/mesa/state_tracker/st_program.c| 18 ++
2 files changed, 61 insertions(+), 5 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
---
src/gallium/auxiliary/tgsi/tgsi_dump.c |3 +-
src/gallium/auxiliary/tgsi/tgsi_text.c |3 +-
src/gallium/auxiliary/tgsi/tgsi_ureg.c | 38 +--
src/gallium/auxiliary/tgsi/tgsi_ureg.h |6
src/gallium/include/pipe/p_shader_tokens.h |3 +
This is the fourth revision of my changes to add support for gl_ClipDistance
with Gallium. The first three revisions were sent in closer succession about
two weeks ago. This revision is identical to v3 except for the inclusion of
the changes suggested by Brian Paul in reply to the v3 patches.
__
On Sun, Jan 1, 2012 at 5:46 PM, Christoph Bumiller
wrote:
> Ping ...
>
Reviewed-by: Dave Airlie
Dave.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
The last major issue (intervening-read) is fixed, so let's turn this
on for real. The only other known issue is a hardware limitation for
tesselation with flat shading.
---
src/mesa/drivers/dri/intel/intel_extensions.c |3 ++-
src/mesa/drivers/dri/intel/intel_screen.c |4
src/mes
Here's my patch series for finishing off gen7 transform feedback.
It's not quite ready because it relies on new libdrm, which relies on
the kernel patch landing, so I won't push without requiring a new
libdrm release.
___
mesa-dev mailing list
mesa-dev@l
We need the kernel to reset our pointers to 0 in between. Note that
the initialization of function pointer had to move to after
InitContext since we didn't have intel->gen set up yet.
Fixes piglit EXT_transform_feedback/immediate-reuse
---
src/mesa/drivers/dri/i965/brw_context.c| 13 ++
Fixes piglit EXT_transform_feedback/intervening-read
---
src/mesa/drivers/dri/i965/brw_context.h|1 +
src/mesa/drivers/dri/i965/brw_vtbl.c |6 ++
src/mesa/drivers/dri/i965/gen6_sol.c |1 +
src/mesa/drivers/dri/i965/gen7_sol_state.c | 13 ++---
4 files chan
The new kernel patch I submitted makes the interface opt-in, so all
batchbuffers aren't preceded by the 4 MI_LOAD_REGISTER_IMMs.
---
src/mesa/drivers/dri/i965/gen7_sol_state.c |2 ++
src/mesa/drivers/dri/intel/intel_batchbuffer.c | 13 +
src/mesa/drivers/dri/intel/intel_conte
The existing glsl_to_tgsi::remove_output_read pass did not work properly
when indirect addressing was involved; this commit replaces it with a
lowering pass that occurs before TGSI code generation.
Fixes varying-array related piglit tests.
Signed-off-by: Vincent Lejeune
Signed-off-by: Kenneth Gr
This is similar to Gallium's existing glsl_to_tgsi::remove_output_read
lowering pass, but done entirely inside the GLSL compiler.
Signed-off-by: Vincent Lejeune
Signed-off-by: Kenneth Graunke
v2 [Kayden]:
- Don't reallocate the array for every shader output.
- Move the class into the .cpp fil
Hi,
thank for your review, your tips about ralloc, and your fixed patches.
I spotted a tab indent instead of a 3 whitespace indent in
lower_output_read.cpp. Other than that, your patch is fine for me.
I don't know how many reviews are needed before a patch is merged so I'm
sending your patches
On 12/31/2011 09:46 AM, Dave Airlie wrote:
Hi,
So in the interest of correctness> perf wrt softpipe, flicking
tgsi_exec.c FAST_MATH to 0 fixes 15 piglit tests with 0 regressions,
I could probably be persuaded to make a SOFTPIPE_FAST_MATH run time
set to 0 by default, anyone care one way or ano
From: Dave Airlie
This fixes
draw-elements-base-vertex user_varrays
draw-elements-instanced-base-vertex user_varrays
for softpipe with no llvm support (DRAW_USE_LLVM=false)
I'm not sure if this is the correct answer, but these tests were showing
a max_index of 7, then trying to fetch up to 43, m
On Mon, Jan 2, 2012 at 5:56 PM, Dave Airlie wrote:
> On Mon, Jan 2, 2012 at 4:40 PM, Marek Olšák wrote:
>> On Mon, Jan 2, 2012 at 3:31 PM, Dave Airlie wrote:
>>> On Mon, Jan 2, 2012 at 1:07 PM, Marek Olšák wrote:
Fetching int as float and vice versa is not allowed.
Fetching unsigned i
I agree that we shouldn't be exposing EXT_texture_integer.
I think it would be cleaner to just say that if GLSL 1.3 is supported,
all the other features which are part of it are supported too. I would
simplify the code like this (see below), and most of the checks for
EXT_texture_integer should al
On Mon, Jan 2, 2012 at 4:40 PM, Marek Olšák wrote:
> On Mon, Jan 2, 2012 at 3:31 PM, Dave Airlie wrote:
>> On Mon, Jan 2, 2012 at 1:07 PM, Marek Olšák wrote:
>>> Fetching int as float and vice versa is not allowed.
>>> Fetching unsigned int as signed int and vice versa is not allowed either.
>>
On Mon, Jan 2, 2012 at 3:31 PM, Dave Airlie wrote:
> On Mon, Jan 2, 2012 at 1:07 PM, Marek Olšák wrote:
>> Fetching int as float and vice versa is not allowed.
>> Fetching unsigned int as signed int and vice versa is not allowed either.
>
> The first is true, the second isn't, otherwise I'd have
>
> EXT_texture_integer isn't strictly equivalent to GL3, the L/I/A integer
> formats are not required in GL3, and some drivers may not require them.
>
> I've no idea if we have any hw that requires it, just sending the patch along
> after discussion on irc.
Another reason was mentioned on irc, EX
From: Dave Airlie
EXT_texture_integer isn't strictly equivalent to GL3, the L/I/A integer
formats are not required in GL3, and some drivers may not require them.
I've no idea if we have any hw that requires it, just sending the patch along
after discussion on irc.
Signed-off-by: Dave Airlie
--
On Thu, Dec 22, 2011 at 9:35 AM, Vadim Girlin wrote:
> 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 t
Hey,
in the last two weeks I did some experiments with xcb, glx and mesa
gallium drivers (nouveau).
Is there any work group that takes care for this part of mesa. There
still seem to be some very basic problems using that configuration.
cheers, ingo
--
i don't do signatures
___
On Mon, Jan 2, 2012 at 1:07 PM, Marek Olšák wrote:
> Fetching int as float and vice versa is not allowed.
> Fetching unsigned int as signed int and vice versa is not allowed either.
The first is true, the second isn't, otherwise I'd have implemented
this that way.
> Doing conversions like that i
This is for GL_ARB_vertex_type_2_10_10_10_rev.
I just took the code from u_format_table.c. It's based on pack_rgba_float.
I had no other choice. The u_format hooks are not exactly compatible
with translate. The cleanup of it is left for future work.
---
.../auxiliary/translate/translate_generic.c
The conversion is limited to only a few cases, because converting to any other
type shouldn't happen in any driver.
---
.../auxiliary/translate/translate_generic.c| 346 ++--
1 files changed, 252 insertions(+), 94 deletions(-)
diff --git a/src/gallium/auxiliary/translate/
Fetching int as float and vice versa is not allowed.
Fetching unsigned int as signed int and vice versa is not allowed either.
Doing conversions like that isn't allowed in OpenGL.
The three hooks could be consolidated into one fetch hook, which would fetch
uint as uint32, sint as sint32, and every
Hi Maarten,
first of all: Happy new Year and sorry for the late reply, have been on
vacation for the last week.
On 29.12.2011 16:41, Maarten Lankhorst wrote:
Hey Christian,
Op 26-12-11 14:00, Christian König schreef:
Based on patches from Maarten Lankhorst
Signed-off-by: Christian König
d
Looks good on first sign. I'm waiting for this for quite some time now,
cause it makes XvMC state tracker implementation more cleaner and faster.
Acked-by: Christian König
On 02.01.2012 01:22, Marek Olšák wrote:
Please see the diff for further info.
This paves the way for moving user buffer
On 12/26/2011 11:51 PM, Mathias Fröhlich wrote:
>
> Hi,
>
> On Tuesday, December 27, 2011 00:17:09 Brian Paul wrote:
>> LGTM. I presume you've done a piglit run and tested with a few other
>> things?
> I have run piglit quick with classic swrast an piglit r600g on gallium rv670
> with this chan
On 12/28/2011 11:14 AM, Eric Anholt wrote:
> We didn't consume these flags in any way that would produce a
> functional difference, but we might have some day.
> ---
> src/mesa/drivers/dri/intel/intel_buffer_objects.c |4 +++-
> 1 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git
This is now handled by the GLSL compiler, so this code is dead.
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 84
1 files changed, 0 insertions(+), 84 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_tracker/st_glsl_to_tgsi.c
From: Vincent Lejeune
The existing glsl_to_tgsi::remove_output_read pass did not work properly
when indirect addressing was involved; this commit replaces it with a
lowering pass that occurs before TGSI code generation.
Fixes varying-array related piglit tests.
Signed-off-by: Vincent Lejeune
S
From: Vincent Lejeune
This is similar to Gallium's existing glsl_to_tgsi::remove_output_read
lowering pass, but done entirely inside the GLSL compiler.
Signed-off-by: Vincent Lejeune
Signed-off-by: Kenneth Graunke
---
src/glsl/Makefile.sources |1 +
src/glsl/ir_optimization.h |
On 12/31/2011 03:52 PM, Vincent Lejeune wrote:
> Current glsl_to_tgsi::remove_output_read pass did not work properly when
> indirect addressing was involved ; this commit replaces it with
> a lowering pass that occurs before glsl_to_tgsi visitor is called.
> This patch fix varying-array related pig
53 matches
Mail list logo
