Re: [Gegl-developer] GEGL OpenCL Porting

Hi Nanley,

I was actually giving a look at that this morning and figuring out
with Daniel Sabo if that's really the case.

Pippin, did you delete that code on purpose?


On Wed, Nov 19, 2014 at 11:26 AM, Nanley Chery <nanleychery gmail com> wrote:
It seems like the code to initialize and run the opencl kernel was lost in
this commit:

I'm not familiar enough with the codebase to understand the commit message.
Why was this functionality removed?
Should I add the deleted code into video degradation's process function?


On Wed, Nov 19, 2014 at 12:57 AM, Nanley Chery <nanleychery gmail com>

I noticed there was more to the brightness-contrast example. I made the
adjustments concerning the kernel name and parameter values.
The code compiles now. The current problem that I'm experiencing is that
the test for video-degradation passes with an empty
I'm not sure which code paths are executing to make this work. Any
pointers? I'll do some grepping of the source tree in the meantime.


On Tue, Nov 18, 2014 at 8:22 PM, Nanley Chery <nanleychery gmail com>

Wow. Thank you for the tip, CL_CHECK is now giving me an output.

This is the error message:
(lt-gegl:10486): GEGL-video-degradation.c-WARNING **: Error in
video-degradation.c:236 cl_process - invalid kernel

I thought that I had followed the kernel compilation process correctly.
Do you notice any mistake? I have pushed my latest change to the branch.


On Tue, Nov 18, 2014 at 8:06 PM, Victor Oliveira
<victormatheus gmail com> wrote:

Hi Nanley,

I'd recommend you follow operations/common/brightness-contrast.c file
for a point-filter operation (i.e. a pixel-wise filter) instead of
doing what you did.

Notice that in operations/common/brightness-contrast.c#n153 there's a
string brightness_contrast_cl_source which is a string in
opencl/, these are auto-generated files from
the kernels in the opencl folder.

Let me know what happens from that.


On Tue, Nov 18, 2014 at 4:45 PM, Nanley Chery <nanleychery gmail com>
Hi Victor,

Thank you very much for taking a look. I understand about the time.

Here's the link to my bitbucket branch:

The latest commit is what's causing the video-degradation.xml test to
(I'm testing using


On Tue, Nov 18, 2014 at 5:11 PM, Victor Oliveira
<victormatheus gmail com>

Hi Nanley,

Just to let you know, I'll need some time to answer that because I'll
need to build GIMP on my new laptop.

Can you share your code so I can give a look?


On Tue, Nov 18, 2014 at 12:49 PM, Nanley Chery
<nanleychery gmail com>
Hi Victor,

I'm a student working on OpenCL porting work for my High
Computing class. I'm trying to implement an OpenCL port for the
newly-committed video-degradation operation. Are you willing to
guidance on the following roadblock?

The issue that I'm finding is that creating a cl_process method and
the following variables in gegl_op_class_init is not enough to get
cl_process method called:

operation_class->opencl_support = TRUE;
point_filter_class->cl_process = cl_process;

If I manually try to call the cl_process function in the process
(like in edge-laplace.c), the program terminates in the
gegl_cl_set_kernel_args method without an error from CL_CHECK;

Is there something I'm missing? I apologize for mailing you
of writing to the mailing list. I'm a little pressed for time, so I
for this option.


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]