add more testing and return default global mem instead of 0

Author: sherry-yuan

src/acl_usm.cpp

@@ -240,7 +240,7 @@ clDeviceMemAllocINTEL(cl_context context, cl_device_id device,
   // Iterate over properties.
   // The end of the properties list is specified with a zero.
   cl_mem_alloc_flags_intel alloc_flags = 0;
-  cl_uint mem_id = 0;
+  cl_uint mem_id = acl_get_default_memory(device->def);
   while (properties != NULL && *properties != 0) {
     switch (*properties) {
     case CL_MEM_ALLOC_FLAGS_INTEL: {
@@ -616,7 +616,7 @@ CL_API_ENTRY cl_int CL_API_CALL clGetMemAllocInfoINTEL(
     if (usm_alloc) {
       RESULT_UINT(usm_alloc->mem->mem_id);
     } else {
-      RESULT_UINT(CL_MEM_TYPE_UNKNOWN_INTEL);
+      RESULT_UINT(0);
     }
   } break;
 

test/acl_usm_test.cpp

@@ -417,13 +417,22 @@ MT_TEST(acl_usm, buffer_location_usm) {
 
   // Check alloc information
   cl_uint read_mem_id = 4;
-  size_t ret_size;
+  size_t ret_size = 9;
   status = clGetMemAllocInfoINTEL(m_context, test_ptr,
                                   CL_MEM_ALLOC_BUFFER_LOCATION_INTEL,
                                   sizeof(cl_uint), &read_mem_id, &ret_size);
 
   CHECK_EQUAL(CL_SUCCESS, status);
   CHECK_EQUAL(0, read_mem_id);
+  CHECK_EQUAL(sizeof(cl_uint), ret_size);
+  // Check when given pointer is invalid
+  status = clGetMemAllocInfoINTEL(m_context, NULL,
+                                  CL_MEM_ALLOC_BUFFER_LOCATION_INTEL,
+                                  sizeof(cl_uint), &read_mem_id, &ret_size);
+
+  CHECK_EQUAL(CL_SUCCESS, status);
+  CHECK_EQUAL(0, read_mem_id);
+  CHECK_EQUAL(sizeof(cl_uint), ret_size);
 
   status = clMemFreeINTEL(m_context, test_ptr);
   CHECK_EQUAL(status, CL_SUCCESS);