From 5f7dda6f415f8f065f86b9f52ee6c5c85bbaa5e4 Mon Sep 17 00:00:00 2001 From: Georgios Pinitas Date: Mon, 18 Nov 2019 14:25:45 +0000 Subject: 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 Reviewed-on: https://review.mlplatform.org/c/2314 Tested-by: Arm Jenkins Reviewed-by: Giuseppe Rossini Comments-Addressed: Arm Jenkins --- tests/framework/Framework.cpp | 21 ++++++++------------- tests/main.cpp | 16 +++++----------- 2 files changed, 13 insertions(+), 24 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(nullptr); + auto queue_properties = CLScheduler::get().queue().getInfo(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(); - 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 diff --git a/tests/main.cpp b/tests/main.cpp index 5757249188..604a51fc0f 100644 --- a/tests/main.cpp +++ b/tests/main.cpp @@ -185,18 +185,12 @@ int main(int argc, char **argv) #ifdef ARM_COMPUTE_CL CLTuner cl_tuner(false); - // Create GPU context - auto cl_ctx = support::cpp14::make_unique(); - assert(cl_ctx != nullptr); - CLScheduler *gpu_scheduler = cl_ctx->gpu_scheduler(); - assert(gpu_scheduler != nullptr); - const auto device_version = cl_ctx->kernel_library().get_device_version(); + if(opencl_is_available()) { - // 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(), &cl_tuner); + auto ctx_dev_err = create_opencl_context_and_device(); + ARM_COMPUTE_ERROR_ON_MSG(std::get<2>(ctx_dev_err) != CL_SUCCESS, "Failed to create OpenCL context"); + CLScheduler::get().default_init_with_context(std::get<1>(ctx_dev_err), std::get<0>(ctx_dev_err), &cl_tuner); } - parameters->set_cl_ctx(std::move(cl_ctx)); if(enable_tuner->is_set()) { @@ -235,7 +229,7 @@ int main(int argc, char **argv) #ifdef ARM_COMPUTE_CL if(opencl_is_available()) { - p->print_entry("CL_DEVICE_VERSION", device_version); + p->print_entry("CL_DEVICE_VERSION", CLKernelLibrary::get().get_device_version()); } else { -- cgit v1.2.1