diff options
author | Georgios Pinitas <georgios.pinitas@arm.com> | 2019-11-18 14:25:45 +0000 |
---|---|---|
committer | SiCong Li <sicong.li@arm.com> | 2019-11-21 10:23:36 +0000 |
commit | 5f7dda6f415f8f065f86b9f52ee6c5c85bbaa5e4 (patch) | |
tree | 634806550555ee5b2932f2cab161b1e7c6299335 /tests/framework/Framework.cpp | |
parent | c16b30c6f85e2a1ccfd6712701ea1537ea15cf65 (diff) | |
download | ComputeLibrary-5f7dda6f415f8f065f86b9f52ee6c5c85bbaa5e4.tar.gz |
COMPMID-2929: Resolve segfault at end of benchmark suite execution.
Disable usage of RuntimeContext for CL in the test and benchmark suite
as ordering of construction/destruction has been altered causing issues.
Change-Id: I86702350fdabeb6d3a9547c405ee82358255d574
Signed-off-by: Georgios Pinitas <georgios.pinitas@arm.com>
Reviewed-on: https://review.mlplatform.org/c/2314
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Giuseppe Rossini <giuseppe.rossini@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Diffstat (limited to 'tests/framework/Framework.cpp')
-rw-r--r-- | tests/framework/Framework.cpp | 21 |
1 files changed, 8 insertions, 13 deletions
diff --git a/tests/framework/Framework.cpp b/tests/framework/Framework.cpp index 0b2ded8a3c..44887a9125 100644 --- a/tests/framework/Framework.cpp +++ b/tests/framework/Framework.cpp @@ -564,20 +564,15 @@ bool Framework::run() // Every 100 tests, reset the OpenCL context to release the allocated memory if(opencl_is_available() && (id_run_test % 100) == 0) { + auto ctx_properties = CLScheduler::get().context().getInfo<CL_CONTEXT_PROPERTIES>(nullptr); + auto queue_properties = CLScheduler::get().queue().getInfo<CL_QUEUE_PROPERTIES>(nullptr); + + cl::Context new_ctx = cl::Context(CL_DEVICE_TYPE_DEFAULT, ctx_properties.data()); + cl::CommandQueue new_queue = cl::CommandQueue(new_ctx, CLKernelLibrary::get().get_device(), queue_properties); + CLKernelLibrary::get().clear_programs_cache(); - auto cl_ctx = support::cpp14::make_unique<CLRuntimeContext>(); - assert(cl_ctx != nullptr); - CLScheduler *gpu_scheduler = cl_ctx->gpu_scheduler(); - assert(gpu_scheduler != nullptr); - { - // Legacy singletons API: This has been deprecated and the singletons will be removed - // Setup singleton for backward compatibility - CLScheduler::get().init(gpu_scheduler->context(), gpu_scheduler->queue(), cl_ctx->kernel_library().get_device()); - } - if(parameters) - { - parameters->set_cl_ctx(std::move(cl_ctx)); - } + CLScheduler::get().set_context(new_ctx); + CLScheduler::get().set_queue(new_queue); } #endif // ARM_COMPUTE_CL |