Skip to content

feat(filesystem): add init in memory fatfs - #568

Merged
Koichi98 merged 6 commits into
mainfrom
feat/init_inmemory_fatfs
Jul 8, 2025
Merged

feat(filesystem): add init in memory fatfs#568
Koichi98 merged 6 commits into
mainfrom
feat/init_inmemory_fatfs

Conversation

@Koichi98

@Koichi98Koichi98 commented Jul 4, 2025

Copy link
Copy Markdown
Contributor

Description

This PR implements an initialization function that allocates a memory block of MEMPOOL_SIZE, formats it with the FAT file system, and stores it as a global variable.

After initialization, this file system can be retrieved using a get function. It is intended that instances implementing AsyncFileSystem will hold this internally.

Related links

#511

How was this PR tested?

Notes for reviewers

The initialization function is planned to be called during kernel initialization, but for now, it is only being defined and not called.

Signed-off-by: Koichi Imai <koichi.imai.2@tier4.jp>
Signed-off-by: Koichi Imai <koichi.imai.2@tier4.jp>
Signed-off-by: Koichi Imai <koichi.imai.2@tier4.jp>
@Koichi98Koichi98 self-assigned this Jul 6, 2025
Base automatically changed from fix/add_debug_faffs to mainJuly 7, 2025 09:02
Signed-off-by: Koichi <koichi.imai.2@tier4.jp>
Signed-off-by: Koichi <koichi.imai.2@tier4.jp>
@Koichi98
Koichi98 marked this pull request as ready for review July 7, 2025 23:14
@Koichi98
Koichi98 requested review from veqcc and ytakanoJuly 7, 2025 23:20
@Koichi98Koichi98 changed the title add init in memory fatfsfeat(filesystem): add init in memory fatfsJul 7, 2025
Comment threadawkernel_lib/src/file/fatfs.rs Outdated
Comment threadawkernel_lib/src/file/fatfs.rs Outdated
Signed-off-by: Koichi Imai <koichi.imai.2@tier4.jp>
@Koichi98
Koichi98 merged commit 5978691 into mainJul 8, 2025
1 check passed
@Koichi98
Koichi98 deleted the feat/init_inmemory_fatfs branch July 8, 2025 08:18
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@Koichi98@ytakano@veqcc