> -----Original Message----- > From: Jes Sorensen [mailto:Jes.Sorensen@xxxxxxxxxx] > Sent: Wednesday, January 18, 2012 5:47 PM > To: Kwolek, Adam > Cc: neilb@xxxxxxx; linux-raid@xxxxxxxxxxxxxxx; dledford@xxxxxxxxxx; > kay@xxxxxxxxxx; harald@xxxxxxxxxx; lpoetter@xxxxxxxxxx; > mschmidt@xxxxxxxxxx > Subject: Re: [PATCH 2/4] Add --initrd argument to mdadm > > On 01/18/12 17:43, Jes Sorensen wrote: > >> We should tell to mdadm for boot device assembly to be prepared for > >> > fs pivot when reshape is in progress. We can do this by specifying > >> > 2 options in command line, but in real world "someone will forget > >> > about something", so I think '--iinitrd' can have '--freeze-reshape' > >> > functionality also /without freeze-reshape option removal for > >> > separate use when raid will not be a boot device/. As I wrote > >> > before it needs to set one variable during command line parsing in > >> > mdadm.c only. > > If there is already a --freeze-reshape option, then I think it is > > better to have the boot scripts set that explicitly. The initramfs > > stage isn't really something we expect users or admins to mess with > > manually, so it ought to work for that. However if there is no option > > for this, and I just need to set the variable when detecting --initrd, > > then I can add that to the patchset no problem. > > I guess I should add here, that I am not strongly opposed to setting > freeze_reshape for initrd. I am just wary of setting multiple flags magically > without it being obvious to the user/admin. > > Cheers, > Jes Up to you. BR Adam -- To unsubscribe from this list: send the line "unsubscribe linux-raid" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
