[PATCH 1/1] Fix deadlock when running run_iter_bw_infinitely

[PATCH 1/1] Fix deadlock when running run_iter_bw_infinitely

[Hans Westgaard Ry]

The test uses a signal-handler to periodically print a line of report.
Snip libc info:
<quote>
24.4.6 Signal Handling and Nonreentrant Functions
Handler functions usually don't do very much.  The best practice is to
write a handler that does nothing but set an external variable that the
program checks regularly, and leave all serious work to the program.
</quote>
The current codebase violates the above and is deadlock prone.
The mutexes taken by the printf functions called from the signal handler
might be held by activity conducted by the main thread.

This commit fixes this by setting a flag in the signal handler and handle the
printout in the mainloop of the test.

Signed-off-by: Hans Westgaard Ry <hans.westgaard.ry-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org>
Reviewed-by: Håkon Bugge <haakon.bugge-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org>
Reviewed-by: Knut Omang <knut.omang-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org>
---
 src/perftest_resources.c | 22 ++++++++++++++++------
 1 file changed, 16 insertions(+), 6 deletions(-)

diff --git a/src/perftest_resources.c b/src/perftest_resources.c
index cb915e0..4d8e6c0 100755
--- a/src/perftest_resources.c
+++ b/src/perftest_resources.c
@@ -25,6 +25,12 @@ static enum ibv_wr_opcode opcode_atomic_array[] = {IBV_WR_ATOMIC_CMP_AND_SWP,IBV
 
 struct perftest_parameters* duration_param;
 struct check_alive_data check_alive_data;
+/*
+ * Flag to trig call to print_report_bw
+ * The flag is set in signal_handler and tested/reset in run_iter_bw_infinitely
+ */
+static volatile int trig_print_report_bw = 0;
+
 
 /******************************************************************************
  * Beginning
@@ -3214,7 +3220,15 @@ int run_iter_bw_infinitely(struct pingpong_context *ctx,struct perftest_paramete
 
 	/* main loop for posting */
 	while (1) {
-
+		if (trig_print_report_bw) {
+			duration_param->tcompleted[0] = get_cycles();
+			print_report_bw(duration_param,NULL);
+			duration_param->iters = 0;
+			alarm(duration_param->duration);
+			duration_param->tposted[0] = get_cycles();
+			trig_print_report_bw = 0;
+		}
+		
 		/* main loop to run over all the qps and post each time n messages */
 		for (index =0 ; index < num_of_qps ; index++) {
 
@@ -4213,11 +4227,7 @@ void check_alive(int sig)
  ******************************************************************************/
 void catch_alarm_infintely(int sig)
 {
-	duration_param->tcompleted[0] = get_cycles();
-	print_report_bw(duration_param,NULL);
-	duration_param->iters = 0;
-	alarm(duration_param->duration);
-	duration_param->tposted[0] = get_cycles();
+	trig_print_report_bw = 1;
 }
 
 /******************************************************************************
-- 
2.4.3

--
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

Ping: [PATCH 1/1] Fix deadlock when running run_iter_bw_infinitely

[Hans Westgaard Ry]

Any comments on this patch ?



On 11/18/2015 01:27 PM, Hans Westgaard Ry wrote:
> The test uses a signal-handler to periodically print a line of report.
> Snip libc info:
> <quote>
> 24.4.6 Signal Handling and Nonreentrant Functions
> Handler functions usually don't do very much.  The best practice is to
> write a handler that does nothing but set an external variable that the
> program checks regularly, and leave all serious work to the program.
> </quote>
> The current codebase violates the above and is deadlock prone.
> The mutexes taken by the printf functions called from the signal handler
> might be held by activity conducted by the main thread.
>
> This commit fixes this by setting a flag in the signal handler and handle the
> printout in the mainloop of the test.
>
> Signed-off-by: Hans Westgaard Ry <hans.westgaard.ry-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org>
> Reviewed-by: Håkon Bugge <haakon.bugge-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org>
> Reviewed-by: Knut Omang <knut.omang-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org>
> ---
>   src/perftest_resources.c | 22 ++++++++++++++++------
>   1 file changed, 16 insertions(+), 6 deletions(-)
>
> diff --git a/src/perftest_resources.c b/src/perftest_resources.c
> index cb915e0..4d8e6c0 100755
> --- a/src/perftest_resources.c
> +++ b/src/perftest_resources.c
> @@ -25,6 +25,12 @@ static enum ibv_wr_opcode opcode_atomic_array[] =IBV_WR_ATOMIC_CMP_AND_SWP,IBV
>   
>   struct perftest_parameters* duration_param;
>   struct check_alive_data check_alive_data;
> +/*
> + * Flag to trig call to print_report_bw
> + * The flag is set in signal_handler and tested/reset in run_iter_bw_infinitely
> + */
> +static volatile int trig_print_report_bw =;
> +
>   
>   /******************************************************************************
>    * Beginning
> @@ -3214,7 +3220,15 @@ int run_iter_bw_infinitely(struct pingpong_context *ctx,struct perftest_paramete
>   
>   	/* main loop for posting */
>   	while (1) {
> -
> +		if (trig_print_report_bw) {
> +			duration_param->tcompleted[0] =et_cycles();
> +			print_report_bw(duration_param,NULL);
> +			duration_param->iters =;
> +			alarm(duration_param->duration);
> +			duration_param->tposted[0] =et_cycles();
> +			trig_print_report_bw =;
> +		}
> +		
>   		/* main loop to run over all the qps and post each time n messages */
>   		for (index =; index < num_of_qps ; index++) {
>   
> @@ -4213,11 +4227,7 @@ void check_alive(int sig)
>    ******************************************************************************/
>   void catch_alarm_infintely(int sig)
>   {
> -	duration_param->tcompleted[0] =et_cycles();
> -	print_report_bw(duration_param,NULL);
> -	duration_param->iters =;
> -	alarm(duration_param->duration);
> -	duration_param->tposted[0] =et_cycles();
> +	trig_print_report_bw =;
>   }
>   
>   /******************************************************************************

--
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

Re: Ping: [PATCH 1/1] Fix deadlock when running run_iter_bw_infinitely

[Leon Romanovsky]

On Wed, Dec 09, 2015 at 09:33:11AM +0100, Hans Westgaard Ry wrote:
> Any comments on this patch ?
> 
> 
> 
> On 11/18/2015 01:27 PM, Hans Westgaard Ry wrote:
> >The test uses a signal-handler to periodically print a line of report.
> >+		if (trig_print_report_bw) {
> >+			duration_param->tcompleted[0] =et_cycles();
> >+			print_report_bw(duration_param,NULL);
> >+			duration_param->iters =;
Excuse me, what did you mean by the line above?

> >+			alarm(duration_param->duration);
> >+			duration_param->tposted[0] =et_cycles();
> >+			trig_print_report_bw =;
And this line too.

--
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

Re: Ping: [PATCH 1/1] Fix deadlock when running run_iter_bw_infinitely

[Hans Westgaard Ry]

Sorry but I didn't see that "something" gobbled part of the mail.
The correct lines are  (as in the original):

src/perftest_resources.c | 22 ++++++++++++++++------
  1 file changed, 16 insertions(+), 6 deletions(-)

diff --git a/src/perftest_resources.c b/src/perftest_resources.c
index cb915e0..4d8e6c0 100755
--- a/src/perftest_resources.c
+++ b/src/perftest_resources.c
@@ -25,6 +25,12 @@ static enum ibv_wr_opcode opcode_atomic_array[] = {IBV_WR_ATOMIC_CMP_AND_SWP,IBV
  
  struct perftest_parameters* duration_param;
  struct check_alive_data check_alive_data;
+/*
+ * Flag to trig call to print_report_bw
+ * The flag is set in signal_handler and tested/reset in run_iter_bw_infinitely
+ */
+static volatile int trig_print_report_bw = 0;
+
  
  /******************************************************************************
   * Beginning
@@ -3214,7 +3220,15 @@ int run_iter_bw_infinitely(struct pingpong_context *ctx,struct perftest_paramete
  
  	/* main loop for posting */
  	while (1) {
-
+		if (trig_print_report_bw) {
+			duration_param->tcompleted[0] = get_cycles();
+			print_report_bw(duration_param,NULL);
+			duration_param->iters = 0;
+			alarm(duration_param->duration);
+			duration_param->tposted[0] = get_cycles();
+			trig_print_report_bw = 0;
+		}
+		
  		/* main loop to run over all the qps and post each time n messages */
  		for (index =0 ; index < num_of_qps ; index++) {
  
@@ -4213,11 +4227,7 @@ void check_alive(int sig)
   ******************************************************************************/
  void catch_alarm_infintely(int sig)
  {
-	duration_param->tcompleted[0] = get_cycles();
-	print_report_bw(duration_param,NULL);
-	duration_param->iters = 0;
-	alarm(duration_param->duration);
-	duration_param->tposted[0] = get_cycles();
+	trig_print_report_bw = 1;
  }



On 12/09/2015 09:52 AM, Leon Romanovsky wrote:
> On Wed, Dec 09, 2015 at 09:33:11AM +0100, Hans Westgaard Ry wrote:
>> Any comments on this patch ?
>>
>>
>>
>> On 11/18/2015 01:27 PM, Hans Westgaard Ry wrote:
>>> The test uses a signal-handler to periodically print a line of report.
>>> +		if (trig_print_report_bw) {
>>> +			duration_param->tcompleted[0] =et_cycles();
>>> +			print_report_bw(duration_param,NULL);
>>> +			duration_param->iters =;
> Excuse me, what did you mean by the line above?
>
>>> +			alarm(duration_param->duration);
>>> +			duration_param->tposted[0] =et_cycles();
>>> +			trig_print_report_bw =;
> And this line too.
>

--
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

Re: Ping: [PATCH 1/1] Fix deadlock when running run_iter_bw_infinitely

[Leon Romanovsky]

On Wed, Dec 09, 2015 at 10:11:55AM +0100, Hans Westgaard Ry wrote:
> Sorry but I didn't see that "something" gobbled part of the mail.
> The correct lines are  (as in the original):
> 
> src/perftest_resources.c | 22 ++++++++++++++++------
>  1 file changed, 16 insertions(+), 6 deletions(-)
> 
> diff --git a/src/perftest_resources.c b/src/perftest_resources.c
> index cb915e0..4d8e6c0 100755
> --- a/src/perftest_resources.c
> +++ b/src/perftest_resources.c
> @@ -25,6 +25,12 @@ static enum ibv_wr_opcode opcode_atomic_array[] = {IBV_WR_ATOMIC_CMP_AND_SWP,IBV
>  struct perftest_parameters* duration_param;
>  struct check_alive_data check_alive_data;
> +/*
> + * Flag to trig call to print_report_bw
> + * The flag is set in signal_handler and tested/reset in run_iter_bw_infinitely
> + */
> +static volatile int trig_print_report_bw = 0;
> +
>  /******************************************************************************
>   * Beginning
> @@ -3214,7 +3220,15 @@ int run_iter_bw_infinitely(struct pingpong_context *ctx,struct perftest_paramete
>  	/* main loop for posting */
>  	while (1) {
> -
> +		if (trig_print_report_bw) {
> +			duration_param->tcompleted[0] = get_cycles();
> +			print_report_bw(duration_param,NULL);
> +			duration_param->iters = 0;
> +			alarm(duration_param->duration);
> +			duration_param->tposted[0] = get_cycles();
> +			trig_print_report_bw = 0;
> +		}
> +		
>  		/* main loop to run over all the qps and post each time n messages */
>  		for (index =0 ; index < num_of_qps ; index++) {
> @@ -4213,11 +4227,7 @@ void check_alive(int sig)
>   ******************************************************************************/
>  void catch_alarm_infintely(int sig)
>  {
> -	duration_param->tcompleted[0] = get_cycles();
> -	print_report_bw(duration_param,NULL);
> -	duration_param->iters = 0;
> -	alarm(duration_param->duration);
> -	duration_param->tposted[0] = get_cycles();
> +	trig_print_report_bw = 1;
>  }
Thanks, it looks good,
Reviewed-by: Leon Romanovsky <leonro-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>

> 
> 
> 
> On 12/09/2015 09:52 AM, Leon Romanovsky wrote:
> >On Wed, Dec 09, 2015 at 09:33:11AM +0100, Hans Westgaard Ry wrote:
> >>Any comments on this patch ?
> >>
> >>
> >>
> >>On 11/18/2015 01:27 PM, Hans Westgaard Ry wrote:
> >>>The test uses a signal-handler to periodically print a line of report.
> >>>+		if (trig_print_report_bw) {
> >>>+			duration_param->tcompleted[0] =et_cycles();
> >>>+			print_report_bw(duration_param,NULL);
> >>>+			duration_param->iters =;
> >Excuse me, what did you mean by the line above?
> >
> >>>+			alarm(duration_param->duration);
> >>>+			duration_param->tposted[0] =et_cycles();
> >>>+			trig_print_report_bw =;
> >And this line too.
> >
> 
> --
> 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
--
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

Re: Ping: [PATCH 1/1] Fix deadlock when running run_iter_bw_infinitely

[Hans Westgaard Ry]

Hi,
I cloned  git://git.openfabrics.org/~grockah/perftest.git today and found
quite a few new commits but not this patch.

Am I looking in the wrong repository or are there problems with this patch ?

Hans

On 12/09/2015 10:30 AM, Leon Romanovsky wrote:
> On Wed, Dec 09, 2015 at 10:11:55AM +0100, Hans Westgaard Ry wrote:
>> Sorry but I didn't see that "something" gobbled part of the mail.
>> The correct lines are  (as in the original):
>>
>> src/perftest_resources.c | 22 ++++++++++++++++------
>>   1 file changed, 16 insertions(+), 6 deletions(-)
>>
>> diff --git a/src/perftest_resources.c b/src/perftest_resources.c
>> index cb915e0..4d8e6c0 100755
>> --- a/src/perftest_resources.c
>> +++ b/src/perftest_resources.c
>> @@ -25,6 +25,12 @@ static enum ibv_wr_opcode opcode_atomic_array[] = {IBV_WR_ATOMIC_CMP_AND_SWP,IBV
>>   struct perftest_parameters* duration_param;
>>   struct check_alive_data check_alive_data;
>> +/*
>> + * Flag to trig call to print_report_bw
>> + * The flag is set in signal_handler and tested/reset in run_iter_bw_infinitely
>> + */
>> +static volatile int trig_print_report_bw = 0;
>> +
>>   /******************************************************************************
>>    * Beginning
>> @@ -3214,7 +3220,15 @@ int run_iter_bw_infinitely(struct pingpong_context *ctx,struct perftest_paramete
>>   	/* main loop for posting */
>>   	while (1) {
>> -
>> +		if (trig_print_report_bw) {
>> +			duration_param->tcompleted[0] = get_cycles();
>> +			print_report_bw(duration_param,NULL);
>> +			duration_param->iters = 0;
>> +			alarm(duration_param->duration);
>> +			duration_param->tposted[0] = get_cycles();
>> +			trig_print_report_bw = 0;
>> +		}
>> +		
>>   		/* main loop to run over all the qps and post each time n messages */
>>   		for (index =0 ; index < num_of_qps ; index++) {
>> @@ -4213,11 +4227,7 @@ void check_alive(int sig)
>>    ******************************************************************************/
>>   void catch_alarm_infintely(int sig)
>>   {
>> -	duration_param->tcompleted[0] = get_cycles();
>> -	print_report_bw(duration_param,NULL);
>> -	duration_param->iters = 0;
>> -	alarm(duration_param->duration);
>> -	duration_param->tposted[0] = get_cycles();
>> +	trig_print_report_bw = 1;
>>   }
> Thanks, it looks good,
> Reviewed-by: Leon Romanovsky <leonro-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
>
>>
>>
>> On 12/09/2015 09:52 AM, Leon Romanovsky wrote:
>>> On Wed, Dec 09, 2015 at 09:33:11AM +0100, Hans Westgaard Ry wrote:
>>>> Any comments on this patch ?
>>>>
>>>>
>>>>
>>>> On 11/18/2015 01:27 PM, Hans Westgaard Ry wrote:
>>>>> The test uses a signal-handler to periodically print a line of report.
>>>>> +		if (trig_print_report_bw) {
>>>>> +			duration_param->tcompleted[0] =et_cycles();
>>>>> +			print_report_bw(duration_param,NULL);
>>>>> +			duration_param->iters =;
>>> Excuse me, what did you mean by the line above?
>>>
>>>>> +			alarm(duration_param->duration);
>>>>> +			duration_param->tposted[0] =et_cycles();
>>>>> +			trig_print_report_bw =;
>>> And this line too.
>>>
>> --
>> 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

--
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

Re: Ping: [PATCH 1/1] Fix deadlock when running run_iter_bw_infinitely

[Leon Romanovsky]

On Fri, Jan 08, 2016 at 08:36:24AM +0100, Hans Westgaard Ry wrote:
> Hi,
> I cloned  git://git.openfabrics.org/~grockah/perftest.git today and found
> quite a few new commits but not this patch.
> 
> Am I looking in the wrong repository or are there problems with this patch ?

Gil/Or
Your feedback will be appreciated.

> 
> Hans
> 
> On 12/09/2015 10:30 AM, Leon Romanovsky wrote:
> >On Wed, Dec 09, 2015 at 10:11:55AM +0100, Hans Westgaard Ry wrote:
> >>Sorry but I didn't see that "something" gobbled part of the mail.
> >>The correct lines are  (as in the original):
> >>
> >>src/perftest_resources.c | 22 ++++++++++++++++------
> >>  1 file changed, 16 insertions(+), 6 deletions(-)
> >>
> >>diff --git a/src/perftest_resources.c b/src/perftest_resources.c
> >>index cb915e0..4d8e6c0 100755
> >>--- a/src/perftest_resources.c
> >>+++ b/src/perftest_resources.c
> >>@@ -25,6 +25,12 @@ static enum ibv_wr_opcode opcode_atomic_array[] = {IBV_WR_ATOMIC_CMP_AND_SWP,IBV
> >>  struct perftest_parameters* duration_param;
> >>  struct check_alive_data check_alive_data;
> >>+/*
> >>+ * Flag to trig call to print_report_bw
> >>+ * The flag is set in signal_handler and tested/reset in run_iter_bw_infinitely
> >>+ */
> >>+static volatile int trig_print_report_bw = 0;
> >>+
> >>  /******************************************************************************
> >>   * Beginning
> >>@@ -3214,7 +3220,15 @@ int run_iter_bw_infinitely(struct pingpong_context *ctx,struct perftest_paramete
> >>  	/* main loop for posting */
> >>  	while (1) {
> >>-
> >>+		if (trig_print_report_bw) {
> >>+			duration_param->tcompleted[0] = get_cycles();
> >>+			print_report_bw(duration_param,NULL);
> >>+			duration_param->iters = 0;
> >>+			alarm(duration_param->duration);
> >>+			duration_param->tposted[0] = get_cycles();
> >>+			trig_print_report_bw = 0;
> >>+		}
> >>+		
> >>  		/* main loop to run over all the qps and post each time n messages */
> >>  		for (index =0 ; index < num_of_qps ; index++) {
> >>@@ -4213,11 +4227,7 @@ void check_alive(int sig)
> >>   ******************************************************************************/
> >>  void catch_alarm_infintely(int sig)
> >>  {
> >>-	duration_param->tcompleted[0] = get_cycles();
> >>-	print_report_bw(duration_param,NULL);
> >>-	duration_param->iters = 0;
> >>-	alarm(duration_param->duration);
> >>-	duration_param->tposted[0] = get_cycles();
> >>+	trig_print_report_bw = 1;
> >>  }
> >Thanks, it looks good,
> >Reviewed-by: Leon Romanovsky <leonro-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
> >
> >>
> >>
> >>On 12/09/2015 09:52 AM, Leon Romanovsky wrote:
> >>>On Wed, Dec 09, 2015 at 09:33:11AM +0100, Hans Westgaard Ry wrote:
> >>>>Any comments on this patch ?
> >>>>
> >>>>
> >>>>
> >>>>On 11/18/2015 01:27 PM, Hans Westgaard Ry wrote:
> >>>>>The test uses a signal-handler to periodically print a line of report.
> >>>>>+		if (trig_print_report_bw) {
> >>>>>+			duration_param->tcompleted[0] =et_cycles();
> >>>>>+			print_report_bw(duration_param,NULL);
> >>>>>+			duration_param->iters =;
> >>>Excuse me, what did you mean by the line above?
> >>>
> >>>>>+			alarm(duration_param->duration);
> >>>>>+			duration_param->tposted[0] =et_cycles();
> >>>>>+			trig_print_report_bw =;
> >>>And this line too.
> >>>
> >>--
> >>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
> 
> --
> 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
--
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

Re: Ping: [PATCH 1/1] Fix deadlock when running run_iter_bw_infinitely

[Hans Westgaard Ry]

On 12/09/2015 10:30 AM, Leon Romanovsky wrote:
> diff --git a/src/perftest_resources.c b/src/perftest_resources.c
> index cb915e0..4d8e6c0 100755
> --- a/src/perftest_resources.c
> +++ b/src/perftest_resources.c
> @@ -25,6 +25,12 @@ static enum ibv_wr_opcode opcode_atomic_array[] = {IBV_WR_ATOMIC_CMP_AND_SWP,IBV
>   struct perftest_parameters* duration_param;
>   struct check_alive_data check_alive_data;
> +/*
> + * Flag to trig call to print_report_bw
> + * The flag is set in signal_handler and tested/reset in run_iter_bw_infinitely
> + */
> +static volatile int trig_print_report_bw = 0;
> +
>   /******************************************************************************
>    * Beginning
> @@ -3214,7 +3220,15 @@ int run_iter_bw_infinitely(struct pingpong_context *ctx,struct perftest_paramete
>   	/* main loop for posting */
>   	while (1) {
> -
> +		if (trig_print_report_bw) {
> +			duration_param->tcompleted[0] = get_cycles();
> +			print_report_bw(duration_param,NULL);
> +			duration_param->iters = 0;
> +			alarm(duration_param->duration);
> +			duration_param->tposted[0] = get_cycles();
> +			trig_print_report_bw = 0;
> +		}
> +		
>   		/* main loop to run over all the qps and post each time n messages */
>   		for (index =0 ; index < num_of_qps ; index++) {
> @@ -4213,11 +4227,7 @@ void check_alive(int sig)
>    ******************************************************************************/
>   void catch_alarm_infintely(int sig)
>   {
> -	duration_param->tcompleted[0] = get_cycles();
> -	print_report_bw(duration_param,NULL);
> -	duration_param->iters = 0;
> -	alarm(duration_param->duration);
> -	duration_param->tposted[0] = get_cycles();
> +	trig_print_report_bw = 1;
>   }
> Thanks, it looks good,
> Reviewed-by: Leon Romanovsky <leonro-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
Hi,
I cloned  git://git.openfabrics.org/~grockah/perftest.git today and found
quite a few new commits but not this patch.

Am I looking in the wrong repository or are there problems with this 
patch ?

Hans
--
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