Commit 25caca9086a12a7658cf401cbc570900c3377977
2012-01-28T16:26:53
Revert "Don't explicitly retain the cl program as it is of no benefit to do so and may lead to problems when trying to release the program." This reverts commit 32910463a3124265b56aca48a6c0fbb107ccfb70. Turns out this does help.
diff --git a/ocl.c b/ocl.c
index d963fc1..28d5a87 100644
--- a/ocl.c
+++ b/ocl.c
@@ -465,6 +465,12 @@ build:
return NULL;
}
+ clRetainProgram(clState->program);
+ if (status != CL_SUCCESS) {
+ applog(LOG_ERR, "Error: Retaining Program (clRetainProgram)");
+ return NULL;
+ }
+
/* create a cl program executable for all the devices specified */
char *CompilerOptions = calloc(1, 256);
@@ -588,6 +594,12 @@ build:
applog(LOG_ERR, "Error: Loading Binary into cl_program (clCreateProgramWithBinary)");
return NULL;
}
+
+ clRetainProgram(clState->program);
+ if (status != CL_SUCCESS) {
+ applog(LOG_ERR, "Error: Retaining Program (clRetainProgram)");
+ return NULL;
+ }
}
free(source);