public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
* [kvm-unit-tests PATCH] Bump Fedora version to 40
@ 2024-06-07  6:51 Thomas Huth
  2024-06-14  0:49 ` Nicholas Piggin
  0 siblings, 1 reply; 2+ messages in thread
From: Thomas Huth @ 2024-06-07  6:51 UTC (permalink / raw)
  To: kvm, Paolo Bonzini
  Cc: Andrew Jones, Nicholas Piggin, Janosch Frank, Claudio Imbrenda,
	Nico Böhr

Version 37 is out of date, let's use a supported one instead.

The "xsave" test now tries to execute more tests with TCG, but one
of them is failing with the QEMU from Fedora 40, so we have to
disable the xsave test for now.
The problem will be fixed in QEMU 9.1 with this patch here:
https://lore.kernel.org/qemu-devel/20240603100405.619418-1-pbonzini@redhat.com/

And there is also an additional problem with the "realmode" test. As
diagnosed by Paolo: "It turns out that with some versions of clang,
realmode.flat has become big enough that it overlaps the stack used
by the multiboot option ROM loader. The result is that a couple
instructions are overwritten." Thus disable the realmode test in the
Clang build until the problem is fixed.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 .gitlab-ci.yml | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 2af47328..b689a0c9 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -1,4 +1,4 @@
-image: fedora:37
+image: fedora:40
 
 before_script:
  - dnf update -y
@@ -216,7 +216,6 @@ build-x86_64:
       vmexit_ple_round_robin
       vmexit_tscdeadline
       vmexit_tscdeadline_immed
-      xsave
       | tee results.txt
  - if grep -q FAIL results.txt ; then exit 1 ; fi
 
@@ -268,7 +267,6 @@ build-clang:
       pks
       pku
       rdpru
-      realmode
       rmap_chain
       setjmp
       sieve
@@ -289,7 +287,6 @@ build-clang:
       vmexit_ple_round_robin
       vmexit_tscdeadline
       vmexit_tscdeadline_immed
-      xsave
       | tee results.txt
  - grep -q PASS results.txt && ! grep -q FAIL results.txt
 
-- 
2.45.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [kvm-unit-tests PATCH] Bump Fedora version to 40
  2024-06-07  6:51 [kvm-unit-tests PATCH] Bump Fedora version to 40 Thomas Huth
@ 2024-06-14  0:49 ` Nicholas Piggin
  0 siblings, 0 replies; 2+ messages in thread
From: Nicholas Piggin @ 2024-06-14  0:49 UTC (permalink / raw)
  To: Thomas Huth, kvm, Paolo Bonzini
  Cc: Andrew Jones, Janosch Frank, Claudio Imbrenda, Nico Böhr

On Fri Jun 7, 2024 at 4:51 PM AEST, Thomas Huth wrote:
> Version 37 is out of date, let's use a supported one instead.
>
> The "xsave" test now tries to execute more tests with TCG, but one
> of them is failing with the QEMU from Fedora 40, so we have to
> disable the xsave test for now.
> The problem will be fixed in QEMU 9.1 with this patch here:
> https://lore.kernel.org/qemu-devel/20240603100405.619418-1-pbonzini@redhat.com/

Could make them kfail and keep it running.

But anyway this works for me.

Acked-by: Nicholas Piggin <npiggin@gmail.com>

> And there is also an additional problem with the "realmode" test. As
> diagnosed by Paolo: "It turns out that with some versions of clang,
> realmode.flat has become big enough that it overlaps the stack used
> by the multiboot option ROM loader. The result is that a couple
> instructions are overwritten." Thus disable the realmode test in the
> Clang build until the problem is fixed.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  .gitlab-ci.yml | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> index 2af47328..b689a0c9 100644
> --- a/.gitlab-ci.yml
> +++ b/.gitlab-ci.yml
> @@ -1,4 +1,4 @@
> -image: fedora:37
> +image: fedora:40
>  
>  before_script:
>   - dnf update -y
> @@ -216,7 +216,6 @@ build-x86_64:
>        vmexit_ple_round_robin
>        vmexit_tscdeadline
>        vmexit_tscdeadline_immed
> -      xsave
>        | tee results.txt
>   - if grep -q FAIL results.txt ; then exit 1 ; fi
>  
> @@ -268,7 +267,6 @@ build-clang:
>        pks
>        pku
>        rdpru
> -      realmode
>        rmap_chain
>        setjmp
>        sieve
> @@ -289,7 +287,6 @@ build-clang:
>        vmexit_ple_round_robin
>        vmexit_tscdeadline
>        vmexit_tscdeadline_immed
> -      xsave
>        | tee results.txt
>   - grep -q PASS results.txt && ! grep -q FAIL results.txt
>  


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2024-06-14  0:49 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-06-07  6:51 [kvm-unit-tests PATCH] Bump Fedora version to 40 Thomas Huth
2024-06-14  0:49 ` Nicholas Piggin

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox