Yeah... that was stupid.  Let's try again with a much smaller,
much improved version. Thanks for pointing out the obvious silliness in
the original patch.<br>
<br>
-b<br>
<br><br><div><span class="gmail_quote">On 10/31/06, <b class="gmail_sendername">B Thomas</b> &lt;<a href="mailto:bjthomas3@gmail.com">bjthomas3@gmail.com</a>&gt; wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Hey, you're probably correct.&nbsp; It was a long, twisty path before I got
to the final patch including many detours and changes of direction.&nbsp;
Let me rework it with the environment variables and resubmit.<br>
<br>
Thanks,<br><span class="sg">
-b</span><div><span class="e" id="q_10ea0c8dbbe7ec72_2"><br>
<br><br><div><span class="gmail_quote">On 10/31/06, <b class="gmail_sendername">Denis Vlasenko</b> &lt;<a href="mailto:vda.linux@googlemail.com" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">vda.linux@googlemail.com
</a>&gt; wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
On Tuesday 31 October 2006 23:22, B Thomas wrote:<br>&gt; Hi,<br>&gt;<br>&gt; There are circumstances in which it would be great to allow the selection of<br>&gt; different inittab file at boot time. This patch adds the ability to select
<br>&gt; the inittab file via a command line option (bb_inittab).<br>&gt;<br>&gt; One of the things that I don't like about this patch is the need to mount<br>&gt; /proc on systems in which it isn't mounted.&nbsp;&nbsp;The mount is cleaned up and
<br>&gt; handled carefully; I just don't like having to do it.&nbsp;&nbsp;Having said that,<br>&gt; this patch has proven to be quite useful.<br>&gt;<br>&gt; Signed-off-by: Ben Thomas (<a href="mailto:ben@virtualiron.com" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">
ben@virtualiron.com
</a>)<br><br>I thought that all unrecognized kernel params are passed in the environment<br>to init: kernel tree, init/main.c:<br><br>/*<br> * Unknown boot options get handed to init, unless they look like<br> * failed parameters
<br> */<br>static int __init unknown_bootoption(char *param, char *val)<br>{<br>....<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;if (val) {<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;/* Environment option */<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;unsigned int i;<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;for (i = 0; envp_init[i]; i++) {
<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;if
(i == MAX_INIT_ENVS) {<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;panic_later = &quot;Too many boot env vars at `%s'&quot;;<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;panic_param = param;<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;}
<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;if
(!strncmp(param, envp_init[i], val - param))<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;break;<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;}<br>....<br><br>So you do not need to parse /proc/commandline<br>--<br>vda<br></blockquote>
</div><br>

</span></div></blockquote></div><br>