Merged as commit b573878f76a893cece197b5969e54099a5fac905.
Merge rejects committed as 9c4be4c0d1b06c066098dafb5aeb914bae1f110b.
Thanks.
On Fri, Nov 18, 2016 at 9:50 PM, Bart Van Assche <bart.vanassche@xxxxxxxxxxx> wrote:
This avoids that the sparse static analyzer complains about
the "extern" keyword.
Signed-off-by: Bart Van Assche <bart.vanassche@xxxxxxxxxxx>
---
kpartx/devmapper.c | 18 ++++-----
kpartx/lopart.c | 15 +++-----
libmpathpersist/mpath_pr_ioctl.c | 2 +-
libmultipath/blacklist.c | 9 ++---
libmultipath/callout.c | 3 +-
libmultipath/checkers/cciss_tur.c | 3 +-
libmultipath/checkers/emc_clariion.c | 3 +-
libmultipath/checkers/hp_sw.c | 3 +-
libmultipath/checkers/rdac.c | 3 +-
libmultipath/checkers/tur.c | 3 +-
libmultipath/config.c | 6 +--
libmultipath/configure.c | 21 +++++-----
libmultipath/devmapper.c | 49 +++++++++--------------
libmultipath/discovery.c | 5 +--
libmultipath/dmparser.c | 8 ++--
libmultipath/hwtable.c | 3 +-
libmultipath/pgpolicies.c | 23 +++++------
libmultipath/print.c | 55 ++++++++++----------------
libmultipath/propsel.c | 75 ++++++++++++------------------------
libmultipath/structs.c | 21 ++++------
libmultipath/structs_vec.c | 49 +++++++++--------------
libmultipath/switchgroup.c | 6 +--
libmultipath/uevent.c | 12 ++----
libmultipath/util.c | 3 +-
24 files changed, 145 insertions(+), 253 deletions(-)
diff --git a/kpartx/devmapper.c b/kpartx/devmapper.c
index 474d879..2acae25 100644
--- a/kpartx/devmapper.c
+++ b/kpartx/devmapper.c
@@ -14,8 +14,7 @@
#define MAX_PREFIX_LEN 8
#define PARAMS_SIZE 1024
-extern int
-dm_prereq (char * str, int x, int y, int z)
+int dm_prereq(char * str, int x, int y, int z)
{
int r = 1;
struct dm_task *dmt;
@@ -52,8 +51,8 @@ out:
return r;
}
-extern int
-dm_simplecmd (int task, const char *name, int no_flush, uint16_t udev_flags) {
+int dm_simplecmd(int task, const char *name, int no_flush, uint16_t udev_flags)
+{
int r = 0;
int udev_wait_flag = (task == DM_DEVICE_RESUME ||
task == DM_DEVICE_REMOVE);
@@ -90,10 +89,10 @@ out:
return r;
}
-extern int
-dm_addmap (int task, const char *name, const char *target,
- const char *params, uint64_t size, int ro, const char *uuid, int part,
- mode_t mode, uid_t uid, gid_t gid) {
+int dm_addmap(int task, const char *name, const char *target,
+ const char *params, uint64_t size, int ro, const char *uuid,
+ int part, mode_t mode, uid_t uid, gid_t gid)
+{
int r = 0;
struct dm_task *dmt;
char *prefixed_uuid = NULL;
@@ -154,8 +153,7 @@ addout:
return r;
}
-extern int
-dm_map_present (char * str, char **uuid)
+int dm_map_present(char * str, char **uuid)
{
int r = 0;
struct dm_task *dmt;
diff --git a/kpartx/lopart.c b/kpartx/lopart.c
index 14af34f..2eb3f63 100644
--- a/kpartx/lopart.c
+++ b/kpartx/lopart.c
@@ -62,8 +62,7 @@ xstrdup (const char *s)
return t;
}
-extern int
-is_loop_device (const char *device)
+int is_loop_device(const char *device)
{
struct stat statbuf;
int loopmajor;
@@ -96,8 +95,7 @@ is_loop_device (const char *device)
#define SIZE(a) (sizeof(a)/sizeof(a[0]))
-extern char *
-find_loop_by_file (const char * filename)
+char *find_loop_by_file(const char *filename)
{
DIR *dir;
struct dirent *dent;
@@ -144,8 +142,7 @@ find_loop_by_file (const char * filename)
return found;
}
-extern char *
-find_unused_loop_device (void)
+char *find_unused_loop_device(void)
{
char dev[20], *next_loop_dev = NULL;
int fd, next_loop = 0, somedev = 0, someloop = 0, loop_known = 0;
@@ -231,8 +228,7 @@ find_unused_loop_device (void)
return NULL;
}
-extern int
-set_loop (const char *device, const char *file, int offset, int *loopro)
+int set_loop(const char *device, const char *file, int offset, int *loopro)
{
struct loop_info loopinfo;
int fd, ffd, mode;
@@ -284,8 +280,7 @@ set_loop (const char *device, const char *file, int offset, int *loopro)
return 0;
}
-extern int
-del_loop (const char *device)
+int del_loop(const char *device)
{
int retries = 5;
int fd;
diff --git a/libmpathpersist/mpath_pr_ioctl.c b/libmpathpersist/mpath_pr_ ioctl.c
index 8b9ac3d..31b2fe6 100644
--- a/libmpathpersist/mpath_pr_ioctl.c
+++ b/libmpathpersist/mpath_pr_ioctl.c
@@ -36,7 +36,7 @@ void decode_transport_id(struct prin_fulldescr *fdesc, unsigned char * p, int le
int get_prin_length(int rq_servact);
int mpath_isLittleEndian(void);
-extern unsigned int mpath_mx_alloc_len;
+unsigned int mpath_mx_alloc_len;
int prout_do_scsi_ioctl(char * dev, int rq_servact, int rq_scope,
unsigned int rq_type, struct prout_param_descriptor *paramp, int noisy)
diff --git a/libmultipath/blacklist.c b/libmultipath/blacklist.c
index f6c4506..d59d37e 100644
--- a/libmultipath/blacklist.c
+++ b/libmultipath/blacklist.c
@@ -13,8 +13,7 @@
#include "config.h"
#include "blacklist.h"
-extern int
-store_ble (vector blist, char * str, int origin)
+int store_ble(vector blist, char * str, int origin)
{
struct blentry * ble;
@@ -47,8 +46,7 @@ out:
}
-extern int
-alloc_ble_device (vector blist)
+int alloc_ble_device(vector blist)
{
struct blentry_device * ble = MALLOC(sizeof(struct blentry_device));
@@ -63,8 +61,7 @@ alloc_ble_device (vector blist)
return 0;
}
-extern int
-set_ble_device (vector blist, char * vendor, char * product, int origin)
+int set_ble_device(vector blist, char * vendor, char * product, int origin)
{
struct blentry_device * ble;
diff --git a/libmultipath/callout.c b/libmultipath/callout.c
index 4d1b067..dc18e02 100644
--- a/libmultipath/callout.c
+++ b/libmultipath/callout.c
@@ -137,8 +137,7 @@ int execute_program(char *path, char *value, int len)
return retval;
}
-extern int
-apply_format (char * string, char * cmd, struct path * pp)
+int apply_format(char * string, char * cmd, struct path * pp)
{
char * pos;
char * dst;
diff --git a/libmultipath/checkers/cciss_tur.c b/libmultipath/checkers/cciss_ tur.c
index a0ffffe..9d79f96 100644
--- a/libmultipath/checkers/cciss_tur.c
+++ b/libmultipath/checkers/cciss_tur.c
@@ -64,8 +64,7 @@ void libcheck_repair (struct checker * c)
return;
}
-extern int
-libcheck_check (struct checker * c)
+int libcheck_check(struct checker * c)
{
int rc;
int ret;
diff --git a/libmultipath/checkers/emc_clariion.c b/libmultipath/checkers/emc_ clariion.c
index a7b9f86..9c1ffed 100644
--- a/libmultipath/checkers/emc_clariion.c
+++ b/libmultipath/checkers/emc_clariion.c
@@ -50,8 +50,7 @@ struct emc_clariion_checker_LU_context {
int inactive_snap;
};
-extern void
-hexadecimal_to_ascii(char * wwn, char *wwnstr)
+void hexadecimal_to_ascii(char * wwn, char *wwnstr)
{
int i,j, nbl;
diff --git a/libmultipath/checkers/hp_sw.c b/libmultipath/checkers/hp_sw. c
index 0cc1111..6019c9d 100644
--- a/libmultipath/checkers/hp_sw.c
+++ b/libmultipath/checkers/hp_sw.c
@@ -128,8 +128,7 @@ do_tur (int fd, unsigned int timeout)
return 0;
}
-extern int
-libcheck_check (struct checker * c)
+int libcheck_check(struct checker * c)
{
char buff[MX_ALLOC_LEN];
diff --git a/libmultipath/checkers/rdac.c b/libmultipath/checkers/rdac.c
index 68682c8..a643a4a 100644
--- a/libmultipath/checkers/rdac.c
+++ b/libmultipath/checkers/rdac.c
@@ -262,8 +262,7 @@ const char
}
}
-extern int
-libcheck_check (struct checker * c)
+int libcheck_check(struct checker * c)
{
struct volume_access_inq inq;
int ret, inqfail;
diff --git a/libmultipath/checkers/tur.c b/libmultipath/checkers/tur.c
index 92200aa..4d6c3c2 100644
--- a/libmultipath/checkers/tur.c
+++ b/libmultipath/checkers/tur.c
@@ -314,8 +314,7 @@ static void copy_msg_to_checker(void *c_p, const char *msg)
strlcpy(c->message, msg, sizeof(c->message));
}
-extern int
-libcheck_check (struct checker * c)
+int libcheck_check(struct checker * c)
{
struct tur_checker_context *ct = c->context;
struct timespec tsp;
diff --git a/libmultipath/config.c b/libmultipath/config.c
index 2d629ef..32b41a2 100644
--- a/libmultipath/config.c
+++ b/libmultipath/config.c
@@ -125,8 +125,7 @@ find_hwe (vector hwtable, char * vendor, char * product, char * revision)
return ret;
}
-extern struct mpentry *
-find_mpe (vector mptable, char * wwid)
+struct mpentry *find_mpe(vector mptable, char *wwid)
{
int i;
struct mpentry * mpe;
@@ -141,8 +140,7 @@ find_mpe (vector mptable, char * wwid)
return NULL;
}
-extern char *
-get_mpe_wwid (vector mptable, char * alias)
+char *get_mpe_wwid(vector mptable, char *alias)
{
int i;
struct mpentry * mpe;
diff --git a/libmultipath/configure.c b/libmultipath/configure.c
index d428099..e5cc5f1 100644
--- a/libmultipath/configure.c
+++ b/libmultipath/configure.c
@@ -253,8 +253,7 @@ int rr_optimize_path_order(struct pathgroup *pgp)
return 0;
}
-extern int
-setup_map (struct multipath * mpp, char * params, int params_size)
+int setup_map(struct multipath *mpp, char *params, int params_size)
{
struct pathgroup * pgp;
struct config *conf;
@@ -505,8 +504,7 @@ select_action (struct multipath * mpp, vector curmp, int force_reload)
return;
}
-extern int
-reinstate_paths (struct multipath * mpp)
+int reinstate_paths(struct multipath *mpp)
{
int i, j;
struct pathgroup * pgp;
@@ -582,8 +580,7 @@ fail:
#define DOMAP_EXIST 2
#define DOMAP_DRY 3
-extern int
-domap (struct multipath * mpp, char * params, int is_daemon)
+int domap(struct multipath *mpp, char *params, int is_daemon)
{
int r = DOMAP_FAIL;
struct config *conf;
@@ -748,8 +745,8 @@ out:
return ret;
}
-extern int
-coalesce_paths (struct vectors * vecs, vector newmp, char * refwwid, int force_reload, enum mpath_cmds cmd)
+int coalesce_paths(struct vectors *vecs, vector newmp, char *refwwid,
+ int force_reload, enum mpath_cmds cmd)
{
int r = 1;
int k, i;
@@ -953,9 +950,8 @@ coalesce_paths (struct vectors * vecs, vector newmp, char * refwwid, int force_r
* 1 - failure
* 2 - blacklist
*/
-extern int
-get_refwwid (enum mpath_cmds cmd, char * dev, enum devtypes dev_type,
- vector pathvec, char **wwid)
+int get_refwwid(enum mpath_cmds cmd, char *dev, enum devtypes dev_type,
+ vector pathvec, char **wwid)
{
int ret = 1;
struct path * pp;
@@ -1126,7 +1122,8 @@ out:
return 1;
}
-extern int reload_map(struct vectors *vecs, struct multipath *mpp, int refresh, int is_daemon)
+int reload_map(struct vectors *vecs, struct multipath *mpp, int refresh,
+ int is_daemon)
{
char params[PARAMS_SIZE] = {0};
struct path *pp;
diff --git a/libmultipath/devmapper.c b/libmultipath/devmapper.c
index 5aea5b6..868d2c0 100644
--- a/libmultipath/devmapper.c
+++ b/libmultipath/devmapper.c
@@ -94,8 +94,8 @@ dm_write_log (int level, const char *file, int line, const char *f, ...)
return;
}
-extern void
-dm_init(int v) {
+void dm_init(int v)
+{
dm_log_init(&dm_write_log);
dm_log_init_verbose(v + 3);
}
@@ -200,8 +200,7 @@ dm_drv_prereq (void)
return 1;
}
-extern int
-dm_prereq (void)
+int dm_prereq(void)
{
if (dm_lib_prereq())
return 1;
@@ -249,13 +248,13 @@ out:
return r;
}
-extern int
-dm_simplecmd_flush (int task, const char *name, uint16_t udev_flags) {
+int dm_simplecmd_flush (int task, const char *name, uint16_t udev_flags)
+{
return dm_simplecmd(task, name, 0, 1, udev_flags, 0);
}
-extern int
-dm_simplecmd_noflush (int task, const char *name, uint16_t udev_flags) {
+int dm_simplecmd_noflush (int task, const char *name, uint16_t udev_flags)
+{
return dm_simplecmd(task, name, 1, 1, udev_flags, 0);
}
@@ -338,8 +337,8 @@ addout:
return r;
}
-extern int
-dm_addmap_create (struct multipath *mpp, char * params) {
+int dm_addmap_create (struct multipath *mpp, char * params)
+{
int ro;
for (ro = 0; ro <= 1; ro++) {
@@ -369,8 +368,7 @@ dm_addmap_create (struct multipath *mpp, char * params) {
#define ADDMAP_RW 0
#define ADDMAP_RO 1
-extern int
-dm_addmap_reload (struct multipath *mpp, char *params, int flush)
+int dm_addmap_reload(struct multipath *mpp, char *params, int flush)
{
int r;
uint16_t udev_flags = (flush ? 0 : MPATH_UDEV_RELOAD_FLAG) |
@@ -397,8 +395,7 @@ dm_addmap_reload (struct multipath *mpp, char *params, int flush)
return r;
}
-extern int
-dm_map_present (const char * str)
+int dm_map_present(const char * str)
{
int r = 0;
struct dm_task *dmt;
@@ -425,8 +422,7 @@ out:
return r;
}
-extern int
-dm_get_map(const char * name, unsigned long long * size, char * outparams)
+int dm_get_map(const char *name, unsigned long long *size, char *outparams)
{
int r = 1;
struct dm_task *dmt;
@@ -492,8 +488,7 @@ uuidout:
return r;
}
-extern int
-dm_get_uuid(char *name, char *uuid)
+int dm_get_uuid(char *name, char *uuid)
{
char uuidtmp[WWID_SIZE];
@@ -533,8 +528,7 @@ dm_compare_uuid(const char* mapname1, const char* mapname2)
return 1;
}
-extern int
-dm_get_status(char * name, char * outstatus)
+int dm_get_status(char *name, char *outstatus)
{
int r = 1;
struct dm_task *dmt;
@@ -577,8 +571,7 @@ out:
* 0 : no match
* -1 : empty map
*/
-extern int
-dm_type(const char * name, char * type)
+int dm_type(const char *name, char *type)
{
int r = 0;
struct dm_task *dmt;
@@ -611,8 +604,7 @@ out:
return r;
}
-extern int
-dm_is_mpath(const char * name)
+int dm_is_mpath(const char *name)
{
int r = 0;
struct dm_task *dmt;
@@ -791,8 +783,7 @@ partmap_in_use(const char *name, void *data)
return 0;
}
-extern int
-_dm_flush_map (const char * mapname, int need_sync, int deferred_remove)
+int _dm_flush_map(const char *mapname, int need_sync, int deferred_remove)
{
int r;
@@ -844,8 +835,7 @@ dm_flush_map_nopaths(const char * mapname, int deferred_remove)
#endif
-extern int
-dm_suspend_and_flush_map (const char * mapname)
+int dm_suspend_and_flush_map(const char *mapname)
{
int s = 0, queue_if_no_path = 0;
unsigned long long mapsize;
@@ -884,8 +874,7 @@ dm_suspend_and_flush_map (const char * mapname)
return 1;
}
-extern int
-dm_flush_maps (void)
+int dm_flush_maps(void)
{
int r = 0;
struct dm_task *dmt;
diff --git a/libmultipath/discovery.c b/libmultipath/discovery.c
index 756344f..786a7c0 100644
--- a/libmultipath/discovery.c
+++ b/libmultipath/discovery.c
@@ -177,7 +177,7 @@ path_discovery (vector pathvec, int flag)
}
#define declare_sysfs_get_str(fname) \
-extern ssize_t \
+ssize_t \
sysfs_get_##fname (struct udev_device * udev, char * buff, size_t len) \
{ \
int l; \
@@ -1714,8 +1714,7 @@ get_uid (struct path * pp, int path_state, struct udev_device *udev)
return 0;
}
-extern int
-pathinfo (struct path *pp, struct config *conf, int mask)
+int pathinfo(struct path *pp, struct config *conf, int mask)
{
int path_state;
diff --git a/libmultipath/dmparser.c b/libmultipath/dmparser.c
index 87e8398..b504961 100644
--- a/libmultipath/dmparser.c
+++ b/libmultipath/dmparser.c
@@ -140,9 +140,8 @@ assemble_map (struct multipath * mp, char * params, int len)
return 0;
}
-extern int
-disassemble_map (vector pathvec, char * params, struct multipath * mpp,
- int is_daemon)
+int disassemble_map(vector pathvec, char *params, struct multipath *mpp,
+ int is_daemon)
{
char * word;
char * p;
@@ -425,8 +424,7 @@ out:
return 1;
}
-extern int
-disassemble_status (char * params, struct multipath * mpp)
+int disassemble_status(char *params, struct multipath *mpp)
{
char * word;
char * p;
diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
index 340035e..afe6c13 100644
--- a/libmultipath/hwtable.c
+++ b/libmultipath/hwtable.c
@@ -1106,8 +1106,7 @@ static struct hwentry default_hw[] = {
},
};
-extern int
-setup_default_hwtable (vector hw)
+int setup_default_hwtable(vector hw)
{
int r = 0;
struct hwentry * hwe = default_hw;
diff --git a/libmultipath/pgpolicies.c b/libmultipath/pgpolicies.c
index 3064aab..4ae4afb 100644
--- a/libmultipath/pgpolicies.c
+++ b/libmultipath/pgpolicies.c
@@ -13,8 +13,7 @@
#include "pgpolicies.h"
#include "switchgroup.h"
-extern int
-get_pgpolicy_id (char * str)
+int get_pgpolicy_id(char * str)
{
if (0 == strncmp(str, "failover", 8))
return FAILOVER;
@@ -30,8 +29,7 @@ get_pgpolicy_id (char * str)
return IOPOLICY_UNDEF;
}
-extern int
-get_pgpolicy_name (char * buff, int len, int id)
+int get_pgpolicy_name(char * buff, int len, int id)
{
char * s;
@@ -89,8 +87,8 @@ sort_pathgroups (struct multipath *mp) {
/*
* One path group per unique tgt_node_name present in the path vector
*/
-extern int
-group_by_node_name (struct multipath * mp) {
+int group_by_node_name(struct multipath * mp)
+{
int i, j;
int * bitmap;
struct path * pp;
@@ -165,8 +163,8 @@ out:
/*
* One path group per unique serial number present in the path vector
*/
-extern int
-group_by_serial (struct multipath * mp) {
+int group_by_serial(struct multipath * mp)
+{
int i, j;
int * bitmap;
struct path * pp;
@@ -237,8 +235,7 @@ out:
return 1;
}
-extern int
-one_path_per_group (struct multipath * mp)
+int one_path_per_group(struct multipath *mp)
{
int i;
struct path * pp;
@@ -275,8 +272,7 @@ out:
return 1;
}
-extern int
-one_group (struct multipath * mp) /* aka multibus */
+int one_group(struct multipath *mp) /* aka multibus */
{
struct pathgroup * pgp;
@@ -313,8 +309,7 @@ out:
return 1;
}
-extern int
-group_by_prio (struct multipath * mp)
+int group_by_prio(struct multipath *mp)
{
int i;
unsigned int prio;
diff --git a/libmultipath/print.c b/libmultipath/print.c
index 865562b..5b03383 100644
--- a/libmultipath/print.c
+++ b/libmultipath/print.c
@@ -942,8 +942,7 @@ snprint_pathgroup (char * line, int len, char * format,
return (c - line);
}
-extern void
-print_multipath_topology (struct multipath * mpp, int verbosity)
+void print_multipath_topology(struct multipath *mpp, int verbosity)
{
int resize;
char *buff = NULL;
@@ -973,9 +972,8 @@ print_multipath_topology (struct multipath * mpp, int verbosity)
FREE(buff);
}
-extern int
-snprint_multipath_topology (char * buff, int len, struct multipath * mpp,
- int verbosity)
+int snprint_multipath_topology(char *buff, int len, struct multipath *mpp,
+ int verbosity)
{
int j, i, fwd = 0;
struct path * pp = NULL;
@@ -1247,8 +1245,7 @@ snprint_hwentry (struct config *conf, char * buff, int len, struct hwentry * hwe
return fwd;
}
-extern int
-snprint_hwtable (struct config *conf, char * buff, int len, vector hwtable)
+int snprint_hwtable(struct config *conf, char *buff, int len, vector hwtable)
{
int fwd = 0;
int i;
@@ -1300,8 +1297,7 @@ snprint_mpentry (struct config *conf, char * buff, int len, struct mpentry * mpe
return fwd;
}
-extern int
-snprint_mptable (struct config *conf, char * buff, int len, vector mptable)
+int snprint_mptable(struct config *conf, char *buff, int len, vector mptable)
{
int fwd = 0;
int i;
@@ -1326,8 +1322,8 @@ snprint_mptable (struct config *conf, char * buff, int len, vector mptable)
return fwd;
}
-extern int
-snprint_overrides (struct config *conf, char * buff, int len, struct hwentry *overrides)
+int snprint_overrides(struct config *conf, char * buff, int len,
+ struct hwentry *overrides)
{
int fwd = 0;
int i;
@@ -1356,8 +1352,7 @@ out:
return fwd;
}
-extern int
-snprint_defaults (struct config *conf, char * buff, int len)
+int snprint_defaults(struct config *conf, char *buff, int len)
{
int fwd = 0;
int i;
@@ -1438,8 +1433,7 @@ snprint_blacklist_devgroup (char *buff, int len, int *fwd, vector *vec)
return pos;
}
-extern int
-snprint_blacklist_report (struct config *conf, char * buff, int len)
+int snprint_blacklist_report(struct config *conf, char *buff, int len)
{
int threshold = MAX_LINE_LEN;
int fwd = 0;
@@ -1501,8 +1495,7 @@ snprint_blacklist_report (struct config *conf, char * buff, int len)
return fwd;
}
-extern int
-snprint_blacklist (struct config *conf, char * buff, int len)
+int snprint_blacklist(struct config *conf, char *buff, int len)
{
int i;
struct blentry * ble;
@@ -1578,8 +1571,7 @@ snprint_blacklist (struct config *conf, char * buff, int len)
return fwd;
}
-extern int
-snprint_blacklist_except (struct config *conf, char * buff, int len)
+int snprint_blacklist_except(struct config *conf, char *buff, int len)
{
int i;
struct blentry * ele;
@@ -1655,8 +1647,7 @@ snprint_blacklist_except (struct config *conf, char * buff, int len)
return fwd;
}
-extern int
-snprint_status (char * buff, int len, struct vectors *vecs)
+int snprint_status(char *buff, int len, struct vectors *vecs)
{
int fwd = 0;
int i;
@@ -1687,8 +1678,8 @@ snprint_status (char * buff, int len, struct vectors *vecs)
return fwd;
}
-extern int
-snprint_devices (struct config *conf, char * buff, int len, struct vectors *vecs)
+int snprint_devices(struct config *conf, char * buff, int len,
+ struct vectors *vecs)
{
DIR *blkdir;
struct dirent *blkdev;
@@ -1756,8 +1747,7 @@ snprint_devices (struct config *conf, char * buff, int len, struct vectors *vecs
/*
* stdout printing helpers
*/
-extern void
-print_path (struct path * pp, char * style)
+void print_path(struct path *pp, char *style)
{
char line[MAX_LINE_LEN];
@@ -1766,8 +1756,7 @@ print_path (struct path * pp, char * style)
printf("%s", line);
}
-extern void
-print_multipath (struct multipath * mpp, char * style)
+void print_multipath(struct multipath *mpp, char *style)
{
char line[MAX_LINE_LEN];
@@ -1776,8 +1765,7 @@ print_multipath (struct multipath * mpp, char * style)
printf("%s", line);
}
-extern void
-print_pathgroup (struct pathgroup * pgp, char * style)
+void print_pathgroup(struct pathgroup *pgp, char *style)
{
char line[MAX_LINE_LEN];
@@ -1786,8 +1774,7 @@ print_pathgroup (struct pathgroup * pgp, char * style)
printf("%s", line);
}
-extern void
-print_map (struct multipath * mpp, char * params)
+void print_map(struct multipath *mpp, char *params)
{
if (mpp->size && params)
printf("0 %llu %s %s\n",
@@ -1795,14 +1782,12 @@ print_map (struct multipath * mpp, char * params)
return;
}
-extern void
-print_all_paths (vector pathvec, int banner)
+void print_all_paths(vector pathvec, int banner)
{
print_all_paths_custo(pathvec, banner, PRINT_PATH_LONG);
}
-extern void
-print_all_paths_custo (vector pathvec, int banner, char *fmt)
+void print_all_paths_custo(vector pathvec, int banner, char *fmt)
{
int i;
struct path * pp;
diff --git a/libmultipath/propsel.c b/libmultipath/propsel.c
index ec1fd92..aa7352f 100644
--- a/libmultipath/propsel.c
+++ b/libmultipath/propsel.c
@@ -81,8 +81,7 @@ do_attr_set(var, mp->mpe, shift, "(LUN setting)")
#define set_attr_conf(var, shift) \
do_attr_set(var, conf, shift, "(config file default)")
-extern int
-select_mode (struct config *conf, struct multipath *mp)
+int select_mode(struct config *conf, struct multipath *mp)
{
char *origin;
@@ -95,8 +94,7 @@ out:
return 0;
}
-extern int
-select_uid (struct config *conf, struct multipath *mp)
+int select_uid(struct config *conf, struct multipath *mp)
{
char *origin;
@@ -109,8 +107,7 @@ out:
return 0;
}
-extern int
-select_gid (struct config *conf, struct multipath *mp)
+int select_gid(struct config *conf, struct multipath *mp)
{
char *origin;
@@ -128,8 +125,7 @@ out:
* traverse the configuration layers from most specific to most generic
* stop at first explicit setting found
*/
-extern int
-select_rr_weight (struct config *conf, struct multipath * mp)
+int select_rr_weight(struct config *conf, struct multipath * mp)
{
char *origin, buff[13];
@@ -144,8 +140,7 @@ out:
return 0;
}
-extern int
-select_pgfailback (struct config *conf, struct multipath * mp)
+int select_pgfailback(struct config *conf, struct multipath * mp)
{
char *origin, buff[13];
@@ -160,8 +155,7 @@ out:
return 0;
}
-extern int
-select_pgpolicy (struct config *conf, struct multipath * mp)
+int select_pgpolicy(struct config *conf, struct multipath * mp)
{
char *origin, buff[POLICY_NAME_SIZE];
@@ -182,8 +176,7 @@ out:
return 0;
}
-extern int
-select_selector (struct config *conf, struct multipath * mp)
+int select_selector(struct config *conf, struct multipath * mp)
{
char *origin;
@@ -236,8 +229,7 @@ out:
return (user_friendly_names == USER_FRIENDLY_NAMES_ON);
}
-extern int
-select_alias (struct config *conf, struct multipath * mp)
+int select_alias(struct config *conf, struct multipath * mp)
{
char *origin = NULL;
@@ -276,8 +268,7 @@ out:
return mp->alias ? 0 : 1;
}
-extern int
-select_features (struct config *conf, struct multipath * mp)
+int select_features(struct config *conf, struct multipath *mp)
{
char *origin;
@@ -302,8 +293,7 @@ out:
return 0;
}
-extern int
-select_hwhandler (struct config *conf, struct multipath * mp)
+int select_hwhandler(struct config *conf, struct multipath *mp)
{
char *origin;
@@ -317,8 +307,7 @@ out:
return 0;
}
-extern int
-select_checker(struct config *conf, struct path *pp)
+int select_checker(struct config *conf, struct path *pp)
{
char *origin, *checker_name;
struct checker * c = &pp->checker;
@@ -346,8 +335,7 @@ out:
return 0;
}
-extern int
-select_getuid (struct config *conf, struct path * pp)
+int select_getuid(struct config *conf, struct path *pp)
{
char *origin;
@@ -400,8 +388,7 @@ do { \
} \
} while(0)
-extern int
-select_prio (struct config *conf, struct path * pp)
+int select_prio(struct config *conf, struct path *pp)
{
char *origin;
struct mpentry * mpe;
@@ -438,8 +425,7 @@ out:
return 0;
}
-extern int
-select_no_path_retry(struct config *conf, struct multipath *mp)
+int select_no_path_retry(struct config *conf, struct multipath *mp)
{
char *origin = NULL;
char buff[12];
@@ -497,8 +483,7 @@ out:
return 0;
}
-extern int
-select_minio (struct config *conf, struct multipath * mp)
+int select_minio(struct config *conf, struct multipath *mp)
{
unsigned int minv_dmrq[3] = {1, 1, 0};
@@ -508,8 +493,7 @@ select_minio (struct config *conf, struct multipath * mp)
return select_minio_bio(conf, mp);
}
-extern int
-select_fast_io_fail(struct config *conf, struct multipath *mp)
+int select_fast_io_fail(struct config *conf, struct multipath *mp)
{
char *origin, buff[12];
@@ -523,8 +507,7 @@ out:
return 0;
}
-extern int
-select_dev_loss(struct config *conf, struct multipath *mp)
+int select_dev_loss(struct config *conf, struct multipath *mp)
{
char *origin, buff[12];
@@ -539,8 +522,7 @@ out:
return 0;
}
-extern int
-select_flush_on_last_del(struct config *conf, struct multipath *mp)
+int select_flush_on_last_del(struct config *conf, struct multipath *mp)
{
char *origin;
@@ -557,8 +539,7 @@ out:
return 0;
}
-extern int
-select_reservation_key (struct config *conf, struct multipath * mp)
+int select_reservation_key(struct config *conf, struct multipath *mp)
{
char *origin, buff[12];
@@ -572,8 +553,7 @@ out:
return 0;
}
-extern int
-select_retain_hwhandler (struct config *conf, struct multipath * mp)
+int select_retain_hwhandler(struct config *conf, struct multipath *mp)
{
char *origin;
unsigned int minv_dm_retain[3] = {1, 5, 0};
@@ -594,8 +574,7 @@ out:
return 0;
}
-extern int
-select_detect_prio (struct config *conf, struct path * pp)
+int select_detect_prio(struct config *conf, struct path *pp)
{
char *origin;
@@ -609,8 +588,7 @@ out:
return 0;
}
-extern int
-select_deferred_remove (struct config *conf, struct multipath *mp)
+int select_deferred_remove(struct config *conf, struct multipath *mp)
{
char *origin;
@@ -635,8 +613,7 @@ out:
return 0;
}
-extern int
-select_delay_watch_checks(struct config *conf, struct multipath *mp)
+int select_delay_watch_checks(struct config *conf, struct multipath *mp)
{
char *origin, buff[12];
@@ -651,8 +628,7 @@ out:
return 0;
}
-extern int
-select_delay_wait_checks(struct config *conf, struct multipath *mp)
+int select_delay_wait_checks(struct config *conf, struct multipath *mp)
{
char *origin, buff[12];
@@ -668,8 +644,7 @@ out:
}
-extern int
-select_skip_kpartx (struct config *conf, struct multipath * mp)
+int select_skip_kpartx (struct config *conf, struct multipath * mp)
{
char *origin;
diff --git a/libmultipath/structs.c b/libmultipath/structs.c
index e4bf4c6..ba9edf9 100644
--- a/libmultipath/structs.c
+++ b/libmultipath/structs.c
@@ -201,8 +201,7 @@ alloc_multipath (void)
return mpp;
}
-extern void
-free_multipath_attributes (struct multipath * mpp)
+void free_multipath_attributes(struct multipath *mpp)
{
if (!mpp)
return;
@@ -439,8 +438,7 @@ find_path_by_devt (vector pathvec, char * dev_t)
return NULL;
}
-extern int
-pathcountgr (struct pathgroup * pgp, int state)
+int pathcountgr(struct pathgroup *pgp, int state)
{
struct path *pp;
int count = 0;
@@ -453,8 +451,7 @@ pathcountgr (struct pathgroup * pgp, int state)
return count;
}
-extern int
-pathcount (struct multipath * mpp, int state)
+int pathcount(struct multipath *mpp, int state)
{
struct pathgroup *pgp;
int count = 0;
@@ -467,8 +464,7 @@ pathcount (struct multipath * mpp, int state)
return count;
}
-extern int
-pathcmp (struct pathgroup *pgp, struct pathgroup *cpgp)
+int pathcmp(struct pathgroup *pgp, struct pathgroup *cpgp)
{
int i, j;
struct path *pp, *cpp;
@@ -498,8 +494,7 @@ first_path (struct multipath * mpp)
return pgp?VECTOR_SLOT(pgp->paths, 0):NULL;
}
-extern void
-setup_feature(struct multipath * mpp, char *feature)
+void setup_feature(struct multipath *mpp, char *feature)
{
if (!strncmp(feature, "queue_if_no_path", 16)) {
if (mpp->no_path_retry <= NO_PATH_RETRY_UNDEF)
@@ -507,8 +502,7 @@ setup_feature(struct multipath * mpp, char *feature)
}
}
-extern int
-add_feature (char **f, char *n)
+int add_feature(char **f, char *n)
{
int c = 0, d, l;
char *e, *p, *t;
@@ -592,8 +586,7 @@ add_feature (char **f, char *n)
return 0;
}
-extern int
-remove_feature(char **f, char *o)
+int remove_feature(char **f, char *o)
{
int c = 0, d, l;
char *e, *p, *n;
diff --git a/libmultipath/structs_vec.c b/libmultipath/structs_vec.c
index e898528..22be8e0 100644
--- a/libmultipath/structs_vec.c
+++ b/libmultipath/structs_vec.c
@@ -20,8 +20,7 @@
/*
* creates or updates mpp->paths reading mpp->pg
*/
-extern int
-update_mpp_paths(struct multipath * mpp, vector pathvec)
+int update_mpp_paths(struct multipath *mpp, vector pathvec)
{
struct pathgroup * pgp;
struct path * pp;
@@ -45,8 +44,7 @@ update_mpp_paths(struct multipath * mpp, vector pathvec)
return 0;
}
-extern int
-adopt_paths (vector pathvec, struct multipath * mpp)
+int adopt_paths(vector pathvec, struct multipath *mpp)
{
int i, ret;
struct path * pp;
@@ -81,8 +79,7 @@ adopt_paths (vector pathvec, struct multipath * mpp)
return 0;
}
-extern void
-orphan_path (struct path * pp, const char *reason)
+void orphan_path(struct path *pp, const char *reason)
{
condlog(3, "%s: orphan path, %s", pp->dev, reason);
pp->mpp = NULL;
@@ -96,8 +93,7 @@ orphan_path (struct path * pp, const char *reason)
pp->fd = -1;
}
-extern void
-orphan_paths (vector pathvec, struct multipath * mpp)
+void orphan_paths(vector pathvec, struct multipath *mpp)
{
int i;
struct path * pp;
@@ -151,15 +147,13 @@ _remove_map (struct multipath * mpp, struct vectors * vecs,
free_multipath(mpp, KEEP_PATHS);
}
-extern void
-remove_map (struct multipath * mpp, struct vectors * vecs, int purge_vec)
+void remove_map(struct multipath *mpp, struct vectors *vecs, int purge_vec)
{
_remove_map(mpp, vecs, KEEP_WAITER, purge_vec);
}
-extern void
-remove_map_and_stop_waiter (struct multipath * mpp, struct vectors * vecs,
- int purge_vec)
+void remove_map_and_stop_waiter(struct multipath *mpp, struct vectors *vecs,
+ int purge_vec)
{
_remove_map(mpp, vecs, STOP_WAITER, purge_vec);
}
@@ -182,14 +176,12 @@ _remove_maps (struct vectors * vecs, int stop_waiter)
vecs->mpvec = NULL;
}
-extern void
-remove_maps (struct vectors * vecs)
+void remove_maps(struct vectors *vecs)
{
_remove_maps(vecs, KEEP_WAITER);
}
-extern void
-remove_maps_and_stop_waiters (struct vectors * vecs)
+void remove_maps_and_stop_waiters(struct vectors *vecs)
{
_remove_maps(vecs, STOP_WAITER);
}
@@ -320,8 +312,8 @@ void sync_paths(struct multipath *mpp, vector pathvec)
pp->mpp = mpp;
}
-extern int
-update_multipath_strings (struct multipath *mpp, vector pathvec, int is_daemon)
+int
+update_multipath_strings(struct multipath *mpp, vector pathvec, int is_daemon)
{
if (!mpp)
return 1;
@@ -343,8 +335,7 @@ update_multipath_strings (struct multipath *mpp, vector pathvec, int is_daemon)
return 0;
}
-extern void
-set_no_path_retry(struct config *conf, struct multipath *mpp)
+void set_no_path_retry(struct config *conf, struct multipath *mpp)
{
mpp->retry_tick = 0;
mpp->nr_active = pathcount(mpp, PATH_UP) + pathcount(mpp, PATH_GHOST);
@@ -373,9 +364,8 @@ set_no_path_retry(struct config *conf, struct multipath *mpp)
}
}
-extern int
-__setup_multipath (struct vectors * vecs, struct multipath * mpp,
- int reset, int is_daemon)
+int __setup_multipath(struct vectors *vecs, struct multipath *mpp,
+ int reset, int is_daemon)
{
struct config *conf;
@@ -425,8 +415,7 @@ out:
return 1;
}
-extern struct multipath *
-add_map_without_path (struct vectors * vecs, char * alias)
+struct multipath *add_map_without_path (struct vectors *vecs, char *alias)
{
struct multipath * mpp = alloc_multipath();
@@ -473,9 +462,8 @@ find_existing_alias (struct multipath * mpp,
}
}
-extern struct multipath *
-add_map_with_path (struct vectors * vecs,
- struct path * pp, int add_vec)
+struct multipath *add_map_with_path(struct vectors *vecs, struct path *pp,
+ int add_vec)
{
struct multipath * mpp;
struct config *conf = NULL;
@@ -514,8 +502,7 @@ out:
return NULL;
}
-extern int
-verify_paths(struct multipath * mpp, struct vectors * vecs)
+int verify_paths(struct multipath *mpp, struct vectors *vecs)
{
struct path * pp;
int count = 0;
diff --git a/libmultipath/switchgroup.c b/libmultipath/switchgroup.c
index 031c3d7..9632ce2 100644
--- a/libmultipath/switchgroup.c
+++ b/libmultipath/switchgroup.c
@@ -7,8 +7,7 @@
#include "structs.h"
#include "switchgroup.h"
-extern void
-path_group_prio_update (struct pathgroup * pgp)
+void path_group_prio_update(struct pathgroup *pgp)
{
int i;
int priority = 0;
@@ -32,8 +31,7 @@ path_group_prio_update (struct pathgroup * pgp)
pgp->priority = 0;
}
-extern int
-select_path_group (struct multipath * mpp)
+int select_path_group(struct multipath *mpp)
{
int i;
int max_priority = 0;
diff --git a/libmultipath/uevent.c b/libmultipath/uevent.c
index 19b910f..7edcce1 100644
--- a/libmultipath/uevent.c
+++ b/libmultipath/uevent.c
@@ -559,8 +559,7 @@ out:
return err;
}
-extern int
-uevent_get_major(struct uevent *uev)
+int uevent_get_major(struct uevent *uev)
{
char *p, *q;
int i, major = -1;
@@ -579,8 +578,7 @@ uevent_get_major(struct uevent *uev)
return major;
}
-extern int
-uevent_get_minor(struct uevent *uev)
+int uevent_get_minor(struct uevent *uev)
{
char *p, *q;
int i, minor = -1;
@@ -599,8 +597,7 @@ uevent_get_minor(struct uevent *uev)
return minor;
}
-extern int
-uevent_get_disk_ro(struct uevent *uev)
+int uevent_get_disk_ro(struct uevent *uev)
{
char *p, *q;
int i, ro = -1;
@@ -619,8 +616,7 @@ uevent_get_disk_ro(struct uevent *uev)
return ro;
}
-extern char *
-uevent_get_dm_name(struct uevent *uev)
+char *uevent_get_dm_name(struct uevent *uev)
{
char *p = NULL;
int i;
diff --git a/libmultipath/util.c b/libmultipath/util.c
index 0a136b4..03a5738 100644
--- a/libmultipath/util.c
+++ b/libmultipath/util.c
@@ -145,8 +145,7 @@ size_t strlcat(char *dst, const char *src, size_t size)
return bytes;
}
-extern int
-devt2devname (char *devname, int devname_len, char *devt)
+int devt2devname(char *devname, int devname_len, char *devt)
{
FILE *fd;
unsigned int tmpmaj, tmpmin, major, minor;
--
2.10.1
-- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel