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

CFSTORE Integration with Storage-Volume-Manager and Flash-Journal API Latest Versions #2701

Merged
merged 2 commits into from
Sep 21, 2016
Merged

Conversation

simonqhughes
Copy link
Contributor

Description

Summary

This pull request if for the following work package:

  • Adoption of the storage-volume-manager module.
  • Adoption of the latest versions of the flash-journal modules.
  • Integration with CFSTORE.

Details

This pull request includes the following CFSTORE/Flash-Journal/Storage updates and fixes:

  • flash-journal basicAPI fix for ARM toolchain
  • Updated storage-abstraction with version 0.4.7
    (commit c7c4a8c52298bbc006a6f53a059fb2599cad73cc).
  • https://github.com/ARMmbed/storage-volume-manager at version v0.2.10.
  • https://github.com/ARMmbed/mtd-k64f v0.4.2 version of flash.c (imported as storage_driver.c).
  • update to CFSTORE to use the storage-volume-manager API to initialize volume manager and
    add a volume for CFSTORE to use.
  • https://github.com/ARMmbed/flash-journal at version v0.5.3
    (commit 4c58165e2fa02c6ed2b9d166a9c96967e81f458f) including readFrom() support.
  • Taking flash-journal-strategy-sequential v0.6.7 strategy.c
    (commit b11a718761aa9f33679956968a21aaef9179bde1).
  • GCC_ARM, ARM and IAR compiler warning fixes for new versions of flash-journal code.
  • Fix storage-volume-manager test cases for concurrent access from 2 volumes to use
    addresses within the 512-1024kB address range, which is within the cfstore added volume.
  • Fix cfstore/storage-volume-manager IAR warnings when building with verbose flag.

Status

Integrated with mbed-os master all tests were found to pass. See attached log.

Migrations

No changes required to existing CFSTORE clients.

Related PRs

No Related pull requests.

Todos

This is part of the 16Q3 deliverables.

Deploy notes

No changes required to

  • build environment
  • compilers
  • other tools

build_tests_master_gcc_all_20160914_0959.txt

…tes and fixes:

- flash-journal basicAPI fix for ARM toolchain
- Updated storage-abstraction with version 0.4.7
  (commit c7c4a8c52298bbc006a6f53a059fb2599cad73cc).
- https://github.com/ARMmbed/storage-volume-manager at version v0.2.10.
- https://github.com/ARMmbed/mtd-k64f v0.4.2 version of flash.c (imported as storage_driver.c).
- update to CFSTORE to use the storage-volume-manager API to initialize volume manager and
  add a volume for CFSTORE to use.
- https://github.com/ARMmbed/flash-journal at version v0.5.3
  (commit 4c58165e2fa02c6ed2b9d166a9c96967e81f458f) including readFrom() support.
- Taking flash-journal-strategy-sequential v0.6.7 strategy.c
  (commit b11a718761aa9f33679956968a21aaef9179bde1).
- GCC_ARM, ARM and IAR compiler warning fixes for new versions of flash-journal code.
- Fix storage-volume-manager test cases for concurrent access from 2 volumes to use
  addresses within the 512-1024kB address range, which is within the cfstore added volume.
- Fix cfstore/storage-volume-manager IAR warnings when building with verbose flag.
@simonqhughes simonqhughes changed the title This commit includes the following CFSTORE/Flash-Journal/Storage upda… CFSTORE Integration with Storage-Volume-Manager and Flash-Journal API Latest Versions Sep 14, 2016
@bridadan
Copy link
Contributor

@simonqhughes Thanks for submitting the GCC test results, they look good! Do you have any results with the ARM and IAR toolchains?

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 16, 2016

@sg- Please can you review this patch

@simonqhughes Was this tested with IAR or ARMCC?

@sg-
Copy link
Contributor

sg- commented Sep 20, 2016

/morph test

@mbed-bot
Copy link

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 899

All builds and test passed!

Copy link
Contributor

@0xc0170 0xc0170 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@0xc0170 0xc0170 merged commit 2866e21 into ARMmbed:master Sep 21, 2016
artokin added a commit to artokin/mbed-os that referenced this pull request Dec 2, 2021
…25a4af..154db5425e

154db5425e Merge branch 'release_internal' into release_external
dca258c3f0 Updated unit test
89df990e6d Set EAPOL relay state function to LLC as a callback
47d8f42bf5 Removed extra call to supplicant delete callback
00ebfd905f Added new configuration options for Border router
421f6a8191 Move RPL code to the 6lr state machine
e1e43b8ac2 Corrected pae controller BR EUI-64 flagging
aaefdaebab Refactored pae controller (ARMmbed#2707)
b82e0ee19a Update copyright text in CMakeLists.txt files (ARMmbed#2706)
5791a22d37 Renamed bootstrap files and separated pae controller callback functions (ARMmbed#2703)
4e3ec07caf MAC beacon payload max MTU allocation limited to for support MTU size 127.
a997ab6653 Iotthd 4698 (ARMmbed#2702)
e64b057bff Adjusted security protocols timeouts (ARMmbed#2701)
cb3db6419f Return invalid values from old API if new configuration is used
6872401f8d RPL address registration update
90a434d318 RPL DAO lifetime update
e39f6432eb Forward RPL configuration unmodified
58da4d6c93 Sync Cmake updates from Mbed OS (ARMmbed#2697)
4c229b46c9 Adjust Wi-SUN host/router config - HAVE_RPL (ARMmbed#2696)
40bd2320ba Update release v15.0.0 details to CHANGELOG (ARMmbed#2694)

git-subtree-dir: connectivity/nanostack/sal-stack-nanostack
git-subtree-split: 154db5425eb9f0df82ee081cb05e7d80cda11415
artokin added a commit to artokin/mbed-os that referenced this pull request Dec 8, 2021
…25a4af..154db5425e

154db5425e Merge branch 'release_internal' into release_external
dca258c3f0 Updated unit test
89df990e6d Set EAPOL relay state function to LLC as a callback
47d8f42bf5 Removed extra call to supplicant delete callback
00ebfd905f Added new configuration options for Border router
421f6a8191 Move RPL code to the 6lr state machine
e1e43b8ac2 Corrected pae controller BR EUI-64 flagging
aaefdaebab Refactored pae controller (ARMmbed#2707)
b82e0ee19a Update copyright text in CMakeLists.txt files (ARMmbed#2706)
5791a22d37 Renamed bootstrap files and separated pae controller callback functions (ARMmbed#2703)
4e3ec07caf MAC beacon payload max MTU allocation limited to for support MTU size 127.
a997ab6653 Iotthd 4698 (ARMmbed#2702)
e64b057bff Adjusted security protocols timeouts (ARMmbed#2701)
cb3db6419f Return invalid values from old API if new configuration is used
6872401f8d RPL address registration update
90a434d318 RPL DAO lifetime update
e39f6432eb Forward RPL configuration unmodified
58da4d6c93 Sync Cmake updates from Mbed OS (ARMmbed#2697)
4c229b46c9 Adjust Wi-SUN host/router config - HAVE_RPL (ARMmbed#2696)
40bd2320ba Update release v15.0.0 details to CHANGELOG (ARMmbed#2694)

git-subtree-dir: connectivity/nanostack/sal-stack-nanostack
git-subtree-split: 154db5425eb9f0df82ee081cb05e7d80cda11415
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants