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

A new non volatile storage system #77930

Merged
merged 4 commits into from
Oct 24, 2024
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
215 changes: 215 additions & 0 deletions include/zephyr/fs/zms.h
Original file line number Diff line number Diff line change
@@ -0,0 +1,215 @@
/* ZMS: Zephyr Memory Storage
*
* Copyright (c) 2024 BayLibre SAS
*
* SPDX-License-Identifier: Apache-2.0
*/
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
#ifndef ZEPHYR_INCLUDE_FS_ZMS_H_
#define ZEPHYR_INCLUDE_FS_ZMS_H_

#include <zephyr/drivers/flash.h>
#include <sys/types.h>
#include <zephyr/kernel.h>
#include <zephyr/device.h>
#include <zephyr/toolchain.h>
tomi-font marked this conversation as resolved.
Show resolved Hide resolved

#ifdef __cplusplus
extern "C" {
#endif

/**
* @brief Zephyr Memory Storage (ZMS)
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
* @defgroup zms Zephyr Memory Storage (ZMS)
* @ingroup file_system_storage
* @{
* @}
*/

/**
* @brief Zephyr Memory Storage Data Structures
* @defgroup zms_data_structures Zephyr Memory Storage Data Structures
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
* @ingroup zms
* @{
*/
tomi-font marked this conversation as resolved.
Show resolved Hide resolved

/**
* @brief Zephyr Memory Storage File system structure
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
*/
struct zms_fs {
/** File system offset in flash **/
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
off_t offset;
/** Allocation table entry write address.
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
* Addresses are stored as uint64_t:
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
* - high 4 bytes correspond to the sector
* - low 4 bytes are the offset in the sector
*/
uint64_t ate_wra;
/** Data write address */
uint64_t data_wra;
/** Storage system is split into sectors, each sector size must be multiple of erase-blocks
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
* if the device has erase capabilities
*/
uint32_t sector_size;
/** Number of sectors in the file system */
uint32_t sector_count;
/** Current cycle counter of the active sector (pointed by ate_wra)*/
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
uint8_t sector_cycle;
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
/** Flag indicating if the file system is initialized */
bool ready;
/** Mutex */
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
struct k_mutex zms_lock;
/** Flash device runtime structure */
const struct device *flash_device;
/** Flash memory parameters structure */
const struct flash_parameters *flash_parameters;
carlescufi marked this conversation as resolved.
Show resolved Hide resolved
/** Size of an Allocation Table Entry */
size_t ate_size;
#if CONFIG_ZMS_LOOKUP_CACHE
/** Lookup table used to cache ATE address of a written ID */
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
uint64_t lookup_cache[CONFIG_ZMS_LOOKUP_CACHE_SIZE];
#endif
};

/**
* @}
*/

/**
* @brief Zephyr Memory Storage APIs
* @defgroup zms_high_level_api Zephyr Memory Storage APIs
* @ingroup zms
* @{
*/

/**
* @brief Mount a ZMS file system onto the device specified in @p fs.
*
* @param fs Pointer to file system
* @retval 0 Success
* @retval -ERRNO errno code if error
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
*/
int zms_mount(struct zms_fs *fs);

/**
* @brief Clear the ZMS file system from device.
*
* @param fs Pointer to file system
* @retval 0 Success
* @retval -ERRNO errno code if error
*/
int zms_clear(struct zms_fs *fs);

/**
* @brief Write an entry to the file system.
*
* @note When @p len parameter is equal to @p 0 then entry is effectively removed (it is
carlescufi marked this conversation as resolved.
Show resolved Hide resolved
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
* equivalent to calling of zms_delete). It is not possible to distinguish between a deleted
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
* entry and an entry with data of length 0.
*
* @param fs Pointer to file system
* @param id Id of the entry to be written
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
* @param data Pointer to the data to be written
* @param len Number of bytes to be written (maximum 64 KB)
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
*
* @return Number of bytes written. On success, it will be equal to the number of bytes requested
* to be written. When a rewrite of the same data already stored is attempted, nothing is written
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
* to flash, thus 0 is returned. On error, returns negative value of errno.h defined error codes.
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
*/
ssize_t zms_write(struct zms_fs *fs, uint32_t id, const void *data, size_t len);

/**
* @brief Delete an entry from the file system
*
* @param fs Pointer to file system
* @param id Id of the entry to be deleted
* @retval 0 Success
* @retval -ERRNO errno code if error
*/
int zms_delete(struct zms_fs *fs, uint32_t id);

/**
* @brief Read an entry from the file system.
*
* @param fs Pointer to file system
* @param id Id of the entry to be read
* @param data Pointer to data buffer
* @param len Number of bytes to be read (or size of the allocated read buffer)
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
*
* @return Number of bytes read. On success, it will be equal to the number of bytes requested
* to be read. When the return value is less than the number of bytes requested to read this
* indicates that ATE contain less data than requested. On error, returns negative value of
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
* errno.h defined error codes.
*/
ssize_t zms_read(struct zms_fs *fs, uint32_t id, void *data, size_t len);

/**
* @brief Read a history entry from the file system.
*
* @param fs Pointer to file system
* @param id Id of the entry to be read
* @param data Pointer to data buffer
* @param len Number of bytes to be read
* @param cnt History counter: 0: latest entry, 1: one before latest ...
*
* @return Number of bytes read. On success, it will be equal to the number of bytes requested
carlescufi marked this conversation as resolved.
Show resolved Hide resolved
* to be read. When the return value is larger than the number of bytes requested to read this
* indicates not all bytes were read, and more data is available. On error, returns negative
* value of errno.h defined error codes.
*/
ssize_t zms_read_hist(struct zms_fs *fs, uint32_t id, void *data, size_t len, uint32_t cnt);

/**
* @brief Gets the data size that is stored in an entry with a given id
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
*
* @param fs Pointer to file system
* @param id Id of the entry that we want to get its data length
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
*
* @return Data length contained in the ATE. On success, it will be equal to the number of bytes
* in the ATE. On error, returns negative value of errno.h defined error codes.
*/
ssize_t zms_get_data_length(struct zms_fs *fs, uint32_t id);
/**
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
* @brief Calculate the available free space in the file system.
*
* @param fs Pointer to file system
*
* @return Number of bytes free. On success, it will be equal to the number of bytes that can
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
* still be written to the file system.
* Calculating the free space is a time consuming operation, especially on spi flash.
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
* On error, returns negative value of errno.h defined error codes.
*/
ssize_t zms_calc_free_space(struct zms_fs *fs);

/**
* @brief Tell how many contiguous free space remains in the currently active ZMS sector.
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
*
* @param fs Pointer to the file system.
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
*
* @return Number of free bytes.
*/
size_t zms_sector_max_data_size(struct zms_fs *fs);
tomi-font marked this conversation as resolved.
Show resolved Hide resolved

/**
* @brief Close the currently active sector and switch to the next one.
*
* @note The garbage collector is called on the new sector.
*
* @warning This routine is made available for specific use cases.
* It collides with the ZMS goal of avoiding any unnecessary flash erase operations.
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
* Using this routine extensively can result in premature failure of the flash device.
*
* @param fs Pointer to the file system.
*
* @return 0 on success. On error, returns negative value of errno.h defined error codes.
*/
int zms_sector_use_next(struct zms_fs *fs);
rghaddab marked this conversation as resolved.
Show resolved Hide resolved

/**
* @}
*/

#ifdef __cplusplus
}
#endif

#endif /* ZEPHYR_INCLUDE_FS_ZMS_H_ */
1 change: 1 addition & 0 deletions subsys/fs/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ endif()

add_subdirectory_ifdef(CONFIG_FCB ./fcb)
add_subdirectory_ifdef(CONFIG_NVS ./nvs)
add_subdirectory_ifdef(CONFIG_ZMS ./zms)

if(CONFIG_FUSE_FS_ACCESS)
zephyr_library_named(FS_FUSE)
Expand Down
1 change: 1 addition & 0 deletions subsys/fs/Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -110,5 +110,6 @@ endif # FILE_SYSTEM

rsource "fcb/Kconfig"
rsource "nvs/Kconfig"
rsource "zms/Kconfig"

endmenu
3 changes: 3 additions & 0 deletions subsys/fs/zms/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
#SPDX-License-Identifier: Apache-2.0

zephyr_sources(zms.c)
57 changes: 57 additions & 0 deletions subsys/fs/zms/Kconfig
Original file line number Diff line number Diff line change
@@ -0,0 +1,57 @@
#Zephyr Memory Storage ZMS

#Copyright (c) 2024 BayLibre SAS

#SPDX-License-Identifier: Apache-2.0
tomi-font marked this conversation as resolved.
Show resolved Hide resolved

config ZMS
bool "Zephyr Memory Storage"
select CRC
help
Enable support of Zephyr Memory Storage.
tomi-font marked this conversation as resolved.
Show resolved Hide resolved

if ZMS

config ZMS_LOOKUP_CACHE
bool "ZMS lookup cache"
help
Enable ZMS cache to reduce the ZMS data lookup time.
Each cache entry holds an address of the most recent allocation
table entry (ATE) for all ZMS IDs that fall into that cache position.

config ZMS_LOOKUP_CACHE_SIZE
int "ZMS Storage lookup cache size"
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
default 128
range 1 65536
depends on ZMS_LOOKUP_CACHE
help
Number of entries in ZMS lookup cache.
It is recommended that it should be a power of 2.
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
Every additional entry in cache will add 8 bytes in RAM
tomi-font marked this conversation as resolved.
Show resolved Hide resolved

config ZMS_DATA_CRC
bool "ZMS DATA CRC"
tomi-font marked this conversation as resolved.
Show resolved Hide resolved
help
Enables DATA CRC
tomi-font marked this conversation as resolved.
Show resolved Hide resolved

config ZMS_CUSTOM_BLOCK_SIZE
bool "Custom buffer size used by ZMS for reads and writes"
help
ZMS uses internal buffers to read/write and compare stored data.
Increasing the size of these buffers should be done carefully in order to not
overflow the stack.
Increasing this buffer means as well that ZMS could work with storage devices
that have larger write-block-size which decreases ZMS performance
tomi-font marked this conversation as resolved.
Show resolved Hide resolved

config ZMS_MAX_BLOCK_SIZE
int "ZMS internal buffer size"
default 32
depends on ZMS_CUSTOM_BLOCK_SIZE
help
Changes the internal buffer size of ZMS

module = ZMS
module-str = zms
source "subsys/logging/Kconfig.template.log_config"

endif # ZMS
Loading