Re: [Mesa-dev] [PATCH] mesa: use signed temporary variable to store _ColorDrawBufferIndexes

2014-01-13 Thread Marek Olšák
Reviewed-by: Marek Olšák marek.ol...@amd.com Marek On Sun, Jan 12, 2014 at 11:52 PM, Emil Velikov emil.l.veli...@gmail.com wrote: _ColorDrawBufferIndexes is defined as GLint* and using a GLuint* will result in the first part of the conditional to be evaluated to true always. Unintentionally

Re: [Mesa-dev] [PATCH] Mark debug_print with __attribute__ ((format(__printf__, 1, 0)))

2014-01-13 Thread Thierry Reding
On Sun, Jan 12, 2014 at 10:34:19AM -0800, Keith Packard wrote: the drmServerInfo member, debug_print, takes a printf format string and varargs list. Tell the compiler about it. Signed-off-by: Keith Packard kei...@keithp.com --- xf86drm.h | 8 +++- 1 file changed, 7 insertions(+), 1

Re: [Mesa-dev] Naming everything in src/gallium/drivers/radeonsi si_*

2014-01-13 Thread Marek Olšák
For the series: Reviewed-by: Marek Olšák marek.ol...@amd.com Feel free to push this. Marek On Sat, Jan 11, 2014 at 4:20 PM, Andreas Hartmetz ahartm...@gmail.com wrote: Continuing here because the threads had diverged... I've updated the patch series under the same URL and applied all the

[Mesa-dev] [PATCH 5/6] r600g, radeonsi: if discarding the whole buffer range, discard the whole resource instead

2014-01-13 Thread Marek Olšák
From: Marek Olšák marek.ol...@amd.com Also set the unsynchronized flag if the whole resource was discarded to avoid doing buffer-busy checks again. --- src/gallium/drivers/radeon/r600_buffer_common.c | 8 1 file changed, 8 insertions(+) diff --git

[Mesa-dev] [PATCH 6/6] radeonsi: handle R600_CONTEXT_PS_PARTIAL_FLUSH in si_emit_cache_flush

2014-01-13 Thread Marek Olšák
From: Marek Olšák marek.ol...@amd.com For consistency only, This unused by radeonsi currently. --- src/gallium/drivers/radeonsi/si_state_draw.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c

[Mesa-dev] [PATCH 1/6] vdpau: flush the context after resolving delayed rendering

2014-01-13 Thread Marek Olšák
From: Marek Olšák marek.ol...@amd.com Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=73191 When VL uploads vertex buffers, it uses PIPE_TRANSFER_DONTBLOCK, which always flushes the context in the winsys if the buffer being mapped is busy. Since I added handling of DISCARD_RANGE,

[Mesa-dev] [PATCH 3/6] gallium/hud: just unmap the upload vertex buffer instead of recreating it

2014-01-13 Thread Marek Olšák
From: Marek Olšák marek.ol...@amd.com --- src/gallium/auxiliary/hud/hud_context.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/hud/hud_context.c b/src/gallium/auxiliary/hud/hud_context.c index c4a4f18..465013c 100644 ---

[Mesa-dev] [PATCH 2/6] gallium/vl: use u_upload_mgr to upload vertices for vl_compositor

2014-01-13 Thread Marek Olšák
From: Marek Olšák marek.ol...@amd.com This is the recommended way for streaming vertices. Always use this if you need to upload vertices every frame. --- src/gallium/auxiliary/vl/vl_compositor.c | 51 src/gallium/auxiliary/vl/vl_compositor.h | 1 + 2 files

[Mesa-dev] [PATCH 4/6] gallium/u_upload_mgr: don't expose u_upload_flush

2014-01-13 Thread Marek Olšák
From: Marek Olšák marek.ol...@amd.com It's unused and shouldn't be used at all in my opinion. If some driver doesn't support the unsynchronized flag, u_upload_mgr should avoid the synchronization by other means, e.g. by using the DONTBLOCK flag. --- src/gallium/auxiliary/util/u_upload_mgr.c |

[Mesa-dev] [PATCH] glx: Add missing null check in glXCreateContextAttribsARB

2014-01-13 Thread Juha-Pekka Heikkila
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com --- src/glx/create_context.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/glx/create_context.c b/src/glx/create_context.c index 38e949a..b15921f 100644 --- a/src/glx/create_context.c +++ b/src/glx/create_context.c @@

Re: [Mesa-dev] [PATCH 2/6] gallium/vl: use u_upload_mgr to upload vertices for vl_compositor

2014-01-13 Thread Christian König
Am 13.01.2014 14:13, schrieb Marek Olšák: From: Marek Olšák marek.ol...@amd.com This patch is: Reviewed-by: Christian König christian.koe...@amd.com This is the recommended way for streaming vertices. Always use this if you need to upload vertices every frame. ---

[Mesa-dev] [PATCH] mesa-demos: Fixes a bug in demo2 application

2014-01-13 Thread Yasir Khan
From: Yasir-Khan yasir_k...@mentor.com Vertices array is being passed to glColorPointer whereas its supposed to pass the color array Signed-off-by: Yasir-Khan yasir_k...@mentor.com diff --git a/src/egl/opengl/demo2.c b/src/egl/opengl/demo2.c index 71a1a31..505b474 100644 ---

[Mesa-dev] [PATCH] vdpau: flush the context before exporting the surface v2

2014-01-13 Thread Christian König
From: Marek Olšák marek.ol...@amd.com Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=73191 When VL uploads vertex buffers, it uses PIPE_TRANSFER_DONTBLOCK, which always flushes the context in the winsys if the buffer being mapped is busy. Since I added handling of DISCARD_RANGE,

Re: [Mesa-dev] [PATCH] Mark debug_print with __attribute__ ((format(__printf__, 1, 0)))

2014-01-13 Thread Keith Packard
Thierry Reding thierry.red...@gmail.com writes: While at it, perhaps the drmMsg() and drmDebugPrint() functions should be similarily annotated as well? I don't know; I'm just fixing X server warnings this week and this was the source of one of them. Additional warning fixes for drm would be a

Re: [Mesa-dev] [PATCH] vdpau: flush the context before exporting the surface v2

2014-01-13 Thread Marek Olšák
This patch doesn't fix the bug. :( Marek On Mon, Jan 13, 2014 at 2:55 PM, Christian König deathsim...@vodafone.de wrote: From: Marek Olšák marek.ol...@amd.com Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=73191 When VL uploads vertex buffers, it uses PIPE_TRANSFER_DONTBLOCK, which

Re: [Mesa-dev] [PATCH] st/egl: Flush resources before presentation

2014-01-13 Thread Marek Olšák
Pushed. Marek On Tue, Jan 7, 2014 at 11:20 PM, Martin Andersson g02ma...@gmail.com wrote: Hi Marek, Since it seems no one else have any comments on this, maybe you could commit it for me? //Martin On Thu, Dec 26, 2013 at 1:15 PM, Marek Olšák mar...@gmail.com wrote: Reviewed-by: Marek

Re: [Mesa-dev] [PATCH] vdpau: flush the context before exporting the surface v2

2014-01-13 Thread Christian König
Yeah, probably because XBMC still (incorrectly) calls the map function only once. Putting the flush into vlVdpResolveDelayedRendering solves the problem because it's called the next time somebody starts rendering, but it's way to late at this point. Need to sync up with the XBMC devs on

[Mesa-dev] [Bug 72895] Missing trees in flightgear 2.12.1 with r600 driver and mesa 10.0.1

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=72895 --- Comment #7 from Barto mister.free...@laposte.net --- does anyone need more informations about this bug ? because this bug is still here with the new mesa 10.0.2, I did a bisect, the bug begins with 59b01ca252bd6706f08cd80a864819d71dfe741c

[Mesa-dev] [Bug 72895] Missing trees in flightgear 2.12.1 with r600 driver and mesa 10.0.1

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=72895 Alex Deucher ag...@yahoo.com changed: What|Removed |Added CC||e...@anholt.net -- You

[Mesa-dev] [Bug 72895] Missing trees in flightgear 2.12.1 with r600 driver and mesa 10.0.1

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=72895 Igor Gnatenko i.gnatenko.br...@gmail.com changed: What|Removed |Added CC|

Re: [Mesa-dev] [PATCH] Use AC_PATH_TOOL instead of AC_PATH_PROG for llvm-config.

2014-01-13 Thread Tom Stellard
On Sat, Dec 28, 2013 at 03:22:09PM +0100, Michał Górny wrote: This should help with cross-compiling and multilib when $CHOST-specific llvm-config is expected rather than build host default one. It will help us a bit in Gentoo where we've started using i686-pc-linux-gnu-llvm-config for 32-bit

Re: [Mesa-dev] [PATCH 2/5] r300g/compiler/tests: Remove an unused variable

2014-01-13 Thread Tom Stellard
On Mon, Jan 06, 2014 at 11:47:39AM +0200, Lauri Kasanen wrote: On Sun, 5 Jan 2014 18:51:18 -0800 Tom Stellard t...@stellard.net wrote: struct rc_test_file test_file; + struct rc_instruction *inst; unsigned optimizations = 1; unsigned do_full_regalloc = 1; - struct

Re: [Mesa-dev] [PATCH 5/8] glsl: Statically cast parameter exec_node to ir_variable.

2014-01-13 Thread Ian Romanick
On 01/11/2014 02:37 AM, Kenneth Graunke wrote: Formal function parameters are always ir_variable objects, not an arbitrary ir_instruction. So there's no need to dynamically cast here. ...especially since we never bother to check that as_variable doesn't return NULL. Signed-off-by: Kenneth

[Mesa-dev] [Bug 73512] [clover] mesa.icd. should contain full path

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73512 --- Comment #4 from Tom Stellard tstel...@gmail.com --- According to the icd spec: http://www.khronos.org/registry/cl/extensions/khr/cl_khr_icd.txt The vendors directory must go in /etc/OpenCL and also only the library name is included in the

Re: [Mesa-dev] [PATCH 7/8] glsl: Replace iterators in ir_reader.cpp with ad-hoc list walking.

2014-01-13 Thread Ian Romanick
On 01/11/2014 02:37 AM, Kenneth Graunke wrote: These can't use foreach_list since they want to skip over the first few list elements. Just doing the ad-hoc list walking isn't too bad. Signed-off-by: Kenneth Graunke kenn...@whitecape.org --- src/glsl/ir_reader.cpp | 18 ++

Re: [Mesa-dev] Removing exec_list iterators

2014-01-13 Thread Ian Romanick
On 01/11/2014 02:37 AM, Kenneth Graunke wrote: Hello, Here's a long overdue cleanup: removing exec_list_iterator and such. Should be fairly easy to review. I ran Piglit on i965, swrast (which uses ir_to_mesa), and softpipe (which uses st_glsl_to_tgsi). Nothing changed. Patches 1 - 5 and

Re: [Mesa-dev] [PATCH 6/8] glsl: Use a new foreach_list2 macro for walking two lists at once.

2014-01-13 Thread Ian Romanick
On 01/11/2014 02:37 AM, Kenneth Graunke wrote: When handling function calls, we often want to walk through the list of formal parameters and list of actual parameters at the same time. (Both are guaranteed to be the same length.) Previously, we used a pattern of: exec_list_iterator

Re: [Mesa-dev] GPU lockup CP stall when calling clBuildProgram on Cayman

2014-01-13 Thread Tom Stellard
On Thu, Jan 09, 2014 at 02:57:20PM +, christophe choquet wrote: Hi, I am using kernel 3.12.6-gentoo, Mesa 10.0.1 and once every two calls to clBuildProgram, the GPU goes to reset after 10 seconds. This also happens on Debian unstable with Mesa 9.2. First hello_world works, the next

Re: [Mesa-dev] [PATCH] Mark debug_print with __attribute__ ((format(__printf__, 1, 0)))

2014-01-13 Thread Ian Romanick
Reviewed-by: Ian Romanick ian.d.roman...@intel.com On 01/12/2014 10:34 AM, Keith Packard wrote: the drmServerInfo member, debug_print, takes a printf format string and varargs list. Tell the compiler about it. Signed-off-by: Keith Packard kei...@keithp.com --- xf86drm.h | 8 +++- 1

Re: [Mesa-dev] OpenCL Clang/Clover Offline Compilation issue

2014-01-13 Thread Tom Stellard
On Thu, Jan 09, 2014 at 12:49:51PM +, Dorrington, Albert wrote: I am not sure if this is the appropriate list on which to ask this question, if not hopefully someone can suggest an alternative. Under Linux, I am attempting to perform an offline compile of an OpenCL kernel example using

Re: [Mesa-dev] [PATCH] Mark debug_print with __attribute__ ((format(__printf__, 1, 0)))

2014-01-13 Thread Keith Packard
Ian Romanick i...@freedesktop.org writes: Reviewed-by: Ian Romanick ian.d.roman...@intel.com Thanks. Pushed. 8279c8f..cb4bc8e master - master -- keith.pack...@intel.com pgpkKf6zSEV5v.pgp Description: PGP signature ___ mesa-dev mailing list

[Mesa-dev] [Bug 70410] egl-static/Makefile: linking fails with llvm = 3.4

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=70410 --- Comment #18 from Krzysztof A. Sobiecki sob...@gmail.com --- (In reply to comment #17) I've tested attachment 91725 [details] and it works with LLVM 3.5 (r198682) in a clean build enviroment (LLVM packages for Debian from llvm.org/apt). I

Re: [Mesa-dev] [wip 1/9] glsl: memory_writer helper class for data serialization

2014-01-13 Thread Paul Berry
On 2 January 2014 03:58, Tapani Pälli tapani.pa...@intel.com wrote: Class will be used by the shader binary cache implementation. Signed-off-by: Tapani Pälli tapani.pa...@intel.com --- src/glsl/memory_writer.h | 147 +++ 1 file changed, 147

Re: [Mesa-dev] [wip 1/9] glsl: memory_writer helper class for data serialization

2014-01-13 Thread Ian Romanick
On 01/02/2014 03:58 AM, Tapani Pälli wrote: Class will be used by the shader binary cache implementation. Signed-off-by: Tapani Pälli tapani.pa...@intel.com --- src/glsl/memory_writer.h | 147 +++ 1 file changed, 147 insertions(+) create mode

[Mesa-dev] [Bug 73512] [clover] mesa.icd. should contain full path

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73512 --- Comment #5 from Igor Gnatenko i.gnatenko.br...@gmail.com --- (In reply to comment #4) According to the icd spec: http://www.khronos.org/registry/cl/extensions/khr/cl_khr_icd.txt The vendors directory must go in /etc/OpenCL and also only

Re: [Mesa-dev] [RFC] Build testing, wine style

2014-01-13 Thread Tom Stellard
On Sat, Jan 11, 2014 at 03:53:58PM +, Emil Velikov wrote: Hello list, While going though mesa's build systems I was wondering what it would take to improve the overall experience of build testing. The only thing I can think of is a more centralised solution similar to the one used by

[Mesa-dev] [Bug 73512] [clover] mesa.icd. should contain full path

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73512 --- Comment #6 from Fabian Deutsch fabian.deut...@gmx.de --- Hey, this can all be a result of me being uninformed (not knowing that only the library name is contained in the .icd file). But I think that the .icd file is still not corect, as it

[Mesa-dev] [Bug 73512] [clover] mesa.icd. should contain full path

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73512 Igor Gnatenko i.gnatenko.br...@gmail.com changed: What|Removed |Added Attachment #91886|0 |1 is

Re: [Mesa-dev] [PATCH] opencl: improved auto-gen .icd

2014-01-13 Thread Tom Stellard
On Sun, Jan 12, 2014 at 03:08:56AM +0400, Igor Gnatenko wrote: From 5b2bf87f1238e44150492a39f5db0ae90d59459b Mon Sep 17 00:00:00 2001 From: Igor Gnatenko i.gnatenko.br...@gmail.com Date: Sun, 12 Jan 2014 02:09:16 +0400 Subject: [PATCH] opencl: improved auto-gen .icd v2: Use

[Mesa-dev] [PATCH 01/19] nv50/ir: fix PFETCH and add RDSV to get VSTRIDE for GPs

2014-01-13 Thread Ilia Mirkin
From: Christoph Bumiller e0425...@student.tuwien.ac.at --- src/gallium/drivers/nouveau/codegen/nv50_ir.h | 1 + .../drivers/nouveau/codegen/nv50_ir_emit_nv50.cpp | 62 -- .../drivers/nouveau/codegen/nv50_ir_print.cpp | 1 + 3 files changed, 59 insertions(+), 5

[Mesa-dev] [PATCH 09/19] nv50: properly set the PRIMITIVE_ID enable flag when it is a gp input.

2014-01-13 Thread Ilia Mirkin
Signed-off-by: Ilia Mirkin imir...@alum.mit.edu --- src/gallium/drivers/nouveau/nv50/nv50_program.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/gallium/drivers/nouveau/nv50/nv50_program.c b/src/gallium/drivers/nouveau/nv50/nv50_program.c index 78a12e3..f46f240 100644 ---

[Mesa-dev] [RFC PATCH 18/19] nv50: report glsl 1.50 now that gp tests pass

2014-01-13 Thread Ilia Mirkin
Signed-off-by: Ilia Mirkin imir...@alum.mit.edu --- There are still some things that fail -- mostly gl_Layer stuff, and also using gl_PositionID without a gp. src/gallium/drivers/nouveau/nv50/nv50_screen.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[Mesa-dev] [PATCH 14/19] nvc0: don't forget to also clear additional layers

2014-01-13 Thread Ilia Mirkin
Signed-off-by: Ilia Mirkin imir...@alum.mit.edu --- src/gallium/drivers/nouveau/nv50/nv50_program.c | 2 ++ src/gallium/drivers/nouveau/nvc0/nvc0_surface.c | 22 -- 2 files changed, 18 insertions(+), 6 deletions(-) diff --git a/src/gallium/drivers/nouveau/nv50/nv50_program.c

[Mesa-dev] [PATCH 08/19] nv50/ir: add support for gl_PrimitiveIDIn

2014-01-13 Thread Ilia Mirkin
Note that the primitive id is stored in a[0x18], while usually the geometry instructions are of the form a[$a1 + 0x4] which gets mapped to p[] space. We need to avoid the change from a[] to p[] here, so it's keyed on whether the access is indirect or not. Note that there's also a use-case for

[Mesa-dev] [PATCH 03/19] nv50: add support for geometry shaders

2014-01-13 Thread Ilia Mirkin
From: Bryan Cain bryanca...@gmail.com Layer output probably doesn't work yet, but other than that everything seems to be working. Signed-off-by: Bryan Cain bryanca...@gmail.com [calim: fix up minor bugs, code formatting] Signed-off-by: Christoph Bumiller e0425...@student.tuwien.ac.at

[Mesa-dev] [PATCH 02/19] nv50/ir: delay calculation of indirect addresses

2014-01-13 Thread Ilia Mirkin
From: Bryan Cain bryanca...@gmail.com Instead of emitting an SHL 4 io an address register on the TGSI ARL and UARL instructions, emit the shift when the loaded address is actually used. This is necessary because input vertex and attribute indices in geometry shaders on nv50 need to be shifted

[Mesa-dev] [RFC PATCH 19/19] nv50: enable seamless cube maps on all hw for OpenGL 3.2

2014-01-13 Thread Ilia Mirkin
Some of the hardware support is missing. The NVIDIA-provided driver, which claims 3.3 support fails a slew of the relevant tests as well. This allows us to expose geometry shaders without doing the additional work involved in supporting ARB_geometry_shader4. Signed-off-by: Ilia Mirkin

[Mesa-dev] [PATCH 00/19] nv50: add sampler2DMS/GP support to get OpenGL 3.2

2014-01-13 Thread Ilia Mirkin
OK, so there's a bunch of stuff in here. The geometry stuff is based on the work started by Bryan Cain and Christoph Bumiller. Patches 01-12: Add support for geometry shaders and fix related issues Patches 13-14: Make it possible for fb clears to operate on texture attachments with

[Mesa-dev] [PATCH 17/19] nv50: add support for textureFetch'ing MS textures, ARB_texture_multisample

2014-01-13 Thread Ilia Mirkin
Creates two areas in the AUX constbuf: - Sample offsets for MS textures - Per-texture MS settings When executing a textureFetch with a MS sampler, looks up that texture's settings and adjusts the parameters given to the texfetch instruction. With this change, all the ARB_texture_multisample

[Mesa-dev] [PATCH 16/19] nv50: copy nvc0's get_sample_position implementation

2014-01-13 Thread Ilia Mirkin
Signed-off-by: Ilia Mirkin imir...@alum.mit.edu --- src/gallium/drivers/nouveau/nv50/nv50_context.c | 46 + 1 file changed, 46 insertions(+) diff --git a/src/gallium/drivers/nouveau/nv50/nv50_context.c b/src/gallium/drivers/nouveau/nv50/nv50_context.c index

[Mesa-dev] [PATCH 07/19] nv50/ir: comment out code to allow input/immed loads

2014-01-13 Thread Ilia Mirkin
This code was missing a break which made it ineffective. But since shader input loads have been disallowed, define the code out. Signed-off-by: Ilia Mirkin imir...@alum.mit.edu --- src/gallium/drivers/nouveau/codegen/nv50_ir_target_nv50.cpp | 3 +++ 1 file changed, 3 insertions(+) diff --git

[Mesa-dev] [PATCH 06/19] nv50/ir: disallow shader input propagation for gp

2014-01-13 Thread Ilia Mirkin
For some reason, shader input accesses don't work correctly in non-ld instructions. Disallow those loads from being propagated. Signed-off-by: Ilia Mirkin imir...@alum.mit.edu --- I'm not particularly happy with this patch. Some investigation needs to happen as to what's going on here. NVIDIA's

[Mesa-dev] [PATCH 10/19] nv50: VP_RESULT_MAP_SIZE has to be positive

2014-01-13 Thread Ilia Mirkin
Make sure that we never try to use a 0-sized map. This can happen when using a gp, so add a dummy mapping when computing vp_gp_mapping in that case. Signed-off-by: Ilia Mirkin imir...@alum.mit.edu --- src/gallium/drivers/nouveau/nv50/nv50_shader_state.c | 4 1 file changed, 4 insertions(+)

[Mesa-dev] [PATCH 04/19] nv50: allow vert_count to be 255

2014-01-13 Thread Ilia Mirkin
--- src/gallium/drivers/nouveau/nv50/nv50_program.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/nouveau/nv50/nv50_program.h b/src/gallium/drivers/nouveau/nv50/nv50_program.h index 13b9516..f63352f 100644 ---

[Mesa-dev] [PATCH 15/19] nv50: add comments about CB_AUX contents

2014-01-13 Thread Ilia Mirkin
Updates a few inconsistencies as well, like the size of the buffer, location of the runout, etc. Signed-off-by: Ilia Mirkin imir...@alum.mit.edu --- src/gallium/drivers/nouveau/nv50/nv50_context.h| 10 ++ src/gallium/drivers/nouveau/nv50/nv50_screen.c | 8

[Mesa-dev] [PATCH 11/19] nv50: GP_REG_ALLOC_RESULT must be positive

2014-01-13 Thread Ilia Mirkin
Set max_out to 1 when there are no outputs. Signed-off-by: Ilia Mirkin imir...@alum.mit.edu --- src/gallium/drivers/nouveau/nv50/nv50_program.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/drivers/nouveau/nv50/nv50_program.c b/src/gallium/drivers/nouveau/nv50/nv50_program.c

[Mesa-dev] [PATCH 05/19] nv50/ir: disallow predicates on emit/restart ops

2014-01-13 Thread Ilia Mirkin
--- src/gallium/drivers/nouveau/codegen/nv50_ir_target_nv50.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_target_nv50.cpp b/src/gallium/drivers/nouveau/codegen/nv50_ir_target_nv50.cpp index ade9be0..52257a8 100644 ---

Re: [Mesa-dev] [PATCH 7/8] glsl: Replace iterators in ir_reader.cpp with ad-hoc list walking.

2014-01-13 Thread Kenneth Graunke
On 01/13/2014 09:49 AM, Ian Romanick wrote: On 01/11/2014 02:37 AM, Kenneth Graunke wrote: These can't use foreach_list since they want to skip over the first few list elements. Just doing the ad-hoc list walking isn't too bad. Signed-off-by: Kenneth Graunke kenn...@whitecape.org ---

Re: [Mesa-dev] [PATCH 6/8] glsl: Use a new foreach_list2 macro for walking two lists at once.

2014-01-13 Thread Kenneth Graunke
On 01/13/2014 09:58 AM, Ian Romanick wrote: On 01/11/2014 02:37 AM, Kenneth Graunke wrote: When handling function calls, we often want to walk through the list of formal parameters and list of actual parameters at the same time. (Both are guaranteed to be the same length.) Previously, we

[Mesa-dev] [PATCH v3] opencl: improved auto-gen .icd

2014-01-13 Thread Igor Gnatenko
From 91796da9c00c0756b90b9e09d404a5357ff32ec6 Mon Sep 17 00:00:00 2001 From: Igor Gnatenko i.gnatenko.br...@gmail.com Date: Sun, 12 Jan 2014 02:09:16 +0400 Subject: [PATCH] opencl: improved auto-gen .icd v2: Use @OPENCL_VERSION@:0 for library replace /etc with @sysconfdir@ macros v3: Drop

[Mesa-dev] [Bug 73571] New: [clover] OpenCL segfault in gegl 'clones' test

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73571 Priority: medium Bug ID: 73571 Assignee: mesa-dev@lists.freedesktop.org Summary: [clover] OpenCL segfault in gegl 'clones' test Severity: normal Classification: Unclassified

Re: [Mesa-dev] Naming everything in src/gallium/drivers/radeonsi si_*

2014-01-13 Thread Andreas Hartmetz
I don't have an fdo account or push rights. Can somebody else push it for me please? I've added the Reviewed-by: lines so the patches only need to be pushed now. On Monday 13 January 2014 11:22:07 Marek Olšák wrote: For the series: Reviewed-by: Marek Olšák marek.ol...@amd.com Feel free to

[Mesa-dev] [Bug 73571] [clover] OpenCL segfault in gegl 'clones' test

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73571 --- Comment #1 from Jan Vesely jano.ves...@gmail.com --- There are 4 more tests that were failing for different reason, but after applying http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20131216/199497.html segfault in the same way:

Re: [Mesa-dev] [PATCH 05/10] i965: Use Global GTT for Sandybridge post-sync non-zero workaround.

2014-01-13 Thread Eric Anholt
Kenneth Graunke kenn...@whitecape.org writes: On 01/09/2014 10:03 PM, Eric Anholt wrote: Eric Anholt e...@anholt.net writes: Kenneth Graunke kenn...@whitecape.org writes: The kernel doesn't even set up the aliasing PPGTT on Sandybridge, so any writes marked as PPGTT will likely just get

Re: [Mesa-dev] [PATCH 08/10] i965: Introduce an OUT_RELOC64 macro.

2014-01-13 Thread Eric Anholt
Kenneth Graunke kenn...@whitecape.org writes: On 01/09/2014 09:31 PM, Eric Anholt wrote: Kenneth Graunke kenn...@whitecape.org writes: On 12/13/2013 09:28 AM, Daniel Vetter wrote: On Thu, Dec 12, 2013 at 01:26:40AM -0800, Kenneth Graunke wrote: Broadwell uses 48-bit addresses. The first

[Mesa-dev] [Bug 73571] [clover] OpenCL segfault in gegl 'clones' test

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73571 --- Comment #2 from Jan Vesely jano.ves...@gmail.com --- I should have noted that my llvm git includes http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20131216/199497.html. Without these patches the backtrace in clones.xml test looks

Re: [Mesa-dev] [PATCH] i965: Use sample barycentric coordinates with per sample shading

2014-01-13 Thread Anuj Phogat
On Fri, Jan 10, 2014 at 5:25 PM, Anuj Phogat anuj.pho...@gmail.com wrote: On Thu, Jan 9, 2014 at 4:34 PM, Chris Forbes chr...@ijw.co.nz wrote: Hi Anuj, There's one fiddly interaction that I don't think this handles quite right, although I think it does conform. Suppose we have this fragment

Re: [Mesa-dev] EXTERNAL: Re: OpenCL Clang/Clover Offline Compilation issue

2014-01-13 Thread Dorrington, Albert
Tom, Thanks for your response. I am very interested in implementing this, so any pointers you can provide would be greatly appreciated. I don't have access to IRC at work (at least I doubt I do) due to firewalls - but I can use the mailing list. I wasn't entirely sure about the proper clang

Re: [Mesa-dev] EXTERNAL: Re: OpenCL Clang/Clover Offline Compilation issue

2014-01-13 Thread Tom Stellard
On Mon, Jan 13, 2014 at 06:44:15PM +, Dorrington, Albert wrote: Tom, Thanks for your response. I am very interested in implementing this, so any pointers you can provide would be greatly appreciated. I'm cc'ing Fransisco since he may also have some feedback. The first step is to build

[Mesa-dev] [Bug 73512] [clover] mesa.icd. should contain full path

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73512 --- Comment #8 from Tom Stellard tstel...@gmail.com --- (In reply to comment #7) Created attachment 91973 [details] [review] [PATCH v3] opencl: improved auto-gen .icd v2: Use @OPENCL_VERSION@:0 for library replace /etc with @sysconfdir@

[Mesa-dev] How to contribute a translation?

2014-01-13 Thread Alex Henrie
Hi, I'd like to translate the DRI driver options (src/mesa/drivers/dri/common/xmlpool) to the Catalan language. What is the procedure for adding new translations? What tool should I use to generate ca.po, and how do I submit the file for review? -Alex

Re: [Mesa-dev] [PATCH 08/10] i965: Introduce an OUT_RELOC64 macro.

2014-01-13 Thread Kenneth Graunke
On 01/13/2014 01:04 PM, Eric Anholt wrote: Kenneth Graunke kenn...@whitecape.org writes: On 01/09/2014 09:31 PM, Eric Anholt wrote: Kenneth Graunke kenn...@whitecape.org writes: On 12/13/2013 09:28 AM, Daniel Vetter wrote: On Thu, Dec 12, 2013 at 01:26:40AM -0800, Kenneth Graunke wrote:

[Mesa-dev] [Bug 72926] Memory corruption (crash) in draw/draw_pt_fetch_shade_pipeline_llvm.c:435

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=72926 Peter Wu lekenst...@gmail.com changed: What|Removed |Added Attachment #91053|0 |1 is obsolete|

[Mesa-dev] [Bug 73512] [clover] mesa.icd. should contain full path

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73512 --- Comment #9 from Igor Gnatenko i.gnatenko.br...@gmail.com --- (In reply to comment #8) (In reply to comment #7) Created attachment 91973 [details] [review] [review] [PATCH v3] opencl: improved auto-gen .icd v2: Use @OPENCL_VERSION@:0

[Mesa-dev] [Bug 72926] Memory corruption (crash) in draw/draw_pt_fetch_shade_pipeline_llvm.c:435

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=72926 Peter Wu lekenst...@gmail.com changed: What|Removed |Added Attachment #91216|0 |1 is obsolete|

Re: [Mesa-dev] [wip 2/9] glsl: serialize methods for IR instructions

2014-01-13 Thread Paul Berry
On 2 January 2014 03:58, Tapani Pälli tapani.pa...@intel.com wrote: diff --git a/src/glsl/ir_serialize.cpp b/src/glsl/ir_serialize.cpp new file mode 100644 index 000..30ca018 --- /dev/null +++ b/src/glsl/ir_serialize.cpp @@ -0,0 +1,392 @@ +/* -*- c++ -*- */ +/* + * Copyright © 2013

[Mesa-dev] [Bug 73512] [clover] mesa.icd. should contain full path

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73512 Igor Gnatenko i.gnatenko.br...@gmail.com changed: What|Removed |Added Attachment #91973|0 |1 is

Re: [Mesa-dev] [PATCH] automake: include the git sha in the opengl version string for oot builds

2014-01-13 Thread Chad Versace
On Mon, Jan 13, 2014 at 02:02:12AM +, Emil Velikov wrote: Because it's a great feature and we should not penalise people for doing out-of-tree builds. Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/mesa/Makefile.am | 12 ++-- 1 file changed, 6 insertions(+), 6

Re: [Mesa-dev] [PATCH 3/7] mesa: implement glBindTextures()

2014-01-13 Thread Ian Romanick
On 01/07/2014 12:05 AM, Fredrik Höglund wrote: On Friday 03 January 2014, Marek Olšák wrote: On Fri, Jan 3, 2014 at 2:04 PM, Marek Olšák mar...@gmail.com wrote: On Fri, Jan 3, 2014 at 1:27 AM, Maxence Le Doré maxence.led...@gmail.com wrote: --- src/mesa/main/texobj.c | 52

Re: [Mesa-dev] [wip 3/9] glsl: memory_map helper class for data deserialization

2014-01-13 Thread Paul Berry
On 2 January 2014 03:58, Tapani Pälli tapani.pa...@intel.com wrote: Class will be used by the shader binary cache implementation. Signed-off-by: Tapani Pälli tapani.pa...@intel.com --- src/glsl/memory_map.h | 174 ++ 1 file changed, 174

[Mesa-dev] Mesa 10.1 release plan strawman

2014-01-13 Thread Ian Romanick
Fast forwarding 3 months from the 10.0 release (November 30th) is February 28th. I'd like to propose the following set of dates: January 31st: Feature freeze / 10.1 branch created. I promise to not let anyone on my team (myself included) dump any giant commit series the day of the freeze. I'll

[Mesa-dev] [Bug 72926] Memory corruption (crash) in draw/draw_pt_fetch_shade_pipeline_llvm.c:435

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=72926 Peter Wu lekenst...@gmail.com changed: What|Removed |Added CC||lekenst...@gmail.com ---

Re: [Mesa-dev] [PATCH] Use AC_PATH_TOOL instead of AC_PATH_PROG for llvm-config.

2014-01-13 Thread Tom Stellard
On Mon, Jan 13, 2014 at 07:04:44PM +0100, Michał Górny wrote: Dnia 2014-01-13, o godz. 08:59:22 Tom Stellard t...@stellard.net napisał(a): On Sat, Dec 28, 2013 at 03:22:09PM +0100, Michał Górny wrote: This should help with cross-compiling and multilib when $CHOST-specific llvm-config

[Mesa-dev] [Bug 72926] Memory corruption (crash) in draw/draw_pt_fetch_shade_pipeline_llvm.c:435

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=72926 Peter Wu lekenst...@gmail.com changed: What|Removed |Added CC||za...@vmware.com -- You

[Mesa-dev] [Bug 73571] [clover] OpenCL segfault in gegl 'clones' test

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73571 --- Comment #3 from Jan Vesely jano.ves...@gmail.com --- Created attachment 92006 -- https://bugs.freedesktop.org/attachment.cgi?id=92006action=edit Don't crash on NULL global mem objects The attached patch fixes the original issue (bt in

[Mesa-dev] [Bug 73571] [clover] Add support for NULL global memory object arguments

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73571 Jan Vesely jano.ves...@gmail.com changed: What|Removed |Added Summary|[clover] OpenCL segfault in |[clover] Add support

[Mesa-dev] [Bug 73512] [clover] mesa.icd. should contain full path

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73512 --- Comment #11 from Igor Gnatenko i.gnatenko.br...@gmail.com --- (In reply to comment #8) (In reply to comment #7) Created attachment 91973 [details] [review] [review] [PATCH v3] opencl: improved auto-gen .icd v2: Use

Re: [Mesa-dev] [PATCH] opencl: improved auto-gen .icd

2014-01-13 Thread Matt Turner
On Mon, Jan 13, 2014 at 11:12 AM, Tom Stellard t...@stellard.net wrote: On Sun, Jan 12, 2014 at 03:08:56AM +0400, Igor Gnatenko wrote: From 5b2bf87f1238e44150492a39f5db0ae90d59459b Mon Sep 17 00:00:00 2001 From: Igor Gnatenko i.gnatenko.br...@gmail.com Date: Sun, 12 Jan 2014 02:09:16 +0400

Re: [Mesa-dev] Naming everything in src/gallium/drivers/radeonsi si_*

2014-01-13 Thread Marek Olšák
Pushed, thanks. Marek On Mon, Jan 13, 2014 at 10:00 PM, Andreas Hartmetz ahartm...@gmail.com wrote: I don't have an fdo account or push rights. Can somebody else push it for me please? I've added the Reviewed-by: lines so the patches only need to be pushed now. On Monday 13 January 2014

Re: [Mesa-dev] gallium endianness and hw drivers

2014-01-13 Thread Marek Olšák
I think the format conversion functions should look like: #ifdef BIG_ENDIAN case PIPE_FORMAT_A8B8G8R8_UNORM: return hw_format_for_R8G8B8A8_UNORM; ... #else case PIPE_FORMAT_R8G8B8A8_UNORM: return hw_format_for_R8G8B8A8_UNORM; #endif which can be simplified to: case

Re: [Mesa-dev] [wip 6/9] glsl: ir_deserializer class for the binary shader cache

2014-01-13 Thread Paul Berry
On 2 January 2014 03:58, Tapani Pälli tapani.pa...@intel.com wrote: + + +/** + * Reads header part of the binary blob. Main purpose of this header is to + * validate that cached shader was produced with same Mesa driver version. + */ +int +ir_deserializer::read_header(struct gl_shader

[Mesa-dev] [Bug 73578] New: egl_pipe.c:46:38: fatal error: radeonsi/radeonsi_public.h: No such file or directory

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73578 Priority: medium Bug ID: 73578 Keywords: regression CC: ahartm...@gmail.com, mar...@gmail.com Assignee: mesa-dev@lists.freedesktop.org Summary: egl_pipe.c:46:38: fatal error:

[Mesa-dev] [Bug 73578] egl_pipe.c:46:38: fatal error: radeonsi/radeonsi_public.h: No such file or directory

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73578 --- Comment #1 from Vinson Lee v...@freedesktop.org --- 786af2f963925df2c2a6fb60b29a83e8340f03c7 is the first bad commit commit 786af2f963925df2c2a6fb60b29a83e8340f03c7 Author: Andreas Hartmetz ahartm...@gmail.com Date: Sat Jan 4 18:44:33 2014

[Mesa-dev] [libdrm PATCH] intel: Create a new drm_intel_bo offset64 field.

2014-01-13 Thread Kenneth Graunke
The existing 'offset' field is unfortunately typed as 'unsigned long', which is unfortunately only 4 bytes with a 32-bit userspace. Traditionally, the hardware has only supported 32-bit virtual addresses, so even though the kernel uses a __u64, the value would always fit. However, Broadwell

[Mesa-dev] [Mesa PATCH 3/3] i965: Introduce an OUT_RELOC64 macro.

2014-01-13 Thread Kenneth Graunke
Broadwell uses 48-bit addresses. The first DWord is the low 32 bits, and the second DWord is the high 16 bits. Cc: Eric Anholt e...@anholt.net Cc: Daniel Vetter daniel.vet...@ffwll.ch Cc: Ben Widawsky b...@bwidawsk.net Signed-off-by: Kenneth Graunke kenn...@whitecape.org ---

Re: [Mesa-dev] FOSDEM14: Graphics DevRoom: Deadline approaching fast.

2014-01-13 Thread Luc Verhaegen
On Tue, Jan 07, 2014 at 02:22:00AM +0100, Luc Verhaegen wrote: Hi, There are still 5 slots open for the FOSDEM graphics DevRoom, and the deadline is this friday, the 10th. Get a move on. If you have requested an account reset with me before, but if you then haven't bothered filing a

Re: [Mesa-dev] [PATCH] Use AC_PATH_TOOL instead of AC_PATH_PROG for llvm-config.

2014-01-13 Thread Michał Górny
Dnia 2014-01-13, o godz. 08:59:22 Tom Stellard t...@stellard.net napisał(a): On Sat, Dec 28, 2013 at 03:22:09PM +0100, Michał Górny wrote: This should help with cross-compiling and multilib when $CHOST-specific llvm-config is expected rather than build host default one. It will help us

[Mesa-dev] [Bug 73578] egl_pipe.c:46:38: fatal error: radeonsi/radeonsi_public.h: No such file or directory

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73578 Vinson Lee v...@freedesktop.org changed: What|Removed |Added Status|NEW |RESOLVED

[Mesa-dev] Nominations for X.Org Foundation Board of Directors are OPEN

2014-01-13 Thread Martin Peres
We are seeking nominations for candidates for election to the X.Org Foundation Board of Directors. All X.Org Foundation members are eligible for election to the board. Nominations for the 2014 election are now open and will remain open until 23.59 GMT on 12 February 2013. The Board consists of

[Mesa-dev] [Mesa PATCH 2/3] i965: Use the new drm_intel_bo offset64 field.

2014-01-13 Thread Kenneth Graunke
libdrm 2.4.52 introduces a new 'uint64_t offset64' field, intended to replace the old 'unsigned long offset' field. To preserve ABI, libdrm continues to store the presumed offset in both locations. On Broadwell, a 64-bit kernel may place BOs at high ( 4G) addresses. However, with a 32-bit

  1   2   >