OpenCL-CTS icon indicating copy to clipboard operation
OpenCL-CTS copied to clipboard

Extend test coverage for `clClonekernel`

Open jainvikas8 opened this issue 4 years ago • 15 comments

Add the following test coverage for the clCloneKernel

With args: Use clSetKernelArg to set args after kernel is cloned. Enqueue and read the buffer to validate.

With Execinfo: Use clSetKernelExecInfo after the kernel is cloned and read the buffer to validate.

With no args: Clone a kernel with no args and enqueue.

With SVM pointer: Use clSetKernelArgSVMPointer to set args after kernel is cloned. Enqueue and read the buffer to validate.

This fixes #1098 and #1234

This PR also replaces test_error with test_assert_error to report the error during failure.

jainvikas8 avatar May 14 '21 16:05 jainvikas8

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar May 14 '21 16:05 CLAassistant

I'm seeing

fatal: unable to access 'https://github.com/KhronosGroup/libclcxx.git/': Could not resolve host: github.com
Command exited with code 128

How can we restart this?

jainvikas8 avatar May 17 '21 08:05 jainvikas8

@gwawiork, please review.

jainvikas8 avatar May 17 '21 15:05 jainvikas8

I'm seeing

fatal: unable to access 'https://github.com/KhronosGroup/libclcxx.git/': Could not resolve host: github.com
Command exited with code 128

How can we restart this?

I had the same issue on another PR. I ended up re-pushing (on the same branch) to retrigger another run. Hopefully there's a more clever alternative with a simple button to press somewhere, but I couldn't find it.

mantognini avatar May 19 '21 07:05 mantognini

Forced pushed to rebase.

jainvikas8 avatar Jun 02 '21 13:06 jainvikas8

Forced pushed again, CI was stuck

jainvikas8 avatar Jun 10 '21 15:06 jainvikas8

Hi @jainvikas8 I've check your changes on Windows on Intel GPU and I am getting ERROR: clFinish failed! (CL_OUT_OF_RESOURCES from C:\NewVPG\Repo\public-github\opensource-qb\OpenCL-CTS\test_conformance\api\test_clone_kernel.cpp:524) test_exec_enqueue_helper failed for cloneKernel_2 on retry. clCloneKernel test_cloned_kernel_exec_info failed. clone_kernel FAILED

The test process is not able to exit looks like hang. Could you double check on your side if everything is ok ? I will analyze the problem as well.

gwawiork avatar Aug 05 '21 13:08 gwawiork

Forced pushed to rebase.

jainvikas8 avatar Aug 05 '21 15:08 jainvikas8

Hi @jainvikas8 I've check your changes on Windows on Intel GPU and I am getting ERROR: clFinish failed! (CL_OUT_OF_RESOURCES from C:\NewVPG\Repo\public-github\opensource-qb\OpenCL-CTS\test_conformance\api\test_clone_kernel.cpp:524) test_exec_enqueue_helper failed for cloneKernel_2 on retry. clCloneKernel test_cloned_kernel_exec_info failed. clone_kernel FAILED

The test process is not able to exit looks like hang. Could you double check on your side if everything is ok ? I will analyze the problem as well.

Have not observed this in the past, normally it should fail for this too. test_exec_enqueue_helper failed for cloneKernel_1 on the retry Will look into reproducing this issue. Thanks.

jainvikas8 avatar Aug 06 '21 10:08 jainvikas8

Forced pushed to rebase and add 2 new commits to address the review comments.

jainvikas8 avatar Aug 23 '21 14:08 jainvikas8

Hi @jainvikas8 I've check your changes on Windows on Intel GPU and I am getting ERROR: clFinish failed! (CL_OUT_OF_RESOURCES from C:\NewVPG\Repo\public-github\opensource-qb\OpenCL-CTS\test_conformance\api\test_clone_kernel.cpp:524) test_exec_enqueue_helper failed for cloneKernel_2 on retry. clCloneKernel test_cloned_kernel_exec_info failed. clone_kernel FAILED The test process is not able to exit looks like hang. Could you double check on your side if everything is ok ? I will analyze the problem as well.

Have not observed this in the past, normally it should fail for this too. test_exec_enqueue_helper failed for cloneKernel_1 on the retry Will look into reproducing this issue. Thanks.

@gwawiork I've been unable to reproduce this issue on MALI-GPU in Linux with OpenCL 2.1 and 3.0. So, please let me know your thoughts on this?

jainvikas8 avatar Aug 23 '21 14:08 jainvikas8

@jainvikas8 Please take a look at my review when you have the time.

ddabek-i avatar Nov 25 '21 11:11 ddabek-i

[Qualcomm] We tested this and it passed on our 3.0 implementation. Will need more time for a detailed review.

lakshmih avatar May 10 '22 01:05 lakshmih

Removing "focused review" until this PR has an updated owner.

bashbaug avatar Jun 01 '22 20:06 bashbaug

@kpet Rebased and CI checks are passing. @bashbaug Addressed outstanding review comments.

ahesham-arm avatar Jul 16 '24 12:07 ahesham-arm