Whamcloud - gitweb
LU-13735 lnet: Loosen restrictions on LNet Health params 28/39228/2
authorChris Horn <chris.horn@hpe.com>
Mon, 22 Jun 2020 15:21:42 +0000 (10:21 -0500)
committerOleg Drokin <green@whamcloud.com>
Tue, 8 Sep 2020 18:08:34 +0000 (18:08 +0000)
commitb8ff97611717d6f4cafd26d1e7e418fd745f5297
tree40bb9758b0fc0d20e1fe9cc52efdc3d811457d62
parentb9d7f8e07ad68a51b4fcc09babe48a68fcbd6041
LU-13735 lnet: Loosen restrictions on LNet Health params

The functions that set various LNet Health related parameters require
that the parameters be set in a specific order depending on whether
health is enabled or disabled. This is not user-friendly.
 - Don't overwrite lnet_transaction_timeout when health is being
   enabled or disabled.
 - Don't overwrite lnet_retry_count when health is being enabled
   (still set it to zero when health is disabled).
 - Allow lnet_retry_count to be set to 0 when health is disabled
 - Correct off-by-one error in transaction_to_set() to ensure
   lnet_transaction_timeout is greater than lnet_retry_count

HPE-bug-id: LUS-8995
Signed-off-by: Chris Horn <chris.horn@hpe.com>
Change-Id: Ic8ca7862543fc667fdf85844e05146c78bf48cd1
Reviewed-on: https://review.whamcloud.com/39228
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Neil Brown <neilb@suse.de>
Reviewed-by: Serguei Smirnov <ssmirnov@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
lnet/lnet/api-ni.c