https://bugs.freedesktop.org/show_bug.cgi?id=38716
--- Comment #10 from Chia-I Wu 2011-06-28 23:09:28 PDT ---
"make realclean" did not seem to remove your i386-linux-gnu (it did "rm -rf
lib*"). When building libEGL later, it failed because 32-bit
xcb-{dri2,xfixes,shape} were missing. So you sti
Commit 1a339b6c(st/mesa: prefer native texture formats when possible)
introduced two new arguments to the st_choose_format() functions.
This patch fixes the order and passes the correct internal_target
rather than GL_NONE
NOTE: This is a candidate for the 7.11 branch
Signed-off-by: Emil Velikov
-
This patch add the support for 24bpp in the dri/swrast implementation.
Signed-off-by: Marc Pignat
---
Hi all!
Here is a fix for https://bugs.freedesktop.org/show_bug.cgi?id=23525
This time it has been tested under kvm/qemu using:
* cirrus emulation (truecolor 24 bits with 24 bpp)
* vga (true
On 06/28/2011 02:48 PM, Dan McCabe wrote:
> Previously we added productions for:
> switch_body
> case_label_list
> case_statement
> case_statement_list
> Now add AST structs corresponding to those productions.
Both 1/3 and 3/3 look good. You might actually want to squash
t
On 06/28/2011 02:48 PM, Dan McCabe wrote:
> The grammar is modified to support switch statements. Rather than follow the
> grammar in the appendix, which allows case labels to be placed ANYWHERE
> as a regular statement, we follow the development of the grammar as
> described in the body of the GLS
https://bugs.freedesktop.org/show_bug.cgi?id=38716
--- Comment #9 from Alexandre Demers 2011-06-28
22:22:29 PDT ---
Created an attachment (id=48539)
--> (https://bugs.freedesktop.org/attachment.cgi?id=48539)
Full log
using:
make realclean
./autogen.sh --enable-32-bit with some other options
ma
On 06/29/2011 01:26 AM, Chia-I Wu wrote:
On Tue, Jun 28, 2011 at 9:19 PM, Thomas Hellstrom wrote:
The api and the state tracker manager code as well as the state tracker code
assumed that only a single context could be bound to a drawable. That is not
a valid assumption, since multiple cont
https://bugs.freedesktop.org/show_bug.cgi?id=38716
--- Comment #8 from Alexandre Demers 2011-06-28
21:05:19 PDT ---
Here are some different results:
-nm -D src/egl/main/eglconfig.o | grep eglFilter
nm: src/egl/main/eglconfig.o: No symbols
-nm src/egl/main/eglconfig.o | grep eglFilter
We didn't allocate them, and the driver will try to free them at the end of the
pipeline stage, so leave them alone.
---
src/gallium/auxiliary/draw/draw_pipe_aaline.c |1 -
src/gallium/auxiliary/draw/draw_pipe_aapoint.c |1 -
2 files changed, 0 insertions(+), 2 deletions(-)
diff --git a
---
src/gallium/auxiliary/draw/draw_pipe_aaline.c |8
src/gallium/auxiliary/draw/draw_pipe_aapoint.c |9 +
2 files changed, 17 insertions(+), 0 deletions(-)
diff --git a/src/gallium/auxiliary/draw/draw_pipe_aaline.c
b/src/gallium/auxiliary/draw/draw_pipe_aaline.c
index
On Tue, Jun 28, 2011 at 5:49 PM, Eric Anholt wrote:
> This error result doesn't appear in the GL 2.1 or 3.2 compatibility
> specs, and triggers an unexpected GL error in Intel's oglconform when
> it tries to reset the feedback state after usage so that the "diff the
> state at error time vs. conte
Comments at the end (where they are supposed to be :).
On 06/17/2011 05:54 PM, Dan McCabe wrote:
There are three changes from the patch set I published on 6/15:
1) Removed IR pointers from AST classes and moved them to
glsl_parser_state,
2) Manage the new IR in a stack-like manner to properly h
---
tests/all.tests|1 +
tests/spec/arb_vertex_program/CMakeLists.gl.txt|1 +
.../arb_vertex_program/getlocal4d-with-error.c | 88
3 files changed, 90 insertions(+), 0 deletions(-)
create mode 100644 tests/spec/arb_vertex_
Fixes a bug caught by oglconform, and now piglit
ARB_vertex_program/getenv4d-with-error. The wrapping of an existing
GL function made it so that we couldn't distinguish an error in
looking up our arguments from an existing error. Instead, make a
helper function to choose the param, and use it fro
Like the previous commit, but fixes
ARB_vertex_program/getlocal4d-with-error.
---
src/mesa/main/arbprogram.c | 116 +--
1 files changed, 57 insertions(+), 59 deletions(-)
diff --git a/src/mesa/main/arbprogram.c b/src/mesa/main/arbprogram.c
index cbc1455..8
---
tests/all.tests|4 +
tests/spec/CMakeLists.txt |1 +
tests/spec/arb_vertex_program/CMakeLists.gl.txt| 16
tests/spec/arb_vertex_program/CMakeLists.txt |1 +
.../spec/arb_vertex_program/getenv4d-with-error
Here's a series to fix the non-oglconform bugs revealed by our
oglconform's VP_binding.c test, and a couple of testcases I had to
make along the way.
I'm curious how others feel about the piglit tests -- would you rather
see them named by the full function name being tested, or would you
rather ha
This error result doesn't appear in the GL 2.1 or 3.2 compatibility
specs, and triggers an unexpected GL error in Intel's oglconform when
it tries to reset the feedback state after usage so that the "diff the
state at error time vs. context init time" code doesn't generate
spurious diffs. The unex
On Tue, Jun 28, 2011 at 9:19 PM, Thomas Hellstrom wrote:
> The api and the state tracker manager code as well as the state tracker code
> assumed that only a single context could be bound to a drawable. That is not
> a valid assumption, since multiple contexts can bind to the same drawable.
>
> Fi
Up until now modifying the GLSL compiler has been pretty straightforward.
This is where things get interesting. But still pretty straightforward.
Switch statements can be thought of a series of if/then/else statements.
Case labels are compared with the value of a test expression and the case
state
We now tie the grammar to the ctors of the ASTs they reference.
This requires that we actually have definitions of the ctors.
In addition, we also need to define "print" and "hir" methods for the AST
classes. The Print methods are pretty simple to flesh out. However, at this
stage of the developm
Previously we added productions for:
switch_body
case_label_list
case_statement
case_statement_list
Now add AST structs corresponding to those productions.
---
src/glsl/ast.h | 59
1 files changed, 59 insert
The grammar is modified to support switch statements. Rather than follow the
grammar in the appendix, which allows case labels to be placed ANYWHERE
as a regular statement, we follow the development of the grammar as
described in the body of the GLSL.
In this variation, the switch statement has a
Data structures for switch statement and case label are created that parallel
the structure of other AST data.
---
src/glsl/ast.h | 24
1 files changed, 20 insertions(+), 4 deletions(-)
diff --git a/src/glsl/ast.h b/src/glsl/ast.h
index 878f48b..2ee0b11 100644
--- a/src
This patch set adds support for switch statements to the GLSL compiler. We
modify the grammar for the compiler with productions for switch statements
and case labels, while adding supporting supporting productions not already
present. New AST classes are defined to support those productions. Howev
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 06/28/2011 11:47 AM, Paul Berry wrote:
Nice work.
Reviewed-by: Ian Romanick
I think we will want to cherry pick this to 7.11 and 7.10, but I want to
give it a little time to settle on master first. I don't want to
inadvertently break some appli
Am 28.06.2011 19:21, schrieb Eric Anholt:
> On Sat, 18 Jun 2011 17:43:16 -0700, Eric Anholt wrote:
>> Fixes oglconform failure about our ARB_texture_float GetTexImage.
>> Note that clamping does not appear in the "Conversion to L" stage of
>> the GL 3.2 spec, and the next stage of "Final conversio
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 06/28/2011 10:01 AM, Eric Anholt wrote:
> On Tue, 28 Jun 2011 06:49:57 -0700, Paul Berry
> wrote:
>> On 27 June 2011 18:30, Ian Romanick wrote:
>>> I like this a lot. It's a really good clean up of a rotting cesspool.
>>
>> Thanks!
>>
tha
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 06/28/2011 06:49 AM, Paul Berry wrote:
> On 27 June 2011 18:30, Ian Romanick wrote:
>> I like this a lot. It's a really good clean up of a rotting cesspool.
>
> Thanks!
>
>>> that are avaiable in geometry shaders.
>>
>> available
>
From: Ian Romanick
Fixes an assertion failure in the piglib out-01.frag
ARB_explicit_attrib_location test. The locations set via the layout
qualifier in fragment shader were not being applied to the shader
outputs. As a result all of these variables still had a location of
-1 set.
This may nee
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 06/28/2011 10:50 AM, Paul Berry wrote:
> From the OpenGL docs for GL_ARB_explicit_attrib_location:
>
> This extension provides a method to pre-assign attribute locations to
> named vertex shader inputs and color numbers to named fragment sh
On 06/28/2011 10:50 AM, Paul Berry wrote:
From the OpenGL docs for GL_ARB_explicit_attrib_location:
This extension provides a method to pre-assign attribute locations to
named vertex shader inputs and color numbers to named fragment shader
outputs.
This was accidentally implemen
Instead of using a chain of manually maintained if/else blocks to
handle "#extension" directives, we now consult a table that specifies,
for each extension, the circumstances under which it is available, and
what flags in _mesa_glsl_parse_state need to be set in order to
activate it.
This makes it
On 28 June 2011 10:01, Eric Anholt wrote:
> If I stumbled on this code, I'd have no idea what was going on. A short
> version of this description near the code might help us poor C
> developers who stumble on it that have never seen this stuff before.
Will do.
>
>> >> + case vertex_shader:
>From the OpenGL docs for GL_ARB_explicit_attrib_location:
This extension provides a method to pre-assign attribute locations to
named vertex shader inputs and color numbers to named fragment shader
outputs.
This was accidentally implemented for fragment shader inputs. This
patch fix
On 06/28/2011 09:12 AM, Paul Berry wrote:
On 27 June 2011 17:35, Keith Packard wrote:
I'd write the simplest possible code that works now and then consider
optimizing it later. This has the advantage that you can get a bunch of
tests working and then use those to validate a later, more complica
On Sat, 18 Jun 2011 17:43:16 -0700, Eric Anholt wrote:
> Fixes oglconform failure about our ARB_texture_float GetTexImage.
> Note that clamping does not appear in the "Conversion to L" stage of
> the GL 3.2 spec, and the next stage of "Final conversion" does
> clamping according to CLAMP_READ_COLO
On Mon, 27 Jun 2011 19:51:51 -0700, "Ian Romanick" wrote:
> From: Ian Romanick
>
> The set of values initially available (before any kills) must be
> tracked with each constant in the set. Otherwise the wrong component
> can be selected after earlier components have been killed.
>
> NOTE: This
On Tue, 28 Jun 2011 06:49:57 -0700, Paul Berry wrote:
> On 27 June 2011 18:30, Ian Romanick wrote:
> > I like this a lot. It's a really good clean up of a rotting cesspool.
>
> Thanks!
>
> >> that are avaiable in geometry shaders.
> >
> > available
>
> *smacks forehead* Oops.
>
On 06/28/2011 04:27 PM, Jose Fonseca wrote:
Looks good to me Thomas.
Jose
Jakob pointed out that the state trackers (vega & mesa) will actually
create a window system draw buffer / framebuffer per context, even if
they represent the same window system drawable, so I need to do some
fixu
On 27 June 2011 17:35, Keith Packard wrote:
> I'd write the simplest possible code that works now and then consider
> optimizing it later. This has the advantage that you can get a bunch of
> tests working and then use those to validate a later, more complicated,
> implementation.
I heartily supp
https://bugs.freedesktop.org/show_bug.cgi?id=38716
--- Comment #7 from Alexandre Demers 2011-06-28
09:02:33 PDT ---
I'll look at it when I'll get home tonight.
--
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You ar
Looks good to me Thomas.
Jose
- Original Message -
> The api and the state tracker manager code as well as the state
> tracker code
> assumed that only a single context could be bound to a drawable. That
> is not
> a valid assumption, since multiple contexts can bind to the same
> drawabl
On 27 June 2011 18:30, Ian Romanick wrote:
> I like this a lot. It's a really good clean up of a rotting cesspool.
Thanks!
>> that are avaiable in geometry shaders.
>
> available
*smacks forehead* Oops.
>> + /* Name of the extension when referred to in a GLSL extension
>> +
It's incorrect to assume a single context bound to a drawable.
Signed-off-by: Thomas Hellstrom
---
.../state_trackers/dri/common/dri_context.c|4
.../state_trackers/dri/common/dri_drawable.h |1 -
2 files changed, 0 insertions(+), 5 deletions(-)
diff --git a/src/galli
The api and the state tracker manager code as well as the state tracker code
assumed that only a single context could be bound to a drawable. That is not
a valid assumption, since multiple contexts can bind to the same drawable.
Fix this by making it the state tracker's responsibility to update al
https://bugs.freedesktop.org/show_bug.cgi?id=38716
--- Comment #6 from Chia-I Wu 2011-06-28 05:04:06 PDT ---
You can still use nm like
$ nm src/egl/main/eglconfig.o
Could you also attach your build log of building src/egl/main? It might shed
some light.
--
Configure bugmail: https://bugs.fr
https://bugs.freedesktop.org/show_bug.cgi?id=38716
--- Comment #5 from Alexandre Demers 2011-06-28
04:41:56 PDT ---
Obviously, after verifying, i386-linux-gnu/libEGL.so shouldn't be outdated
since I always make a "make realclean" at the beginning and the file is then
removed (checked). I underst
https://bugs.freedesktop.org/show_bug.cgi?id=38716
--- Comment #4 from Chia-I Wu 2011-06-28 04:30:59 PDT ---
Is your i386-linux-gnu/libEGL.so outdated? It should have the symbol.
(another place to check is src/egl/main/eglconfig.o where the symbol is
defined)
--
Configure bugmail: https://bug
https://bugs.freedesktop.org/show_bug.cgi?id=38716
Alexandre Demers changed:
What|Removed |Added
Status|RESOLVED|REOPENED
Resolution|FIXED
https://bugs.freedesktop.org/show_bug.cgi?id=38699
Alexandre Demers changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|
https://bugs.freedesktop.org/show_bug.cgi?id=38716
Alexandre Demers changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|
52 matches
Mail list logo
