This patch fixes
error: void value not ignored as it ought to be in the C/R code by ignoring the return value of functions which no longer return a value (only void). Signed-off-by: Adrian Reber <adrian.reber@hs-esslingen.de> This commit was SVN r29816.
Этот коммит содержится в:
родитель
d556b60b21
Коммит
ed9aba3896
@ -290,16 +290,10 @@ static int errmgr_base_tool_stop_cmdline_listener(void)
|
||||
OPAL_OUTPUT_VERBOSE((5, orte_errmgr_base_framework.framework_output,
|
||||
"errmgr:base:tool: Shutdown Command Line Channel"));
|
||||
|
||||
if (ORTE_SUCCESS != (ret = orte_rml.recv_cancel(ORTE_NAME_WILDCARD,
|
||||
ORTE_RML_TAG_MIGRATE))) {
|
||||
ORTE_ERROR_LOG(ret);
|
||||
exit_status = ret;
|
||||
goto cleanup;
|
||||
}
|
||||
orte_rml.recv_cancel(ORTE_NAME_WILDCARD, ORTE_RML_TAG_MIGRATE);
|
||||
|
||||
errmgr_cmdline_recv_issued = false;
|
||||
|
||||
cleanup:
|
||||
return exit_status;
|
||||
}
|
||||
|
||||
|
@ -63,7 +63,7 @@ BEGIN_C_DECLS
|
||||
/*
|
||||
* Set URI
|
||||
*/
|
||||
int orte_rml_ftrm_set_contact_info(const char* uri);
|
||||
void orte_rml_ftrm_set_contact_info(const char* uri);
|
||||
|
||||
/*
|
||||
* Ping
|
||||
@ -148,7 +148,7 @@ BEGIN_C_DECLS
|
||||
/*
|
||||
* Recv Cancel
|
||||
*/
|
||||
int orte_rml_ftrm_recv_cancel(orte_process_name_t* peer, orte_rml_tag_t tag);
|
||||
void orte_rml_ftrm_recv_cancel(orte_process_name_t* peer, orte_rml_tag_t tag);
|
||||
|
||||
/*
|
||||
* Register a callback on loss of connection
|
||||
@ -161,7 +161,7 @@ BEGIN_C_DECLS
|
||||
*/
|
||||
int orte_rml_ftrm_ft_event(int state);
|
||||
|
||||
int orte_rml_ftrm_purge(orte_process_name_t *peer);
|
||||
void orte_rml_ftrm_purge(orte_process_name_t *peer);
|
||||
|
||||
END_C_DECLS
|
||||
|
||||
|
@ -94,20 +94,14 @@ char * orte_rml_ftrm_get_contact_info(void)
|
||||
/*
|
||||
* Set CONTACT_INFO
|
||||
*/
|
||||
int orte_rml_ftrm_set_contact_info(const char* contact_info)
|
||||
void orte_rml_ftrm_set_contact_info(const char* contact_info)
|
||||
{
|
||||
int ret;
|
||||
|
||||
opal_output_verbose(20, rml_ftrm_output_handle,
|
||||
"orte_rml_ftrm: set_contact_info()");
|
||||
|
||||
if( NULL != orte_rml_ftrm_wrapped_module.set_contact_info ) {
|
||||
if( ORTE_SUCCESS != (ret = orte_rml_ftrm_wrapped_module.set_contact_info(contact_info) ) ) {
|
||||
return ret;
|
||||
}
|
||||
orte_rml_ftrm_wrapped_module.set_contact_info(contact_info);
|
||||
}
|
||||
|
||||
return ORTE_SUCCESS;
|
||||
}
|
||||
|
||||
|
||||
@ -330,20 +324,14 @@ int orte_rml_ftrm_recv_buffer_nb(orte_process_name_t* peer,
|
||||
/*
|
||||
* Recv Cancel
|
||||
*/
|
||||
int orte_rml_ftrm_recv_cancel(orte_process_name_t* peer, orte_rml_tag_t tag)
|
||||
void orte_rml_ftrm_recv_cancel(orte_process_name_t* peer, orte_rml_tag_t tag)
|
||||
{
|
||||
int ret;
|
||||
|
||||
opal_output_verbose(20, rml_ftrm_output_handle,
|
||||
"orte_rml_ftrm: recv_cancel()");
|
||||
|
||||
if( NULL != orte_rml_ftrm_wrapped_module.recv_cancel ) {
|
||||
if( ORTE_SUCCESS != (ret = orte_rml_ftrm_wrapped_module.recv_cancel(peer, tag) ) ) {
|
||||
return ret;
|
||||
}
|
||||
orte_rml_ftrm_wrapped_module.recv_cancel(peer, tag);
|
||||
}
|
||||
|
||||
return ORTE_SUCCESS;
|
||||
}
|
||||
|
||||
|
||||
@ -436,18 +424,12 @@ int orte_rml_ftrm_ft_event(int state)
|
||||
return ORTE_SUCCESS;
|
||||
}
|
||||
|
||||
int orte_rml_ftrm_purge(orte_process_name_t *peer)
|
||||
void orte_rml_ftrm_purge(orte_process_name_t *peer)
|
||||
{
|
||||
int ret;
|
||||
|
||||
opal_output_verbose(20, rml_ftrm_output_handle,
|
||||
"orte_rml_ftrm: purge()");
|
||||
|
||||
if( NULL != orte_rml_ftrm_wrapped_module.purge ) {
|
||||
if( ORTE_SUCCESS != (ret = orte_rml_ftrm_wrapped_module.purge(peer) ) ) {
|
||||
return ret;
|
||||
}
|
||||
orte_rml_ftrm_wrapped_module.purge(peer);
|
||||
}
|
||||
|
||||
return ORTE_SUCCESS;
|
||||
}
|
||||
|
@ -905,8 +905,6 @@ static int snapc_full_global_start_listener(void)
|
||||
|
||||
static int snapc_full_global_stop_listener(void)
|
||||
{
|
||||
int ret, exit_status = ORTE_SUCCESS;
|
||||
|
||||
if (!snapc_orted_recv_issued && ORTE_PROC_IS_HNP) {
|
||||
return ORTE_SUCCESS;
|
||||
}
|
||||
@ -914,17 +912,10 @@ static int snapc_full_global_stop_listener(void)
|
||||
OPAL_OUTPUT_VERBOSE((5, mca_snapc_full_component.super.output_handle,
|
||||
"Global) Shutdown Coordinator Channel"));
|
||||
|
||||
if (ORTE_SUCCESS != (ret = orte_rml.recv_cancel(ORTE_NAME_WILDCARD,
|
||||
ORTE_RML_TAG_SNAPC_FULL))) {
|
||||
ORTE_ERROR_LOG(ret);
|
||||
exit_status = ret;
|
||||
goto cleanup;
|
||||
}
|
||||
orte_rml.recv_cancel(ORTE_NAME_WILDCARD, ORTE_RML_TAG_SNAPC_FULL);
|
||||
|
||||
snapc_orted_recv_issued = false;
|
||||
|
||||
cleanup:
|
||||
return exit_status;
|
||||
return ORTE_SUCCESS;
|
||||
}
|
||||
|
||||
static int snapc_full_global_start_cmdline_listener(void)
|
||||
@ -959,8 +950,6 @@ static int snapc_full_global_start_cmdline_listener(void)
|
||||
|
||||
static int snapc_full_global_stop_cmdline_listener(void)
|
||||
{
|
||||
int ret, exit_status = ORTE_SUCCESS;
|
||||
|
||||
if (!snapc_cmdline_recv_issued && ORTE_PROC_IS_HNP) {
|
||||
return ORTE_SUCCESS;
|
||||
}
|
||||
@ -968,17 +957,10 @@ static int snapc_full_global_stop_cmdline_listener(void)
|
||||
OPAL_OUTPUT_VERBOSE((5, mca_snapc_full_component.super.output_handle,
|
||||
"Global) Shutdown Command Line Channel"));
|
||||
|
||||
if (ORTE_SUCCESS != (ret = orte_rml.recv_cancel(ORTE_NAME_WILDCARD,
|
||||
ORTE_RML_TAG_CKPT))) {
|
||||
ORTE_ERROR_LOG(ret);
|
||||
exit_status = ret;
|
||||
goto cleanup;
|
||||
}
|
||||
orte_rml.recv_cancel(ORTE_NAME_WILDCARD, ORTE_RML_TAG_CKPT);
|
||||
|
||||
snapc_cmdline_recv_issued = false;
|
||||
|
||||
cleanup:
|
||||
return exit_status;
|
||||
return ORTE_SUCCESS;
|
||||
}
|
||||
|
||||
/*****************
|
||||
|
@ -378,8 +378,6 @@ static int snapc_full_local_start_hnp_listener(void)
|
||||
|
||||
static int snapc_full_local_stop_hnp_listener(void)
|
||||
{
|
||||
int ret, exit_status = ORTE_SUCCESS;
|
||||
|
||||
/*
|
||||
* Global Coordinator: Does not register a Local listener
|
||||
*/
|
||||
@ -394,17 +392,10 @@ static int snapc_full_local_stop_hnp_listener(void)
|
||||
OPAL_OUTPUT_VERBOSE((5, mca_snapc_full_component.super.output_handle,
|
||||
"Local) Shutdown Coordinator Channel"));
|
||||
|
||||
if (ORTE_SUCCESS != (ret = orte_rml.recv_cancel(ORTE_NAME_WILDCARD,
|
||||
ORTE_RML_TAG_SNAPC_FULL))) {
|
||||
ORTE_ERROR_LOG(ret);
|
||||
exit_status = ret;
|
||||
goto cleanup;
|
||||
}
|
||||
orte_rml.recv_cancel(ORTE_NAME_WILDCARD, ORTE_RML_TAG_SNAPC_FULL);
|
||||
|
||||
snapc_local_hnp_recv_issued = false;
|
||||
|
||||
cleanup:
|
||||
return exit_status;
|
||||
return ORTE_SUCCESS;
|
||||
}
|
||||
|
||||
static int snapc_full_local_start_app_listener(void)
|
||||
@ -439,8 +430,6 @@ static int snapc_full_local_start_app_listener(void)
|
||||
|
||||
static int snapc_full_local_stop_app_listener(void)
|
||||
{
|
||||
int ret, exit_status = ORTE_SUCCESS;
|
||||
|
||||
if (!snapc_local_app_recv_issued ) {
|
||||
return ORTE_SUCCESS;
|
||||
}
|
||||
@ -448,17 +437,10 @@ static int snapc_full_local_stop_app_listener(void)
|
||||
OPAL_OUTPUT_VERBOSE((5, mca_snapc_full_component.super.output_handle,
|
||||
"Local) Shutdown Application State Channel"));
|
||||
|
||||
if (ORTE_SUCCESS != (ret = orte_rml.recv_cancel(ORTE_NAME_WILDCARD,
|
||||
ORTE_RML_TAG_SNAPC))) {
|
||||
ORTE_ERROR_LOG(ret);
|
||||
exit_status = ret;
|
||||
goto cleanup;
|
||||
}
|
||||
orte_rml.recv_cancel(ORTE_NAME_WILDCARD, ORTE_RML_TAG_SNAPC);
|
||||
|
||||
snapc_local_app_recv_issued = false;
|
||||
|
||||
cleanup:
|
||||
return exit_status;
|
||||
return ORTE_SUCCESS;
|
||||
}
|
||||
|
||||
/******************
|
||||
|
@ -824,19 +824,10 @@ static int sstore_central_global_start_listener(void)
|
||||
|
||||
static int sstore_central_global_stop_listener(void)
|
||||
{
|
||||
int ret, exit_status = ORTE_SUCCESS;
|
||||
|
||||
if (ORTE_SUCCESS != (ret = orte_rml.recv_cancel(ORTE_NAME_WILDCARD,
|
||||
ORTE_RML_TAG_SSTORE_INTERNAL))) {
|
||||
ORTE_ERROR_LOG(ret);
|
||||
exit_status = ret;
|
||||
goto cleanup;
|
||||
}
|
||||
orte_rml.recv_cancel(ORTE_NAME_WILDCARD, ORTE_RML_TAG_SSTORE_INTERNAL);
|
||||
|
||||
is_global_listener_active = false;
|
||||
|
||||
cleanup:
|
||||
return exit_status;
|
||||
return ORTE_SUCCESS;
|
||||
}
|
||||
|
||||
static void sstore_central_global_recv(int status,
|
||||
|
@ -643,19 +643,10 @@ static int sstore_central_local_start_listener(void)
|
||||
|
||||
static int sstore_central_local_stop_listener(void)
|
||||
{
|
||||
int ret, exit_status = ORTE_SUCCESS;
|
||||
|
||||
if (ORTE_SUCCESS != (ret = orte_rml.recv_cancel(ORTE_NAME_WILDCARD,
|
||||
ORTE_RML_TAG_SSTORE_INTERNAL))) {
|
||||
ORTE_ERROR_LOG(ret);
|
||||
exit_status = ret;
|
||||
goto cleanup;
|
||||
}
|
||||
orte_rml.recv_cancel(ORTE_NAME_WILDCARD, ORTE_RML_TAG_SSTORE_INTERNAL);
|
||||
|
||||
is_global_listener_active = false;
|
||||
|
||||
cleanup:
|
||||
return exit_status;
|
||||
return ORTE_SUCCESS;
|
||||
}
|
||||
|
||||
static int process_global_pull(orte_process_name_t* peer, opal_buffer_t* buffer, orte_sstore_central_local_snapshot_info_t *handle_info)
|
||||
|
@ -1016,19 +1016,10 @@ static int sstore_stage_global_start_listener(void)
|
||||
|
||||
static int sstore_stage_global_stop_listener(void)
|
||||
{
|
||||
int ret, exit_status = ORTE_SUCCESS;
|
||||
|
||||
if (ORTE_SUCCESS != (ret = orte_rml.recv_cancel(ORTE_NAME_WILDCARD,
|
||||
ORTE_RML_TAG_SSTORE_INTERNAL))) {
|
||||
ORTE_ERROR_LOG(ret);
|
||||
exit_status = ret;
|
||||
goto cleanup;
|
||||
}
|
||||
orte_rml.recv_cancel(ORTE_NAME_WILDCARD, ORTE_RML_TAG_SSTORE_INTERNAL);
|
||||
|
||||
is_global_listener_active = false;
|
||||
|
||||
cleanup:
|
||||
return exit_status;
|
||||
return ORTE_SUCCESS;
|
||||
}
|
||||
|
||||
static void sstore_stage_global_recv(int status,
|
||||
|
@ -1067,19 +1067,9 @@ static int sstore_stage_local_start_listener(void)
|
||||
|
||||
static int sstore_stage_local_stop_listener(void)
|
||||
{
|
||||
int ret, exit_status = ORTE_SUCCESS;
|
||||
|
||||
if (ORTE_SUCCESS != (ret = orte_rml.recv_cancel(ORTE_NAME_WILDCARD,
|
||||
ORTE_RML_TAG_SSTORE_INTERNAL))) {
|
||||
ORTE_ERROR_LOG(ret);
|
||||
exit_status = ret;
|
||||
goto cleanup;
|
||||
}
|
||||
|
||||
orte_rml.recv_cancel(ORTE_NAME_WILDCARD, ORTE_RML_TAG_SSTORE_INTERNAL);
|
||||
is_global_listener_active = false;
|
||||
|
||||
cleanup:
|
||||
return exit_status;
|
||||
return ORTE_SUCCESS;
|
||||
}
|
||||
|
||||
static void sstore_stage_local_recv(int status,
|
||||
|
@ -671,22 +671,14 @@ static int start_listener(void)
|
||||
|
||||
static int stop_listener(void)
|
||||
{
|
||||
int ret, exit_status = ORTE_SUCCESS;
|
||||
|
||||
if( !listener_started ) {
|
||||
exit_status = ORTE_ERROR;
|
||||
goto cleanup;
|
||||
return ORTE_ERROR;
|
||||
}
|
||||
|
||||
if (ORTE_SUCCESS != (ret = orte_rml.recv_cancel(ORTE_NAME_WILDCARD,
|
||||
ORTE_RML_TAG_CKPT))) {
|
||||
exit_status = ret;
|
||||
goto cleanup;
|
||||
}
|
||||
orte_rml.recv_cancel(ORTE_NAME_WILDCARD, ORTE_RML_TAG_CKPT);
|
||||
|
||||
listener_started = false;
|
||||
cleanup:
|
||||
return exit_status;
|
||||
return ORTE_SUCCESS;
|
||||
}
|
||||
|
||||
static void hnp_receiver(int status,
|
||||
|
@ -532,22 +532,14 @@ static int start_listener(void)
|
||||
|
||||
static int stop_listener(void)
|
||||
{
|
||||
int ret, exit_status = ORTE_SUCCESS;
|
||||
|
||||
if( !listener_started ) {
|
||||
exit_status = ORTE_ERROR;
|
||||
goto cleanup;
|
||||
return ORTE_ERROR;
|
||||
}
|
||||
|
||||
if (ORTE_SUCCESS != (ret = orte_rml.recv_cancel(ORTE_NAME_WILDCARD,
|
||||
ORTE_RML_TAG_MIGRATE))) {
|
||||
exit_status = ret;
|
||||
goto cleanup;
|
||||
}
|
||||
orte_rml.recv_cancel(ORTE_NAME_WILDCARD, ORTE_RML_TAG_MIGRATE);
|
||||
|
||||
listener_started = false;
|
||||
cleanup:
|
||||
return exit_status;
|
||||
return ORTE_SUCCESS;
|
||||
}
|
||||
|
||||
static void hnp_receiver(int status,
|
||||
|
Загрузка…
x
Ссылка в новой задаче
Block a user