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.abbar...@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.si...@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
signature.asc
Description: PGP signature
