<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugzilla.yoctoproject.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugzilla.yoctoproject.org/"
          
          maintainer="it-coreprojects-helpdesk@linuxfoundation.org"
>

    <bug>
          <bug_id>10013</bug_id>
          
          <creation_ts>2016-07-24 17:11:23 +0000</creation_ts>
          <short_desc>Suggested changes to the INITRAMFS_IMAGE_BUNDLE glossary entry</short_desc>
          <delta_ts>2016-08-01 18:24:48 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>9</classification_id>
          <classification>Documentation</classification>
          <product>Reference</product>
          <component>handbook</component>
          <version>2.2</version>
          <rep_platform>x86</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard>01 August 2016: RESOLVED</status_whiteboard>
          <keywords></keywords>
          <priority>Medium</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>2.2 M3</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter>ulfalizer</reporter>
          <assigned_to name="Scott Rifenbark">srifenbark</assigned_to>
          <cc>bluelightning</cc>
    
    <cc>poky.doc.watcher</cc>
    
    <cc>poky.watcher</cc>
    
    <cc>richard.purdie</cc>
    
    <cc>srifenbark</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>Done (doc changes complete)</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>64398</commentid>
    <comment_count>0</comment_count>
    <who name="">ulfalizer</who>
    <bug_when>2016-07-24 17:11:23 +0000</bug_when>
    <thetext>Hello,

The INITRAMFS_IMAGE_BUNDLE glossary entry gets caught up in implementation details and doesn&apos;t make it clear that this variable is simply an on/off switch for creating a kernel image with the initramfs bundled inside it. The description of the implementation details are hard to understand too (I had to stare at it for a long time).

I suggest the following changes:

1. Rewrite

     Using an extra compilation pass ensures that when a kernel attempts
     to use an initramfs, it does not encounter circular dependencies should
     the initramfs include kernel modules.

   to

     Using an extra compilation pass to bundle the initramfs avoids a circular
     dependency between the kernel recipe and the initramfs recipe in case
     the initramfs includes kernel modules. In that case, the initramfs recipe
     depends on the kernel for the kernel modules, and the kernel on the
     initramfs recipe since it bundles in the initramfs inside the kernel image.


2. Turn the rewritten part into a Note.


3. Rewrite

     Setting the variable to &quot;1&quot; in a configuration file causes the
     OpenEmbedded build system to make the extra pass during kernel
     compilation:

   to

     Setting the variable to &quot;1&quot; in a configuration file causes the
     OpenEmbedded build system to generate a kernel image with the
     initramfs specified in INITRAMFS_IMAGE bundled inside it.


4. Change

     INITRAMFS_IMAGE_BUNDLE = &quot;&quot;

   to

     INITRAMFS_IMAGE_BUNDLE ?= &quot;&quot;

   This matches what kernel.bbclass actually does.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>64401</commentid>
    <comment_count>1</comment_count>
    <who name="">ulfalizer</who>
    <bug_when>2016-07-24 19:02:20 +0000</bug_when>
    <thetext>Here&apos;s two more suggested changes to make things a bit more explicit:

5. Changing

     ...is run through an extra pass during kernel compilation...

   to

     ...is run through an extra pass ([do_bundle_initramfs]) during kernel compilation...

6. Adding the following sentence after &quot;...and the initial RAM disk (initramfs).&quot;

    This makes use of the CONFIG_INITRAMFS_SOURCE kernel feature.

Cheers,
Ulf</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>64691</commentid>
    <comment_count>2</comment_count>
    <who name="Scott Rifenbark">srifenbark</who>
    <bug_when>2016-07-29 23:13:09 +0000</bug_when>
    <thetext>Ulf, 

See http://www.yoctoproject.org/docs/2.2/ref-manual/ref-manual.html#var-INITRAMFS_IMAGE_BUNDLE for the changes.  I did some re-writing in the new note.  Please make sure I did not destroy any technical meaning.

Scott</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>64699</commentid>
    <comment_count>3</comment_count>
    <who name="">ulfalizer</who>
    <bug_when>2016-07-30 12:25:31 +0000</bug_when>
    <thetext>Looks fine to me.

Cheers,
Ulf</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>64737</commentid>
    <comment_count>4</comment_count>
    <who name="Scott Rifenbark">srifenbark</who>
    <bug_when>2016-08-01 18:24:48 +0000</bug_when>
    <thetext>Thanks Ulf, 

Updated to RESOLVED and set the doc flag to &quot;Done.&quot;

Scott</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>