Skip to content

Commit c85c2a3

Browse files
committed
[OS memory provider] Split initialize
1 parent aa4ff00 commit c85c2a3

File tree

1 file changed

+15
-16
lines changed

1 file changed

+15
-16
lines changed

src/provider/provider_os_memory.c

Lines changed: 15 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -602,20 +602,6 @@ static umf_result_t os_initialize(const void *params, void **provider) {
602602
}
603603
}
604604

605-
os_provider->nodeset_str_buf = umf_ba_global_alloc(NODESET_STR_BUF_LEN);
606-
if (!os_provider->nodeset_str_buf) {
607-
LOG_INFO("allocating memory for printing NUMA nodes failed");
608-
} else {
609-
LOG_INFO("OS provider initialized with NUMA nodes:");
610-
for (unsigned i = 0; i < os_provider->nodeset_len; i++) {
611-
if (hwloc_bitmap_list_snprintf(os_provider->nodeset_str_buf,
612-
NODESET_STR_BUF_LEN,
613-
os_provider->nodeset[i])) {
614-
LOG_INFO("%s", os_provider->nodeset_str_buf);
615-
}
616-
}
617-
}
618-
619605
*provider = os_provider;
620606

621607
return UMF_RESULT_SUCCESS;
@@ -1392,8 +1378,21 @@ static umf_result_t os_ctl(void *hProvider,
13921378
}
13931379

13941380
static umf_result_t os_post_initialize(void *provider) {
1395-
(void)provider;
1396-
// For initial version, just return success
1381+
os_memory_provider_t *os_provider = (os_memory_provider_t *)provider;
1382+
1383+
os_provider->nodeset_str_buf = umf_ba_global_alloc(NODESET_STR_BUF_LEN);
1384+
if (!os_provider->nodeset_str_buf) {
1385+
LOG_INFO("allocating memory for printing NUMA nodes failed");
1386+
} else {
1387+
LOG_INFO("OS provider initialized with NUMA nodes:");
1388+
for (unsigned i = 0; i < os_provider->nodeset_len; i++) {
1389+
if (hwloc_bitmap_list_snprintf(os_provider->nodeset_str_buf,
1390+
NODESET_STR_BUF_LEN,
1391+
os_provider->nodeset[i])) {
1392+
LOG_INFO("%s", os_provider->nodeset_str_buf);
1393+
}
1394+
}
1395+
}
13971396
return UMF_RESULT_SUCCESS;
13981397
}
13991398

0 commit comments

Comments
 (0)