Re: [Beignet] beignet and LLVM 8+

2020-01-14 Thread Simon Richter
Hi, On Tue, Jan 14, 2020 at 07:45:57AM +0100, Jérôme Kieffer wrote: > By the way, Thanks Rebecca for you work on the driver. +1. Simon ___ Beignet mailing list Beignet@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/beignet

Re: [Beignet] Status of cl_khr_gl_sharing

2017-08-02 Thread Simon Richter
Hi, On 01.08.2017 23:39, Rebecca N. Palmer wrote: > As beignet's Debian maintainer, I am considering whether to enable CL-GL > sharing in our package, given its incomplete state. I haven't looked at it since the initial package, because back then it required the Mesa source code in order to get

Re: [Beignet] [PATCH] Backend: add double support to convert_uchar|char|short|ushort|int|uint|long|ulong_sat(double x) HW support Double to int16, int32 from IVB, others done by software. Double to in

2017-03-08 Thread Simon Richter
Hi, On 08.03.2017 06:27, rander wrote: > backend/src/libocl/script/ocl_convert.sh | 10 +- I think you also need conversion from double to other types. Also (somewhat unrelated): are there no-op conversions for same source and destination type? Simon signature.asc

Re: [Beignet] [PATCH] Only provide device IDs if the type matches

2017-02-10 Thread Simon Richter
Hi, On Fri, Feb 10, 2017 at 06:36:35AM +, Pan, Xiuli wrote: > You point about the device type is right, but the git commit you found is > not right the function is just copy form somewhere else. And the this > patch is not follow the OpenCL spec. As the Beignet does not have a CPU > device,

Re: [Beignet] [PATCH] FindLLVM: Accept LLVM 3.9

2017-01-23 Thread Simon Richter
Hi, On Sun, Jan 22, 2017 at 05:07:07PM +, Rebecca N. Palmer wrote: > - NAMES llvm-config-37 llvm-config-3.7 llvm-config-36 > llvm-config-3.6 llvm-config-38 llvm-config-3.8 llvm-config llvm-config-35 > llvm-config-3.5 llvm-config-34 llvm-config-3.4 > + NAMES

Re: [Beignet] Proposed patch: add CMake option for toggling OCL ICD Loader compatibility

2016-12-28 Thread Simon Richter
Hi, On 01.12.2016 13:24, Marek Szuba wrote: > As suggested in https://bugs.freedesktop.org/show_bug.cgi?id=98885 , > here is a Git-friendly patch which makes CMake looking for OCL ICD > header files depend on a new configuration option, so that the loader > compatibility mode can be explicitly

Re: [Beignet] [PATCH] Refine mem.h and improve the related macro defination.

2016-12-20 Thread Simon Richter
Hi, On 20.12.2016 04:23, He Junyan wrote: >>> +if (!CL_OBJECT_IS_BUFFER(memobj)) { >> That would match pipes as well, is that intended? > Already redefine CL_OBJECT_IS_BUFFER to just match buffer and subbuffer. > +#define CL_OBJECT_IS_BUFFER(mem) ((mem &&

Re: [Beignet] [PATCH] Refine mem.h and improve the related macro defination.

2016-12-19 Thread Simon Richter
Hi, On 19.12.2016 10:21, junyan...@inbox.com wrote: > --- a/src/cl_api_mem.c > +++ b/src/cl_api_mem.c > @@ -71,54 +71,54 @@ clGetMemObjectInfo(cl_mem memobj, > break; >case CL_MEM_HOST_PTR: { > ptr = 0; > -if (memobj->type == CL_MEM_IMAGE_TYPE) { > +if

Re: [Beignet] [PATCH] Runtime: Use cl_ulong as CL_DEVICE_MAX_MEM_ALLOC_SIZE's return type.

2016-12-09 Thread Simon Richter
Hi, On Fri, Dec 09, 2016 at 06:49:56AM +, Yang, Rong R wrote: > The change is for the case that param_value_size > sizeof(device->FIELD). > For example: > cl_ulong max_alloc_size; > clGetDeviceInfo(device, CL_DEVICE_MAX_MEM_ALLOC_SIZE, > sizeof(max_alloc_size), _alloc_size, NULL); >

Re: [Beignet] [PATCH] Runtime: Use cl_ulong as CL_DEVICE_MAX_MEM_ALLOC_SIZE's return type.

2016-12-08 Thread Simon Richter
Hi, On Thu, Dec 08, 2016 at 03:47:28PM +0800, Yang Rong wrote: > diff --git a/src/cl_device_id.c b/src/cl_device_id.c > index 24334fd..71a7be1 100644 > --- a/src/cl_device_id.c > +++ b/src/cl_device_id.c > @@ -926,6 +926,7 @@ cl_get_device_ids(cl_platform_idplatform, > }

Re: [Beignet] [PATCH] Runtime: Add CL base object for all cl objects.

2016-07-15 Thread Simon Richter
Hi, On 14.07.2016 10:15, junyan...@inbox.com wrote: > The runtime code is a little verbose in CL object handle. > Every CL objects should have a reference, a lock to protect itself > and an ICD dispatcher. We can organize them to a struct and place > it at the beginning of each CL object. Does

Re: [Beignet] Error during build on Linux Mint 17 -- segmentation fault

2014-11-30 Thread Simon Richter
Hi, On 30.11.2014 19:34, al acran wrote: I'm using llvm version 3.4 and clang 3.5. Can anyone help me? Generally, these have tighter coupling than expected. Using two different versions has led to problems for me quite often. Can you try to use 3.5 of each? Simon signature.asc

Re: [Beignet] Libdir

2014-01-14 Thread Simon Richter
Hi, I'm not sure, but why we installing libs to prefix/lib but not libdir? I.e. on x86_64 I have installed beignet in /usr/lib, but not in /usr/lib64. /usr/lib64 is dead and should never be used. In principle, it would be good to install to /usr/lib/x86_64-linux-gnu, but that is actually the

Re: [Beignet] [PATCH] build: use auto libdir

2014-01-14 Thread Simon Richter
Hi, On 14.01.2014 18:51, Igor Gnatenko wrote: From 4a47aac87719b0b56565cd257cac0868ae574cd4 Mon Sep 17 00:00:00 2001 From: Igor Gnatenko i.gnatenko.br...@gmail.com Date: Tue, 14 Jan 2014 21:22:42 +0400 Subject: [PATCH] build: use auto libdir That looks immensely useful for distribution

[Beignet] Beignet vs LuxRays, again

2013-11-22 Thread Simon Richter
Hi, I've just rendered the first picture using LuxRays and Beignet. Observations: - During rendering, colored dots appeared all over the display at random locations - In the final result, there are black pixels - Fonts in some applications appear garbled - Rendering was slow (AMD's CPU-based

Re: [Beignet] Backend visible to other apps?

2013-10-14 Thread Simon Richter
Hi Ian, On 14.10.2013 20:22, Ian Daniher wrote: I'm pretty sure I need to add something to /etc/OpenCL/vendors to make beignet's presence known, but this doesn't seem to be done automatically and documentation is spotty. You need two things: the library needs to be built with ICD support

[Beignet] Debian will freeze soon(tm)

2013-10-13 Thread Simon Richter
Hi, I just got the announcement that Debian is going to freeze the next release soon. Presently, Beignet 0.2 with ICD support and EGL enabled is in the branch being frozen (the more recent stuff is on the separate experimental branch). Since that is a rather old state, I wonder whether I should

Re: [Beignet] [PATCH V2 2/4] Add the internal used kernels for buffer copy

2013-10-09 Thread Simon Richter
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Hi, On 09.10.2013 09:55, junyan...@inbox.com wrote: +macro (MakeKernelBinStr KERNEL_PATH KERNEL_FILES) +foreach (KF ${KERNEL_FILES}) + set (input_file ${KERNEL_PATH}/${KF}.cl) + set (output_file ${KERNEL_PATH}/${KF}_str.c) + list (APPEND

[Beignet] Piglit results

2013-10-02 Thread Simon Richter
Hi, I've run the testsuite against the current Debian packages on a Core i7/IvyBridge2 system, results are at http://psi5.com/~geier/beignet/ Simon ___ Beignet mailing list Beignet@lists.freedesktop.org

[Beignet] Failure to compile more complex kernels

2013-09-28 Thread Simon Richter
Hi, With luxrays, I still keep running into ASSERTION FAILED: it != instrinsicMap.map.end() at file /home/geier/debian/beignet-0.2+git20130928+187c17e/backend/src/llvm/llvm_scalarize.cpp, function bool gbe::Scalarize::scalarizeFuncCall(llvm::CallInst*), line 635 Stack dump: 0. Running

[Beignet] Bug#724808: ocl-icd-libopencl1: Handle invalid object references

2013-09-28 Thread Simon Richter
Package: ocl-icd-libopencl1 Version: 1.3-3+debug Severity: minor Running the Piglit test suite reports several errors where NULL pointers are dereferenced in libOpenCL.so. Apparently, OpenCL implementations are expected to recognize NULL pointers as invalid object references. I could not find

Re: [Beignet] [PATCH 2/3] add clCreateBufferFromLibva() api

2013-09-27 Thread Simon Richter
Hi, This patch and the following patch is to introduce new API to cl lib. It's better to put it into include/CL/cl_intel.h, and the naming rule is to follow other non-standard API : clMapBufferIntel(cl_mem, cl_int*); clXXXIntel. In addition, extension functions need to be listed in the