<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://jogamp.org/bugzilla/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.2"
          urlbase="https://jogamp.org/bugzilla/"
          
          maintainer="sgothel@jausoft.com"
>

    <bug>
          <bug_id>1302</bug_id>
          
          <creation_ts>2016-04-12 02:58:58 +0200</creation_ts>
          <short_desc>CLKernel missing several constants used to access clGetKernelWorkGroupInfo</short_desc>
          <delta_ts>2023-07-12 01:52:32 +0200</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>3</classification_id>
          <classification>JogAmp</classification>
          <product>Jocl</product>
          <component>opencl</component>
          <version>2.5.0</version>
          <rep_platform>All</rep_platform>
          <op_sys>all</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P4</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Wade Walker">wwalker3</reporter>
          <assigned_to name="Wade Walker">wwalker3</assigned_to>
          <cc>gouessej</cc>
    
    <cc>pierre.diguet</cc>
    
    <cc>sgothel</cc>
    
    <cc>wwalker3</cc>
          
          <cf_type>DEFECT</cf_type>
          <cf_scm_refs></cf_scm_refs>
          <cf_workaround>---</cf_workaround>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>5590</commentid>
    <comment_count>0</comment_count>
    <who name="Wade Walker">wwalker3</who>
    <bug_when>2016-04-12 02:58:58 +0200</bug_when>
    <thetext>We&apos;re missing several constants for newer versions of CL, like CL_KERNEL_COMPILE_WORK_GROUP_SIZE. Add new functions for these, and make sure there aren&apos;t any others missing.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>5591</commentid>
    <comment_count>1</comment_count>
    <who name="ArToX">pierre.diguet</who>
    <bug_when>2016-04-12 09:25:58 +0200</bug_when>
    <thetext>Thanks Wade,

Also feel free to check presence or add other methods for other bindings

    public static final int CL_KERNEL_COMPILE_WORK_GROUP_SIZE = 4529;
    public static final int CL_KERNEL_CONTEXT = 4499;
    public static final int CL_KERNEL_FUNCTION_NAME = 4496;
    public static final int CL_KERNEL_LOCAL_MEM_SIZE = 4530;
    public static final int CL_KERNEL_NUM_ARGS = 4497;
    public static final int CL_KERNEL_PREFERRED_WORK_GROUP_SIZE_MULTIPLE = 4531;
    public static final int CL_KERNEL_PRIVATE_MEM_SIZE = 4532;
    public static final int CL_KERNEL_PROGRAM = 4500;
    public static final int CL_KERNEL_REFERENCE_COUNT = 4498;
    public static final int CL_KERNEL_WORK_GROUP_SIZE = 4528;

or switch the getWorkGroupInfo() to public.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>5598</commentid>
    <comment_count>2</comment_count>
    <who name="Wade Walker">wwalker3</who>
    <bug_when>2016-04-12 22:34:53 +0200</bug_when>
    <thetext>Yes, I plan to add those others, I just need to check that those constants are getting properly translated from the CL header files. There may be a little weirdness because there&apos;s only one version of CLKernel, but there are three versions of the CL object (for v1.1, v1.2, and v2.0), so I&apos;ll need to throw an error if you try to query a non-existent value on an older CL version, but that&apos;s no big deal.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>5603</commentid>
    <comment_count>3</comment_count>
    <who name="Wade Walker">wwalker3</who>
    <bug_when>2016-04-18 02:40:05 +0200</bug_when>
    <thetext>I&apos;ve checked in the fix on my version of the JOCL repo, you can see the commit at https://github.com/WadeWalker/jocl/commit/7a26673570fd77839672d472b04dbd509395de5b.

For now, what I&apos;ve done is add new functions for CL_KERNEL_PREFERRED_WORK_GROUP_SIZE_MULTIPLE and CL_KERNEL_PRIVATE_MEM_SIZE, since those two were added in OpenCL 1.1, which is the version the Java OpenCL objects (the &quot;high-level interface&quot;) present to the user.

For later ones such as CL_KERNEL_GLOBAL_WORK_SIZE (introduced in OpenCL 1.2) I included a new test that shows how to access them. Essentially, you just call the raw OpenCL function instead of using an OO method.

Right now our cluster is offline, so I&apos;ll have to build you an unofficial release. Which platform and bit-width do you need this for? I&apos;ll build it and send you a download link via email.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>5604</commentid>
    <comment_count>4</comment_count>
    <who name="ArToX">pierre.diguet</who>
    <bug_when>2016-04-18 08:35:36 +0200</bug_when>
    <thetext>Hi Wade,

Thanks for the fix, i&apos;m using a win64 platform. I&apos;ll check your git and try the release you&apos;ll send me.

Thanks a lot.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>5605</commentid>
    <comment_count>5</comment_count>
    <who name="Wade Walker">wwalker3</who>
    <bug_when>2016-04-20 04:09:53 +0200</bug_when>
    <thetext>Having a spot of difficulty building the new library, I get some access violations on Windows 10, which I hadn&apos;t tested on before (though it works on Windows 7). I need to track it down and make sure this isn&apos;t a general problem with JOCL on Windows 10, so it may take another few days depending on what&apos;s wrong.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>5607</commentid>
    <comment_count>6</comment_count>
    <who name="Wade Walker">wwalker3</who>
    <bug_when>2016-04-23 18:01:14 +0200</bug_when>
    <thetext>I created a test build and send it as a Dropbox link to the bug reporter. We&apos;ll see if it works for him too. I&apos;m getting some HotSpot crashes in JOCL with the latest nvidia drivers on 64-bit Windows 10, but these seem to be unrelated as far as I can tell. I&apos;m working on that as a separate problem.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>5609</commentid>
    <comment_count>7</comment_count>
    <who name="ArToX">pierre.diguet</who>
    <bug_when>2016-04-29 17:04:56 +0200</bug_when>
    <thetext>Thanks i&apos;ll try it soon, I was a bit off the subject last week.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>6797</commentid>
    <comment_count>8</comment_count>
    <who name="Sven Gothel">sgothel</who>
    <bug_when>2023-07-12 01:52:32 +0200</bug_when>
    <thetext>finally merged w/ Wade&apos;s master branch of 2016 :)</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>