On 06/07/2011 08:16 AM, Jan Friesse wrote:
> Signed-off-by: Jan Friesse <[email protected]>
> ---
>  exec/coroipcs.c |    7 +++++++
>  1 files changed, 7 insertions(+), 0 deletions(-)
> 
> diff --git a/exec/coroipcs.c b/exec/coroipcs.c
> index e06acb5..ff78a92 100644
> --- a/exec/coroipcs.c
> +++ b/exec/coroipcs.c
> @@ -1564,6 +1564,7 @@ static void coroipcs_init_conn_stats (
>       char conn_name[CS_MAX_NAME_LENGTH];
>       char proc_name[CS_MAX_NAME_LENGTH];
>       char int_str[4];
> +     struct timeval tv;
>  
>       if (conn->client_pid > 0) {
>               if (pid_to_name (conn->client_pid, proc_name, 
> sizeof(proc_name))) {
> @@ -1584,6 +1585,12 @@ static void coroipcs_init_conn_stats (
>                       short_service_name_get(conn->service, int_str, 4),
>                       conn->fd);
>       }
> +
> +     gettimeofday(&tv, NULL);
> +     log_printf (LOGSYS_LEVEL_DEBUG, "coroipcs_init_conn_stats from %s at 
> %llu.%llu\n",
> +                     conn_name, (unsigned long long int)tv.tv_sec,
> +                     (unsigned long long int)tv.tv_usec);
> +
>       conn->stats_handle = api->stats_create_connection (conn_name, 
> conn->client_pid, conn->fd);
>       api->stats_update_value (conn->stats_handle, "service_id",
>               &conn->service, sizeof(conn->service));

The patch looks good minus the timestamping.  The timestamping should be
handled directly as part of the log_printf infrasturcture.  There is a
timestamp mode.
_______________________________________________
Openais mailing list
[email protected]
https://lists.linux-foundation.org/mailman/listinfo/openais

Reply via email to