feat(fs): complete ext2 implementation with directory ops and large file support #71
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ile support This commit completes the ext2 filesystem implementation with: Directory Operations: - mkdir: Create new directories with proper . and .. entries - rmdir: Remove empty directories with ENOTEMPTY validation Link Operations: - link: Create hard links with proper link count management - symlink: Create symbolic links (fast symlinks for targets ≤60 bytes) - readlink: Read symbolic link targets Large File Support: - Double indirect blocks (files up to ~64MB with 1KB blocks) - Triple indirect blocks (files up to ~4TB with 4KB blocks) - Extended set_block_num() to handle all indirect levels Block Management: - Proper block deallocation on unlink - Recursive traversal of indirect blocks for cleanup - increment_free_blocks() and increment_free_inodes() for superblock updates New syscall numbers: - Link (86), Symlink (88), Readlink (89) New errno constants: - EPERM (1), EBUSY (16), ENOTEMPTY (39) All filesystem tests pass (96/98 boot stages, failures unrelated to fs). 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add fs_directory_test.rs with 9 tests for mkdir/rmdir: - Create/verify directory, permission verification via fstat - EEXIST, ENOENT (missing parent), ENOTEMPTY error handling - Add fs_link_test.rs with 11 tests for link/symlink/readlink: - Hard link creation, survives original deletion - Symlink creation, dangling symlinks, long targets (>60 bytes) - ENOENT, EEXIST, EPERM, EINVAL error handling - Fix VirtIO block driver DMA buffer exhaustion: - Pre-allocate 3 DMA buffers during device init - Reuse buffers for all I/O operations - Prevents frame allocator exhaustion under heavy I/O - Increase kernel heap from 16MB to 32MB: - Bump allocator only reclaims when ALL allocations freed - 43+ test processes accumulate memory across test suite - Fix fs_large_file_test.rs: - Update stale seek positions to valid values within 512KB - Document triple indirect blocks limitation (requires >64MB file) - Fix fs_rename_test.rs: - Strictly require EISDIR errno (was accepting any error) - Convert ext2 Vec allocations to stack buffers: - free_indirect_block, free_double_indirect_block - create_directory, write_directory_data, read_symlink_target Boot stages: 100/100 passing 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR completes the ext2 filesystem implementation for Breenix:
.and..entriesNew Syscalls
Files Changed (12 files, +1629 lines)
kernel/src/fs/ext2/- Core filesystem operationskernel/src/syscall/- Syscall handlers and dispatchlibs/libbreenix/- Userspace syscall wrappersuserspace/tests/- Updated large file testTest plan
🤖 Generated with Claude Code