Re: [PATCH] kvm: account allocation in generic version of kvm_arch_alloc_vm()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Nit, please capitalize "KVM:" in the shortlog.

On Thu, Nov 17, 2022, Alexey Dobriyan wrote:
> This is persistent allocation and arch-specific versions account it.

The justification isn't that "x86 does it", e.g. the x86 code could be wrong.
The justification is that the allocation is tied to the current task as opposed
to being a KVM-wide allocation.  It's worth calling out that x86 already accounts
its allocation, but that should be a footnote, not the reason for converting the
generic allocation.

> Signed-off-by: Alexey Dobriyan <adobriyan@xxxxxxxxx>
> ---

With a reworded changelog,

Reviewed-by: Sean Christopherson <seanjc@xxxxxxxxxx>



[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux