Bug#684708: mdadm: support external metadata arrays correctly

2012-10-21 Thread Michael Tokarev
On 21.10.2012 13:46, Michael Tokarev wrote: > So, finally, this whole "stop" case - I'd write it like this: > > stop) >sync # XXX it can be a bad idea to sync here? ># check if there are ANY arrays, and stop any ongoing sync_actions >wait= >for sf in /sys/block/md* ; do

Bug#684708: mdadm: support external metadata arrays correctly

2012-10-21 Thread Michael Tokarev
I'm sorry this took too long. I was very busy last ~3 weeks. Now I looked at it all, and have a few comments. I'm not sure there's a need to respin/resend this patch, if you agree I'll take care of it myself. Comments are inline. On 02.10.2012 17:20, Miquel van Smoorenburg wrote: > Package: mda

Bug#684708: mdadm: support external metadata arrays correctly

2012-10-02 Thread Michael Tokarev
On 02.10.2012 17:20, Miquel van Smoorenburg wrote: > Package: mdadm > Version: 3.2.5-1 > > [version 2 of the patch] > > The initramfs hook supplied by mdadm doesn't install mdmon. Also, mdmon > is not included in the .udeb for the installer. > > This means that if you have an array with external

Bug#684708: mdadm: support external metadata arrays correctly

2012-10-02 Thread Miquel van Smoorenburg
Package: mdadm Version: 3.2.5-1 [version 2 of the patch] The initramfs hook supplied by mdadm doesn't install mdmon. Also, mdmon is not included in the .udeb for the installer. This means that if you have an array with external metadata (ddf or, more widely used, imsm - Intel Matrix Raid) that i

Bug#684708: mdadm: support external metadata arrays correctly

2012-09-23 Thread Michael Tokarev
On 23.09.2012 13:31, Jan Kleinsorge wrote: > Is that patch related to http://permalink.gmane.org/gmane.linux.raid/36907 ? I'm not sure I understand your question. Yes both this bugreport and that patch are about initramfs and running mdadm&Co from there. /mjt -- To UNSUBSCRIBE, email to debia

Bug#684708: mdadm: support external metadata arrays correctly

2012-09-23 Thread Jan Kleinsorge
Is that patch related to http://permalink.gmane.org/gmane.linux.raid/36907 ? -- To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Bug#684708: mdadm: support external metadata arrays correctly

2012-09-05 Thread Miquel van Smoorenburg
On 13.08.2012 14:10, Miquel van Smoorenburg wrote: Package: mdadm Version: 3.2.5-1 Severity: serious Tags: patch wheezy sid The initramfs hook supplied by mdadm doesn't install mdmon. Also, mdmon is not included in the .udeb for the installer. [this time with patch attached, oops.] Here is a

Bug#684708: mdadm: support external metadata arrays correctly

2012-09-05 Thread Miquel van Smoorenburg
On 13.08.2012 14:10, Miquel van Smoorenburg wrote: Package: mdadm Version: 3.2.5-1 Severity: serious Tags: patch wheezy sid The initramfs hook supplied by mdadm doesn't install mdmon. Also, mdmon is not included in the .udeb for the installer. Here is a revised patch. I proposed the patch to m

Bug#684708: mdadm: support external metadata arrays correctly

2012-08-20 Thread Miquel van Smoorenburg
On 20-08-12 11:42 AM, Michael Tokarev wrote: There's one more thing missing in there: mdmon should be re-started from real root after switching from rootfs -- the takeover. I guess it needs to be added to mdadm-raid. Is that really necessary? Can't we just leave it as-is, so the number of cha

Bug#684708: mdadm: support external metadata arrays correctly

2012-08-20 Thread Michael Tokarev
On 20.08.2012 14:05, Dmitrijs Ledkovs wrote: > On 20/08/12 10:42, Michael Tokarev wrote: [] >> There's one more thing missing in there: mdmon should be re-started >> from real root after switching from rootfs -- the takeover. I guess >> it needs to be added to mdadm-raid. > > Why restarted... sur

Bug#684708: mdadm: support external metadata arrays correctly

2012-08-20 Thread Dmitrijs Ledkovs
On 20/08/12 10:42, Michael Tokarev wrote: > On 13.08.2012 14:10, Miquel van Smoorenburg wrote: >> Package: mdadm >> Version: 3.2.5-1 >> Severity: serious >> Tags: patch wheezy sid >> >> The initramfs hook supplied by mdadm doesn't install mdmon. Also, mdmon >> is not included in the .udeb for the i

Bug#684708: mdadm: support external metadata arrays correctly

2012-08-20 Thread Michael Tokarev
On 13.08.2012 14:10, Miquel van Smoorenburg wrote: > Package: mdadm > Version: 3.2.5-1 > Severity: serious > Tags: patch wheezy sid > > The initramfs hook supplied by mdadm doesn't install mdmon. Also, mdmon > is not included in the .udeb for the installer. > > This means that if you have an arra

Bug#684708: mdadm: support external metadata arrays correctly

2012-08-14 Thread Miquel van Smoorenburg
On 08/13/2012 01:43 PM, Michael Tokarev wrote: On 13.08.2012 14:10, Miquel van Smoorenburg wrote: Package: mdadm Version: 3.2.5-1 Severity: serious Tags: patch wheezy sid The initramfs hook supplied by mdadm doesn't install mdmon. Also, mdmon is not included in the .udeb for the installer. Thi

Bug#684708: mdadm: support external metadata arrays correctly

2012-08-13 Thread Michael Tokarev
On 13.08.2012 19:40, Dmitrijs Ledkovs wrote: > On 13/08/12 16:31, Michael Tokarev wrote: [] >> Well. Sort of. mdmon utility is supposed to be used in the initramfs, >> with support of taking over, etc. But this is apparently due to defects >> in some filesystems (I guess it is ext[34]fs), which

Bug#684708: mdadm: support external metadata arrays correctly

2012-08-13 Thread Dmitrijs Ledkovs
On 13/08/12 16:31, Michael Tokarev wrote: > On 13.08.2012 18:20, Dmitrijs Ledkovs wrote: >> On 13/08/12 12:43, Michael Tokarev wrote: >>> On 13.08.2012 14:10, Miquel van Smoorenburg wrote: Package: mdadm Version: 3.2.5-1 Severity: serious Tags: patch wheezy sid The ini

Bug#684708: mdadm: support external metadata arrays correctly

2012-08-13 Thread Michael Tokarev
On 13.08.2012 18:20, Dmitrijs Ledkovs wrote: > On 13/08/12 12:43, Michael Tokarev wrote: >> On 13.08.2012 14:10, Miquel van Smoorenburg wrote: >>> Package: mdadm >>> Version: 3.2.5-1 >>> Severity: serious >>> Tags: patch wheezy sid >>> >>> The initramfs hook supplied by mdadm doesn't install mdmon.

Bug#684708: mdadm: support external metadata arrays correctly

2012-08-13 Thread martin f krafft
also sprach Dmitrijs Ledkovs [2012.08.13.1620 +0200]: > Agree, sounds risky but doing this right is worth the effort. Yes, but it's definitely too late for wheezy. I suggest that we wait until wheezy is out and then push a new mdadm to unstable. -- martin | http://madduck.net/ | http://two.sen

Bug#684708: mdadm: support external metadata arrays correctly

2012-08-13 Thread Dmitrijs Ledkovs
On 13/08/12 12:43, Michael Tokarev wrote: > On 13.08.2012 14:10, Miquel van Smoorenburg wrote: >> Package: mdadm >> Version: 3.2.5-1 >> Severity: serious >> Tags: patch wheezy sid >> >> The initramfs hook supplied by mdadm doesn't install mdmon. Also, mdmon >> is not included in the .udeb for the i

Bug#684708: mdadm: support external metadata arrays correctly

2012-08-13 Thread Michael Tokarev
On 13.08.2012 14:10, Miquel van Smoorenburg wrote: > Package: mdadm > Version: 3.2.5-1 > Severity: serious > Tags: patch wheezy sid > > The initramfs hook supplied by mdadm doesn't install mdmon. Also, mdmon > is not included in the .udeb for the installer. > > This means that if you have an arra

Bug#684708: mdadm: support external metadata arrays correctly

2012-08-13 Thread Miquel van Smoorenburg
Package: mdadm Version: 3.2.5-1 Severity: serious Tags: patch wheezy sid The initramfs hook supplied by mdadm doesn't install mdmon. Also, mdmon is not included in the .udeb for the installer. This means that if you have an array with external metadata (ddf or, more widely used, imsm - Intel Matr