Re: [Mesa-dev] [PATCH] build: remove forced -fno-rtti

2013-10-16 Thread Jan Vesely
directory, while I can see in */flags.make (in build dir) and */link.txt that CXX_FLAGS do include -fno-rtti after compiling and installing llvm in local dir the output of llvm-config --cxx-flags is without -fno-rtti regards, Jan Vesely -- Alex

Re: [Mesa-dev] [PATCH] build: remove forced -fno-rtti

2013-10-17 Thread Jan Vesely
-D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -fno-rtti Hi, this patch fixes the issue for me as well. Would you mind attaching it to to http://llvm.org/bugs/show_bug.cgi?id=14200 ? thank you, Jan Vesely ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http

[Mesa-dev] [PATCH] clover: Refuse to create context with invalid properties

2013-10-21 Thread Jan Vesely
the specs say that clCreateContext reutrns error if platform value specified in properties is not a valid platform The orignal approach fials if invalid valu other than NULL pointer is provided. Fixes piglit cl-api-create-context. Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- src

Re: [Mesa-dev] [PATCH] clover: Refuse to create context with invalid properties

2013-10-22 Thread Jan Vesely
On Mon, 2013-10-21 at 22:20 -0700, Francisco Jerez wrote: Jan Vesely jan.ves...@rutgers.edu writes: the specs say that clCreateContext reutrns error if platform value specified in properties is not a valid platform The orignal approach fials if invalid valu other than NULL pointer

Re: [Mesa-dev] [PATCH] clover: Refuse to create context with invalid properties

2013-10-24 Thread Jan Vesely
On Tue, 2013-10-22 at 13:38 -0700, Francisco Jerez wrote: Jan Vesely jan.ves...@rutgers.edu writes: On Mon, 2013-10-21 at 22:20 -0700, Francisco Jerez wrote: Jan Vesely jan.ves...@rutgers.edu writes: the specs say that clCreateContext reutrns error if platform value specified

[Mesa-dev] [PATCH 0/3] Buffer creation and consistency

2013-12-15 Thread Jan Vesely
Hi, this series fixes clCreatBUffer and buffer-flags piglit tests. The first two patches add just error checking, the last patch fixes memory initialization when using CL_MEM_USE_HOST_PTR flag. regards, Jan Jan Vesely (3): clover: Check invalid flag combinations in clCreateBuffer. clover

[Mesa-dev] [PATCH 3/3] clover: Append buffers that use CL_MEM_USE_HOST_PTR

2013-12-15 Thread Jan Vesely
Specs say it's legal for implementations to use internal copies, and the write synchronization seems to work. Fixes clCreateBuffer (together with previous patches) and buffer-flags piglits. Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- src/gallium/state_trackers/clover/core/memory.cpp

[Mesa-dev] [PATCH 2/3] clover: Check allocation size against context maximum

2013-12-15 Thread Jan Vesely
From: Jan Vesely jan.ves...@cs.rutgers.edu Check maximum size across all devices in the context Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- src/gallium/state_trackers/clover/api/memory.cpp | 2 +- src/gallium/state_trackers/clover/core/context.cpp | 11 +++ src/gallium

[Mesa-dev] [PATCH 1/3] clover: Check invalid flag combinations in clCreateBuffer.

2013-12-15 Thread Jan Vesely
Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- src/gallium/state_trackers/clover/api/memory.cpp | 15 +++ 1 file changed, 15 insertions(+) diff --git a/src/gallium/state_trackers/clover/api/memory.cpp b/src/gallium/state_trackers/clover/api/memory.cpp index 785a509..15f5b7f

[Mesa-dev] [PATCH v2 1/1] clover: Add parameter checks to clCreateBuffer.

2013-12-17 Thread Jan Vesely
v2: use fewer if statements and functional tricks instead of single-use method, suggested by Francisco Jerez squash two small patches into one Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- Hi, this is v2 of the first two patches incorporating Francisco's comments, i.e. it's

[Mesa-dev] [PATCH v2 1/1] R600: Allow ftrunc

2013-12-18 Thread Jan Vesely
v2: Add ftrunc-TRUNC pattern instead of replacing int_AMDGPU_trunc Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- lib/Target/R600/AMDGPUISelLowering.cpp | 2 ++ lib/Target/R600/R600Instructions.td| 2 ++ 2 files changed, 4 insertions(+) diff --git a/lib/Target/R600

[Mesa-dev] [PATCH 1/1] R600: Allow ftrunc

2013-12-18 Thread Jan Vesely
v2: Add ftrunc-TRUNC pattern instead of replacing int_AMDGPU_trunc v3: move ftrunc pattern next to TRUNC definition, it's available since R600 Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- lib/Target/R600/AMDGPUISelLowering.cpp | 2 ++ lib/Target/R600/R600Instructions.td| 3 +++ 2

[Mesa-dev] [PATCH RFC 1/1] clover: Don't crash on NULL global buffer objects.

2014-01-14 Thread Jan Vesely
Specs say it's legal. Fixes clones.xml gegl test. --- Hi, this patch is an attempt to support NULL buffer objects in clover. It adds NULL handling to few places, and it's enough to get 'clones' gegl test running. The specs say: If the argument is a buffer object, the arg_value pointer can be

[Mesa-dev] [PATCH v2 RFC 1/1] clover: Don't crash on NULL global buffer objects.

2014-01-15 Thread Jan Vesely
NULL into input buffer instead of buffer handle pair Fix constant_argument too Drop r600 driver changes Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- Hi Francisco, I could not find much info about how ctx.input is supposed to be used. It looks like it stores begin-end intervals

[Mesa-dev] [PATCH v3 1/1] clover: Don't crash on NULL global buffer objects.

2014-01-16 Thread Jan Vesely
NULL into input buffer instead of buffer handle pair Fix constant_argument too Drop r600 driver changes v3: Fix inserting NULL pointer Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- src/gallium/state_trackers/clover/core/kernel.cpp | 34 +++ 1 file changed, 23

Re: [Mesa-dev] [PATCH v2 RFC 1/1] clover: Don't crash on NULL global buffer objects.

2014-01-16 Thread Jan Vesely
On Wed, 2014-01-15 at 18:41 +0100, Francisco Jerez wrote: Jan Vesely jan.ves...@rutgers.edu writes: Specs say If the argument is a buffer object, the arg_value pointer can be NULL or point to a NULL value in which case a NULL value will be used as the value for the argument declared

Re: [Mesa-dev] [PATCH v3 1/1] clover: Don't crash on NULL global buffer objects.

2014-01-20 Thread Jan Vesely
On Sat, 2014-01-18 at 13:34 +0100, Francisco Jerez wrote: Jan Vesely jan.ves...@rutgers.edu writes: Specs say If the argument is a buffer object, the arg_value pointer can be NULL or point to a NULL value in which case a NULL value will be used as the value for the argument declared

Re: [Mesa-dev] [PATCH 2/2] pipe-loader: Add support for render nodes v2

2014-01-30 Thread Jan Vesely
mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev -- Jan Vesely jan.ves...@rutgers.edu

Re: [Mesa-dev] [PATCH 2/2] pipe-loader: Add support for render nodes v2

2014-01-30 Thread Jan Vesely
On Thu, 2014-01-30 at 15:52 -0500, Tom Stellard wrote: On Thu, Jan 30, 2014 at 01:55:40PM -0500, Jan Vesely wrote: Hi, are these patches the last thing necessary to get CL programs running without needing X permission? You also need a 3.12 or newer kernel and you must enable render

Re: [Mesa-dev] [PATCH 1/3] r600g, radeonsi: use a fallback in dma_copy instead of failing

2014-03-11 Thread Jan Vesely
si_set_occlusion_query_state(struct pipe_context *ctx, bool enable) This patch together with 2/3 also fixes ~900 opencl piglit regressions introduced by f112ba03bbd6072df9f8879bb231f7f0abb14d2e radeon: Use upload manager for buffer downloads you can add my tested by regards, -- Jan Vesely jan.ves

Re: [Mesa-dev] [PATCH 1/3] r600g, radeonsi: use a fallback in dma_copy instead of failing

2014-03-11 Thread Jan Vesely
On Tue, 2014-03-11 at 11:37 -0400, Jan Vesely wrote: On Tue, 2014-03-11 at 15:09 +0100, Marek Olšák wrote: From: Marek Olšák marek.ol...@amd.com v2: - allow byte-aligned DMA buffer copies on Evergreen - fix piglit/texsubimage regression - use the fallback for 3D copies (depth

Re: [Mesa-dev] [PATCH 1/3] r600g, radeonsi: use a fallback in dma_copy instead of failing

2014-03-11 Thread Jan Vesely
On Tue, 2014-03-11 at 12:49 -0400, Tom Stellard wrote: On Tue, Mar 11, 2014 at 12:37:15PM -0400, Jan Vesely wrote: On Tue, 2014-03-11 at 11:37 -0400, Jan Vesely wrote: On Tue, 2014-03-11 at 15:09 +0100, Marek Olšák wrote: From: Marek Olšák marek.ol...@amd.com v2: - allow byte

[Mesa-dev] 3 element vectors in opencl 1.1+

2014-04-21 Thread Jan Vesely
elements in libclc, it caused clang to complain in thousand places. I don't think this can be handled cleanly in clang, unless we add something like __attribute__((padding)). I have attached a workaround that I use now. any advice welcome, Jan -- Jan Vesely jan.ves...@rutgers.edu diff --git a/src

Re: [Mesa-dev] [cfe-dev] 3 element vectors in opencl 1.1+

2014-04-22 Thread Jan Vesely
On Tue, 2014-04-22 at 14:40 -0700, Matt Arsenault wrote: On 04/22/2014 02:35 PM, Tom Stellard wrote: On Mon, Apr 21, 2014 at 10:02:27PM -0400, Jan Vesely wrote: Hi, I ran into a problem caused by this part of the OCL specs (6.1.5 Alignment of Types): For 3-component vector data types

Re: [Mesa-dev] [cfe-dev] 3 element vectors in opencl 1.1+

2014-04-23 Thread Jan Vesely
and using padding was just a tool to do the job. anyway, thanks for the hint, using getTypeAllocSize works nicely. since we are allocating space in the argument vector I think getAllocSize is the right function to use. I'll post a patch. regards, Jan -- Jan Vesely jan.ves...@rutgers.edu

[Mesa-dev] [PATCH 1/1] clover: use getTypeAllocSize() for kernel arguments

2014-04-23 Thread Jan Vesely
3 element vectors have the size of 4 element ones. See Ch 6.1.5 of OCL 1.1+ specs Thx Matt Arsenault for the hint Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- src/gallium/state_trackers/clover/llvm/invocation.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src

Re: [Mesa-dev] [cfe-dev] 3 element vectors in opencl 1.1+

2014-04-23 Thread Jan Vesely
On Wed, 2014-04-23 at 19:49 +0200, Francisco Jerez wrote: Jan Vesely jan.ves...@rutgers.edu writes: On Tue, 2014-04-22 at 17:50 -0700, Matt Arsenault wrote: snip I think this is what v96:128 is for according to [0], it specifies only alignment, not size. I could not find

Re: [Mesa-dev] [cfe-dev] 3 element vectors in opencl 1.1+

2014-04-24 Thread Jan Vesely
the following is illegal in both: 0x8: float3 here So, I don't think that the quoted text requires all builtin types to have 2^x size (although all but 3 element vectors do). regards, Jan regards, Jan Thanks. Thanks. regards, Jan -- Jan Vesely jan.ves

Re: [Mesa-dev] [cfe-dev] 3 element vectors in opencl 1.1+

2014-04-24 Thread Jan Vesely
On Thu, 2014-04-24 at 17:53 +0200, Francisco Jerez wrote: Jan Vesely jan.ves...@rutgers.edu writes: On Thu, 2014-04-24 at 12:05 +0200, Francisco Jerez wrote: snip I don't think that using getTypeAllocSize() instead of getTypeStoreSize() to calculate clover::argument::size would

[Mesa-dev] [PATCH 1/1] clover: Align kernel argument sizes to nearest power of 2

2014-04-24 Thread Jan Vesely
Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- Hi, this is the alternative (power of 2) approach to hadling kernel args, as discussed in this thread: http://lists.freedesktop.org/archives/mesa-dev/2014-April/thread.html#58209 Jan src/gallium/state_trackers/clover/llvm/invocation.cpp | 4

[Mesa-dev] [PATCH 1/2] r600: Fix possible endless loop in compute_memory_pool allocations.

2014-06-19 Thread Jan Vesely
-by: Jan Vesely jan.ves...@rutgers.edu CC: Bruno Jimenez brunoji...@gmail.com --- This fixes hang in gegl colors.xml test src/gallium/drivers/r600/compute_memory_pool.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/r600/compute_memory_pool.c b/src

[Mesa-dev] [PATCH 2/2] r600: Handle failures in compute_memory_pool_finalize

2014-06-19 Thread Jan Vesely
Signed-off-by: Jan Vesely jan.ves...@rutgers.edu CC: Bruno Jimenez brunoji...@gmail.com --- The failure now hits assertion compute_memory_pool.c:408, instead of u_inlines.h:275:pipe_buffer_map_range: Assertion `offset buffer-width0' src/gallium/drivers/r600/evergreen_compute.c | 9 +++-- 1

Re: [Mesa-dev] [PATCH 2/2] r600: Handle failures in compute_memory_pool_finalize

2014-06-19 Thread Jan Vesely
. regards, Jan Thanks! Bruno On Thu, 2014-06-19 at 10:21 -0400, Jan Vesely wrote: Signed-off-by: Jan Vesely jan.ves...@rutgers.edu CC: Bruno Jimenez brunoji...@gmail.com --- The failure now hits assertion compute_memory_pool.c:408, instead of u_inlines.h:275

Re: [Mesa-dev] [PATCH 1/2] r600: Fix possible endless loop in compute_memory_pool allocations.

2014-06-19 Thread Jan Vesely
] on mesa/clover/r600. you 'll need babl git[1] too. Let me know if I can help with setting it up. Sry, I don't have any smaller reproducer. Jan [0] git://git.gnome.org/gegl [1] git://git.gnome.org/babl On Thu, 2014-06-19 at 10:21 -0400, Jan Vesely wrote: The important part is the change

Re: [Mesa-dev] [PATCH 1/2] r600: Fix possible endless loop in compute_memory_pool allocations.

2014-06-19 Thread Jan Vesely
On Thu, 2014-06-19 at 08:27 -0700, Tom Stellard wrote: On Thu, Jun 19, 2014 at 10:21:32AM -0400, Jan Vesely wrote: The important part is the change of the condition to = 0. Otherwise the loop gets stuck never actually growing the pool. The change in the aux-need calculation guarantees

Re: [Mesa-dev] [PATCH 2/2] r600: Handle failures in compute_memory_pool_finalize

2014-06-19 Thread Jan Vesely
On Thu, 2014-06-19 at 08:34 -0700, Tom Stellard wrote: On Thu, Jun 19, 2014 at 11:22:28AM -0400, Jan Vesely wrote: On Thu, 2014-06-19 at 17:12 +0200, Bruno Jimenez wrote: Hi, To which failure are you refering? Could you please send me a test/program that I can try to track

Re: [Mesa-dev] [PATCH 1/2] r600: Fix possible endless loop in compute_memory_pool allocations.

2014-06-19 Thread Jan Vesely
On Thu, 2014-06-19 at 18:22 +0200, Bruno Jimenez wrote: On Thu, 2014-06-19 at 11:58 -0400, Jan Vesely wrote: On Thu, 2014-06-19 at 08:27 -0700, Tom Stellard wrote: On Thu, Jun 19, 2014 at 10:21:32AM -0400, Jan Vesely wrote: The important part is the change of the condition to = 0

Re: [Mesa-dev] Testing OpenCL with gegl

2014-06-20 Thread Jan Vesely
', 'accelerator' HTH, let me know if you have further questions Jan Thanks in advance! Bruno -- Jan Vesely jan.ves...@rutgers.edu signature.asc Description: This is a digitally signed message part ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org

Re: [Mesa-dev] Testing OpenCL with gegl

2014-06-20 Thread Jan Vesely
On Fri, 2014-06-20 at 19:46 +0200, Bruno Jimenez wrote: On Fri, 2014-06-20 at 09:31 -0400, Jan Vesely wrote: On Fri, 2014-06-20 at 10:27 +0200, Bruno Jimenez wrote: Hi, Hi Bruno, I have added mesa list to CC as others might be interested. Hope you don't mind. Sure, no problem

Re: [Mesa-dev] Testing OpenCL with gegl

2014-06-21 Thread Jan Vesely
I readded the mesa list again, as the info might be interesting to others. On Sat, 2014-06-21 at 11:50 +0200, Bruno Jimenez wrote: On Fri, 2014-06-20 at 17:01 -0400, Jan Vesely wrote: On Fri, 2014-06-20 at 21:58 +0200, Bruno Jimenez wrote: [snip] I also found a reference to BABL_PATH env

Re: [Mesa-dev] Testing OpenCL with gegl

2014-06-21 Thread Jan Vesely
doesnt seem to be used at all. If i make all tests, all opencl tests are skipped and one (non-opencl i suppose) even fails (gegl.xml). Any ideas how i could use opencl? -- Jan Vesely jan.ves...@rutgers.edu signature.asc Description: This is a digitally signed message part

[Mesa-dev] [PATCH 1/1] r600: Fix use after free in compute_memory_promote_item.

2014-06-23 Thread Jan Vesely
The dst pointer needs to be initialized after any calls to compute_memory_grow_pool, as the function might change the pool-vbo pointer. This fixes crashes and assertion failures in two gegl tests. Signed-off-by: Jan Vesely jan.ves...@rutgers.edu CC: Bruno Jimenez brunoji...@gmail.com CC: Tom

Re: [Mesa-dev] [PATCH 1/1] r600: Fix use after free in compute_memory_promote_item.

2014-06-24 Thread Jan Vesely
On Mon, 2014-06-23 at 17:15 +0200, Bruno Jimenez wrote: On Mon, 2014-06-23 at 10:39 -0400, Jan Vesely wrote: The dst pointer needs to be initialized after any calls to compute_memory_grow_pool, as the function might change the pool-vbo pointer. You are completely right. Good catch

Re: [Mesa-dev] Updated debdiff for mesa to compile on m68k

2014-07-17 Thread Jan Vesely
? the attached program produces the following output on my x64 machine: natural: size 8, offset c: 0, offset i: 4 explicit 8: size 16, offset c: 0, offset i: 8 explicit 2: size 8, offset c: 0, offset i: 4 and I get the same output on arm32. regards, Jan bye, //mirabilos -- Jan Vesely jan.ves

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

2014-07-24 Thread Jan Vesely
mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev -- Jan Vesely jan.ves...@rutgers.edu From 48536e44883f8f3f4846235dc60982328cfc6805 Mon Sep 17 00:00:00 2001 From: Jan Vesely jan.ves...@rutgers.edu Date: Thu, 24 Jul 2014 09:58:40 -0400 Subject

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

2014-07-24 Thread Jan Vesely
cache of 64 128 bit constants, which uses the CR mentioned above, and I believe is used to pass kernel arguments by clover. I know little about OpenGL, what are these different ways to do the same thing good for? thanks Jan Marek On Thu, Jul 24, 2014 at 4:05 PM, Jan Vesely jan.ves

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

2014-07-24 Thread Jan Vesely
On Thu, 2014-07-24 at 13:52 -0400, Alex Deucher wrote: On Thu, Jul 24, 2014 at 12:05 PM, Jan Vesely jan.ves...@rutgers.edu wrote: On Thu, 2014-07-24 at 17:07 +0200, Marek Olšák wrote: Sorry, GL 3.1 actually only requires 1024 vec4s. The UBO extension spec contains a mistake. Marek

[Mesa-dev] [PATCH 1/3] r600g/compute: Fix singed/unsigned comparison compiler warnings.

2014-07-25 Thread Jan Vesely
The iteration variables go from 0 anyway. Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- A collection of fixes for gcc warnings I ran across. src/gallium/drivers/r600/evergreen_compute.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/src/gallium

[Mesa-dev] [PATCH 2/3] mesa/st: Fix compiler warnings

2014-07-25 Thread Jan Vesely
both array and index are unsigned types Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp b/src/mesa/state_tracker

[Mesa-dev] [PATCH 3/3] gallium: Fix compiler warning.

2014-07-25 Thread Jan Vesely
warning: type qualifiers ignored on function return type Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- src/gallium/auxiliary/tgsi/tgsi_ureg.c | 2 +- src/gallium/auxiliary/tgsi/tgsi_ureg.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/auxiliary/tgsi

[Mesa-dev] [PATCH 1/1] r600: Use llvm intrinsic to read work dimension information

2014-07-30 Thread Jan Vesely
Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- generic/include/clc/workitem/get_work_dim.h | 1 + r600/lib/SOURCES| 1 + r600/lib/workitem/get_work_dim.ll | 6 ++ 3 files changed, 8 insertions(+) create mode 100644 generic/include/clc/workitem

[Mesa-dev] [PATCH 1/1] R600: Add new intrinsic to read work dimensions

2014-07-30 Thread Jan Vesely
CC: Tom Stellard t...@stellard.net CC: Matt Arsenault matthew.arsena...@amd.com Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- include/llvm/IR/IntrinsicsR600.td| 2 ++ lib/Target/R600/R600ISelLowering.cpp | 6 -- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git

[Mesa-dev] [PATCH 1/2] gallium: Add dimension parameter to launch_grid

2014-07-30 Thread Jan Vesely
This is needed for OpenCL CC: Francisco Jerez curroje...@riseup.net CC: Tom Stellard t...@stellard.net Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- I tried to find another way how to get the information, but in the end there is no way to distinguish between 1,1,1 dim 1, and 1,1,1 dim 2

[Mesa-dev] [PATCH 2/2] r600g: Pass dimension parameter to compute shader.

2014-07-30 Thread Jan Vesely
Make the function static. This needs corresponding change in LLVM otherwise it breaks parameter passing CC: Tom Stellard t...@stellard.net CC: Matt Arsenault matthew.arsena...@amd.com Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- src/gallium/drivers/r600/evergreen_compute.c | 26

Re: [Mesa-dev] [PATCH 2/2] r600g: Pass dimension parameter to compute shader.

2014-07-31 Thread Jan Vesely
On Thu, 2014-07-31 at 11:02 -0400, Tom Stellard wrote: On Wed, Jul 30, 2014 at 07:11:35PM -0400, Jan Vesely wrote: Make the function static. No need to cc llvm-commits on these mesa patches. Reviewers follow both lists. sorry about that This needs corresponding change in LLVM

Re: [Mesa-dev] [PATCH 2/2] r600g: Pass dimension parameter to compute shader.

2014-07-31 Thread Jan Vesely
On Thu, 2014-07-31 at 14:58 -0400, Tom Stellard wrote: On Thu, Jul 31, 2014 at 01:28:45PM -0400, Jan Vesely wrote: On Thu, 2014-07-31 at 11:02 -0400, Tom Stellard wrote: On Wed, Jul 30, 2014 at 07:11:35PM -0400, Jan Vesely wrote: Make the function static. No need to cc llvm

Re: [Mesa-dev] [PATCH 3/3] gallium: Fix compiler warning.

2014-08-01 Thread Jan Vesely
On Mon, 2014-07-28 at 10:56 -0600, Brian Paul wrote: On 07/25/2014 08:33 AM, Jan Vesely wrote: warning: type qualifiers ignored on function return type Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- src/gallium/auxiliary/tgsi/tgsi_ureg.c | 2 +- src/gallium/auxiliary/tgsi

[Mesa-dev] [PATCH 1/1] gallivm: Fix build with latest LLVM

2014-08-04 Thread Jan Vesely
Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- src/gallium/auxiliary/gallivm/lp_bld_debug.cpp | 8 1 file changed, 8 insertions(+) diff --git a/src/gallium/auxiliary/gallivm/lp_bld_debug.cpp b/src/gallium/auxiliary/gallivm/lp_bld_debug.cpp index 413a0c2..b6b52c8 100644 --- a/src

[Mesa-dev] [PATCH 1/1] targets/clover: link against libmesautil.la

2014-08-04 Thread Jan Vesely
Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- src/gallium/targets/opencl/Makefile.am | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/targets/opencl/Makefile.am b/src/gallium/targets/opencl/Makefile.am index ca047f0..43ba5dd 100644 --- a/src/gallium/targets/opencl

Re: [Mesa-dev] [PATCH 1/1] gallivm: Fix build with latest LLVM

2014-08-05 Thread Jan Vesely
On Tue, 2014-08-05 at 12:59 +0900, Michel Dänzer wrote: On 05.08.2014 07:50, Jan Vesely wrote: Signed-off-by: Jan Vesely jan.ves...@rutgers.edu I came up with exactly the same patches independently, so I pushed both your patches. :) thanks -- Jan Vesely jan.ves...@rutgers.edu

Re: [Mesa-dev] [PATCH 1/2] gallium: Add dimension parameter to launch_grid

2014-08-05 Thread Jan Vesely
On Tue, 2014-08-05 at 13:45 +0300, Francisco Jerez wrote: Jan Vesely jan.ves...@rutgers.edu writes: This is needed for OpenCL CC: Francisco Jerez curroje...@riseup.net CC: Tom Stellard t...@stellard.net Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- I tried to find

[Mesa-dev] [PATCH 3/3] r600, radeonsi: Copy implicit args provided by clover

2014-08-06 Thread Jan Vesely
Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- src/gallium/drivers/r600/evergreen_compute.c | 14 -- src/gallium/drivers/r600/evergreen_compute.h | 1 - src/gallium/drivers/radeonsi/si_compute.c| 6 +++--- 3 files changed, 11 insertions(+), 10 deletions(-) diff --git

[Mesa-dev] [PATCH 2/3] clover: Add work dimension implicit param to input

2014-08-06 Thread Jan Vesely
Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- src/gallium/state_trackers/clover/core/kernel.cpp | 162 -- 1 file changed, 85 insertions(+), 77 deletions(-) diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp b/src/gallium/state_trackers/clover/core

[Mesa-dev] [PATCH 1/3] gallium: Pass input data size to launch_grid

2014-08-06 Thread Jan Vesely
Future commits add implicit parameters so we can no longer rely on shader param size Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- src/gallium/drivers/ilo/ilo_gpgpu.c | 2 +- src/gallium/drivers/nouveau/nvc0/nvc0_compute.c | 2 +- src/gallium/drivers/nouveau/nvc0

[Mesa-dev] [PATCH 0/3] cl workdim v2

2014-08-06 Thread Jan Vesely
on turks without any regression, I have not tested SI as I don't have the hw. jan Jan Vesely (3): gallium: Pass input data size to launch_grid clover: Add work dimension implicit param to input r600,radeonsi: Copy implicit args provided by clover src/gallium/drivers/ilo/ilo_gpgpu.c

Re: [Mesa-dev] [PATCH 0/3] cl workdim v2

2014-08-08 Thread Jan Vesely
On Thu, 2014-08-07 at 16:02 +0300, Francisco Jerez wrote: Jan Vesely jan.ves...@rutgers.edu writes: This respin includes Francisco's approach of providing implicit in the arg vector passed from clover, and Tom's idea of appending implicit args after the kernel args. Hmmm... Maybe

[Mesa-dev] [PATCH 1/1] glapi: Fix compiler warning and script name

2014-08-13 Thread Jan Vesely
Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- src/mapi/glapi/gen/gl_gentable.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mapi/glapi/gen/gl_gentable.py b/src/mapi/glapi/gen/gl_gentable.py index 7577b66..c49f9a5 100644 --- a/src/mapi/glapi/gen

[Mesa-dev] [PATCH 1/1] configure.ac: Fix build with git-svn llvm version string

2014-08-13 Thread Jan Vesely
Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- My llvm-config --version is 3.6.0git-svn-r215564-cd35a3b3 This patch assumes that the interesting part consists of only digits and dots. configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b

Re: [Mesa-dev] [PATCH 1/1] configure.ac: Fix build with git-svn llvm version string

2014-08-14 Thread Jan Vesely
On Thu, 2014-08-14 at 06:35 -0700, Tom Stellard wrote: On Wed, Aug 13, 2014 at 04:46:56PM -0400, Jan Vesely wrote: Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- My llvm-config --version is 3.6.0git-svn-r215564-cd35a3b3 This patch assumes that the interesting part consists

Re: [Mesa-dev] [PATCH] r600/compute: Don't leak compute pool item_list/unallocated_list

2014-08-25 Thread Jan Vesely
/mailman/listinfo/mesa-dev -- Jan Vesely jan.ves...@rutgers.edu signature.asc Description: This is a digitally signed message part ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH 0/3] cl workdim v2

2014-09-01 Thread Jan Vesely
On Sat, 2014-08-16 at 13:13 +0300, Francisco Jerez wrote: Jan Vesely jan.ves...@rutgers.edu writes: On Thu, 2014-08-07 at 16:02 +0300, Francisco Jerez wrote: Jan Vesely jan.ves...@rutgers.edu writes: This respin includes Francisco's approach of providing implicit in the arg vector

Re: [Mesa-dev] [PATCH 0/3] cl workdim v2

2014-09-02 Thread Jan Vesely
On Tue, 2014-09-02 at 15:36 +0300, Francisco Jerez wrote: Jan Vesely jan.ves...@rutgers.edu writes: On Sat, 2014-08-16 at 13:13 +0300, Francisco Jerez wrote: Jan Vesely jan.ves...@rutgers.edu writes: On Thu, 2014-08-07 at 16:02 +0300, Francisco Jerez wrote: Jan Vesely jan.ves

Re: [Mesa-dev] [PATCH 1/1] configure.ac: Fix build with git-svn llvm version string

2014-09-04 Thread Jan Vesely
On Thu, 2014-08-14 at 11:15 -0400, Jan Vesely wrote: On Thu, 2014-08-14 at 06:35 -0700, Tom Stellard wrote: On Wed, Aug 13, 2014 at 04:46:56PM -0400, Jan Vesely wrote: Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- My llvm-config --version is 3.6.0git-svn-r215564-cd35a3b3

[Mesa-dev] [PATCH 1/1] clover: Remove compat::string::c_str

2014-09-04 Thread Jan Vesely
It's unsafe as compat::string is not null terminated. Fixes garbage log on successful build. Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- It's obviously not needed in program::build, but I'm not sure if runtime_error solution is the best one. src/gallium/state_trackers/clover/core

Re: [Mesa-dev] [PATCH 1/3] clover: fix _logs string creation

2014-09-05 Thread Jan Vesely
of these should just be using the conversion operator. See attachment. I came up with an almost identical patch to your attachment. you can add my Reviewed-by: Jan Vesely jan.ves...@rutgers.edu throw; } } -- 2.0.4

Re: [Mesa-dev] [PATCH 1/1] clover: Remove compat::string::c_str

2014-09-05 Thread Jan Vesely
On Fri, 2014-09-05 at 15:51 +0200, EdB wrote: On Friday, September 05, 2014 09:30:52 AM Francisco Jerez wrote: Jan Vesely jan.ves...@rutgers.edu writes: It's unsafe as compat::string is not null terminated. Fixes garbage log on successful build. Signed-off-by: Jan Vesely jan.ves

Re: [Mesa-dev] [PATCH] configure.ac: unbreak the build with non gnu grep

2014-09-10 Thread Jan Vesely
should be able to handle EREs everywhere. This allows Mesa to build on OpenBSD again. Signed-off-by: Jonathan Gray j...@jsg.id.au I don't have a BSD machine to test, but this works OK on my Linux box. Acked-by: Jan Vesely jan.ves...@rutgers.edu --- configure.ac | 2 +- 1 file changed, 1

Re: [Mesa-dev] [PATCH 0/3] cl workdim v2

2014-09-22 Thread Jan Vesely
instead of attachments? thanks, jan [SNIP] -- Jan Vesely jan.ves...@rutgers.edu signature.asc Description: This is a digitally signed message part ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo

Re: [Mesa-dev] [PATCH 1/1] glapi: Fix compiler warning and script name

2014-09-25 Thread Jan Vesely
On Wed, 2014-08-13 at 16:47 -0400, Jan Vesely wrote: Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- src/mapi/glapi/gen/gl_gentable.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mapi/glapi/gen/gl_gentable.py b/src/mapi/glapi/gen/gl_gentable.py index

Re: [Mesa-dev] [PATCH] Revert configure: ask vdpau.pc for the default location of the vdpau drivers

2014-10-06 Thread Jan Vesely
@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev -- Jan Vesely jan.ves...@rutgers.edu signature.asc Description: This is a digitally signed message part ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org

Re: [Mesa-dev] [PATCH 0/3] cl workdim v2

2014-10-10 Thread Jan Vesely
On Wed, 2014-10-08 at 18:02 +0300, Francisco Jerez wrote: Jan Vesely jan.ves...@rutgers.edu writes: [SNIP] I also don't like that this way there is no difference between explicit and implicit kernel arguments. On the other hand it's simple, and does not need additional per

Re: [Mesa-dev] [PATCH 0/3] cl workdim v2

2014-10-11 Thread Jan Vesely
On Sat, 2014-10-11 at 12:47 +0300, Francisco Jerez wrote: Jan Vesely jan.ves...@rutgers.edu writes: On Wed, 2014-10-08 at 18:02 +0300, Francisco Jerez wrote: Jan Vesely jan.ves...@rutgers.edu writes: [SNIP] I also don't like that this way there is no difference between

Re: [Mesa-dev] [PATCH 0/3] cl workdim v2

2014-10-11 Thread Jan Vesely
On Sun, 2014-10-12 at 01:56 +0300, Francisco Jerez wrote: Jan Vesely jan.ves...@rutgers.edu writes: On Sat, 2014-10-11 at 12:47 +0300, Francisco Jerez wrote: Jan Vesely jan.ves...@rutgers.edu writes: On Wed, 2014-10-08 at 18:02 +0300, Francisco Jerez wrote: Jan Vesely jan.ves

Re: [Mesa-dev] [PATCH 0/3] cl workdim v2

2014-10-11 Thread Jan Vesely
On Sun, 2014-10-12 at 01:56 +0300, Francisco Jerez wrote: Jan Vesely jan.ves...@rutgers.edu writes: On Sat, 2014-10-11 at 12:47 +0300, Francisco Jerez wrote: Jan Vesely jan.ves...@rutgers.edu writes: On Wed, 2014-10-08 at 18:02 +0300, Francisco Jerez wrote: Jan Vesely jan.ves

[Mesa-dev] [PATCH] clover: Require libelf

2014-10-20 Thread Jan Vesely
From: Jan Vesely jan.ves...@rutgers.edu CC: Tom Stellard t...@stellard.net CC: Emil Velikov emil.l.veli...@gmail.com CC: Francisco Jerez curroje...@riseup.net Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- This fixes clover build failures when building without radeon configure.ac | 2

[Mesa-dev] [PATCH v2 1/1] clover: Require libelf

2014-10-21 Thread Jan Vesely
v2: test for libelf once, check in both radeon and clover CC: Tom Stellard t...@stellard.net CC: Emil Velikov emil.l.veli...@gmail.com CC: Francisco Jerez curroje...@riseup.net Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- configure.ac | 10 -- 1 file changed, 8 insertions(+), 2

Re: [Mesa-dev] [PATCH 3/5] clover: Add support for compiling to native object code v3

2014-10-21 Thread Jan Vesely
\ -lclangFrontendTool \ -- Jan Vesely jan.ves...@rutgers.edu signature.asc Description: This is a digitally signed message part ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH 3/5] clover: Add support for compiling to native object code v3

2014-10-21 Thread Jan Vesely
patch). --as-needed also looked as a good alternative to hand picking LLVM_COMPONENTS in configure.ac, but I'm not sure how it is supported across platforms. jan -Tom [snip] -- Jan Vesely jan.ves...@rutgers.edu ___ mesa-dev mailing

Re: [Mesa-dev] [PATCH v2 1/1] clover: Require libelf

2014-10-22 Thread Jan Vesely
On Wed, 2014-10-22 at 19:16 +0100, Emil Velikov wrote: On 21/10/14 16:59, Jan Vesely wrote: v2: test for libelf once, check in both radeon and clover CC: Tom Stellard t...@stellard.net CC: Emil Velikov emil.l.veli...@gmail.com CC: Francisco Jerez curroje...@riseup.net Signed-off

Re: [Mesa-dev] [PATCH 1/1] glapi: Fix compiler warning and script name

2014-10-23 Thread Jan Vesely
On Thu, 2014-10-23 at 16:04 +0100, Emil Velikov wrote: On 25/09/14 23:45, Jan Vesely wrote: On Wed, 2014-08-13 at 16:47 -0400, Jan Vesely wrote: Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- src/mapi/glapi/gen/gl_gentable.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions

[Mesa-dev] [PATCH RFC 1/1] configure: Drop LLVM_COMPONENTS

2014-10-23 Thread Jan Vesely
-project.org Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- I'm not sure whether -Wl,--as-needed is supported by all build systems that emsa targets, or I need to add a test for it. configure.ac | 40 +++- 1 file changed, 7 insertions(+), 33 deletions

Re: [Mesa-dev] [PATCH RFC 1/1] configure: Drop LLVM_COMPONENTS

2014-10-23 Thread Jan Vesely
it with Tom few days ago. so it was not that quick :) since the --as-needed trick does not work I don't know what the correct solution is jan Cheers, Kai Jan Vesely wrote on 23.10.2014 21:58: Use -Wl,--as-needed instead This should fix the problem of using llvm split libraries. Since

[Mesa-dev] [PATCH 1/1] configure: fix typos

2014-10-24 Thread Jan Vesely
Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- configure.ac | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/configure.ac b/configure.ac index 93b25a2..a588d55 100644 --- a/configure.ac +++ b/configure.ac @@ -1970,7 +1970,7 @@ if test -n $with_gallium_drivers

[Mesa-dev] [PATCH WIP 1/1] configure: include llvm systemlibs when using static llvm

2014-10-24 Thread Jan Vesely
-Wl,--exclude-libs prevents automatic export of symbols CC: Kai Wasserbach k...@dev.carbon-project.org CC: Emil Velikov emil.l.veli...@gmail.com Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- Kai, can you try this patch with your setup, and check whether LLVM symbols are exported from

Re: [Mesa-dev] [PATCH WIP 1/1] configure: include llvm systemlibs when using static llvm

2014-10-24 Thread Jan Vesely
On Fri, 2014-10-24 at 22:42 +0200, Kai Wasserbäch wrote: Hi Jan, Jan Vesely wrote on 24.10.2014 19:03: -Wl,--exclude-libs prevents automatic export of symbols CC: Kai Wasserbach k...@dev.carbon-project.org CC: Emil Velikov emil.l.veli...@gmail.com Signed-off-by: Jan Vesely jan.ves

Re: [Mesa-dev] [PATCH WIP 1/1] configure: include llvm systemlibs when using static llvm

2014-10-24 Thread Jan Vesely
On Fri, 2014-10-24 at 23:48 +0200, Kai Wasserbäch wrote: Hi Jan, Jan Vesely wrote on 24.10.2014 23:30: On Fri, 2014-10-24 at 22:42 +0200, Kai Wasserbäch wrote: Jan Vesely wrote on 24.10.2014 19:03: -Wl,--exclude-libs prevents automatic export of symbols CC: Kai Wasserbach k

Re: [Mesa-dev] [PATCH WIP 1/1] configure: include llvm systemlibs when using static llvm

2014-10-26 Thread Jan Vesely
On Fri, 2014-10-24 at 23:54 +, Emil Velikov wrote: On 24/10/14 17:03, Jan Vesely wrote: -Wl,--exclude-libs prevents automatic export of symbols CC: Kai Wasserbach k...@dev.carbon-project.org CC: Emil Velikov emil.l.veli...@gmail.com Signed-off-by: Jan Vesely jan.ves

Re: [Mesa-dev] [PATCH WIP 1/1] configure: include llvm systemlibs when using static llvm

2014-10-27 Thread Jan Vesely
On Mon, 2014-10-27 at 02:24 +, Emil Velikov wrote: On 26/10/14 19:36, Jan Vesely wrote: On Fri, 2014-10-24 at 23:54 +, Emil Velikov wrote: On 24/10/14 17:03, Jan Vesely wrote: -Wl,--exclude-libs prevents automatic export of symbols CC: Kai Wasserbach k...@dev.carbon

Re: [Mesa-dev] [PATCH WIP 1/1] configure: include llvm systemlibs when using static llvm

2014-10-27 Thread Jan Vesely
On Mon, 2014-10-27 at 20:22 +, Emil Velikov wrote: On 27/10/14 18:05, Jan Vesely wrote: On Mon, 2014-10-27 at 02:24 +, Emil Velikov wrote: On 26/10/14 19:36, Jan Vesely wrote: On Fri, 2014-10-24 at 23:54 +, Emil Velikov wrote: On 24/10/14 17:03, Jan Vesely wrote: -Wl

[Mesa-dev] [PATCH 1/1] r600, llvm: Fix mem leak

2014-11-03 Thread Jan Vesely
Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- src/gallium/drivers/r600/r600_llvm.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/r600/r600_llvm.c b/src/gallium/drivers/r600/r600_llvm.c index c19693a..5f74bf7 100644 --- a/src/gallium/drivers/r600/r600_llvm.c +++ b

  1   2   3   4   5   6   >