intel/compute-runtime
1
0
Fork 0
mirror of https://github.com/intel/compute-runtime.git synced 2025-12-20 00:24:58 +08:00
Code Issues Packages Projects Releases Wiki Activity

Revert "Ensure that builtin kernel is patched"

Browse Source 
This reverts commit c9c752526e.

Signed-off-by: Compute-Runtime-Validation <compute-runtime-validation@intel.com>
This commit is contained in:
Compute-Runtime-Validation
2021-06-21 12:12:30 +02:00
committed by Compute-Runtime-Automation
parent 2e97aeccfd
commit 9bc4f02a88
5 changed files with 6 additions and 49 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
opencl/source/helpers/task_information.h
View File

@@ -137,7 +137,6 @@ class CommandComputeKernel : public Command {
CompletionStamp &submit(uint32_t taskLevel, bool terminated) override;
LinearStream *getCommandStream() override { return kernelOperation->commandStream.get(); }
Kernel *peekKernel() const { return kernel; }
protected:
std::vector<Surface *> surfaces;

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

@@ -766,14 +766,6 @@ void Kernel::resizeSurfaceStateHeap(void *pNewSsh, size_t newSshSize, size_t new
localBindingTableOffset = newBindingTableOffset;
}
void Kernel::markArgPatchedAndResolveArgs(uint32_t argIndex) {
if (!kernelArguments[argIndex].isPatched) {
patchedArgumentsNum++;
kernelArguments[argIndex].isPatched = true;
}
resolveArgs();
}
cl_int Kernel::setArg(uint32_t argIndex, size_t argSize, const void *argVal) {
cl_int retVal = CL_SUCCESS;
bool updateExposedKernel = true;
@@ -790,9 +782,13 @@ cl_int Kernel::setArg(uint32_t argIndex, size_t argSize, const void *argVal) {
retVal = (this->*argHandler)(argIndex, argSize, argVal);
}
if (retVal == CL_SUCCESS) {
if (!kernelArguments[argIndex].isPatched) {
patchedArgumentsNum++;
kernelArguments[argIndex].isPatched = true;
}
auto argIsUncacheable = kernelArguments[argIndex].isStatelessUncacheable;
statelessUncacheableArgsCount += (argIsUncacheable ? 1 : 0) - (argWasUncacheable ? 1 : 0);
markArgPatchedAndResolveArgs(argIndex);
resolveArgs();
}
return retVal;
}
@@ -810,11 +806,7 @@ cl_int Kernel::setArg(uint32_t argIndex, cl_mem argVal) {
}
cl_int Kernel::setArg(uint32_t argIndex, cl_mem argVal, uint32_t mipLevel) {
auto retVal = setArgImageWithMipLevel(argIndex, sizeof(argVal), &argVal, mipLevel);
if (retVal == CL_SUCCESS) {
markArgPatchedAndResolveArgs(argIndex);
}
return retVal;
return setArgImageWithMipLevel(argIndex, sizeof(argVal), &argVal, mipLevel);
}
void *Kernel::patchBufferOffset(const ArgDescPointer &argAsPtr, void *svmPtr, GraphicsAllocation *svmAlloc) {

1
opencl/source/kernel/kernel.h
View File

@@ -503,7 +503,6 @@ class Kernel : public ReferenceTrackedObject<Kernel> {
void patchBlocksCurbeWithConstantValues();
void markArgPatchedAndResolveArgs(uint32_t argIndex);
void resolveArgs();
void reconfigureKernel();

25
opencl/test/unit_test/command_queue/enqueue_read_image_tests.cpp
View File

@@ -66,31 +66,6 @@ HWTEST_F(EnqueueReadImageTest, GivenBlockingEnqueueWhenReadingImageThenTaskLevel
EXPECT_EQ(oldCsrTaskLevel, pCmdQ->taskLevel);
}
HWTEST_F(EnqueueReadImageTest, whenEnqueueReadImageThenBuiltinKernelIsResolved) {
UserEvent userEvent{};
cl_event inputEvent = &userEvent;
cl_event outputEvent{};
EnqueueReadImageHelper<>::enqueueReadImage(pCmdQ, srcImage, CL_FALSE,
EnqueueReadImageTraits::origin,
EnqueueReadImageTraits::region,
EnqueueReadImageTraits::rowPitch,
EnqueueReadImageTraits::slicePitch,
EnqueueReadImageTraits::hostPtr,
EnqueueReadImageTraits::mapAllocation,
1u,
&inputEvent,
&outputEvent);
auto pEvent = castToObject<Event>(outputEvent);
auto pCommand = static_cast<CommandComputeKernel *>(pEvent->peekCommand());
EXPECT_TRUE(pCommand->peekKernel()->isPatched());
userEvent.setStatus(CL_COMPLETE);
pEvent->release();
pCmdQ->finish();
}
HWTEST_F(EnqueueReadImageTest, GivenNonBlockingEnqueueWhenReadingImageThenTaskLevelIsIncremented) {
//this test case assumes IOQ
auto &csr = pDevice->getUltCommandStreamReceiver<FamilyType>();

8
opencl/test/unit_test/kernel/kernel_image_arg_tests.cpp
View File

@@ -157,14 +157,6 @@ TEST_F(KernelImageArgTest, givenImageWithWriteOnlyAccessAndReadOnlyArgWhenCheckC
EXPECT_EQ(retVal, CL_INVALID_ARG_INDEX);
}
TEST_F(KernelImageArgTest, givenInvalidImageWhenSettingArgImageThenInvalidArgValueErrorIsReturned) {
cl_mem memObj = reinterpret_cast<cl_mem>(pKernel);
retVal = pKernel->setArg(0, memObj, 0u);
EXPECT_EQ(retVal, CL_INVALID_ARG_VALUE);
}
TEST_F(KernelImageArgTest, givenImageWithReadOnlyAccessAndWriteOnlyArgWhenCheckCorrectImageAccessQualifierIsCalledThenReturnsInvalidArgValue) {
cl_image_format imgFormat = {CL_RGBA, CL_UNORM_INT8};
cl_image_desc imgDesc = {};