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

libmultipath: fix max_sectors_kb on adding path #69

Closed
wants to merge 1 commit into from

Conversation

ajmes
Copy link
Contributor

@ajmes ajmes commented Jul 19, 2023

A user can change the value of max_sectors_kb on the multipath device and its path devices.
So when a path is deleted and then re-added, its value will not be the same as the multipath device. In that case the IOs could be mis-sized.

This was originally hit on Lustre FS on:
https://jira.whamcloud.com/browse/LU-9551

On reload, this patch re-apply max_sectors_kb value of the multipath device on its path devices.

A user can change the value of max_sectors_kb on the multipath device
and its path devices.
So when a path is deleted and then re-added, its value will not be the
same as the multipath device. In that case the IOs could be mis-sized.

This was originally hit on Lustre FS on:
https://jira.whamcloud.com/browse/LU-9551

On reload, this patch re-apply max_sectors_kb value of the multipath
device on its path devices.

Signed-off-by: Etienne AUJAMES <[email protected]>
@ajmes
Copy link
Contributor Author

ajmes commented Jul 19, 2023

More information in: https://jira.whamcloud.com/browse/LU-12029

@bmarzins
Copy link
Contributor

bmarzins commented Aug 3, 2023

This looks good to me, but in general, pull requests to multipath-tools are handled using an email based workflow. Can you please send these patches to the dm-devel mailing list, and CC mwilck at suse.com and bmarzins at redhat.com? Thanks.

@mwilck mwilck mentioned this pull request Sep 6, 2023
@mwilck
Copy link
Contributor

mwilck commented Sep 6, 2023

Fixed in #68.

@mwilck mwilck closed this Sep 6, 2023
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.

3 participants