1
1

Cleanup a set of typos on the orte_get_attribute call

This commit was SVN r31942.
Этот коммит содержится в:
Ralph Castain 2014-06-03 20:36:38 +00:00
родитель 3ed4aaea99
Коммит f1978fba7c
15 изменённых файлов: 26 добавлений и 26 удалений

Просмотреть файл

@ -179,7 +179,7 @@ void orte_plm_base_recv(int status, orte_process_name_t* sender,
app = (orte_app_context_t*)opal_pointer_array_get_item(parent->apps, 0);
child_app = (orte_app_context_t*)opal_pointer_array_get_item(jdata->apps, 0);
prefix_dir = NULL;
if (orte_get_attribute(&app->attributes, ORTE_APP_PREFIX_DIR, (void**)prefix_dir, OPAL_STRING) &&
if (orte_get_attribute(&app->attributes, ORTE_APP_PREFIX_DIR, (void**)&prefix_dir, OPAL_STRING) &&
!orte_get_attribute(&child_app->attributes, ORTE_APP_PREFIX_DIR, NULL, OPAL_STRING)) {
orte_set_attribute(&child_app->attributes, ORTE_APP_PREFIX_DIR, ORTE_ATTR_GLOBAL, prefix_dir, OPAL_STRING);
}

Просмотреть файл

@ -1190,7 +1190,7 @@ static void launch_daemons(int fd, short args, void *cbdata)
/* setup node name */
free(argv[node_name_index1]);
username = NULL;
if (orte_get_attribute(&node->attributes, ORTE_NODE_USERNAME, (void**)username, OPAL_STRING)) {
if (orte_get_attribute(&node->attributes, ORTE_NODE_USERNAME, (void**)&username, OPAL_STRING)) {
(void)asprintf (&argv[node_name_index1], "%s@%s",
username, node->name);
free(username);

Просмотреть файл

@ -82,7 +82,7 @@ static void reset_usage(orte_node_t *node, orte_jobid_t jobid)
continue;
}
bound = NULL;
if (!orte_get_attribute(&proc->attributes, ORTE_PROC_HWLOC_BOUND, (void**)bound, OPAL_PTR) ||
if (!orte_get_attribute(&proc->attributes, ORTE_PROC_HWLOC_BOUND, (void**)&bound, OPAL_PTR) ||
NULL == bound) {
/* this proc isn't bound - ignore it */
opal_output_verbose(10, orte_rmaps_base_framework.framework_output,

Просмотреть файл

@ -379,7 +379,7 @@ void orte_rmaps_base_map_job(int fd, short args, void *cbdata)
char *cpu_bitmap;
loc = NULL;
if (orte_get_attribute(&proc->attributes, ORTE_PROC_HWLOC_LOCALE, (void**)loc, OPAL_PTR)) {
if (orte_get_attribute(&proc->attributes, ORTE_PROC_HWLOC_LOCALE, (void**)&loc, OPAL_PTR)) {
hwloc_bitmap_list_snprintf(locale, 64, loc->cpuset);
}
cpu_bitmap = NULL;

Просмотреть файл

@ -133,7 +133,7 @@ static int rank_span(orte_job_t *jdata,
}
/* protect against bozo case */
locale = NULL;
if (!orte_get_attribute(&proc->attributes, ORTE_PROC_HWLOC_LOCALE, (void**)locale, OPAL_PTR)) {
if (!orte_get_attribute(&proc->attributes, ORTE_PROC_HWLOC_LOCALE, (void**)&locale, OPAL_PTR)) {
ORTE_ERROR_LOG(ORTE_ERROR);
return ORTE_ERROR;
}
@ -246,7 +246,7 @@ static int rank_fill(orte_job_t *jdata,
}
/* protect against bozo case */
locale = NULL;
if (!orte_get_attribute(&proc->attributes, ORTE_PROC_HWLOC_LOCALE, (void**)locale, OPAL_PTR)) {
if (!orte_get_attribute(&proc->attributes, ORTE_PROC_HWLOC_LOCALE, (void**)&locale, OPAL_PTR)) {
ORTE_ERROR_LOG(ORTE_ERROR);
return ORTE_ERROR;
}

Просмотреть файл

@ -826,7 +826,7 @@ static int orte_rmaps_lama_map_core(orte_job_t *jdata)
continue;
}
cpu_bitmap = NULL;
orte_get_attribute(&proc->attributes, ORTE_PROC_CPU_BITMAP, (void**)cpu_bitmap, OPAL_STRING);
orte_get_attribute(&proc->attributes, ORTE_PROC_CPU_BITMAP, (void**)&cpu_bitmap, OPAL_STRING);
opal_output_verbose(5, orte_rmaps_base_framework.framework_output,
"mca:rmaps:lama: Ordering: Proc. %2d on Node %10s - Slot %s",
proc->name.vpid, proc->node->name, cpu_bitmap);

Просмотреть файл

@ -511,7 +511,7 @@ static void prune(orte_jobid_t jobid,
continue;
}
locale = NULL;
if (orte_get_attribute(&proc->attributes, ORTE_PROC_HWLOC_LOCALE, (void**)locale, OPAL_PTR)) {
if (orte_get_attribute(&proc->attributes, ORTE_PROC_HWLOC_LOCALE, (void**)&locale, OPAL_PTR)) {
ORTE_ERROR_LOG(ORTE_ERR_NOT_FOUND);
return;
}
@ -565,7 +565,7 @@ static void prune(orte_jobid_t jobid,
continue;
}
locale = NULL;
if (orte_get_attribute(&proc->attributes, ORTE_PROC_HWLOC_LOCALE, (void**)locale, OPAL_PTR)) {
if (orte_get_attribute(&proc->attributes, ORTE_PROC_HWLOC_LOCALE, (void**)&locale, OPAL_PTR)) {
ORTE_ERROR_LOG(ORTE_ERR_NOT_FOUND);
return;
}

Просмотреть файл

@ -582,7 +582,7 @@ void orte_state_base_check_all_complete(int fd, short args, void *cbdata)
}
i32ptr = &i32;
if (orte_get_attribute(&jdata->attributes, ORTE_JOB_NUM_NONZERO_EXIT, (void**)i32ptr, OPAL_INT32) && !orte_abort_non_zero_exit) {
if (orte_get_attribute(&jdata->attributes, ORTE_JOB_NUM_NONZERO_EXIT, (void**)&i32ptr, OPAL_INT32) && !orte_abort_non_zero_exit) {
if (!orte_report_child_jobs_separately || 1 == ORTE_LOCAL_JOBID(jdata->jobid)) {
/* update the exit code */
ORTE_UPDATE_EXIT_STATUS(lowest);

Просмотреть файл

@ -309,7 +309,7 @@ int orte_dt_print_node(char **output, char *prefix, orte_node_t *src, opal_data_
(int)src->slots, (int)src->slots_max);
/* does this node have any aliases? */
tmp3 = NULL;
if (orte_get_attribute(&src->attributes, ORTE_NODE_ALIAS, (void**)tmp3, OPAL_STRING)) {
if (orte_get_attribute(&src->attributes, ORTE_NODE_ALIAS, (void**)&tmp3, OPAL_STRING)) {
alias = opal_argv_split(tmp3, ',');
for (i=0; NULL != alias[i]; i++) {
asprintf(&tmp2, "%s%s\t<noderesolve resolved=\"%s\"/>\n", tmp, pfx2, alias[i]);
@ -335,7 +335,7 @@ int orte_dt_print_node(char **output, char *prefix, orte_node_t *src, opal_data_
(long)src->slots, (long)src->slots_max);
/* does this node have any aliases? */
tmp3 = NULL;
if (orte_get_attribute(&src->attributes, ORTE_NODE_ALIAS, (void**)tmp3, OPAL_STRING)) {
if (orte_get_attribute(&src->attributes, ORTE_NODE_ALIAS, (void**)&tmp3, OPAL_STRING)) {
alias = opal_argv_split(tmp3, ',');
for (i=0; NULL != alias[i]; i++) {
asprintf(&tmp2, "%s%s\tresolved from %s\n", tmp, pfx2, alias[i]);
@ -356,7 +356,7 @@ int orte_dt_print_node(char **output, char *prefix, orte_node_t *src, opal_data_
(long)src->slots, (long)src->slots_max, (long)src->num_procs);
/* does this node have any aliases? */
tmp3 = NULL;
if (orte_get_attribute(&src->attributes, ORTE_NODE_ALIAS, (void**)tmp3, OPAL_STRING)) {
if (orte_get_attribute(&src->attributes, ORTE_NODE_ALIAS, (void**)&tmp3, OPAL_STRING)) {
alias = opal_argv_split(tmp3, ',');
for (i=0; NULL != alias[i]; i++) {
asprintf(&tmp2, "%s%s\tresolved from %s\n", tmp, pfx2, alias[i]);
@ -375,7 +375,7 @@ int orte_dt_print_node(char **output, char *prefix, orte_node_t *src, opal_data_
pfx2, (NULL == src->name) ? "UNKNOWN" : src->name, src->state);
/* does this node have any aliases? */
tmp3 = NULL;
if (orte_get_attribute(&src->attributes, ORTE_NODE_ALIAS, (void**)tmp3, OPAL_STRING)) {
if (orte_get_attribute(&src->attributes, ORTE_NODE_ALIAS, (void**)&tmp3, OPAL_STRING)) {
alias = opal_argv_split(tmp3, ',');
for (i=0; NULL != alias[i]; i++) {
asprintf(&tmp2, "%s%s\tresolved from %s\n", tmp, pfx2, alias[i]);
@ -411,7 +411,7 @@ int orte_dt_print_node(char **output, char *prefix, orte_node_t *src, opal_data_
tmp = tmp2;
tmp3 = NULL;
if (orte_get_attribute(&src->attributes, ORTE_NODE_USERNAME, (void**)tmp3, OPAL_STRING)) {
if (orte_get_attribute(&src->attributes, ORTE_NODE_USERNAME, (void**)&tmp3, OPAL_STRING)) {
asprintf(&tmp2, "%s\n%s\tUsername on node: %s", tmp, pfx2, tmp3);
free(tmp3);
free(tmp);
@ -503,7 +503,7 @@ int orte_dt_print_proc(char **output, char *prefix, orte_proc_t *src, opal_data_
char tmp1[1024], tmp2[1024];
char *str=NULL, *cpu_bitmap=NULL;
;
if (orte_get_attribute(&src->attributes, ORTE_PROC_CPU_BITMAP, (void**)cpu_bitmap, OPAL_STRING)) {
if (orte_get_attribute(&src->attributes, ORTE_PROC_CPU_BITMAP, (void**)&cpu_bitmap, OPAL_STRING)) {
mycpus = hwloc_bitmap_alloc();
hwloc_bitmap_list_sscanf(mycpus, cpu_bitmap);
if (OPAL_ERR_NOT_BOUND == opal_hwloc_base_cset2str(tmp1, sizeof(tmp1), src->node->topology, mycpus)) {
@ -634,7 +634,7 @@ int orte_dt_print_app_context(char **output, char *prefix, orte_app_context_t *s
}
tmp3 = NULL;
orte_get_attribute(&src->attributes, ORTE_APP_PREFIX_DIR, (void**)tmp3, OPAL_STRING);
orte_get_attribute(&src->attributes, ORTE_APP_PREFIX_DIR, (void**)&tmp3, OPAL_STRING);
asprintf(&tmp2, "%s\n%s\tWorking dir: %s\n%s\tPrefix: %s\n%s\tUsed on node: %s", tmp,
pfx2, (NULL == src->cwd) ? "NULL" : src->cwd,
pfx2, (NULL == tmp3) ? "NULL" : tmp3,

Просмотреть файл

@ -726,14 +726,14 @@ static void orte_job_destruct(orte_job_t* job)
/* release any pointers in the attributes */
evtimer = NULL;
if (orte_get_attribute(&job->attributes, ORTE_JOB_FAILURE_TIMER_EVENT,
(void**)evtimer, OPAL_PTR)) {
(void**)&evtimer, OPAL_PTR)) {
orte_remove_attribute(&job->attributes, ORTE_JOB_FAILURE_TIMER_EVENT);
/* the timer is a pointer to orte_timer_t */
OBJ_RELEASE(evtimer);
}
proc = NULL;
if (orte_get_attribute(&job->attributes, ORTE_JOB_ABORTED_PROC,
(void**)proc, OPAL_PTR)) {
(void**)&proc, OPAL_PTR)) {
orte_remove_attribute(&job->attributes, ORTE_JOB_ABORTED_PROC);
/* points to an orte_proc_t */
OBJ_RELEASE(proc);

Просмотреть файл

@ -170,7 +170,7 @@ static void dump_aborted_procs(void)
ORTE_JOB_STATE_ABORT_ORDERED != job->state) {
/* this is a guilty party */
proc = NULL;
orte_get_attribute(&job->attributes, ORTE_JOB_ABORTED_PROC, (void**)proc, OPAL_PTR);
orte_get_attribute(&job->attributes, ORTE_JOB_ABORTED_PROC, (void**)&proc, OPAL_PTR);
/* cycle through and count the number that were killed or aborted */
for (i=0; i < job->procs->size; i++) {
if (NULL == (pptr = (orte_proc_t*)opal_pointer_array_get_item(job->procs, i))) {

Просмотреть файл

@ -961,7 +961,7 @@ static int parseable_print(orte_ps_mpirun_info_t *hnpinfo)
appname = opal_basename(app->app);
}
nodename = NULL;
orte_get_attribute(&proc->attributes, ORTE_PROC_NODENAME, (void**)nodename, OPAL_STRING);
orte_get_attribute(&proc->attributes, ORTE_PROC_NODENAME, (void**)&nodename, OPAL_STRING);
printf("process:%s:rank:%s:pid:%lu:node:%s:state:%s\n",
appname, ORTE_VPID_PRINT(proc->name.vpid),
(unsigned long)proc->pid,

Просмотреть файл

@ -2893,7 +2893,7 @@ void orte_debugger_init_after_spawn(int fd, short event, void *cbdata)
if (orte_retain_aliases) {
aliases = NULL;
aptr = NULL;
if (orte_get_attribute(&proc->node->attributes, ORTE_NODE_ALIAS, (void**)aptr, OPAL_STRING)) {
if (orte_get_attribute(&proc->node->attributes, ORTE_NODE_ALIAS, (void**)&aptr, OPAL_STRING)) {
aliases = opal_argv_split(aptr, ',');
free(aptr);
if (orte_use_hostname_alias <= opal_argv_count(aliases)) {

Просмотреть файл

@ -140,7 +140,7 @@ int orte_util_add_dash_host_nodes(opal_list_t *nodes,
/* add to list of aliases for this node - only add if unique */
aptr = NULL;
aliases = NULL;
orte_get_attribute(&node->attributes, ORTE_NODE_ALIAS, (void**)aptr, OPAL_STRING);
orte_get_attribute(&node->attributes, ORTE_NODE_ALIAS, (void**)&aptr, OPAL_STRING);
if (NULL != aptr) {
aliases = opal_argv_split(aptr, ',');
free(aptr);

Просмотреть файл

@ -478,7 +478,7 @@ static int decode_app_nodemap(opal_byte_object_t *bo)
*/
hostid = 0;
hostidptr = &hostid;
if (orte_get_attribute(&node->attributes, ORTE_NODE_HOSTID, (void**)hostidptr, ORTE_VPID)) {
if (orte_get_attribute(&node->attributes, ORTE_NODE_HOSTID, (void**)&hostidptr, ORTE_VPID)) {
orte_coprocessors_detected = true; // obviously, coprocessors were detected
OBJ_CONSTRUCT(&kv, opal_value_t);
kv.key = strdup(ORTE_DB_HOSTID);
@ -786,7 +786,7 @@ static int decode_app_pidmap(opal_byte_object_t *bo)
OBJ_DESTRUCT(&kv);
}
cpu_bitmap = NULL;
if (orte_get_attribute(&pptr->attributes, ORTE_PROC_CPU_BITMAP, (void**)cpu_bitmap, OPAL_STRING) &&
if (orte_get_attribute(&pptr->attributes, ORTE_PROC_CPU_BITMAP, (void**)&cpu_bitmap, OPAL_STRING) &&
NULL != cpu_bitmap) {
orte_process_info.cpuset = cpu_bitmap;
}
@ -833,7 +833,7 @@ static int decode_app_pidmap(opal_byte_object_t *bo)
}
OBJ_DESTRUCT(&kv);
cpu_bitmap = NULL;
if (orte_get_attribute(&pptr->attributes, ORTE_PROC_CPU_BITMAP, (void**)cpu_bitmap, OPAL_STRING) &&
if (orte_get_attribute(&pptr->attributes, ORTE_PROC_CPU_BITMAP, (void**)&cpu_bitmap, OPAL_STRING) &&
NULL != cpu_bitmap) {
OBJ_CONSTRUCT(&kv, opal_value_t);
kv.key = strdup(OPAL_DSTORE_CPUSET);