On Tue, Jan 11, 2022 at 1:55 AM Andrew Jones <drjones@xxxxxxxxxx> wrote: > > On Mon, Jan 10, 2022 at 09:04:41PM +0000, Jing Zhang wrote: > > For ARM64, if no vgic is setup before the dirty log perf test, the > > userspace irqchip would be used, which would affect the dirty log perf > > test result. > > > > Signed-off-by: Jing Zhang <jingzhangos@xxxxxxxxxx> > > --- > > tools/testing/selftests/kvm/dirty_log_perf_test.c | 10 ++++++++++ > > 1 file changed, 10 insertions(+) > > > > diff --git a/tools/testing/selftests/kvm/dirty_log_perf_test.c b/tools/testing/selftests/kvm/dirty_log_perf_test.c > > index 1954b964d1cf..b501338d9430 100644 > > --- a/tools/testing/selftests/kvm/dirty_log_perf_test.c > > +++ b/tools/testing/selftests/kvm/dirty_log_perf_test.c > > @@ -18,6 +18,12 @@ > > #include "test_util.h" > > #include "perf_test_util.h" > > #include "guest_modes.h" > > +#ifdef __aarch64__ > > +#include "aarch64/vgic.h" > > + > > +#define GICD_BASE_GPA 0x8000000ULL > > +#define GICR_BASE_GPA 0x80A0000ULL > > +#endif > > > > /* How many host loops to run by default (one KVM_GET_DIRTY_LOG for each loop)*/ > > #define TEST_HOST_LOOP_N 2UL > > @@ -200,6 +206,10 @@ static void run_test(enum vm_guest_mode mode, void *arg) > > vm_enable_cap(vm, &cap); > > } > > > > +#ifdef __aarch64__ > > + vgic_v3_setup(vm, nr_vcpus, 64, GICD_BASE_GPA, GICR_BASE_GPA); > ^^ extra parameter The patch is based on kvm/queue, which has a patch adding an extra parameter nr_irqs. > > Thanks, > drew > > > +#endif > > + > > /* Start the iterations */ > > iteration = 0; > > host_quit = false; > > -- > > 2.34.1.575.g55b058a8bb-goog > > > > _______________________________________________ > > kvmarm mailing list > > kvmarm@xxxxxxxxxxxxxxxxxxxxx > > https://lists.cs.columbia.edu/mailman/listinfo/kvmarm > > > Thanks, Jing
