[u-boot-test-hooks][PATCH] travis-ci: Skip the saveenv and reset test cases on xilinx zynq qemu
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@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
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@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@amd.com
Thanks, Michal
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@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@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.
participants (3)
-
Michal Simek -
Tom Rini -
Venkatesh Yadav Abbarapu