-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add kernel time tracing support for gpu #381
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PTI has not yet released OpenCL support as of now for its SDK. The current estimation is that it should take about a month to complete. The API will follow the existing one for L0. There will be a proper library to link against as well.
Thanks for the information! We could switch to use the SDK API when the OpenCL support is ready |
if(GC_ENABLE_GPU_PROFILE) | ||
include(ptigpu) | ||
get_property(GC_PTIGPU_BINARY_DIR GLOBAL PROPERTY GC_PTIGPU_BINARY_DIR) | ||
target_link_libraries(GcGpuOclRuntime PRIVATE ${GC_PTIGPU_BINARY_DIR}/lib/libonetrace_tool.so) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
target_link_libraries(GcGpuOclRuntime PRIVATE ${GC_PTIGPU_BINARY_DIR}/lib/libonetrace_tool.so) | |
target_link_libraries(GcGpuOclRuntime PRIVATE onetrace_tool) |
Should this work?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@zhczhong, would it be possible to put the changes inside PTI instead of a fork? A branch maybe?
I tried it but don't have written permission for the PTI repo and cannot create a branch for it. Do you have any suggestion? |
Here's a public branch for the PTI's interface https://github.com/intel/pti-gpu/tree/exp_opencl_0.11.0. @zhczhong, could you please revive this? |
ok! I will switch the profiling support to the one based on the sdk |
3536d6f
to
19dbdda
Compare
This PR depends on intel/pti-gpu#80 for a correct compilation |
4b9bb13
to
188fef9
Compare
188fef9
to
78a1f3c
Compare
Use pti-gpu to trace the OpenCL kernel execution and print the trace result after the program is finished. The feature is default OFF and could be enable by setting
-DGC_ENABLE_GPU_PROFILE=ON
.Example: