[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#273182: a possible fix/workaround



severity 273182 critical
merge 273182 294404
tags 294404 = patch confirmed pending
tags 301560 = patch confirmed pending
thanks

Okay, Steve's config.c patch in addition to Marco's suggestion
appear to solve this bug. Attached is the final patch for an NMU.

I have tested this only on VMware (but with SCSI, so modules are
being used, but via initrd), and not using a RAID-on-root. Anyway,
RAID-on-root pivot happens way before init, so it would not make
a difference anyway (and RAID-on-root just won't work for now).

I will now post a testers-sought message to debian-devel so that we
can verify that the three bugs are fixed. NMU packages are available
from

  deb http://debian.madduck.net ~madduck/packages/nmu/mdadm/
  deb-src http://debian.madduck.net ~madduck/packages/nmu/mdadm/

-- 
 .''`.     martin f. krafft <madduck@debian.org>
: :'  :    proud Debian developer, admin, user, and author
`. `'`
  `-  Debian - when you have better things to do than fixing a system
 
Invalid/expired PGP subkeys? Use subkeys.pgp.net as keyserver!
 
your eyes are weary from staring at the CRT. you feel sleepy. notice
how restful it is to watch the cursor blink. close your eyes. the
opinions stated above are yours. you cannot imagine why you ever felt
otherwise.
diff -u mdadm-1.9.0/debian/changelog mdadm-1.9.0/debian/changelog
--- mdadm-1.9.0/debian/changelog
+++ mdadm-1.9.0/debian/changelog
@@ -1,3 +1,19 @@
+mdadm (1.9.0-2.2) unstable; urgency=high
+
+  * Non-maintainer upload.
+  * High-urgency upload for sarge targeted RC bugfix.
+  * Move mdadm-raid back to S25 as it needs to run after modules have been
+    loaded at S20 (see followups to #294404, #301560).
+  * Added code suggested by Marco d'Itri (see #294404) to create /dev/md*
+    device nodes before running mdadm in mdadm-raid if not yet present, and if
+    udev is being used.
+    (closes: #294404, #273182)
+  * Verified that Steve Langasek's patch to config.c (see item 4 of the
+    1.9.0-2.1 changelog) is necessary for `mdadm -A -s` to work.
+    (closes: #301560)
+
+ -- martin f. krafft <madduck@debian.org>  Sat, 21 May 2005 17:27:34 +0200
+
 mdadm (1.9.0-2.1) unstable; urgency=high
 
   * Non-maintainer upload.
diff -u mdadm-1.9.0/debian/mdadm.postinst mdadm-1.9.0/debian/mdadm.postinst
--- mdadm-1.9.0/debian/mdadm.postinst
+++ mdadm-1.9.0/debian/mdadm.postinst
@@ -48,9 +48,15 @@
     db_stop
 fi
 
+# remove S04 installed as a fix to #294404 as it does not work
+# we only remove the startup links if the previous version was 1.9.0-2.1 (the
+# one with the erroneous fix) and the S04 link exists to make sure that
+# chances are minimised to overwrite admin changes (even though this would
+# not affect woody upgraders)
 if [ "$1" = "configure" ] && [ -n "$2" ] \
-   && dpkg --compare-versions "$2" le "1.9.0-2";
-then
+   && dpkg --compare-versions "$2" eq "1.9.0-2.1" \
+   && test -L /etc/rcS.d/S04mdadm-raid;
+then 
 	update-rc.d -f mdadm-raid remove
 fi
 
diff -u mdadm-1.9.0/debian/rules mdadm-1.9.0/debian/rules
--- mdadm-1.9.0/debian/rules
+++ mdadm-1.9.0/debian/rules
@@ -53,7 +53,7 @@
 #	dh_installemacsen
 #	dh_installpam
 #	dh_installmime
-	dh_installinit --init-script=mdadm-raid -- start 4 S . start 50 0 6 .
+	dh_installinit --init-script=mdadm-raid -- start 25 S . start 50 0 6 .
 	dh_installinit -- defaults 25
 #	dh_installcron
 	dh_installman
diff -u mdadm-1.9.0/debian/mdadm-raid mdadm-1.9.0/debian/mdadm-raid
--- mdadm-1.9.0/debian/mdadm-raid
+++ mdadm-1.9.0/debian/mdadm-raid
@@ -23,7 +23,12 @@
                 /sbin/modprobe -k md > /dev/null 2>&1  
             fi
             test -f /proc/mdstat || exit 0
-	    echo "Starting raid devices: "
+      if [ -d /dev/.udevdb -a ! -e /dev/md0 -a ! -e /dev/md/0 ]; then
+        echo -n "Creating raid device nodes: "
+        cd /dev && WRITE_ON_UDEV=1 ./MAKEDEV md
+        echo "done."
+      fi
+      echo -n "Starting raid devices: "
 	    if [ -f $CONFIG ] && [ -x $MDADM ] ; then
 	        $MDADM -A -s
 	    elif [ -x $MDRUN ] ; then

Attachment: signature.asc
Description: Digital signature


Reply to: