[ovs-dev] [PATCH 1/5] raft: Add log length to the memory report.

Dumitru Ceara dceara at redhat.com
Wed Oct 28 10:56:42 UTC 2020


On 10/26/20 2:42 AM, Ilya Maximets wrote:
> In many cases a big part of a memory consumed by ovsdb-server process
> is a raft log, so it's important to add its length to the memory
> report.
> 
> Signed-off-by: Ilya Maximets <i.maximets at ovn.org>
> ---
>  ovsdb/raft.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/ovsdb/raft.c b/ovsdb/raft.c
> index 657eed813..ac85c6b67 100644
> --- a/ovsdb/raft.c
> +++ b/ovsdb/raft.c
> @@ -1029,6 +1029,7 @@ raft_get_memory_usage(const struct raft *raft, struct simap *usage)
>      }
>      simap_increase(usage, "raft-backlog-kB", backlog / 1000);
>      simap_increase(usage, "raft-connections", cnt);
> +    simap_increase(usage, "raft-log", raft->log_end - raft->log_start);
>  }
>  
>  /* Returns true if 'raft' has completed joining its cluster, has not left or
> 

Looks good to me, thanks!

Acked-by: Dumitru Ceara <dceara at redhat.com



More information about the dev mailing list