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

[Stefano Stabellini <sstabellini@xxxxxxxxxx>]

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 <marmarek@xxxxxxxxxxxxxxxxxxxxxx>

Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>


> ---
> 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
>