On 4/3/20 4:37 PM, w.bumil...@proxmox.com wrote:
> ...
> +# Parse some memory data from `memory.stat`
> +sub get_memory_stat {
> +    my ($self) = @_;
> +
> +    my $res = {
> +     mem => 0,
> +     swap => 0,
> +    };
> +
> +    if (cgroup_mode() == 2) {
> +     if (defined(my $path = $self->get_path('memory'))) {
> +         my $mem = file_get_contents("$path/memory.current");
> +         my $swap = file_get_contents("$path/memory.swap.current");
> +
> +         chomp ($mem, $swap);
> +
> +         # FIXME: For the cgv1 equivalent of `total_cache` we may need to 
> sum up
> +         # the values in `memory.stat`...

I guess you plan to fix this, as "wrong" memory stats seem to startle our
users quite a bit ^^

> +
> +         $res->{mem} = $mem;
> +         $res->{swap} = $swap;

_______________________________________________
pve-devel mailing list
pve-devel@pve.proxmox.com
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to