Bug 4036 - scripts/contrib/mkefidisk.sh hardcodes 'rw' as root mount option
Summary: scripts/contrib/mkefidisk.sh hardcodes 'rw' as root mount option
Status: RESOLVED FIXED
Alias: None
Product: OE-Core
Classification: Build System, Metadata & Runtime
Component: deployment (show other bugs)
Version: unspecified
Hardware: All Multiple
: Medium+ normal
Target Milestone: 1.4 M6
Assignee: Darren Hart
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2013-03-16 09:17 UTC by Koen Kooi
Modified: 2013-04-05 17:23 UTC (History)
0 users

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Koen Kooi 2013-03-16 09:17:44 UTC
scripts/contrib/mkefidisk.sh hardcodes 'rw' as root mount option, which breaks fsck which expects a 'ro' param to work.
Comment 1 Darren Hart 2013-03-19 18:36:51 UTC
When is fsck breaking? During a boot test?

What would you like to see instead? An option to use 'rw', unilateral use of 'ro'?
Comment 2 Koen Kooi 2013-03-19 18:48:07 UTC
you can't run fsck on a rw fs, I'd like to have an option to choose ro or rw, but just replacing rw with ro is fine by me as well
Comment 3 Darren Hart 2013-03-19 18:54:11 UTC
If this is just running it after boot.... why not just remount the fs?

$ mount -o remount,ro /dev/sda2 /
$ fsck /dev/sda2

This is what I've always done. It seems backwards to change the normal boot method and parameters so a debugging tool can be run without any setup....
Comment 4 Koen Kooi 2013-03-19 19:00:12 UTC
well, you need to boot with 'ro' if you don't use an initramfs and want the builtin fsck to work, which it doesn't do right now. 
I realize that oe-core sysv scripts don't have any way to run fsck, but systemd does, and it has worked great for years now on other platforms (e.g. beagle*). I don't see why mkefidisk.sh needs to be special and force rw mode.
Comment 5 Darren Hart 2013-03-19 19:05:13 UTC
OK, that is why asked if it was part of boot or later. I'm fine adding the option to choose which to use.
Comment 6 Darren Hart 2013-04-05 17:23:43 UTC
commit 24cd3ddeb70a3d3f2985ec71c87ae0108f3d2777
Author: Darren Hart <dvhart@linux.intel.com>
Date:   Tue Mar 19 15:15:03 2013 -0700

    mkefidisk: Boot with ro kernel parameter instead of rw