<?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>4880</bug_id>
          
          <creation_ts>2013-07-22 02:16:55 +0000</creation_ts>
          <short_desc>Use a uniform way to determine whether rootfs is read-only</short_desc>
          <delta_ts>2013-08-27 04:32:26 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>7</classification_id>
          <classification>Build System, Metadata &amp; Runtime</classification>
          <product>OE-Core</product>
          <component>core</component>
          <version>1.4</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></status_whiteboard>
          <keywords></keywords>
          <priority>Medium</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>1.5 M5</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Chen Qi">Qi.Chen</reporter>
          <assigned_to name="Chen Qi">Qi.Chen</assigned_to>
          <cc>liezhi.yang</cc>
    
    <cc>mark.hatle</cc>
    
    <cc>meta.mr.watcher</cc>
    
    <cc>meta.watcher</cc>
    
    <cc>sgw</cc>
    
    <cc>yi.zhao</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>Don&apos;t know</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>34732</commentid>
    <comment_count>0</comment_count>
    <who name="Chen Qi">Qi.Chen</who>
    <bug_when>2013-07-22 02:16:55 +0000</bug_when>
    <thetext>In some part of the system, we determine whether the rootfs is read-only by checking the fstab or /proc/mounts, in other part of the system, we use the value of ROOTFS_READ_ONLY in /etc/default/rcS as a criteria. This is confusing and makes system inconsistent.

We should drop the use of ROOTFS_READ_ONLY and use a uniform way to determine whether rootfs is read-only.

On target, we should use /proc/mounts to check whether / is read-only; on host, we should use $ROOT_DIR/etc/fstab to check whether the rootfs is going to be mounted as read-only or not.

//Chen Qi</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>34768</commentid>
    <comment_count>1</comment_count>
    <who name="Saul Wold">sgw</who>
    <bug_when>2013-07-22 20:57:09 +0000</bug_when>
    <thetext>A question is it important to know if the rootfs is READ-ONLY during build or install time?  If so, the rootfs and thus ${ROOT_DIR} will not be available yet.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>34771</commentid>
    <comment_count>2</comment_count>
    <who name="Chen Qi">Qi.Chen</who>
    <bug_when>2013-07-23 01:39:22 +0000</bug_when>
    <thetext>(In reply to comment #1)
&gt; A question is it important to know if the rootfs is READ-ONLY during build
&gt; or install time?

The answer would be &apos;No&apos; with current approach.

//Chen Qi</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>35730</commentid>
    <comment_count>3</comment_count>
    <who name="Chen Qi">Qi.Chen</who>
    <bug_when>2013-08-19 02:07:01 +0000</bug_when>
    <thetext>The first solution spawns too many processes at boot time.
We need a better solution.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>35902</commentid>
    <comment_count>4</comment_count>
    <who name="Chen Qi">Qi.Chen</who>
    <bug_when>2013-08-26 06:32:04 +0000</bug_when>
    <thetext>Considering the overhead brought in by checking the /proc/mounts or /etc/fstab in each boot script, the READ_ONLY_ROOTFS value in /etc/default/rcS is still useful. However, we should at least add a check for conflicting configurations between /etc/fstab and /etc/default/rcS.

//Chen Qi</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>35939</commentid>
    <comment_count>5</comment_count>
    <who name="Chen Qi">Qi.Chen</who>
    <bug_when>2013-08-27 04:32:26 +0000</bug_when>
    <thetext>merged into poky/master: e6e44cf9c10116b9bd2260eb344031d9b897f669

http://git.openembedded.org/?p=openembedded-core.git&amp;a=commit;h=1565a0c5a3f245703e280ca90cf11d3f9374788a</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>