[Mesa-dev] [PATCH v3 2/2] clover: add GetKernelArgInfo (CL 1.2)

2016-11-27 Thread Serge Martin
--- src/gallium/state_trackers/clover/api/kernel.cpp | 92 +- src/gallium/state_trackers/clover/core/module.cpp | 14 src/gallium/state_trackers/clover/core/module.hpp | 11 +++ .../state_trackers/clover/llvm/codegen/common.cpp | 11 +++ 4 files changed, 125

[Mesa-dev] [PATCH v3 1/2] clover: fix sampler argument type detection

2016-11-27 Thread Serge Martin
--- src/gallium/state_trackers/clover/llvm/codegen/common.cpp | 5 + 1 file changed, 5 insertions(+) diff --git a/src/gallium/state_trackers/clover/llvm/codegen/common.cpp b/src/gallium/state_trackers/clover/llvm/codegen/common.cpp index 13ccd59..aa6ca50 100644 ---

Re: [Mesa-dev] [PATCH v2] clover: add GetKernelArgInfo (CL 1.2)

2016-11-10 Thread Serge Martin
On Sunday 30 October 2016 16:07:25 Francisco Jerez wrote: > Serge Martin <edb+m...@sigluy.net> writes: > > --- > > > > src/gallium/state_trackers/clover/api/kernel.cpp | 47 > > -- > > src/gallium/state_trackers/clover/core/kernel.cpp |

Re: [Mesa-dev] clover: Add CL_PROGRAM_BINARY_TYPE support (CL1.2).

2016-11-08 Thread Serge Martin
On Sunday 06 November 2016 17:02:26 Dieter Nützel wrote: > After latest clover commit 'luxmark-v3.0' sigfault immediately: Hello Did you bisect it? Luxmark seems to crash just the same here without this commit. Serge > > SOURCE/luxmark-v3.0> ./luxmark > ./luxmark.bin:

Re: [Mesa-dev] [PATCH] clover: Allow OpenCL version override

2016-10-07 Thread Serge Martin
On Thursday 06 October 2016 16:26:21 Vedran Miletić wrote: > CLOVER_CL_VERSION_OVERRIDE allows overriding default OpenCL version > supported by Clover, analogous to MESA_GL_VERSION_OVERRIDE for OpenGL. > CLOVER_CL_C_VERSION_OVERRIDE allows overridng default OpenCL C version. Thanks. I've made

Re: [Mesa-dev] [PATCH 1/2] clover: support CL_PROGRAM_BINARY_TYPE (CL1.2)

2016-10-07 Thread Serge Martin
On Thursday 06 October 2016 13:20:36 Jan Vesely wrote: > On Fri, 2014-12-19 at 16:42 +0100, EdB wrote: > > CL_PROGRAM_BINARY_TYPE have been added to clGetProgramBuildInfo in > > CL1.2 > > --- > > src/gallium/state_trackers/clover/api/program.cpp | 4 +++ > >

Re: [Mesa-dev] [PATCH v2] clover: add GetKernelArgInfo (CL 1.2)

2016-10-04 Thread Serge Martin
On Saturday 01 October 2016 15:54:49 Serge Martin wrote: CC curro > --- > src/gallium/state_trackers/clover/api/kernel.cpp | 47 -- > src/gallium/state_trackers/clover/core/kernel.cpp | 6 +++ > src/gallium/state_trackers/clover/core/kernel.hpp | 1 + &g

Re: [Mesa-dev] [PATCH v2 0/2] Add support for some of the missing CL1.2 queries

2016-10-04 Thread Serge Martin
On Saturday 01 October 2016 18:51:09 Serge Martin wrote: > Updated serie, please review. CC curro > > Serge Martin (2): > clover: add CL_PROGRAM_BINARY_TYPE support (CL1.2) > clover: add missing clGetDeviceInfo CL1.2 queries > > src/gallium/state_trackers/clover/

Re: [Mesa-dev] [PATCH] clover: clGetExtensionFunctionAddressForPlatform

2016-10-04 Thread Serge Martin
On Saturday 01 October 2016 19:03:11 Serge Martin wrote: > On Sunday 27 September 2015 11:15:14 Serge Martin wrote: > > add clGetExtensionFunctionAddressForPlatform (CL 1.2) > > ping (one year reminder :p ) CC curro > > > --- > > > > src/gallium/state_t

Re: [Mesa-dev] [PATCH 0/2] clover: clEnqueueMigrateMemObjects

2016-10-04 Thread Serge Martin
On Monday 03 October 2016 19:37:53 Serge Martin wrote: > Ping CC curro > > On Saturday 12 September 2015 21:08:20 Serge Martin wrote: > > Now that mem object can be move back to the host, I think we should latter > > come with a way to optimize read mapping for such o

Re: [Mesa-dev] [PATCH 0/2] clover: clEnqueueMigrateMemObjects

2016-10-03 Thread Serge Martin
Ping On Saturday 12 September 2015 21:08:20 Serge Martin wrote: > Now that mem object can be move back to the host, I think we should latter > come with a way to optimize read mapping for such objets. For the moment if > they are mapped for reading after been moved to the host, they wil

Re: [Mesa-dev] [PATCH] clover: clGetExtensionFunctionAddressForPlatform

2016-10-01 Thread Serge Martin
On Sunday 27 September 2015 11:15:14 Serge Martin wrote: > add clGetExtensionFunctionAddressForPlatform (CL 1.2) ping (one year reminder :p ) > --- > src/gallium/state_trackers/clover/api/dispatch.cpp | 2 +- > src/gallium/state_trackers/clover/api/dispatch.hpp | 4 &g

[Mesa-dev] [PATCH v2 1/2] clover: add CL_PROGRAM_BINARY_TYPE support (CL1.2)

2016-10-01 Thread Serge Martin
CL_PROGRAM_BINARY_TYPE have been added to clGetProgramBuildInfo in CL1.2 --- src/gallium/state_trackers/clover/api/program.cpp | 5 + src/gallium/state_trackers/clover/core/module.cpp | 1 + src/gallium/state_trackers/clover/core/module.hpp | 5 +

[Mesa-dev] [PATCH v2 2/2] clover: add missing clGetDeviceInfo CL1.2 queries

2016-10-01 Thread Serge Martin
--- src/gallium/state_trackers/clover/api/device.cpp | 23 +++ src/gallium/state_trackers/clover/core/device.cpp | 10 ++ src/gallium/state_trackers/clover/core/device.hpp | 2 ++ 3 files changed, 35 insertions(+) diff --git

[Mesa-dev] [PATCH v2 0/2] Add support for some of the missing CL1.2 queries

2016-10-01 Thread Serge Martin
Updated serie, please review. Serge Martin (2): clover: add CL_PROGRAM_BINARY_TYPE support (CL1.2) clover: add missing clGetDeviceInfo CL1.2 queries src/gallium/state_trackers/clover/api/device.cpp | 23 ++ src/gallium/state_trackers/clover/api/program.cpp | 5

[Mesa-dev] [PATCH v2] clover: add GetKernelArgInfo (CL 1.2)

2016-10-01 Thread Serge Martin
--- src/gallium/state_trackers/clover/api/kernel.cpp | 47 -- src/gallium/state_trackers/clover/core/kernel.cpp | 6 +++ src/gallium/state_trackers/clover/core/kernel.hpp | 1 + src/gallium/state_trackers/clover/core/module.hpp | 19 +--

Re: [Mesa-dev] [PATCH v3] clover: Introduce CLOVER_EXTRA_COMPILER_OPTIONS

2016-09-07 Thread Serge Martin
On Wednesday 07 September 2016 19:28:36 Serge Martin wrote: > On Wednesday 07 September 2016 18:37:51 Vedran Miletić wrote: > > On 09/07/2016 04:14 PM, Jan Vesely wrote: > > > On Wed, 2016-09-07 at 13:43 +0200, Vedran Miletić wrote: > > >> Options specified via the

Re: [Mesa-dev] [PATCH v4] clover: Introduce CLOVER_EXTRA_{COMPILER, LINKER}_OPTIONS

2016-09-07 Thread Serge Martin
On Wednesday 07 September 2016 19:17:39 Vedran Miletić wrote: > The options specified in the CLOVER_EXTRA_COMPILER_OPTIONS shell > variable are appended to the compiler options specified by the OpenCL > program, if any. > Analogously, the options specified in the CLOVER_EXTRA_LINKER_OPTIONS >

Re: [Mesa-dev] [PATCH v3] clover: Introduce CLOVER_EXTRA_COMPILER_OPTIONS

2016-09-07 Thread Serge Martin
On Wednesday 07 September 2016 18:37:51 Vedran Miletić wrote: > On 09/07/2016 04:14 PM, Jan Vesely wrote: > > On Wed, 2016-09-07 at 13:43 +0200, Vedran Miletić wrote: > >> Options specified via the CLOVER_EXTRA_COMPILER_OPTIONS shell > >> variable > >> are appended to the compiler and linker

Re: [Mesa-dev] [PATCH] st/clover: Define __OPENCL_VERSION__ on the device side

2016-08-31 Thread Serge Martin
On Wednesday 31 August 2016 12:39:23 Vedran Miletić wrote: > On 08/28/2016 04:42 PM, Niels Ole Salscheider wrote: > > This is required by the OpenCL standard. > > > > Signed-off-by: Niels Ole Salscheider > Reviewed-by: Vedran Miletić > >

Re: [Mesa-dev] [PATCH] clover: Introduce CLOVER_EXTRA_COMPILER_OPTIONS

2016-08-31 Thread Serge Martin
R_OPTIONS > * use debug_get_option > * append to linker options as well > > Signed-off-by: Vedran Miletić <ved...@miletic.net> > Reviewed-by: Edward O'Callaghan <funfunc...@folklore1984.net> > Reviewed-by: Serge Martin <edb+m...@sigluy.net> I didn't R-b. > --- >

[Mesa-dev] [PATCH v3] clover: fix getting scalar args api size

2016-08-30 Thread Serge Martin
This fix getting the size of a struct arg. vec3 types still work ok. Only buit-in args need to have power of two alignment, getTypeAllocSize reports the correct size in all cases. --- This v3 is v1.1 modified for the curent ToT src/gallium/state_trackers/clover/llvm/codegen/common.cpp | 7

Re: [Mesa-dev] [PATCH] clover: Introduce CLOVER_COMPILER_OPTIONS

2016-08-30 Thread Serge Martin
On Tuesday 30 August 2016 01:20:55 Vedran Miletić wrote: > Options specified via the CLOVER_COMPILER_OPTIONS shell variable are > appended to the compiler options specified by the OpenCL program (if > any). > > Signed-off-by: Vedran Miletić > --- > docs/envvars.html

Re: [Mesa-dev] [PATCH mesa] Introduce .editorconfig

2016-08-29 Thread Serge Martin
On Monday 29 August 2016 22:30:58 Eric Engestrom wrote: > A few weeks ago, Jose Fonseca suggested [0] we use .editorconfig files > to try and enforce the formatting of the code, to which Michel Dänzer > suggested [1] we start by importing the existing .dir-locals.el > settings. The first draft was

Re: [Mesa-dev] [PATCH 2/2] clover: fix getting struct args api size

2016-06-08 Thread Serge Martin
On Thursday 09 June 2016 00:47:27 Serge Martin wrote: > This fix getting the size of a struct arg. vec3 types still work ok. > Only buit-in args need to have power of two alignment, getTypeAllocSize > reports the correct size. M. I forgot to put V2 and to fix the commit

[Mesa-dev] [PATCH 2/2] clover: fix getting struct args api size

2016-06-08 Thread Serge Martin
This fix getting the size of a struct arg. vec3 types still work ok. Only buit-in args need to have power of two alignment, getTypeAllocSize reports the correct size. --- src/gallium/state_trackers/clover/llvm/invocation.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[Mesa-dev] [PATCH v1.1 2/2] clover: fix getting scalar args api size

2016-06-02 Thread Serge Martin
This fix getting the size of a struct arg. vec3 types still work ok. Only buit-in args need to have power of two alignment, getTypeAllocSize reports the correct size. Cc: 12.0 --- src/gallium/state_trackers/clover/llvm/invocation.cpp | 7 +++ 1 file

[Mesa-dev] [PATCH 2/2] clover: fix getting scalar args api size

2016-06-01 Thread Serge Martin
This fix getting the size of a struct arg. vec3 types still work ok. Only buit-in args need to have power of two alignement, getTypeAllocSize reports the corect size. Cc: 12.0 --- src/gallium/state_trackers/clover/llvm/invocation.cpp | 7 +++ 1 file

[Mesa-dev] [PATCH 1/2] clover: assert struct argument is compiled usably

2016-06-01 Thread Serge Martin
rectly once Clang gets patched with http://reviews.llvm.org/D20168 or an equivalent patch. Reviewed-by: Serge Martin <edb+m...@sigluy.net> Cc: 12.0 <mesa-sta...@lists.freedesktop.org> --- src/gallium/state_trackers/clover/llvm/invocation.cpp | 4 1 file changed, 4 insertions(+)

Re: [Mesa-dev] [PATCH 2/2] Add LLVM version to Mesa version strings

2016-05-23 Thread Serge Martin
On Monday 23 May 2016 09:21:51 Giuseppe Bilotta wrote: > Code generation (kernel compilation) may sometimes hit LLVM-specific > bugs. Adding the used LLVM version to the version string may make bug > triaging easier. (This was inspired by a similar patch recently > proposed for pocl.) > >

Re: [Mesa-dev] [RFC PATCH] clover: add LLVM version to device and platform version

2016-05-22 Thread Serge Martin
On Tuesday 17 May 2016 22:54:01 Giuseppe Bilotta wrote: > On Sun, May 15, 2016 at 10:54 PM, Giuseppe Bilotta > > wrote: > > On Sun, May 15, 2016 at 9:35 AM, Vedran Miletić wrote: > >> How about also adding Mesa git version, like "OpenGL core

Re: [Mesa-dev] [PATCH] clover/llvm: Fix copyright attribution of invocation.cpp.

2016-05-10 Thread Serge Martin
.", let me know if this > >> is not correct -- Or if you think you have contributed a significant > >> amount of work to this file and I've missed it. > > > > The second paragraph probably shouldn't be in the commit log. Either way > > though, > > S

Re: [Mesa-dev] [PATCH v7 0/7] add clLinkProgram

2016-04-09 Thread Serge Martin
On Monday 29 February 2016 12:36:36 Francisco Jerez wrote: > Serge Martin <edb+m...@sigluy.net> writes: > > Ping > > Thanks for trying to split some of the LLVM interfacing changes into > separate patches. I'm about 80% done cleaning up the mess that > llvm/invoca

[Mesa-dev] [PATCH] require libdrm 2.4.66 for AMD drivers

2016-03-13 Thread Serge Martin
since 737b6ed13e8f813987b5566004f0f45e9c55f1e8 src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c not longer compile: error: unknown type name ‘drmDevicePtr’ --- configure.ac | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index 49be147..55b1c95

Re: [Mesa-dev] [PATCH v7 0/7] add clLinkProgram

2016-02-29 Thread Serge Martin
Ping On Saturday 13 February 2016 23:08:34 Serge Martin wrote: > This serie add clLinkProgram function needed for CL 1.2. > However, it lacks the binary type part that is mandatory for input > validation and also for CL_PROGRAM_BINARY_TYPE query. This will be adressed > in another s

[Mesa-dev] [PATCH] clover: fix build failure since bfd695e

2016-02-13 Thread Serge Martin
--- src/gallium/state_trackers/clover/core/kernel.cpp | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp b/src/gallium/state_trackers/clover/core/kernel.cpp index 41b3852..a4ef2b1 100644 ---

[Mesa-dev] [PATCH v2] clover: fix build failure since bfd695e

2016-02-13 Thread Serge Martin
--- src/gallium/state_trackers/clover/core/kernel.cpp | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp b/src/gallium/state_trackers/clover/core/kernel.cpp index 41b3852..3daec3b 100644 ---

[Mesa-dev] [PATCH v7 2/7] clover: make use of llvm_ir_compiler

2016-02-13 Thread Serge Martin
--- .../state_trackers/clover/llvm/invocation.cpp | 234 ++--- 1 file changed, 23 insertions(+), 211 deletions(-) diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp b/src/gallium/state_trackers/clover/llvm/invocation.cpp index 4d11c24..d3347de 100644 ---

[Mesa-dev] [PATCH v7 3/7] clover: program::build change opts to std::string

2016-02-13 Thread Serge Martin
--- src/gallium/state_trackers/clover/core/program.cpp | 4 ++-- src/gallium/state_trackers/clover/core/program.hpp | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/gallium/state_trackers/clover/core/program.cpp b/src/gallium/state_trackers/clover/core/program.cpp index

[Mesa-dev] [PATCH v7 6/7] clover: add clLinkProgram (CL 1.2)

2016-02-13 Thread Serge Martin
Reviewed-by: Francisco Jerez --- src/gallium/state_trackers/clover/api/program.cpp | 31 +++--- src/gallium/state_trackers/clover/core/program.hpp | 4 +-- 2 files changed, 29 insertions(+), 6 deletions(-) diff --git

[Mesa-dev] [PATCH v7 4/7] clover: separate compilation and link stages

2016-02-13 Thread Serge Martin
--- src/gallium/state_trackers/clover/api/program.cpp | 19 -- .../state_trackers/clover/core/compiler.hpp| 7 +- src/gallium/state_trackers/clover/core/error.hpp | 7 ++ src/gallium/state_trackers/clover/core/program.cpp | 38 ++-

[Mesa-dev] [PATCH v7 7/7] clover: add -create-library option support

2016-02-13 Thread Serge Martin
--- .../state_trackers/clover/llvm/invocation.cpp | 87 -- .../state_trackers/clover/llvm/ir_compiler.cpp | 46 +--- .../state_trackers/clover/llvm/ir_compiler.hpp | 4 +- 3 files changed, 85 insertions(+), 52 deletions(-) diff --git

[Mesa-dev] [PATCH v7 0/7] add clLinkProgram

2016-02-13 Thread Serge Martin
This serie add clLinkProgram function needed for CL 1.2. However, it lacks the binary type part that is mandatory for input validation and also for CL_PROGRAM_BINARY_TYPE query. This will be adressed in another serie once we agree on the way to store it. Serge Martin (7): clover: add a LLVM

[Mesa-dev] [PATCH v7 1/7] clover: add a LLVM compiler class

2016-02-13 Thread Serge Martin
a/src/gallium/state_trackers/clover/llvm/ir_compiler.cpp b/src/gallium/state_trackers/clover/llvm/ir_compiler.cpp new file mode 100644 index 000..8042a3e --- /dev/null +++ b/src/gallium/state_trackers/clover/llvm/ir_compiler.cpp @@ -0,0 +1,282 @@ +// Copyright 2016 Serge Martin

[Mesa-dev] [PATCH v7 5/7] clover: override ret_object

2016-02-13 Thread Serge Martin
Return an API object from an intrusive reference to a Clover object, incrementing the reference count of the object. Reviewed-by: Francisco Jerez --- src/gallium/state_trackers/clover/api/util.hpp | 12 1 file changed, 12 insertions(+) diff --git

[Mesa-dev] [PATCH v3] clover: fix build failure since bfd695e

2016-02-13 Thread Serge Martin
--- src/gallium/state_trackers/clover/core/kernel.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp b/src/gallium/state_trackers/clover/core/kernel.cpp index 41b3852..8396be9 100644 ---

Re: [Mesa-dev] [PATCH] tgsi: use TGSI_WRITEMASK_XYZW instead of hardcoding the mask

2016-02-04 Thread Serge Martin
On Thursday 04 February 2016 22:25:00 Samuel Pitoiset wrote: > Signed-off-by: Samuel Pitoiset <samuel.pitoi...@gmail.com> Reviewed-by: Serge Martin <edb+m...@sigluy.net> > --- > src/gallium/auxiliary/tgsi/tgsi_transform.h | 2 +- > src/gallium/auxiliary/tgsi/tgsi_ure

[Mesa-dev] [PATCH v6 0/3] add clLinkProgram

2015-12-31 Thread Serge Martin
, except for LLVM api change. Serge Martin (3): clover: separate compile and link stages clover: override ret_object clover: add clLinkProgram (CL 1.2) src/gallium/state_trackers/clover/api/program.cpp | 50 +++- src/gallium/state_trackers/clover/api/util.hpp | 12 + .../state_trackers

[Mesa-dev] [PATCH v6 3/3] clover: add clLinkProgram (CL 1.2)

2015-12-31 Thread Serge Martin
Reviewed-by: Francisco Jerez --- src/gallium/state_trackers/clover/api/program.cpp | 31 +++--- src/gallium/state_trackers/clover/core/program.hpp | 4 +-- 2 files changed, 29 insertions(+), 6 deletions(-) diff --git

[Mesa-dev] [PATCH v6 1/3] clover: separate compile and link stages

2015-12-31 Thread Serge Martin
--- src/gallium/state_trackers/clover/api/program.cpp | 19 +- .../state_trackers/clover/core/compiler.hpp| 7 +- src/gallium/state_trackers/clover/core/error.hpp | 7 + src/gallium/state_trackers/clover/core/program.cpp | 41 +++-

[Mesa-dev] [PATCH v6 2/3] clover: override ret_object

2015-12-31 Thread Serge Martin
Return an API object from an intrusive reference to a Clover object, incrementing the reference count of the object. Reviewed-by: Francisco Jerez --- src/gallium/state_trackers/clover/api/util.hpp | 12 1 file changed, 12 insertions(+) diff --git

[Mesa-dev] [PATCH 2/3] freedreno: change to goto fail

2015-12-06 Thread Serge Martin
in fd_resource_transfer_map, like the others error cases --- src/gallium/drivers/freedreno/freedreno_resource.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/gallium/drivers/freedreno/freedreno_resource.c b/src/gallium/drivers/freedreno/freedreno_resource.c index

[Mesa-dev] [PATCH 3/3] freedreno: little clean up in fd_create_surface

2015-12-06 Thread Serge Martin
in order to avoid returing invalid adress if CALLOC_STRUCT return NULL. --- src/gallium/drivers/freedreno/freedreno_surface.c | 31 --- 1 file changed, 16 insertions(+), 15 deletions(-) diff --git a/src/gallium/drivers/freedreno/freedreno_surface.c

[Mesa-dev] [PATCH 1/3] freedreno: fix bind_sampler_states when hwcso is NULL

2015-12-06 Thread Serge Martin
src/gallium/tests/trivial/compute.c expects samplers to be cleaned when the samplers list is NULL. Like in radeon, the function behave like when the number of samplers parameter is set to 0. --- src/gallium/drivers/freedreno/a2xx/fd2_texture.c | 3 +++

Re: [Mesa-dev] [PATCH v2] gallium/tests: fix build with clang compiler

2015-12-01 Thread Serge Martin
On Sunday 29 November 2015 15:23:37 Samuel Pitoiset wrote: > Nested functions are supported as an extension in GNU C, but Clang > don't support them. > > This fixes compilation errors when (manually) building compute.c, > or by setting --enable-gallium-tests to the configure script. > > Changes

[Mesa-dev] [PATCH] clover: fix tgsi compiler crash with invalid src

2015-10-26 Thread Serge Martin
--- src/gallium/state_trackers/clover/tgsi/compiler.cpp | 5 + 1 file changed, 5 insertions(+) diff --git a/src/gallium/state_trackers/clover/tgsi/compiler.cpp b/src/gallium/state_trackers/clover/tgsi/compiler.cpp index 54cb747..4d05666 100644 ---

[Mesa-dev] [PATCH v5 0/3] add clLinkProgram

2015-10-04 Thread Serge Martin
changed, this v5 is a rebased v4 in hope that the first patch in the serie can be reviewed. CC: Francisco Jerez <curroje...@riseup.net> Serge Martin (3): clover: separate compile and link stages clover: override ret_object clover: add clLinkProgramm (CL 1.2) src/gallium/state_tr

[Mesa-dev] [PATCH v5 3/3] clover: add clLinkProgramm (CL 1.2)

2015-10-04 Thread Serge Martin
Reviewed-by: Francisco Jerez --- src/gallium/state_trackers/clover/api/program.cpp | 31 +++--- src/gallium/state_trackers/clover/core/program.hpp | 4 +-- 2 files changed, 29 insertions(+), 6 deletions(-) diff --git

[Mesa-dev] [PATCH v5 2/3] clover: override ret_object

2015-10-04 Thread Serge Martin
Return an API object from an intrusive reference to a Clover object, incrementing the reference count of the object. Reviewed-by: Francisco Jerez --- src/gallium/state_trackers/clover/api/util.hpp | 12 1 file changed, 12 insertions(+) diff --git

[Mesa-dev] [PATCH v5 1/3] clover: separate compile and link stages

2015-10-04 Thread Serge Martin
--- src/gallium/state_trackers/clover/api/program.cpp | 19 +- .../state_trackers/clover/core/compiler.hpp| 7 +- src/gallium/state_trackers/clover/core/error.hpp | 7 + src/gallium/state_trackers/clover/core/program.cpp | 41 +++-

[Mesa-dev] [PATCH] clover: clGetExtensionFunctionAddressForPlatform

2015-09-27 Thread Serge Martin
add clGetExtensionFunctionAddressForPlatform (CL 1.2) --- src/gallium/state_trackers/clover/api/dispatch.cpp | 2 +- src/gallium/state_trackers/clover/api/dispatch.hpp | 4 src/gallium/state_trackers/clover/api/platform.cpp | 16 3 files changed, 21 insertions(+), 1

[Mesa-dev] [PATCH v2 0/3] clover: implement clCreateImage

2015-09-19 Thread Serge Martin
Francisco Jerez (1): clover: Move down canonicalization of memory object flags into validate_flags(). Serge Martin (2): clover: implement clCreateImage clover: implement clCreateImage?D w/ clCreateImage src/gallium/state_trackers/clover/api/memory.cpp | 215 +-- 1

[Mesa-dev] [PATCH v2 2/3] clover: implement clCreateImage

2015-09-19 Thread Serge Martin
--- src/gallium/state_trackers/clover/api/memory.cpp | 101 --- 1 file changed, 91 insertions(+), 10 deletions(-) diff --git a/src/gallium/state_trackers/clover/api/memory.cpp b/src/gallium/state_trackers/clover/api/memory.cpp index 69c3e95..0c2d353 100644 ---

[Mesa-dev] [PATCH v2 1/3] clover: Move down canonicalization of memory object flags into validate_flags().

2015-09-19 Thread Serge Martin
From: Francisco Jerez <curroje...@riseup.net> This will be used to share the same logic between buffer and image creation. Reviewed-by: Serge Martin <edb+m...@sigluy.net> --- src/gallium/state_trackers/clover/api/memory.cpp | 80 1 file changed, 41 inser

[Mesa-dev] [PATCH v2 3/3] clover: implement clCreateImage?D w/ clCreateImage

2015-09-19 Thread Serge Martin
remplace clCreateImage2D and clCreateImage3D implementation with call to clCreateImage --- src/gallium/state_trackers/clover/api/memory.cpp | 60 1 file changed, 8 insertions(+), 52 deletions(-) diff --git a/src/gallium/state_trackers/clover/api/memory.cpp

[Mesa-dev] [PATCH 1/2] clover: implement clCreateImage

2015-09-17 Thread Serge Martin
--- src/gallium/state_trackers/clover/api/memory.cpp | 117 +-- 1 file changed, 107 insertions(+), 10 deletions(-) diff --git a/src/gallium/state_trackers/clover/api/memory.cpp b/src/gallium/state_trackers/clover/api/memory.cpp index 1efb95b..6e95931 100644 ---

[Mesa-dev] [PATCH 2/2] clover: implement clCreateImage?D w/ clCreateImage

2015-09-17 Thread Serge Martin
remplace clCreateImage2D and clCreateImage3D implementation with call to clCreateImage --- src/gallium/state_trackers/clover/api/memory.cpp | 92 +--- 1 file changed, 32 insertions(+), 60 deletions(-) diff --git a/src/gallium/state_trackers/clover/api/memory.cpp

[Mesa-dev] [PATCH 0/2] clover: clEnqueueMigrateMemObjects

2015-09-12 Thread Serge Martin
Now that mem object can be move back to the host, I think we should latter come with a way to optimize read mapping for such objets. For the moment if they are mapped for reading after been moved to the host, they will be send back to the device... Serge Martin (2): clover

[Mesa-dev] [PATCH 1/2] clover: clEnqueueMigrateMemObjects (device)

2015-09-12 Thread Serge Martin
implement memory objects migration to device --- src/gallium/state_trackers/clover/api/transfer.cpp | 48 ++ src/gallium/state_trackers/clover/core/memory.cpp | 10 ++--- src/gallium/state_trackers/clover/core/memory.hpp | 12 --

[Mesa-dev] [PATCH 2/2] clover: clEnqueueMigrateMemObjects (host)

2015-09-12 Thread Serge Martin
implement memory objects migration back to host --- src/gallium/state_trackers/clover/api/transfer.cpp | 5 +++- src/gallium/state_trackers/clover/core/memory.cpp | 28 ++ src/gallium/state_trackers/clover/core/memory.hpp | 4

Re: [Mesa-dev] [RFC] Fix for clang compiler issue as reported in Bug 91826

2015-09-01 Thread Serge Martin (EdB)
On Tuesday 01 September 2015 17:10:33 Albert Freeman wrote: > Clang tryed to declare the non type member of struct module (enum type type) > (in clover/core/module.hpp) instead of a variable of type enum (enum type). > > Signed-off-by: Albert Freeman <albertwdfree...@gmail.com>

Re: [Mesa-dev] [PATCH] clover: fix llvm 3.5 build error

2015-08-21 Thread Serge Martin (EdB)
to llvm 3.5 this is good for me. And it no longer make clover crash. Reviewed-by: Serge Martin (EdB) edb+m...@sigluy.net But since I don't have commit access, not sure what my r-b worth. Could it be pushed before the branch point? --- .../state_trackers/clover/llvm/invocation.cpp | 33

Re: [Mesa-dev] [PATCH] clover: fix llvm 3.5 build error

2015-08-16 Thread Serge Martin (EdB)
On Sunday 09 August 2015 11:52:13 orome wrote: On Sun, 2015-08-09 at 15:39 +0200, Zoltan Gilian wrote: There is no MDOperand in llvm 3.5. v2: Check if kernel metadata is present to avoid crash (EdB). still builds for me. Sadly, it's still crashing for me. auto kernel_node =

Re: [Mesa-dev] [PATCH] clover: block on transfer api calls when requested

2015-08-15 Thread Serge Martin (EdB)
Hello This looks like what I've send a few weeks ago [0]. It's seems we should come with something a little different as explain by curro in [1] [0] http://lists.freedesktop.org/archives/mesa-dev/2015-August/090479.html [1] http://lists.freedesktop.org/archives/mesa-dev/2015-June/086110.html

[Mesa-dev] [PATCH v4] clover: stub missing CL 1.2 functions

2015-08-07 Thread Serge Martin (EdB)
As sugested by Tom a long time ago and in order to be able to create Piglit tests v2: replace NOT_SUPPORTED_BY_CL_1_1 macro with an inline function remove extra space in clLinkProgram arg v3: use __func__ v4: back to a macro, it make more sense to use it with __func__ ---

Re: [Mesa-dev] [PATCH v4] clover: stub missing CL 1.2 functions

2015-08-07 Thread Serge Martin
On Friday 07 August 2015 14:45:56 Francisco Jerez wrote: Serge Martin (EdB) edb+m...@sigluy.net writes: As sugested by Tom a long time ago and in order to be able to create Piglit tests v2: replace NOT_SUPPORTED_BY_CL_1_1 macro with an inline function remove extra space