Re: Redundant nfsroot cmdline options

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

 



On 06/16/2009 11:48 PM, Dave Dillow wrote:
On Mon, Jun 15, 2009 at 03:46:24PM -0400, Warren Togami wrote:
I am concerned about the proliferation of too many redundant ways of
configuring netboot.  We should clearly document each supported method
on this Wiki page and figure out which methods should be cut.  The most
obvious methods to cut are NEW inventions of dracut that are redundant
to other methods.

root=nfs       nfsroot=...
root=nfs4      nfsroot=...
root=/dev/nfs4 nfsroot=...

These are all new inventions in dracut, redundant to the following
native methods already supported.

These short cuts are simple, and fall naturally out of the current command
line parsing. I don't expect validating them to be any harder than the
translation I already do into the canoncical nfs[4]:IP:/path:options
format -- you validate the canonical. Documenting them is not very
difficult, either. Maybe there is a potential of user confusion, but you
hide them away in the "Legacy support" or "Non-recommended" appendix.

I simply don't see the problem.

But, I'm tired of having this same coversation every few days and I
don't care enough to continue it now. I'm not going to lose sleep if these
options are gone, so rip them out if that's your fancy.

I have great respect for the work and effort you've put into this, but it just doesn't make sense to add multiple redundant syntaxes of doing exactly the same thing. I accepted it earlier because I was under the impression that all of the above were Legacy syntax from kernel's nfsroot.txt. But now I realize it is invented for dracut.

Furthermore it makes less sense to add redundant syntaxes immediately for them to be classified as Legacy.

Also,

root=/dev/nfs4 nfsroot=...

How is this a shortcut for

root=nfs4:...

?

Warren Togami
wtogami@xxxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe initramfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux SCSI]

  Powered by Linux