Re: [PATCH 1/2] gitlab: Introduce Loongarch64 runner

[gaosong]

在 2024/1/12 下午5:52, gaosong 写道:
> 在 2024/1/11 下午4:26, Thomas Huth 写道:
> > On 11/01/2024 08.25, gaosong wrote:
> > > Hi,
> > >
> > > 在 2024/1/11 下午3:08, Thomas Huth 写道:
> > > > On 02/01/2024 18.22, Philippe Mathieu-Daudé wrote:
> > > > > Full build config to run CI tests on a Loongarch64 host.
> > > > >
> > > > > Forks might enable this by setting LOONGARCH64_RUNNER_AVAILABLE
> > > > > in their CI namespace settings, see:
> > > > > https://www.qemu.org/docs/master/devel/ci.html#maintainer-controlled-job-variables 
> > > > >
> > > > >
> > > > > Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> > > > > ---
> > > > >   docs/devel/ci-jobs.rst.inc                    |  6 ++++++
> > > > >   .gitlab-ci.d/custom-runners.yml               |  1 +
> > > > >   .../openeuler-22.03-loongarch64.yml           | 21 
> > > > > +++++++++++++++++++
> > > > >   3 files changed, 28 insertions(+)
> > > > >   create mode 100644 
> > > > > .gitlab-ci.d/custom-runners/openeuler-22.03-loongarch64.yml
> > > > ...
> > > > > diff --git a/.gitlab-ci.d/custom-runners.yml 
> > > > > b/.gitlab-ci.d/custom-runners.yml
> > > > > index 8e5b9500f4..152ace4492 100644
> > > > > --- a/.gitlab-ci.d/custom-runners.yml
> > > > > +++ b/.gitlab-ci.d/custom-runners.yml
> > > > > @@ -32,3 +32,4 @@ include:
> > > > >     - local: '/.gitlab-ci.d/custom-runners/ubuntu-22.04-aarch64.yml'
> > > > >     - local: '/.gitlab-ci.d/custom-runners/ubuntu-22.04-aarch32.yml'
> > > > >     - local: 
> > > > > '/.gitlab-ci.d/custom-runners/centos-stream-8-x86_64.yml'
> > > > > +  - local: 
> > > > > '/.gitlab-ci.d/custom-runners/openeuler-22.03-loongarch64.yml'
> > > > > diff --git 
> > > > > a/.gitlab-ci.d/custom-runners/openeuler-22.03-loongarch64.yml 
> > > > > b/.gitlab-ci.d/custom-runners/openeuler-22.03-loongarch64.yml
> > > > > new file mode 100644
> > > > > index 0000000000..86d18f820e
> > > > > --- /dev/null
> > > > > +++ b/.gitlab-ci.d/custom-runners/openeuler-22.03-loongarch64.yml
> > > > > @@ -0,0 +1,21 @@
> > > > > +openeuler-22.03-loongarch64-all:
> > > > > + extends: .custom_runner_template :-)
> > > > > + needs: []
> > > > > + stage: build
> > > > > + tags:
> > > > > + - oe2203
> > > > > + - loongarch64
> > > > > + rules:
> > > > > + - if: '$CI_PROJECT_NAMESPACE == "qemu-project" && 
> > > > > $CI_COMMIT_BRANCH =~ /^staging/'
> > > > > +   when: manual
> > > > > +   allow_failure: true
> > > > > + - if: "$LOONGARCH64_RUNNER_AVAILABLE"
> > > > > +   when: manual
> > > > > +   allow_failure: true
> > > > > + script:
> > > > > + - mkdir build
> > > > > + - cd build
> > > > > + - ../configure
> > > > > +   || { cat config.log meson-logs/meson-log.txt; exit 1; }
> > > > > + - make --output-sync -j`nproc --ignore=40`
> > > > > + - make --output-sync -j`nproc --ignore=40` check
> > > >
> > > > Does this system really have more than 40 CPU threads? Or is this a 
> > > > copy-n-past from one of the other scripts? In the latter case, I'd 
> > > > suggest to adjust the --ignore=40 to a more reasonable value.
> > > >
> > > >  Thomas
> > > No,  only 32.   I think it should be --ignore=32 or 16.
> >
> > --ignore=32 then also does not make much sense, that would still be 
> > the same as simply omitting the -j parameter. I guess --ignore=16 
> > should be fine.
> >
> > > I create a same runner on this machine, and I  find  some check error.
> > > but I am not sure how to fix it. :-)
> > >
> > > See:
> > >
> > > https://gitlab.com/gaosong/qemu/-/jobs/5906269934
> >
> > Seems to be related to RAM backing... for example, the erst-test is 
> > failing, which is doing something like:
> >
> >     setup_vm_cmd(&state,
> >         "-object memory-backend-file,"
> >             "mem-path=acpi-erst.XXXXXX,"
> >             "size=64K,"
> Hi,
>
> We tested this  on
> LoongArch host with the 5.10 kernel,  (openEuler 22.03),
> x86_64 host with 5.10 kernel, (openEuler 22.03)
> x86_64 host with 5.15kernel , (Ubuntu 20.04.3 LTS)
>
> and didn't get any error.
>
> but the CI machine use the  6.7_rc4 kernel.
> we didn't update the x86_64 host kernel to tested this.
>
> Is it possible that the new kernel is causing the problem?
Hi,

The kernel adds the patch[1] can fix this problem.

[1] 
https://patchew.org/linux/20240106145501.3370364-1-chenhuacai@loongson.cn/

So
Tested-by: Song Gao <gaosong@loongson.cn>
Reviewed-by: Song Gao <gaosong@loongson.cn>

Thanks.
Song Gao

> > "share=on,"
> >             "id=nvram "
> >         "-device acpi-erst,"
> >             "memdev=nvram");
> >
> > So it seems like -object memory-backend-file" is not correctly 
> > working in your gitlab runner? Is there some setup missing?
> >
> >  Thomas