On 01/08/2013 02:27 PM, Paul Berry wrote:
[snip]
diff --git a/src/mesa/drivers/dri/intel/intel_mipmap_tree.h
b/src/mesa/drivers/dri/intel/intel_mipmap_tree.h
index 8e84bef..195fa88 100644
--- a/src/mesa/drivers/dri/intel/intel_mipmap_tree.h
+++ b/src/mesa/drivers/dri/intel/intel_mipmap_tree.h
@@
Kenneth Graunke writes:
> According to bug #54524, I regressed oglconform's multicontext test
> when I reenabled the fragment shader precompile.
Reviewed-by: Eric Anholt
pgpoJefVy5P9D.pgp
Description: PGP signature
___
mesa-dev mailing list
mesa-dev
To be squashed in with the previous patch. Hopefully I'm doing this
right.
---
src/mesa/drivers/dri/i965/Makefile.am | 2 ++
1 file changed, 2 insertions(+)
I didn't need this to build, but Matt tells me it's probably needed.
diff --git a/src/mesa/drivers/dri/i965/Makefile.am
b/src/mesa/driver
According to bug #54524, I regressed oglconform's multicontext test
when I reenabled the fragment shader precompile.
However, these test cases only passed by miraculous coincedence. We
assign each fragment program a unique ID (brw_fragment_program::id which
becomes brw_wm_prog_key::program_string
On Fri, Dec 14, 2012 at 03:28:13PM +0200, Abdiel Janulgue wrote:
> Add create image from texture extension.
>
> Signed-off-by: Abdiel Janulgue
> ---
> include/GL/internal/dri_interface.h | 12 +
> src/egl/drivers/dri2/egl_dri2.c | 83
> +++
> 2 files
For the series:
Reviewed-by: Chris Forbes
On Wed, Jan 9, 2013 at 11:26 AM, Paul Berry wrote:
> In the process of reviewing Chris Forbes' patches to implement
> ARB_texture_multisample, we discovered considerable confusion in the
> code that refers to surface dimensions: some functions expected
In most cases, the width, height, and depth of the physical surface
used by the driver to implement a texture or renderbuffer is equal to
the logical width, height, and depth exposed to the client through
functions such as glTexImage3D(). However, there are two exceptions:
cube maps (which have a
This allows intel_miptree_alloc_mcs() to force Y tiling for the MCS
buffer. Previously we accomplished this by the hack of passing
INTEL_MSAA_LAYOUT_CMS as the msaa_layout parameter, but that parameter
is going to be going away soon.
---
src/mesa/drivers/dri/intel/intel_fbo.c | 3 +-
sr
No functional change. This patch moves the compute_msaa_layout()
function earlier in intel_mipmap_tree.c so that it can be used by
other functions in that file.
---
src/mesa/drivers/dri/intel/intel_mipmap_tree.c | 79 +-
1 file changed, 41 insertions(+), 38 deletions(-)
d
In the process of reviewing Chris Forbes' patches to implement
ARB_texture_multisample, we discovered considerable confusion in the
code that refers to surface dimensions: some functions expected
logical surface dimensions (in units of pixels, as used by GL API
functions such as glTexImage3D()), an
On Mon, Jan 7, 2013 at 12:32 AM, Anuj Phogat wrote:
> On Sat, Jan 5, 2013 at 8:11 AM, Jordan Justen
> wrote:
>> Changes based on GTF/gles3 conformance test suite.
>>
>> Signed-off-by: Jordan Justen
>> ---
>> src/mesa/main/teximage.c | 62
>> +-
>>
---
src/gallium/drivers/r600/r600_llvm.c | 24 +-
src/gallium/drivers/r600/r600_shader.c | 7 +--
.../drivers/radeon/radeon_setup_tgsi_llvm.c| 4
3 files changed, 23 insertions(+), 12 deletions(-)
diff --git a/src/gallium/drivers/r6
---
src/gallium/drivers/r600/r600_llvm.c | 194 ++-
src/gallium/drivers/r600/r600_shader.c | 6 +-
src/gallium/drivers/radeon/radeon_llvm.h | 1 +
3 files changed, 144 insertions(+), 57 deletions(-)
diff --git a/src/gallium/drivers/r600/r600_llvm.c
b/src/gal
---
src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c | 14 --
1 file changed, 12 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
b/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
index d390f92..acbd387 100644
--- a/src/gallium/dri
---
src/gallium/drivers/r600/eg_asm.c| 2 ++
src/gallium/drivers/r600/r600_asm.c | 2 ++
src/gallium/drivers/r600/r600_llvm.c | 20
src/gallium/drivers/r600/r600_shader.c | 3 ++-
src/gallium/drivers/radeon/radeon_llvm.h | 1 +
5 files changed, 27 insert
From: Vadim Girlin
Pack kc_bank, const_index and chan into the load.const operand, unpack
them in r600_alu_from_byte_stream and use to override sel, kc_bank and chan
for the constants.
Expected operand value is (((512 + (kc_bank << 12) + const_index) << 2) + chan).
---
src/gallium/drivers/r600/
---
lib/Target/R600/AMDILISelDAGToDAG.cpp | 61 ++-
lib/Target/R600/R600ISelLowering.cpp | 14 +++-
2 files changed, 73 insertions(+), 2 deletions(-)
diff --git a/lib/Target/R600/AMDILISelDAGToDAG.cpp
b/lib/Target/R600/AMDILISelDAGToDAG.cpp
index 8fbf153..567
---
lib/Target/R600/AMDGPU.h | 1 +
lib/Target/R600/AMDGPUTargetMachine.cpp| 1 +
lib/Target/R600/AMDIL.h| 19 +++-
lib/Target/R600/AMDILISelDAGToDAG.cpp | 104 +
lib/Target/R600/MCTargetDesc/R6
From: Vadim Girlin
Remove Cxxx registers, add new special register - "ALU_CONST" and new
operand for each alu src - "sel". ALU_CONST is used to designate that the
new operand contains the value to override src.sel, src.kc_bank, src.chan
for constants in the driver.
---
lib/Target/R600/AMDILISelD
---
lib/Target/R600/R600Instructions.td | 110 +---
1 file changed, 65 insertions(+), 45 deletions(-)
diff --git a/lib/Target/R600/R600Instructions.td
b/lib/Target/R600/R600Instructions.td
index c3539ab..112f769 100644
--- a/lib/Target/R600/R600Instructions.td
+++
On Tue, Jan 8, 2013 at 12:46 PM, Kenneth Graunke wrote:
> Technically, variable sized arrays are a required feature of C99,
> redacted to be optional in C11, and not actually part of C++ whatsoever.
>
> Gcc allows using them in C++ unless you specify -pedantic, and Clang
> appears to allow them fo
On Tue, Jan 8, 2013 at 12:36 PM, Kenneth Graunke wrote:
> Clang doesn't support the TLS mode we use, so building Mesa with Clang
> and --enable-glx-tls simply causes every Piglit test under the sun to
> crash when calling glGetString(). Good times.
>
> To avoid catastrophic failure and confusion,
Technically, variable sized arrays are a required feature of C99,
redacted to be optional in C11, and not actually part of C++ whatsoever.
Gcc allows using them in C++ unless you specify -pedantic, and Clang
appears to allow them for simple/POD types.
exec_list is arguably POD, since it doesn't h
On 01/08/2013 11:40 AM, Marek Olšák wrote:
https://bugs.freedesktop.org/show_bug.cgi?id=59143
---
src/mesa/state_tracker/st_cb_texture.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/state_tracker/st_cb_texture.c
b/src/mesa/state_tracker/st_cb_texture.c
index
Clang doesn't support the TLS mode we use, so building Mesa with Clang
and --enable-glx-tls simply causes every Piglit test under the sun to
crash when calling glGetString(). Good times.
To avoid catastrophic failure and confusion, simply bail in configure.
---
configure.ac | 4
1 file chan
On 08/01/13 20:05, Matt Turner wrote:
> On Tue, Jan 8, 2013 at 11:26 AM, Emil Velikov
> wrote:
>> Allows us to easily disable compiler optimizations without directly dealing
>> with C{,XX}FLAGS
>>
>> Signed-off-by: Emil Velikov
>> ---
>
> This is getting silly. Do Dan's patches not satisfy ever
https://bugs.freedesktop.org/show_bug.cgi?id=59094
--- Comment #2 from Ian Romanick ---
Does this test pass on NVIDIA? AMD? The bisected commit was made to match the
GL spec and the behavior (of a different GL function) of the other vendors.
--
You are receiving this mail because:
You are the
On 01/08/2013 12:06 PM, Matt Turner wrote:
On Tue, Jan 8, 2013 at 8:38 AM, Marek Olšák wrote:
Some distributions use --enable-debug for testing packages and the commit
results in terrible CPU performance. It can be embarrassing for us.
If distros want debug symbols, they should put -g in thei
On Tue, Jan 8, 2013 at 8:38 AM, Marek Olšák wrote:
> Some distributions use --enable-debug for testing packages and the commit
> results in terrible CPU performance. It can be embarrassing for us.
If distros want debug symbols, they should put -g in their CFLAGS
instead of --enable-debug. --enabl
On Tue, Jan 8, 2013 at 11:26 AM, Emil Velikov wrote:
> Allows us to easily disable compiler optimizations without directly dealing
> with C{,XX}FLAGS
>
> Signed-off-by: Emil Velikov
> ---
This is getting silly. Do Dan's patches not satisfy everyone?
__
On 01/08/2013 02:03 AM, Anuj Phogat wrote:
On Tue, Jan 8, 2013 at 5:25 AM, Kenneth Graunke wrote:
From: Anuj Phogat
V2:
If mask has GL_STENCIL_BUFFER_BIT set, the depth formats for readRenderBuffer
and drawRenderBuffer must match unless one of the two buffers doesn't have
depth, in which case
On 01/08/2013 03:56 AM, Anuj Phogat wrote:
This patch fixes a blitting case when drawAttachment->Texture ==
readAttachment->Texture. It was causing an assertion failure in
intel_miptree_attach_map() with gles3 conformance test case:
framebuffer_blit_functionality_minifying_blit
Number of changes
On 01/08/2013 03:54 AM, Anuj Phogat wrote:
This patch adds required error checking in _mesa_BlitFramebuffer() when
blitting to multiple color render targets. It also fixes a case when
blitting to a framebuffer with renderbuffer/texture attached to
GL_COLOR_ATTACHMENT{i} (where i!=0). Earlier it s
https://bugs.freedesktop.org/show_bug.cgi?id=59143
---
src/mesa/state_tracker/st_cb_texture.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/state_tracker/st_cb_texture.c
b/src/mesa/state_tracker/st_cb_texture.c
index 02fc675..e869d61 100644
--- a/src/mesa/state_t
Allows us to easily disable compiler optimizations without directly dealing
with C{,XX}FLAGS
Signed-off-by: Emil Velikov
---
configure.ac | 22 ++
1 file changed, 22 insertions(+)
diff --git a/configure.ac b/configure.ac
index 1414ce8..4d96ba0 100644
--- a/configure.ac
+++ b
https://bugs.freedesktop.org/show_bug.cgi?id=59143
Vinson Lee changed:
What|Removed |Added
CC||mar...@gmail.com
--- Comment #1 from Vinson
On 08/01/13 17:27, Marek Olšák wrote:
> On Tue, Jan 8, 2013 at 6:11 PM, Matt Turner wrote:
>> On Tue, Jan 8, 2013 at 8:38 AM, Marek Olšák wrote:
>>> This reverts commit 122345876479cf5cf553e38162ab105658614ab7.
>>>
>>> Some distributions use --enable-debug for testing packages and the commit
>>>
https://bugs.freedesktop.org/show_bug.cgi?id=59143
Priority: medium
Bug ID: 59143
Keywords: regression
Assignee: mesa-dev@lists.freedesktop.org
Summary: src\mesa\state_tracker\st_cb_texture.c(832) : error
C2036: 'GLvoid *' :
Marek Olšák writes:
> I've had no problem with debugging even with -O2. Yeah, gdb cannot
> show some variables sometimes, but I rarely had to force CFLAGS to -O0
> to debug an issue. That said, I can continue overriding CFLAGS and
> CXXFLAGS like I do now if the majority of people think that
> --
I've had no problem with debugging even with -O2. Yeah, gdb cannot
show some variables sometimes, but I rarely had to force CFLAGS to -O0
to debug an issue. That said, I can continue overriding CFLAGS and
CXXFLAGS like I do now if the majority of people think that
--disable-optimizations would be u
https://bugs.freedesktop.org/show_bug.cgi?id=59094
Paul Berry changed:
What|Removed |Added
CC||stereotype...@gmail.com
--- Comment #1 from
On 01/08/2013 08:38 AM, Marek Olšák wrote:
This reverts commit 122345876479cf5cf553e38162ab105658614ab7.
Some distributions use --enable-debug for testing packages and the commit
results in terrible CPU performance. It can be embarrassing for us.
Surely people should understand that enabling d
On 08.01.2013 18:06, Matt Turner wrote:
> On Tue, Jan 8, 2013 at 5:00 AM, Andreas Boll
> wrote:
>> From: Arvind R
>>
>> applications cannot load libXvMC.so due to nv30_screen_create being
>> undefined.
>> This patch fixes that. And MPlayer successfully uses XvMC for
>> video-out on GeForce 9400
On Tue, Jan 8, 2013 at 6:11 PM, Matt Turner wrote:
> On Tue, Jan 8, 2013 at 8:38 AM, Marek Olšák wrote:
>> This reverts commit 122345876479cf5cf553e38162ab105658614ab7.
>>
>> Some distributions use --enable-debug for testing packages and the commit
>> results in terrible CPU performance. It can b
On Mon, Jan 7, 2013 at 1:19 PM, Ian Romanick wrote:
> On 01/04/2013 06:41 PM, Jordan Justen wrote:
>>
>> GTF/gles3 test suite wants this error to have higher priority
>> than the type checking.
>
>
> If the ReadPixels call in the test has multiple errors, the test should be
> changed. There is no
Some instructions like memory reads/writes are executed
asynchronously, so we need to insert S_WAITCNT instructions
to block before accessing their results. Previously we have
just inserted S_WAITCNT instructions after each async
instruction, this patch fixes this and adds a prober
insertion pass.
We shouldn't insert KILL optimization if we don't have a
kill instruction at all.
Signed-off-by: Christian König
---
lib/Target/R600/SIISelLowering.cpp | 14
lib/Target/R600/SIISelLowering.h |2 -
lib/Target/R600/SIInstructions.td | 24 +++---
lib/Target/R600/SILowerC
Hi Tom,
here are two patches I already send out last year, but are still missing
in your master branch on http://cgit.freedesktop.org/~tstellar/llvm/.
I just rebased them, so they should apply cleanly ontop of that branch.
Christian.
___
mesa-dev mail
On Tue, Jan 8, 2013 at 8:38 AM, Marek Olšák wrote:
> This reverts commit 122345876479cf5cf553e38162ab105658614ab7.
>
> Some distributions use --enable-debug for testing packages and the commit
> results in terrible CPU performance. It can be embarrassing for us.
>
> Those who do not want optimizat
On Mon, Jan 7, 2013 at 1:25 PM, Ian Romanick wrote:
> On 01/04/2013 06:41 PM, Jordan Justen wrote:
>>
>> Signed-off-by: Jordan Justen
>> ---
>> src/mesa/drivers/dri/intel/intel_tex_image.c |1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/src/mesa/drivers/dri/intel/intel_tex_image
On Tue, Jan 8, 2013 at 5:00 AM, Andreas Boll wrote:
> From: Arvind R
>
> applications cannot load libXvMC.so due to nv30_screen_create being undefined.
> This patch fixes that. And MPlayer successfully uses XvMC for
> video-out on GeForce 9400GT
>
> Log:
> vo_xvmc: Using Xv Adaptor #0 (Nouveau Ge
This reverts commit 122345876479cf5cf553e38162ab105658614ab7.
Some distributions use --enable-debug for testing packages and the commit
results in terrible CPU performance. It can be embarrassing for us.
Those who do not want optimizations should set the CFLAGS and CXXFLAGS
environment variables.
On Tue, Jan 8, 2013 at 4:51 PM, Jerome Glisse wrote:
> On Tue, Jan 8, 2013 at 10:15 AM, Marek Olšák wrote:
>> On Mon, Jan 7, 2013 at 9:30 PM, wrote:
>>> From: Jerome Glisse
>>>
>>> Signed-off-by: Jerome Glisse
>>> ---
>>> src/gallium/drivers/r300/r300_context.c | 2 +-
>>> src/ga
On Tue, Jan 8, 2013 at 10:15 AM, Marek Olšák wrote:
> On Mon, Jan 7, 2013 at 9:30 PM, wrote:
>> From: Jerome Glisse
>>
>> Signed-off-by: Jerome Glisse
>> ---
>> src/gallium/drivers/r300/r300_context.c | 2 +-
>> src/gallium/drivers/r600/r600_pipe.c | 2 +-
>> src/ga
On Mon, Jan 7, 2013 at 9:30 PM, wrote:
> From: Jerome Glisse
>
> Signed-off-by: Jerome Glisse
> ---
> src/gallium/drivers/r300/r300_context.c | 2 +-
> src/gallium/drivers/r600/r600_pipe.c | 2 +-
> src/gallium/drivers/radeonsi/radeonsi_pipe.c | 2 +-
> src/ga
On Sun, Jan 6, 2013 at 6:59 PM, Marek Olšák wrote:
> R6xx doesn't work - the issue seems to be with flushing (sometimes
> the destination buffer contains garbage). There are no hangs, so we're good.
>
> R7xx doesn't seem to have any alignment restriction despite our initial
> thinking. Everything
From: Arvind R
applications cannot load libXvMC.so due to nv30_screen_create being undefined.
This patch fixes that. And MPlayer successfully uses XvMC for
video-out on GeForce 9400GT
Log:
vo_xvmc: Using Xv Adaptor #0 (Nouveau GeForce 8/9 Textured Video)
vo_xvmc: Port 63 grabed
vo_xvmc: Allocate
This patch fixes a blitting case when drawAttachment->Texture ==
readAttachment->Texture. It was causing an assertion failure in
intel_miptree_attach_map() with gles3 conformance test case:
framebuffer_blit_functionality_minifying_blit
Number of changes in this file look scary. But most of them ar
This patch adds required error checking in _mesa_BlitFramebuffer() when
blitting to multiple color render targets. It also fixes a case when
blitting to a framebuffer with renderbuffer/texture attached to
GL_COLOR_ATTACHMENT{i} (where i!=0). Earlier it skips color blitting if
nothing is found attac
On Tue, Jan 8, 2013 at 5:22 AM, Kenneth Graunke wrote:
> On 01/07/2013 08:16 AM, Anuj Phogat wrote:
>>
>> Changes in fbobject.c fix a case when blitting to a framebuffer with
>> renderbuffers/textures attached to GL_COLOR_ATTACHMENT{i} (where i!=0).
>> Earlier it skips color blitting if nothing is
On 07.01.2013 21:30, j.gli...@gmail.com wrote:
From: Jerome Glisse
Signed-off-by: Jerome Glisse
Looks good to me.
Reviewed-by: Christian König
---
src/gallium/drivers/r300/r300_context.c | 2 +-
src/gallium/drivers/r600/r600_pipe.c | 2 +-
src/gallium/driver
On Tue, Jan 8, 2013 at 5:25 AM, Kenneth Graunke wrote:
> From: Anuj Phogat
>
> V2:
> If mask has GL_STENCIL_BUFFER_BIT set, the depth formats for readRenderBuffer
> and drawRenderBuffer must match unless one of the two buffers doesn't have
> depth, in which case it's not blitted, so the format ch
62 matches
Mail list logo
