devmapper: explicit config settings
authorHannes Reinecke <hare@suse.de>
Mon, 6 Jun 2016 09:52:42 +0000 (11:52 +0200)
committerHannes Reinecke <hare@suse.de>
Mon, 4 Jul 2016 06:34:12 +0000 (08:34 +0200)
Rather than access 'conf' from within the code this patch moves
those settings to function arguments. So with this patch we don't
need to access 'struct config' anymore.

Signed-off-by: Hannes Reinecke <hare@suse.com>
libmultipath/configure.c
libmultipath/devmapper.c
libmultipath/devmapper.h
multipath/main.c
multipathd/main.c

index e49e7ea..a98b1ca 100644 (file)
@@ -633,11 +633,13 @@ domap (struct multipath * mpp, char * params)
                break;
 
        case ACT_RENAME:
-               r = dm_rename(mpp->alias_old, mpp->alias);
+               r = dm_rename(mpp->alias_old, mpp->alias,
+                             conf->partition_delim);
                break;
 
        case ACT_FORCERENAME:
-               r = dm_rename(mpp->alias_old, mpp->alias);
+               r = dm_rename(mpp->alias_old, mpp->alias,
+                             conf->partition_delim);
                if (r)
                        r = dm_addmap_reload(mpp, params, 0);
                break;
index 926d2f5..b50e9e6 100644 (file)
@@ -19,7 +19,6 @@
 #include "debug.h"
 #include "memory.h"
 #include "devmapper.h"
-#include "config.h"
 #include "sysfs.h"
 
 #include "log_pthread.h"
@@ -32,6 +31,8 @@
 #define UUID_PREFIX "mpath-"
 #define UUID_PREFIX_LEN 6
 
+static int dm_conf_verbosity;
+
 #ifdef LIBDM_API_DEFERRED
 static int dm_cancel_remove_partmaps(const char * mapname);
 #endif
@@ -65,7 +66,7 @@ dm_write_log (int level, const char *file, int line, const char *f, ...)
        if (level > 6)
                level = 6;
 
-       thres = (conf) ? conf->verbosity : 0;
+       thres = dm_conf_verbosity;
        if (thres <= 3 || level > thres)
                return;
 
@@ -94,9 +95,9 @@ dm_write_log (int level, const char *file, int line, const char *f, ...)
 }
 
 extern void
-dm_init(void) {
+dm_init(int v) {
        dm_log_init(&dm_write_log);
-       dm_log_init_verbose(conf ? conf->verbosity + 3 : 0);
+       dm_log_init_verbose(v + 3);
 }
 
 static int
@@ -1375,21 +1376,21 @@ rename_partmap (const char *name, void *data)
        for (offset = strlen(rd->old); name[offset] && !(isdigit(name[offset])); offset++); /* do nothing */
        snprintf(buff, PARAMS_SIZE, "%s%s%s", rd->new, rd->delim,
                 name + offset);
-       dm_rename(name, buff);
+       dm_rename(name, buff, rd->delim);
        condlog(4, "partition map %s renamed", name);
        return 0;
 }
 
 int
-dm_rename_partmaps (const char * old, char * new)
+dm_rename_partmaps (const char * old, char * new, char *delim)
 {
        struct rename_data rd;
 
        rd.old = old;
        rd.new = new;
 
-       if (conf->partition_delim)
-               rd.delim = conf->partition_delim;
+       if (delim)
+               rd.delim = delim;
        if (isdigit(new[strlen(new)-1]))
                rd.delim = "p";
        else
@@ -1398,13 +1399,13 @@ dm_rename_partmaps (const char * old, char * new)
 }
 
 int
-dm_rename (const char * old, char * new)
+dm_rename (const char * old, char * new, char *delim)
 {
        int r = 0;
        struct dm_task *dmt;
        uint32_t cookie;
 
-       if (dm_rename_partmaps(old, new))
+       if (dm_rename_partmaps(old, new, delim))
                return r;
 
        if (!(dmt = dm_task_create(DM_DEVICE_RENAME)))
index b5df369..4bc3b11 100644 (file)
@@ -12,7 +12,7 @@
 #define MPATH_UDEV_RELOAD_FLAG 0
 #endif
 
-void dm_init(void);
+void dm_init(int verbosity);
 int dm_prereq (void);
 int dm_drv_version (unsigned int * version, char * str);
 int dm_simplecmd_flush (int, const char *, uint16_t);
@@ -46,7 +46,7 @@ int dm_remove_partmaps (const char * mapname, int need_sync,
                        int deferred_remove);
 int dm_get_uuid(char *name, char *uuid);
 int dm_get_info (char * mapname, struct dm_info ** dmi);
-int dm_rename (const char * old, char * new);
+int dm_rename (const char * old, char * new, char * delim);
 int dm_reassign(const char * mapname);
 int dm_reassign_table(const char *name, char *old, char *new);
 int dm_setgeometry(struct multipath *mpp);
index aadebec..0dbe281 100644 (file)
@@ -583,6 +583,7 @@ main (int argc, char *argv[])
                exit(1);
        }
 
+       dm_init(conf->verbosity);
        if (dm_prereq())
                exit(1);
        dm_drv_version(conf->version, TGT_MPATH);
@@ -627,7 +628,6 @@ main (int argc, char *argv[])
                condlog(0, "failed to initialize prioritizers");
                goto out;
        }
-       dm_init();
 
        if (conf->cmd == CMD_VALID_PATH &&
            (!conf->dev || conf->dev_type == DEV_DEVMAP)) {
index 3b79aef..8592982 100644 (file)
@@ -2111,6 +2111,7 @@ child (void * param)
 
        uxsock_timeout = conf->uxsock_timeout;
 
+       dm_init(conf->verbosity);
        dm_drv_version(conf->version, TGT_MPATH);
        if (init_checkers()) {
                condlog(0, "failed to initialize checkers");
@@ -2368,7 +2369,6 @@ main (int argc, char *argv[])
        int foreground = 0;
 
        logsink = 1;
-       dm_init();
 
        if (getuid() != 0) {
                fprintf(stderr, "need to be root\n");