Only pass buffer location property if in simulation environment

ericxu233 · pcolberg · commit f8b6babcf6ff · 2022-07-20T18:43:17.000-04:00
This addresses #147 (comment)
diff --git a/src/acl_usm.cpp b/src/acl_usm.cpp
@@ -150,8 +150,12 @@ CL_API_ENTRY void *CL_API_CALL clHostMemAllocINTEL(
     {
       auto mmd_properties_it = mmd_properties.begin();
       if (mem_id) {
-        *mmd_properties_it++ = AOCL_MMD_MEM_PROPERTIES_BUFFER_LOCATION;
-        *mmd_properties_it++ = *mem_id;
+        int use_offline_only;
+        acl_get_offline_device_user_setting(&use_offline_only);
+        if (use_offline_only == ACL_CONTEXT_MPSIM) {
+          *mmd_properties_it++ = AOCL_MMD_MEM_PROPERTIES_BUFFER_LOCATION;
+          *mmd_properties_it++ = *mem_id;
+        }
       }
       *mmd_properties_it++ = 0;
     }
@@ -436,8 +440,12 @@ clSharedMemAllocINTEL(cl_context context, cl_device_id device,
     {
       auto mmd_properties_it = mmd_properties.begin();
       if (mem_id) {
-        *mmd_properties_it++ = AOCL_MMD_MEM_PROPERTIES_BUFFER_LOCATION;
-        *mmd_properties_it++ = *mem_id;
+        int use_offline_only;
+        acl_get_offline_device_user_setting(&use_offline_only);
+        if (use_offline_only == ACL_CONTEXT_MPSIM) {
+          *mmd_properties_it++ = AOCL_MMD_MEM_PROPERTIES_BUFFER_LOCATION;
+          *mmd_properties_it++ = *mem_id;
+        }
       }
       *mmd_properties_it++ = 0;
     }

Original file line number	Diff line number	Diff line change
`@@ -150,8 +150,12 @@ CL_API_ENTRY void *CL_API_CALL clHostMemAllocINTEL(`
`150`	`150`	`{`
`151`	`151`	`auto mmd_properties_it = mmd_properties.begin();`
`152`	`152`	`if (mem_id) {`
`153`		`- *mmd_properties_it++ = AOCL_MMD_MEM_PROPERTIES_BUFFER_LOCATION;`
`154`		`- mmd_properties_it++ = mem_id;`
	`153`	`+ int use_offline_only;`
	`154`	`+ acl_get_offline_device_user_setting(&use_offline_only);`
	`155`	`+ if (use_offline_only == ACL_CONTEXT_MPSIM) {`
	`156`	`+ *mmd_properties_it++ = AOCL_MMD_MEM_PROPERTIES_BUFFER_LOCATION;`
	`157`	`+ mmd_properties_it++ = mem_id;`
	`158`	`+ }`
`155`	`159`	`}`
`156`	`160`	`*mmd_properties_it++ = 0;`
`157`	`161`	`}`
`@@ -436,8 +440,12 @@ clSharedMemAllocINTEL(cl_context context, cl_device_id device,`
`436`	`440`	`{`
`437`	`441`	`auto mmd_properties_it = mmd_properties.begin();`
`438`	`442`	`if (mem_id) {`
`439`		`- *mmd_properties_it++ = AOCL_MMD_MEM_PROPERTIES_BUFFER_LOCATION;`
`440`		`- mmd_properties_it++ = mem_id;`
	`443`	`+ int use_offline_only;`
	`444`	`+ acl_get_offline_device_user_setting(&use_offline_only);`
	`445`	`+ if (use_offline_only == ACL_CONTEXT_MPSIM) {`
	`446`	`+ *mmd_properties_it++ = AOCL_MMD_MEM_PROPERTIES_BUFFER_LOCATION;`
	`447`	`+ mmd_properties_it++ = mem_id;`
	`448`	`+ }`
`441`	`449`	`}`
`442`	`450`	`*mmd_properties_it++ = 0;`
`443`	`451`	`}`