| Message ID | 20221201131623.18ebc8d8@canb.auug.org.au (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | linux-next: manual merge of the kvm-arm tree with the kvm tree | expand |
On Thu, 01 Dec 2022 02:16:23 +0000, Stephen Rothwell <sfr@canb.auug.org.au> wrote: > > Hi all, > > Today's linux-next merge of the kvm-arm tree got a conflict in: > > tools/testing/selftests/kvm/include/perf_test_util.h > > between commit: > > 9fda6753c9dd ("KVM: selftests: Rename perf_test_util.[ch] to memstress.[ch]") > > from the kvm tree and commit: > > 9ec1eb1bccee ("KVM: selftests: Have perf_test_util signal when to stop vCPUs") > > from the kvm-arm tree. > > I fixed it up (I removed the file and applied the following patch) and > can carry the fix as necessary. This is now fixed as far as linux-next > is concerned, but any non trivial conflicts should be mentioned to your > upstream maintainer when your tree is submitted for merging. You may > also want to consider cooperating with the maintainer of the conflicting > tree to minimise any particularly complex conflicts. > > (this patch covered another occurrence fo the needed rename) Huh, this is a lot of conflicts. Paolo, do you have a stable branch I can pull in my tree to kill those? Or do you want to deal with the conflicts yourself? M.
diff --git a/tools/testing/selftests/kvm/access_tracking_perf_test.c b/tools/testing/selftests/kvm/access_tracking_perf_test.c index 2c9e0abcac94..57a16371e9c2 100644 --- a/tools/testing/selftests/kvm/access_tracking_perf_test.c +++ b/tools/testing/selftests/kvm/access_tracking_perf_test.c @@ -208,7 +208,7 @@ static bool spin_wait_for_next_iteration(int *current_iteration) int last_iteration = *current_iteration; do { - if (READ_ONCE(perf_test_args.stop_vcpus)) + if (READ_ONCE(memstress_args.stop_vcpus)) return false; *current_iteration = READ_ONCE(iteration); diff --git a/tools/testing/selftests/kvm/demand_paging_test.c b/tools/testing/selftests/kvm/demand_paging_test.c index f980ed8aeead..b0e1fc4de9e2 100644 --- a/tools/testing/selftests/kvm/demand_paging_test.c +++ b/tools/testing/selftests/kvm/demand_paging_test.c @@ -161,7 +161,7 @@ static void run_test(enum vm_guest_mode mode, void *arg) vcpu_alias = addr_gpa2alias(vm, vcpu_args->gpa); prefault_mem(vcpu_alias, - vcpu_args->pages * perf_test_args.guest_page_size); + vcpu_args->pages * memstress_args.guest_page_size); /* * Set up user fault fd to handle demand paging diff --git a/tools/testing/selftests/kvm/include/memstress.h b/tools/testing/selftests/kvm/include/memstress.h index bbd2a302df10..4ae63963c14f 100644 --- a/tools/testing/selftests/kvm/include/memstress.h +++ b/tools/testing/selftests/kvm/include/memstress.h @@ -40,6 +40,8 @@ struct memstress_args { /* Run vCPUs in L2 instead of L1, if the architecture supports it. */ bool nested; + /* Test is done, stop running vCPUs. */ + bool stop_vcpus; /* Randomize which pages are accessed by the guest. */ bool random_access; /* True if all vCPUs are pinned to pCPUs */ diff --git a/tools/testing/selftests/kvm/lib/memstress.c b/tools/testing/selftests/kvm/lib/memstress.c index 3742ce238129..5f1d3173c238 100644 --- a/tools/testing/selftests/kvm/lib/memstress.c +++ b/tools/testing/selftests/kvm/lib/memstress.c @@ -292,7 +292,7 @@ void memstress_start_vcpu_threads(int nr_vcpus, vcpu_thread_fn = vcpu_fn; WRITE_ONCE(all_vcpu_threads_running, false); - WRITE_ONCE(perf_test_args.stop_vcpus, false); + WRITE_ONCE(memstress_args.stop_vcpus, false); for (i = 0; i < nr_vcpus; i++) { struct vcpu_thread *vcpu = &vcpu_threads[i]; @@ -315,7 +315,7 @@ void memstress_join_vcpu_threads(int nr_vcpus) { int i; - WRITE_ONCE(perf_test_args.stop_vcpus, true); + WRITE_ONCE(memstress_args.stop_vcpus, true); for (i = 0; i < nr_vcpus; i++) pthread_join(vcpu_threads[i].thread, NULL); diff --git a/tools/testing/selftests/kvm/memslot_modification_stress_test.c b/tools/testing/selftests/kvm/memslot_modification_stress_test.c index 431dbc47d345..9855c41ca811 100644 --- a/tools/testing/selftests/kvm/memslot_modification_stress_test.c +++ b/tools/testing/selftests/kvm/memslot_modification_stress_test.c @@ -43,7 +43,7 @@ static void vcpu_worker(struct memstress_vcpu_args *vcpu_args) run = vcpu->run; /* Let the guest access its memory until a stop signal is received */ - while (!READ_ONCE(perf_test_args.stop_vcpus)) { + while (!READ_ONCE(memstress_args.stop_vcpus)) { ret = _vcpu_run(vcpu); TEST_ASSERT(ret == 0, "vcpu_run failed: %d\n", ret);
Hi all, Today's linux-next merge of the kvm-arm tree got a conflict in: tools/testing/selftests/kvm/include/perf_test_util.h between commit: 9fda6753c9dd ("KVM: selftests: Rename perf_test_util.[ch] to memstress.[ch]") from the kvm tree and commit: 9ec1eb1bccee ("KVM: selftests: Have perf_test_util signal when to stop vCPUs") from the kvm-arm tree. I fixed it up (I removed the file and applied the following patch) and can carry the fix as necessary. This is now fixed as far as linux-next is concerned, but any non trivial conflicts should be mentioned to your upstream maintainer when your tree is submitted for merging. You may also want to consider cooperating with the maintainer of the conflicting tree to minimise any particularly complex conflicts. (this patch covered another occurrence fo the needed rename) From: Stephen Rothwell <sfr@canb.auug.org.au> Date: Thu, 1 Dec 2022 12:19:45 +1100 Subject: [PATCH] fix up for rename of perf_test_args to memstress_args Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au> --- tools/testing/selftests/kvm/access_tracking_perf_test.c | 2 +- tools/testing/selftests/kvm/demand_paging_test.c | 2 +- tools/testing/selftests/kvm/include/memstress.h | 2 ++ tools/testing/selftests/kvm/lib/memstress.c | 4 ++-- .../testing/selftests/kvm/memslot_modification_stress_test.c | 2 +- 5 files changed, 7 insertions(+), 5 deletions(-)