Commit 5f623a59 authored by Oliver Upton's avatar Oliver Upton
Browse files

KVM: arm64: Mark some VM-scoped allocations as __GFP_ACCOUNT 



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: default avatarMarc Zyngier <maz@kernel.org>
Reviewed-by: default avatarAkihiko Odaki <akihiko.odaki@daynix.com>
Reviewed-by: default avatarGavin Shan <gshan@redhat.com>
Link: https://lore.kernel.org/r/20230206235229.4174711-1-oliver.upton@linux.dev


Signed-off-by: default avatarOliver Upton <oliver.upton@linux.dev>
parent 7af0c253

arch/arm64/kvm/arm.c

+1 −1
Original line number Diff line number Diff line
@@ -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;

	}

arch/arm64/kvm/sys_regs.c

+1 −1
Original line number Diff line number Diff line
@@ -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;