Bug 334171

Summary: check of rootfs is already mounted rw is insufficient
Product: Red Hat Enterprise Linux 5 Reporter: Mark Hlawatschek <hlawatschek>
Component: initscriptsAssignee: initscripts Maintenance Team <initscripts-maint-list>
Status: CLOSED ERRATA QA Contact: Brock Organ <borgan>
Severity: medium Docs Contact:
Priority: low    
Version: 5.1CC: grimme
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: RHBA-2008-0443 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-05-21 17:23:55 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
patch for this none

Description Mark Hlawatschek 2007-10-16 12:36:50 UTC
Description of problem:
During the bootprocess, the root filesystem will be remounted rw if it's not 
already. The check that us done here for this is insufficient.

In /etc/rc.sysinit:
    700 state=`LC_ALL=C awk '/ \/ / && ($3 !~ /rootfs/) { print 
$4 }' /proc/mounts`
    701 [ "$state" != "rw" -a "$READONLY" != "yes" ] && \
    702   action $"Remounting root filesystem in read-write mode: " 
mount -n -o remount,rw /

state may be rw,option1,option2 here

Version-Release number of selected component (if applicable):
initscripts-8.45.17.EL-1

How reproducible:
Always

Steps to Reproduce:
1. mount rootfs rw togehter with other options (e.g. noatime)
2. boot system
3.
  
Actual results:
rootfs will be remounted rw  

Expected results:
rootfs should not be remounted rw as it already is mounted rw

Additional info:

Comment 1 RHEL Program Management 2007-10-16 16:35:23 UTC
This request was evaluated by Red Hat Product Management for inclusion in a Red
Hat Enterprise Linux maintenance release.  Product Management has requested
further review of this request by Red Hat Engineering, for potential
inclusion in a Red Hat Enterprise Linux Update release for currently deployed
products.  This request is not yet committed for inclusion in an Update
release.

Comment 3 Bill Nottingham 2008-01-10 20:49:52 UTC
Created attachment 291330 [details]
patch for this

Here's what's going in CVS.

Comment 6 Mark Hlawatschek 2008-02-14 09:30:47 UTC
I just tested the new rpms.
The proposed patch and new rpms seem to fix the problem.
Thank you very much.

Mark

Comment 9 errata-xmlrpc 2008-05-21 17:23:55 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on the solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHBA-2008-0443.html