[Openais] [PATCH corosync/trunk] add some missing stats "++"

Steven Dake sdake at redhat.com
Mon Oct 12 14:42:39 PDT 2009


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
> Openais at lists.linux-foundation.org
> https://lists.linux-foundation.org/mailman/listinfo/openais



More information about the Openais mailing list