On 5/12/25 7:11 PM, Benjamin Marzinski wrote:
On Fri, May 09, 2025 at 04:22:40PM +0200, Martin Wilck wrote:
On Fri, 2025-05-09 at 16:08 +0200, Martin Wilck wrote:
This code is from 55da608 ("libmultipath: update INFINIDAT builtin
config"), signed-off by arn...@infinidat.com. We usually don't change
defaults from explicit vendor recommendations without consent from
the
vendor. I've added Arnon to cc, maybe he wants to comment on your
suggestion.
Arnon's email address bounces. Unless we find someone else from
Infinidat to comment, I think we should keep the current default. Users
are free to override the defaults if the aren't happy with the
performance.
I agree. This device config was set to round-robin after we switched the
default to service-time, so I think it's reasonable to assume that this
was intentional. Barring an email or updated documentation from
INFINIDAT that shows it's no longer the recommended config, I feel like
we should leave it as is.
At Infinidat support site(user required), there are new configs for SLES 15 and
RHEL 8, or above:
https://lh3.googleusercontent.com/pw/AP1GczPhlOIUXYWsuCYQJ42ORS_77xnwTmlN02ZkDU4__6jAvQ_6bBLvvcFsQ9luhSMcB887pXZOaVCrCNZUITTl8eEztD7l568BTSNHL2cB9Dh2SfAAXye7K9Tu8nkjCKMkhQM4H7LRiY4Rp0ygQqCrMmM=w0-h0
https://lh3.googleusercontent.com/pw/AP1GczMchJ6bcMIohp_g3Ik5DS6BZv_IW0iwaEXf968sJsR8fr_p3mR4ThRpmHpZE_VMnlcV8j0uuuI2kz-eoqekRCUBxyTBKS7n-4WFMsUiecq8i7nIjEuhfZFrV54DBQLDzGt6ofakAAF1L-ZcAuCWW18=w0-h0
device {
vendor "NFINIDAT"
product "InfiniBox"
path_grouping_policy "group_by_prio"
path_checker "tur"
features 0
hardware_handler "1 alua"
prio "alua"
rr_weight "priorities"
no_path_retry "queue"
rr_min_io 1
rr_min_io_rq 1
flush_on_last_del "yes"
fast_io_fail_tmo 15
dev_loss_tmo "infinity"
path_selector "service-time 0"
failback "immediate"
detect_prio "no"
user_friendly_names "no"
}
I will send a new patch.
