On Wed, Jan 31, 2007, Alasdair G Kergon wrote:
> Some distributions chose to configure udev to create /dev/dm-N inodes where
> N is the minor number. (I have always discouraged that as a recipe for
> confusion: I believe that using configurable names is a better approach.)
I seem to recall even RedHat or Fedora attempted to add these devices,
but I suppose you know best. The reasons why these were added in udev
in Debian and Ubuntu are mentionned in http://bugs.debian.org/392623,
but in short gnome-mount required this to work; perhaps this should be
fixed in gnome-mount. The details are in a comment of #401393 though:
<http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=401393;msg=37>
(attaching the patch I mentionned in the preceding mail but forgot,
sorry)
--
Loïc Minier <lool@xxxxxxxx>
--- lilo-22.7.3/debian/patches/00list
+++ lilo-22.7.3/debian/patches/00list
@@ -31,0 +32 @@
+20_devmapper-use-major-minor
--- lilo-22.7.3/debian/changelog
+++ lilo-22.7.3/debian/changelog
@@ -1,3 +1,11 @@
+lilo (1:22.7.3-1.4) unstable; urgency=low
+
+ * Non-maintainer upload.
+ * New dpatch, 20_devmapper-use-major-minor, to address device-mapper devices
+ via major/minor instead of name for DM_DEVICE_TABLE; closes: #401393.
+
+ -- Loic Minier <lool@xxxxxxxx> Wed, 31 Jan 2007 15:21:06 +0100
+
lilo (1:22.7.3-1.3) unstable; urgency=low
* Non-maintainer upload to fix a few more l10n issues.
--- lilo-22.7.3.orig/debian/patches/20_devmapper-use-major-minor.dpatch
+++ lilo-22.7.3/debian/patches/20_devmapper-use-major-minor.dpatch
@@ -0,0 +1,22 @@
+#! /bin/sh /usr/share/dpatch/dpatch-run
+## 20_devmapper-use-major-minor.patch.dpatch by Loic Minier <lool@xxxxxxxx>
+##
+## All lines beginning with `## DP:' are a description of the patch.
+## DP: No description.
+
+@DPATCH@
+diff -urNad lilo-22.7.3~/geometry.c lilo-22.7.3/geometry.c
+--- lilo-22.7.3~/geometry.c 2007-01-31 15:20:37.000000000 +0100
++++ lilo-22.7.3/geometry.c 2007-01-31 15:20:43.000000000 +0100
+@@ -930,8 +930,9 @@
+ slash++;
+ else
+ slash = dmdev;
+- if (!dm_task_set_name(dmt, slash))
+- die("device-mapper: dm_task_set_name(\"%s\") failed",dmdev);
++ if (!dm_task_set_major(dmt, MAJOR(device)) ||
++ !dm_task_set_minor(dmt, MINOR(device)))
++ die("device-mapper: dm_task_set_major() or dm_task_set_minor() failed");
+ if (!dm_task_run(dmt))
+ die("device-mapper: dm_task_run(DM_DEVICE_TABLE) failed");
+
--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel
