* [PATCH] osmtest/osmtest.c: handle timeouts in PR stress test
@ 2010-09-07 15:01 Yevgeny Kliteynik
[not found] ` <4C8653D6.1000201-VPRAkNaXOzVS1MOuV/RT9w@public.gmane.org>
0 siblings, 1 reply; 2+ messages in thread
From: Yevgeny Kliteynik @ 2010-09-07 15:01 UTC (permalink / raw)
To: Sasha Khapyorsky, Linux RDMA
There was some "preparation" code for handling
PathRecord timeouts in SA stress codes: variable
num_timeouts exists, and it's value is checked.
What was missing is actually increasing the
num_timeouts value when timeout happens.
Signed-off-by: Yevgeny Kliteynik <kliteyn-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org>
---
opensm/osmtest/osmtest.c | 12 ++++++++++--
1 files changed, 10 insertions(+), 2 deletions(-)
diff --git a/opensm/osmtest/osmtest.c b/opensm/osmtest/osmtest.c
index 7412ce7..b2ec3fa 100644
--- a/opensm/osmtest/osmtest.c
+++ b/opensm/osmtest/osmtest.c
@@ -2753,7 +2753,11 @@ static ib_api_status_t osmtest_stress_small_rmpp(IN osmtest_t * const p_osmt)
status = osmtest_stress_port_recs_small(p_osmt, &delta_recs,
&delta_queries);
- if (status != IB_SUCCESS)
+ if (status == IB_TIMEOUT) {
+ num_timeouts++;
+ continue;
+ }
+ else if (status != IB_SUCCESS)
goto Exit;
num_recs += delta_recs;
@@ -2889,7 +2893,11 @@ static ib_api_status_t osmtest_stress_get_pr(IN osmtest_t * const p_osmt)
status = osmtest_stress_path_recs_by_lid(p_osmt,
&delta_recs,
&delta_queries);
- if (status != IB_SUCCESS)
+ if (status == IB_TIMEOUT) {
+ num_timeouts++;
+ continue;
+ }
+ else if (status != IB_SUCCESS)
goto Exit;
num_recs += delta_recs;
--
1.6.2.4
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] osmtest/osmtest.c: handle timeouts in PR stress test
[not found] ` <4C8653D6.1000201-VPRAkNaXOzVS1MOuV/RT9w@public.gmane.org>
@ 2010-09-09 14:22 ` Sasha Khapyorsky
0 siblings, 0 replies; 2+ messages in thread
From: Sasha Khapyorsky @ 2010-09-09 14:22 UTC (permalink / raw)
To: kliteyn-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb; +Cc: Linux RDMA
On 18:01 Tue 07 Sep , Yevgeny Kliteynik wrote:
> There was some "preparation" code for handling
> PathRecord timeouts in SA stress codes: variable
> num_timeouts exists, and it's value is checked.
> What was missing is actually increasing the
> num_timeouts value when timeout happens.
>
> Signed-off-by: Yevgeny Kliteynik <kliteyn-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org>
Applied. Thanks.
Sasha
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2010-09-09 14:22 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-09-07 15:01 [PATCH] osmtest/osmtest.c: handle timeouts in PR stress test Yevgeny Kliteynik
[not found] ` <4C8653D6.1000201-VPRAkNaXOzVS1MOuV/RT9w@public.gmane.org>
2010-09-09 14:22 ` Sasha Khapyorsky
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox