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

pkg/lwext4: add lightweight implementation of the ext2/3/4 filesystem #18682

Merged
merged 4 commits into from
Feb 25, 2023

Conversation

benpicco
Copy link
Contributor

@benpicco benpicco commented Oct 3, 2022

Contribution description

This adds lwEXT4 as a package and provides integration with MTD and VFS.

lwEXT4 is an implementation of the ext2/3/4 journaling file system. The different feature levels can be selected by the pseudo-modules lwext2_vfs, lwext3_vfs and lwext4_vfs respectively.

It is intended for use on SD cards (maybe large NAND flashes once we have support for those) as a more fault tolerant alternative to FAT. (If this is really the case has to be seen 😉)

Testing procedure

In tests/vfs_default enable one of the lwext%_vfs modules either on native or same54-xpro. (Other boards need the mount point configured)

Issues/PRs references

@github-actions github-actions bot added Area: boards Area: Board ports Area: build system Area: Build system Area: doc Area: Documentation Area: pkg Area: External package ports Area: sys Area: System Area: tests Area: tests and testing framework Platform: native Platform: This PR/issue effects the native platform labels Oct 3, 2022
@benpicco benpicco requested a review from maribu October 4, 2022 12:49
@benpicco benpicco changed the title pkg/lwext4: add lightweight implementation of ext2/3/4 pkg/lwext4: add lightweight implementation of the ext2/3/4 filesystem Oct 14, 2022
Copy link
Member

@maribu maribu left a comment

Choose a reason for hiding this comment

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

a few nitpicks, other lgtm

pkg/lwext4/fs/lwext4_fs.c Outdated Show resolved Hide resolved
pkg/lwext4/fs/lwext4_fs.c Outdated Show resolved Hide resolved
@benpicco benpicco added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Feb 25, 2023
@riot-ci
Copy link

riot-ci commented Feb 25, 2023

Murdock results

✔️ PASSED

020aac6 boards/same54-xpro: add support for lwEXT4

Success Failures Total Runtime
6865 0 6865 08m:37s

Artifacts

This only reflects a subset of all builds from https://ci-prod.riot-os.org. Please refer to https://ci.riot-os.org for a complete build for now.

@github-actions github-actions bot removed the Area: tests Area: tests and testing framework label Feb 25, 2023
@benpicco
Copy link
Contributor Author

Thank you!

bors merge

@bors
Copy link
Contributor

bors bot commented Feb 25, 2023

Build succeeded:

@bors bors bot merged commit d4422a0 into RIOT-OS:master Feb 25, 2023
@benpicco benpicco deleted the pkg/lwext4 branch February 25, 2023 20:36
@MrKevinWeiss MrKevinWeiss added this to the Release 2023.04 milestone Apr 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: boards Area: Board ports Area: build system Area: Build system Area: doc Area: Documentation Area: pkg Area: External package ports Area: sys Area: System CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: native Platform: This PR/issue effects the native platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants