Add logs to query_attribute_value()

Signed-off-by: Lukasz Dorau <[email protected]>

Author: ldorau

src/memory_targets/memory_target_numa.c

@@ -201,27 +201,32 @@ static umf_result_t query_attribute_value(void *srcMemoryTarget,
                                           memattr_type_t type) {
     hwloc_topology_t topology = umfGetTopology();
     if (!topology) {
+        LOG_PERR("Retrieving cached topology failed");
         return UMF_RESULT_ERROR_NOT_SUPPORTED;
     }
 
     hwloc_obj_t srcNumaNode = hwloc_get_obj_by_type(
         topology, HWLOC_OBJ_NUMANODE,
         ((struct numa_memory_target_t *)srcMemoryTarget)->physical_id);
     if (!srcNumaNode) {
+        LOG_PERR("Getting HWLOC object by type failed");
         return UMF_RESULT_ERROR_INVALID_ARGUMENT;
     }
 
     hwloc_obj_t dstNumaNode = hwloc_get_obj_by_type(
         topology, HWLOC_OBJ_NUMANODE,
         ((struct numa_memory_target_t *)dstMemoryTarget)->physical_id);
     if (!dstNumaNode) {
+        LOG_PERR("Getting HWLOC object by type failed");
         return UMF_RESULT_ERROR_INVALID_ARGUMENT;
     }
 
     // Given NUMA nodes aren't local, HWLOC returns an error in such case.
     if (!hwloc_bitmap_intersects(srcNumaNode->cpuset, dstNumaNode->cpuset)) {
         // Since we want to skip such query, we return the worst possible
         // value for given memory attribute.
+        LOG_PDEBUG("Testing whether two bitmaps intersect failed, using the "
+                   "worst value");
         *value = memattr_get_worst_value(type);
         return UMF_RESULT_SUCCESS;
     }
@@ -246,6 +251,8 @@ static umf_result_t query_attribute_value(void *srcMemoryTarget,
     int ret = hwloc_memattr_get_value(topology, hwlocMemAttrType, dstNumaNode,
                                       &initiator, 0, &memAttrValue);
     if (ret) {
+        LOG_PERR("Getting an attribute value for a specific target NUMA node "
+                 "failed");
         return (errno == EINVAL) ? UMF_RESULT_ERROR_NOT_SUPPORTED
                                  : UMF_RESULT_ERROR_UNKNOWN;
     }