libmultipath: make "checkint" unsigned
authorMartin Wilck <mwilck@suse.com>
Sat, 12 Oct 2019 21:28:40 +0000 (21:28 +0000)
committerChristophe Varoqui <christophe.varoqui@opensvc.com>
Mon, 2 Mar 2020 08:19:28 +0000 (09:19 +0100)
path->checkint was unsigned already, now use this type everywhere.

Signed-off-by: Martin Wilck <mwilck@suse.com>
libmultipath/config.h
libmultipath/dict.c
libmultipath/discovery.c
libmultipath/discovery.h
libmultipath/io_err_stat.c
libmultipath/structs_vec.c
multipathd/main.c

index ffec310..2ab7b42 100644 (file)
@@ -137,8 +137,8 @@ struct config {
        int pgpolicy;
        int minio;
        int minio_rq;
-       int checkint;
-       int max_checkint;
+       unsigned int checkint;
+       unsigned int max_checkint;
        int pgfailback;
        int remove;
        int rr_weight;
index c23d525..a90690f 100644 (file)
@@ -314,7 +314,7 @@ snprint_mp_ ## option (struct config *conf, char * buff, int len,   \
 
 static int checkint_handler(struct config *conf, vector strvec)
 {
-       int rc = set_int(strvec, &conf->checkint);
+       int rc = set_uint(strvec, &conf->checkint);
 
        if (rc)
                return rc;
@@ -325,7 +325,7 @@ static int checkint_handler(struct config *conf, vector strvec)
 
 declare_def_snprint(checkint, print_int)
 
-declare_def_handler(max_checkint, set_int)
+declare_def_handler(max_checkint, set_uint)
 declare_def_snprint(max_checkint, print_int)
 
 declare_def_handler(verbosity, set_int)
index 6288c3e..4288c9f 100644 (file)
@@ -754,7 +754,7 @@ sysfs_set_nexus_loss_tmo(struct multipath *mpp, struct path *pp)
 }
 
 int
-sysfs_set_scsi_tmo (struct multipath *mpp, int checkint)
+sysfs_set_scsi_tmo (struct multipath *mpp, unsigned int checkint)
 {
        struct path *pp;
        int i;
index 8d04c2a..0516e41 100644 (file)
@@ -42,7 +42,7 @@ int alloc_path_with_pathinfo (struct config *conf, struct udev_device *udevice,
 int store_pathinfo (vector pathvec, struct config *conf,
                    struct udev_device *udevice, int flag,
                    struct path **pp_ptr);
-int sysfs_set_scsi_tmo (struct multipath *mpp, int checkint);
+int sysfs_set_scsi_tmo (struct multipath *mpp, unsigned int checkint);
 int sysfs_get_timeout(const struct path *pp, unsigned int *timeout);
 int sysfs_get_host_pci_name(const struct path *pp, char *pci_name);
 int sysfs_get_iscsi_ip_address(const struct path *pp, char *ip_address);
index 6b3a3e1..d7e296f 100644 (file)
@@ -357,7 +357,7 @@ int io_err_stat_handle_pathfail(struct path *path)
                if (path->state != PATH_DOWN) {
                        struct config *conf;
                        int oldstate = path->state;
-                       int checkint;
+                       unsigned int checkint;
 
                        conf = get_multipath_config();
                        checkint = conf->checkint;
index c43b58f..6991f9a 100644 (file)
@@ -292,7 +292,7 @@ update_multipath_strings(struct multipath *mpp, vector pathvec, int is_daemon)
 
 void enter_recovery_mode(struct multipath *mpp)
 {
-       int checkint;
+       unsigned int checkint;
        struct config *conf = get_multipath_config();
        checkint = conf->checkint;
        put_multipath_config(conf);
index 26d5aa9..bb5c1f1 100644 (file)
@@ -493,7 +493,7 @@ int update_multipath (struct vectors *vecs, char *mapname, int reset)
                        if (pp->state != PATH_DOWN) {
                                struct config *conf;
                                int oldstate = pp->state;
-                               int checkint;
+                               unsigned int checkint;
 
                                conf = get_multipath_config();
                                checkint = conf->checkint;
@@ -1985,7 +1985,7 @@ should_skip_path(struct path *pp){
  * and '0' otherwise
  */
 int
-check_path (struct vectors * vecs, struct path * pp, int ticks)
+check_path (struct vectors * vecs, struct path * pp, unsigned int ticks)
 {
        int newstate;
        int new_path_up = 0;
@@ -1993,7 +1993,8 @@ check_path (struct vectors * vecs, struct path * pp, int ticks)
        int add_active;
        int disable_reinstate = 0;
        int oldchkrstate = pp->chkrstate;
-       int retrigger_tries, checkint, max_checkint, verbosity;
+       int retrigger_tries, verbosity;
+       unsigned int checkint, max_checkint;
        struct config *conf;
        int marginal_pathgroups, marginal_changed = 0;
        int ret;
@@ -2333,7 +2334,8 @@ checkerloop (void *ap)
 
        while (1) {
                struct timespec diff_time, start_time, end_time;
-               int num_paths = 0, ticks = 0, strict_timing, rc = 0;
+               int num_paths = 0, strict_timing, rc = 0;
+               unsigned int ticks = 0;
 
                get_monotonic_time(&start_time);
                if (start_time.tv_sec && last_time.tv_sec) {