diff --git a/lfs.c b/lfs.c index eb832fa0..d5c84124 100644 --- a/lfs.c +++ b/lfs.c @@ -5,7 +5,7 @@ * SPDX-License-Identifier: BSD-3-Clause */ #include "lfs.h" -#include "lfs_util.h" +#include #define LFS_BLOCK_NULL ((lfs_block_t)-1) #define LFS_BLOCK_INLINE ((lfs_block_t)-2) @@ -1525,7 +1525,7 @@ static int lfs_dir_compact(lfs_t *lfs, if (lfs_pair_cmp(dir->pair, (const lfs_block_t[2]){0, 1}) == 0) { // oh no! we're writing too much to the superblock, // should we expand? - lfs_ssize_t res = lfs_fs_size(lfs); + lfs_ssize_t res = lfs_fs_size_raw(lfs); if (res < 0) { return res; } @@ -1906,7 +1906,7 @@ static int lfs_dir_commit(lfs_t *lfs, lfs_mdir_t *dir, /// Top level directory operations /// -int lfs_mkdir(lfs_t *lfs, const char *path) { +int lfs_mkdir_raw(lfs_t *lfs, const char *path) { LFS_TRACE("lfs_mkdir(%p, \"%s\")", (void*)lfs, path); // deorphan if we haven't yet, needed at most once after poweron int err = lfs_fs_forceconsistency(lfs); @@ -2005,7 +2005,7 @@ int lfs_mkdir(lfs_t *lfs, const char *path) { return 0; } -int lfs_dir_open(lfs_t *lfs, lfs_dir_t *dir, const char *path) { +int lfs_dir_open_raw(lfs_t *lfs, lfs_dir_t *dir, const char *path) { LFS_TRACE("lfs_dir_open(%p, %p, \"%s\")", (void*)lfs, (void*)dir, path); lfs_stag_t tag = lfs_dir_find(lfs, &dir->m, &path, NULL); if (tag < 0) { @@ -2056,7 +2056,7 @@ int lfs_dir_open(lfs_t *lfs, lfs_dir_t *dir, const char *path) { return 0; } -int lfs_dir_close(lfs_t *lfs, lfs_dir_t *dir) { +int lfs_dir_close_raw(lfs_t *lfs, lfs_dir_t *dir) { LFS_TRACE("lfs_dir_close(%p, %p)", (void*)lfs, (void*)dir); // remove from list of mdirs for (struct lfs_mlist **p = &lfs->mlist; *p; p = &(*p)->next) { @@ -2070,7 +2070,7 @@ int lfs_dir_close(lfs_t *lfs, lfs_dir_t *dir) { return 0; } -int lfs_dir_read(lfs_t *lfs, lfs_dir_t *dir, struct lfs_info *info) { +int lfs_dir_read_raw(lfs_t *lfs, lfs_dir_t *dir, struct lfs_info *info) { LFS_TRACE("lfs_dir_read(%p, %p, %p)", (void*)lfs, (void*)dir, (void*)info); memset(info, 0, sizeof(*info)); @@ -2123,11 +2123,11 @@ int lfs_dir_read(lfs_t *lfs, lfs_dir_t *dir, struct lfs_info *info) { return true; } -int lfs_dir_seek(lfs_t *lfs, lfs_dir_t *dir, lfs_off_t off) { +int lfs_dir_seek_raw(lfs_t *lfs, lfs_dir_t *dir, lfs_off_t off) { LFS_TRACE("lfs_dir_seek(%p, %p, %"PRIu32")", (void*)lfs, (void*)dir, off); // simply walk from head dir - int err = lfs_dir_rewind(lfs, dir); + int err = lfs_dir_rewind_raw(lfs, dir); if (err) { LFS_TRACE("lfs_dir_seek -> %d", err); return err; @@ -2166,14 +2166,14 @@ int lfs_dir_seek(lfs_t *lfs, lfs_dir_t *dir, lfs_off_t off) { return 0; } -lfs_soff_t lfs_dir_tell(lfs_t *lfs, lfs_dir_t *dir) { +lfs_soff_t lfs_dir_tell_raw(lfs_t *lfs, lfs_dir_t *dir) { LFS_TRACE("lfs_dir_tell(%p, %p)", (void*)lfs, (void*)dir); (void)lfs; LFS_TRACE("lfs_dir_tell -> %"PRId32, dir->pos); return dir->pos; } -int lfs_dir_rewind(lfs_t *lfs, lfs_dir_t *dir) { +int lfs_dir_rewind_raw(lfs_t *lfs, lfs_dir_t *dir) { LFS_TRACE("lfs_dir_rewind(%p, %p)", (void*)lfs, (void*)dir); // reload the head dir int err = lfs_dir_fetch(lfs, &dir->m, dir->head); @@ -2380,7 +2380,7 @@ static int lfs_ctz_traverse(lfs_t *lfs, /// Top level file operations /// -int lfs_file_opencfg(lfs_t *lfs, lfs_file_t *file, +int lfs_file_opencfg_raw(lfs_t *lfs, lfs_file_t *file, const char *path, int flags, const struct lfs_file_config *cfg) { LFS_TRACE("lfs_file_opencfg(%p, %p, \"%s\", %x, %p {" @@ -2529,26 +2529,26 @@ int lfs_file_opencfg(lfs_t *lfs, lfs_file_t *file, cleanup: // clean up lingering resources file->flags |= LFS_F_ERRED; - lfs_file_close(lfs, file); + lfs_file_close_raw(lfs, file); LFS_TRACE("lfs_file_opencfg -> %d", err); return err; } -int lfs_file_open(lfs_t *lfs, lfs_file_t *file, +int lfs_file_open_raw(lfs_t *lfs, lfs_file_t *file, const char *path, int flags) { LFS_TRACE("lfs_file_open(%p, %p, \"%s\", %x)", (void*)lfs, (void*)file, path, flags); static const struct lfs_file_config defaults = {0}; - int err = lfs_file_opencfg(lfs, file, path, flags, &defaults); + int err = lfs_file_opencfg_raw(lfs, file, path, flags, &defaults); LFS_TRACE("lfs_file_open -> %d", err); return err; } -int lfs_file_close(lfs_t *lfs, lfs_file_t *file) { +int lfs_file_close_raw(lfs_t *lfs, lfs_file_t *file) { LFS_TRACE("lfs_file_close(%p, %p)", (void*)lfs, (void*)file); LFS_ASSERT(file->flags & LFS_F_OPENED); - int err = lfs_file_sync(lfs, file); + int err = lfs_file_sync_raw(lfs, file); // remove from list of mdirs for (struct lfs_mlist **p = &lfs->mlist; *p; p = &(*p)->next) { @@ -2679,12 +2679,12 @@ static int lfs_file_flush(lfs_t *lfs, lfs_file_t *file) { // copy over a byte at a time, leave it up to caching // to make this efficient uint8_t data; - lfs_ssize_t res = lfs_file_read(lfs, &orig, &data, 1); + lfs_ssize_t res = lfs_file_read_raw(lfs, &orig, &data, 1); if (res < 0) { return res; } - res = lfs_file_write(lfs, file, &data, 1); + res = lfs_file_write_raw(lfs, file, &data, 1); if (res < 0) { return res; } @@ -2731,7 +2731,7 @@ static int lfs_file_flush(lfs_t *lfs, lfs_file_t *file) { return 0; } -int lfs_file_sync(lfs_t *lfs, lfs_file_t *file) { +int lfs_file_sync_raw(lfs_t *lfs, lfs_file_t *file) { LFS_TRACE("lfs_file_sync(%p, %p)", (void*)lfs, (void*)file); LFS_ASSERT(file->flags & LFS_F_OPENED); @@ -2788,7 +2788,7 @@ int lfs_file_sync(lfs_t *lfs, lfs_file_t *file) { return 0; } -lfs_ssize_t lfs_file_read(lfs_t *lfs, lfs_file_t *file, +lfs_ssize_t lfs_file_read_raw(lfs_t *lfs, lfs_file_t *file, void *buffer, lfs_size_t size) { LFS_TRACE("lfs_file_read(%p, %p, %p, %"PRIu32")", (void*)lfs, (void*)file, buffer, size); @@ -2868,7 +2868,7 @@ lfs_ssize_t lfs_file_read(lfs_t *lfs, lfs_file_t *file, return size; } -lfs_ssize_t lfs_file_write(lfs_t *lfs, lfs_file_t *file, +lfs_ssize_t lfs_file_write_raw(lfs_t *lfs, lfs_file_t *file, const void *buffer, lfs_size_t size) { LFS_TRACE("lfs_file_write(%p, %p, %p, %"PRIu32")", (void*)lfs, (void*)file, buffer, size); @@ -2903,7 +2903,7 @@ lfs_ssize_t lfs_file_write(lfs_t *lfs, lfs_file_t *file, file->pos = file->ctz.size; while (file->pos < pos) { - lfs_ssize_t res = lfs_file_write(lfs, file, &(uint8_t){0}, 1); + lfs_ssize_t res = lfs_file_write_raw(lfs, file, &(uint8_t){0}, 1); if (res < 0) { LFS_TRACE("lfs_file_write -> %"PRId32, res); return res; @@ -2999,7 +2999,7 @@ lfs_ssize_t lfs_file_write(lfs_t *lfs, lfs_file_t *file, return size; } -lfs_soff_t lfs_file_seek(lfs_t *lfs, lfs_file_t *file, +lfs_soff_t lfs_file_seek_raw(lfs_t *lfs, lfs_file_t *file, lfs_soff_t off, int whence) { LFS_TRACE("lfs_file_seek(%p, %p, %"PRId32", %d)", (void*)lfs, (void*)file, off, whence); @@ -3034,7 +3034,7 @@ lfs_soff_t lfs_file_seek(lfs_t *lfs, lfs_file_t *file, return npos; } -int lfs_file_truncate(lfs_t *lfs, lfs_file_t *file, lfs_off_t size) { +int lfs_file_truncate_raw(lfs_t *lfs, lfs_file_t *file, lfs_off_t size) { LFS_TRACE("lfs_file_truncate(%p, %p, %"PRIu32")", (void*)lfs, (void*)file, size); LFS_ASSERT(file->flags & LFS_F_OPENED); @@ -3046,7 +3046,7 @@ int lfs_file_truncate(lfs_t *lfs, lfs_file_t *file, lfs_off_t size) { } lfs_off_t pos = file->pos; - lfs_off_t oldsize = lfs_file_size(lfs, file); + lfs_off_t oldsize = lfs_file_size_raw(lfs, file); if (size < oldsize) { // need to flush since directly changing metadata int err = lfs_file_flush(lfs, file); @@ -3070,7 +3070,7 @@ int lfs_file_truncate(lfs_t *lfs, lfs_file_t *file, lfs_off_t size) { } else if (size > oldsize) { // flush+seek if not already at end if (file->pos != oldsize) { - lfs_soff_t res = lfs_file_seek(lfs, file, 0, LFS_SEEK_END); + lfs_soff_t res = lfs_file_seek_raw(lfs, file, 0, LFS_SEEK_END); if (res < 0) { LFS_TRACE("lfs_file_truncate -> %"PRId32, res); return (int)res; @@ -3079,7 +3079,7 @@ int lfs_file_truncate(lfs_t *lfs, lfs_file_t *file, lfs_off_t size) { // fill with zeros while (file->pos < size) { - lfs_ssize_t res = lfs_file_write(lfs, file, &(uint8_t){0}, 1); + lfs_ssize_t res = lfs_file_write_raw(lfs, file, &(uint8_t){0}, 1); if (res < 0) { LFS_TRACE("lfs_file_truncate -> %"PRId32, res); return (int)res; @@ -3088,7 +3088,7 @@ int lfs_file_truncate(lfs_t *lfs, lfs_file_t *file, lfs_off_t size) { } // restore pos - lfs_soff_t res = lfs_file_seek(lfs, file, pos, LFS_SEEK_SET); + lfs_soff_t res = lfs_file_seek_raw(lfs, file, pos, LFS_SEEK_SET); if (res < 0) { LFS_TRACE("lfs_file_truncate -> %"PRId32, res); return (int)res; @@ -3098,7 +3098,7 @@ int lfs_file_truncate(lfs_t *lfs, lfs_file_t *file, lfs_off_t size) { return 0; } -lfs_soff_t lfs_file_tell(lfs_t *lfs, lfs_file_t *file) { +lfs_soff_t lfs_file_tell_raw(lfs_t *lfs, lfs_file_t *file) { LFS_TRACE("lfs_file_tell(%p, %p)", (void*)lfs, (void*)file); LFS_ASSERT(file->flags & LFS_F_OPENED); (void)lfs; @@ -3106,9 +3106,9 @@ lfs_soff_t lfs_file_tell(lfs_t *lfs, lfs_file_t *file) { return file->pos; } -int lfs_file_rewind(lfs_t *lfs, lfs_file_t *file) { +int lfs_file_rewind_raw(lfs_t *lfs, lfs_file_t *file) { LFS_TRACE("lfs_file_rewind(%p, %p)", (void*)lfs, (void*)file); - lfs_soff_t res = lfs_file_seek(lfs, file, 0, LFS_SEEK_SET); + lfs_soff_t res = lfs_file_seek_raw(lfs, file, 0, LFS_SEEK_SET); if (res < 0) { LFS_TRACE("lfs_file_rewind -> %"PRId32, res); return (int)res; @@ -3118,7 +3118,7 @@ int lfs_file_rewind(lfs_t *lfs, lfs_file_t *file) { return 0; } -lfs_soff_t lfs_file_size(lfs_t *lfs, lfs_file_t *file) { +lfs_soff_t lfs_file_size_raw(lfs_t *lfs, lfs_file_t *file) { LFS_TRACE("lfs_file_size(%p, %p)", (void*)lfs, (void*)file); LFS_ASSERT(file->flags & LFS_F_OPENED); (void)lfs; @@ -3134,7 +3134,7 @@ lfs_soff_t lfs_file_size(lfs_t *lfs, lfs_file_t *file) { /// General fs operations /// -int lfs_stat(lfs_t *lfs, const char *path, struct lfs_info *info) { +int lfs_stat_raw(lfs_t *lfs, const char *path, struct lfs_info *info) { LFS_TRACE("lfs_stat(%p, \"%s\", %p)", (void*)lfs, path, (void*)info); lfs_mdir_t cwd; lfs_stag_t tag = lfs_dir_find(lfs, &cwd, &path, NULL); @@ -3148,7 +3148,7 @@ int lfs_stat(lfs_t *lfs, const char *path, struct lfs_info *info) { return err; } -int lfs_remove(lfs_t *lfs, const char *path) { +int lfs_remove_raw(lfs_t *lfs, const char *path) { LFS_TRACE("lfs_remove(%p, \"%s\")", (void*)lfs, path); // deorphan if we haven't yet, needed at most once after poweron int err = lfs_fs_forceconsistency(lfs); @@ -3229,7 +3229,7 @@ int lfs_remove(lfs_t *lfs, const char *path) { return 0; } -int lfs_rename(lfs_t *lfs, const char *oldpath, const char *newpath) { +int lfs_rename_raw(lfs_t *lfs, const char *oldpath, const char *newpath) { LFS_TRACE("lfs_rename(%p, \"%s\", \"%s\")", (void*)lfs, oldpath, newpath); // deorphan if we haven't yet, needed at most once after poweron @@ -3374,7 +3374,7 @@ int lfs_rename(lfs_t *lfs, const char *oldpath, const char *newpath) { return 0; } -lfs_ssize_t lfs_getattr(lfs_t *lfs, const char *path, +lfs_ssize_t lfs_getattr_raw(lfs_t *lfs, const char *path, uint8_t type, void *buffer, lfs_size_t size) { LFS_TRACE("lfs_getattr(%p, \"%s\", %"PRIu8", %p, %"PRIu32")", (void*)lfs, path, type, buffer, size); @@ -3437,7 +3437,7 @@ static int lfs_commitattr(lfs_t *lfs, const char *path, {LFS_MKTAG(LFS_TYPE_USERATTR + type, id, size), buffer})); } -int lfs_setattr(lfs_t *lfs, const char *path, +int lfs_setattr_raw(lfs_t *lfs, const char *path, uint8_t type, const void *buffer, lfs_size_t size) { LFS_TRACE("lfs_setattr(%p, \"%s\", %"PRIu8", %p, %"PRIu32")", (void*)lfs, path, type, buffer, size); @@ -3451,7 +3451,7 @@ int lfs_setattr(lfs_t *lfs, const char *path, return err; } -int lfs_removeattr(lfs_t *lfs, const char *path, uint8_t type) { +int lfs_removeattr_raw(lfs_t *lfs, const char *path, uint8_t type) { LFS_TRACE("lfs_removeattr(%p, \"%s\", %"PRIu8")", (void*)lfs, path, type); int err = lfs_commitattr(lfs, path, type, NULL, 0x3ff); LFS_TRACE("lfs_removeattr -> %d", err); @@ -3584,7 +3584,7 @@ static int lfs_deinit(lfs_t *lfs) { return 0; } -int lfs_format(lfs_t *lfs, const struct lfs_config *cfg) { +int lfs_format_raw(lfs_t *lfs, const struct lfs_config *cfg) { LFS_TRACE("lfs_format(%p, %p {.context=%p, " ".read=%p, .prog=%p, .erase=%p, .sync=%p, " ".read_size=%"PRIu32", .prog_size=%"PRIu32", " @@ -3665,7 +3665,7 @@ int lfs_format(lfs_t *lfs, const struct lfs_config *cfg) { return err; } -int lfs_mount(lfs_t *lfs, const struct lfs_config *cfg) { +int lfs_mount_raw(lfs_t *lfs, const struct lfs_config *cfg) { LFS_TRACE("lfs_mount(%p, %p {.context=%p, " ".read=%p, .prog=%p, .erase=%p, .sync=%p, " ".read_size=%"PRIu32", .prog_size=%"PRIu32", " @@ -3804,12 +3804,12 @@ int lfs_mount(lfs_t *lfs, const struct lfs_config *cfg) { return 0; cleanup: - lfs_unmount(lfs); + lfs_unmount_raw(lfs); LFS_TRACE("lfs_mount -> %d", err); return err; } -int lfs_unmount(lfs_t *lfs) { +int lfs_unmount_raw(lfs_t *lfs) { LFS_TRACE("lfs_unmount(%p)", (void*)lfs); int err = lfs_deinit(lfs); LFS_TRACE("lfs_unmount -> %d", err); @@ -3914,7 +3914,7 @@ int lfs_fs_traverseraw(lfs_t *lfs, return 0; } -int lfs_fs_traverse(lfs_t *lfs, +int lfs_fs_traverse_raw(lfs_t *lfs, int (*cb)(void *data, lfs_block_t block), void *data) { LFS_TRACE("lfs_fs_traverse(%p, %p, %p)", (void*)lfs, (void*)(uintptr_t)cb, data); @@ -4235,7 +4235,7 @@ static int lfs_fs_size_count(void *p, lfs_block_t block) { return 0; } -lfs_ssize_t lfs_fs_size(lfs_t *lfs) { +lfs_ssize_t lfs_fs_size_raw(lfs_t *lfs) { LFS_TRACE("lfs_fs_size(%p)", (void*)lfs); lfs_size_t size = 0; int err = lfs_fs_traverseraw(lfs, lfs_fs_size_count, &size, false); @@ -4669,7 +4669,7 @@ static int lfs1_unmount(lfs_t *lfs) { } /// v1 migration /// -int lfs_migrate(lfs_t *lfs, const struct lfs_config *cfg) { +int lfs_migrate_raw(lfs_t *lfs, const struct lfs_config *cfg) { LFS_TRACE("lfs_migrate(%p, %p {.context=%p, " ".read=%p, .prog=%p, .erase=%p, .sync=%p, " ".read_size=%"PRIu32", .prog_size=%"PRIu32", " @@ -4911,3 +4911,398 @@ int lfs_migrate(lfs_t *lfs, const struct lfs_config *cfg) { } #endif + +#if LFS_THREAD_SAFE + +int lfs_format_ts(lfs_t *lfs, const struct lfs_config *config) +{ + int err = config->lock(config); + if(err) + { + return err; + } + err = lfs_format_raw(lfs, config); + config->unlock(config); + + return err; +} +int lfs_mount_ts(lfs_t *lfs, const struct lfs_config *config) +{ + int err = config->lock(config); + if(err) + { + return err; + } + err = lfs_mount_raw(lfs, config); + config->unlock(config); + + return err; +} +int lfs_unmount_ts(lfs_t *lfs) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_unmount_raw(lfs); + lfs->cfg->unlock(lfs->cfg); + + return err; +} +int lfs_remove_ts(lfs_t *lfs, const char *path) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_remove_raw(lfs, path); + lfs->cfg->unlock(lfs->cfg); + + return err; +} +int lfs_rename_ts(lfs_t *lfs, const char *oldpath, const char *newpath) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_rename_raw(lfs, oldpath, newpath); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +int lfs_stat_ts(lfs_t *lfs, const char *path, struct lfs_info *info) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_stat_raw(lfs, path, info); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +lfs_ssize_t lfs_getattr_ts(lfs_t *lfs, const char *path, uint8_t type, void *buffer, lfs_size_t size) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_getattr_raw(lfs, path, type, buffer, size); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +int lfs_setattr_ts(lfs_t *lfs, const char *path, uint8_t type, const void *buffer, lfs_size_t size) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_setattr_raw(lfs, path, type, buffer, size); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +int lfs_removeattr_ts(lfs_t *lfs, const char *path, uint8_t type) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_removeattr_raw(lfs, path, type); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +int lfs_file_open_ts(lfs_t *lfs, lfs_file_t *file, const char *path, int flags) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_file_open_raw(lfs, file, path, flags); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +int lfs_file_opencfg_ts(lfs_t *lfs, lfs_file_t *file, const char *path, int flags, const struct lfs_file_config *config) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_file_opencfg_raw(lfs, file, path, flags, config); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +int lfs_file_close_ts(lfs_t *lfs, lfs_file_t *file) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_file_close_raw(lfs, file); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +int lfs_file_sync_ts(lfs_t *lfs, lfs_file_t *file) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_file_sync_raw(lfs, file); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +lfs_ssize_t lfs_file_read_ts(lfs_t *lfs, lfs_file_t *file, void *buffer, lfs_size_t size) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_file_read_raw(lfs, file, buffer, size); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +lfs_ssize_t lfs_file_write_ts(lfs_t *lfs, lfs_file_t *file, const void *buffer, lfs_size_t size) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_file_write_raw(lfs, file, buffer, size); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +lfs_soff_t lfs_file_seek_ts(lfs_t *lfs, lfs_file_t *file, lfs_soff_t off, int whence) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_file_seek_raw(lfs, file, off, whence); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +int lfs_file_truncate_ts(lfs_t *lfs, lfs_file_t *file, lfs_off_t size) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_file_truncate_raw(lfs, file, size); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +lfs_soff_t lfs_file_tell_ts(lfs_t *lfs, lfs_file_t *file) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_file_tell_raw(lfs, file); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +int lfs_file_rewind_ts(lfs_t *lfs, lfs_file_t *file) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_file_rewind_raw(lfs, file); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +lfs_soff_t lfs_file_size_ts(lfs_t *lfs, lfs_file_t *file) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_file_size_raw(lfs, file); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +int lfs_mkdir_ts(lfs_t *lfs, const char *path) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_mkdir_raw(lfs, path); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +int lfs_dir_open_ts(lfs_t *lfs, lfs_dir_t *dir, const char *path) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_dir_open_raw(lfs, dir, path); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +int lfs_dir_close_ts(lfs_t *lfs, lfs_dir_t *dir) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_dir_close_raw(lfs, dir); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +int lfs_dir_read_ts(lfs_t *lfs, lfs_dir_t *dir, struct lfs_info *info) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_dir_read_raw(lfs, dir, info); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +int lfs_dir_seek_ts(lfs_t *lfs, lfs_dir_t *dir, lfs_off_t off) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_dir_seek_raw(lfs, dir, off); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +lfs_soff_t lfs_dir_tell_ts(lfs_t *lfs, lfs_dir_t *dir) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_dir_tell_raw(lfs, dir); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +int lfs_dir_rewind_ts(lfs_t *lfs, lfs_dir_t *dir) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_dir_rewind_raw(lfs, dir); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +lfs_ssize_t lfs_fs_size_ts(lfs_t *lfs) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_fs_size_raw(lfs); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} +int lfs_fs_traverse_ts(lfs_t *lfs, int (*cb)(void*, lfs_block_t), void *data) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_fs_traverse_raw(lfs, cb, data); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} + +#ifdef LFS_MIGRATE + +int lfs_migrate_ts(lfs_t *lfs, const struct lfs_config *cfg) +{ + int err = lfs->cfg->lock(lfs->cfg); + if(err) + { + return err; + } + err = lfs_migrate_raw(lfs, cfg); + + lfs->cfg->unlock(lfs->cfg); + + return err; +} + +#endif +#endif \ No newline at end of file diff --git a/lfs.h b/lfs.h index 35bbbabf..b4d65cbb 100644 --- a/lfs.h +++ b/lfs.h @@ -9,6 +9,7 @@ #include #include +#include #ifdef __cplusplus extern "C" @@ -53,7 +54,7 @@ typedef uint32_t lfs_block_t; // Maximum size of a file in bytes, may be redefined to limit to support other // drivers. Limited on disk to <= 4294967296. However, above 2147483647 the -// functions lfs_file_seek, lfs_file_size, and lfs_file_tell will return +// functions lfs_file_seek_raw, lfs_file_size_raw, and lfs_file_tell_raw will return // incorrect values due to using signed integers. Stored in superblock and // must be respected by other littlefs drivers. #ifndef LFS_FILE_MAX @@ -174,6 +175,16 @@ struct lfs_config { // are propogated to the user. int (*sync)(const struct lfs_config *c); + #if LFS_THREAD_SAFE + // Lock the underlying block device. Negative error codes + // are propogated to the user. + int (*lock)(const struct lfs_config *c); + + // Unlock the underlying block device. Negative error codes + // are propogated to the user. + int (*unlock)(const struct lfs_config *c); + #endif + // Minimum size of a block read. All read operations will be a // multiple of this value. lfs_size_t read_size; @@ -406,7 +417,7 @@ typedef struct lfs { // be zeroed for defaults and backwards compatibility. // // Returns a negative error code on failure. -int lfs_format(lfs_t *lfs, const struct lfs_config *config); +int lfs_format_raw(lfs_t *lfs, const struct lfs_config *config); // Mounts a littlefs // @@ -416,13 +427,13 @@ int lfs_format(lfs_t *lfs, const struct lfs_config *config); // be zeroed for defaults and backwards compatibility. // // Returns a negative error code on failure. -int lfs_mount(lfs_t *lfs, const struct lfs_config *config); +int lfs_mount_raw(lfs_t *lfs, const struct lfs_config *config); // Unmounts a littlefs // // Does nothing besides releasing any allocated resources. // Returns a negative error code on failure. -int lfs_unmount(lfs_t *lfs); +int lfs_unmount_raw(lfs_t *lfs); /// General operations /// @@ -430,7 +441,7 @@ int lfs_unmount(lfs_t *lfs); // // If removing a directory, the directory must be empty. // Returns a negative error code on failure. -int lfs_remove(lfs_t *lfs, const char *path); +int lfs_remove_raw(lfs_t *lfs, const char *path); // Rename or move a file or directory // @@ -438,13 +449,13 @@ int lfs_remove(lfs_t *lfs, const char *path); // If the destination is a directory, the directory must be empty. // // Returns a negative error code on failure. -int lfs_rename(lfs_t *lfs, const char *oldpath, const char *newpath); +int lfs_rename_raw(lfs_t *lfs, const char *oldpath, const char *newpath); // Find info about a file or directory // // Fills out the info structure, based on the specified file or directory. // Returns a negative error code on failure. -int lfs_stat(lfs_t *lfs, const char *path, struct lfs_info *info); +int lfs_stat_raw(lfs_t *lfs, const char *path, struct lfs_info *info); // Get a custom attribute // @@ -458,7 +469,7 @@ int lfs_stat(lfs_t *lfs, const char *path, struct lfs_info *info); // Note, the returned size is the size of the attribute on disk, irrespective // of the size of the buffer. This can be used to dynamically allocate a buffer // or check for existance. -lfs_ssize_t lfs_getattr(lfs_t *lfs, const char *path, +lfs_ssize_t lfs_getattr_raw(lfs_t *lfs, const char *path, uint8_t type, void *buffer, lfs_size_t size); // Set custom attributes @@ -468,7 +479,7 @@ lfs_ssize_t lfs_getattr(lfs_t *lfs, const char *path, // implicitly created. // // Returns a negative error code on failure. -int lfs_setattr(lfs_t *lfs, const char *path, +int lfs_setattr_raw(lfs_t *lfs, const char *path, uint8_t type, const void *buffer, lfs_size_t size); // Removes a custom attribute @@ -476,7 +487,7 @@ int lfs_setattr(lfs_t *lfs, const char *path, // If an attribute is not found, nothing happens. // // Returns a negative error code on failure. -int lfs_removeattr(lfs_t *lfs, const char *path, uint8_t type); +int lfs_removeattr_raw(lfs_t *lfs, const char *path, uint8_t type); /// File operations /// @@ -487,7 +498,7 @@ int lfs_removeattr(lfs_t *lfs, const char *path, uint8_t type); // are values from the enum lfs_open_flags that are bitwise-ored together. // // Returns a negative error code on failure. -int lfs_file_open(lfs_t *lfs, lfs_file_t *file, +int lfs_file_open_raw(lfs_t *lfs, lfs_file_t *file, const char *path, int flags); // Open a file with extra configuration @@ -500,7 +511,7 @@ int lfs_file_open(lfs_t *lfs, lfs_file_t *file, // config struct must be zeroed for defaults and backwards compatibility. // // Returns a negative error code on failure. -int lfs_file_opencfg(lfs_t *lfs, lfs_file_t *file, +int lfs_file_opencfg_raw(lfs_t *lfs, lfs_file_t *file, const char *path, int flags, const struct lfs_file_config *config); @@ -510,19 +521,19 @@ int lfs_file_opencfg(lfs_t *lfs, lfs_file_t *file, // sync had been called and releases any allocated resources. // // Returns a negative error code on failure. -int lfs_file_close(lfs_t *lfs, lfs_file_t *file); +int lfs_file_close_raw(lfs_t *lfs, lfs_file_t *file); // Synchronize a file on storage // // Any pending writes are written out to storage. // Returns a negative error code on failure. -int lfs_file_sync(lfs_t *lfs, lfs_file_t *file); +int lfs_file_sync_raw(lfs_t *lfs, lfs_file_t *file); // Read data from file // // Takes a buffer and size indicating where to store the read data. // Returns the number of bytes read, or a negative error code on failure. -lfs_ssize_t lfs_file_read(lfs_t *lfs, lfs_file_t *file, +lfs_ssize_t lfs_file_read_raw(lfs_t *lfs, lfs_file_t *file, void *buffer, lfs_size_t size); // Write data to file @@ -531,38 +542,38 @@ lfs_ssize_t lfs_file_read(lfs_t *lfs, lfs_file_t *file, // actually be updated on the storage until either sync or close is called. // // Returns the number of bytes written, or a negative error code on failure. -lfs_ssize_t lfs_file_write(lfs_t *lfs, lfs_file_t *file, +lfs_ssize_t lfs_file_write_raw(lfs_t *lfs, lfs_file_t *file, const void *buffer, lfs_size_t size); // Change the position of the file // // The change in position is determined by the offset and whence flag. // Returns the new position of the file, or a negative error code on failure. -lfs_soff_t lfs_file_seek(lfs_t *lfs, lfs_file_t *file, +lfs_soff_t lfs_file_seek_raw(lfs_t *lfs, lfs_file_t *file, lfs_soff_t off, int whence); // Truncates the size of the file to the specified size // // Returns a negative error code on failure. -int lfs_file_truncate(lfs_t *lfs, lfs_file_t *file, lfs_off_t size); +int lfs_file_truncate_raw(lfs_t *lfs, lfs_file_t *file, lfs_off_t size); // Return the position of the file // -// Equivalent to lfs_file_seek(lfs, file, 0, LFS_SEEK_CUR) +// Equivalent to lfs_file_seek_raw(lfs, file, 0, LFS_SEEK_CUR) // Returns the position of the file, or a negative error code on failure. -lfs_soff_t lfs_file_tell(lfs_t *lfs, lfs_file_t *file); +lfs_soff_t lfs_file_tell_raw(lfs_t *lfs, lfs_file_t *file); // Change the position of the file to the beginning of the file // -// Equivalent to lfs_file_seek(lfs, file, 0, LFS_SEEK_SET) +// Equivalent to lfs_file_seek_raw(lfs, file, 0, LFS_SEEK_SET) // Returns a negative error code on failure. -int lfs_file_rewind(lfs_t *lfs, lfs_file_t *file); +int lfs_file_rewind_raw(lfs_t *lfs, lfs_file_t *file); // Return the size of the file // -// Similar to lfs_file_seek(lfs, file, 0, LFS_SEEK_END) +// Similar to lfs_file_seek_raw(lfs, file, 0, LFS_SEEK_END) // Returns the size of the file, or a negative error code on failure. -lfs_soff_t lfs_file_size(lfs_t *lfs, lfs_file_t *file); +lfs_soff_t lfs_file_size_raw(lfs_t *lfs, lfs_file_t *file); /// Directory operations /// @@ -570,26 +581,26 @@ lfs_soff_t lfs_file_size(lfs_t *lfs, lfs_file_t *file); // Create a directory // // Returns a negative error code on failure. -int lfs_mkdir(lfs_t *lfs, const char *path); +int lfs_mkdir_raw(lfs_t *lfs, const char *path); // Open a directory // // Once open a directory can be used with read to iterate over files. // Returns a negative error code on failure. -int lfs_dir_open(lfs_t *lfs, lfs_dir_t *dir, const char *path); +int lfs_dir_open_raw(lfs_t *lfs, lfs_dir_t *dir, const char *path); // Close a directory // // Releases any allocated resources. // Returns a negative error code on failure. -int lfs_dir_close(lfs_t *lfs, lfs_dir_t *dir); +int lfs_dir_close_raw(lfs_t *lfs, lfs_dir_t *dir); // Read an entry in the directory // // Fills out the info structure, based on the specified file or directory. // Returns a positive value on success, 0 at the end of directory, // or a negative error code on failure. -int lfs_dir_read(lfs_t *lfs, lfs_dir_t *dir, struct lfs_info *info); +int lfs_dir_read_raw(lfs_t *lfs, lfs_dir_t *dir, struct lfs_info *info); // Change the position of the directory // @@ -597,7 +608,7 @@ int lfs_dir_read(lfs_t *lfs, lfs_dir_t *dir, struct lfs_info *info); // an absolute offset in the directory seek. // // Returns a negative error code on failure. -int lfs_dir_seek(lfs_t *lfs, lfs_dir_t *dir, lfs_off_t off); +int lfs_dir_seek_raw(lfs_t *lfs, lfs_dir_t *dir, lfs_off_t off); // Return the position of the directory // @@ -605,12 +616,12 @@ int lfs_dir_seek(lfs_t *lfs, lfs_dir_t *dir, lfs_off_t off); // sense, but does indicate the current position in the directory iteration. // // Returns the position of the directory, or a negative error code on failure. -lfs_soff_t lfs_dir_tell(lfs_t *lfs, lfs_dir_t *dir); +lfs_soff_t lfs_dir_tell_raw(lfs_t *lfs, lfs_dir_t *dir); // Change the position of the directory to the beginning of the directory // // Returns a negative error code on failure. -int lfs_dir_rewind(lfs_t *lfs, lfs_dir_t *dir); +int lfs_dir_rewind_raw(lfs_t *lfs, lfs_dir_t *dir); /// Filesystem-level filesystem operations @@ -621,7 +632,7 @@ int lfs_dir_rewind(lfs_t *lfs, lfs_dir_t *dir); // size may be larger than the filesystem actually is. // // Returns the number of allocated blocks, or a negative error code on failure. -lfs_ssize_t lfs_fs_size(lfs_t *lfs); +lfs_ssize_t lfs_fs_size_raw(lfs_t *lfs); // Traverse through all blocks in use by the filesystem // @@ -630,12 +641,12 @@ lfs_ssize_t lfs_fs_size(lfs_t *lfs); // blocks are in use or how much of the storage is available. // // Returns a negative error code on failure. -int lfs_fs_traverse(lfs_t *lfs, int (*cb)(void*, lfs_block_t), void *data); +int lfs_fs_traverse_raw(lfs_t *lfs, int (*cb)(void*, lfs_block_t), void *data); #ifdef LFS_MIGRATE // Attempts to migrate a previous version of littlefs // -// Behaves similarly to the lfs_format function. Attempts to mount +// Behaves similarly to the lfs_format_raw function. Attempts to mount // the previous version of littlefs and update the filesystem so it can be // mounted with the current version of littlefs. // @@ -644,7 +655,106 @@ int lfs_fs_traverse(lfs_t *lfs, int (*cb)(void*, lfs_block_t), void *data); // be zeroed for defaults and backwards compatibility. // // Returns a negative error code on failure. -int lfs_migrate(lfs_t *lfs, const struct lfs_config *cfg); +int lfs_migrate_raw(lfs_t *lfs, const struct lfs_config *cfg); +#endif + +#if LFS_THREAD_SAFE + +int lfs_format_ts(lfs_t *lfs, const struct lfs_config *config); +int lfs_mount_ts(lfs_t *lfs, const struct lfs_config *config); +int lfs_unmount_ts(lfs_t *lfs); +int lfs_remove_ts(lfs_t *lfs, const char *path); +int lfs_rename_ts(lfs_t *lfs, const char *oldpath, const char *newpath); +int lfs_stat_ts(lfs_t *lfs, const char *path, struct lfs_info *info); +lfs_ssize_t lfs_getattr_ts(lfs_t *lfs, const char *path, uint8_t type, void *buffer, lfs_size_t size); +int lfs_setattr_ts(lfs_t *lfs, const char *path, uint8_t type, const void *buffer, lfs_size_t size); +int lfs_removeattr_ts(lfs_t *lfs, const char *path, uint8_t type); +int lfs_file_open_ts(lfs_t *lfs, lfs_file_t *file, const char *path, int flags); +int lfs_file_opencfg_ts(lfs_t *lfs, lfs_file_t *file, const char *path, int flags, const struct lfs_file_config *config); +int lfs_file_close_ts(lfs_t *lfs, lfs_file_t *file); +int lfs_file_sync_ts(lfs_t *lfs, lfs_file_t *file); +lfs_ssize_t lfs_file_read_ts(lfs_t *lfs, lfs_file_t *file, void *buffer, lfs_size_t size); +lfs_ssize_t lfs_file_write_ts(lfs_t *lfs, lfs_file_t *file, const void *buffer, lfs_size_t size); +lfs_soff_t lfs_file_seek_ts(lfs_t *lfs, lfs_file_t *file, lfs_soff_t off, int whence); +int lfs_file_truncate_ts(lfs_t *lfs, lfs_file_t *file, lfs_off_t size); +lfs_soff_t lfs_file_tell_ts(lfs_t *lfs, lfs_file_t *file); +int lfs_file_rewind_ts(lfs_t *lfs, lfs_file_t *file); +lfs_soff_t lfs_file_size_ts(lfs_t *lfs, lfs_file_t *file); +int lfs_mkdir_ts(lfs_t *lfs, const char *path); +int lfs_dir_open_ts(lfs_t *lfs, lfs_dir_t *dir, const char *path); +int lfs_dir_close_ts(lfs_t *lfs, lfs_dir_t *dir); +int lfs_dir_read_ts(lfs_t *lfs, lfs_dir_t *dir, struct lfs_info *info); +int lfs_dir_seek_ts(lfs_t *lfs, lfs_dir_t *dir, lfs_off_t off); +lfs_soff_t lfs_dir_tell_ts(lfs_t *lfs, lfs_dir_t *dir); +int lfs_dir_rewind_ts(lfs_t *lfs, lfs_dir_t *dir); +lfs_ssize_t lfs_fs_size_ts(lfs_t *lfs); +int lfs_fs_traverse_ts(lfs_t *lfs, int (*cb)(void*, lfs_block_t), void *data); +int lfs_migrate_ts(lfs_t *lfs, const struct lfs_config *cfg); + +#define lfs_format lfs_format_ts +#define lfs_mount lfs_mount_ts +#define lfs_unmount lfs_unmount_ts +#define lfs_remove lfs_remove_ts +#define lfs_rename lfs_rename_ts +#define lfs_stat lfs_stat_ts +#define lfs_getattr lfs_getattr_ts +#define lfs_setattr lfs_setattr_ts +#define lfs_removeattr lfs_removeattr_ts +#define lfs_file_open lfs_file_open_ts +#define lfs_file_opencfg lfs_file_opencfg_ts +#define lfs_file_close lfs_file_close_ts +#define lfs_file_sync lfs_file_sync_ts +#define lfs_file_read lfs_file_read_ts +#define lfs_file_write lfs_file_write_ts +#define lfs_file_seek lfs_file_seek_ts +#define lfs_file_truncate lfs_file_truncate_ts +#define lfs_file_tell lfs_file_tell_ts +#define lfs_file_rewind lfs_file_rewind_ts +#define lfs_file_size lfs_file_size_ts +#define lfs_mkdir lfs_mkdir_ts +#define lfs_dir_open lfs_dir_open_ts +#define lfs_dir_close lfs_dir_close_ts +#define lfs_dir_read lfs_dir_read_ts +#define lfs_dir_seek lfs_dir_seek_ts +#define lfs_dir_tell lfs_dir_tell_ts +#define lfs_dir_rewind lfs_dir_rewind_ts +#define lfs_fs_size lfs_fs_size_ts +#define lfs_fs_traverse lfs_fs_traverse_ts +#define lfs_migrate lfs_migrate_ts + +#else + +#define lfs_format lfs_format_raw +#define lfs_mount lfs_mount_raw +#define lfs_unmount lfs_unmount_raw +#define lfs_remove lfs_remove_raw +#define lfs_rename lfs_rename_raw +#define lfs_stat lfs_stat_raw +#define lfs_getattr lfs_getattr_raw +#define lfs_setattr lfs_setattr_raw +#define lfs_removeattr lfs_removeattr_raw +#define lfs_file_open lfs_file_open_raw +#define lfs_file_opencfg lfs_file_opencfg_raw +#define lfs_file_close lfs_file_close_raw +#define lfs_file_sync lfs_file_sync_raw +#define lfs_file_read lfs_file_read_raw +#define lfs_file_write lfs_file_write_raw +#define lfs_file_seek lfs_file_seek_raw +#define lfs_file_truncate lfs_file_truncate_raw +#define lfs_file_tell lfs_file_tell_raw +#define lfs_file_rewind lfs_file_rewind_raw +#define lfs_file_size lfs_file_size_raw +#define lfs_mkdir lfs_mkdir_raw +#define lfs_dir_open lfs_dir_open_raw +#define lfs_dir_close lfs_dir_close_raw +#define lfs_dir_read lfs_dir_read_raw +#define lfs_dir_seek lfs_dir_seek_raw +#define lfs_dir_tell lfs_dir_tell_raw +#define lfs_dir_rewind lfs_dir_rewind_raw +#define lfs_fs_size lfs_fs_size_raw +#define lfs_fs_traverse lfs_fs_traverse_raw +#define lfs_migrate lfs_migrate_raw + #endif diff --git a/lfs_util.h b/lfs_util.h index dbb4c5ba..2db2cba9 100644 --- a/lfs_util.h +++ b/lfs_util.h @@ -43,6 +43,10 @@ extern "C" { #endif +// Enables thread-safe wrappers using the lock/unlock callbacks in lfs_config +#ifndef LFS_THREAD_SAFE +#define LFS_THREAD_SAFE 0 +#endif // Macros, may be replaced by system specific wrappers. Arguments to these // macros must not have side-effects as the macros can be removed for a smaller