[PATCH] fabrics: ensure zero kato for non-persistent controllers

[George, Martin]

Ping?

On Thu, 2021-02-18 at 15:24 +0530, Martin George wrote:
> Along with ensuring a non-zero kato is passed to a
> persistent discovery controller, also ensure a zero
> kato is passed to a non-persistent discovery controller.
> 
> Signed-off-by: Martin George <marting at netapp.com>
> ---
>  fabrics.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/fabrics.c b/fabrics.c
> index 1880a5d..ed15e08 100644
> --- a/fabrics.c
> +++ b/fabrics.c
> @@ -1399,6 +1399,8 @@ static int discover_from_conf_file(const char
> *desc, char *argstr,
> 
>                 if (cfg.persistent && !cfg.keep_alive_tmo)
>                         cfg.keep_alive_tmo = NVMF_DEF_DISC_TMO;
> +               else if (!cfg.persistent && (cfg.keep_alive_tmo > 0))
> +                       cfg.keep_alive_tmo = 0;
> 
>                 if (traddr_is_hostname(&cfg)) {
>                         ret = hostname2traddr(&cfg);
> @@ -1474,6 +1476,8 @@ int fabrics_discover(const char *desc, int
> argc, char **argv, bool connect)
>         } else {
>                 if (cfg.persistent && !cfg.keep_alive_tmo)
>                         cfg.keep_alive_tmo = NVMF_DEF_DISC_TMO;
> +               else if (!cfg.persistent && (cfg.keep_alive_tmo > 0))
> +                       cfg.keep_alive_tmo = 0;
> 
>                 if (traddr_is_hostname(&cfg)) {
>                         ret = hostname2traddr(&cfg);
> --
> 2.30.0
>