-
Notifications
You must be signed in to change notification settings - Fork 39
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
RPP Audio Support HIP - Spectrogram #433
base: develop
Are you sure you want to change the base?
RPP Audio Support HIP - Spectrogram #433
Conversation
r-abishek
commented
Aug 22, 2024
•
edited
Loading
edited
- Updates version to RPP 1.9.3
- Adds RPP audio support for Spectrogram on HIP backend.
- Adds relevant qa tests and performance tests.
made changes to compute num_windows in executor instead of recomputing everytime inside kernel
combined processing of FT and TF layouts in single kernel
made changes to compute windowOutput stride from dstDescPtr
Spectrogram HIP TOT Merge
Spectrogram HIP TOT Merge
@r-abishek -- merge conflicts |
RPP Spectrogram HIP TOT Merge
// copy the hanning window values to hip memory | ||
Rpp32f *d_windowFn = handle.GetInitHandle()->mem.mgpu.scratchBufferHip.floatmem; | ||
CHECK_RETURN_STATUS(hipMemcpyAsync(d_windowFn, windowFn, windowLength * sizeof(Rpp32f), hipMemcpyHostToDevice, handle.GetStream())); | ||
CHECK_RETURN_STATUS(hipStreamSynchronize(handle.GetStream())); |
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.
Please remove CHECK_RETURN_STATUS(hipStreamSynchronize(handle.GetStream())));
from here and line 198 and move it to after the last hipLaunchKernelGGL call.
…goes out of bounds in HOST kernel
Spectrogram HIP changes based on review comments
@AryanSalmanpour This PR addresses verbal comments from the code review now |
All Jenkins CI passing. Waiting on Azure - https://dev.azure.com/ROCm-CI/ROCm-CI/_build/results?buildId=10403&view=results |
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.
Looks good to me.
…upport_6_spectrogram_hip