On Tue, Feb 5, 2013 at 11:09 PM, Kenneth Graunke wrote:
> According to the OpenGL 2.1 specification, Table 6.11 (Coloring),
> GL_FOG_COORD_SRC is supposed to be in the fog attribute.
>
> NOTE: This is a candidate for stable branches.
>
> Signed-off-by: Kenneth Graunke
> ---
> src/mesa/main/attri
According to the GL 2.1 specification, Table 6.11 (Coloring),
GL_COLOR_SUM is supposed to be in the fog/enable attributes.
NOTE: This is a candidate for stable branches.
Signed-off-by: Kenneth Graunke
---
src/mesa/main/attrib.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/mesa/mai
According to the OpenGL 2.1 specification, Table 6.11 (Coloring),
GL_FOG_COORD_SRC is supposed to be in the fog attribute.
NOTE: This is a candidate for stable branches.
Signed-off-by: Kenneth Graunke
---
src/mesa/main/attrib.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/main/a
According to the GL 2.1 specification, Table 6.11 (Coloring),
GL_COLOR_SUM is supposed to be in the fog/enable attributes.
This patch just adds it to GL_FOG_BIT. GL_ENABLE_BIT is still broken.
NOTE: This is a candidate for stable branches.
Signed-off-by: Kenneth Graunke
---
src/mesa/main/attr
Fixes uninitialized pointer field defect reported by Coverity.
Signed-off-by: Vinson Lee
---
src/glsl/link_varyings.cpp | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/glsl/link_varyings.cpp b/src/glsl/link_varyings.cpp
index e2cb46e..b1317c8 100644
--- a/src/glsl/link
The BLT engine has many limitations. Currently, it can only blit
X-tiled buffers (since we don't have a kernel API to whack the BLT
tiling mode register), which means all depth/stencil operations get
punted to meta code, which can be very CPU-intensive.
Even if we used the BLT engine, it can't bl
On Wed, Feb 6, 2013 at 1:20 PM, wrote:
> From: Roland Scheidegger
>
> None of the filters used it (why would they). Maybe that param
> was just there because some of the lines were considered to be
> too short...
Looks good to me,
Reviewed-by: Dave Airlie
_
Hmm that commit message doesn't quite look right, I should have said
explicitly that it doesn't work for ordinary dx10 sampling instructions,
since the tgsi sample interface doesn't pass separate sampler and
sampler_view ids for fetch_texel neither, not just for get_dims and
get_texels (so unlike t
From: Roland Scheidegger
None of the filters used it (why would they). Maybe that param
was just there because some of the lines were considered to be
too short...
---
src/gallium/drivers/softpipe/sp_tex_sample.c | 76 +-
src/gallium/drivers/softpipe/sp_tex_sample.h |
From: Roland Scheidegger
There were several bugs how this was handled, most opcodes wouldn't even
have fetched the right arguments.
Also, the tex "target" is coming from the sampler view, hence it cannot
have information about shadow comparisons - fortunately this is not only
sampler state but al
Yes, that's much more sensible, thanks.
-- Chris
On Wed, Feb 6, 2013 at 12:00 PM, Eric Anholt wrote:
> Chris Forbes writes:
>
>> GL_TEXTURE_2D_MULTISAMPLE and GL_TEXTURE_2D_MULTISAMPLE_ARRAY targets
>> don't allow filtering state to be set, but the default state fails the
>> completeness check
I've re-read the spec and am pretty certain now that allowing
STENCIL_INDEX is bogus.
I had got confused about what was actually required where the
extension spec says:
must be color-renderable, depth-renderable, or
stencil- renderable (as defined in section 4.4.4).
-- Chris
___
When the user specifies an unsupported GLSL version,
_mesa_glsl_parse_state::process_version_directive() nicely gives them
an error message telling them which GLSL versions are supported.
Previous to this patch, the logic for determining whether a given
language version was supported was independen
Chris Forbes writes:
> GL_TEXTURE_2D_MULTISAMPLE and GL_TEXTURE_2D_MULTISAMPLE_ARRAY targets
> don't allow filtering state to be set, but the default state fails the
> completeness check for integer formats.
>
> We still care about other reasons the texture might be considered
> incomplete though
> diff --git a/src/glsl/builtins/tools/texture_builtins.py
> b/src/glsl/builtins/tools/texture_builtins.py
> index 654eb06..37fb749 100755
> --- a/src/glsl/builtins/tools/texture_builtins.py
> +++ b/src/glsl/builtins/tools/texture_builtins.py
> @@ -57,6 +57,12 @@ def get_txs_dim(sampler_type):
>
Chris Forbes writes:
> diff --git a/src/mesa/main/get.c b/src/mesa/main/get.c
> index 056bacb..3a618fc 100644
> --- a/src/mesa/main/get.c
> +++ b/src/mesa/main/get.c
> @@ -1635,6 +1635,15 @@ find_value_indexed(const char *func, GLenum pname,
> GLuint index, union value *v)
>goto invalid_e
Chris Forbes writes:
> V2: - Drop placeholder
> - Align enum values
> - Remove explicit exec=mesa; it *is* the dispatch flavor we want,
> but it's also the default. I misunderstood how this worked before;
> after actually reading the generator it makes good sense.
>
> Signed-o
Chris Forbes writes:
> - sample count must be the same on all attachments
> - fixedsamplepositions must be the same on all attachments
> (renderbuffers have fixedsamplepositions=true implicitly; only
> multisample textures can choose to have it false)
> - stencil-only formats become legal.
>
> V2
This patch is
Reviewed-by: Eric Anholt
pgpK3yuwYw8QK.pgp
Description: PGP signature
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
From: Christian König
Remove all the unused code.
Signed-off-by: Christian König
---
lib/Target/R600/AMDGPUAsmPrinter.cpp |2 +-
lib/Target/R600/MCTargetDesc/SIMCCodeEmitter.cpp | 141 +-
lib/Target/R600/SIInstrInfo.h| 12 +-
lib/Targe
From: Roland Scheidegger
Need to calculate the number of mip levels (if it would be worthwile could
store it in dynamic state).
While here, the query code also used chan 2 for the lod value.
This worked with mesa state tracker but it seems safer to use chan 0.
Still passes piglit textureSize (wit
Am Dienstag, 5. Februar 2013 schrieb Daniel Vetter:
> On Sat, Feb 02, 2013 at 09:22:55PM +0100, Martin Steigerwald wrote:
> > Am Samstag, 2. Februar 2013 schrieb Kenneth Graunke:
> > > On 02/02/2013 04:50 AM, Martin Steigerwald wrote:
> > > > Am Samstag, 2. Februar 2013 schrieb Kenneth Graunke:
> >
https://bugs.freedesktop.org/show_bug.cgi?id=59304
Matt Turner changed:
What|Removed |Added
Depends on|59332 |
--
You are receiving this mail because:
https://bugs.freedesktop.org/show_bug.cgi?id=59304
Bug 59304 depends on bug 59332, which changed state.
Bug 59332 Summary: Problems building 32bit Mesa
https://bugs.freedesktop.org/show_bug.cgi?id=59332
What|Removed |Added
--
On Tue, Feb 05, 2013 at 09:03:47PM +0100, Michel Dänzer wrote:
> From: Michel Dänzer
>
> Allows nexuiz to run with radeonsi.
>
Nice!
Reviewed-by: Tom Stellard
> Signed-off-by: Michel Dänzer
> ---
> lib/Target/R600/SIInstrInfo.cpp | 10 +-
> 1 file changed, 9 insertions(+), 1 delet
From: Michel Dänzer
Allows nexuiz to run with radeonsi.
Signed-off-by: Michel Dänzer
---
lib/Target/R600/SIInstrInfo.cpp | 10 +-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/lib/Target/R600/SIInstrInfo.cpp b/lib/Target/R600/SIInstrInfo.cpp
index 011ec50..2a6271c 1006
- Original Message -
> On 02/05/2013 06:43 AM, jfons...@vmware.com wrote:
> > From: José Fonseca
> >
> > I'd like to test Mesa OpenGL ES along side with NVIDIA libGL
> > drivers. But
> > without this change, I get a NULL pointer dereference.
> > ---
> > src/egl/drivers/dri2/egl_dri2.c |
On 02/05/2013 06:43 AM, jfons...@vmware.com wrote:
From: José Fonseca
I'd like to test Mesa OpenGL ES along side with NVIDIA libGL drivers. But
without this change, I get a NULL pointer dereference.
---
src/egl/drivers/dri2/egl_dri2.c |3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
On 4 February 2013 21:48, Chris Forbes wrote:
> This series adds the core mesa bits and i965 driver support for
> ARB_texture_multisample.
>
> Big changes from V1:
>- Generally cleaner
>- GLSL texelFetch() with a multisample sampler is converted to
> a new ir_txf_ms opcode in the GL
On 4 February 2013 21:48, Chris Forbes wrote:
> - sample count must be the same on all attachments
> - fixedsamplepositions must be the same on all attachments
> (renderbuffers have fixedsamplepositions=true implicitly; only
> multisample textures can choose to have it false)
> - stencil-only for
On 4 February 2013 21:48, Chris Forbes wrote:
> V2: - fix multiline comment style
> - stop using ASSERT_OUTSIDE_BEGIN_END_AND_FLUSH since that
> doesn't exist anymore.
>
> Signed-off-by: Chris Forbes
> ---
> src/mesa/main/enable.c | 15 +++
> src/mesa/main/get.c
https://bugs.freedesktop.org/show_bug.cgi?id=59876
--- Comment #4 from Brian Paul ---
(In reply to comment #3)
>
> BTW: The repository paths on http://www.mesa3d.org/repository.html are
> slightly wrong, its missing a trailing ".git", i.e.
> git clone git://anongit.freedesktop.org/git/mesa/mesa.
A single element in a GLX reply is contained in the header itself.
The number of elements is denoted in the "n" field of the reply, if
"n" is 1, the length of additional data is 0.
The XXX_data_length() function of xcb does not return the length of
the (optional, n>1) data but the number of element
https://bugs.freedesktop.org/show_bug.cgi?id=59967
--- Comment #14 from Tom Stellard ---
(In reply to comment #12)
> I suggest to remove this configure check since it prevents building on an
> otherwise working setup. Better risking of failing during make rather than
> during configure if the mak
https://bugs.freedesktop.org/show_bug.cgi?id=59967
Johannes Obermayr changed:
What|Removed |Added
Status|REOPENED|RESOLVED
Resolution|---
From: Michel Dänzer
18 more little piglits.
Signed-off-by: Michel Dänzer
---
src/gallium/drivers/radeonsi/si_state.c | 27 ++-
1 file changed, 18 insertions(+), 9 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state.c
b/src/gallium/drivers/radeonsi/si_sta
From: Michel Dänzer
The hardware can't do it.
Signed-off-by: Michel Dänzer
---
src/gallium/drivers/radeonsi/si_state.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state.c
b/src/gallium/drivers/radeonsi/si_state.c
index 6e50832..a5
https://bugs.freedesktop.org/show_bug.cgi?id=59876
Stefan Brüns changed:
What|Removed |Added
CC||stefan.bruens@rwth-aachen.d
https://bugs.freedesktop.org/show_bug.cgi?id=59876
--- Comment #3 from Stefan Brüns ---
(In reply to comment #2)
> Nice. Please send a git-formatted patch to mesa-dev@lists.freedesktop.org
> and CC Ian Romanick .
Done.
BTW: The repository paths on http://www.mesa3d.org/repository.html are sligh
On Die, 2013-02-05 at 15:46 +0100, Christian König wrote:
> From: Christian König
>
> With the llvm patches, fixing 14 piglit tests in total.
>
> v2: increase the const limit
>
> Signed-off-by: Christian König
> ---
> src/gallium/drivers/radeonsi/radeonsi_pipe.c |2 +-
> src/gallium/dr
On Mon, Feb 04, 2013 at 07:14:22PM +0100, Michel Dänzer wrote:
> From: Michel Dänzer
>
> 20 more little piglits with radeonsi.
>
Reviewed-by: Tom Stellard
> Signed-off-by: Michel Dänzer
> ---
> lib/Target/R600/SIInstructions.td |4
> 1 file changed, 4 insertions(+)
>
> diff --git
On Mon, Feb 04, 2013 at 05:56:32PM +0100, Christian König wrote:
> From: Christian König
>
For both patches.
Reviewed-by: Tom Stellard
> Signed-off-by: Christian König
> ---
> src/gallium/drivers/radeonsi/radeonsi_shader.c |9 +
> 1 file changed, 9 insertions(+)
>
> diff --git
From: Christian König
Signed-off-by: Christian König
---
src/gallium/drivers/radeonsi/radeonsi_shader.c |9 +
1 file changed, 9 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
b/src/gallium/drivers/radeonsi/radeonsi_shader.c
index e639566..65372b7 100644
From: Christian König
With the llvm patches, fixing 14 piglit tests in total.
v2: increase the const limit
Signed-off-by: Christian König
---
src/gallium/drivers/radeonsi/radeonsi_pipe.c |2 +-
src/gallium/drivers/radeonsi/radeonsi_shader.c |6 --
2 files changed, 1 insertion(+)
https://bugs.freedesktop.org/show_bug.cgi?id=60326
Rodrigo Vivi changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On Mon, Feb 04, 2013 at 05:54:14PM +0100, Christian König wrote:
> From: Christian König
>
> The _SGPR variants where wrong.
>
*were* wrong.
This looks good, it's nice to see so much of the c++ code disapear.
Reviewed-by: Tom Stellard
> Signed-off-by: Christian König
> ---
> lib/Target/R60
From: Christian König
The _SGPR variants where wrong.
Signed-off-by: Christian König
---
lib/Target/R600/AMDGPUCodeEmitter.h|4 -
lib/Target/R600/AMDILISelDAGToDAG.cpp | 53
lib/Target/R600/MCTargetDesc/AMDGPUMCCodeEmitter.h |4 -
lib/Target/
From: Christian König
v2: rebased on current upstream
Signed-off-by: Christian König
Reviewed-by: Tom Stellard
---
lib/Target/R600/SIInstrInfo.td | 10 ++
lib/Target/R600/SIInstructions.td | 19 ---
lib/Target/R600/SILowerLiteralConstants.cpp
On Mon, Feb 04, 2013 at 05:54:13PM +0100, Christian König wrote:
> From: Christian König
>
Reviewed-by: Tom Stellard
> Signed-off-by: Christian König
> ---
> lib/Target/R600/SIInstrInfo.td | 10 ++
> lib/Target/R600/SIInstructions.td | 19 ---
On Mon, Feb 04, 2013 at 04:04:25PM +0100, Michel Dänzer wrote:
> From: Michel Dänzer
>
> 22 more little piglits with radeonsi.
>
Reviewed-by: Tom Stellard
> Signed-off-by: Michel Dänzer
> ---
> lib/Target/R600/SIInstructions.td |4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
On Sat, Feb 02, 2013 at 07:03:03PM +0100, Vincent Lejeune wrote:
> This allows MachineInstScheduler to reorder them, and thus make scheduling
> more
> efficient.
> ---
> lib/Target/R600/R600Instructions.td | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Reviewed-by: Tom Stellard
>
On Sat, Feb 02, 2013 at 07:03:01PM +0100, Vincent Lejeune wrote:
> This fixes a couple of regressions on (probably not just) cayman
> ---
> lib/Target/R600/AMDILISelDAGToDAG.cpp | 7 +--
> 1 file changed, 5 insertions(+), 2 deletions(-)
Reviewed-by: Tom Stellard
>
> diff --git a/lib/Target
On Sat, Feb 02, 2013 at 07:03:02PM +0100, Vincent Lejeune wrote:
> ---
> lib/Target/R600/R600ISelLowering.cpp | 111
> ---
> lib/Target/R600/R600Instructions.td | 20 ++-
> lib/Target/R600/R600Intrinsics.td| 3 -
> 3 files changed, 55 insertions(+), 79
On Sat, Feb 02, 2013 at 07:03:00PM +0100, Vincent Lejeune wrote:
> ---
Hi Vincent,
Could you add a test case for this. Also, if the AMDGPUISD::MAD node
isn't being used anymore, it should be removed.
-Tom
> lib/Target/R600/AMDGPUISelLowering.cpp | 6 +++---
> lib/Target/R600/AMDILISelLowering.
https://bugs.freedesktop.org/show_bug.cgi?id=60216
--- Comment #2 from Quentin Glidic ---
(In reply to comment #1)
> Where is HAVE_WINSYS_XLIB defined? I don't think it ever is, which makes the
> existing check to use it wrong.
Oh, right, it’s only used in configure.ac but never defined code-wis
https://bugs.freedesktop.org/show_bug.cgi?id=60326
Priority: medium
Bug ID: 60326
Assignee: mesa-dev@lists.freedesktop.org
Summary: new account request
Severity: normal
Classification: Unclassified
OS: All
Reporter:
From: José Fonseca
I'd like to test Mesa OpenGL ES along side with NVIDIA libGL drivers. But
without this change, I get a NULL pointer dereference.
---
src/egl/drivers/dri2/egl_dri2.c |3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/e
https://bugs.freedesktop.org/show_bug.cgi?id=44743
--- Comment #14 from Vasily Khoruzhick ---
(In reply to comment #13)
> I tried the trace here w/ NVIDIA GL driver and the rendering here looks just
> like your screenshot.
>
> I though by artefacts you meant the countours in the balls, but I'm n
https://bugs.freedesktop.org/show_bug.cgi?id=60197
--- Comment #4 from Quentin Glidic ---
(In reply to comment #3)
> The wayland patch is totally believable. I can reproduce that and I'll
> commit it.
Thanks
> What flags do you have to build with to reproduce the problem the first
> problem fi
https://bugs.freedesktop.org/show_bug.cgi?id=44743
--- Comment #13 from José Fonseca ---
I tried the trace here w/ NVIDIA GL driver and the rendering here looks just
like your screenshot.
I though by artefacts you meant the countours in the balls, but I'm not sure
anymore. Could you describe exa
On Sat, Feb 02, 2013 at 09:22:55PM +0100, Martin Steigerwald wrote:
> Am Samstag, 2. Februar 2013 schrieb Kenneth Graunke:
> > On 02/02/2013 04:50 AM, Martin Steigerwald wrote:
> > > Am Samstag, 2. Februar 2013 schrieb Kenneth Graunke:
> > >> The old calculation was off by one in certain cases. Th
https://bugs.freedesktop.org/show_bug.cgi?id=60294
Michel Dänzer changed:
What|Removed |Added
Assignee|mesa-dev@lists.freedesktop. |dri-devel@lists.freedesktop
Mike Frysinger writes:
> The current gen_matypes logic assumes that the host compiler will produce
> information that is useful for the target compiler. Unfortunately, this
> is not the case whenever cross-compiling.
>
> When we detect that we're cross-compiling and using GCC, use the target
> c
On Mon, Feb 4, 2013 at 7:30 PM, Eric Anholt wrote:
> Jordan Justen writes:
>
>> On Fri, Feb 1, 2013 at 1:34 PM, Eric Anholt wrote:
>>> Otherwise, the stderr and stdout debug end up interleaved wrong
>>> when I pipe them to a file.
>>
>> Shouldn't we convert dlist.c to _mesa_debug instead? It doe
https://bugs.freedesktop.org/show_bug.cgi?id=44743
--- Comment #12 from Vasily Khoruzhick ---
Here's apitrace:
https://www.dropbox.com/s/bfljhocswxfqtxc/foobillardplus.1.trace.bz2
--
You are receiving this mail because:
You are the assignee for the bug.
_
https://bugs.freedesktop.org/show_bug.cgi?id=44743
--- Comment #11 from Vasily Khoruzhick ---
(In reply to comment #10)
> (In reply to comment #9)
> > Created attachment 74194 [details]
> > cpuinfo
>
> Thanks. Your processor has SSE4.1, which is what we test the most, so it
> can't be for lack o
On Mon, 2013-02-04 at 20:17 +0100, Michel Dänzer wrote:
>
> [...] how about something like this:
>
> Define the packing to be in the host byte order. However, do not define
> array formats as packed values (which makes little sense e.g. for
> *32*32*32*32 anyway) but really just as arrays. 16- o
https://bugs.freedesktop.org/show_bug.cgi?id=44743
--- Comment #10 from José Fonseca ---
(In reply to comment #9)
> Created attachment 74194 [details]
> cpuinfo
Thanks. Your processor has SSE4.1, which is what we test the most, so it can't
be for lack of processor features.
Could you please ta
V2: Works on Ivy Bridge now too, so this can be 6+.
Signed-off-by: Chris Forbes
---
src/mesa/drivers/dri/intel/intel_extensions.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/drivers/dri/intel/intel_extensions.c
b/src/mesa/drivers/dri/intel/intel_extensions.c
index 6b5f678..7a2d
69 matches
Mail list logo