KVM: arm64: Mark some VM-scoped allocations as __GFP_ACCOUNT
authorOliver Upton <oliver.upton@linux.dev>
Mon, 6 Feb 2023 23:52:29 +0000 (23:52 +0000)
committerOliver Upton <oliver.upton@linux.dev>
Tue, 7 Feb 2023 13:56:18 +0000 (13:56 +0000)
Generally speaking, any memory allocations that can be associated with a
particular VM should be charged to the cgroup of its process.
Nonetheless, there are a couple spots in KVM/arm64 that aren't currently
accounted:

 - the ccsidr array containing the virtualized cache hierarchy

 - the cpumask of supported cpus, for use of the vPMU on heterogeneous
   systems

Go ahead and set __GFP_ACCOUNT for these allocations.

Reviewed-by: Marc Zyngier <maz@kernel.org>
Reviewed-by: Akihiko Odaki <akihiko.odaki@daynix.com>
Reviewed-by: Gavin Shan <gshan@redhat.com>
Link: https://lore.kernel.org/r/20230206235229.4174711-1-oliver.upton@linux.dev
Signed-off-by: Oliver Upton <oliver.upton@linux.dev>
arch/arm64/kvm/arm.c
arch/arm64/kvm/sys_regs.c

index 9c5573b..3619f1e 100644 (file)
@@ -146,7 +146,7 @@ int kvm_arch_init_vm(struct kvm *kvm, unsigned long type)
        if (ret)
                goto err_unshare_kvm;
 
-       if (!zalloc_cpumask_var(&kvm->arch.supported_cpus, GFP_KERNEL)) {
+       if (!zalloc_cpumask_var(&kvm->arch.supported_cpus, GFP_KERNEL_ACCOUNT)) {
                ret = -ENOMEM;
                goto err_unshare_kvm;
        }
index 6a742af..5dc13d0 100644 (file)
@@ -160,7 +160,7 @@ static int set_ccsidr(struct kvm_vcpu *vcpu, u32 csselr, u32 val)
                if (val == get_ccsidr(vcpu, csselr))
                        return 0;
 
-               ccsidr = kmalloc_array(CSSELR_MAX, sizeof(u32), GFP_KERNEL);
+               ccsidr = kmalloc_array(CSSELR_MAX, sizeof(u32), GFP_KERNEL_ACCOUNT);
                if (!ccsidr)
                        return -ENOMEM;