Re: [XEN PATCH v1 1/4] arch/riscv: initial RISC-V support to build/run minimal Xen

On Wed, 2022-12-28 at 19:01 +, Andrew Cooper wrote: > On 28/12/2022 4:51 am, Alistair Francis wrote: > > On Mon, Dec 26, 2022 at 9:14 PM Oleksii > > wrote: > > > On Fri, 2022-12-23 at 13:50 +, Julien Grall wrote: > > > > On 23/12/2022 11:16, Oleksii Kurochko wrote: > > > > > + . = ALIGN(P

Re: [XEN PATCH v1 2/4] automation: add cross-compiler support for the build script

On Wed, 2022-12-28 at 23:24 +, Andrew Cooper wrote: > On 23/12/2022 11:16 am, Oleksii Kurochko wrote: > > diff --git a/automation/scripts/build b/automation/scripts/build > > index a593419063..026f480e76 100755 > > --- a/automation/scripts/build > > +++ b/automation/scripts/build > > @@ -2,12 +

Re: [XEN PATCH v1 1/4] arch/riscv: initial RISC-V support to build/run minimal Xen

On Wed, 2022-12-28 at 22:22 +, Julien Grall wrote: > > > On Mon, 26 Dec 2022 at 12:14, Oleksii > wrote: > > > > > > > +/* > > > > + * PAGE_SIZE is defined in <{asm,xen}/page-*.h> but to > > > > + * remove unnecessary headers for minimal > > > > + * build headers it will be better to set PAG

Re: [XEN PATCH v1 1/4] arch/riscv: initial RISC-V support to build/run minimal Xen

On Wed, 2022-12-28 at 18:56 +, Andrew Cooper wrote: > On 23/12/2022 11:16 am, Oleksii Kurochko wrote: > > diff --git a/xen/arch/riscv/Makefile b/xen/arch/riscv/Makefile > > index 942e4ffbc1..893dd19ea6 100644 > > --- a/xen/arch/riscv/Makefile > > +++ b/xen/arch/riscv/Makefile > > @@ -1,2 +1,32

[linux-linus test] 175518: regressions - FAIL

flight 175518 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/175518/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-qemuu-nested-intel 8 xen-boot fail REGR. vs. 173462 test-amd64-amd64-xl

Re: [PATCH 4/6] util/error: add G_GNUC_PRINTF for various functions

On 19/12/2022 14.02, Daniel P. Berrangé wrote: Signed-off-by: Daniel P. Berrangé --- util/error-report.c | 1 + util/error.c| 1 + 2 files changed, 2 insertions(+) diff --git a/util/error-report.c b/util/error-report.c index 5edb2e6040..6e44a55732 100644 --- a/util/error-report.c ++

Re: [PATCH 5/6] tests: add G_GNUC_PRINTF for various functions

On 19/12/2022 14.02, Daniel P. Berrangé wrote: Signed-off-by: Daniel P. Berrangé --- tests/qtest/ahci-test.c | 3 +++ tests/qtest/arm-cpu-features.c| 1 + tests/qtest/erst-test.c | 2 +- tests/qtest/ide-test.c| 3 ++- tests/qtest/ivshmem-test.c

[linux-5.4 test] 175519: regressions - FAIL

flight 175519 linux-5.4 real [real] http://logs.test-lab.xenproject.org/osstest/logs/175519/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-credit2 broken in 175407 test-amd64-amd64-xl-qemut-debianh

Re: [XEN PATCH v1 3/4] automation: add python3 package for riscv64.dockerfile

On 29/12/2022 7:45 am, Oleksii wrote: > On Wed, 2022-12-28 at 23:26 +, Andrew Cooper wrote: >> On 23/12/2022 11:16 am, Oleksii Kurochko wrote: >>> Pyhton3 package is requited by automation/scripts/build >>> script so it shoud be installed to riscv64 docker image >> Is it?  This series runs fine

Re: [XEN PATCH v1 2/4] automation: add cross-compiler support for the build script

On Thu, 2022-12-29 at 10:13 +0200, Oleksii wrote: > On Wed, 2022-12-28 at 23:24 +, Andrew Cooper wrote: > > On 23/12/2022 11:16 am, Oleksii Kurochko wrote: > > > diff --git a/automation/scripts/build b/automation/scripts/build > > > index a593419063..026f480e76 100755 > > > --- a/automation/scr

[xen-unstable test] 175520: tolerable FAIL

flight 175520 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/175520/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-amd64-i386-xl-qemuu-dmrestrict-amd64-dmrestrict 7 xen-install fail pass in 175517 test-armhf-armhf-xl-arn

[linux-linus test] 175521: regressions - FAIL

flight 175521 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/175521/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-qemuu-nested-intel 8 xen-boot fail REGR. vs. 173462 test-amd64-amd64-xl

[linux-5.4 test] 175522: regressions - FAIL

flight 175522 linux-5.4 real [real] http://logs.test-lab.xenproject.org/osstest/logs/175522/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-credit2 broken in 175407 test-amd64-i386-qemuu-rhel6hvm-in

[PATCH 3/6] CI: Only calculate ./configure args if needed

This is purely code motion of the cfgargs construction, into the case where it is used. No practical change. Signed-off-by: Andrew Cooper --- CC: Doug Goldstein CC: Stefano Stabellini CC: Anthony PERARD CC: Michal Orzel CC: Oleksii Kurochko --- automation/scripts/build | 63 +++

[PATCH 5/6] CI: Fix build script when CROSS_COMPILE is in use

Some testcases use a cross compiler. Presently it's only arm32 and due to previous cleanup the only thing which is now wrong is printing the compiler version at the start of day. Construct $cc to match what `make` will eventually choose given CROSS_COMPILE, taking care not to modify $CC. Use $cc

[PATCH 0/6] CI: Fixes/cleanup in preparation for RISCV

This series is intended to avoid the need for any editing of automation/scripts/build in the main RISCV series. https://gitlab.com/xen-project/people/andyhhp/xen/-/pipelines/734584203 https://gitlab.com/xen-project/people/andyhhp/xen/-/commits/auto-build The two Ubuntu Bionic failures are known a

[PATCH 4/6] CI: Express HYPERVISOR_ONLY in build.yml

Whether to build only Xen, or everything, is a property of container, toolchain and/or testcase. It is not a property of XEN_TARGET_ARCH. Capitalise HYPERVISOR_ONLY and have it set by the debian-unstable-gcc-arm32-* testcases at the point that arm32 get matched with a container that can only buil

[PATCH 2/6] CI: Remove guesswork about which artefacts to preserve

Preserve the artefacts based on the `make` rune we actually ran, rather than guesswork about which rune we would have run based on other settings. Note that the ARM qemu smoke tests depend on finding binaries/xen even from full builds. Also that the Jessie-32 containers build tools but not Xen.

[PATCH 6/6] CI: Simplify the MUSL check

There's no need to do ad-hoc string parsing. Use grep -q instead. No functional change. Signed-off-by: Andrew Cooper --- CC: Doug Goldstein CC: Stefano Stabellini CC: Anthony PERARD CC: Michal Orzel CC: Oleksii Kurochko --- automation/scripts/build | 2 +- 1 file changed, 1 insertion(+),

[PATCH 1/6] CI: Drop automation/configs/

Having 3 extra hypervisor builds on the end of a full build is deeply confusing to debug if one of them fails, because the .config file presented in the artefacts is not the one which caused a build failure. Also, the log tends to be truncated in the UI. PV-only is tested as part of PV-Shim in a

[linux-linus test] 175523: regressions - FAIL

flight 175523 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/175523/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-qemuu-nested-intel 8 xen-boot fail REGR. vs. 173462 test-amd64-amd64-xl

[linux-5.4 test] 175524: FAIL

flight 175524 linux-5.4 real [real] http://logs.test-lab.xenproject.org/osstest/logs/175524/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemut-debianhvm-amd64 broken in 175407 test-amd64-a

[ovmf test] 175527: all pass - PUSHED

flight 175527 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/175527/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf c5d68ef6e7553ab2894f541eba4e982428ecbd53 baseline version: ovmf a086f4a63bc0295b0b02f