Add spec file for Tizen (#1)

Change-Id: I68d77d5783d79f7178a4919bd8e9999fec931f95
Signed-off-by: Xuelian Bai <xuelian.bai@samsung.com>

Fix: stat was done on an incomplete file path. (#161)

Pedantic (#159)

* Replace usage of `d_type` of `struct dirent` by `stat()`.

* Remove unsigned comparison warning.

* Remove warning conversion for function pointers.

* Fix missing initializers.

* Remove unused variable warnings.

* Update loader/linux/icd_linux.c

Co-authored-by: Ronan Keryell <ronan@keryell.fr>
Co-authored-by: Ronan Keryell <ronan@keryell.fr>

Remove const warning (#158)

* Remove spurious trailing spaces

* Add missing const on the pointed data

This removes compilation warning:
Building C object _deps/ocl-icd-build/CMakeFiles/OpenCL.dir/loader/linux/icd_linux.c.o
In file included from _deps/ocl-icd-src/loader/icd.h:22,
                 from _deps/ocl-icd-src/loader/linux/icd_linux.c:19:
_deps/ocl-icd-src/loader/linux/icd_linux.c: In function ‘khrIcdOsVendorsEnumerate’:
_deps/ocl-icd-src/loader/icd_platform.h:29:25: warning: initialization discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers]
   29 | #define ICD_VENDOR_PATH "/etc/OpenCL/vendors";
      |                         ^~~~~~~~~~~~~~~~~~~~~
_deps/ocl-icd-src/loader/linux/icd_linux.c:43:24: note: in expansion of macro ‘ICD_VENDOR_PATH’
   43 |     char* vendorPath = ICD_VENDOR_PATH;
      |                        ^~~~~~~~~~~~~~~

declare ret_val global variable static (#147)

It would be better to remove all global ret_val variables, but
this would be a much bigger change.

Fix compiler warnings (#148)

* Fix compiler warnings

* Define and use printf format macros for DWORD and ULONG.

Co-authored-by: Brice Videau <bvideau@anl.gov>

update ICD loader generation scripts to handle empty arguments (#150)

Change OCL_ICD_LAYERS to OPENCL_LAYERS to conform to the specification. (#145)

do not add the API suffix macro to function definitions (#124)

Remove cl_gl_ext.h includes (#144)

All OpenGL extensions are now in cl_gl.h.

Replace uses of CL_EXT_{PRE,SUF}FIX* with CL_API_{PRE,SUF}FIX* (#142)

Generation done using https://github.com/KhronosGroup/OpenCL-Docs/pull/569

Signed-off-by: Kévin Petit <kpet@free.fr>

Removed uneeded lines in Cmakelists.txt file. (#141)

Removed cl_icd_layer.h as it is now in the OpenCL-Headers repository under CL/cl_layer.h. (#140)

Fixed generators to better account for the lack of return value of clSVMFree. (#139)

* Fixed generators to better account for the lack of return value of clSVMFree.

* Added generated file.

Add C99 flag as last commit added some c99 variable declarations (#135)

* Add C99 flag as last commit added some c99 variable declarations

Signed-off-by: John Kesapides <john.kesapides@arm.com>
* Add CMAKE_C_STANDARD_REQUIRED

Signed-off-by: John Kesapides <john.kesapides@arm.com>

Layers (#125)

* remove CMake option for OpenCL 3.0 provisional

* generate the OpenCL 3.0 APIs also

* Added layers.

* Added test print layer.

* Added missing cpp guards to cl_icd_layer.h.

* Rearrange includes in icd_windows.c to ensure DXGI.h has INITGUID defined when it's included

* Fix extern "C" usage in Windows build

* Ensure platforms are initialized before initializing layers.

* Fix faulty merge.

* Fix copy paste errors.

* Added ENABLE_OPENCL_LAYERS option to CMakeLists.

* Activate layers by default and fix bad copy paste...

* Added formal definition of Layer API entry points.

* Better variable name.

* Added a section dedicated to layers in the README.

* Changed value to proposed reserved enum range.

* Bumped version number for windows release.

* Added guard for MacOS.

Co-authored-by: Ben Ashbaugh <ben.ashbaugh@intel.com>
Co-authored-by: Jesse Natalie <jenatali@microsoft.com>

Add missing includes (#131)

* Add missing includes

This is causing problems with recent Xcode version on macOS.

* Enable macOS in GitHub Actions

Switch from Travis CI to Github Actions (#132)

Add force fPIC option

Minor fixes to Windows build to improve header include order resiliency (#126)

* Rearrange includes in icd_windows.c to ensure DXGI.h has INITGUID defined when it's included

* Fix extern "C" usage in Windows build

remove CMake option for OpenCL 3.0 provisional (#121)

* remove CMake option for OpenCL 3.0 provisional

* generate the OpenCL 3.0 APIs also

Revert "Merge branch 'master' into master"

This reverts commit 97f0eb551154ac28decc89f546eb9f4082948887, reversing
changes made to 98ca71fb9f8484f1cd1999f55224bf9e8d18693b.

Merge branch 'master' into master

Define CL_USE_DEPRECATED_OPENCL_2_{1,2}_APIS (#122)

skip testing except for icd loader project (#117)

* skip testing except for icd loader project

* add option for testing when embedding the loader in other projects

Generate the ICD Loader Dispatch Functions (#114)

* initial version of generated ICD dispatch code

* add new OpenCL 3.0 API clSetContextDestructorCallback

* add a README describing how to generate files
increment the file version

* address review comments

correctly pass user_data to destructor callbacks

add clSetContextDestructorCallback

Update and modernize some aspects of CMake (#92)

* Update and modernize some aspects of CMake

Ease integration with projects targeting more recent CMake versions:

* Update minimum required CMake version to 3.1.
* Specify a version `1.2` to the `project` CMake command, this silences
  developer warnings when the OpenCL-ICD-Loader is integrated in another
  project using `add_subdirectory`.
* Specify include directory on the `OpenCL` target, this removes the
  need to set the OpenCL-Headers include directory globally (old CMake)
  or set them manually per target which links the `OpenCL` target
  library.

* Update AppVeyor project config

runtimeobject.lib should be lower case

expand ifdefs to remove include file dependencies

add an option to disable support for OpenCLOn12

setting this option breaks a dependency on the Windows SDK

Add support for OpenCL 3.0 Provisional (#108)

* Add support for experimental entry points

The new entry points will only be enabled if `CL_EXPERIMENTAL` is manually set.

* Update experimental implementation points to 3.0 provisional

* Use configure_file to selectively include symbols

remove WDK dependency for OpenCL ICD loader (#102)

* remove dependency on Windows WDK for Windows ICD Loader builds

After this change, instead of including a WDK header file, a
private header file is used with just the definitions needed
for the ICD loader.  The header file is copied unchanged from
the Vulkan loader repo.

* update README to remove WDK dependency

* updated file version to v2.2.8

Add support for OpenCLOn12 ICD (#103)

* Add search for OpenCLOn12 mapping layer package.

* Use altered search path for ICD loading.

* Update version to 2.2.7

Add a path separator to OCL_ICD_VENDORS

When using the `OCL_ICD_VENDORS` environment variable a trailing `/`
path separator must be provided by the user, this is surprising.

This patch updates the `sprintf` call to include a `/` when constructing
the `.icd` file path. It also removes the trailing `/` from instances of
the `ICD_VENDOR_PATH` to canonicalised path strings. Existing scripts
which specify a trailing `/` when setting `OCL_ICD_VENDORS` will
continue working as expected since duplicate path separators `//`, while
not canonical, work as if a single path separator is specified.

Make ret_val and ret_mem local to the functions in which they are used.

Corrected inconsistent indentation

Incorporating PR refinement requests so REG_MULTI_SZ is handled in the HKR codepath, adding KHR_ICD_TRACE messages and incrementing the revision version number

Fix to allow windows ICD detection to detect AMD ICD drivers

Use API function pointer and dispatch table definitions from the CL headers

Depends on https://github.com/KhronosGroup/OpenCL-Headers/pull/50.

Fixes #72.

Signed-off-by: Kevin Petit <kevin.petit@arm.com>

Fix indentation with previous changes

Fix whitespaces in indentation added by
previous commit
0b3756e8ac11fb35e696e13d3cfdbc2e699cb6eb

Merge branch 'origin/para-virtual-fixes'

Update nikhiljnv:master with Khronos:master

Merge branch 'KhronosGroup-master'

Remove link-time dependency on WDK

Recent changes for using QueryAdapter and DXGI APIs
for enumerating platforms on Windows added a dependency on
WDK for linking OpenCL driver to Windows specific
libraries.

Querying function address using GetProcAddress for new QueryAdapterInfo
function so that there is no need to link to WDK while
building OpenCL ICD loader.

Also, removing knob OPENCL_ICD_LOADER_REQUIRE_WDK
needed because of link-time dependency on WDK.

Update README to reflect the changes wrt WDK dependency.

While at it -
1. Fix variable names to use camelCase.
2. Use realloc instead of malloc for reallocating memory for adapters.

Update branch with Khronos GitHub Master branch.
Merge https://github.com/KhronosGroup/OpenCL-ICD-Loader into new_para_virtual

Use QueryAdapter APIs as per MS directives

Update ICD loader to use QueryAdapter APIs
as per MS directives for para-virtualization.

This includes changes to -
1. Order OpenCL platforms consistent with Windows adapters,
   to honor user-settings for multi-adapter machines.
2. Fix OS version detection.
3. Some minor fixes including
    1. Add trace prints at required places.
    2. Fix build failure in getting the address of D3DKMTQueryAdapterInfo.
    3. Fix early return platform enumeartion if there are no entries found for old reg key
       under "HKLM\\SOFTWARE\\Khronos\\OpenCL\\Vendors" to allow add for the entries found
       using DXGK interface and HKR entries.
4. Some other fixes including
    1. Update README.md with WDK dependency information.
    2. Move AddAdapter* call inside for which got misplaced while resolving conflicts.
    3. Free WinAdapter allocations and avoid overriding NULL platform.
5. Fixes based on review comments including -
    1. WinAdapter is reallocated with previous allocation freed.
       However, the final allocation is not freed.
       Fixed the same to free at the end of enumeration.
    2. Fix header file ordering by including windows headers
       inside necessary header files.

Minor fixes to para-virtualization support

Changes to following aspects of para-virtualization changes -
1. Revert platform[0] override when platform is NULL

Fix variable naming, array size and cleanup paths.

Fix changes added as part of para_virtualization support
1. to use camelCase for variable names
2. Use sizeof(element) while calculating array size.
3. Free/unload allocations/libraries during cleanup.

Add return value checks for adapterAdd.

Return boolean result from adapterAdd API
and add checks for success of this API
for better error reporting.

Free adapter names and add check for DXGI interface queries

Add adapterFree to free memory allocated to hold WinAdapter
handles.
Also, check for successful query of DXGIFactory interfaces.

Minor fixes to cleanup Windows para-virtualization changes

Fix indentation and variable name casing.
Fix adapterFree to remove null-pointer check and set it to null after
free.
Add missing header required for adapterAdd.
Add checks for malloced pointers.

Bump up OpenCL ICD Loader version

Bump up OpenCL ICD Loader version
with para-virtualization changes.

Update status based on DXGK and HKR enumearation

khrIcdOsVendorsEnumerate currently does not consider
the status returned by DXGK and HKR enumeration
and returns TRUE only if HKLM regkey enumeration
results in successful adapterAdd.

Fix the returned status to consider successful
adapterAdd from DXGK and HKR enumeration.

Also, set freed adapger pointers to NULL to avoid
potential double-free issue.

Refactor ICD loader windows header files

Add icd_windows.h to include Windows specific
types and functions.
Keep icd_windows_hkr.h and icd_windows_dxgk.h
limited to the HKR and DXGK specific information.

Refactor ICD loader windows header files

Add icd_windows.h to include Windows specific
types and functions.
Keep icd_windows_hkr.h and icd_windows_dxgk.h
limited to the HKR and DXGK specific information.

Update status based on DXGK and HKR enumearation

khrIcdOsVendorsEnumerate currently does not consider
the status returned by DXGK and HKR enumeration
and returns TRUE only if HKLM regkey enumeration
results in successful adapterAdd.

Fix the returned status to consider successful
adapterAdd from DXGK and HKR enumeration.

Also, set freed adapger pointers to NULL to avoid
potential double-free issue.

Bump up OpenCL ICD Loader version

Bump up OpenCL ICD Loader version
with para-virtualization changes.

Merge pull request #10 from KhronosGroup/master

added checks for high integrity for Windows secure_getenv

Minor fixes to cleanup Windows para-virtualization changes

Fix indentation and variable name casing.
Fix adapterFree to remove null-pointer check and set it to null after
free.
Add missing header required for adapterAdd.
Add checks for malloced pointers.

added checks for high integrity for Windows secure_getenv

Free adapter names and add check for DXGI interface queries

Add adapterFree to free memory allocated to hold WinAdapter
handles.
Also, check for successful query of DXGIFactory interfaces.

Add return value checks for adapterAdd.

Return boolean result from adapterAdd API
and add checks for success of this API
for better error reporting.

Fix variable naming, array size and cleanup paths.

Fix changes added as part of para_virtualization support
1. to use camelCase for variable names
2. Use sizeof(element) while calculating array size.
3. Free/unload allocations/libraries during cleanup.

Minor fixes to para-virtualization support

Changes to following aspects of para-virtualization changes -
1. Revert platform[0] override when platform is NULL

Merge branch 'KhronosGroup-master' into new_para_virtual

Update nikhiljnv:new_para_virtual with Khronos master

Merge branch 'master' of https://github.com/KhronosGroup/OpenCL-ICD-Loader into KhronosGroup-master

Install to standard locations using GNUInstallDirs

Different distributions may specify different paths; using GNUInstallDirs ensures that everything installed to the correct location for all systems. For example, amd64 Debian uses lib64 as opposed to lib, so hardcoding lib would be wrong for Debian.

Use QueryAdapter APIs as per MS directives

Update ICD loader to use QueryAdapter APIs
as per MS directives for para-virtualization.

This includes changes to -
1. Order OpenCL platforms consistent with Windows adapters,
   to honor user-settings for multi-adapter machines.
2. Fix OS version detection.
3. Some minor fixes including
    1. Add trace prints at required places.
    2. Fix build failure in getting the address of D3DKMTQueryAdapterInfo.
    3. Fix early return platform enumeartion if there are no entries found for old reg key
       under "HKLM\\SOFTWARE\\Khronos\\OpenCL\\Vendors" to allow add for the entries found
       using DXGK interface and HKR entries.
4. Some other fixes including
    1. Update README.md with WDK dependency information.
    2. Move AddAdapter* call inside for which got misplaced while resolving conflicts.
    3. Free WinAdapter allocations and avoid overriding NULL platform.
5. Fixes based on review comments including -
    1. WinAdapter is reallocated with previous allocation freed.
       However, the final allocation is not freed.
       Fixed the same to free at the end of enumeration.
    2. Fix header file ordering by including windows headers
       inside necessary header files.

increase loader rev for environment variable support

need to include a lower-case windows.h for cross-compilation

initial ICD loader environment variable support (#77)

* basic ICD loader env var support for OCL_ICD_VENDORS

* added envvars source files

* added OCL_ICD_FILENAMES support

* added README section describing debug environment variables

* refactored platform specific defines into icd_platform.h

* disable tracing

Merge pull request #7 from KhronosGroup/master

Merge master branch of Khronos

Windows 10 RS4 Support for the OpenCL ICD Loader w/ Build Option (#78)

* Windows 10 RS4 support for OpenCL ICD as per MS directive

Motivation - MS is adopting DCHU complaince and for For RS4 it will be mandatory for OpenCL to load via registry keys contained only within the HKR section of the registry as writes outside of HKR will not be allowed for the INF. It is also to support ParaV feature of the OS to make it "virtualization-ready".

* Update Copyright year
* Fix vendors enumeration on 32-bit Windows OS
* Update CMake file for detecting necessary includes for the Windows WDK
* Add option to build with or without the WDK (default: with)
* Documented Windows WDK dependency

add CI badges for travis-ci and appveyor

basic appveyor enabling

simple travis CI enabling

fix README typos

removed non-CMake build files
updated README with CMake build instructions

Use QueryAdapter APIs as per MS directives

Update ICD loader to use QueryAdapter APIs
as per MS directives for paravirtualization.

Merge branch 'skudchad-ocl-wddm24'
of https://github.com/BKlamik-MSFT/OpenCL-ICD-Loader

Windows 10 RS4 support for OpenCL ICD as per MS directive

These are set of changes required by MS for Para-Virtualization
support for Windows 10 RS4.
Pull changes in pull request
https://github.com/KhronosGroup/OpenCL-ICD-Loader/pull/33

Downstream MinGW fixes

* Don't include DX SDK at all on MinGW (MinGW have their own headers)
* Lower case header name for cross-compilation on Linux

Create directory tree, fix warnings, cmake improvements (#58)

* fix make test

* include CTest and wrap tests with BUILD_TESTING

* move loader files to their own directory

* create specific directories for Windows and Linux

* cmake improvements

* fix path to map file for linux build

* add definition for CL_TARGET_OPENCL_VERSION

* fix warnings

* don't use target_link_options for cmake compatibility

* remove SHARED from a few cmake files

Option to allow for dynamic linking with VC++ Runtime (#60)

By default ICD loader will use static linking. However, you can specifiy
cmake parameter USE_DYNAMIC_VCXX_RUNTIME to switch to dynamic. For
example:
  cmake -DUSE_DYNAMIC_VCXX_RUNTIME=TRUE ..

Signed-off-by: Artur Harasimiuk <artur.harasimiuk@intel.com>

Do parameter validation in clGetExtensionFunctionAddress

Also undefine the local macros as soon as possible.

Patch originally from Jeroen Ketema.

Statically link with VC++ Runtime (#57)

* Statically link with VC++ Runtime

VC++ Runtime dynamic library may not be available in some cases. Thus,
by doing static linking it makes dll independent of already installed
software and can be used without problems.

Change-Id: I8bb71db2f3dc48bc9ec976cdbb597d2ec812ce54
Signed-off-by: Artur Harasimiuk <artur.harasimiuk@intel.com>
* Extend static linking of VC++ Runtime

adding MinSizeRel and RelWithDebInfo targets

Change-Id: I740002ca47ad4a65548b9fcad87817f730511c13
Signed-off-by: Artur Harasimiuk <artur.harasimiuk@intel.com>

fix compilation with VS2010

Update icd_windows.c (#51)

Return TRUE from khrIcdOsVendorsEnumerate() even if RegOpenKeyExA() fails to open the key.

Change header license to Apache 2.0 (#53)

switch README from plain text to markdown

no major content updates, just minor wording fixes.

Create LICENSE

Delete LICENSE.txt

Fix bugs encountered while building with VS 2017 64bit (#39)

Use PROJECT_SOURCE_DIR in CMakefile

Instead of using CMAKE_CURRENT_SOURCE_DIR. This facilitates using the
ICD loader as a subproject of another project.

Update .gitignore to ignore build directory (#42)

Add option to build static library.

Also add explanatory comment on why shared library should be preferred.

add extension function comments

As per working group discussion, most extension functions do not need to
be "ICD-aware", and hence do not need to be handled in the ICD loader
and dispatch tables.

Support building on macOS (#40)

Use the the linux code in that case.

The resulting library hasn't been tested but it is already
useful as is as a library to link against to build-test
things like the CTS on macOS.

Signed-off-by: Kevin Petit <kevin.petit@arm.com>

Create CODE_OF_CONDUCT.md

Fix vendors enumeration on 32-bit Windows OS (#28)

When running 32-bit OpenCL applications on a 32-bit OS, we need to
use the registry keys without the "Wow" suffix. On 64-bit OSes and
32-bit applications, OTOH, we must use the Wow suffix.

Remove GL/EGL headers

driver stub: Fix dangling & NULL ptr dereference (#27)

1. NULL pointer dereference
2. dangling pointer dereference
3. Memory leak (newly created device isn't returned to user)
4. Ignoring errors + invalid error logging

Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>

Static analysis fixes.

Added OpenCL Universal Driver Support for Win10 RS3 (#21)

Starting from Windows 10 RS3+ Graphics Drivers are no longer allowed to
write HKLM entries, and hence the current registry key:
HKLM\SOFTWARE\Khronos\OpenCL\Vendors

Is no longer applicable to graphics drivers for registering their ICD.
Instead, graphics drivers can write to either of the 2 registry keys
below:
1) Display Adapter HKR
2) Software Components HKR

Graphics Drivers (starting from RS3) have dedicated component INF files and are
referenced via the CopyINF directive and/or (optional) "INF AddComponent directive".

Every such component, e.g. OpenCL instance (ICD), is assigned a unique device
instance ID (e.g. 0000, 0001, etc) under the "Software Component" GUID and use it to
store the path to its ICD.

Therefore we enumerate ICDs in the following places (and also in the following order):
1) Display Adapters HKRs (using the display adapter GUID)
2) Software Component HKRs (using the CM_* APIs to traverse the device tree)
3) The usual "Vendors" registry key (see above)

Corresponding ICD extension updates were made with KhronosGroup/OpenCL-Registry#30.

Allow building ICD loader as a CMake subproject.

Tweaked CMakeLists.txt to make it possible to build OpenCL-ICD-Loader as a
subproject.

Update Windows DLL version to 2.2 (#19)

Add OpenCL 2.2 API to ICD Loader.

correct the case that properties is NULL for clCreateContextFromType (#8)

Handle properties==NULL in clCreateContextFromType

According to spec, for function clCreateContextFromType, properties can also be
NULL in which case the platform that is selected is implementation-defined,
while the current code returns CL_INVALID_PLATFORM.