Skip to content

Derive the block size from the log block size#662

Merged
crosbymichael merged 3 commits intoapple:mainfrom
dkovba:block-size
Apr 9, 2026
Merged

Derive the block size from the log block size#662
crosbymichael merged 3 commits intoapple:mainfrom
dkovba:block-size

Conversation

@dkovba
Copy link
Copy Markdown
Contributor

@dkovba dkovba commented Apr 7, 2026

Derives the block size from the log block size.

@dkovba dkovba requested a review from wlan0 April 7, 2026 20:00
@crosbymichael
Copy link
Copy Markdown
Contributor

Just for my reference, where did you find the log block size to be 2?

@dkovba
Copy link
Copy Markdown
Contributor Author

dkovba commented Apr 9, 2026

Just for my reference, where did you find the log block size to be 2?

This is what it has to be to have the block size of 4 KiB (1024 << 2 = 4 KiB). It was previously defined in:

            superblock.logBlockSize = 2
            superblock.logClusterSize = 2

@dkovba dkovba requested a review from crosbymichael April 9, 2026 19:12
@crosbymichael crosbymichael merged commit 2c4012e into apple:main Apr 9, 2026
3 checks passed
@dkovba dkovba deleted the block-size branch April 9, 2026 19:27
wlan0 pushed a commit that referenced this pull request Apr 10, 2026
Restores a configurable block size in the EXT4 formatter. 

#662 resolved a conflict
between the hardcoded log block size and a user-provided block size, but
removed the later. This PR restores the configurable block size without
creating the conflict.
Sign up for free to 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.

2 participants