mpathpersist: initialize data structures only once
authorMartin Wilck <mwilck@suse.com>
Mon, 27 May 2019 12:59:39 +0000 (14:59 +0200)
committerChristophe Varoqui <christophe.varoqui@opensvc.com>
Wed, 3 Jul 2019 06:28:33 +0000 (08:28 +0200)
We now have the possibility to run several PR commands in a single
mpathpersist invocation. Run initialization / discovery and teardown
only once at program invocation / exit.

Signed-off-by: Martin Wilck <mwilck@suse.com>
mpathpersist/main.c

index 6e27a1c..5ad06a9 100644 (file)
@@ -375,6 +375,9 @@ static int handle_args(int argc, char * argv[], int nline)
                /* set verbosity */
                noisy = (loglevel >= 3) ? 1 : hex;
                verbose = (loglevel >= 3)? 3: loglevel;
+               ret = mpath_persistent_reserve_init_vecs(verbose);
+               if (ret != MPATH_PR_SUCCESS)
+                       goto out;
        }
 
        if ((prout_flag + prin_flag) == 0 && batch_fn == NULL)
@@ -481,7 +484,7 @@ static int handle_args(int argc, char * argv[], int nline)
                        goto out;
                }
 
-               ret = mpath_persistent_reserve_in (fd, prin_sa, resp, noisy, verbose);
+               ret = __mpath_persistent_reserve_in (fd, prin_sa, resp, noisy);
                if (ret != MPATH_PR_SUCCESS )
                {
                        fprintf (stderr, "Persistent Reserve IN command failed\n");
@@ -541,8 +544,8 @@ static int handle_args(int argc, char * argv[], int nline)
                }
 
                /* PROUT commands other than 'register and move' */
-               ret = mpath_persistent_reserve_out (fd, prout_sa, 0, prout_type,
-                               paramp, noisy, verbose);
+               ret = __mpath_persistent_reserve_out (fd, prout_sa, 0, prout_type,
+                               paramp, noisy);
                for (j = 0 ; j < num_transport; j++)
                {
                        tmp = paramp->trnptid_list[j];
@@ -581,6 +584,8 @@ out :
                free(batch_fn);
                ret = ret == 0 ? rv : ret;
        }
+       if (nline == 0)
+               mpath_persistent_reserve_free_vecs();
        return (ret >= 0) ? ret : MPATH_PR_OTHER;
 }