Re: [PATCH] multipath-tools: replace round-robin path selector with the default for NFINIDAT/InfiniBox

[Xose Vazquez Perez <xose.vazquez@gmail.com>]

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 arnony@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.