The initrd.scripts from genkernel package do not parse nfs options that are appended with "," after nfs server and path for bootparameter "nfsroot".
If nfsroot is given, the scripts assume only server:path.
But normally, one can append nfs mount options like this:
Steps to Reproduce:
1. setup PXE NFS boot
2. add NFS mount options after server and path in parameter nfsroot
The scripts will try to mount "server:path,options" which will not work, because only "server:path" has to be mounted. The options should be used as nfs mount options.
The scripts should recognize and use the given nfs mount options in nfsroot parameter.
Created attachment 185440 [details, diff]
Patch for /usr/share/genkernel/generic/initrd.scripts
This patch is for "normal" nfs roots. cd boot has not be touched. It will use the nfs mount options in nfsroot, if given. Otherwise, it will use the old default mount options.
(In reply to comment #1)
> This patch is for "normal" nfs roots. cd boot has not be touched. It will use
> the nfs mount options in nfsroot, if given. Otherwise, it will use the old
> default mount options.
Your patch worked just fine here. May be a better one would allow the mount options to supersede only related default option.
For example, I would like to just append ",rw" to mount my nfsroot readwrite instead o ro. The problem is in that case the other default options (nolock,rsize=1024,wsize=1024) are lost.
I'll try to make a patch for this some time
We could just have the custom mount options appended to the end of the default ones. For example, with 'nfsroot=host:export,rw', you'd get 'ro,nolock,rsize=1024,wsize=1024,rw'.
However, I'm not sure if later options override earlier ones like this.
I did this slightly differently than in your patch. I moved the parsing code further up, so it works for mounting the CD via NFS. I also had it append to the default NFSOPTIONS.
I want to vote to *process* this bug and use the patch!
A "normal" kernel feature further missing is the expansion of a '%s' in the path to the hostname (if known, IP otherwise).
Removing keyword InSVN (as there seems to be dispute about it) so this bug doens't get closed by mistake.
PS: Adding keyword "Inclusion" and "[patch] " prefix to better show this bugs nature in searches...
Was fixed long time ago via https://gitweb.gentoo.org/proj/genkernel.git/commit/?id=62f353b65be33b49380900d3cddf150e403cc455