From: Paolo Bonzini <pbonzini@redhat.com>
To: Andrew Jones <drjones@redhat.com>, kvm@vger.kernel.org
Cc: borntraeger@de.ibm.com, frankja@linux.ibm.com,
bgardon@google.com, peterx@redhat.com
Subject: Re: [PATCH 00/11] KVM: selftests: Cleanups
Date: Fri, 6 Nov 2020 14:01:12 +0100 [thread overview]
Message-ID: <3975bb56-3af2-6387-3b45-a3cac4787829@redhat.com> (raw)
In-Reply-To: <20201104212357.171559-1-drjones@redhat.com>
On 04/11/20 22:23, Andrew Jones wrote:
> This series attempts to clean up demand_paging_test and dirty_log_test
> by factoring out common code, creating some new API along the way. It's
> main goal is to prepare for even more factoring that Ben and Peter want
> to do. The series would have a nice negative diff stat, but it also
> picks up a few of Peter's patches for his new dirty log test. So, the
> +/- diff stat is close to equal. It's not as close as an electoral vote
> count, but it's close.
>
> I've tested on x86 and AArch64 (one config each), but not s390x.
>
> Thanks,
> drew
>
>
> Andrew Jones (8):
> KVM: selftests: Add x86_64/tsc_msrs_test to .gitignore
> KVM: selftests: Drop pointless vm_create wrapper
> KVM: selftests: Make the per vcpu memory size global
> KVM: selftests: Make the number of vcpus global
> KVM: selftests: Factor out guest mode code
> KVM: selftests: Make vm_create_default common
> KVM: selftests: Introduce vm_create_[default_]vcpus
> KVM: selftests: Remove create_vm
>
> Peter Xu (3):
> KVM: selftests: Always clear dirty bitmap after iteration
> KVM: selftests: Use a single binary for dirty/clear log test
> KVM: selftests: Introduce after_vcpu_run hook for dirty log test
>
> tools/testing/selftests/kvm/.gitignore | 2 +-
> tools/testing/selftests/kvm/Makefile | 4 +-
> .../selftests/kvm/clear_dirty_log_test.c | 6 -
> .../selftests/kvm/demand_paging_test.c | 213 +++-------
> tools/testing/selftests/kvm/dirty_log_test.c | 372 ++++++++++--------
> .../selftests/kvm/include/aarch64/processor.h | 3 +
> .../selftests/kvm/include/guest_modes.h | 21 +
> .../testing/selftests/kvm/include/kvm_util.h | 20 +-
> .../selftests/kvm/include/s390x/processor.h | 4 +
> .../selftests/kvm/include/x86_64/processor.h | 4 +
> .../selftests/kvm/lib/aarch64/processor.c | 17 -
> tools/testing/selftests/kvm/lib/guest_modes.c | 70 ++++
> tools/testing/selftests/kvm/lib/kvm_util.c | 62 ++-
> .../selftests/kvm/lib/s390x/processor.c | 22 --
> .../selftests/kvm/lib/x86_64/processor.c | 32 --
> 15 files changed, 445 insertions(+), 407 deletions(-)
> delete mode 100644 tools/testing/selftests/kvm/clear_dirty_log_test.c
> create mode 100644 tools/testing/selftests/kvm/include/guest_modes.h
> create mode 100644 tools/testing/selftests/kvm/lib/guest_modes.c
>
Queued (or overridden by patches already in queue) patches 1-8, thanks.
prev parent reply other threads:[~2020-11-06 13:01 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-11-04 21:23 [PATCH 00/11] KVM: selftests: Cleanups Andrew Jones
2020-11-04 21:23 ` [PATCH 01/11] KVM: selftests: Add x86_64/tsc_msrs_test to .gitignore Andrew Jones
2020-11-04 21:23 ` [PATCH 02/11] KVM: selftests: Drop pointless vm_create wrapper Andrew Jones
2020-11-04 21:23 ` [PATCH 03/11] KVM: selftests: Always clear dirty bitmap after iteration Andrew Jones
2020-11-04 21:23 ` [PATCH 04/11] KVM: selftests: Use a single binary for dirty/clear log test Andrew Jones
2020-11-04 21:23 ` [PATCH 05/11] KVM: selftests: Introduce after_vcpu_run hook for dirty " Andrew Jones
2020-11-04 21:23 ` [PATCH 06/11] KVM: selftests: Make the per vcpu memory size global Andrew Jones
2020-11-04 21:23 ` [PATCH 07/11] KVM: selftests: Make the number of vcpus global Andrew Jones
2020-11-04 21:23 ` [PATCH 08/11] KVM: selftests: Factor out guest mode code Andrew Jones
2020-11-04 21:23 ` [PATCH 09/11] KVM: selftests: Make vm_create_default common Andrew Jones
2020-11-04 21:36 ` Andrew Jones
2020-11-05 7:18 ` Christian Borntraeger
2020-11-05 9:59 ` Andrew Jones
2020-11-05 18:45 ` Peter Xu
2020-11-05 18:54 ` Christian Borntraeger
2020-11-04 21:23 ` [PATCH 10/11] KVM: selftests: Introduce vm_create_[default_]vcpus Andrew Jones
2020-11-04 21:23 ` [PATCH 11/11] KVM: selftests: Remove create_vm Andrew Jones
2020-11-05 7:08 ` [PATCH 00/11] KVM: selftests: Cleanups Christian Borntraeger
2020-11-05 18:55 ` Peter Xu
2020-11-05 19:41 ` Ben Gardon
2020-11-06 9:45 ` Andrew Jones
2020-11-06 15:04 ` Peter Xu
2020-11-09 17:11 ` Ben Gardon
2020-11-06 13:01 ` Paolo Bonzini [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=3975bb56-3af2-6387-3b45-a3cac4787829@redhat.com \
--to=pbonzini@redhat.com \
--cc=bgardon@google.com \
--cc=borntraeger@de.ibm.com \
--cc=drjones@redhat.com \
--cc=frankja@linux.ibm.com \
--cc=kvm@vger.kernel.org \
--cc=peterx@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.