[Mesa-dev] [Bug 45348] [swrast] piglit fbo-drawbuffers-arbfp regression

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=45348 Vinson Lee changed: What|Removed |Added Blocks||79039 -- You are receiving this mail becau

[Mesa-dev] [Bug 79039] [TRACKER] Mesa 10.2 release tracker

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79039 Vinson Lee changed: What|Removed |Added Depends on||45348 -- You are receiving this mail becau

[Mesa-dev] [Bug 59777] [softpipe] piglit interpolation-noperspective-gl_BackColor-flat-distance regression

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=59777 Vinson Lee changed: What|Removed |Added CC||jfons...@vmware.com Blocks|

[Mesa-dev] [Bug 79039] [TRACKER] Mesa 10.2 release tracker

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79039 Vinson Lee changed: What|Removed |Added Depends on||59777 -- You are receiving this mail becau

[Mesa-dev] [Bug 59777] [softpipe] piglit interpolation-noperspective-gl_BackColor-flat-distance regression

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=59777 --- Comment #2 from Vinson Lee --- This test is still failing on the 10.2 branch. This test passes on the 9.0 branch and also fails on the 9.1 branch. -- You are receiving this mail because: You are the assignee for the bug. __

[Mesa-dev] [Bug 79039] [TRACKER] Mesa 10.2 release tracker

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79039 Vinson Lee changed: What|Removed |Added Depends on||61153 -- You are receiving this mail becau

[Mesa-dev] [Bug 61153] [softpipe] piglit interpolation-noperspective-gl_BackColor-flat-vertex regression

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=61153 Vinson Lee changed: What|Removed |Added Blocks||79039 -- You are receiving this mail becau

[Mesa-dev] [Bug 61326] [softpipe] piglit interpolation-noperspective-gl_BackSecondaryColor-flat-vertex regression

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=61326 Vinson Lee changed: What|Removed |Added Blocks||79039 -- You are receiving this mail becau

[Mesa-dev] [Bug 79039] [TRACKER] Mesa 10.2 release tracker

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79039 Vinson Lee changed: What|Removed |Added Depends on||61326 -- You are receiving this mail becau

[Mesa-dev] [Bug 49125] sp_tex_sample.c:802:get_texel_2d_array: Assertion `layer < texture->array_size' failed.

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=49125 Vinson Lee changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [Bug 49539] [softpipe] piglit vs-attrib-ivec4-implied regression

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=49539 Vinson Lee changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [Bug 79039] [TRACKER] Mesa 10.2 release tracker

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79039 Vinson Lee changed: What|Removed |Added Depends on||44519 -- You are receiving this mail becau

[Mesa-dev] [Bug 44519] translate_test generic regression

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=44519 Vinson Lee changed: What|Removed |Added Summary|SIGABRT |translate_test generic |s

[Mesa-dev] [Bug 79472] New: [llvmpipe] SIGSEGV src/gallium/auxiliary/draw/draw_cliptest_tmp.h:91

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79472 Priority: medium Bug ID: 79472 Keywords: have-backtrace Assignee: mesa-dev@lists.freedesktop.org Summary: [llvmpipe] SIGSEGV src/gallium/auxiliary/draw/draw_cliptest_tmp.h:91

[Mesa-dev] [Bug 79471] New: [llvmpipe] [softpipe] state_tracker/st_glsl_to_tgsi.cpp:4495:translate_src: Assertion `src_reg->file != PROGRAM_TEMPORARY' failed.

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79471 Priority: medium Bug ID: 79471 Keywords: have-backtrace Assignee: mesa-dev@lists.freedesktop.org Summary: [llvmpipe] [softpipe] state_tracker/st_glsl_to_tgsi.cpp:4495:transla

[Mesa-dev] [Bug 79470] New: [llvmpipe] src/gallium/auxiliary/gallivm/lp_bld_tgsi_soa.c:2343:emit_fetch_texels: Assertion `0' failed.

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79470 Priority: medium Bug ID: 79470 Keywords: have-backtrace Assignee: mesa-dev@lists.freedesktop.org Summary: [llvmpipe] src/gallium/auxiliary/gallivm/lp_bld_tgsi_soa.c:2343:e

Re: [Mesa-dev] [PATCH 1/9] i965: Remove the dst and src0 parameters from brw_JMPI.

2014-05-30 Thread Matt Turner
Series is Reviewed-by: Matt Turner ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [Bug 79469] Commit e3cc0d90e14e62a0a787b6c07a6df0f5c84039be breaks unigine heaven

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79469 --- Comment #2 from veb...@hotmail.fr --- After looking closer at the weston log, the swrast_dri reference seems to be GLX X side, so probably indirect rendering failing, which has nothing to do with the bug I'm reporting. -- You are receiving t

[Mesa-dev] [Bug 79469] Commit e3cc0d90e14e62a0a787b6c07a6df0f5c84039be breaks unigine heaven

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79469 --- Comment #1 from veb...@hotmail.fr --- Created attachment 100188 --> https://bugs.freedesktop.org/attachment.cgi?id=100188&action=edit Weston log -- You are receiving this mail because: You are the assignee for the bug.

[Mesa-dev] [Bug 79469] New: Commit e3cc0d90e14e62a0a787b6c07a6df0f5c84039be breaks unigine heaven

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79469 Priority: medium Bug ID: 79469 Assignee: mesa-dev@lists.freedesktop.org Summary: Commit e3cc0d90e14e62a0a787b6c07a6df0f5c84039be breaks unigine heaven Severity: major Cla

[Mesa-dev] [PATCH 2/9] i965: Make brw_JMPI set predicate_control based on a parameter.

2014-05-30 Thread Kenneth Graunke
We use both predicated and unconditional JMPI instructions. But in each case, it's clear which we want. It's simpler to just specify it as a parameter, rather than relying on default state. Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_eu.h | 3 ++- src/mesa/drivers/dr

[Mesa-dev] [PATCH 1/9] i965: Remove the dst and src0 parameters from brw_JMPI.

2014-05-30 Thread Kenneth Graunke
In all cases, we set both dst and src0 to brw_ip_reg(). This is no accident: according to the ISA reference, both are required to be the IP register. So, we may as well drop the parameters. Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_eu.h | 3 +-- src/mesa/drivers/dr

[Mesa-dev] [PATCH 8/9] i965: Eliminate brw_set_conditionalmod from the Gen4-5 compilers.

2014-05-30 Thread Kenneth Graunke
With the predication changes eliminated, all this does is set the conditional modifier on a single instruction. Doing that directly is easy, and avoids mucking about with default state. Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_clip_line.c | 8 src/mesa/driv

[Mesa-dev] [PATCH 6/9] i965: Stop setting predication from brw_set_conditionalmod.

2014-05-30 Thread Kenneth Graunke
brw_set_conditionalmod has traditionally been complex: it causes conditionalmod to be set for the next instruction, and then predication to be set on all future instructions after that. We may want to generate a flag condition and not use it immediately, due to instruction scheduling or the like.

[Mesa-dev] [PATCH 9/9] i965: Delete brw_set_conditionalmod.

2014-05-30 Thread Kenneth Graunke
This removes the ability to set the default conditional modifier on all future instructions. Nothing uses it, and it's not really a sensible thing to do anyway. Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_eu.c | 5 - src/mesa/drivers/dri/i965/brw_eu.h | 1 - s

[Mesa-dev] [PATCH 4/9] i965/clip: Use the new brw_last_inst macro instead of temporaries.

2014-05-30 Thread Kenneth Graunke
It wasn't too bad before, but the macro is going to be nicer once I start modifying a lot more instructions in this pattern. Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_clip_line.c | 9 --- src/mesa/drivers/dri/i965/brw_clip_tri.c | 34 +

[Mesa-dev] [PATCH 7/9] i965: Don't use brw_set_conditionalmod in the FS and vec4 compilers.

2014-05-30 Thread Kenneth Graunke
brw_set_conditionalmod and brw_next_insn work together to set the conditional modifier for the next instruction, then turn it off. The Gen8+ generators don't implement this: we just set it for all future instructions, and whack it for each fs_inst/vec4_instruction. Both approaches work out because

[Mesa-dev] [PATCH 5/9] i965: Drop unnecessary brw_set_conditionalmod() before brw_CMP().

2014-05-30 Thread Kenneth Graunke
brw_CMP already takes a conditional modifier as a parameter, and sets it accordingly. brw_set_conditionalmod() also makes everything after the next instruction predicated, but we don't need that: we always emit an IF instruction after load_clip_distance(), and that's already predicated. Signed-of

[Mesa-dev] [PATCH 3/9] i965: Create a "brw_last_inst" convenience macro.

2014-05-30 Thread Kenneth Graunke
Often times, we want to emit an instruction, then set one field on it, such as predication or a conditional modifier. Normally, we'd have to declare "struct brw_instruction *inst;" and then use "inst = brw_FOO(...)" to emit the instruction, which can hurt readability. The new "brw_last_inst" macr

[Mesa-dev] [Bug 65224] piglit arb_uniform_buffer_object-maxuniformblocksize fs regression

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=65224 Vinson Lee changed: What|Removed |Added Blocks||79039 -- You are receiving this mail becau

[Mesa-dev] [Bug 79039] [TRACKER] Mesa 10.2 release tracker

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79039 Vinson Lee changed: What|Removed |Added Depends on||65224 -- You are receiving this mail becau

[Mesa-dev] [PATCH 3/5] i965: Give dump_instructions() a filename argument.

2014-05-30 Thread Matt Turner
This will allow debugging code to dump the IR after an optimization pass makes progress (the next patch). Only let it open and write to a file if the effective user isn't root. --- src/mesa/drivers/dri/i965/brw_fs.cpp | 22 +++--- src/mesa/drivers/dri/i965/brw_fs.h

[Mesa-dev] [PATCH 5/5] i965: Skip IR annotations with INTEL_DEBUG=noann.

2014-05-30 Thread Matt Turner
Running shader-db with INTEL_DEBUG=noann reduces the runtime from ~90 to ~80 seconds on my machine. It also reduces the disk space consumed by the .out files from 660 MB (676 on disk) to 343 MB (358 on disk). --- src/mesa/drivers/dri/i965/brw_shader.cpp | 6 -- src/mesa/drivers/dri/i965/intel_

[Mesa-dev] [PATCH 4/5] i965/fs: Debug the optimization passes by dumping instr to file.

2014-05-30 Thread Matt Turner
With INTEL_DEBUG=optimizer, write the output of dump_instructions() to a file each time an optimization pass makes progress. This lets you easily diff successive files to see what an optimization pass did. Example filenames written when running glxgears: fs8-00-00-start fs8-00-01-04-opt_copy

[Mesa-dev] [PATCH 1/5] i965: Add envvar to debug the optimization passes.

2014-05-30 Thread Matt Turner
--- src/mesa/drivers/dri/i965/intel_debug.c | 1 + src/mesa/drivers/dri/i965/intel_debug.h | 1 + 2 files changed, 2 insertions(+) diff --git a/src/mesa/drivers/dri/i965/intel_debug.c b/src/mesa/drivers/dri/i965/intel_debug.c index 621a571..bba873b 100644 --- a/src/mesa/drivers/dri/i965/intel_de

[Mesa-dev] [PATCH 2/5] i965: Give dump_instruction() a FILE* argument.

2014-05-30 Thread Matt Turner
Use function overloading rather than default arguments, since gdb doesn't know about default arguments. --- src/mesa/drivers/dri/i965/brw_fs.cpp | 108 + src/mesa/drivers/dri/i965/brw_fs.h | 1 + src/mesa/drivers/dri/i965/brw_shader.h | 1 + src/mesa/drive

Re: [Mesa-dev] [PATCH 09/21] glsl: Set ir_instruction::ir_type in the base class constructor

2014-05-30 Thread Kenneth Graunke
On Tuesday, May 27, 2014 07:49:04 PM Ian Romanick wrote: > From: Ian Romanick > > This has the added perk that if you forget to set ir_type in the > constructor of a new subclass (or a new constructor of an existing > subclass) the compiler will tell you... instead of relying on > ir_validate or

Re: [Mesa-dev] Mesa IR as a list of instructions

2014-05-30 Thread Kenneth Graunke
On Wednesday, May 28, 2014 11:37:31 AM Eric Anholt wrote: > Here's a series I started back in January as a little experiment. > Basically, I feel guilty for pushing GLSL IR into the driver, and wish I'd > just fixed up Mesa IR back in the day. But, given that we're still > feeding Mesa IR into dri

Re: [Mesa-dev] [PATCH 00/21] Reduce ir_variable memory usage

2014-05-30 Thread Ian Romanick
On 05/30/2014 05:52 PM, Kenneth Graunke wrote: > On Friday, May 30, 2014 11:07:03 AM Eric Anholt wrote: >> Ian Romanick writes: >>> This series reduces the memory usage of ir_variable quite significantly. >>> >>> The first couple patches add a mechanism to determine the amount of >>> memory used b

Re: [Mesa-dev] [PATCH 00/21] Reduce ir_variable memory usage

2014-05-30 Thread Kenneth Graunke
On Friday, May 30, 2014 11:07:03 AM Eric Anholt wrote: > Ian Romanick writes: > > This series reduces the memory usage of ir_variable quite significantly. > > > > The first couple patches add a mechanism to determine the amount of > > memory used by any kind of IR object. This is used to collect

Re: [Mesa-dev] [PATCH 03/21] glsl: Eliminate ir_variable::data.atomic.buffer_index

2014-05-30 Thread Kenneth Graunke
On Tuesday, May 27, 2014 07:48:58 PM Ian Romanick wrote: > From: Ian Romanick > > Just use ir_variable::data.binding... because that's the where the > binding is stored for everything else that can use layout(binding=). > > No change in the peak ir_variable memory usage in a trimmed apitrace of

[Mesa-dev] Mesa 10.2 release candidate 5

2014-05-30 Thread Ian Romanick
(Resend with GPG signature.) Mesa 10.2 release candidate 5 is now available for testing. The current plan of record was for this to be the last release candidate, and the 10.2 release is planned for Friday, June 6th. The tag in the GIT repository for Mesa 10.2-rc5 is 'mesa-10.2-rc5'. I have ver

[Mesa-dev] Mesa 10.2 release candidate 5

2014-05-30 Thread Ian Romanick
Mesa 10.2 release candidate 5 is now available for testing. The current plan of record was for this to be the last release candidate, and the 10.2 release is planned for Friday, June 6th. The tag in the GIT repository for Mesa 10.2-rc5 is 'mesa-10.2-rc5'. I have verified that the tag is in the c

Re: [Mesa-dev] [PATCH] Check for NULL native_window into dri2_wl_create_surface

2014-05-30 Thread Ian Romanick
On 05/30/2014 04:58 PM, Sinclair Yeh wrote: > We have customers using NULL as a way to test the robustness of > the API. Without this check, EGL will segfault trying to > dereference dri2_surf->wl_win->private because wl_win is > NULL. It sounds like we need a piglit test for this... :) Should t

[Mesa-dev] [PATCH] Check for NULL native_window into dri2_wl_create_surface

2014-05-30 Thread Sinclair Yeh
We have customers using NULL as a way to test the robustness of the API. Without this check, EGL will segfault trying to dereference dri2_surf->wl_win->private because wl_win is NULL. This fix adds a check and sets EGL_BAD_NATIVE_WINDOW --- src/egl/drivers/dri2/platform_wayland.c | 7 +++ 1

Re: [Mesa-dev] [PATCH 2/3] radeon/compute: Implement PIPE_COMPUTE_CAP_MAX_COMPUTE_UNITS

2014-05-30 Thread Alex Deucher
On Fri, May 30, 2014 at 11:31 AM, Bruno Jiménez wrote: > The data has been extracted from: > AMD Accelerated Parallel Processing OpenCL Programming Guide (rev 2.7) > Appendix D: Device Parameters You should add a query for the number of compute units to the RADEON_INFO ioctl and then just ask the

Re: [Mesa-dev] [PATCH] mesa: Free the compiled shader IR after it has been linked.

2014-05-30 Thread Ian Romanick
On 05/28/2014 01:57 PM, Eric Anholt wrote: > If the shader compiled once, then we can compile it again. Compiled > shaders almost always get used in just one program, so holding that > compiled IR until the program is freed is just a waste of memory. Would this work with some madness like: gl

[Mesa-dev] [Bug 79230] After upgrade from 10.1.4 to 10.2-rc4 cross-compile fails

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79230 --- Comment #6 from Emil Velikov --- A couple of distinct issues here (rather unrelated): 1: Missing gl* declarations as posted originally. Most likely a duplicate of bug 79294. Should be addressed with http://patchwork.freedesktop.org/patch/269

[Mesa-dev] [Bug 79230] After upgrade from 10.1.4 to 10.2-rc4 cross-compile fails

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79230 --- Comment #5 from war...@o2.pl --- On 30/05/14 15:38, bugzilla-dae...@freedesktop.org wrote: > > *Comment # 3 > on bug 79230 from > Em

Re: [Mesa-dev] [PATCH] glapi: Duplicate GLES1 prototypes in glapi_dispatch.c

2014-05-30 Thread Matt Turner
On Fri, May 30, 2014 at 1:57 PM, Ian Romanick wrote: > From: Ian Romanick > > These prototypes are necessary because GLES1 library builds will create > dispatch functions for them. We can't directly include GLES/gl.h > because it would conflict the previously-included GL/gl.h. Since GLES1 > ABI

[Mesa-dev] [PATCH] glapi: Duplicate GLES1 prototypes in glapi_dispatch.c

2014-05-30 Thread Ian Romanick
From: Ian Romanick These prototypes are necessary because GLES1 library builds will create dispatch functions for them. We can't directly include GLES/gl.h because it would conflict the previously-included GL/gl.h. Since GLES1 ABI is not expected to every add more functions, the path of least r

[Mesa-dev] [Bug 79230] After upgrade from 10.1.4 to 10.2-rc4 cross-compile fails

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79230 --- Comment #4 from war...@o2.pl --- That was my suspect. But 2 things are worth to mention: -exactly the same environment builds OK with 10.1.4 -I verified pkgconfig.pc for libdrm_radeon: [piotro@myth-devel-vm pkgconfig]$ cat libdrm_radeon.pc pr

Re: [Mesa-dev] Mesa IR as a list of instructions

2014-05-30 Thread Ilia Mirkin
On Fri, May 30, 2014 at 3:22 PM, Eric Anholt wrote: > Marek Olšák writes: > >> I have just some comments below. >> >> On Wed, May 28, 2014 at 8:37 PM, Eric Anholt wrote: >>> Here's a series I started back in January as a little experiment. >>> Basically, I feel guilty for pushing GLSL IR into th

[Mesa-dev] [PATCH 3/4] radeon/vce: implement h264 profile support

2014-05-30 Thread Leo Liu
Signed-off-by: Leo Liu --- src/gallium/drivers/radeon/radeon_vce_40_2_2.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeon/radeon_vce_40_2_2.c b/src/gallium/drivers/radeon/radeon_vce_40_2_2.c index 3010c5b..c67f8f9 100644 --- a/src/gallium/driver

[Mesa-dev] [PATCH 4/4] st/omx/enc: enable b frames

2014-05-30 Thread Leo Liu
Signed-off-by: Leo Liu --- src/gallium/state_trackers/omx/vid_enc.c | 6 -- src/gallium/state_trackers/omx/vid_enc.h | 2 +- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/src/gallium/state_trackers/omx/vid_enc.c b/src/gallium/state_trackers/omx/vid_enc.c index f828d05..d16fa

[Mesa-dev] [PATCH 2/4] st/omx/enc: implement h264 profile support

2014-05-30 Thread Leo Liu
Signed-off-by: Leo Liu --- src/gallium/state_trackers/omx/vid_enc.c | 50 ++-- src/gallium/state_trackers/omx/vid_enc.h | 1 + 2 files changed, 49 insertions(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/omx/vid_enc.c b/src/gallium/state_trackers/omx/vi

[Mesa-dev] [PATCH 1/4] vl: add more avc profiles

2014-05-30 Thread Leo Liu
Signed-off-by: Leo Liu --- src/gallium/auxiliary/util/u_video.h | 4 src/gallium/include/pipe/p_video_enums.h | 6 +- 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/util/u_video.h b/src/gallium/auxiliary/util/u_video.h index da65a58..d1ca736 1006

[Mesa-dev] [Bug 77596] [r600g] random and reproducible crash in Left 4 Dead 2 (bisected)

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=77596 --- Comment #7 from Benjamin Bellec --- I finally found that I omitted to specify "--prefix=/usr" on build time. Could this be the source of the crash (eg. if old "includes" were in this place) ? The game didn't crash since many days, I think tha

Re: [Mesa-dev] Mesa IR as a list of instructions

2014-05-30 Thread Eric Anholt
Connor Abbott writes: > On Wed, May 28, 2014 at 8:50 PM, Eric Anholt wrote: >> Connor Abbott writes: >> >>> On Wed, May 28, 2014 at 2:37 PM, Eric Anholt wrote: Here's a series I started back in January as a little experiment. Basically, I feel guilty for pushing GLSL IR into the driv

Re: [Mesa-dev] Mesa IR as a list of instructions

2014-05-30 Thread Eric Anholt
Marek Olšák writes: > I have just some comments below. > > On Wed, May 28, 2014 at 8:37 PM, Eric Anholt wrote: >> Here's a series I started back in January as a little experiment. >> Basically, I feel guilty for pushing GLSL IR into the driver, and wish I'd >> just fixed up Mesa IR back in the d

Re: [Mesa-dev] [PATCH] i965: Fix Line Stipple enable bit in 3DSTATE_SF for Haswell.

2014-05-30 Thread Matt Turner
I've committed this and marked it as a candidate for the 10.1 and 10.2 stable branches. ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [Bug 79294] Xlib-based build broken on non x86/x86-64 architectures

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79294 Ian Romanick changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|mesa-dev@list

Re: [Mesa-dev] [PATCH 00/21] Reduce ir_variable memory usage

2014-05-30 Thread Eric Anholt
Ian Romanick writes: > This series reduces the memory usage of ir_variable quite significantly. > > The first couple patches add a mechanism to determine the amount of > memory used by any kind of IR object. This is used to collect the data > that is shown in the commit messages through the seri

Re: [Mesa-dev] [PATCH] glx: Don't uselessly dlopen libGL within libGL

2014-05-30 Thread Adam Jackson
On Thu, 2014-05-29 at 15:10 -0700, Ian Romanick wrote: > That code was originally added by: > > commit 061a3fe34051327fba418cc99599ecff0016ee15 > Author: Michel Dänzer > Date: Mon Aug 14 15:53:37 2006 + > > Bug #7169: Attempt to make libGL symbols visible to drivers. > > Some

Re: [Mesa-dev] [PATCH] llvmpipe: fix crash when not all attachments are populated in a fb

2014-05-30 Thread Jose Fonseca
- Original Message - > From: Roland Scheidegger > > Framebuffers can have NULL attachments since a while. llvmpipe handled > that properly for lp_rast_shade_quads_mask but it seems the change didn't > make it to lp_rast_shade_tile. > This fixes piglit fbo-drawbuffers-none test (though I

[Mesa-dev] [PATCH 0/3] clover: Allow quering for the number of max compute units

2014-05-30 Thread Bruno Jiménez
This series allows clover to answer the query for the number of max compute units of the devices. The data has been extracted from: AMD Accelerated Parallel Processing OpenCL Programming Guide (rev 2.7) Appendix D: Device Parameters For the rest of the devices I haven't found data. Patch 2 is ve

[Mesa-dev] [PATCH 1/3] gallium: Add PIPE_COMPUTE_CAP_MAX_COMPUTE_UNITS

2014-05-30 Thread Bruno Jiménez
--- src/gallium/docs/source/screen.rst | 2 ++ src/gallium/include/pipe/p_defines.h | 3 ++- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/src/gallium/docs/source/screen.rst b/src/gallium/docs/source/screen.rst index b292257..dde2c38 100644 --- a/src/gallium/docs/source/screen.r

[Mesa-dev] [PATCH 2/3] radeon/compute: Implement PIPE_COMPUTE_CAP_MAX_COMPUTE_UNITS

2014-05-30 Thread Bruno Jiménez
The data has been extracted from: AMD Accelerated Parallel Processing OpenCL Programming Guide (rev 2.7) Appendix D: Device Parameters --- src/gallium/drivers/radeon/r600_pipe_common.c | 90 +++ 1 file changed, 90 insertions(+) diff --git a/src/gallium/drivers/radeon/r600_

[Mesa-dev] [PATCH 3/3] clover: query driver for the max number of compute units

2014-05-30 Thread Bruno Jiménez
--- src/gallium/state_trackers/clover/api/device.cpp | 2 +- src/gallium/state_trackers/clover/core/device.cpp | 6 ++ src/gallium/state_trackers/clover/core/device.hpp | 1 + 3 files changed, 8 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/clover/api/device.cpp b/src

[Mesa-dev] [PATCH] [RFC] r600g/compute: Solving the mapping bug

2014-05-30 Thread Bruno Jiménez
Hi, This is my first attempt to fix a bug that the r600g/compute code has when handling mapped buffers in OpenCL. I have written a small document explaining more or less what is the bug I am trying to solve, and the approach I have chosen to solve it with this patch: https://www.dropbox.com/s/qwk

[Mesa-dev] [PATCH] r600g/compute: solve the mapping bug

2014-05-30 Thread Bruno Jiménez
This patch changes completely the way buffers are added to the compute_memory_pool. Before this, whenever we were going to map a buffer or write to or read from it, it would get placed into the pool. Now, every unallocated buffer has its own r600_resource until it is allocated in the pool. This al

[Mesa-dev] [Bug 79294] Xlib-based build broken on non x86/x86-64 architectures

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79294 Andreas Boll changed: What|Removed |Added Keywords||bisected --- Comment #2 from Andreas Boll

Re: [Mesa-dev] [PATCH] glx/dri3: request async pixmap present for swap interval 0

2014-05-30 Thread Chris Wilson
On Thu, Apr 24, 2014 at 01:19:44PM +0100, Frank Binns wrote: > Typically we want swaps to occur during vertical blank to avoid > tearing. However, when the swap interval is 0 we want to go > as quickly as possible even if this results in tearing. > > Indicate to the X server that it should attempt

[Mesa-dev] [Bug 79230] After upgrade from 10.1.4 to 10.2-rc4 cross-compile fails

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79230 --- Comment #3 from Emil Velikov --- (In reply to comment #2) > Created attachment 100147 [details] > fixing libdrm includes when cross-compile Strange, neither of these changes should be needed. When winsys/radeon/drm is build the RADEON_CFLAGS

Re: [Mesa-dev] [PATCH 2/2] i965: in set_read_rb_tex_image() check _mesa_meta_bind_rb_as_tex_image() did succeed

2014-05-30 Thread Pohjolainen, Topi
On Fri, May 30, 2014 at 01:54:48PM +0300, Juha-Pekka Heikkila wrote: > Check if _mesa_meta_bind_rb_as_tex_image() did give the texture. > If no texture was given there is already either > GL_INVALID_VALUE or GL_OUT_OF_MEMORY error set in context. > > Signed-off-by: Juha-Pekka Heikkila > --- > sr

Re: [Mesa-dev] [PATCH 0/2] Klocwork patches

2014-05-30 Thread Pohjolainen, Topi
On Fri, May 30, 2014 at 01:54:46PM +0300, Juha-Pekka Heikkila wrote: > These pass piglit quick set on my Ivybridge. The stencil meta path is not in use on ivb - you would need bdw for that. > > For patch number 2 it bring the error all the way to set_read_rb_tex_image() > which is "static void"

Re: [Mesa-dev] [PATCH V2 06/11] loader: Use drirc device_id parameter in complement to DRI_PRIME

2014-05-30 Thread Axel Davy
On 30/05/2014 06:05, Michel Dänzer wrote : On 30.05.2014 12:49, Axel Davy wrote: On 29/05/2014 23:31, Michel Dänzer a wrote : I tried with your compile options (except for the opencl flags, didn't have the required libs), and it worked. Did you try adding -lm to libloader_la_LIBADD ? That fixe

[Mesa-dev] [PATCH 0/2] Klocwork patches

2014-05-30 Thread Juha-Pekka Heikkila
These pass piglit quick set on my Ivybridge. For patch number 2 it bring the error all the way to set_read_rb_tex_image() which is "static void" function. Was pondering if it be changed to "static bool" to be able to carry the error further? (it is anyhow marked into the context at this point) Thi

[Mesa-dev] [PATCH 2/2] i965: in set_read_rb_tex_image() check _mesa_meta_bind_rb_as_tex_image() did succeed

2014-05-30 Thread Juha-Pekka Heikkila
Check if _mesa_meta_bind_rb_as_tex_image() did give the texture. If no texture was given there is already either GL_INVALID_VALUE or GL_OUT_OF_MEMORY error set in context. Signed-off-by: Juha-Pekka Heikkila --- src/mesa/drivers/common/meta_blit.c | 10 +- src/mesa/drivers/d

[Mesa-dev] [PATCH 1/2] i965/fs: Add missing null checks in fs_generator::generate_assembly()

2014-05-30 Thread Juha-Pekka Heikkila
In fs_generator::generate_assembly() was avoided null access when fetching variable prog but then prog was left null and used as pointer. Signed-off-by: Juha-Pekka Heikkila --- src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/

Re: [Mesa-dev] [PATCH V2 06/11] loader: Use drirc device_id parameter in complement to DRI_PRIME

2014-05-30 Thread Michel Dänzer
On 30.05.2014 12:49, Axel Davy wrote: > On 29/05/2014 23:31, Michel Dänzer a wrote : > > I tried with your compile options (except for the opencl flags, didn't > have the required libs), and it worked. > Did you try adding -lm to libloader_la_LIBADD ? That fixes it, but is it a proper solution? Do

Re: [Mesa-dev] [PATCH 00/21] Reduce ir_variable memory usage

2014-05-30 Thread Tapani Pälli
Just one nitpick about spec for patch 4 (feel free to ignore), otherwise Reviewed-by: Tapani Pälli (I also run Piglit quick, no regressions) On 05/28/2014 05:48 AM, Ian Romanick wrote: > This series reduces the memory usage of ir_variable quite significantly. > > The first couple patches add a

Re: [Mesa-dev] [PATCH 04/21] glsl: Use a single bit for the dual-source blend index

2014-05-30 Thread Tapani Pälli
On 05/28/2014 05:48 AM, Ian Romanick wrote: > From: Ian Romanick > > The only values allowed are 0 and 1, and the value is checked before > assigning. > > With the previous changes, reduces the peak ir_variable memory usage in > a trimmed apitrace of dota2 by 204KiB on 64-bit. > > Before: IR MEM:

[Mesa-dev] [Bug 79230] After upgrade from 10.1.4 to 10.2-rc4 cross-compile fails

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79230 --- Comment #2 from war...@o2.pl --- Created attachment 100147 --> https://bugs.freedesktop.org/attachment.cgi?id=100147&action=edit fixing libdrm includes when cross-compile -- You are receiving this mail because: You are the assignee for the

Re: [Mesa-dev] [PATCH] i965/vec4: Allow writemasking on math instructions on Gen7+.

2014-05-30 Thread Kenneth Graunke
On Thursday, May 29, 2014 09:55:42 PM Matt Turner wrote: > The math instruction was Align1-only on Gen6 and we never updated this > to let it use Align16 features like writemasking on newer platforms. > > total instructions in shared programs: 1686120 -> 1685507 (-0.04%) > instructions in affected

[Mesa-dev] [Bug 79440] prog_hash_table.c:146: undefined reference to `_mesa_error_no_memory'

2014-05-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79440 Tapani Pälli changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

Re: [Mesa-dev] [PATCH] scons: add common.c as part of glcpp build

2014-05-30 Thread Juha-Pekka Heikkilä
Reviewed-by: Juha-Pekka Heikkila On Fri, May 30, 2014 at 10:11 AM, Tapani Pälli wrote: > to have _mesa_error_no_memory function available > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=79440 > Signed-off-by: Tapani Pälli > --- > src/glsl/SConscript | 2 +- > 1 file changed, 1 inser

[Mesa-dev] [PATCH] scons: add common.c as part of glcpp build

2014-05-30 Thread Tapani Pälli
to have _mesa_error_no_memory function available Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=79440 Signed-off-by: Tapani Pälli --- src/glsl/SConscript | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/glsl/SConscript b/src/glsl/SConscript index fe9d507..dc35477 1