[u-boot-test-hooks][PATCH] travis-ci: Skip the saveenv and reset test cases on xilinx zynq qemu

[Tom Rini]

On Mon, Sep 02, 2024 at 12:11:22PM +0200, Michal Simek wrote:
> Hi Tom,
> 
> On 9/2/24 11:53, Venkatesh Yadav Abbarapu wrote:
> > There is an issue with the xilinx zynq qemu qspi driver during the erase
> > operation in dual parallel configuration, so skip these
> > tests for now.
> > 
> > Signed-off-by: Venkatesh Yadav Abbarapu <venkatesh.abbarapu at amd.com>
> > ---
> >   py/travis-ci/u_boot_boardenv_xilinx_zynq_virt_qemu.py | 2 ++
> >   py/travis-ci/u_boot_boardenv_zynq_zc702_qemu.py       | 2 ++
> >   2 files changed, 4 insertions(+)
> > 
> > diff --git a/py/travis-ci/u_boot_boardenv_xilinx_zynq_virt_qemu.py b/py/travis-ci/u_boot_boardenv_xilinx_zynq_virt_qemu.py
> > index 267285f..42685ad 100644
> > --- a/py/travis-ci/u_boot_boardenv_xilinx_zynq_virt_qemu.py
> > +++ b/py/travis-ci/u_boot_boardenv_xilinx_zynq_virt_qemu.py
> > @@ -1 +1,3 @@
> >   env__spl_skipped = True
> > +env__saveenv_test_skip = True
> > +env__reset_test_skip = True
> > diff --git a/py/travis-ci/u_boot_boardenv_zynq_zc702_qemu.py b/py/travis-ci/u_boot_boardenv_zynq_zc702_qemu.py
> > index 267285f..42685ad 100644
> > --- a/py/travis-ci/u_boot_boardenv_zynq_zc702_qemu.py
> > +++ b/py/travis-ci/u_boot_boardenv_zynq_zc702_qemu.py
> > @@ -1 +1,3 @@
> >   env__spl_skipped = True
> > +env__saveenv_test_skip = True
> > +env__reset_test_skip = True
> 
> 
> CI found bug in Qemu which is reported here.
> https://source.denx.de/u-boot/custodians/u-boot-microblaze/-/jobs/878781
> 
> as
> 
> ----------------------------- Captured stdout call -----------------------------
> Saving Environment to SPIFlash... Erasing SPI flash...qemu-system-arm:
> ../util/fifo8.c:62: fifo8_pop: Assertion `fifo->num > 0' failed.
> 
> It is bug in Qemu and we have already asked our Qemu team to fix this but
> hard to say when fix reach upstream and then there is CI pipeline update
> too.
> That's why the most sensible thing to do is disable these tests to be able
> to pass CI for dual stacked/parallel configurations.
> 
> Acked-by: Michal Simek <michal.simek at amd.com>

Thanks for explaining, I'm sure once QEMU is fixed and we've updated our
Dockerfile to have it someone will send a revert for this change.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20240902/e767d958/attachment.sig>