qemu-ppc
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 2/3] tests/qtest/migration-test: Enable on ppc64


From: Thomas Huth
Subject: Re: [PATCH 2/3] tests/qtest/migration-test: Enable on ppc64
Date: Mon, 27 May 2024 09:35:10 +0200
User-agent: Mozilla Thunderbird

On 25/05/2024 05.13, Nicholas Piggin wrote:
ppc64 with TCG seems to no longer be failing this test. Let's try to
enable it. s390x is still hanging about 1 in 10 runs.

Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
---
  tests/qtest/migration-test.c | 16 +++-------------
  1 file changed, 3 insertions(+), 13 deletions(-)

diff --git a/tests/qtest/migration-test.c b/tests/qtest/migration-test.c
index c13535c37d..b8617cc843 100644
--- a/tests/qtest/migration-test.c
+++ b/tests/qtest/migration-test.c
@@ -3454,19 +3454,9 @@ int main(int argc, char **argv)
  #endif
/*
-     * On ppc64, the test only works with kvm-hv, but not with kvm-pr and TCG
-     * is touchy due to race conditions on dirty bits (especially on PPC for
-     * some reason)
-     */
-    if (g_str_equal(arch, "ppc64") &&
-        (!has_kvm || access("/sys/module/kvm_hv", F_OK))) {
-        g_test_message("Skipping tests: kvm_hv not available");
-        goto test_add_done;
-    }
-
-    /*
-     * Similar to ppc64, s390x seems to be touchy with TCG, so disable it
-     * there until the problems are resolved
+     * On s390x, the test seems to be touchy with TCG, perhaps due to race
+     * conditions on dirty bits, so disable it there until the problems are
+     * resolved.
       */
      if (g_str_equal(arch, "s390x") && !has_kvm) {
          g_test_message("Skipping tests: s390x host with KVM is required");

Since you've identified the problem on s390x, you could maybe adjust the comment in case you respin ... OTOH, it will get removed anyway once we merge the s390x fix, so no need to respin just because of this.

Reviewed-by: Thomas Huth <thuth@redhat.com>




reply via email to

[Prev in Thread] Current Thread [Next in Thread]