Re: [V2 01/11] KVM: selftests: move vm_phy_pages_alloc() earlier in file

From: Andrew Jones
Date: Tue Aug 02 2022 - 06:00:01 EST


On Mon, Aug 01, 2022 at 01:10:59PM -0700, Peter Gonda wrote:
> From: Michael Roth <michael.roth@xxxxxxx>
>
> Subsequent patches will break some of this code out into file-local
> helper functions, which will be used by functions like vm_vaddr_alloc(),
> which currently are defined earlier in the file, so a forward
> declaration would be needed.
>
> Instead, move it earlier in the file, just above vm_vaddr_alloc() and
> and friends, which are the main users.
>
> Reviewed-by: Mingwei Zhang <mizhang@xxxxxxxxxx>
> Signed-off-by: Michael Roth <michael.roth@xxxxxxx>
> Signed-off-by: Peter Gonda <pgonda@xxxxxxxxxx>
> ---
> tools/testing/selftests/kvm/lib/kvm_util.c | 145 ++++++++++-----------
> 1 file changed, 72 insertions(+), 73 deletions(-)
>

Reviewed-by: Andrew Jones <andrew.jones@xxxxxxxxx>