On Thu, Apr 09, 2020 at 01:28:37PM +0200, Wolfgang Bumiller wrote:
> Signed-off-by: Wolfgang Bumiller <w.bumil...@proxmox.com>
> ---
> Changes to v1:
> * Add a convenience method for querying a set of multiple possible
>   controllers.
> 
>  src/PVE/LXC.pm        |  14 +----
>  src/PVE/LXC/CGroup.pm | 139 ++++++++++++++++++++++++++++++++++--------
>  2 files changed, 116 insertions(+), 37 deletions(-)
> 
> diff --git a/src/PVE/LXC.pm b/src/PVE/LXC.pm
> index 914dc2a..711ddd9 100644
> --- a/src/PVE/LXC.pm
> +++ b/src/PVE/LXC.pm
> @@ -401,19 +401,9 @@ sub parse_ipv4_cidr {
>      die "unable to parse ipv4 address/mask\n";
>  }
>  
> +# Deprecated. Use `PVE::LXC::CGroup::get_cgroup_controllers()` instead.
>  sub get_cgroup_subsystems {
> -     my $v1 = {};
> -     my $v2 = 0;
> -     my $data = PVE::Tools::file_get_contents('/proc/self/cgroup');
> -     while ($data =~ /^\d+:([^:\n]*):.*$/gm) {
> -             my $type = $1;
> -             if (length($type)) {
> -                     $v1->{$_} = 1 foreach split(/,/, $type);
> -             } else {
> -                     $v2 = 1;
> -             }
> -     }
> -     return wantarray ? ($v1, $v2) : $v1;
> +    PVE::LXC::Cgroup::get_v1_controllers();

^ This should be a captail 'G' here, CGroup.

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

Reply via email to