Skip to contentSkip to conversation

opensm/libvendor//osm_vendor_ibumad_sa.c: uninitialized fields

PATCH Applied Yevgeny Kliteynik 2 participants 2 messages Dec 9, 2009 – Dec 13, 2009 1 message not in the archive InfiniBand

lore
Share & tools
lore

Yevgeny Kliteynik

author
#0 lore raw

PATCH 1/1 opensm/libvendor//osm_vendor_ibumad_sa.c: uninitialized fields

Taking care of couple of uninitialized fields.
Discovered with 'osmtest -f a' - bad things
start to happen when a packet is dropped, and
osmtest should really get a timeout.

Signed-off-by Yevgeny Kliteynik <[email protected]>
---
 opensm/libvendor/osm_vendor_ibumad_sa.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)
opensm/libvendor/osm_vendor_ibumad_sa.c +2 -0
index e05a5ef..78f2bce 100644
@@ -241,6 +241,8 @@ osmv_bind_sa(IN osm_vendor_t * const p_vend,
 	bind_info.is_report_processor = FALSE;
 	bind_info.send_q_size = OSM_SM_DEFAULT_QP1_RCV_SIZE;
 	bind_info.recv_q_size = OSM_SM_DEFAULT_QP1_SEND_SIZE;
+	bind_info.timeout = OSM_DEFAULT_TRANS_TIMEOUT_MILLISEC;
+	bind_info.retries = OSM_DEFAULT_RETRY_COUNT;
 
 	/* allocate the new sa bind info */
 	p_sa_bind_info =
Signature · 7 lines
1.5.1.4


--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

A message that is not in the archive (sent off-list or not yet received): 4B1F6454.8060001-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org

Applied
#2 lore raw
Quoted 6 lines · Yevgeny Kliteynik

Taking care of couple of uninitialized fields. Discovered with 'osmtest -f a' - bad things start to happen when a packet is dropped, and osmtest should really get a timeout.

Signed-off-by Yevgeny Kliteynik <[email protected]>

Applied. Thanks.

Sasha
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html