IMHO, if the system has to fatal() here, something VERY wrong is going on.<div><br></div><div>This isn&#39;t your general debug option, this is a last-resort emergency rescue. I suppose that if you need to debug this, the system is so problematic you need to deep-dive into it anyway.</div>
<div><br></div><div>I suggest taking it as it is now, and fill in a bugzilla entry about handling it better in initramfs-framework as it should be.</div><div><br></div><div>Alex</div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Mon, Dec 17, 2012 at 7:06 PM, Darren Hart <span dir="ltr">&lt;<a href="mailto:dvhart@linux.intel.com" target="_blank">dvhart@linux.intel.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
<br>
On 12/17/2012 04:55 AM, Alex DAMIAN wrote:<br>
&gt; From: Alexandru DAMIAN &lt;<a href="mailto:alexandru.damian@intel.com">alexandru.damian@intel.com</a>&gt;<br>
&gt;<br>
&gt; Adds &quot;debugshell&quot; command line parameter for live/install images.<br>
&gt;<br>
&gt; If the init live fails to find and mount a root-fs image,<br>
&gt; dumps to a shell after timeout so that the developer can figure<br>
&gt; what&#39;s wrong.<br>
&gt;<br>
&gt; Timeout defaults to 30 seconds, but it can be changed as param<br>
&gt; argument.<br>
&gt;<br>
&gt; Prior art in Ubuntu. Also, leaving a system stale isn&#39;t good form.<br>
&gt;<br>
&gt; Signed-off-by: Alexandru DAMIAN &lt;<a href="mailto:alexandru.damian@intel.com">alexandru.damian@intel.com</a>&gt;<br>
&gt; ---<br>
&gt;  meta/recipes-core/initrdscripts/files/init-live.sh |   18 +++++++++++++++++-<br>
&gt;  1 file changed, 17 insertions(+), 1 deletion(-)<br>
&gt;<br>
&gt; diff --git a/meta/recipes-core/initrdscripts/files/init-live.sh b/meta/recipes-core/initrdscripts/files/init-live.sh<br>
&gt; index c591f0d..d99a8ea 100644<br>
&gt; --- a/meta/recipes-core/initrdscripts/files/init-live.sh<br>
&gt; +++ b/meta/recipes-core/initrdscripts/files/init-live.sh<br>
&gt; @@ -45,7 +45,13 @@ read_args() {<br>
&gt;                      console_params=$arg<br>
&gt;                  else<br>
&gt;                      console_params=&quot;$console_params $arg&quot;<br>
&gt; -                fi<br>
&gt; +                fi ;;<br>
&gt; +            debugshell*)<br>
&gt; +                if [ -z &quot;$optarg&quot; ]; then<br>
&gt; +                        shelltimeout=30<br>
&gt; +                else<br>
&gt; +                        shelltimeout=$optarg<br>
&gt; +                fi<br>
&gt;          esac<br>
&gt;      done<br>
&gt;  }<br>
&gt; @@ -75,6 +81,7 @@ early_setup<br>
&gt;  read_args<br>
&gt;<br>
&gt;  echo &quot;Waiting for removable media...&quot;<br>
&gt; +C=0<br>
&gt;  while true<br>
&gt;  do<br>
&gt;    for i in `ls /media 2&gt;/dev/null`; do<br>
&gt; @@ -90,6 +97,15 @@ do<br>
&gt;    if [ &quot;$found&quot; = &quot;yes&quot; ]; then<br>
&gt;        break;<br>
&gt;    fi<br>
&gt; +  # don&#39;t wait for more than $shelltimeout seconds, if it&#39;s set<br>
&gt; +  if [ -n &quot;$shelltimeout&quot; ]; then<br>
&gt; +      echo -n &quot; &quot; $(( $shelltimeout - $C ))<br>
&gt; +      if [ $C -ge $shelltimeout ]; then<br>
&gt; +           echo &quot;...&quot;<br>
&gt; +           fatal &quot;Cannot find root image on media, dropping to shell &quot;<br>
<br>
</div></div>I&#39;d recommend being explicity about what it is looking for when<br>
debugshell is used. Which devices where probed, what was the file it<br>
looked for. Save the developer from having to find this script in the<br>
repository and look it up.<br>
<br>
--<br>
Darren<br>
<div class="HOEnZb"><div class="h5"><br>
&gt; +      fi<br>
&gt; +      C=$(( C + 1 ))<br>
&gt; +  fi<br>
&gt;    sleep 1<br>
&gt;  done<br>
&gt;<br>
&gt;<br>
<br>
--<br>
</div></div><span class="HOEnZb"><font color="#888888">Darren Hart<br>
Intel Open Source Technology Center<br>
Yocto Project - Technical Lead - Linux Kernel<br>
</font></span></blockquote></div><br></div>