On Thu, Apr 11, 2013 at 3:05 PM, Ben Pfaff <[email protected]> wrote:

> Commit b9de4087685 (memory: Avoid an empty log message if there's nothing
> to log.) introduced a 100% CPU usage condition when there is nothing to
> log.  This fixes the problem.
>
> Reported-by: Ansis Atteka <[email protected]>
> Signed-off-by: Ben Pfaff <[email protected]>
> ---
>  lib/memory.c |    6 ++++--
>  1 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/lib/memory.c b/lib/memory.c
> index c26c743..1137390 100644
> --- a/lib/memory.c
> +++ b/lib/memory.c
> @@ -137,8 +137,10 @@ memory_report(const struct simap *usage)
>      ds_init(&s);
>      compose_report(usage, &s);
>
> -    if (want_report && s.length) {
> -        VLOG_INFO("%s", ds_cstr(&s));
> +    if (want_report) {
> +        if (s.length) {
> +            VLOG_INFO("%s", ds_cstr(&s));
> +        }
>          want_report = false;
>      }
>      if (n_conns) {
> --
> 1.7.2.5
>
> Acked-by: Ansis Atteka <[email protected]>
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to