Re: [PATCH 4/4] automation: add PV passthrough tests on a AMD Zen3+ runner

2023-05-15 Thread Stefano Stabellini
On Sat, 13 May 2023, Marek Marczykowski-Górecki wrote:
> The PV passthrough test currently fails on this system
> with:
> (d1) Can't find new memory area for initrd needed due to E820 map conflict
> 
> Setting e820_host=1 does not help. So, add this test with
> "allow_failure: true" until the problem is fixed.
> 
> Signed-off-by: Marek Marczykowski-Górecki 

Reviewed-by: Stefano Stabellini 


> ---
> I'm unsure if this should be included. On one hand, the test case will
> help verifying potential fix. But on the other hand, until fixed it will
> be wasting time.

I am not sure about this one either. I committed the other patches. I'll
give it a few days for others to comment on this one.


> ---
>  automation/gitlab-ci/test.yaml |  9 +
>  1 file changed, 9 insertions(+)
> 
> diff --git a/automation/gitlab-ci/test.yaml b/automation/gitlab-ci/test.yaml
> index 81d027532cca..7becb7a6b782 100644
> --- a/automation/gitlab-ci/test.yaml
> +++ b/automation/gitlab-ci/test.yaml
> @@ -194,6 +194,15 @@ zen3p-smoke-x86-64-gcc-debug:
>  - *x86-64-test-needs
>  - alpine-3.12-gcc-debug
>  
> +zen3p-pci-pv-x86-64-gcc-debug:
> +  extends: .zen3p-x86-64
> +  allow_failure: true
> +  script:
> +- ./automation/scripts/qubes-x86-64.sh pci-pv 2>&1 | tee ${LOGFILE}
> +  needs:
> +- *x86-64-test-needs
> +- alpine-3.12-gcc-debug
> +
>  zen3p-pci-hvm-x86-64-gcc-debug:
>extends: .zen3p-x86-64
>script:
> -- 
> git-series 0.9.1
> 

[PATCH 4/4] automation: add PV passthrough tests on a AMD Zen3+ runner

2023-05-12 Thread Marek Marczykowski-Górecki
The PV passthrough test currently fails on this system
with:
(d1) Can't find new memory area for initrd needed due to E820 map conflict

Setting e820_host=1 does not help. So, add this test with
"allow_failure: true" until the problem is fixed.

Signed-off-by: Marek Marczykowski-Górecki 
---
I'm unsure if this should be included. On one hand, the test case will
help verifying potential fix. But on the other hand, until fixed it will
be wasting time.
---
 automation/gitlab-ci/test.yaml |  9 +
 1 file changed, 9 insertions(+)

diff --git a/automation/gitlab-ci/test.yaml b/automation/gitlab-ci/test.yaml
index 81d027532cca..7becb7a6b782 100644
--- a/automation/gitlab-ci/test.yaml
+++ b/automation/gitlab-ci/test.yaml
@@ -194,6 +194,15 @@ zen3p-smoke-x86-64-gcc-debug:
 - *x86-64-test-needs
 - alpine-3.12-gcc-debug
 
+zen3p-pci-pv-x86-64-gcc-debug:
+  extends: .zen3p-x86-64
+  allow_failure: true
+  script:
+- ./automation/scripts/qubes-x86-64.sh pci-pv 2>&1 | tee ${LOGFILE}
+  needs:
+- *x86-64-test-needs
+- alpine-3.12-gcc-debug
+
 zen3p-pci-hvm-x86-64-gcc-debug:
   extends: .zen3p-x86-64
   script:
-- 
git-series 0.9.1