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.

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

tests/subsys/fs/fcb fails on nRF52840-DK #24207

Closed
mbolivar-nordic opened this issue Apr 8, 2020 · 4 comments · Fixed by #24375
Closed

tests/subsys/fs/fcb fails on nRF52840-DK #24207

mbolivar-nordic opened this issue Apr 8, 2020 · 4 comments · Fixed by #24375
Assignees
Labels
area: File System bug The issue is a bug, or the PR is fixing a bug platform: nRF Nordic nRFx priority: low Low impact/importance bug Regression Something, which was working, does not anymore

Comments

@mbolivar-nordic
Copy link
Contributor

Describe the bug

I'm getting a test failure on tests/subsys/fs/fcb for nrf52840dk_nrf52840.

To Reproduce
Steps to reproduce the behavior:

  1. cd zephyrproject/zephyr
  2. west build -b nrf52840dk_nrf52840 -p -t flash -s tests/subsys/fs/fcb
  3. test fails on device serial console

Expected behavior

The board is listed in platform_whitelist inside tests/subsys/fs/fcb/testcase.yaml, so I expected this to pass.

Impact

Unclear.

Screenshots or console output

Running test suite test_fcb
===================================================================
starting test - fcb_test_len
PASS - fcb_test_len
===================================================================
starting test - fcb_test_init
PASS - fcb_test_init
===================================================================
starting test - fcb_test_empty_walk
PASS - fcb_test_empty_walk
===================================================================
starting test - fcb_test_append

    Assertion failed at WEST_TOPDIR/zephyr/tests/subsys/fs/fcb/src/fcb_test_append.c:30: fcb_test_append: (rc == 0 is false)
flash_area_write call failure
FAIL - fcb_test_append
===================================================================
starting test - fcb_test_append_too_big

    Assertion failed at WEST_TOPDIR/zephyr/tests/subsys/fs/fcb/src/fcb_test_append_too_big.c:42: fcb_test_append_too_big: (rc == 0 is false)
fcb_append call failure
FAIL - fcb_test_append_too_big
===================================================================
starting test - fcb_test_append_fill
PASS - fcb_test_append_fill
===================================================================
starting test - fcb_test_rotate
PASS - fcb_test_rotate
===================================================================
starting test - fcb_test_multi_scratch
PASS - fcb_test_multi_scratch
===================================================================
starting test - fcb_test_last_of_n
PASS - fcb_test_last_of_n
===================================================================
starting test - fcb_test_reset

    Assertion failed at WEST_TOPDIR/zephyr/tests/subsys/fs/fcb/src/fcb_test_reset.c:82: fcb_test_reset: (rc == 0 is false)
flash_area_write call failure
FAIL - fcb_test_reset
===================================================================
Test suite test_fcb failed.
===================================================================
PROJECT EXECUTION FAILED

Environment:

  • OS: linux
  • Toolchain Zephyr SDK 0.11.1
  • Commit SHA or Version used: recent master (e.g. d35567b)
@mbolivar-nordic mbolivar-nordic added the bug The issue is a bug, or the PR is fixing a bug label Apr 8, 2020
@mbolivar-nordic
Copy link
Contributor Author

@nvlsianpu not sure if you are the right assignee. Please feel free to change.

@carlescufi carlescufi added area: File System priority: low Low impact/importance bug labels Apr 14, 2020
@carlescufi
Copy link
Member

@nvlsianpu should we deprecate FCB at this point?

@mbolivar-nordic
Copy link
Contributor Author

@nvlsianpu should we deprecate FCB at this point?

Ah, so this is the one which might get deprecated. I couldn't remember the status of all these storage backends.

@nvlsianpu nvlsianpu added the Regression Something, which was working, does not anymore label Apr 15, 2020
@nvlsianpu
Copy link
Collaborator

@nvlsianpu should we deprecate FCB at this point?

Not today - we can deprecate fcb at some point, but it was not problematic for user and initially it was the default setting back-end, so better to make a query before.

de-nordic added a commit to de-nordic/zephyr that referenced this issue Apr 15, 2020
The commit fixes problem with test attempting to perform 1 byte writes
to flash, which are not emulated by default.

GH issue: zephyrproject-rtos#24207

Signed-off-by: Dominik Ermel <dominik.ermel@nordicsemi.no>
@carlescufi carlescufi added the platform: nRF Nordic nRFx label Apr 15, 2020
@mbolivar-nordic mbolivar-nordic changed the title fcb test fails on nRF52840-DK tests/subsys/fs/fcb fails on nRF52840-DK Apr 15, 2020
ioannisg pushed a commit that referenced this issue Apr 17, 2020
The commit fixes problem with test attempting to perform 1 byte writes
to flash, which are not emulated by default.

GH issue: #24207

Signed-off-by: Dominik Ermel <dominik.ermel@nordicsemi.no>
hakehuang pushed a commit to hakehuang/zephyr that referenced this issue Jun 20, 2020
The commit fixes problem with test attempting to perform 1 byte writes
to flash, which are not emulated by default.

GH issue: zephyrproject-rtos#24207

Signed-off-by: Dominik Ermel <dominik.ermel@nordicsemi.no>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: File System bug The issue is a bug, or the PR is fixing a bug platform: nRF Nordic nRFx priority: low Low impact/importance bug Regression Something, which was working, does not anymore
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants