-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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 NVTX instrumentation #54873
Add NVTX instrumentation #54873
Conversation
Currently failing due to a linker error:
|
This is now working thanks to @giordano! |
How can I test this? I need to use https://github.com/JuliaGPU/NVTX.jl and install Nsight Systems? |
Co-authored-by: Mosè Giordano <[email protected]>
You don't need NVTX.jl, just Nsight Systems (the .run file is pretty easy to install). A simple test case
|
cc @maleadt (in case you hadn't already seen it) |
It took me a while to get a usable Nsight Systems on the Grace Hopper machine (@fspiga suggested me to simply use I need to practice more to get used to the interface, but it looks cool! |
@simonbyrne do I understand correctly this doesn't support JIT-ed functions? When calling a user function in the list of events I see loads of names like |
Yes, this just instruments the compiler and runtime. |
Should we merge this? |
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.
I haven't tested this, but LGTM.
Allow profiling with Nsight Systems. To try it out, add ``` WITH_NVTX=1 ``` to `Make.user` --------- Co-authored-by: Mosè Giordano <[email protected]>
Allow profiling with Nsight Systems.
I've added the headers to the tree until I can figure out how thedeps
stuff works.To try it out, add
to
Make.user