Skip to content
Snippets Groups Projects

Add SYCL Support to CalorimeterIslandCluster

Merged Ujwal Kundur requested to merge ujwalk-sycl-integration into master

GSoC '22 Project - Electromagnetic Cluster Finding on GPUs

SYCL support can be enabled with cmake options: -DUSE_SYCL=ON and specifying a compatible SYCL compiler using -DCMAKE_CXX_COMPILER=<dpcpp>

As of now only Intel OneAPI DPCPP is supported.

eicweb/oneapi_jug_xl:nightly provides a OneAPI environment for running / developing SYCL-enabled code.

More information about the project and the proposal can be found in the student blog here

Edited by Ujwal Kundur

Merge request reports

Approval is optional

Merged by Wouter DeconinckWouter Deconinck 2 years ago (Sep 26, 2022 9:29pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Ujwal Kundur requested review from @wdconinc

    requested review from @wdconinc

  • Author Developer

    Performance Report on SYCL Code

    The following performance studies were done on Compute Canada's HPC Systems on an AMD EPYC 32-core CPU with -O2 flag enabled for both Control (Sequential) and Test (SYCL) code. Simple hotspots collection was performed for both types to determine wall-clock time of execution (includes overhead time). No GPUs were used due to limitations

    Few points to note:

    • Only OpenCL is currently supported and hence performance gains can only be seen on an AMD GPU. CUDA has experimental support for OneAPI and is currently not supported. Since we don't have access to AMD GPUs for testing the following performance runs were carried out on the same AMD EPYC CPUs. Therefore, we see negligible / worse performance from SYCL Code. Take the following with a grain of salt.
    • Testing was carried out with a separate freestanding test build as VTune doesn't work with Gaudi. Test repo here

    Report:

    1. Grouped Barplot of Wall Time vs Number of Events WallTime-compare

    2. Screenshots of VTune Hotspots results perf_cmpr.pdf

  • Ujwal Kundur marked this merge request as ready

    marked this merge request as ready

  • Wouter Deconinck added 2 commits

    added 2 commits

    • 08e2552c - 1 commit from branch master
    • 5613e4a9 - Add SYCL Support to CalorimeterIslandCluster

    Compare with previous version

  • Wouter Deconinck enabled an automatic merge when the pipeline for 5613e4a9 succeeds

    enabled an automatic merge when the pipeline for 5613e4a9 succeeds

Please register or sign in to reply
Loading