Catch a potential bug spanning several ESS modules. The node_rank and local_rank types were changed to uint16_t, however the modules returned UINT8_MAX as an "invalid" value. To clean this up, define an INVALID value for these types, and change the various modules so they return this value to indicate an invalid response.

This commit was SVN r20303.

orte/include/orte/types.h

@@ -47,8 +47,10 @@ typedef uint16_t orte_local_rank_t;
 typedef uint16_t orte_node_rank_t;
 #define ORTE_LOCAL_RANK         OPAL_UINT16
 #define ORTE_NODE_RANK          OPAL_UINT16
-#define ORTE_LOCAL_RANK_MAX     UINT16_MAX
-#define ORTE_NODE_RANK_MAX      UINT16_MAX
+#define ORTE_LOCAL_RANK_MAX     UINT16_MAX-1
+#define ORTE_NODE_RANK_MAX      UINT16_MAX-1
+#define ORTE_LOCAL_RANK_INVALID UINT16_MAX
+#define ORTE_NODE_RANK_INVALID  UINT16_MAX
 
 
 /*

orte/mca/ess/alps/ess_alps_module.c

@@ -264,7 +264,7 @@ static orte_local_rank_t proc_get_local_rank(orte_process_name_t *proc)
     
     if (NULL == (pmap = orte_util_lookup_pmap(proc))) {
         ORTE_ERROR_LOG(ORTE_ERR_NOT_FOUND);
-        return UINT8_MAX;
+        return ORTE_LOCAL_RANK_INVALID;
     }    
 
     OPAL_OUTPUT_VERBOSE((2, orte_ess_base_output,
@@ -282,7 +282,7 @@ static orte_node_rank_t proc_get_node_rank(orte_process_name_t *proc)
     
     if (NULL == (pmap = orte_util_lookup_pmap(proc))) {
         ORTE_ERROR_LOG(ORTE_ERR_NOT_FOUND);
-        return UINT8_MAX;
+        return ORTE_NODE_RANK_INVALID;
     }    
     
     OPAL_OUTPUT_VERBOSE((2, orte_ess_base_output,

orte/mca/ess/env/ess_env_module.c

@@ -314,7 +314,7 @@ static orte_local_rank_t proc_get_local_rank(orte_process_name_t *proc)
     
     if (NULL == (pmap = orte_util_lookup_pmap(proc))) {
         ORTE_ERROR_LOG(ORTE_ERR_NOT_FOUND);
-        return UINT8_MAX;
+        return ORTE_LOCAL_RANK_INVALID;
     }    
     
     OPAL_OUTPUT_VERBOSE((2, orte_ess_base_output,
@@ -332,7 +332,7 @@ static orte_node_rank_t proc_get_node_rank(orte_process_name_t *proc)
     
     if (NULL == (pmap = orte_util_lookup_pmap(proc))) {
         ORTE_ERROR_LOG(ORTE_ERR_NOT_FOUND);
-        return UINT8_MAX;
+        return ORTE_NODE_RANK_INVALID;
     }    
     
     OPAL_OUTPUT_VERBOSE((2, orte_ess_base_output,

orte/mca/ess/hnp/ess_hnp_module.c

@@ -693,7 +693,7 @@ static orte_local_rank_t proc_get_local_rank(orte_process_name_t *proc)
     
     if (NULL == (pdata = find_proc(proc))) {
         ORTE_ERROR_LOG(ORTE_ERR_NOT_FOUND);
-        return UINT8_MAX;
+        return ORTE_LOCAL_RANK_INVALID;
     }
     
     OPAL_OUTPUT_VERBOSE((2, orte_ess_base_output,
@@ -711,7 +711,7 @@ static orte_node_rank_t proc_get_node_rank(orte_process_name_t *proc)
     
     if (NULL == (pdata = find_proc(proc))) {
         ORTE_ERROR_LOG(ORTE_ERR_NOT_FOUND);
-        return UINT8_MAX;
+        return ORTE_NODE_RANK_INVALID;
     }
     
     OPAL_OUTPUT_VERBOSE((2, orte_ess_base_output,

orte/mca/ess/lsf/ess_lsf_module.c

@@ -276,7 +276,7 @@ static orte_local_rank_t proc_get_local_rank(orte_process_name_t *proc)
     
     if (NULL == (pmap = orte_util_lookup_pmap(proc))) {
         ORTE_ERROR_LOG(ORTE_ERR_NOT_FOUND);
-        return UINT8_MAX;
+        return ORTE_LOCAL_RANK_INVALID;
     }    
 
     OPAL_OUTPUT_VERBOSE((2, orte_ess_base_output,
@@ -294,7 +294,7 @@ static orte_node_rank_t proc_get_node_rank(orte_process_name_t *proc)
     
     if (NULL == (pmap = orte_util_lookup_pmap(proc))) {
         ORTE_ERROR_LOG(ORTE_ERR_NOT_FOUND);
-        return UINT8_MAX;
+        return ORTE_NODE_RANK_INVALID;
     }    
     
     OPAL_OUTPUT_VERBOSE((2, orte_ess_base_output,

orte/mca/ess/singleton/ess_singleton_module.c

@@ -507,7 +507,7 @@ static orte_local_rank_t proc_get_local_rank(orte_process_name_t *proc)
     
     if (NULL == (pmap = orte_util_lookup_pmap(proc))) {
         ORTE_ERROR_LOG(ORTE_ERR_NOT_FOUND);
-        return UINT8_MAX;
+        return ORTE_LOCAL_RANK_INVALID;
     }    
     
     OPAL_OUTPUT_VERBOSE((2, orte_ess_base_output,
@@ -525,7 +525,7 @@ static orte_node_rank_t proc_get_node_rank(orte_process_name_t *proc)
     
     if (NULL == (pmap = orte_util_lookup_pmap(proc))) {
         ORTE_ERROR_LOG(ORTE_ERR_NOT_FOUND);
-        return UINT8_MAX;
+        return ORTE_NODE_RANK_INVALID;
     }    
     
     OPAL_OUTPUT_VERBOSE((2, orte_ess_base_output,

orte/mca/ess/slurm/ess_slurm_module.c

@@ -278,7 +278,7 @@ static orte_local_rank_t proc_get_local_rank(orte_process_name_t *proc)
     
     if (NULL == (pmap = orte_util_lookup_pmap(proc))) {
         ORTE_ERROR_LOG(ORTE_ERR_NOT_FOUND);
-        return UINT8_MAX;
+        return ORTE_LOCAL_RANK_INVALID;
     }    
     
     OPAL_OUTPUT_VERBOSE((2, orte_ess_base_output,
@@ -296,7 +296,7 @@ static orte_node_rank_t proc_get_node_rank(orte_process_name_t *proc)
     
     if (NULL == (pmap = orte_util_lookup_pmap(proc))) {
         ORTE_ERROR_LOG(ORTE_ERR_NOT_FOUND);
-        return UINT8_MAX;
+        return ORTE_NODE_RANK_INVALID;
     }    
     
     OPAL_OUTPUT_VERBOSE((2, orte_ess_base_output,