From 35201ff32d5df3cc301006019cc86d358fae7f5e Mon Sep 17 00:00:00 2001 From: Saleel Kudchadker Date: Fri, 1 Nov 2024 00:46:30 +0000 Subject: [PATCH] SWDEV-483586 - Do not take pinned path for read/write - When GPU_FORCE_BLIT_COPY_SIZE is set do not take pinned path Change-Id: Iaa065db63cc8fda61f82e6c9701e9fdaec5c54cb [ROCm/clr commit: f1e98ab6e445a2dc8b9f2bad33d478e9e19f3ee1] --- projects/clr/rocclr/device/rocm/rocblit.cpp | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/projects/clr/rocclr/device/rocm/rocblit.cpp b/projects/clr/rocclr/device/rocm/rocblit.cpp index e05dd8f6f9..737529aa0e 100644 --- a/projects/clr/rocclr/device/rocm/rocblit.cpp +++ b/projects/clr/rocclr/device/rocm/rocblit.cpp @@ -1832,10 +1832,8 @@ bool KernelBlitManager::readBuffer(device::Memory& srcMemory, void* dstHost, size_t pinSize = size[0]; // Check if a pinned transfer can be executed with a single pin - // If sdmaCopyThreshold is set ignore pinning restrictions and always pin to make sure we use - // Blit for copies - if (((pinSize <= dev().settings().pinnedXferSize_) && (pinSize > MinSizeForPinnedTransfer)) || - (pinSize <= dev().settings().sdmaCopyThreshold_)) { + + if (((pinSize <= dev().settings().pinnedXferSize_) && (pinSize > MinSizeForPinnedTransfer))) { size_t partial; amd::Memory* amdMemory = pinHostMemory(dstHost, pinSize, partial); @@ -1939,10 +1937,7 @@ bool KernelBlitManager::writeBuffer(const void* srcHost, device::Memory& dstMemo size_t pinSize = size[0]; // Check if a pinned transfer can be executed with a single pin - // If sdmaCopyThreshold is set ignore pinning restrictions and always pin to make sure we use - // Blit for copies - if (((pinSize <= dev().settings().pinnedXferSize_) && (pinSize > MinSizeForPinnedTransfer)) || - (pinSize <= dev().settings().sdmaCopyThreshold_)) { + if ((pinSize <= dev().settings().pinnedXferSize_) && (pinSize > MinSizeForPinnedTransfer)) { size_t partial; amd::Memory* amdMemory = pinHostMemory(srcHost, pinSize, partial);