libibumad: Increase max ports supported to 192 Increase UMAD_MAX_PORTS from 64 to 192. Since this value was used to size arrays on the stack, the affected arrays (counts, mapping and cas_pair) are now allocated dynamically on the heap to avoid large stack frames, and freed on all return paths. Signed-off-by: Asaf Mazor <amazor@nvidia.com>

diff --git a/libibumad/umad.c b/libibumad/umad.c
index d18fbb5..1618381 100644
--- a/libibumad/umad.c
+++ b/libibumad/umad.c

@@ -1482,13 +1482,22 @@
 	ca_names_t *legacy_ca_names = malloc(sizeof(ca_names_t));
 	if (!legacy_ca_names)
 		return -1;
-	struct port_guid_port_count counts[UMAD_MAX_PORTS] = {};
-	struct guid_ca_pairs_mapping mapping[UMAD_MAX_PORTS] = {};
+	struct port_guid_port_count *counts = calloc(UMAD_MAX_PORTS, sizeof(struct port_guid_port_count));
+	struct guid_ca_pairs_mapping *mapping = calloc(UMAD_MAX_PORTS, sizeof(struct guid_ca_pairs_mapping));
+
+	if (!counts || !mapping) {
+		free(counts);
+		free(mapping);
+		free(legacy_ca_names);
+		return -1;
+	}
 
 	memset(cas, 0, sizeof(struct umad_ca_pair) * max);
 	int cas_found = umad_get_cas_names(legacy_ca_names->_ca_names_arr, UMAD_MAX_DEVICES);
 
 	if (cas_found < 0) {
+		free(counts);
+		free(mapping);
 		free(legacy_ca_names);
 		return 0;
 	}
@@ -1532,6 +1541,8 @@
 				break;
 			} else {
 				umad_release_ca(&curr_ca);
+				free(counts);
+				free(mapping);
 				free(legacy_ca_names);
 				return -1;
 			}
@@ -1540,6 +1551,8 @@
 		umad_release_ca(&curr_ca);
 	}
 
+	free(counts);
+	free(mapping);
 	free(legacy_ca_names);
 	return added_devices;
 }
@@ -1625,18 +1638,23 @@
 	bool is_gsi		= false;
 
 	umad_ca_t ca;
-	struct umad_ca_pair cas_pair[UMAD_MAX_PORTS] = {};
+	struct umad_ca_pair *cas_pair;
 
 	if (!ca_pair)
 		return -1;
 
-	memset(cas_pair, 0, sizeof(cas_pair));
+	cas_pair = calloc(UMAD_MAX_PORTS, sizeof(struct umad_ca_pair));
+	if (!cas_pair)
+		return -1;
+
 	memset(ca_pair, 0, sizeof(*ca_pair));
 
 	num_cas = umad_get_smi_gsi_pairs(cas_pair, UMAD_MAX_PORTS);
 
-	if (num_cas <= 0)
+	if (num_cas <= 0) {
+		free(cas_pair);
 		return num_cas;
+	}
 
 	for (i = 0; i < (size_t)num_cas; ++i) {
 		if (enforce_smi && !cas_pair[i].smi_name[0])
@@ -1678,9 +1696,11 @@
 
 	if (rc) {
 		errno = ENODEV;
+		free(cas_pair);
 		return -errno;
 	}
 
+	free(cas_pair);
 	return rc;
 }
 

diff --git a/libibumad/umad.h b/libibumad/umad.h
index 3ca436b..e56a132 100644
--- a/libibumad/umad.h
+++ b/libibumad/umad.h

@@ -116,7 +116,7 @@
 #define SYS_IB_MAD_PORT		"port"
 #define SYS_IB_MAD_DEV		"ibdev"
 
-#define UMAD_MAX_PORTS		64
+#define UMAD_MAX_PORTS		192
 
 #define SYS_CA_PORTS_DIR	"ports"