Re: [XEN PATCH v2 2/2] automation: add RISC-V 64 cross-build tests for Xen

[Oleksii <oleksii.kurochko@xxxxxxxxx>]

On Wed, 2023-01-04 at 20:39 +0000, Andrew Cooper wrote:
> On 30/12/2022 1:01 pm, Oleksii Kurochko wrote:
> > diff --git a/automation/gitlab-ci/build.yaml b/automation/gitlab-
> > ci/build.yaml
> > index e6a9357de3..11eb1c6b82 100644
> > --- a/automation/gitlab-ci/build.yaml
> > +++ b/automation/gitlab-ci/build.yaml
> > @@ -617,6 +644,21 @@ alpine-3.12-gcc-debug-arm64-boot-cpupools:
> >      EXTRA_XEN_CONFIG: |
> >        CONFIG_BOOT_TIME_CPUPOOLS=y
> >  
> > +# RISC-V 64 cross-build
> > +riscv64-cross-gcc:
> > +  extends: .gcc-riscv64-cross-build
> > +  variables:
> > +    CONTAINER: archlinux:riscv64
> > +    KBUILD_DEFCONFIG: tiny64_defconfig
> > +    HYPERVISOR_ONLY: y
> > +
> > +riscv64-cross-gcc-debug:
> > +  extends: .gcc-riscv64-cross-build-debug
> > +  variables:
> > +    CONTAINER: archlinux:riscv64
> > +    KBUILD_DEFCONFIG: tiny64_defconfig
> > +    HYPERVISOR_ONLY: y
> > +
> 
> Judging by the Kconfig which gets written out, I suggest inserting
> the
> two RANDCONFIG jobs right now.
> 
> >  ## Test artifacts common
> >  
> >  .test-jobs-artifact-common:
> > @@ -692,3 +734,6 @@ kernel-5.10.74-export:
> >        - binaries/bzImage
> >    tags:
> >      - x86_64
> > +
> > +# # RISC-V 64 test artificats
> > +# # TODO: add RISC-V 64 test artitifacts
> 
> Drop this hunk.  All you're going to be doing is deleting it in the
> next
> series...
> 

Thanks for the comments.
They were fixed in patch series v4:
[PATCH v4 2/2] automation: add RISC-V 64 cross-build tests for Xen

~ Oleksii
> ~Andrew