[ovs-dev] [PATCH] timeval: Only log poll intervals longer than 50 ms.

Ethan Jackson ethan at nicira.com
Wed Mar 30 18:25:37 UTC 2011


Looks Good.

On Mon, Mar 28, 2011 at 1:43 PM, Ben Pfaff <blp at nicira.com> wrote:
> When poll interval-based logging was introduced a long time, we were
> actively interested in looking at almost every long poll interval.  But
> these days, with OVS working rather well, with pretty good latency, most
> of the messages are red herrings that bother some administrators and
> provoke false reports.  So this commit suppresses all but the most
> egregious long poll intervals that may in fact be worth looking at.
>
> NIC-366.
> ---
>  lib/timeval.c |    7 ++++---
>  1 files changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/lib/timeval.c b/lib/timeval.c
> index d24ba03..84c90f3 100644
> --- a/lib/timeval.c
> +++ b/lib/timeval.c
> @@ -1,5 +1,5 @@
>  /*
> - * Copyright (c) 2008, 2009, 2010 Nicira Networks.
> + * Copyright (c) 2008, 2009, 2010, 2011 Nicira Networks.
>  *
>  * Licensed under the Apache License, Version 2.0 (the "License");
>  * you may not use this file except in compliance with the License.
> @@ -409,8 +409,9 @@ log_poll_interval(long long int last_wakeup, const struct rusage *last_rusage)
>     now = time_msec();
>     interval = MIN(10000, now - last_wakeup) << 4;
>
> -    /* Warn if we took too much time between polls. */
> -    if (n_samples > 10 && interval > mean_interval * 8) {
> +    /* Warn if we took too much time between polls: at least 50 ms and at least
> +     * 8X the mean interval. */
> +    if (n_samples > 10 && interval > mean_interval * 8 && interval > 50 * 16) {
>         struct rusage rusage;
>
>         getrusage(RUSAGE_SELF, &rusage);
> --
> 1.7.1
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>



More information about the dev mailing list