[PATCH v2 2/2] KVM: arm64: Add memcg accounting to KVM allocations

[Justin He]

Hi Marc

> -----Original Message-----
> From: Oliver Upton <oupton at google.com>
> Sent: Monday, September 13, 2021 8:21 PM
> To: Justin He <Justin.He at arm.com>
> Cc: Marc Zyngier <maz at kernel.org>; James Morse <James.Morse at arm.com>;
> Alexandru Elisei <Alexandru.Elisei at arm.com>; Suzuki Poulose
> <Suzuki.Poulose at arm.com>; Xiaoming Ni <nixiaoming at huawei.com>; Lorenzo
> Pieralisi <Lorenzo.Pieralisi at arm.com>; Kees Cook <keescook at chromium.org>;
> Catalin Marinas <Catalin.Marinas at arm.com>; Nick Desaulniers
> <ndesaulniers at google.com>; linux-kernel at vger.kernel.org; Liu Shixin
> <liushixin2 at huawei.com>; Sami Tolvanen <samitolvanen at google.com>; Will
> Deacon <will at kernel.org>; kvmarm at lists.cs.columbia.edu; linux-arm-
> kernel at lists.infradead.org
> Subject: Re: [PATCH v2 2/2] KVM: arm64: Add memcg accounting to KVM
> allocations
> 
[...]
> > Do you mean kvm_hyp_zalloc_page() here?
> > Seems kvm_hyp_zalloc_page() is in both global and VM scopes.
> 
> Doh! kvm_hyp_zalloc_page() is only for the hyp's page tables, hence I
> believe your patch is correct as it stands. As such:
> 
> Reviewed-by: Oliver Upton <oupton at google.com>
Kindly ping for the whole series. 😉

--
Cheers,
Justin (Jia He)