On 01/23/12 17:24, Doug Ledford wrote: > On 01/23/2012 06:18 AM, Jes.Sorensen@xxxxxxxxxx wrote: >> > From: Jes Sorensen <Jes.Sorensen@xxxxxxxxxx> >> > >> > Hi, >> > >> > Here is v2 of the systemd/dracut fixes for preboot handling in >> > mdadm/mdmon. I have changed the argument over from --initrd to >> > --preboot as Neil suggested, as well as fixed up the minor code nits >> > and added documentation to the man pages. >> > >> > It shows up that I didn't need to add a separate enum for the command >> > line arguments in mdmon since mdadm.h gets pulled in so we can reuse >> > those. If we find that to be an issue, we can always create a separate >> > one later. > I'm not sure I like --preboot. I understand the difficulty in finding > just the right option name, but to me, preboot sounds like an option you > would use if you are doing pre-processing of something for a later > invokation to use. And that isn't the case here, so the gut level > understanding won't match up with the man page/help output understanding > and I try to avoid that as much as possible. Since the distinct element > is that we want to keep running both before and after the normal root > filesystem is mounted and we want all files we open to be off of the > root filesystem namespace, maybe --offroot (which is odd enough by > itself to forestall any gut reactions and prompt a trip to the help/man > pages as opposed to guessing what the option means)? > Kay suggested --preroot on irc - would that do the job? I think that is a bit better than --offroot. Cheers, Jes -- 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
