Re: [PATCH v2 0/4] add --preboot argument to mdadm/mdmon

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

 



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


[Index of Archives]     [Linux RAID Wiki]     [ATA RAID]     [Linux SCSI Target Infrastructure]     [Linux Block]     [Linux IDE]     [Linux SCSI]     [Linux Hams]     [Device Mapper]     [Device Mapper Cryptographics]     [Kernel]     [Linux Admin]     [Linux Net]     [GFS]     [RPM]     [git]     [Yosemite Forum]


  Powered by Linux