Remove hard-coding and create device-mapper directory if required

diff --git a/lib/activate/activate.c b/lib/activate/activate.c
index e7937e54fda5ea1bd790f1790defe7639b0736e6..d32cca63a5cd6f1c9a11310ec21137bf21383d82 100644 (file)
--- a/lib/activate/activate.c
+++ b/lib/activate/activate.c
@@ -148,7 +148,7 @@ int _load(struct logical_volume *lv, int task)
         */
        while (le < lv->le_count) {
                if (!_emit_target(dmt, lv, &le)) {
-                       log_err("unable to activate logical volume '%s'",
+                       log_error("Unable to activate logical volume '%s'",
                                lv->name);
                        goto out;
                }

diff --git a/lib/activate/fs.c b/lib/activate/fs.c
index d8abc29551d1f8d1ed1d2a7576e1bec432b87efe..07f6d9cfddf7b63a3469882b233f2a9c66f16a37 100644 (file)
--- a/lib/activate/fs.c
+++ b/lib/activate/fs.c
@@ -15,6 +15,8 @@
 #include "fs.h"
 #include "log.h"
 
+#include <devmapper/libdevmapper.h>
+
 /*
  * FIXME: copied straight from LVM1.
  *
@@ -54,8 +56,8 @@ static int _check_devfs(const char *dev_prefix)
 
 void _build_lv_path(char *buffer, size_t len, struct logical_volume *lv)
 {
-       snprintf(buffer, len, "%s/device-mapper/%s-%s",
-                lv->vg->cmd->dev_dir, lv->vg->name, lv->name);
+       snprintf(buffer, len, "%s%s/%s-%s",
+                lv->vg->cmd->dev_dir, dm_dir(), lv->vg->name, lv->name);
 }
 
 void _build_vg_path(char *buffer, size_t len, struct volume_group *vg)
@@ -72,16 +74,23 @@ void _build_link_path(char *buffer, size_t len, struct logical_volume *lv)
 static int _mk_node(struct logical_volume *lv)
 {
        char lv_path[PATH_MAX];
+       char dm_path[PATH_MAX];
        dev_t dev;
        const char *dev_dir = lv->vg->cmd->dev_dir;
 
        if (_check_devfs(dev_dir))
                return 1;
 
-       _build_lv_path(lv_path, sizeof(lv_path), lv);
+       snprintf(dm_path, PATH_MAX, "%s%s", dev_dir, dm_dir());
+       if (mkdir(dm_path, 0555) && errno != EEXIST) {
+               log_sys_error("mkdir", dm_path);
+               return 0;
+       }
 
+       _build_lv_path(lv_path, sizeof(lv_path), lv);
+       
        if (mknod(lv_path, S_IFBLK | S_IRUSR | S_IWUSR | S_IRGRP, dev) < 0) {
-               log_sys_error("mknod", "creating lv device node");
+               log_sys_error("mknod", lv_path);
                return 0;
        }
 
@@ -99,7 +108,7 @@ static int _rm_node(struct logical_volume *lv)
        _build_lv_path(lv_path, sizeof(lv_path), lv);
 
        if (unlink(lv_path) < 0) {
-               log_sys_error("unlink", "removing lv device node");
+               log_sys_error("unlink", lv_path);
                return 0;
        }
 
@@ -137,7 +146,7 @@ static int _mk_link(struct logical_volume *lv)
        _build_link_path(link_path, sizeof(link_path), lv);
 
        if (symlink(lv_path, link_path) < 0) {
-               log_sys_error("symlink", "creating lv symlink");
+               log_sys_error("symlink", link_path);
                return 0;
        }
 
@@ -151,7 +160,7 @@ static int _rm_link(struct logical_volume *lv)
        _build_link_path(link_path, sizeof(link_path), lv);
 
        if (unlink(link_path) < 0) {
-               log_sys_error("unlink", "unlink lv symlink");
+               log_sys_error("unlink", link_path);
                return 0;
        }