On Fri, 2024-12-13 at 17:07 -0800, Sean Christopherson wrote: > Actually run all requested iterations, instead of iterations-1 (the count > starts at '1' due to the need to avoid '0' as an in-memory value for a > dirty page). > > Signed-off-by: Sean Christopherson <seanjc@xxxxxxxxxx> > --- > tools/testing/selftests/kvm/dirty_log_test.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/testing/selftests/kvm/dirty_log_test.c b/tools/testing/selftests/kvm/dirty_log_test.c > index f156459bf1ae..ccc5d9800bbf 100644 > --- a/tools/testing/selftests/kvm/dirty_log_test.c > +++ b/tools/testing/selftests/kvm/dirty_log_test.c > @@ -695,7 +695,7 @@ static void run_test(enum vm_guest_mode mode, void *arg) > > pthread_create(&vcpu_thread, NULL, vcpu_worker, vcpu); > > - for (iteration = 1; iteration < p->iterations; iteration++) { > + for (iteration = 1; iteration <= p->iterations; iteration++) { > unsigned long i; > > sync_global_to_guest(vm, iteration); Reviewed-by: Maxim Levitsky <mlevitsk@xxxxxxxxxx>