Skip to content

Commit

Permalink
Merge pull request #2586 from AllanZyne/review/yang/fix_kernel_release
Browse files Browse the repository at this point in the history
[DeviceASAN] Fix kernel release order
  • Loading branch information
kbenzie authored Jan 22, 2025
2 parents f058cb2 + 1e1d7e5 commit 75eb585
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion source/loader/layers/sanitizer/asan/asan_ddi.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1418,12 +1418,12 @@ __urdlllocal ur_result_t urKernelRelease(
}

getContext()->logger.debug("==== urKernelRelease");
UR_CALL(pfnRelease(hKernel));

auto &KernelInfo = getAsanInterceptor()->getOrCreateKernelInfo(hKernel);
if (--KernelInfo.RefCount == 0) {
UR_CALL(getAsanInterceptor()->eraseKernelInfo(hKernel));
}
UR_CALL(pfnRelease(hKernel));

return UR_RESULT_SUCCESS;
}
Expand Down
2 changes: 1 addition & 1 deletion source/loader/layers/sanitizer/msan/msan_ddi.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1320,12 +1320,12 @@ ur_result_t urKernelRelease(
auto pfnRelease = getContext()->urDdiTable.Kernel.pfnRelease;

getContext()->logger.debug("==== urKernelRelease");
UR_CALL(pfnRelease(hKernel));

auto &KernelInfo = getMsanInterceptor()->getOrCreateKernelInfo(hKernel);
if (--KernelInfo.RefCount == 0) {
UR_CALL(getMsanInterceptor()->eraseKernelInfo(hKernel));
}
UR_CALL(pfnRelease(hKernel));

return UR_RESULT_SUCCESS;
}
Expand Down

0 comments on commit 75eb585

Please sign in to comment.