diff mbox series

[1/4] travis: Remove qemu-riscv64 testing

Message ID 1585238724-2928-1-git-send-email-bmeng.cn@gmail.com
State New
Headers show
Series [1/4] travis: Remove qemu-riscv64 testing | expand

Commit Message

Bin Meng March 26, 2020, 4:05 p.m. UTC
As of today there is no pre-built UEFI GRUB image for RISC-V 32/64
available on the internet, and with travis-ci we don't build GRUB
images like we do for azure and gitlab.

Remove qemu-riscv64 testing temporarily.

Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
---

 .travis.yml | 7 -------
 1 file changed, 7 deletions(-)

Comments

Tom Rini March 26, 2020, 4:10 p.m. UTC | #1
On Thu, Mar 26, 2020 at 09:05:21AM -0700, Bin Meng wrote:

> As of today there is no pre-built UEFI GRUB image for RISC-V 32/64
> available on the internet, and with travis-ci we don't build GRUB
> images like we do for azure and gitlab.
> 
> Remove qemu-riscv64 testing temporarily.
> 
> Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
> ---
> 
>  .travis.yml | 7 -------
>  1 file changed, 7 deletions(-)
> 
> diff --git a/.travis.yml b/.travis.yml
> index c59bd77..55b94cf 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -481,13 +481,6 @@ matrix:
>            QEMU_TARGET="ppc-softmmu"
>            BUILDMAN="^qemu-ppce500$"
>            TOOLCHAIN="powerpc"
> -    - name: "test/py qemu-riscv64"
> -      env:
> -        - TEST_PY_BD="qemu-riscv64"
> -          TEST_PY_TEST_SPEC="not sleep"
> -          QEMU_TARGET="riscv64-softmmu"
> -          BUILDMAN="^qemu-riscv64$"
> -          TOOLCHAIN="riscv"
>      - name: "test/py qemu-x86"
>        env:
>          - TEST_PY_BD="qemu-x86"

But why are we removing this?  We run all of the rest of the test suites
and I don't think this one test adds noticeably to the overall Travis
time.  Thanks!
Bin Meng March 27, 2020, 1:51 a.m. UTC | #2
Hi Tom,

On Fri, Mar 27, 2020 at 12:10 AM Tom Rini <trini at konsulko.com> wrote:
>
> On Thu, Mar 26, 2020 at 09:05:21AM -0700, Bin Meng wrote:
>
> > As of today there is no pre-built UEFI GRUB image for RISC-V 32/64
> > available on the internet, and with travis-ci we don't build GRUB
> > images like we do for azure and gitlab.
> >
> > Remove qemu-riscv64 testing temporarily.
> >
> > Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
> > ---
> >
> >  .travis.yml | 7 -------
> >  1 file changed, 7 deletions(-)
> >
> > diff --git a/.travis.yml b/.travis.yml
> > index c59bd77..55b94cf 100644
> > --- a/.travis.yml
> > +++ b/.travis.yml
> > @@ -481,13 +481,6 @@ matrix:
> >            QEMU_TARGET="ppc-softmmu"
> >            BUILDMAN="^qemu-ppce500$"
> >            TOOLCHAIN="powerpc"
> > -    - name: "test/py qemu-riscv64"
> > -      env:
> > -        - TEST_PY_BD="qemu-riscv64"
> > -          TEST_PY_TEST_SPEC="not sleep"
> > -          QEMU_TARGET="riscv64-softmmu"
> > -          BUILDMAN="^qemu-riscv64$"
> > -          TOOLCHAIN="riscv"
> >      - name: "test/py qemu-x86"
> >        env:
> >          - TEST_PY_BD="qemu-x86"
>
> But why are we removing this?  We run all of the rest of the test suites
> and I don't think this one test adds noticeably to the overall Travis
> time.  Thanks!

The reason was mentioned in the commit message.

As of today there is no pre-built UEFI GRUB image for RISC-V 32/64
available on the internet, and with travis-ci we don't build GRUB
images like we do for azure and gitlab.

Do you want to build GRUB image in travis, like what's done in the
docker image used for azure/gitlab?

Regards,
Bin
Tom Rini March 27, 2020, 2:01 a.m. UTC | #3
On Fri, Mar 27, 2020 at 09:51:03AM +0800, Bin Meng wrote:
> Hi Tom,
> 
> On Fri, Mar 27, 2020 at 12:10 AM Tom Rini <trini at konsulko.com> wrote:
> >
> > On Thu, Mar 26, 2020 at 09:05:21AM -0700, Bin Meng wrote:
> >
> > > As of today there is no pre-built UEFI GRUB image for RISC-V 32/64
> > > available on the internet, and with travis-ci we don't build GRUB
> > > images like we do for azure and gitlab.
> > >
> > > Remove qemu-riscv64 testing temporarily.
> > >
> > > Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
> > > ---
> > >
> > >  .travis.yml | 7 -------
> > >  1 file changed, 7 deletions(-)
> > >
> > > diff --git a/.travis.yml b/.travis.yml
> > > index c59bd77..55b94cf 100644
> > > --- a/.travis.yml
> > > +++ b/.travis.yml
> > > @@ -481,13 +481,6 @@ matrix:
> > >            QEMU_TARGET="ppc-softmmu"
> > >            BUILDMAN="^qemu-ppce500$"
> > >            TOOLCHAIN="powerpc"
> > > -    - name: "test/py qemu-riscv64"
> > > -      env:
> > > -        - TEST_PY_BD="qemu-riscv64"
> > > -          TEST_PY_TEST_SPEC="not sleep"
> > > -          QEMU_TARGET="riscv64-softmmu"
> > > -          BUILDMAN="^qemu-riscv64$"
> > > -          TOOLCHAIN="riscv"
> > >      - name: "test/py qemu-x86"
> > >        env:
> > >          - TEST_PY_BD="qemu-x86"
> >
> > But why are we removing this?  We run all of the rest of the test suites
> > and I don't think this one test adds noticeably to the overall Travis
> > time.  Thanks!
> 
> The reason was mentioned in the commit message.
> 
> As of today there is no pre-built UEFI GRUB image for RISC-V 32/64
> available on the internet, and with travis-ci we don't build GRUB
> images like we do for azure and gitlab.
> 
> Do you want to build GRUB image in travis, like what's done in the
> docker image used for azure/gitlab?

But this isn't just for testing GRUB via UEFI, it runs our test/py suite
too.  And that's worth doing, right?
Bin Meng March 27, 2020, 2:19 a.m. UTC | #4
Hi Tom,

On Fri, Mar 27, 2020 at 10:01 AM Tom Rini <trini at konsulko.com> wrote:
>
> On Fri, Mar 27, 2020 at 09:51:03AM +0800, Bin Meng wrote:
> > Hi Tom,
> >
> > On Fri, Mar 27, 2020 at 12:10 AM Tom Rini <trini at konsulko.com> wrote:
> > >
> > > On Thu, Mar 26, 2020 at 09:05:21AM -0700, Bin Meng wrote:
> > >
> > > > As of today there is no pre-built UEFI GRUB image for RISC-V 32/64
> > > > available on the internet, and with travis-ci we don't build GRUB
> > > > images like we do for azure and gitlab.
> > > >
> > > > Remove qemu-riscv64 testing temporarily.
> > > >
> > > > Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
> > > > ---
> > > >
> > > >  .travis.yml | 7 -------
> > > >  1 file changed, 7 deletions(-)
> > > >
> > > > diff --git a/.travis.yml b/.travis.yml
> > > > index c59bd77..55b94cf 100644
> > > > --- a/.travis.yml
> > > > +++ b/.travis.yml
> > > > @@ -481,13 +481,6 @@ matrix:
> > > >            QEMU_TARGET="ppc-softmmu"
> > > >            BUILDMAN="^qemu-ppce500$"
> > > >            TOOLCHAIN="powerpc"
> > > > -    - name: "test/py qemu-riscv64"
> > > > -      env:
> > > > -        - TEST_PY_BD="qemu-riscv64"
> > > > -          TEST_PY_TEST_SPEC="not sleep"
> > > > -          QEMU_TARGET="riscv64-softmmu"
> > > > -          BUILDMAN="^qemu-riscv64$"
> > > > -          TOOLCHAIN="riscv"
> > > >      - name: "test/py qemu-x86"
> > > >        env:
> > > >          - TEST_PY_BD="qemu-x86"
> > >
> > > But why are we removing this?  We run all of the rest of the test suites
> > > and I don't think this one test adds noticeably to the overall Travis
> > > time.  Thanks!
> >
> > The reason was mentioned in the commit message.
> >
> > As of today there is no pre-built UEFI GRUB image for RISC-V 32/64
> > available on the internet, and with travis-ci we don't build GRUB
> > images like we do for azure and gitlab.
> >
> > Do you want to build GRUB image in travis, like what's done in the
> > docker image used for azure/gitlab?
>
> But this isn't just for testing GRUB via UEFI, it runs our test/py suite
> too.  And that's worth doing, right?

Yes, and this testing is covered in azure/gitlab. The issue is that
whether we need build GRUB in travis. If yes, I can update the travis
scripts to sync with azure/gitlab.

Regards,
Bin
Tom Rini March 27, 2020, 2:20 a.m. UTC | #5
On Fri, Mar 27, 2020 at 10:19:05AM +0800, Bin Meng wrote:
> Hi Tom,
> 
> On Fri, Mar 27, 2020 at 10:01 AM Tom Rini <trini at konsulko.com> wrote:
> >
> > On Fri, Mar 27, 2020 at 09:51:03AM +0800, Bin Meng wrote:
> > > Hi Tom,
> > >
> > > On Fri, Mar 27, 2020 at 12:10 AM Tom Rini <trini at konsulko.com> wrote:
> > > >
> > > > On Thu, Mar 26, 2020 at 09:05:21AM -0700, Bin Meng wrote:
> > > >
> > > > > As of today there is no pre-built UEFI GRUB image for RISC-V 32/64
> > > > > available on the internet, and with travis-ci we don't build GRUB
> > > > > images like we do for azure and gitlab.
> > > > >
> > > > > Remove qemu-riscv64 testing temporarily.
> > > > >
> > > > > Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
> > > > > ---
> > > > >
> > > > >  .travis.yml | 7 -------
> > > > >  1 file changed, 7 deletions(-)
> > > > >
> > > > > diff --git a/.travis.yml b/.travis.yml
> > > > > index c59bd77..55b94cf 100644
> > > > > --- a/.travis.yml
> > > > > +++ b/.travis.yml
> > > > > @@ -481,13 +481,6 @@ matrix:
> > > > >            QEMU_TARGET="ppc-softmmu"
> > > > >            BUILDMAN="^qemu-ppce500$"
> > > > >            TOOLCHAIN="powerpc"
> > > > > -    - name: "test/py qemu-riscv64"
> > > > > -      env:
> > > > > -        - TEST_PY_BD="qemu-riscv64"
> > > > > -          TEST_PY_TEST_SPEC="not sleep"
> > > > > -          QEMU_TARGET="riscv64-softmmu"
> > > > > -          BUILDMAN="^qemu-riscv64$"
> > > > > -          TOOLCHAIN="riscv"
> > > > >      - name: "test/py qemu-x86"
> > > > >        env:
> > > > >          - TEST_PY_BD="qemu-x86"
> > > >
> > > > But why are we removing this?  We run all of the rest of the test suites
> > > > and I don't think this one test adds noticeably to the overall Travis
> > > > time.  Thanks!
> > >
> > > The reason was mentioned in the commit message.
> > >
> > > As of today there is no pre-built UEFI GRUB image for RISC-V 32/64
> > > available on the internet, and with travis-ci we don't build GRUB
> > > images like we do for azure and gitlab.
> > >
> > > Do you want to build GRUB image in travis, like what's done in the
> > > docker image used for azure/gitlab?
> >
> > But this isn't just for testing GRUB via UEFI, it runs our test/py suite
> > too.  And that's worth doing, right?
> 
> Yes, and this testing is covered in azure/gitlab. The issue is that
> whether we need build GRUB in travis. If yes, I can update the travis
> scripts to sync with azure/gitlab.

So you're trying to keep them all consistent in terms of what's covered.
I'm not sure that's viable and we have other cases where I believe some
tests aren't run in Travis but only gitlab such as the filesystem tests.
David Abdurachmanov March 27, 2020, 5:19 a.m. UTC | #6
On Thu, Mar 26, 2020 at 6:06 PM Bin Meng <bmeng.cn at gmail.com> wrote:
>
> As of today there is no pre-built UEFI GRUB image for RISC-V 32/64
> available on the internet, and with travis-ci we don't build GRUB
> images like we do for azure and gitlab.

There are RPM packages for GRUB in OpenSUSE and Fedora.

https://build.opensuse.org/package/binaries/openSUSE:Factory:RISCV/grub2/standard
http://fedora.riscv.rocks/koji/buildinfo?buildID=133151

I have not tried them on QEMU or Unleashed yet.

We should be able to boot with GRUB2 and EFI stubs to the kernel soonish.

>
> Remove qemu-riscv64 testing temporarily.
>
> Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
> ---
>
>  .travis.yml | 7 -------
>  1 file changed, 7 deletions(-)
>
> diff --git a/.travis.yml b/.travis.yml
> index c59bd77..55b94cf 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -481,13 +481,6 @@ matrix:
>            QEMU_TARGET="ppc-softmmu"
>            BUILDMAN="^qemu-ppce500$"
>            TOOLCHAIN="powerpc"
> -    - name: "test/py qemu-riscv64"
> -      env:
> -        - TEST_PY_BD="qemu-riscv64"
> -          TEST_PY_TEST_SPEC="not sleep"
> -          QEMU_TARGET="riscv64-softmmu"
> -          BUILDMAN="^qemu-riscv64$"
> -          TOOLCHAIN="riscv"
>      - name: "test/py qemu-x86"
>        env:
>          - TEST_PY_BD="qemu-x86"
> --
> 2.7.4
>
Bin Meng March 27, 2020, 5:31 a.m. UTC | #7
On Fri, Mar 27, 2020 at 1:20 PM David Abdurachmanov
<david.abdurachmanov at gmail.com> wrote:
>
> On Thu, Mar 26, 2020 at 6:06 PM Bin Meng <bmeng.cn at gmail.com> wrote:
> >
> > As of today there is no pre-built UEFI GRUB image for RISC-V 32/64
> > available on the internet, and with travis-ci we don't build GRUB
> > images like we do for azure and gitlab.
>
> There are RPM packages for GRUB in OpenSUSE and Fedora.
>
> https://build.opensuse.org/package/binaries/openSUSE:Factory:RISCV/grub2/standard
> http://fedora.riscv.rocks/koji/buildinfo?buildID=133151
>

I've looked at this alternatives before, it turns out there are only
riscv64 images. Do you know any riscv32 prebuilt images?

> I have not tried them on QEMU or Unleashed yet.
>
> We should be able to boot with GRUB2 and EFI stubs to the kernel soonish.

Regards,
Bin
David Abdurachmanov March 27, 2020, 5:39 a.m. UTC | #8
On Fri, Mar 27, 2020 at 7:31 AM Bin Meng <bmeng.cn at gmail.com> wrote:
>
> On Fri, Mar 27, 2020 at 1:20 PM David Abdurachmanov
> <david.abdurachmanov at gmail.com> wrote:
> >
> > On Thu, Mar 26, 2020 at 6:06 PM Bin Meng <bmeng.cn at gmail.com> wrote:
> > >
> > > As of today there is no pre-built UEFI GRUB image for RISC-V 32/64
> > > available on the internet, and with travis-ci we don't build GRUB
> > > images like we do for azure and gitlab.
> >
> > There are RPM packages for GRUB in OpenSUSE and Fedora.
> >
> > https://build.opensuse.org/package/binaries/openSUSE:Factory:RISCV/grub2/standard
> > http://fedora.riscv.rocks/koji/buildinfo?buildID=133151
> >
>
> I've looked at this alternatives before, it turns out there are only
> riscv64 images. Do you know any riscv32 prebuilt images?

I am not aware of any distribution supporting RV32 (or plans to
do it). Furthermore RV32 Linux ABI is not frozen yet. OE definitely
has RV32 Linux support, but it doesn't incl. RV32 GRUB2 for booting.

Simply put, I don't think there are RV32 GRUB2 binaries somewhere
online.

>
> > I have not tried them on QEMU or Unleashed yet.
> >
> > We should be able to boot with GRUB2 and EFI stubs to the kernel soonish.
>
> Regards,
> Bin
diff mbox series

Patch

diff --git a/.travis.yml b/.travis.yml
index c59bd77..55b94cf 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -481,13 +481,6 @@  matrix:
           QEMU_TARGET="ppc-softmmu"
           BUILDMAN="^qemu-ppce500$"
           TOOLCHAIN="powerpc"
-    - name: "test/py qemu-riscv64"
-      env:
-        - TEST_PY_BD="qemu-riscv64"
-          TEST_PY_TEST_SPEC="not sleep"
-          QEMU_TARGET="riscv64-softmmu"
-          BUILDMAN="^qemu-riscv64$"
-          TOOLCHAIN="riscv"
     - name: "test/py qemu-x86"
       env:
         - TEST_PY_BD="qemu-x86"