Media
/
compute-runtime
mirror of https://github.com/intel/compute-runtime.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

refactor(ocl): remove redundant check 

Remove redundant check for graphicsAllocations.size() in set kernel arg
svm method.

Signed-off-by: Dominik Dabek <dominik.dabek@intel.com>
This commit is contained in:
Dominik Dabek 2023-03-15 12:43:30 +00:00 committed by Compute-Runtime-Automation
parent 93a30f002b
commit 5ffb93cf2a
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
opencl/source/kernel/kernel.cpp
View File

@ -949,7 +949,7 @@ cl_int Kernel::setArgSvmAlloc(uint32_t argIndex, void *svmPtr, GraphicsAllocatio
bool disableL3 = false; bool disableL3 = false;
bool forceNonAuxMode = false; bool forceNonAuxMode = false;
bool isAuxTranslationKernel = (AuxTranslationDirection::None != auxTranslationDirection); const bool isAuxTranslationKernel = (AuxTranslationDirection::None != auxTranslationDirection);
auto &rootDeviceEnvironment = getDevice().getRootDeviceEnvironment(); auto &rootDeviceEnvironment = getDevice().getRootDeviceEnvironment();
auto &clGfxCoreHelper = rootDeviceEnvironment.getHelper<ClGfxCoreHelper>(); auto &clGfxCoreHelper = rootDeviceEnvironment.getHelper<ClGfxCoreHelper>();
@ -963,8 +963,8 @@ cl_int Kernel::setArgSvmAlloc(uint32_t argIndex, void *svmPtr, GraphicsAllocatio
forceNonAuxMode = true; forceNonAuxMode = true;
} }
bool argWasUncacheable = kernelArgInfo.isStatelessUncacheable; const bool argWasUncacheable = kernelArgInfo.isStatelessUncacheable;
bool argIsUncacheable = svmAlloc ? svmAlloc->isUncacheable() : false; const bool argIsUncacheable = svmAlloc ? svmAlloc->isUncacheable() : false;
statelessUncacheableArgsCount += (argIsUncacheable ? 1 : 0) - (argWasUncacheable ? 1 : 0); statelessUncacheableArgsCount += (argIsUncacheable ? 1 : 0) - (argWasUncacheable ? 1 : 0);
void *ptrToPatch = patchBufferOffset(argAsPtr, svmPtr, svmAlloc); void *ptrToPatch = patchBufferOffset(argAsPtr, svmPtr, svmAlloc);

2
opencl/source/kernel/multi_device_kernel.cpp
View File

@ -82,7 +82,7 @@ cl_int MultiDeviceKernel::setArgSvmAlloc(uint32_t argIndex, void *svmPtr, MultiG
for (auto rootDeviceIndex = 0u; rootDeviceIndex < kernels.size(); rootDeviceIndex++) { for (auto rootDeviceIndex = 0u; rootDeviceIndex < kernels.size(); rootDeviceIndex++) {
auto pKernel = getKernel(rootDeviceIndex); auto pKernel = getKernel(rootDeviceIndex);
if (pKernel) { if (pKernel) {
if (svmAllocs && (svmAllocs->getGraphicsAllocations().size() <= rootDeviceIndex || !svmAllocs->getGraphicsAllocation(rootDeviceIndex))) { if (svmAllocs && !svmAllocs->getGraphicsAllocation(rootDeviceIndex)) {
continue; continue;
} }
auto svmAlloc = svmAllocs ? svmAllocs->getGraphicsAllocation(rootDeviceIndex) : nullptr; auto svmAlloc = svmAllocs ? svmAllocs->getGraphicsAllocation(rootDeviceIndex) : nullptr;