Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

esp32: mtd tests failing on CI #15123

Closed
kaspar030 opened this issue Sep 30, 2020 · 8 comments · Fixed by #15186
Closed

esp32: mtd tests failing on CI #15123

kaspar030 opened this issue Sep 30, 2020 · 8 comments · Fixed by #15186
Assignees
@leandrolanzieri @gschorcht
Labels
Platform: ESP Platform: This PR/issue effects ESP-based platforms Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Comments

@kaspar030
Copy link
Contributor

Description

The following tests regularly fail on CI:

tests/mtd_mapper/esp32-wroom-32:gnu
tests/pkg_littlefs/esp32-wroom-32:gnu
tests/pkg_littlefs2/esp32-wroom-32:gnu
tests/pkg_spiffs/esp32-wroom-32:gnu

Steps to reproduce the issue

Expected results

Actual results

Versions
@kaspar030 kaspar030 added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Platform: ESP Platform: This PR/issue effects ESP-based platforms labels Sep 30, 2020
@kaspar030 kaspar030 mentioned this issue Sep 30, 2020
Merged
@fjmolinas
Copy link
Contributor

Is it reproducible locally as well?

@kaspar030
Copy link
Contributor Author

Is it reproducible locally as well?

Does anyone have the board? Mine are in the office (where I currently never go because of pandemia).

@aabadie
Copy link
Contributor

aabadie commented Sep 30, 2020

Does anyone have the board?

I have one, but won't be able to test before tomorrow.

@fjmolinas
Copy link
Contributor

With tests/mtd_wrapper:

2020-09-30 13:09:11,517 # rst:0x1 (POWERON_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
2020-09-30 13:09:11,520 # configsip: 0, SPIWP:0xee
2020-09-30 13:09:11,526 # clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
2020-09-30 13:09:11,528 # mode:DOUT, clock div:2
2020-09-30 13:09:11,530 # load:0x3fff0018,len:4
2020-09-30 13:09:11,532 # load:0x3fff001c,len:4004
2020-09-30 13:09:11,534 # load:0x40078000,len:7304
2020-09-30 13:09:11,536 # load:0x40080000,len:7224
2020-09-30 13:09:11,538 # entry 0x40080350

@fjmolinas
Copy link
Contributor

With tests/mtd_wrapper:

2020-09-30 13:09:11,517 # rst:0x1 (POWERON_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
2020-09-30 13:09:11,520 # configsip: 0, SPIWP:0xee
2020-09-30 13:09:11,526 # clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
2020-09-30 13:09:11,528 # mode:DOUT, clock div:2
2020-09-30 13:09:11,530 # load:0x3fff0018,len:4
2020-09-30 13:09:11,532 # load:0x3fff001c,len:4004
2020-09-30 13:09:11,534 # load:0x40078000,len:7304
2020-09-30 13:09:11,536 # load:0x40080000,len:7224
2020-09-30 13:09:11,538 # entry 0x40080350

There is no issue in 2020.07, this is a regression.

@fjmolinas
Copy link
Contributor

I'll bisect

@fjmolinas
Copy link
Contributor

Git bisect is pointing to #14754:

81cb769cadd31e4fb30711bf6a3da85fa88b2135 is the first bad commit
commit 81cb769cadd31e4fb30711bf6a3da85fa88b2135
Author: Leandro Lanzieri <leandro.lanzieri@haw-hamburg.de>
Date:   Mon Aug 10 14:45:39 2020 +0200

    makefiles: avoid building archives when compiling

 Makefile.base                                      | 16 ++---
 Makefile.include                                   | 75 +++++-----------------
 boards/native/Makefile.include                     |  2 -
 cpu/arm7_common/Makefile.include                   |  2 -
 cpu/cc26x0/Makefile.include                        |  2 -
 cpu/cc26x2_cc13x2/Makefile.include                 |  2 -
 cpu/efm32/families/efm32gg/Makefile.include        |  3 -
 cpu/efm32/families/efm32lg/Makefile.include        |  3 -
 cpu/efm32/families/efm32pg12b/Makefile.include     |  3 -
 cpu/efm32/families/efm32pg1b/Makefile.include      |  3 -
 cpu/efm32/families/efr32mg12p/Makefile.include     |  3 -
 cpu/efm32/families/efr32mg1p/Makefile.include      |  3 -
 cpu/esp32/Makefile.include                         | 12 ++--
 cpu/esp8266/Makefile.include                       |  4 +-
 cpu/esp_common/Makefile.include                    |  5 +-
 cpu/kinetis/Makefile.include                       |  3 -
 .../newlib_syscalls_mips_uhi/syscalls.c            |  3 +
 cpu/msp430_common/Makefile.include                 |  2 -
 cpu/native/Makefile                                |  5 +-
 makefiles/arch/atmega.inc.mk                       |  4 --
 makefiles/arch/cortexm.inc.mk                      |  7 --
 makefiles/cflags.inc.mk                            | 10 ++-
 makefiles/docker.inc.mk                            |  2 -
 makefiles/modules.inc.mk                           |  2 +-
 makefiles/scan-build.inc.mk                        |  1 -
 makefiles/vars.inc.mk                              |  3 +-
 pkg/ccn-lite/Makefile.include                      |  1 +
 pkg/cryptoauthlib/Makefile.include                 |  1 +
 pkg/jerryscript/Makefile.include                   |  3 +
 pkg/micropython/Makefile.include                   |  3 +
 pkg/nordic_softdevice_ble/Makefile.include         |  1 +
 pkg/openthread/Makefile.dep                        |  1 -
 pkg/openthread/Makefile.include                    |  3 +
 pkg/relic/Makefile.include                         |  1 +
 pkg/tlsf/Makefile.include                          |  6 --
 sys/Makefile.include                               | 11 ----
 sys/newlib_syscalls_default/Makefile.include       |  1 -
 sys/picolibc_syscalls_default/Makefile.include     |  1 -
 tests/unittests/Makefile                           |  2 +-
 39 files changed, 58 insertions(+), 157 deletions(-)
 delete mode 100644 sys/newlib_syscalls_default/Makefile.include
 delete mode 100644 sys/picolibc_syscalls_default/Makefile.include
bisect run success

@benpicco
Copy link
Contributor

benpicco commented Oct 4, 2020

Same issue as #15137

@leandrolanzieri leandrolanzieri mentioned this issue Oct 8, 2020
Merged
@leandrolanzieri leandrolanzieri mentioned this issue Oct 9, 2020
Merged
@kaspar030 kaspar030 mentioned this issue Mar 3, 2021
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@leandrolanzieri leandrolanzieri

@gschorcht gschorcht

Labels
Platform: ESP Platform: This PR/issue effects ESP-based platforms Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

cpu/esp*: fix linker scripts leandrolanzieri/RIOT
6 participants
@benpicco @aabadie @kaspar030 @leandrolanzieri @fjmolinas @gschorcht