Skip to content

Commit 1c41a07

Browse files
authored
Merge pull request #9768 from jsquyres/pr/v5.0.x/minor-fixes
v5.0.x: common_ofi.c: fix uninit variable warning
2 parents c7e390f + 7e81531 commit 1c41a07

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

opal/mca/common/ofi/common_ofi.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -602,7 +602,6 @@ struct fi_info *opal_mca_common_ofi_select_provider(struct fi_info *provider_lis
602602
struct fi_pci_attr pci;
603603
#endif
604604
int ret;
605-
uint32_t package_rank;
606605
unsigned int num_provider = 0, provider_limit = 0;
607606
bool provider_found = false, cpusets_match = false;
608607

@@ -660,9 +659,9 @@ struct fi_info *opal_mca_common_ofi_select_provider(struct fi_info *provider_lis
660659
}
661660

662661
/* Select provider from local rank % number of providers */
662+
uint32_t package_rank = get_package_rank(process_info);
663663
if (num_provider >= 2) {
664664
// If there are multiple NICs "close" to the process, try to calculate package_rank
665-
package_rank = get_package_rank(process_info);
666665
provider = provider_table[package_rank % num_provider];
667666
} else if (num_provider == 1) {
668667
provider = provider_table[num_provider - 1];

0 commit comments

Comments
 (0)