-
Notifications
You must be signed in to change notification settings - Fork 38
mdadm --create supports --logical-block-size option #216
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
base: main
Are you sure you want to change the base?
mdadm --create supports --logical-block-size option #216
Conversation
b0c8ca4 to
4b2d71b
Compare
mtkaczyk
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, bunch of questions :)
The kernel already supports configuring logical block size, therefore a new
parameter should be added to allow specifying the logical block size when
creating a RAID device.
Please mention when it was added. Should we check kernel version to determine if it is supported?
If it is fresh change, what will happen if kernel is not supporting it?
Please also update README.
What about recovery path? if change for one disk fail, should we restore previous the logical block size for all?
What will happen if we will try to set same block size twice?
LGTM,. but we need README do be updated!
| ClusterConfirm, | ||
| WriteJournal, | ||
| ConsistencyPolicy, | ||
| LogicalBlockSize, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand the special_options at all concept so please care to explain why it is added here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My understanding is that these options are not general options but specific to certain modes. LogicalBlocksize only takes effect when used with --create to create something. Then, we need to define one value through O(Create, xxx) to reach the corresponding branch.
|
4414325 to
e65c9f1
Compare
The kernel block branch has started supporting the configuration of logical block size after mergeing commit 62ed1b582246 (md:allow configuring logical block size), therefore a new parameter should be added to allow specifying the logical block size when creating a RAID device. Links: https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux.git/commit/?h=for-6.19/block&id=62ed1b58224 Signed-off-by: Wu Guanghao <[email protected]>
e65c9f1 to
0f38140
Compare
Uh oh!
There was an error while loading. Please reload this page.