multipath-tools: mark unused params in signal and cleanup handlers
authorMartin Wilck <mwilck@suse.com>
Sat, 12 Oct 2019 21:28:07 +0000 (21:28 +0000)
committerChristophe Varoqui <christophe.varoqui@opensvc.com>
Mon, 2 Mar 2020 08:17:29 +0000 (09:17 +0100)
These parameters are all rightfully unused, yet need to be declared
because of the function prototypes.

Signed-off-by: Martin Wilck <mwilck@suse.com>
libmultipath/file.c
libmultipath/foreign.c
libmultipath/io_err_stat.c
mpathpersist/main.c
multipath/main.c
multipathd/dmevents.c
multipathd/main.c

index 8727f16..5cb06e1 100644 (file)
@@ -74,7 +74,7 @@ ensure_directories_exist(const char *str, mode_t dir_mode)
 }
 
 static void
-sigalrm(int sig)
+sigalrm(__attribute__((unused)) int sig)
 {
        /* do nothing */
 }
index 9ef8ad1..f05db16 100644 (file)
@@ -50,7 +50,7 @@ static void wrlock_foreigns(void)
        pthread_rwlock_wrlock(&foreign_lock);
 }
 
-static void unlock_foreigns(void *unused)
+static void unlock_foreigns(__attribute__((unused)) void *unused)
 {
        pthread_rwlock_unlock(&foreign_lock);
 }
index 554b777..6b3a3e1 100644 (file)
@@ -84,7 +84,7 @@ io_context_t  ioctx;
 
 static void cancel_inflight_io(struct io_err_stat_path *pp);
 
-static void rcu_unregister(void *param)
+static void rcu_unregister(__attribute__((unused)) void *param)
 {
        rcu_unregister_thread();
 }
@@ -689,7 +689,7 @@ static void cleanup_unlock(void *arg)
        pthread_mutex_unlock((pthread_mutex_t*) arg);
 }
 
-static void cleanup_exited(void *arg)
+static void cleanup_exited(__attribute__((unused)) void *arg)
 {
        uatomic_set(&io_err_thread_running, 0);
 }
index 278b8d5..5676166 100644 (file)
@@ -50,7 +50,7 @@ struct config *get_multipath_config(void)
        return multipath_conf;
 }
 
-void put_multipath_config(void * arg)
+void put_multipath_config(__attribute__((unused)) void * arg)
 {
        /* Noop for now */
 }
index 4f4d8e8..b3f8c99 100644 (file)
@@ -86,7 +86,7 @@ struct config *get_multipath_config(void)
        return multipath_conf;
 }
 
-void put_multipath_config(void *arg)
+void put_multipath_config(__attribute__((unused)) void *arg)
 {
        /* Noop for now */
 }
index 0034892..b22b47d 100644 (file)
@@ -370,12 +370,12 @@ static int dmevent_loop (void)
        return -1; /* never reach there */
 }
 
-static void rcu_unregister(void *param)
+static void rcu_unregister(__attribute__((unused)) void *param)
 {
        rcu_unregister_thread();
 }
 
-void *wait_dmevents (void *unused)
+void *wait_dmevents (__attribute__((unused)) void *unused)
 {
        int r;
 
index 34a5768..ef128a3 100644 (file)
@@ -215,7 +215,7 @@ static void do_sd_notify(enum daemon_status old_state,
 }
 #endif
 
-static void config_cleanup(void *arg)
+static void config_cleanup(__attribute__((unused)) void *arg)
 {
        pthread_mutex_unlock(&config_lock);
 }
@@ -306,7 +306,7 @@ struct config *get_multipath_config(void)
        return rcu_dereference(multipath_conf);
 }
 
-void put_multipath_config(void *arg)
+void put_multipath_config(__attribute__((unused)) void *arg)
 {
        rcu_read_unlock();
 }
@@ -1503,7 +1503,7 @@ out:
        return r;
 }
 
-static void rcu_unregister(void *param)
+static void rcu_unregister(__attribute__((unused)) void *param)
 {
        rcu_unregister_thread();
 }
@@ -2685,25 +2685,25 @@ handle_signals(bool nonfatal)
 }
 
 static void
-sighup (int sig)
+sighup(__attribute__((unused)) int sig)
 {
        reconfig_sig = 1;
 }
 
 static void
-sigend (int sig)
+sigend(__attribute__((unused)) int sig)
 {
        exit_sig = 1;
 }
 
 static void
-sigusr1 (int sig)
+sigusr1(__attribute__((unused)) int sig)
 {
        log_reset_sig = 1;
 }
 
 static void
-sigusr2 (int sig)
+sigusr2(__attribute__((unused)) int sig)
 {
        condlog(3, "SIGUSR2 received");
 }