good for merge

regards
-steve

On Tue, 2009-10-13 at 10:40 +1300, angus salkeld wrote:
> Hi
> 
> Here are 5 missing calls to increment the relevant stats.
> 
> -Angus
> 
> --- exec/totemsrp.c   2009-10-13 10:27:45.000000000 +1300
> +++ ../writable_corosync/exec/totemsrp.c      2009-10-13 08:01:32.000000000 
> +1300
> @@ -2787,6 +2787,8 @@
>       memcpy (instance->orf_token_retransmit, commit_token, 
> commit_token_size);
>       instance->orf_token_retransmit_size = commit_token_size;
>  
> +     instance->stats->memb_commit_token_tx++;
> +
>       totemrrp_token_send (instance->totemrrp_context,
>               commit_token,
>               commit_token_size);
> --- exec/coroipcs.c   2009-10-13 10:31:52.000000000 +1300
> +++ ../writable_corosync/exec/coroipcs.c      2009-10-13 10:39:03.000000000 
> +1300
> @@ -1063,6 +1063,7 @@
>  retry_semop:
>       res = semop (conn_info->semid, &sop, 1);
>       if ((res == -1) && (errno == EINTR || errno == EAGAIN)) {
> +             stats_api->stats_increment_value (conn_info->stats_handle, 
> "sem_retry_count");
>               goto retry_semop;
>       } else
>       if ((res == -1) && (errno == EINVAL || errno == EIDRM)) {
> @@ -1102,12 +1103,14 @@
>  retry_semop:
>       res = semop (conn_info->semid, &sop, 1);
>       if ((res == -1) && (errno == EINTR || errno == EAGAIN)) {
> +             stats_api->stats_increment_value (conn_info->stats_handle, 
> "sem_retry_count");
>               goto retry_semop;
>       } else
>       if ((res == -1) && (errno == EINVAL || errno == EIDRM)) {
>               return (0);
>       }
>  #endif
> +     stats_api->stats_increment_value (conn_info->stats_handle, "responses");
>       return (0);
>  }
>  
> @@ -1179,6 +1182,7 @@
>  retry_semop:
>       res = semop (conn_info->semid, &sop, 1);
>       if ((res == -1) && (errno == EINTR || errno == EAGAIN)) {
> +             stats_api->stats_increment_value (conn_info->stats_handle, 
> "sem_retry_count");
>               goto retry_semop;
>       } else
>       if ((res == -1) && (errno == EINVAL || errno == EIDRM)) {
> 
> 
> _______________________________________________
> Openais mailing list
> [email protected]
> https://lists.linux-foundation.org/mailman/listinfo/openais

_______________________________________________
Openais mailing list
[email protected]
https://lists.linux-foundation.org/mailman/listinfo/openais

Reply via email to