[Mesa-dev] [PATCH v2 3/4] r600,compute: Plug memory leaks

2015-06-06 Thread Jan Vesely
v2: drop inline keyword drop radeon_llvm_dispose_kernel_module wrapper Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- I decided to keep radeon_shader_binary_clean,a nd dropped the other one. src/gallium/drivers/r600/evergreen_compute.c | 24 +---

Re: [Mesa-dev] [PATCH 2/4] nouveau/compiler: init the size and code variables to NULL/0

2015-06-06 Thread Eirik Byrkjeflot Anonsen
Ilia Mirkin imir...@alum.mit.edu writes: Sure. Fix it in GCC :) disable the bogus warning, whatever. I really really hate pandering to broken tools, sending the message that it's ok for the tools to be broken. While I largely agree with the sentiment, I'm not sure I agree with what is a

Re: [Mesa-dev] [PATCH 2/4] nouveau/compiler: init the size and code variables to NULL/0

2015-06-06 Thread Ilia Mirkin
On Jun 6, 2015 9:51 AM, Eirik Byrkjeflot Anonsen ei...@eirikba.org wrote: Ilia Mirkin imir...@alum.mit.edu writes: Sure. Fix it in GCC :) disable the bogus warning, whatever. I really really hate pandering to broken tools, sending the message that it's ok for the tools to be broken.

Re: [Mesa-dev] [PATCH] gallivm: Only build lp_profile() body when PROFILE is defined

2015-06-06 Thread Jose Fonseca
On 06/06/15 02:30, Tom Stellard wrote: The only use of lp_profile() is wrapped in #if defined(PROFILE), so there is no reason to build it unless this macro is defined. --- src/gallium/auxiliary/gallivm/lp_bld_debug.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [Mesa-dev] [PATCH 2/3] i915: Blit RGBX-RGBA drawpixels

2015-06-06 Thread Chris Wilson
On Sat, Jun 06, 2015 at 12:58:05PM +0300, Alexander Monakov wrote: On Fri, Jun 5, 2015 at 5:14 PM, Chris Wilson ch...@chris-wilson.co.uk wrote: The blitter already has code to accommodate filling in the alpha channel for BGRX destination formats, so expand this to also allow filling the

Re: [Mesa-dev] [PATCH 2/3] i915: Blit RGBX-RGBA drawpixels

2015-06-06 Thread Alexander Monakov
On Fri, Jun 5, 2015 at 5:14 PM, Chris Wilson ch...@chris-wilson.co.uk wrote: The blitter already has code to accommodate filling in the alpha channel for BGRX destination formats, so expand this to also allow filling the alpha channgel in RGBX formats. More importantly for the next patch is

Re: [Mesa-dev] [PATCH 1/2] gallium: add PIPE_COMPUTE_CAP_SUBGROUP_SIZE

2015-06-06 Thread Francisco Jerez
Giuseppe Bilotta giuseppe.bilo...@gmail.com writes: On Fri, Jun 5, 2015 at 10:35 PM, Francisco Jerez curroje...@riseup.net wrote: OTOH, at least in OpenCL, this cap wouldn't be used 'raw' as performance hint, since the actual value returned (the PREFERRED_WORK_GROUP_SIZE_MULTIPLE) is a

[Mesa-dev] [Bug 89018] Civilization: Beyond Earth terrain section not rendered

2015-06-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=89018 --- Comment #18 from Sami Liedes sami.lie...@iki.fi --- I'm not seeing this anymore on mesa/llvm git from today either on Radeon R9 270 or swrast. -- You are receiving this mail because: You are the QA Contact for the bug. You are the assignee

Re: [Mesa-dev] [PATCH v2 3/4] r600,compute: Plug memory leaks

2015-06-06 Thread Marek Olšák
Please move the inline functions to a .c file. Also, if they are not used by drivers/radeon or radeonsi, they shouldn't be in drivers/radeon. Marek On Sat, Jun 6, 2015 at 10:33 PM, Jan Vesely jan.ves...@rutgers.edu wrote: v2: drop inline keyword drop radeon_llvm_dispose_kernel_module

[Mesa-dev] [Bug 89819] WebGL Conformance swrast failure in conformance/uniforms/uniform-default-values.html

2015-06-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=89819 --- Comment #4 from Luke lukebe...@hotmail.com --- I did see a crash though when loading that page but it didn't seem related Roland, I just retested and only google-chrome is giving the results above. Firefox is crashing like you reported. To

[Mesa-dev] [Bug 89819] WebGL Conformance swrast failure in conformance/uniforms/uniform-default-values.html

2015-06-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=89819 --- Comment #3 from Luke lukebe...@hotmail.com --- Roland, Thanks for looking into this! If you need the tests in a piglit form, see: Bug 89754 - http://cgit.freedesktop.org/piglit/commit/?id=aa97ac98dd8f1eab8f288f9509fdf3f91ffbecd6 and Bug

[Mesa-dev] [PATCH 2/2] fs/reg_allocate: Remove the MRF hack helpers from fs_visitor

2015-06-06 Thread Jason Ekstrand
These are helpers that only exist in this one file. No reason to put them in the visitor. --- src/mesa/drivers/dri/i965/brw_fs.h| 4 src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp | 25 --- 2 files changed, 13 insertions(+), 16 deletions(-) diff

[Mesa-dev] [PATCH 1/2] i965/fs: Don't let the EOT send message interfere with the MRF hack

2015-06-06 Thread Jason Ekstrand
Previously, we just put the message for the EOT send as high in the file as it would go. This is because the register pre-filling hardware will stop all over the early registers in the file in preparation for the next thread while you're still sending the last message. However, if something