AI/rocm-systems
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

SWDEV-270599 - hipExtLaunchKernelGGL Event Timing Incorrect Between Two Launches

Browse Source 
Change-Id: I599a51720f297209072c6be8fffe2aa61e1a071e
This commit is contained in:
Rahul Garg
2021-03-02 15:34:19 +00:00
parent daafd303ec
commit cc17122987
1 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
hipamd/rocclr/hip_event.cpp
+6 -3
View File
@@ -100,6 +100,10 @@ hipError_t Event::elapsedTime(Event& eStop, float& ms) {
if (event_ != eStop.event_ && recorded_ && eStop.recorded_) {
ms = static_cast<float>(static_cast<int64_t>(eStop.event_->profilingInfo().end_ -
event_->profilingInfo().end_))/1000000.f;
} else if (event_ != eStop.event_ && !recorded_ && !eStop.recorded_) {
//It is invalid to use unrecorded events across multiple commands to get time
//For example start and stop events from different hipExtLaunchKernelGGL calls
return hipErrorInvalidHandle;
} else if (event_ == eStop.event_ && (recorded_ || eStop.recorded_)) {
// Events are the same, which indicates the stream is empty and likely
// eventRecord is called on another stream. For such cases insert and measure a
@@ -110,10 +114,9 @@ hipError_t Event::elapsedTime(Event& eStop, float& ms) {
ms = static_cast<float>(static_cast<int64_t>(command->event().profilingInfo().end_ -
event_->profilingInfo().end_))/1000000.f;
command->release();
} else {
// For certain HIP API's that take both start and stop event
// or scenarios where HIP API takes one of the events and the other event is recorded with hipEventRecord
// For certain HIP API's that take both start and stop event
// or scenarios where HIP API takes one of the events and the other event is recorded with hipEventRecord
ms = static_cast<float>(static_cast<int64_t>(eStop.event_->profilingInfo().end_ -
event_->profilingInfo().start_))/1000000.f;
}